| *** iokiwi has joined #openstack-containers | 00:02 | |
| openstackgerrit | Feilong Wang proposed openstack/magnum master: [k8s] Support updating k8s cluster health status https://review.opendev.org/710384 | 01:09 |
|---|---|---|
| flwang1 | brtknr: ^ | 01:09 |
| *** sapd1 has quit IRC | 01:22 | |
| brtknr | flwang1: thanks | 01:23 |
| brtknr | left a comment | 01:23 |
| flwang1 | brtknr: i tested locally and it passed the test | 01:24 |
| flwang1 | what do you mean a KeyError? | 01:24 |
| flwang1 | brtknr: ^ | 01:28 |
| *** sapd1_x has joined #openstack-containers | 02:39 | |
| *** ykarel|away is now known as ykarel | 04:52 | |
| *** udesale has joined #openstack-containers | 05:06 | |
| *** ykarel is now known as ykarel|afk | 05:39 | |
| *** ykarel|afk is now known as ykarel | 06:33 | |
| *** vishalmanchanda has joined #openstack-containers | 07:16 | |
| *** ykarel is now known as ykarel|lunch | 08:24 | |
| flwang1 | brtknr: around? | 08:53 |
| flwang1 | strigazi: ? | 08:53 |
| brtknr | flwang1: hi | 08:53 |
| brtknr | sorry i was mostly sleeping in the last 12 hours :) | 08:53 |
| flwang1 | no problem ;) | 08:53 |
| brtknr | flwang1: ignore my comment about keyerror | 08:54 |
| flwang1 | i ran the conformance test and finally there 2 cases failed | 08:54 |
| brtknr | TIL python dictionary get function returns None | 08:54 |
| brtknr | flwang1: did you run --certified-conformance? | 08:54 |
| flwang1 | yes | 08:54 |
| flwang1 | [Fail] [sig-storage] EmptyDir wrapper volumes [It] should not cause race condition when used for configmaps [Serial] [Conformance] | 08:54 |
| flwang1 | /workspace/anago-v1.17.3-beta.0.40+c94b9acd4b784f/src/k8s.io/kubernetes/_output/dockerized/go/src/k8s.io/kubernetes/test/e2e/storage/empty_dir_wrapper.go:418 | 08:54 |
| flwang1 | [Fail] [sig-network] Services [It] should be able to change the type from ExternalName to NodePort [Conformance] | 08:54 |
| flwang1 | /workspace/anago-v1.17.3-beta.0.40+c94b9acd4b784f/src/k8s.io/kubernetes/_output/dockerized/go/src/k8s.io/kubernetes/test/e2e/network/service.go:1117 | 08:54 |
| brtknr | hmm how did you get it to run in the first place? | 08:55 |
| flwang1 | i just created a new cluster based on v1.17.4 and rerun, will let you guys know the result | 08:55 |
| brtknr | mine refused to start due to the taints | 08:55 |
| brtknr | mine refused to start due to the taints on the master | 08:55 |
| brtknr | did you remove the taints before the conformance test? | 08:55 |
| flwang1 | sonobuoy run --plugin-env=e2e.E2E_EXTRA_ARGS="--allowed-not-ready-nodes=1" | 08:55 |
| flwang1 | 08:55 | |
| flwang1 | with above command, it can start to run | 08:55 |
| flwang1 | but i agree, the taint error is annoying | 08:56 |
| flwang1 | remove the taint also works, but it doesn't make sense | 08:56 |
| brtknr | flwang1: oh | 08:56 |
| flwang1 | this is a new issue only existing in v1.17.x i think | 08:56 |
| flwang1 | i didn't see it in v1.16.x | 08:57 |
| brtknr | ah okay | 08:57 |
| brtknr | btw yesterday i spawned a 1.18.0 cluster and it "just worked" | 08:57 |
| flwang1 | just worked means? | 08:58 |
| brtknr | flwang1: do you understand why those failures are happening? | 08:58 |
| flwang1 | conformance passed? | 08:58 |
| brtknr | there is no conformance test available yet for 1.18.0 | 08:58 |
| brtknr | there is no conformance test available yet for 1.18.0 yet | 08:59 |
| flwang1 | i don't know, the emptydir one shouldn't related to network | 08:59 |
| flwang1 | but i'm not sure the one under 'sig-network' | 08:59 |
| flwang1 | i tried rerun on the same cluster, but still failed | 08:59 |
| *** ykarel|lunch is now known as ykarel | 09:00 | |
| flwang1 | brtknr: do you have any new comment on https://review.opendev.org/#/c/710384/ ? | 09:01 |
| brtknr | flwang1: yes, what version of magnum auto healer supports this feature? | 09:03 |
| brtknr | should we try and ensure that the magnum_auto_healer_tag is a supported version? | 09:03 |
| flwang1 | i don't know yet, since that patch will depend on magnum, not the reverse | 09:03 |
| flwang1 | we can get the PR in before magnum support update health status | 09:04 |
| brtknr | ok makes sense to make that a follow on PR | 09:06 |
| flwang1 | I will submit a follow patch to update the version, as long as the PR merged in CPO | 09:08 |
| brtknr | flwang1: ok sounds good | 09:09 |
| flwang1 | brtknr: i appreciate your review on this one | 09:09 |
| brtknr | flwang1: no probs | 09:10 |
| brtknr | flwang1: i wonder how we can fix the calico issue | 09:10 |
| flwang1 | brtknr: any technical issue can be fixed :) | 09:11 |
| flwang1 | sometimes a very hard, tricky problem could be a simple oneline config change :) | 09:13 |
| brtknr | flwang1: true | 09:16 |
| flwang1 | on v1.17.4, seems the storage case i mentioned above passed | 09:17 |
| *** udesale_ has joined #openstack-containers | 09:35 | |
| *** udesale has quit IRC | 09:38 | |
| *** flwang1 has quit IRC | 09:59 | |
| openstackgerrit | Bharat Kunwar proposed openstack/magnum master: Add selinux_mode label https://review.opendev.org/713193 | 10:48 |
| openstackgerrit | Bharat Kunwar proposed openstack/magnum master: fcos: Mount /:/rootfs:ro to Kubelet https://review.opendev.org/714423 | 10:54 |
| openstackgerrit | Merged openstack/magnum master: k8s: Fix logic of when a cluster API is accessible https://review.opendev.org/714589 | 11:13 |
| *** sapd1_x has quit IRC | 11:37 | |
| openstackgerrit | Merged openstack/magnum master: Fix calico regression issue caused by default ipv4pool change https://review.opendev.org/715093 | 11:49 |
| *** sapd1_x has joined #openstack-containers | 11:55 | |
| openstackgerrit | Bharat Kunwar proposed openstack/magnum stable/train: Fix join of status_reason https://review.opendev.org/713928 | 12:13 |
| openstackgerrit | Bharat Kunwar proposed openstack/magnum stable/train: Fix join of status_reason https://review.opendev.org/713928 | 12:15 |
| openstackgerrit | Bharat Kunwar proposed openstack/magnum stable/train: k8s: Fix logic of when a cluster API is accessible https://review.opendev.org/715410 | 12:16 |
| *** openstackstatus has quit IRC | 13:01 | |
| *** openstack has joined #openstack-containers | 13:04 | |
| *** ChanServ sets mode: +o openstack | 13:04 | |
| *** rcernin has quit IRC | 13:23 | |
| *** sapd1_x has quit IRC | 13:46 | |
| openstackgerrit | Merged openstack/magnum stable/train: Fix join of status_reason https://review.opendev.org/713928 | 14:23 |
| *** vishalmanchanda has quit IRC | 14:32 | |
| *** ykarel is now known as ykarel|afk | 14:34 | |
| *** sapd1_x has joined #openstack-containers | 14:42 | |
| *** dave-mccowan has joined #openstack-containers | 14:53 | |
| *** udesale_ has quit IRC | 15:20 | |
| *** ykarel|afk is now known as ykarel | 15:36 | |
| *** ykarel is now known as ykarel|away | 16:31 | |
| *** pcaruana has quit IRC | 17:10 | |
| *** pcaruana has joined #openstack-containers | 17:23 | |
| *** sapd1_x has quit IRC | 18:12 | |
| *** ramishra has quit IRC | 18:33 | |
| *** flwang1 has joined #openstack-containers | 19:40 | |
| flwang1 | brtknr: strigazi: can you please revisit https://review.opendev.org/#/c/710384/ ? | 19:40 |
| *** kgz has quit IRC | 19:45 | |
| *** kgz has joined #openstack-containers | 19:49 | |
| openstackgerrit | Merged openstack/magnum stable/train: k8s: Fix logic of when a cluster API is accessible https://review.opendev.org/715410 | 20:36 |
| brtknr | flwang1: sure thing | 21:37 |
| brtknr | i am mostly happy with it when i read the code, just need to find time to test drive | 21:37 |
| brtknr | are you happy to take this: https://review.opendev.org/#/c/714423/ | 21:37 |
| brtknr | would be nice to get it in as we'd like to backport to train if possible | 21:38 |
| brtknr | along with https://review.opendev.org/#/c/713699/ | 21:38 |
| brtknr | to support cinder csi | 21:38 |
| brtknr | flwang1: btw do we also need the client side patch? or was that just for testing? | 21:39 |
| brtknr | python-magnumclient health_status patch | 21:40 |
| flwang1 | brtknr: we do need the client side support | 21:43 |
| flwang1 | brtknr: are you going to address strigazi's comments on https://review.opendev.org/#/c/714423/ ? | 21:44 |
| openstackgerrit | Bharat Kunwar proposed openstack/magnum master: [k8s] Upgrade calico/coredns to the latest stable version https://review.opendev.org/705599 | 21:49 |
| openstackgerrit | Bharat Kunwar proposed openstack/magnum master: k8s: Upgrade etcd to v3.4.5 and use quay.io/coreos/etcd https://review.opendev.org/714719 | 21:49 |
| openstackgerrit | Bharat Kunwar proposed openstack/magnum master: k8s: Upgrade flannel to v0.12.0 https://review.opendev.org/714720 | 21:49 |
| openstackgerrit | Bharat Kunwar proposed openstack/magnum master: docs: Bring user docs up to date with recent changes https://review.opendev.org/714721 | 21:50 |
| brtknr | flwang1: oops sorry didnt mean to upload that calico/coredns change | 21:52 |
| flwang1 | brtknr: :) | 22:52 |
| openstackgerrit | Merged openstack/magnum master: Use cluster name for fixed_network instead of private https://review.opendev.org/714574 | 23:14 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!