Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add version labels to release manifest Deployment/DaemonSet #77

Merged
merged 1 commit into from
Jul 8, 2024

Conversation

roehrich-hpe
Copy link
Collaborator

Change make-kustomization.sh to add component/version labels only to Deployment/Daemonset in the manifest. By doing only these resources, rather than all resources, we minimize the noise in a manifest update to the gitops repo.

If run locally the labels will be:

app.kubernetes.io/component: lustre-csi-driver
app.kubernetes.io/version:

If run from nnf-deploy's "make manifests" the labels will be:

app.kubernetes.io/component: lustre-csi-driver
app.kubernetes.io/version: <lustre-csi-driver's git-version-gen>
app.kubernetes.io/part-of: nnf
app.kubernetes.io/nnf-version: <nnf-deploy's git-version-gen>

Change make-kustomization.sh to add component/version labels only to
Deployment/Daemonset in the manifest. By doing only these resources, rather
than all resources, we minimize the noise in a manifest update to the gitops
repo.

If run locally the labels will be:

  app.kubernetes.io/component: lustre-csi-driver
  app.kubernetes.io/version: <git-version-gen>

If run from nnf-deploy's "make manifests" the labels will be:

  app.kubernetes.io/component: lustre-csi-driver
  app.kubernetes.io/version: <lustre-csi-driver's git-version-gen>
  app.kubernetes.io/part-of: nnf
  app.kubernetes.io/nnf-version: <nnf-deploy's git-version-gen>

Signed-off-by: Dean Roehrich <[email protected]>
@roehrich-hpe roehrich-hpe merged commit 977ac7e into master Jul 8, 2024
3 checks passed
@roehrich-hpe roehrich-hpe deleted the rel-labels branch July 8, 2024 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants