-
Notifications
You must be signed in to change notification settings - Fork 115
Add default kubernetes labels #324
Conversation
Signed-off-by: Anatolii Bazko <abazko@redhat.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do not service and ingress needs default labels set as well?
name: che-devfile-registry | ||
labels: | ||
app: che |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do we need app? Should not it be replaced with app.kubernetes.io/name: che
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We use these labels on chectl side to find a corresponding deployment.
spec: | ||
replicas: 1 | ||
revisionHistoryLimit: 2 | ||
selector: | ||
matchLabels: | ||
app: che-devfile-registry | ||
app: che | ||
component: devfile-registry |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should not it be replaced with app.kubernetes.io/component
from https://kubernetes.io/docs/concepts/overview/working-with-objects/common-labels/ ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We keep those labels in che-operator since selector
field is immutable.
So I decided to keep the same labels here as well.
app: che
component: devfile-registry
Signed-off-by: Anatolii Bazko <abazko@redhat.com>
@ibuziuk |
Signed-off-by: Anatolii Bazko abazko@redhat.com
What does this PR do?
What issues does this PR fix or reference?
eclipse-che/che#18357
PR Checklist
As the author of this Pull Request I made sure that:
What issues does this PR fix or reference
andHow to test this PR
completedReviewers
Reviewers, please comment how you tested the PR when approving it.