| noonedeadpunk | so I think now we should land and backport to Z https://review.opendev.org/c/openstack/openstack-ansible/+/882090/1 to unblock magnum role | 10:02 |
|---|---|---|
| noonedeadpunk | and maybe even Y.... | 10:02 |
| noonedeadpunk | I think with fixed magnum we can create stable/2023.1 branch for roles... | 10:04 |
| noonedeadpunk | We have Zun ofc... but well, we'd need to backport fix for it quite deep anyway | 10:04 |
| damiandabrowski | please note that we also have an alternate patch for magnum | 10:06 |
| damiandabrowski | https://review.opendev.org/c/openstack/openstack-ansible/+/881566 | 10:06 |
| damiandabrowski | but well, I noticed you don't really like it | 10:07 |
| noonedeadpunk | I'm treating more as a complementary rather then alternative to be frank. as eventually we can just not create cluster templates at all according to your coomit mesaage | 10:09 |
| noonedeadpunk | as what;s the point to define magnum_cluster_templates at all? | 10:09 |
| noonedeadpunk | It will reduce code to maintain even more :) | 10:15 |
| damiandabrowski | the thing is: cluster templates are covered by tempest tests. label values(like kube_tag) wont be used because we dont create magnum clusters. | 10:19 |
| damiandabrowski | i get your point that setting some label may be a good option, but maybe it would be better to set some dummy one? | 10:19 |
| damiandabrowski | Look here: we even bump values for these labels even they are not used(but commit msg suggests that they are) | 10:19 |
| damiandabrowski | https://opendev.org/openstack/openstack-ansible/commit/3881b5c7cb419b67047a09c76842b3c738547f3b | 10:20 |
| damiandabrowski | at the beginning i was pretty sure that if we define these labels, we also spin up magnum clusters😀 | 10:20 |
| damiandabrowski | idk, i leave it up to you but i think that current setup is a bit confusing | 10:23 |
| noonedeadpunk | to be frank would be revise disabled tempest tests that did spin up clusters... | 10:27 |
| noonedeadpunk | As eventually would be great to resume testing that... | 10:27 |
| damiandabrowski | +1 | 10:27 |
| damiandabrowski | but i also tried to run these tests outside CI and there was something wrong with them :/ | 10:28 |
| noonedeadpunk | yeah, that's frustrating part :( | 10:30 |
| opendevreview | Merged openstack/openstack-ansible-os_swift master: Add TLS support to swift backends https://review.opendev.org/c/openstack/openstack-ansible-os_swift/+/880396 | 11:38 |
| opendevreview | Merged openstack/openstack-ansible-os_octavia master: Add TLS support to octavia backends https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/880680 | 11:51 |
| spatel | Can i do host evacuation without shutting down compute nodes? | 15:45 |
| Tarlan | Hi | 16:41 |
| Tarlan | If you were to go for Kubernetes-as-a-service, which one of these tools would you choose? | 16:42 |
| spatel | magnum? | 16:43 |
| Tarlan | 1. Magnum 2. Cluster-API 3. Gardener | 16:43 |
| Tarlan | spatel: You offer Magnum? Have you ever run magnum? seems it's not a working module! | 16:45 |
| NeilHanlon | Tarlan: we're working on fixing magnum in OSA | 16:46 |
| spatel | NeilHanlon +1 | 16:49 |
| supamatt | I run Magnum, works pretty well | 16:49 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!