Add Helm configuration for dashboard apps#263
Merged
dbutenhof merged 4 commits intocloud-bulldozer:mainfrom Aug 20, 2025
Merged
Add Helm configuration for dashboard apps#263dbutenhof merged 4 commits intocloud-bulldozer:mainfrom
dbutenhof merged 4 commits intocloud-bulldozer:mainfrom
Conversation
dbutenhof
requested review from
MVarshini,
josecastillolema,
jtaleric and
khandrew-redhat
This modifies the existing dashboard-dev Helm Chart to make it and the external dashboard Helm Chart more reusable, and adds a new Helm Chart to define OpenShift-GitOps (ArgoCD) Application resources for both versions of the dashboard. Note that the deployment layout had diverged when the container build moved the application (and secret) from /backend to /opt/backend. There was never a good reason for the move and to merge the two environments I've somewhat arbitrarily reversed that change.
I still don't understand why I need to add the namespace label, when the existing namespaces seem to have worked fine without this. It's more than just the "oc new-project" setup, because I deployment fails with the same namespace permission errors (unable to create resources) when I create `dashboard-test` with `new-project` and attempt to deploy. For now, I'm just accepting this odd experimental reality and moving on...
MVarshini
approved these changes
Aug 13, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Type of change
Description
This modifies the existing dashboard-dev Helm Chart to make it and the external dashboard Helm Chart more reusable, and adds a new Helm Chart to define OpenShift-GitOps (ArgoCD) Application resources for both versions of the dashboard.
Note that the deployment layout had diverged when the container build moved the application (and secret) from /backend to /opt/backend. There was never a good reason for the move and to merge the two environments I've somewhat arbitrarily reversed that change.
Related Tickets & Documents
PANDA-771 (update CD) – INTLAB
Checklist before requesting a review
Testing
Ad hoc testing on the INTLAB cluster using a set of hacked charts to deploy a new temporary "dashboard-test" application.
I had some problems getting OpenShift-GitOps to create resources in the
dashboard-testnamespace until adding the label "argocd.argoproj.io/managed-by=openshift-gitops"; however as the existingdashboardanddashboard-devnamespaces don't have this label, I don't understand why it's needed ... or perhaps, why ArgoCD was able to deploy in those other namespaces without it.Clearly there's a lot going on under the covers (under multiple layers of covers) that I don't understand, so please review carefully and comment liberally based on any knowledge of OpenShift-GitOps/ArgoCD and/or OpenShift/Kubernetes you may have that might be relevant. 😆