| *** ricolin has joined #openstack-containers | 02:27 | |
| *** ykarel|away has joined #openstack-containers | 02:36 | |
| *** ykarel|away has quit IRC | 04:05 | |
| *** gouthamr has quit IRC | 04:15 | |
| *** gouthamr has joined #openstack-containers | 04:16 | |
| *** ykarel|away has joined #openstack-containers | 04:25 | |
| *** ykarel|away is now known as ykarel | 04:31 | |
| *** ykarel is now known as ykarel|afk | 04:50 | |
| *** iokiwi has quit IRC | 05:23 | |
| *** iokiwi has joined #openstack-containers | 05:24 | |
| *** goldyfruit_ has joined #openstack-containers | 05:26 | |
| *** ramishra has joined #openstack-containers | 05:47 | |
| *** ykarel|afk is now known as ykarel | 06:00 | |
| *** spsurya has joined #openstack-containers | 06:33 | |
| *** udesale has joined #openstack-containers | 06:46 | |
| *** ttsiouts has joined #openstack-containers | 07:03 | |
| *** ramishra has quit IRC | 07:06 | |
| *** pcaruana has joined #openstack-containers | 07:15 | |
| *** ttsiouts has quit IRC | 07:26 | |
| *** ttsiouts has joined #openstack-containers | 07:26 | |
| *** ttsiouts has quit IRC | 07:31 | |
| *** ivve has joined #openstack-containers | 07:41 | |
| *** ttsiouts has joined #openstack-containers | 07:56 | |
| brtknr | guilhermesp: get any further | 07:58 |
|---|---|---|
| strigazi | brtknr: flwang https://review.opendev.org/#/c/685749/ let's take this? | 08:09 |
| *** ttsiouts has quit IRC | 08:10 | |
| *** ttsiouts has joined #openstack-containers | 08:11 | |
| *** ttsiouts_ has joined #openstack-containers | 08:14 | |
| *** ttsiouts has quit IRC | 08:15 | |
| *** rcernin has quit IRC | 08:15 | |
| *** lpetrut has joined #openstack-containers | 08:25 | |
| strigazi | brtknr: thanks | 09:12 |
| brtknr | strigazi: I'm happy with it... worked for both cases | 09:12 |
| brtknr | look forward to the fcos part! | 09:13 |
| strigazi | brtknr: I'm doing the keypair thing now | 09:14 |
| *** ramishra has joined #openstack-containers | 09:17 | |
| brtknr | Can you help me understand a bit about the upgrades strigazi | 09:27 |
| brtknr | If the template we’re upgrading to has a different os or a different network plugin, what happens? | 09:27 |
| strigazi | it will be ignored | 09:28 |
| brtknr | Just the kube tag is used? | 09:28 |
| brtknr | Is anything else used? | 09:28 |
| strigazi | some reasomable usage is required by the operator | 09:28 |
| strigazi | yes | 09:29 |
| strigazi | kube_tag only | 09:29 |
| brtknr | Hmm any reason we don’t just pass the label during upgrade? | 09:29 |
| *** openstackgerrit has joined #openstack-containers | 09:29 | |
| openstackgerrit | Leopard Ma proposed openstack/magnum master: Add wiki Admin guide and Contributing notes link to README https://review.opendev.org/686558 | 09:29 |
| strigazi | because it is just wrong | 09:30 |
| brtknr | E.g use the same template but a different label | 09:30 |
| strigazi | it will be a nightmare to maintain | 09:30 |
| brtknr | What do you mean it is just wrong | 09:30 |
| strigazi | how you tell users to upgrade? | 09:30 |
| strigazi | maintain docs? | 09:30 |
| brtknr | Elaborate pleae | 09:30 |
| strigazi | magnum tries to be a managed service | 09:31 |
| strigazi | so the operators publish CTs and users consume | 09:31 |
| strigazi | already passing labels in cluster creationg is pretty bad | 09:31 |
| brtknr | Hmm the thing is you can override labels during cluster creation , it would make sense to do the same during upgrade | 09:31 |
| brtknr | Ok I’d prefer if it was consistently bad :) | 09:32 |
| strigazi | really? | 09:32 |
| strigazi | you prefer consistently bad? | 09:32 |
| brtknr | What I meant was, I’d prefer a consistent behaviour for cluster creation and upgrade | 09:33 |
| brtknr | E.g if it’s pissible to override labels during creations, it nakes sense to have the same for upgrade | 09:34 |
| strigazi | in any case, it would be way more dangerous to add the label randomness in uprgades | 09:34 |
| strigazi | also now it is merged :) | 09:34 |
| brtknr | Well maybe we need to remove label override for cluster creation | 09:35 |
| brtknr | I agree with the philosophy | 09:36 |
| strigazi | +100 to this, or refactor it? add another field, I don't know | 09:36 |
| strigazi | anyway, I'm going back to focs | 09:36 |
| strigazi | fcos | 09:36 |
| brtknr | Thanks for helping me understand! | 09:38 |
| strigazi | np :) | 09:38 |
| *** ttsiouts_ has quit IRC | 10:12 | |
| *** ttsiouts has joined #openstack-containers | 10:13 | |
| *** yasemin has joined #openstack-containers | 10:15 | |
| *** ttsiouts has quit IRC | 10:17 | |
| *** ttsiouts has joined #openstack-containers | 11:12 | |
| *** spsurya has quit IRC | 12:10 | |
| *** udesale has quit IRC | 12:16 | |
| *** udesale has joined #openstack-containers | 12:16 | |
| *** goldyfruit_ has quit IRC | 12:22 | |
| *** ivve has quit IRC | 12:41 | |
| *** vesper11 has quit IRC | 12:53 | |
| *** vesper11 has joined #openstack-containers | 12:54 | |
| *** udesale has quit IRC | 12:56 | |
| *** ramishra has quit IRC | 12:58 | |
| *** vesper11 has quit IRC | 12:59 | |
| *** vesper11 has joined #openstack-containers | 13:27 | |
| *** vesper11 has quit IRC | 13:31 | |
| *** vesper11 has joined #openstack-containers | 13:32 | |
| *** goldyfruit_ has joined #openstack-containers | 13:40 | |
| *** ttsiouts has quit IRC | 13:44 | |
| *** ttsiouts has joined #openstack-containers | 13:45 | |
| *** ttsiouts has quit IRC | 13:46 | |
| *** ttsiouts has joined #openstack-containers | 13:46 | |
| *** sapd1_x has joined #openstack-containers | 13:57 | |
| *** goldyfruit___ has joined #openstack-containers | 14:16 | |
| *** goldyfruit_ has quit IRC | 14:19 | |
| openstackgerrit | Theodoros Tsioutsias proposed openstack/magnum master: ng-10: Fix cluster template conditions https://review.opendev.org/685620 | 14:22 |
| openstackgerrit | Theodoros Tsioutsias proposed openstack/magnum master: ng-11: API microversion 1.9 https://review.opendev.org/686089 | 14:22 |
| openstackgerrit | Theodoros Tsioutsias proposed openstack/magnum master: ng-12: Label nodegroup nodes https://review.opendev.org/686362 | 14:22 |
| openstackgerrit | Theodoros Tsioutsias proposed openstack/magnum master: [WIP] ng-13: Support nodegroup upgrade https://review.opendev.org/686733 | 14:22 |
| ttsiouts | brtknr: Hi! I addressed your comments ^ | 14:23 |
| ttsiouts | brtknr: ng-13 adds support for upgrading nodegroups | 14:24 |
| ttsiouts | brtknr: but we should discuss a bit more about this | 14:25 |
| *** ricolin_ has joined #openstack-containers | 14:25 | |
| ttsiouts | brtknr: that's why I posted it as WIP | 14:26 |
| *** ricolin has quit IRC | 14:26 | |
| brtknr | ttsiouts: nice thanks! | 14:29 |
| brtknr | so upgrading nodegroups is currently unsupported? | 14:32 |
| brtknr | ttsiouts: ^ | 14:32 |
| *** goldyfruit_ has joined #openstack-containers | 14:36 | |
| *** goldyfruit___ has quit IRC | 14:39 | |
| ttsiouts | brtknr: yes. up till now you could upgrade only the deafult stack | 14:47 |
| ttsiouts | s/deafult/default | 14:48 |
| ttsiouts | ng-13 allows you to use properly the --nodegroup option in the upgrade api | 14:49 |
| *** ricolin_ has quit IRC | 14:49 | |
| brtknr | ttsiouts: makes sense, i only used upgrade function for the first time yesterday... we've always gone for the destroy and recreate approach so its never been an issue. | 14:51 |
| ttsiouts | brtknr: as I mentioned, we should discuss how to properly handle upgrades. | 14:54 |
| ttsiouts | brtknr: so next week we should do that with strigazi and flwang too. | 14:55 |
| brtknr | ttsiouts: sure np, please add to agenda :) | 14:57 |
| ttsiouts | brtknr: done! | 14:58 |
| brtknr | ttsiouts: you added it to last week's agenda :P | 14:58 |
| ttsiouts | brtknr: haha friday afternoon | 14:59 |
| brtknr | ttsiouts: :P | 15:00 |
| brtknr | ttsiouts: wwhat are the options for upgrades? | 15:00 |
| brtknr | just wondering about what design decision would require wider consensus | 15:00 |
| ttsiouts | brtknr: for now the safest thing is to just update the kube_tag using a new cluster_template | 15:01 |
| brtknr | ttsiouts: yes, i only learnt that yesterday... | 15:02 |
| ttsiouts | brtknr: but there is something I don't really like in the current implementation. | 15:02 |
| brtknr | ttsiouts: which is? | 15:03 |
| ttsiouts | brtknr: you can override labels including availability zone, etc | 15:03 |
| ttsiouts | brtknr: which are things that shouldn't be touched IMHO | 15:03 |
| ttsiouts | brtknr: because it will cause the nodes to be rebuilt | 15:04 |
| brtknr | I understood from my conversation with strigazi this morning that all labels are ignored apart from kube_tags during upgrades? | 15:06 |
| *** usr2033 has quit IRC | 15:08 | |
| *** ykarel is now known as ykarel|afk | 15:15 | |
| ttsiouts | brtknr: it seems that they are not ignored. Which makes sense for some of them | 15:32 |
| ttsiouts | brtknr: this is what I wanted to discuss next week | 15:32 |
| brtknr | ttsiouts: do you know which labels are not ignored and what is the impact of these labels? would be good if you could describe the problem in detail either in a story on the storyboard or on the agenda itself | 15:34 |
| ttsiouts | brtknr: I'll try to have something for the meeting | 15:35 |
| ttsiouts | brtknr: I've got to run | 15:36 |
| brtknr | ttsiouts: have a good weekend! | 15:36 |
| ttsiouts | brtknr: have a good weekend! | 15:36 |
| *** ttsiouts has quit IRC | 15:37 | |
| *** ttsiouts has joined #openstack-containers | 15:38 | |
| *** ttsiouts has quit IRC | 15:42 | |
| *** iokiwi has quit IRC | 15:59 | |
| *** iokiwi has joined #openstack-containers | 16:02 | |
| *** ykarel|afk is now known as ykarel | 16:07 | |
| *** sapd1_x has quit IRC | 16:21 | |
| *** ykarel has quit IRC | 16:24 | |
| *** goldyfruit_ has quit IRC | 16:32 | |
| *** goldyfruit_ has joined #openstack-containers | 16:39 | |
| *** lpetrut has quit IRC | 16:41 | |
| *** goldyfruit_ has quit IRC | 16:46 | |
| *** ykarel has joined #openstack-containers | 16:53 | |
| *** lpetrut has joined #openstack-containers | 17:15 | |
| *** lpetrut has quit IRC | 17:49 | |
| *** ykarel is now known as ykarel|away | 17:56 | |
| *** pcaruana has quit IRC | 18:51 | |
| *** ykarel|away has quit IRC | 19:31 | |
| openstackgerrit | Bharat Kunwar proposed openstack/magnum master: Add compatibility matrix for kube_tag https://review.opendev.org/685675 | 20:55 |
| openstackgerrit | Bharat Kunwar proposed openstack/magnum master: Add compatibility matrix for kube_tag https://review.opendev.org/685675 | 21:20 |
| *** goldyfruit_ has joined #openstack-containers | 21:38 | |
| *** goldyfruit_ has quit IRC | 22:04 | |
| *** goldyfruit_ has joined #openstack-containers | 22:07 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!