Friday, 2019-10-18

*** sapd1 has joined #openstack-containers01:09
*** sapd1 has quit IRC01:39
*** sapd1 has joined #openstack-containers02:37
*** jerryz has quit IRC02:45
*** ricolin_ has joined #openstack-containers02:51
*** razorfishsl has joined #openstack-containers02:58
*** sapd1 has quit IRC03:06
*** razorfishsl has quit IRC03:10
*** trungnv has joined #openstack-containers03:12
*** trungnv has joined #openstack-containers03:13
*** dave-mccowan has quit IRC03:21
*** goldyfruit has quit IRC03:46
*** ykarel|away has joined #openstack-containers04:00
*** ykarel|away is now known as ykarel04:00
*** Qiwei has quit IRC04:23
*** trungnv has quit IRC05:00
*** ykarel is now known as ykarel|afk05:23
*** ykarel|afk is now known as ykarel06:08
*** trungnv has joined #openstack-containers06:40
*** sapd1 has joined #openstack-containers06:50
*** trident has quit IRC07:00
*** ramishra has joined #openstack-containers07:01
*** trident has joined #openstack-containers07:04
strigaziflwang: here?07:05
*** lpetrut has joined #openstack-containers07:05
strigazidid you have time to look at the backports?07:06
*** udesale has joined #openstack-containers07:08
*** pcaruana has joined #openstack-containers07:26
*** ricolin_ is now known as ricolin07:30
brtknrflwang: also this: https://review.opendev.org/68892807:43
openstackgerritMerged openstack/magnum stable/train: Fixing typos and spelling errors in driver template files  https://review.opendev.org/68919708:10
openstackgerritMerged openstack/magnum stable/train: Convert fixed_subnet name to uuid for OCCM  https://review.opendev.org/68919808:10
openstackgerritMerged openstack/magnum stable/train: update api-ref for clustertemplate  https://review.opendev.org/68919908:10
openstackgerritMerged openstack/magnum stable/train: Support Fedora CoreOS 30  https://review.opendev.org/68920008:10
openstackgerritMerged openstack/magnum stable/train: ng-10: Fix cluster template conditions  https://review.opendev.org/68920108:15
openstackgerritMerged openstack/magnum stable/train: ng-11: API microversion 1.9  https://review.opendev.org/68920208:15
openstackgerritMerged openstack/magnum stable/train: ng-12: Label nodegroup nodes  https://review.opendev.org/68920308:20
openstackgerritMerged openstack/magnum stable/train: ng-13: Support nodegroup upgrade  https://review.opendev.org/68920408:20
*** ttsiouts has joined #openstack-containers08:23
brtknrstrigazi: do you know where I can find reference to ostree_commit and ostree_remote?08:44
*** ykarel is now known as ykarel|lunch08:46
strigazibrtknr: I don't remember, I'll have to check.08:55
*** sapd1 has quit IRC09:01
brtknrstrigazi: dont worry, I found it, its rpm-ostree upgrade --check09:10
brtknralthough there must be an easier way09:15
*** ykarel|lunch is now known as ykarel09:39
strigazibrtknr: we don't need this for long09:40
strigazibrtknr: I guess09:40
*** ttsiouts has quit IRC09:51
*** ttsiouts has joined #openstack-containers09:51
*** sapd1 has joined #openstack-containers09:53
*** ttsiouts has quit IRC09:56
*** ramishra has quit IRC09:59
*** ykarel is now known as ykarel|afk10:07
*** ramishra has joined #openstack-containers10:27
*** ykarel|afk is now known as ykarel10:43
*** zigo has quit IRC10:54
*** ttsiouts has joined #openstack-containers10:59
brtknrstrigazi: is OS upgrade really supported by specifying a new image_name?11:01
brtknrNot what I see in the code11:02
*** zigo has joined #openstack-containers11:03
openstackgerritBharat Kunwar proposed openstack/magnum master: [fedora-atomic][k8s] Support operating system upgrade  https://review.opendev.org/66959311:11
brtknrstrigazi: I tested the above for OS upgrade and it works :)11:11
brtknralthough it doesnt appear to be working for kube tags11:22
brtknrcc ttsiouts11:22
brtknrany thoughts why?11:22
*** trungnv has quit IRC11:30
openstackgerritBharat Kunwar proposed openstack/magnum master: [fedora-atomic][k8s] Support operating system upgrade  https://review.opendev.org/66959311:46
*** mgariepy has joined #openstack-containers12:07
*** ramishra has quit IRC12:12
*** udesale has quit IRC12:13
*** udesale has joined #openstack-containers12:16
ttsioutsbrtknr: I'll try to reproduce now.12:18
ttsioutsI reviewed the code and left some comments.12:18
ttsioutsbrtknr: I'm -1 cause update_max_batch_size is ignored if kube_tag is set.12:19
ttsioutsbrtknr: kube_tag seems to be working for me..12:42
ttsioutsbrtknr: kube_tag upgrade seems to be working for me on 669593..12:42
*** ttsiouts has quit IRC12:53
*** ttsiouts has joined #openstack-containers12:54
*** ttsiouts has quit IRC12:58
*** mgariepy has quit IRC13:00
*** ttsiouts has joined #openstack-containers13:10
brtknrttsiouts: thank you :)13:11
ttsioutsbrtknr: http://paste.openstack.org/show/784719/13:15
ttsioutsbrtknr: next test is to upgrade both kube_tag and ostree commit in one go13:15
brtknrttsiouts: i was about to try that!13:16
ttsioutsbrtknr: I'll let you know how it goes13:16
brtknrttsiouts: i dont understand if we need to specify both ostree_commit and ostree_remote or its either or13:16
brtknrlooking at the code, it seems to be either ostree_commit or ostree_remote13:17
*** goldyfruit has joined #openstack-containers13:28
*** ykarel is now known as ykarel|afk13:36
ttsioutsbrtknr: from the code it seems that if you give both of them only ostree_commit is used13:36
brtknrttsiouts: yes, should it use both13:37
ttsioutsbrtknr: s/both/ostree_commit and ostree_remote/13:37
ttsioutsbrtknr: not sure.13:37
*** ykarel|afk is now known as ykarel13:37
ttsioutsbrtknr: https://review.opendev.org/#/c/669593/17/magnum/drivers/common/templates/kubernetes/fragments/upgrade-kubernetes.sh@9113:38
ttsioutsbrtknr: according to this I think that it's either or13:38
ttsioutsbrtknr: flwang's note I mean13:39
brtknrhmm13:40
*** rcernin has quit IRC13:40
*** ricolin has quit IRC13:43
*** mgariepy has joined #openstack-containers13:46
*** dave-mccowan has joined #openstack-containers13:46
ttsioutsbrtknr: if the cluster template has one of them (ostree_commit and ostree_remote) nothing will pass heat13:48
ttsioutsbrtknr: https://review.opendev.org/#/c/669593/17/magnum/drivers/heat/driver.py@38413:48
ttsioutsbrtknr: which means that with the current implementation both are needed but only one is used.13:50
ttsioutsbrtknr: it's a bit weird.13:51
*** pcaruana has quit IRC13:53
*** dave-mccowan has quit IRC13:56
*** munimeha1 has joined #openstack-containers13:57
brtknrjust fixing the logic14:15
brtknri will make it accept args in the same order as the bash script14:16
brtknrbest thing to do is to document it14:16
ttsioutsbrtknr: everything got upgraded for me. both os and kube_tag14:25
ttsioutsbrtknr: my only problem is this14:25
ttsioutskube-oz6lyl4vtlqf-master-0   Ready,SchedulingDisabled   master   17m   v1.14.714:25
brtknrit worked?14:25
ttsioutsbrtknr: yeap14:25
brtknrwhat is the problem?14:25
brtknrmaster didn't uncordon?14:26
ttsioutsyeap14:26
openstackgerritBharat Kunwar proposed openstack/magnum master: [fedora-atomic][k8s] Support operating system upgrade  https://review.opendev.org/66959314:26
openstackgerritBharat Kunwar proposed openstack/magnum master: [fedora-atomic][k8s] Support operating system upgrade  https://review.opendev.org/66959314:44
*** sapd1 has quit IRC14:47
*** sapd1 has joined #openstack-containers14:52
*** FlorianFa has quit IRC15:01
*** ttsiouts has quit IRC15:03
*** mgariepy has quit IRC15:03
*** ttsiouts has joined #openstack-containers15:03
*** sapd1 has quit IRC15:08
*** ttsiouts has quit IRC15:08
*** udesale has quit IRC15:10
*** udesale has joined #openstack-containers15:11
*** mgariepy has joined #openstack-containers15:16
*** ykarel is now known as ykarel|afk15:55
*** ykarel|afk is now known as ykarel15:55
*** lpetrut has quit IRC17:09
*** ykarel is now known as ykarel|away17:44
*** jerryz has joined #openstack-containers17:46
*** udesale has quit IRC17:56
*** ykarel|away has quit IRC17:59
*** trident has quit IRC18:57
*** trident has joined #openstack-containers19:00
*** dave-mccowan has joined #openstack-containers19:10
*** mgariepy has quit IRC19:47
*** jerryz has quit IRC19:58
*** jerryz has joined #openstack-containers20:22
*** Qiwei has joined #openstack-containers20:58
*** lpetrut has joined #openstack-containers21:06
*** goldyfruit has quit IRC21:10
*** lpetrut has quit IRC21:13

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!