Skip to content

feat(cluster): Add labels to cluster config#9553

Merged
katiexzhang merged 3 commits into
GoogleContainerTools:mainfrom
idsulik:issue-9549
Nov 18, 2024
Merged

feat(cluster): Add labels to cluster config#9553
katiexzhang merged 3 commits into
GoogleContainerTools:mainfrom
idsulik:issue-9549

Conversation

@idsulik

@idsulik idsulik commented Oct 25, 2024

Copy link
Copy Markdown
Contributor

Fixes: #9549

Description
Add build.cluster:labels config so user can specify labels for pod

User facing changes
Before: no way to specify labels
After: there is a way to specify labels

Signed-off-by: Suleiman Dibirov <idsulik@gmail.com>
Signed-off-by: Suleiman Dibirov <idsulik@gmail.com>
Signed-off-by: Suleiman Dibirov <idsulik@gmail.com>
@idsulik

idsulik commented Oct 30, 2024

Copy link
Copy Markdown
Contributor Author

Hi @ericzzzzzzz
What do you think about the PR?

@ericzzzzzzz

Copy link
Copy Markdown
Contributor

Hi @idsulik Thanks for the pr, due to org structure change, I'm no longer working on skaffold, please use skaffold-team alias to tag maintainers for reviews : )

@idsulik

idsulik commented Oct 30, 2024

Copy link
Copy Markdown
Contributor Author

hi @ericzzzzzzz, okay, thank you!)

@idsulik

idsulik commented Oct 31, 2024

Copy link
Copy Markdown
Contributor Author

@plumpy hi! Could you please check the PR?)

@idsulik

idsulik commented Nov 3, 2024

Copy link
Copy Markdown
Contributor Author

@ericzzzzzzz hi! Could you please write me on linkedin, I want to ask something. I'm writing here, because I couldn't find anything to find you, I'll delete this message

@ericzzzzzzz

Copy link
Copy Markdown
Contributor

Hi @idsulik , I don't use linkedin, please send me an email for your questions, ericwork@google.com

Comment thread pkg/skaffold/build/cluster/pod_test.go

@katiexzhang katiexzhang left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@katiexzhang katiexzhang merged commit e55f954 into GoogleContainerTools:main Nov 18, 2024
plumpy added a commit to plumpy/skaffold that referenced this pull request Nov 20, 2024
@menahyouyeah menahyouyeah mentioned this pull request Mar 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3 participants