-
Notifications
You must be signed in to change notification settings - Fork 6.3k
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
Verify and alert when kubeadm_version is unsupported #6302
Verify and alert when kubeadm_version is unsupported #6302
Conversation
Hi @caruccio. Thanks for your PR. I'm waiting for a kubernetes-sigs member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/ok-to-test |
@caruccio can you update the PR and fix the lint errors |
@caruccio still here ? |
Sorry for the delay, I'm not used to this linter. Took me a while to see the error message (maybe print it red?). |
/lgtm |
Oops sorry you need to rebase master .. :) @caruccio |
@floryut there are any script to do that? |
|
1be9f1c
to
91bec57
Compare
Ok tests are failing |
91bec57
to
489e33d
Compare
@floryut tnx, was missing |
@floryut Pipeline has failed. Is it because I've squashed my commits? |
Nop all good no worries :) |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: caruccio, floryut The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
oops.... closed by mistake |
/lgtm |
* 'master' of https://github.com/kubernetes-sigs/kubespray: remove variable 'etcd_ionice', because ionice removed from container image etcd:v3.4.x (kubernetes-sigs#6735) calico: default to using kdd datastore (kubernetes-sigs#6693) Update docker packages to 19.03.13 + add docker f32 (kubernetes-sigs#6712) Fix snapshot.storage apiVersion (kubernetes-sigs#6711) properly generate extravolumes in kubeadmconfig for centos (kubernetes-sigs#6708) Fix reserved memory unit in kubelet configuration (kubernetes-sigs#6725) Fix unintended SIGPIPE (kubernetes-sigs#6721) Expose offline install overrides in inventory (kubernetes-sigs#6728) Added ability to set calico vxlan vni and port. defaults to calico's … (kubernetes-sigs#6678) Change health check from TCP to HTTPS (kubernetes-sigs#6487) Add multi architeture support to flannel (kubernetes-sigs#6166) Remove pypi repo and pip extra flags (kubernetes-sigs#6729) Fails if kubeadm_version do not matches kubernetes version (kubernetes-sigs#6302) Add external_openstack_lbaas_provider setting for occm (kubernetes-sigs#6566) add new variable allowing additionnal audit webhook server options (kubernetes-sigs#6726) Fix example value for etcd_quota_backend_bytes (kubernetes-sigs#6724) Added support for setting tiller_service_account and tiller_replicas (kubernetes-sigs#6696)
What type of PR is this?
What this PR does / why we need it:
Which issue(s) this PR fixes:
Just a small sugar
Special notes for your reviewer:
Does this PR introduce a user-facing change?: