| opendevreview | Merged openstack/openstack-ansible-os_glance stable/2023.2: Fix iteration over backends config https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/906491 | 00:55 |
|---|---|---|
| opendevreview | Merged openstack/openstack-ansible-os_nova master: Always distribute qemu config file https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/908160 | 10:51 |
| jrosser | good morning | 10:55 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_nova stable/2023.2: Always distribute qemu config file https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/908265 | 10:55 |
| noonedeadpunk | o/ | 10:55 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_nova stable/2023.1: Always distribute qemu config file https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/908266 | 10:55 |
| * noonedeadpunk still remember about the poll | 10:55 | |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Add SLURP upgrade jobs https://review.opendev.org/c/openstack/openstack-ansible/+/907729 | 10:57 |
| opendevreview | Merged openstack/openstack-ansible-os_masakari stable/2023.2: Add sqlalchemy-migrate to workaround masakari issues https://review.opendev.org/c/openstack/openstack-ansible-os_masakari/+/907722 | 10:59 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Add Octavia OVN provider to required repos https://review.opendev.org/c/openstack/openstack-ansible/+/908292 | 11:33 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_octavia master: Implement support for octavia-ovn-provider driver https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/868462 | 11:33 |
| * NeilHanlon is willing to help w/ bug squash too | 11:45 | |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-ops master: Add collection to deploy magnum cluster-api with vexxhost driver https://review.opendev.org/c/openstack/openstack-ansible-ops/+/901450 | 13:05 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-ops master: Cluster API Bootstrapping playbook https://review.opendev.org/c/openstack/openstack-ansible-ops/+/902178 | 13:09 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-ops master: Add role to install and run sonobouy k8s validation tests https://review.opendev.org/c/openstack/openstack-ansible-ops/+/906054 | 13:09 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-ops master: Add playbook to run functional test of magnum capi driver https://review.opendev.org/c/openstack/openstack-ansible-ops/+/906361 | 13:09 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-ops master: Add hook playbook install and test magnum capi driver https://review.opendev.org/c/openstack/openstack-ansible-ops/+/906363 | 13:10 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-ops master: Add hook playbook install and test magnum capi driver https://review.opendev.org/c/openstack/openstack-ansible-ops/+/906363 | 13:10 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_magnum master: Add job to test Vexxhost cluster API driver https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/905199 | 13:12 |
| jrosser | noonedeadpunk: how about this as well for small but messy patch https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/906641 | 13:14 |
| jrosser | tempest_install is a current variable, but it `end_play` in the playbook so the role never runs to create the resources like public network | 13:15 |
| jrosser | so feels ugly haveing two badly named vars really | 13:15 |
| jrosser | mnasiadka: i'd be interested to see what you do with sonobouy in magnum, to make a CI friendly thing that tests more than just the API endpoint | 13:35 |
| mnasiadka | jrosser: that's the plan | 13:36 |
| jrosser | i have an openstack-ansible job that deploys OSA/cluster_api/workload cluster then runs sonobouy --quick, kind of similar to what you have in https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/907717 | 13:36 |
| jrosser | but in ansible rather than tempest plugin | 13:36 |
| noonedeadpunk | jrosser: so that's do avoid defining "expected" resources elsewhere for https://opendev.org/openstack/openstack-ansible/src/commit/55e9baecac9def6d3dc8d9232a6f79307d3c1754/playbooks/openstack-resources.yml? | 13:36 |
| jrosser | noonedeadpunk: yeah so i wanted to run the tempest role and have it just create the resources | 13:38 |
| jrosser | but thats not possible right now because `tempest_run` is checked in the playbook, not the role | 13:38 |
| jrosser | `tempest_install` i mean | 13:38 |
| jrosser | here https://github.com/openstack/openstack-ansible/blob/master/playbooks/os-tempest-install.yml#L34 | 13:39 |
| noonedeadpunk | that kinda creates another special case for tempest, on top of all special cases we already have for it | 13:40 |
| jrosser | yes, this is why i don't like my patch, its just quick hack to get the capi stuff going at all | 13:40 |
| noonedeadpunk | but kinda alternative would be to move all defaults and logic and crap out of tempest role to integrated repo | 13:41 |
| jrosser | yeah, or even from tempest role to tempest playbook | 13:41 |
| noonedeadpunk | yeah... | 13:41 |
| noonedeadpunk | and you need/expect what specifically for capi? | 13:42 |
| jrosser | some of the data is sadly in os_tempest/defaults/main.yml that deswcribes it all | 13:42 |
| jrosser | i just needed the public network, really as simple as that | 13:42 |
| jrosser | but i didnt want to put that into the capi stuff itself, because that also needs to run on a real cloud | 13:43 |
| noonedeadpunk | I mean. Maybe it's possible to define that for openstack-resources.yml playbook? | 13:43 |
| jrosser | ah, yes | 13:43 |
| noonedeadpunk | and add it to some bigger playbook? | 13:43 |
| jrosser | perhaps we take some super-generic common things like that out of the tempest role as a first step | 13:43 |
| noonedeadpunk | or using hooks | 13:43 |
| jrosser | and then have some AIO specific vars which make a fake public network for the AIO case | 13:44 |
| jrosser | i'll take a look at that | 13:45 |
| noonedeadpunk | or maybe just set `tempest_install: true; tempest_run: false`? | 13:45 |
| noonedeadpunk | It will take some extra time.... | 13:45 |
| jrosser | well it still does the install and venv and everything | 13:45 |
| jrosser | which was just time | 13:45 |
| jrosser | same for rally | 13:45 |
| noonedeadpunk | But probably not too much | 13:45 |
| jrosser | so i think there is another kind of seperate thing which is properly disabling tempest / rally | 13:46 |
| jrosser | but if we move some fundamentals like public network out of there, then it becomes easier | 13:46 |
| noonedeadpunk | How much I don't like that there's no way to provide tags during playbook import.... | 13:46 |
| noonedeadpunk | while I don;t like it very much, we can land it, as I totally see how much work it is otherwise | 13:47 |
| jrosser | i can do something better than that patch | 13:48 |
| amarao | Hello. I found that option horizon_container_mysql_password is not used anywere (nor in os_horizon role, nor in playbooks of openstack-ansible). Should I send PR for removal? | 14:01 |
| amarao | The same with ceilometer_container_db_password. Or, may be, I searched them not thorough enough? | 14:06 |
| noonedeadpunk | amarao: ceilomter totally don't need database as of today and it's not used for sure | 14:07 |
| noonedeadpunk | about horizon... I can recall some usecase where DB might be needed for horizon.... | 14:08 |
| noonedeadpunk | But I don't see anywhere where we'd configured it | 14:09 |
| noonedeadpunk | so I guess in these both cases should be fine to clean out | 14:17 |
| amarao | Okay, thanks, I'll send PRs. | 14:47 |
| amarao | One more thing. I send PR for openstack module (subnet, allow cidr and subnet_pool to be used together). I asked to see it few times in #openstack-ansible-sig, but there is no much activity here, and I don't know where and whom to nag. https://review.opendev.org/c/openstack/ansible-collections-openstack/+/907301 | 14:48 |
| noonedeadpunk | amarao: can you add some test to the patch? somewhere to https://opendev.org/openstack/ansible-collections-openstack/src/branch/master/ci/roles/subnet | 14:54 |
| noonedeadpunk | but you can also try pinging in #openstack-sdks I can try to bother ppl from my side as well. | 14:55 |
| amarao | Thank you, I will. The actual setup I used it for, is subnet pool with muliple cidrs, and cidr is mandatory parameter when creating subnet from such subnet pool. | 14:56 |
| noonedeadpunk | ah, yes, fair enough actually | 14:57 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Remove distro_ceph template from project defenition https://review.opendev.org/c/openstack/openstack-ansible/+/908322 | 15:39 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Add Octavia OVN provider to required repos https://review.opendev.org/c/openstack/openstack-ansible/+/908292 | 15:39 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Remove galera_client from required projects https://review.opendev.org/c/openstack/openstack-ansible/+/908324 | 15:47 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-ops master: Add collection to deploy magnum cluster-api with vexxhost driver https://review.opendev.org/c/openstack/openstack-ansible-ops/+/901450 | 16:42 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-ops master: Cluster API Bootstrapping playbook https://review.opendev.org/c/openstack/openstack-ansible-ops/+/902178 | 16:42 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-ops master: Add role to install and run sonobouy k8s validation tests https://review.opendev.org/c/openstack/openstack-ansible-ops/+/906054 | 16:42 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-ops master: Add playbook to run functional test of magnum capi driver https://review.opendev.org/c/openstack/openstack-ansible-ops/+/906361 | 16:42 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-ops master: Add hook playbook install and test magnum capi driver https://review.opendev.org/c/openstack/openstack-ansible-ops/+/906363 | 16:42 |
| mgariepy | spatel, reading your post on k8s/capi :) | 17:02 |
| spatel | Please.. pass feedback :) | 17:03 |
| mgariepy | url in blue on the black background are hard to read ;) | 17:04 |
| * jrosser reminds about the docs here https://review.opendev.org/c/openstack/openstack-ansible-ops/+/901450/14/mcapi_vexxhost/doc/source/README.rst | 17:05 | |
| spatel | :( I need to fix template | 17:05 |
| spatel | Just highlight and copy paste :) | 17:05 |
| mgariepy | i know :) | 17:05 |
| mgariepy | haha | 17:05 |
| jrosser | i have a bunch of debugging stuff to write up too | 17:06 |
| jrosser | as thats slightly 8-O if this is all new to you | 17:06 |
| mgariepy | debuging unknown stuff is ... fun .. ; | 17:07 |
| mgariepy | unless it's in prod haha | 17:07 |
| spatel | In my next blog I am going to add some debug and backup/restore stuff | 17:08 |
| spatel | mgariepy my deployment is all about kolla-ansible. You may need to adjust for your deployment tool | 17:09 |
| mgariepy | yeah i guess so :) | 17:09 |
| jrosser | did anyone try out my openstack-ansible patches yet? | 17:10 |
| mgariepy | i didn't had time to test it yet. | 17:10 |
| mgariepy | but i will for sure. | 17:10 |
| jrosser | it will run OK in a 32G AIO, so it's not too bad to set up a test | 17:11 |
| mgariepy | when i run aio i always do 30ish gb or so unless i'm testing only a small thing. then i do 15gb one. | 17:11 |
| noonedeadpunk | jrosser: let's ask Mohammed to include this writing as well :D | 17:13 |
| jrosser | this actually is interesting point | 17:13 |
| jrosser | as documentation in the ops repo is pretty hard | 17:13 |
| jrosser | the usual openstack/sphinx/tox setup really does not seem to like multiple pretty independant subdirs of stuff | 17:14 |
| noonedeadpunk | I _think_ this can be structured and included.... | 17:15 |
| jrosser | so this is pretty useless right now https://docs.openstack.org/openstack-ansible-ops/latest/ | 17:15 |
| noonedeadpunk | As you can do literalinclude from any part of the repo? | 17:15 |
| jrosser | but the experience of just visiting https://github.com/openstack/openstack-ansible-ops/tree/master/elk_metrics_7x is pretty good | 17:15 |
| noonedeadpunk | well, it doesn't have to be that useless... | 17:15 |
| jrosser | so i did have some attempt to add the capi docs to the offical docs / sphinx stuff | 17:16 |
| noonedeadpunk | I can spend couple of cycles to see what can be done | 17:16 |
| jrosser | and was kind of down a rabbit hole of other folk also struggling with paths that needed to go out "above" the root of the doc index | 17:16 |
| jrosser | oh i remember, it can't do a "doc of docs" with a top level index | 17:18 |
| jrosser | and so everything in adjacent subdirs turns a bit messy | 17:18 |
| noonedeadpunk | hm. I kinda know that some of our projects do include of readme files from root folder rather then docs | 17:21 |
| jrosser | it might also have been including images was wierd | 17:22 |
| jrosser | as the path is well what? relative to the place you literalinclude from, or relative to the thing you include | 17:23 |
| jrosser | basically i did give up :) | 17:23 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_neutron master: Do not configure RPC for Neutron with OVN driver https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/907245 | 18:43 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_octavia master: Implement support for octavia-ovn-provider driver https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/868462 | 19:35 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_neutron master: Add VPNaaS OVN support https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/908341 | 19:43 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!