Update Calico to v3.13.2, Multus to v3.4.1. #5894

alexkross · 2020-04-05T21:08:31Z

What type of PR is this?

/kind api-change
/kind bug
/kind cleanup
/kind design
/kind documentation
/kind failing-test

/kind feature

/kind flake

What this PR does / why we need it:
Update.

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?:
NONE

…cess to kubeadm config.

k8s-ci-robot · 2020-04-05T21:08:39Z

Hi @alexkross. 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 /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

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.

k8s-ci-robot · 2020-04-05T21:08:40Z

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: alexkross
To complete the pull request process, please assign chadswen
You can assign the PR to them by writing /assign @chadswen in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

floryut · 2020-04-06T09:24:11Z

@alexkross Didn't you forget to add calicoctl hashes for arm64 and arm (even if set to 0) ?

…igs#5890)

…spray into upd-calico-multus

alexkross · 2020-04-06T10:46:35Z

@alexkross Didn't you forget to add calicoctl hashes for arm64 and arm (even if set to 0) ?

Yes. I do. Updated. Thanks.

…ck (kubernetes-sigs#5847) * requirements.txt: Bump versions Ansible 2.8+ allow ansible_python_interpreter autodetection Signed-off-by: Etienne Champetier <champetier.etienne@gmail.com> * tests: do not force ansible_python_interpreter we do not expect people to set ansible_python_interpreter, so we should not set it in the CI Signed-off-by: Etienne Champetier <champetier.etienne@gmail.com>

* requirements.txt: Bump versions Ansible 2.8+ allow ansible_python_interpreter autodetection Signed-off-by: Etienne Champetier <champetier.etienne@gmail.com> * tests: do not force ansible_python_interpreter we do not expect people to set ansible_python_interpreter, so we should not set it in the CI Signed-off-by: Etienne Champetier <champetier.etienne@gmail.com> * Add CentOS 8 Calico to CI Signed-off-by: Etienne Champetier <champetier.etienne@gmail.com>

…igs#5898)

…kubernetes-sigs#5896) * Fix chicken and egg problem with proxy_env not defined on the first envinronment usage. * Disable fact gathering for the first proxy_env evaluation. * Move proxy_env var set up from the role defaults to the root playbooks as fact.

Miouge1 · 2020-04-08T08:25:47Z

CI is showing CrashLoopBackOff in packet_centos7-calico-ha-once-localhost kubectl logs shows:

2020-04-08 08:24:37.677 [ERROR][8] startup.go 182: failed to query kubeadm's config map error=configmaps "kubeadm-config" is forbidden: User "system:serviceaccount:kube-system:calico-node" cannot get resource "configmaps" in API group "" in the namespace "kube-system"

This PR might require #5895 ?

* etcd: etcd-events doesn't depend on etcd_cluster_setup Signed-off-by: Etienne Champetier <champetier.etienne@gmail.com> * etcd: remove condition already present on include_tasks Signed-off-by: Etienne Champetier <champetier.etienne@gmail.com> * etcd: fix scaling up Signed-off-by: Etienne Champetier <champetier.etienne@gmail.com> * etcd: use *access_addresses, do not delegate to etcd[0] We want to wait for the full cluster to be healthy, so use all the cluster addresses Also we should be able to run the playbook when etcd[0] is down (not tested), so do not delegate to etcd[0] Signed-off-by: Etienne Champetier <champetier.etienne@gmail.com> * etcd: use failed_when for health check unhealthy cluster is expected on first run, so use failed_when instead of ignore_errors to remove scary red messages Also use run_once Signed-off-by: Etienne Champetier <champetier.etienne@gmail.com> * kubernetes/preinstall: ensure ansible_fqdn is up to date after changing /etc/hosts Signed-off-by: Etienne Champetier <champetier.etienne@gmail.com> * kubernetes/master: regenerate apiserver cert if needed Signed-off-by: Etienne Champetier <champetier.etienne@gmail.com>

alexkross · 2020-04-08T11:59:16Z

CI is showing CrashLoopBackOff in packet_centos7-calico-ha-once-localhost kubectl logs shows:
2020-04-08 08:24:37.677 [ERROR][8] startup.go 182: failed to query kubeadm's config map error=configmaps "kubeadm-config" is forbidden: User "system:serviceaccount:kube-system:calico-node" cannot get resource "configmaps" in API group "" in the namespace "kube-system"
This PR might require #5895 ?

I think you are right.
It would be better to rebase onto #5895 .

Miouge1 · 2020-04-08T13:46:48Z

@alexkross can you do a single PR for the changes in #5894 and #5895 ?

Maybe keep this one and close the other one.

…able. (kubernetes-sigs#5872)

…tes-sigs#5853)

…ltus

…spray into upd-calico-multus

alexkross · 2020-04-08T18:20:17Z

@Miouge1 before merge calico-cr-missing-cm-perm ( #5895 ) i did rebase to the latest master and now see that history looks bloated.
Maybe I still need to follow your original variant suggestion to make another clean single PR for the changes in #5894 and #5895 based on current master branch?

floryut · 2020-04-08T18:21:14Z

@Miouge1 before merge calico-cr-missing-cm-perm ( #5895 ) i did rebase master and now see that history looks bloated.
Maybe I still need to follow your original variant suggestion to make another clean single PR for the changes in #5894 and #5895 based on current master branch?

If you can and have some time to do so, that'd be great, cleaner and easier :)

alexkross · 2020-04-08T18:22:24Z

/close

In favor of #5912

k8s-ci-robot · 2020-04-08T18:22:35Z

@alexkross: Closed this PR.

In response to this:

/close

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.

alexkross added 2 commits April 5, 2020 19:55

Add ConfigMap get permission to allow calico-node (startup.go:182) ac…

a58b28a

…cess to kubeadm config.

Update Calico to v3.13.2, Multus to v3.4.1.

6dd53d2

k8s-ci-robot added kind/feature Categorizes issue or PR as related to a new feature. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. labels Apr 5, 2020

k8s-ci-robot requested review from bozzo and holmsten April 5, 2020 21:08

k8s-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Apr 5, 2020

qvicksilver and others added 3 commits April 6, 2020 02:41

Terraform/OpenStack: Enable usage of an existing router (kubernetes-s…

0d29905

…igs#5890)

Update Calico to v3.13.2, Multus to v3.4.1.

540c814

Merge branch 'upd-calico-multus' of https://github.com/alexkross/kube…

42a2938

…spray into upd-calico-multus

champtar and others added 8 commits April 7, 2020 01:47

Replace latest tags for csi drivers (kubernetes-sigs#5899)

ae29296

Change docker.io repo to variable and upgrade alb image (kubernetes-s…

9b1980c

…igs#5898)

remove unused kubelet options (kubernetes-sigs#5903)

0c51352

add local-path-provosioner helper image def (kubernetes-sigs#5817)

45a177e

make explicit that doc is at kubespray.io (kubernetes-sigs#5878)

2c21e7b

champtar and others added 3 commits April 8, 2020 01:27

fix readonly flexvolume in fcos and coreos (kubernetes-sigs#5885)

157c247

Add crictl 1.18.0 hashes for k8s 1.18 (kubernetes-sigs#5877)

f055ba7

alexkross mentioned this pull request Apr 8, 2020

Add ConfigMap get permission to allow calico-node … #5895

Closed

floryut mentioned this pull request Apr 8, 2020

Remove 1.16.x hook on test tf-ovh_coreos-calico #5853

Merged

Miouge1 and others added 6 commits April 8, 2020 07:37

Fix conntrack for opensuse and docker support (kubernetes-sigs#5880)

7eaa7c9

Update docker RHEL/CentOS versions to the latest patch versions avail…

c33a049

…able. (kubernetes-sigs#5872)

Remove 1.16.x flag for tf-ovh_coreos-calico (now 1.17 ready) (kuberne…

01b9b26

…tes-sigs#5853)

Update Calico to v3.13.2, Multus to v3.4.1.

ce2a673

Merge branch 'calico-cr-missing-cm-perm' (PR#5895) into upd-calico-mu…

cf09f19

…ltus

Merge branch 'upd-calico-multus' of https://github.com/alexkross/kube…

6ebadb5

…spray into upd-calico-multus

k8s-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Apr 8, 2020

k8s-ci-robot closed this Apr 8, 2020

alexkross deleted the upd-calico-multus branch April 9, 2020 10:30

Miouge1 mentioned this pull request May 1, 2020

Contiv in Kubespray needs a maintainer #6059

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Update Calico to v3.13.2, Multus to v3.4.1. #5894

Update Calico to v3.13.2, Multus to v3.4.1. #5894

alexkross commented Apr 5, 2020

k8s-ci-robot commented Apr 5, 2020

k8s-ci-robot commented Apr 5, 2020

floryut commented Apr 6, 2020

alexkross commented Apr 6, 2020

Miouge1 commented Apr 8, 2020

alexkross commented Apr 8, 2020

Miouge1 commented Apr 8, 2020

alexkross commented Apr 8, 2020 •

edited

Loading

floryut commented Apr 8, 2020

alexkross commented Apr 8, 2020 •

edited

Loading

k8s-ci-robot commented Apr 8, 2020

Update Calico to v3.13.2, Multus to v3.4.1. #5894

Update Calico to v3.13.2, Multus to v3.4.1. #5894

Conversation

alexkross commented Apr 5, 2020

k8s-ci-robot commented Apr 5, 2020

k8s-ci-robot commented Apr 5, 2020

floryut commented Apr 6, 2020

alexkross commented Apr 6, 2020

Miouge1 commented Apr 8, 2020

alexkross commented Apr 8, 2020

Miouge1 commented Apr 8, 2020

alexkross commented Apr 8, 2020 • edited Loading

floryut commented Apr 8, 2020

alexkross commented Apr 8, 2020 • edited Loading

k8s-ci-robot commented Apr 8, 2020

alexkross commented Apr 8, 2020 •

edited

Loading

alexkross commented Apr 8, 2020 •

edited

Loading