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

Bug 1995614: Fix beta.kubernetes.io/os deprecated warning #1348

Merged
merged 4 commits into from Aug 30, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
2 changes: 1 addition & 1 deletion assets/thanos-querier/deployment.yaml
Expand Up @@ -187,7 +187,7 @@ spec:
- mountPath: /etc/kube-rbac-proxy
name: secret-thanos-querier-kube-rbac-proxy-rules
nodeSelector:
beta.kubernetes.io/os: linux
kubernetes.io/os: linux
priorityClassName: system-cluster-critical
serviceAccountName: thanos-querier
terminationGracePeriodSeconds: 120
Expand Down
4 changes: 4 additions & 0 deletions jsonnet/components/thanos-querier.libsonnet
Expand Up @@ -286,6 +286,10 @@ function(params)
},
template+: {
spec+: {
// TODO(slashpai): remove once new kube-thanos is released which has this change
nodeSelector: {
'kubernetes.io/os': 'linux',
},
// TODO(dgrisonnet): remove once the upstream anti-affinity addon
// can be extended.
affinity+: {
Expand Down
Expand Up @@ -77,7 +77,7 @@ spec:
- mountPath: /etc/cluster-monitoring-operator/telemetry
name: telemetry-config
nodeSelector:
beta.kubernetes.io/os: linux
kubernetes.io/os: linux
priorityClassName: system-cluster-critical
serviceAccountName: cluster-monitoring-operator
tolerations:
Expand Down
Expand Up @@ -22,7 +22,7 @@ spec:
spec:
serviceAccountName: cluster-monitoring-operator
nodeSelector:
beta.kubernetes.io/os: linux
kubernetes.io/os: linux
node-role.kubernetes.io/master: ""
priorityClassName: "system-cluster-critical"
tolerations:
Expand Down