Fix loading logs when ui and workflow are in different namespace

Author: Alexander Matyushentsev

VERSION

@@ -0,0 +1 @@
+v2.2.1

scripts/build_docker.sh

@@ -1,8 +1,12 @@
 #!/bin/bash
 
+CURRENT_DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" >/dev/null && pwd )"
+GIT_COMMIT=`git rev-parse --short HEAD`
+VERSION=`cat ${CURRENT_DIR}/../VERSION`
+
 set -e
 
-TAG=${IMAGE_TAG:-'latest'}
+TAG=${IMAGE_TAG:-"$VERSION-$GIT_COMMIT"}
 
 docker build --build-arg ARGO_VERSION=${TAG} -t ${IMAGE_NAMESPACE:-`whoami`}/argoui:${TAG} .
 

src/api/app.ts

@@ -101,18 +101,20 @@ export function create(
         async (req, res) => serve(res, () => (forceNamespaceIsolation ? crd.ns(namespace) : crd.ns(req.params.namespace)).workflows.get(req.params.name)));
 
     app.get('/api/workflows/live', async (req, res) => {
+        const ns = getNamespace(req);
+
         let updatesSource = new Observable((observer: Observer<any>) => {
             const labelSelector = getWorkflowLabelSelector(req);
-            let stream = crd.ns(req.params.namespace).workflows.getStream({ qs: { watch: true, labelSelector: labelSelector.join(',') } });
+            let stream = (ns ? crd.ns(ns) : crd).workflows.getStream({ qs: { watch: true, labelSelector: labelSelector.join(',') } });
             stream.on('end', () => observer.complete());
             stream.on('error', (e) => observer.error(e));
             stream.on('close', () => observer.complete());
             stream = stream.pipe(new JSONStream());
             stream.on('data', (data) => data && observer.next(data));
         });
-        if (forceNamespaceIsolation || req.query.namespace) {
+        if (ns) {
             updatesSource = updatesSource.filter((change) => {
-                return change.object.metadata.namespace === forceNamespaceIsolation ? namespace : req.query.namespace;
+                return change.object.metadata.namespace === ns;
             });
         }
         if (req.query.name) {