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

[velero] Add version label to deployment #624

Merged
merged 4 commits into from
Oct 30, 2024

Conversation

wyattfry
Copy link
Contributor

@wyattfry wyattfry commented Oct 24, 2024

Special notes for your reviewer:

This PR adds the Kubernetes-recommended label "app.kubernetes.io/version" to the Deployment and its Pods.
https://kubernetes.io/docs/concepts/overview/working-with-objects/common-labels/#labels

The value of the version label:

The current version of the application (e.g., a SemVer 1.0, revision hash, etc.)

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • DCO signed
  • Chart Version bumped, please refer to the chart version instruction
  • Variables are documented in the values.yaml or README.md
  • Title of the PR starts with chart name (e.g. [velero])

@wyattfry wyattfry changed the title Add version label to deployment [velero] Add version label to deployment Oct 24, 2024
maxsxu and others added 3 commits October 24, 2024 15:47
…nzu#616)

Signed-off-by: Max Xu <xuhuan@live.cn>
Signed-off-by: Wyatt Fry <wfry@bridgephase.com>
Signed-off-by: Wyatt Fry <wfry@bridgephase.com>
Signed-off-by: Wyatt Fry <wfry@bridgephase.com>
@wyattfry wyattfry force-pushed the add-version-label-to-deployment branch from 9b4853f to 5c7dba2 Compare October 24, 2024 19:47
Copy link
Collaborator

@jenting jenting left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@wyattfry
Copy link
Contributor Author

@aramprice @weseven could i please have one more review?

@aramprice
Copy link
Member

@wyattfry - you may have tagged the wrong person, I don't have context on this codebase.

@ywk253100 ywk253100 merged commit bbb36cc into vmware-tanzu:main Oct 30, 2024
15 checks passed
@wyattfry
Copy link
Contributor Author

@wyattfry - you may have tagged the wrong person, I don't have context on this codebase.

Ah, sorry about that

@wyattfry wyattfry deleted the add-version-label-to-deployment branch October 30, 2024 13:00
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.

5 participants