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

jsonnet/manifests/example: Update/generate nodeSelector to kubernetes.io/os instead of beta.kubernetes.io/os #241

Merged
merged 2 commits into from
Aug 24, 2021

Conversation

slashpai
Copy link
Contributor

@slashpai slashpai commented Aug 24, 2021

  • I added CHANGELOG entry for this change.
  • Change is not relevant to the end user.

Changes

Update nodeSelector to kubernetes.io/os since beta.kubernetes.io/os deprecated since v1.14 and shows warning in logs

Verification

NA

@slashpai slashpai force-pushed the update_nodeselector branch 3 times, most recently from 13c3c05 to 7182f40 Compare August 24, 2021 07:35
Signed-off-by: Jayapriya Pai <janantha@redhat.com>
Signed-off-by: Jayapriya Pai <janantha@redhat.com>
@slashpai slashpai force-pushed the update_nodeselector branch from 7182f40 to 93e56f1 Compare August 24, 2021 07:37
@slashpai
Copy link
Contributor Author

@s-urbaniak @bwplotka Could you please review

@simonpasquier
Copy link

lgtm

Copy link
Contributor

@yeya24 yeya24 left a comment

Choose a reason for hiding this comment

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

LGTM

@squat squat merged commit 812e6d6 into thanos-io:main Aug 24, 2021
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.

4 participants