From 9c8124670e361f0b408d410038ef0522059bc6ef Mon Sep 17 00:00:00 2001
From: Ionwyn Sean <ionwynsean@gmail.com>
Date: Thu, 2 Apr 2026 07:48:44 -0700
Subject: [PATCH 1/5] [SMT-63] Fix an issue where header colours are overridden

---
 client/src/js/components/fragments/Header.js |  5 ++--
 client/src/scss/_nav.scss                    | 26 ++++++++++++++++++++
 2 files changed, 29 insertions(+), 2 deletions(-)

diff --git a/client/src/js/components/fragments/Header.js b/client/src/js/components/fragments/Header.js
index 11bb5e39..1b5a80da 100644
--- a/client/src/js/components/fragments/Header.js
+++ b/client/src/js/components/fragments/Header.js
@@ -29,6 +29,7 @@ const Header = ({ currentUser }) => {
   const location = useLocation();
   const [collapsed, setCollapsed] = useState(true);
   const [version, setVersion] = useState(null);
+  const environmentClass = version || 'unknown';
 
   useEffect(() => {
     api.getVersion().then((response) => setVersion(response.data.environment.toLowerCase()));
@@ -48,7 +49,7 @@ const Header = ({ currentUser }) => {
 
   return (
     <header className="mb-3">
-      <Navbar expand="lg" className="navbar-dark">
+      <Navbar expand="lg" className={`navbar-dark header-nav ${environmentClass}`}>
         <Container>
           <NavbarBrand tag={Link} onClick={hideNavbar} to="/">
             <img
@@ -72,7 +73,7 @@ const Header = ({ currentUser }) => {
           <Collapse isOpen={!collapsed} navbar />
         </Container>
       </Navbar>
-      <Navbar expand="lg" className={`navbar-dark main-nav ${version}`}>
+      <Navbar expand="lg" className={`navbar-dark main-nav ${environmentClass}`}>
         <Container>
           <Collapse isOpen={!collapsed} navbar>
             <Nav className="navbar-nav">
diff --git a/client/src/scss/_nav.scss b/client/src/scss/_nav.scss
index 1b650ad4..ec708598 100644
--- a/client/src/scss/_nav.scss
+++ b/client/src/scss/_nav.scss
@@ -1,5 +1,9 @@
 @import './vendor/bootstrap-theme.scss';
 
+@mixin header-nav-colors($background-color) {
+  background-color: $background-color;
+}
+
 @mixin main-nav-colors($background-color) {
   $nav-link-active-bg: lighten($background-color, 10%);
 
@@ -45,6 +49,28 @@
   }
 }
 
+.header-nav {
+  @include header-nav-colors(#053662);
+}
+
+.header-nav.dev,
+.header-nav.unknown {
+  @include header-nav-colors(#448a38);
+}
+
+.header-nav.test,
+.header-nav.tst {
+  @include header-nav-colors(#f9a825);
+}
+
+.header-nav.prod {
+  @include header-nav-colors(#053662);
+}
+
+.header-nav.uat {
+  @include header-nav-colors(#744e91);
+}
+
 .main-nav {
   @include main-nav-colors(#053662);
 }

From b3f9b6aac8cdc00bc2dc49e96f58e8a363c675c9 Mon Sep 17 00:00:00 2001
From: Ionwyn Sean <ionwynsean@gmail.com>
Date: Thu, 2 Apr 2026 07:50:32 -0700
Subject: [PATCH 2/5] Fix deployment issues

- Patched the pipeline so deploy reruns recover from that case instead of failing immediately. In .pipeline/lib/
  deploy.js:10, the deploy now checks whether the required versioned images exist in d3d940-tools; if they are missing,
  it rebuilds them from the checked-out commit before calling importImageStreams.

- Also made the build helper return
  its promise in .pipeline/lib/build.js:5, and updated the entrypoints in .pipeline/build.js:1 and .pipeline/deploy.js:1
  to fail cleanly on async errors.
---
 .pipeline/build.js      |  5 +++-
 .pipeline/deploy.js     |  5 +++-
 .pipeline/lib/build.js  |  2 +-
 .pipeline/lib/deploy.js | 51 ++++++++++++++++++++++++++++++++++-------
 4 files changed, 52 insertions(+), 11 deletions(-)

diff --git a/.pipeline/build.js b/.pipeline/build.js
index 3ac899f8..c20aa94c 100644
--- a/.pipeline/build.js
+++ b/.pipeline/build.js
@@ -2,4 +2,7 @@
 const task = require('./lib/build.js')
 const settings = require('./lib/config.js')
 
-task(Object.assign(settings, { phase: 'build'}))
+Promise.resolve(task(Object.assign(settings, { phase: 'build' }))).catch((error) => {
+  console.error(error);
+  process.exit(1);
+})
diff --git a/.pipeline/deploy.js b/.pipeline/deploy.js
index 59550945..37992788 100644
--- a/.pipeline/deploy.js
+++ b/.pipeline/deploy.js
@@ -2,4 +2,7 @@
 const settings = require('./lib/config.js')
 const task = require('./lib/deploy.js')
 
-task(Object.assign(settings, { phase: settings.options.env}));
+Promise.resolve(task(Object.assign(settings, { phase: settings.options.env }))).catch((error) => {
+  console.error(error);
+  process.exit(1);
+})
diff --git a/.pipeline/lib/build.js b/.pipeline/lib/build.js
index 7141f95b..59a59f22 100644
--- a/.pipeline/lib/build.js
+++ b/.pipeline/lib/build.js
@@ -70,5 +70,5 @@ module.exports = (settings) => {
     phases[phase].changeId,
     phases[phase].instance
   );
-  oc.applyAndBuild(objects);
+  return oc.applyAndBuild(objects);
 };
diff --git a/.pipeline/lib/deploy.js b/.pipeline/lib/deploy.js
index a80cc0b3..f895a479 100644
--- a/.pipeline/lib/deploy.js
+++ b/.pipeline/lib/deploy.js
@@ -2,9 +2,48 @@
 const { OpenShiftClientX } = require("@bcgov/pipeline-cli");
 const path = require("path");
 
+const buildTask = require("./build");
 const util = require("./util");
 
-module.exports = (settings) => {
+const imageNames = ["hmcr-api", "hmcr-client", "hmcr-hangfire"];
+
+function missingBuildImages(oc, buildNamespace, version) {
+  return imageNames.filter((imageName) => {
+    try {
+      oc.raw(
+        "get",
+        ["istag", `${imageName}:${version}`, "-o", "name"],
+        { namespace: buildNamespace }
+      );
+      return false;
+    } catch (error) {
+      return true;
+    }
+  });
+}
+
+async function ensureBuildImages(settings, oc, version) {
+  const buildNamespace = settings.phases.build.namespace;
+  const missingImages = missingBuildImages(oc, buildNamespace, version);
+
+  if (missingImages.length === 0) {
+    return;
+  }
+
+  console.log(
+    `⚠️ Missing build image tags for ${version}: ${missingImages.join(", ")}. Rebuilding before deploy.`
+  );
+  await buildTask(settings);
+
+  const remainingMissingImages = missingBuildImages(oc, buildNamespace, version);
+  if (remainingMissingImages.length > 0) {
+    throw new Error(
+      `Build images are still missing after rebuild for ${version}: ${remainingMissingImages.join(", ")}`
+    );
+  }
+}
+
+module.exports = async (settings) => {
   const phases = settings.phases;
   const options = settings.options;
   const phase = options.env;
@@ -137,6 +176,7 @@ module.exports = (settings) => {
     `${changeId}`,
     phases[phase].instance
   );
+  await ensureBuildImages(settings, oc, version);
   oc.importImageStreams(
     objects,
     phases[phase].tag,
@@ -145,7 +185,6 @@ module.exports = (settings) => {
   );
 
   // Ensure image streams are imported before proceeding
-  const imageNames = ["hmcr-api", "hmcr-client", "hmcr-hangfire"];
   imageNames.forEach((imageName) => {
     try {
       console.log(`🔄 Importing image stream for ${imageName}`);
@@ -164,10 +203,8 @@ module.exports = (settings) => {
 
   let imageExists = false;
   
-  oc.applyAndDeploy(objects, phases[phase].instance)
+  return oc.applyAndDeploy(objects, phases[phase].instance)
     .then(() => {
-      const imageNames = ["hmcr-api", "hmcr-client", "hmcr-hangfire"];
-
       imageExists = imageNames.every((imageName) => {
         try {
           const imageSha = oc.raw("get", [
@@ -209,8 +246,6 @@ module.exports = (settings) => {
         console.log("❌ Skipping final tagging because image does not exist.");
         return;
       }
-      const imageNames = ["hmcr-api", "hmcr-client", "hmcr-hangfire"];
-
       imageNames.forEach((imageName) => {
         const sourceImage = `d3d940-tools/${imageName}:latest`;
         const targetImage = `${phases[phase].namespace}/${imageName}`;
@@ -227,4 +262,4 @@ module.exports = (settings) => {
         );
       });
     });
-};
\ No newline at end of file
+};

From 769f68818161032bd9848fecbde3a894ad213613 Mon Sep 17 00:00:00 2001
From: Ionwyn Sean <ionwynsean@gmail.com>
Date: Thu, 2 Apr 2026 08:40:09 -0700
Subject: [PATCH 3/5] Remove unneeded postgres image

---
 openshift/api-build-config.yaml | 15 ---------------
 1 file changed, 15 deletions(-)

diff --git a/openshift/api-build-config.yaml b/openshift/api-build-config.yaml
index 8e4fb036..a44864b6 100644
--- a/openshift/api-build-config.yaml
+++ b/openshift/api-build-config.yaml
@@ -6,21 +6,6 @@ metadata:
 labels:
   template: api-build
 objects:
-  - apiVersion: "v1"
-    kind: ImageStream
-    metadata:
-      name: postgresql-10
-    spec:
-      lookupPolicy:
-        local: false
-      tags:
-        - annotations: null
-          from:
-            kind: DockerImage
-            name: registry.redhat.io/rhel8/postgresql-10:1
-          name: "1"
-          referencePolicy:
-            type: Local
   - apiVersion: "v1"
     kind: ImageStream
     metadata:

From d1ef25fcb48eeed13ee6a4d6cea483c7d24333d2 Mon Sep 17 00:00:00 2001
From: Ionwyn Sean <ionwynsean@gmail.com>
Date: Thu, 2 Apr 2026 09:56:09 -0700
Subject: [PATCH 4/5] Customise old school pipeline cli for HMCR

---
 .pipeline/lib/build.js            |  4 ++--
 .pipeline/lib/deploy.js           | 21 ++------------------
 .pipeline/lib/openshift-client.js | 32 +++++++++++++++++++++++++++++++
 3 files changed, 36 insertions(+), 21 deletions(-)
 create mode 100644 .pipeline/lib/openshift-client.js

diff --git a/.pipeline/lib/build.js b/.pipeline/lib/build.js
index 59a59f22..c3d20046 100644
--- a/.pipeline/lib/build.js
+++ b/.pipeline/lib/build.js
@@ -1,11 +1,11 @@
 "use strict";
-const { OpenShiftClientX } = require("@bcgov/pipeline-cli");
 const path = require("path");
+const { HmcrOpenShiftClientX } = require("./openshift-client");
 
 module.exports = (settings) => {
   const phases = settings.phases;
   const options = settings.options;
-  const oc = new OpenShiftClientX(
+  const oc = new HmcrOpenShiftClientX(
     Object.assign({ namespace: phases.build.namespace }, options)
   );
   const phase = "build";
diff --git a/.pipeline/lib/deploy.js b/.pipeline/lib/deploy.js
index f895a479..9d71ebde 100644
--- a/.pipeline/lib/deploy.js
+++ b/.pipeline/lib/deploy.js
@@ -1,8 +1,8 @@
 "use strict";
-const { OpenShiftClientX } = require("@bcgov/pipeline-cli");
 const path = require("path");
 
 const buildTask = require("./build");
+const { HmcrOpenShiftClientX } = require("./openshift-client");
 const util = require("./util");
 
 const imageNames = ["hmcr-api", "hmcr-client", "hmcr-hangfire"];
@@ -52,7 +52,7 @@ module.exports = async (settings) => {
   const version = options.version || `v1.0.${githubRunNumber}`;
   console.log(`🚀 Using version: ${version}`);
 
-  const oc = new OpenShiftClientX(
+  const oc = new HmcrOpenShiftClientX(
     Object.assign({ namespace: phases[phase].namespace }, options)
   );
 
@@ -184,23 +184,6 @@ module.exports = async (settings) => {
     version
   );
 
-  // Ensure image streams are imported before proceeding
-  imageNames.forEach((imageName) => {
-    try {
-      console.log(`🔄 Importing image stream for ${imageName}`);
-      oc.raw("import-image", [
-        `${imageName}:${phases.build.tag}`,
-        `--from=d3d940-tools/${imageName}:${phases.build.tag}`,
-        "--confirm",
-        "-n",
-        phases.build.namespace,
-      ]);
-      console.log(`✅ Successfully imported image stream for ${imageName}`);
-    } catch (error) {
-      console.error(`❌ Failed to import image stream for ${imageName}: ${error.message}`);
-    }
-  });
-
   let imageExists = false;
   
   return oc.applyAndDeploy(objects, phases[phase].instance)
diff --git a/.pipeline/lib/openshift-client.js b/.pipeline/lib/openshift-client.js
new file mode 100644
index 00000000..a8dd440d
--- /dev/null
+++ b/.pipeline/lib/openshift-client.js
@@ -0,0 +1,32 @@
+"use strict";
+
+const { OpenShiftClientX } = require("@bcgov/pipeline-cli");
+
+function normalizeImageStream(resource) {
+  if (
+    resource &&
+    resource.kind === "ImageStream" &&
+    resource.status &&
+    Array.isArray(resource.status.tags)
+  ) {
+    resource.status.tags.forEach((tag) => {
+      if (!Array.isArray(tag.items)) {
+        tag.items = [];
+      }
+    });
+  }
+
+  return resource;
+}
+
+class HmcrOpenShiftClientX extends OpenShiftClientX {
+  object(name, args) {
+    return normalizeImageStream(super.object(name, args));
+  }
+
+  objects(names, args) {
+    return super.objects(names, args).map(normalizeImageStream);
+  }
+}
+
+module.exports = { HmcrOpenShiftClientX };

From cd13a8ed825123f2a8cd295fffe9040374896d5f Mon Sep 17 00:00:00 2001
From: Ionwyn Sean <ionwynsean@gmail.com>
Date: Thu, 2 Apr 2026 10:23:39 -0700
Subject: [PATCH 5/5] Preserve existing logdb pvc size on deploy

---
 .pipeline/lib/deploy.js | 17 ++++++++++++++++-
 .pipeline/lib/util.js   | 19 +++++++++++++++++++
 2 files changed, 35 insertions(+), 1 deletion(-)

diff --git a/.pipeline/lib/deploy.js b/.pipeline/lib/deploy.js
index 9d71ebde..4ba9679f 100644
--- a/.pipeline/lib/deploy.js
+++ b/.pipeline/lib/deploy.js
@@ -60,11 +60,23 @@ module.exports = async (settings) => {
     path.resolve(__dirname, "../../openshift")
   );
   var objects = [];
+  const logDbClaimName = `${phases[phase].name}-logdb${phases[phase].suffix}`;
   const logDbSecret = util.getSecret(
     oc,
     phases[phase].namespace,
-    `${phases[phase].name}-logdb${phases[phase].suffix}`
+    logDbClaimName
   );
+  const logDbPersistentVolumeSize = util.getPersistentVolumeClaimSize(
+    oc,
+    phases[phase].namespace,
+    logDbClaimName
+  );
+
+  if (logDbPersistentVolumeSize) {
+    console.log(
+      `Reusing existing logDb PVC size for ${logDbClaimName}: ${logDbPersistentVolumeSize}`
+    );
+  }
 
   objects.push(
     ...oc.processDeploymentTemplate(
@@ -109,6 +121,9 @@ module.exports = async (settings) => {
           SUFFIX: phases[phase].suffix,
           VERSION: version,
           ENV: phases[phase].phase,
+          ...(logDbPersistentVolumeSize
+            ? { PERSISTENT_VOLUME_SIZE: logDbPersistentVolumeSize }
+            : {}),
         },
       }
     )
diff --git a/.pipeline/lib/util.js b/.pipeline/lib/util.js
index 652ea76e..605cc992 100644
--- a/.pipeline/lib/util.js
+++ b/.pipeline/lib/util.js
@@ -23,6 +23,25 @@ function getSecret(oc, namespace, secretId) {
   return secret;
 }
 
+function getPersistentVolumeClaimSize(oc, namespace, claimName) {
+  try {
+    const raw = oc.raw("get", [
+      "-n",
+      namespace,
+      "pvc",
+      claimName,
+      "-o",
+      "json",
+    ]);
+    const claim = JSON.parse(raw.stdout);
+
+    return claim?.spec?.resources?.requests?.storage || null;
+  } catch (error) {
+    return null;
+  }
+}
+
 module.exports = {
   getSecret,
+  getPersistentVolumeClaimSize,
 };