| *** dviroel is now known as dviroel|out | 00:59 | |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible stable/xena: Bump os_zun role SHA to pick up new location for kata repo https://review.opendev.org/c/openstack/openstack-ansible/+/827412 | 05:54 |
|---|---|---|
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-nspawn_hosts stable/victoria: Remove openstack-ansible role test jobs. https://review.opendev.org/c/openstack/openstack-ansible-nspawn_hosts/+/827319 | 05:56 |
| jrosser | this is ready https://review.opendev.org/c/openstack/openstack-ansible-os_designate/+/824335 | 06:15 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-lxc_hosts master: Ansible systemd module can reload units without specifying a service https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/827217 | 08:59 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-lxc_hosts master: Remove legacy network scripts cleanup task https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/827425 | 08:59 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-lxc_hosts master: Remove vars files for EOL version of centos-8 https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/827426 | 08:59 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-openstack_hosts master: Remove condtionals for kernel module installation on debian https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/827427 | 09:01 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-openstack_hosts master: Remove ubuntu bionic vars file https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/827428 | 09:01 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_aodh master: Remove policy.json cleanup handler https://review.opendev.org/c/openstack/openstack-ansible-os_aodh/+/827430 | 09:04 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_aodh master: Remove legacy apache variables https://review.opendev.org/c/openstack/openstack-ansible-os_aodh/+/827431 | 09:04 |
| opendevreview | Merged openstack/openstack-ansible master: Fix tempest plugins versions https://review.opendev.org/c/openstack/openstack-ansible/+/826129 | 09:05 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_barbican master: Remove legacy policy.json cleanup handler https://review.opendev.org/c/openstack/openstack-ansible-os_barbican/+/827432 | 09:05 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_ceilometer master: Remove py2/py3 compatibility notes https://review.opendev.org/c/openstack/openstack-ansible-os_ceilometer/+/827433 | 09:07 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_ceilometer master: Remove legacy pipeline cleanup task https://review.opendev.org/c/openstack/openstack-ansible-os_ceilometer/+/827434 | 09:07 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_cinder master: Remove legacy policy.json cleanup handler https://review.opendev.org/c/openstack/openstack-ansible-os_cinder/+/827435 | 09:08 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_cloudkitty master: Remove legacy policy.json cleanup handler https://review.opendev.org/c/openstack/openstack-ansible-os_cloudkitty/+/827436 | 09:09 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_designate master: Remove legacy policy.json cleanup handler https://review.opendev.org/c/openstack/openstack-ansible-os_designate/+/827437 | 09:10 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_glance master: Remove legacy policy.json cleanup handler https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/827438 | 09:11 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_gnocchi master: Remove apache to uwsgi migration tasks and variables https://review.opendev.org/c/openstack/openstack-ansible-os_gnocchi/+/827439 | 09:12 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_heat master: Remove legacy policy.json cleanup handler https://review.opendev.org/c/openstack/openstack-ansible-os_heat/+/827440 | 09:13 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_ironic master: Remove legacy policy.json cleanup handler https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/827442 | 09:13 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_keystone master: Remove legacy policy.json cleanup handler https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/827443 | 09:15 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_magnum master: Do not install python development packages https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/824212 | 09:16 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_magnum master: Remove legacy policy.json cleanup handler https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/827444 | 09:16 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_manila master: Remove legacy policy.json cleanup handler https://review.opendev.org/c/openstack/openstack-ansible-os_manila/+/827445 | 09:17 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_masakari master: Remove legacy policy.json cleanup handler https://review.opendev.org/c/openstack/openstack-ansible-os_masakari/+/827446 | 09:18 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_mistral master: Remove legacy policy.json cleanup handler https://review.opendev.org/c/openstack/openstack-ansible-os_mistral/+/827448 | 09:18 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_neutron master: Remove legacy policy.json cleanup handler https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/827449 | 09:20 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_nova master: Remove legacy policy.json cleanup handler https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/827450 | 09:21 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_octavia master: Remove legacy policy.json cleanup handler https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/827451 | 09:21 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_sahara master: Remove legacy policy.json cleanup handler https://review.opendev.org/c/openstack/openstack-ansible-os_sahara/+/827452 | 09:22 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_senlin master: Remove legacy policy.json cleanup handler https://review.opendev.org/c/openstack/openstack-ansible-os_senlin/+/827453 | 09:23 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_tacker master: Remove legacy policy.json cleanup handler https://review.opendev.org/c/openstack/openstack-ansible-os_tacker/+/827454 | 09:23 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_trove master: Remove legacy policy.json cleanup handler https://review.opendev.org/c/openstack/openstack-ansible-os_trove/+/827455 | 09:24 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-lxc_hosts master: Simplify installtion of libpython https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/827458 | 09:40 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-ceph_client master: Simpify selection of the python interpreter. https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/827459 | 09:41 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-openstack_hosts master: Simplify generation of osbpo repo name https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/827460 | 09:42 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_horizon master: Only support python3 for apache wsgi module https://review.opendev.org/c/openstack/openstack-ansible-os_horizon/+/827462 | 09:43 |
| opendevreview | Jonathan Rosser proposed openstack/ansible-role-qdrouterd master: Only support python3 for the qpid bindings https://review.opendev.org/c/openstack/ansible-role-qdrouterd/+/827463 | 09:44 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Simplify mount options for new kernels. https://review.opendev.org/c/openstack/openstack-ansible/+/827464 | 09:46 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Remove workaround for OpenSUSE when setting AIO hostname https://review.opendev.org/c/openstack/openstack-ansible/+/827465 | 09:46 |
| jrosser | hmmmm fatal: [aio1]: FAILED! => {"attempts": 5, "changed": false, "msg": "Failed to download metadata for repo 'advanced-virtualization' | 09:58 |
| opendevreview | Merged openstack/openstack-ansible-openstack_hosts stable/ussuri: Remove legacy centos-8 jobs https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/827266 | 09:59 |
| agemuend | jrosser: Same error as me yesterday? ;) | 10:00 |
| jrosser | maybe :) | 10:00 |
| jrosser | i think the opendev CI mirrors rsync only a small number of times a day | 10:01 |
| jrosser | agemuend: what did you do? http://mirror.centos.org/centos/8-stream/virt/x86_64/advancedvirt-common/ ? | 10:03 |
| agemuend | the package centos-release-advanced-virtualization gave us this: http://mirrorlist.centos.org/?release=$avstream&arch=$basearch&repo=virt-$avdir | 10:05 |
| jrosser | hmm well i wonder if we did the right thing on master with this https://github.com/openstack/openstack-ansible-openstack_hosts/commit/b13adf0bef950449d33a69f7cd62fb5baab99605 | 10:07 |
| opendevreview | Merged openstack/openstack-ansible-nspawn_hosts stable/victoria: Remove openstack-ansible role test jobs. https://review.opendev.org/c/openstack/openstack-ansible-nspawn_hosts/+/827319 | 10:10 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-openstack_hosts master: Use advancedvirt-common repo for redhat-8 https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/827468 | 10:12 |
| jrosser | agemuend: ^ lets see how that goes, looks like it needs to be backported to all our branches with centos-8-stream support | 10:12 |
| jrosser | agemuend: have you been submitting bugs for any of these things? | 10:13 |
| agemuend | Nope, we just worked on it yesterday while you helped us | 10:19 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_manila stable/ussuri: Remove legacy centos-8 CI jobs. https://review.opendev.org/c/openstack/openstack-ansible-os_manila/+/827470 | 10:20 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-lxc_container_create stable/xena: Remove legacy centos-8 jobs https://review.opendev.org/c/openstack/openstack-ansible-lxc_container_create/+/827473 | 10:40 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-lxc_container_create stable/wallaby: Remove legacy centos-8 jobs https://review.opendev.org/c/openstack/openstack-ansible-lxc_container_create/+/827399 | 10:40 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_keystone stable/wallaby: Remove legacy centos-8 CI jobs https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/827474 | 10:44 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_keystone stable/victoria: Remove legacy centos-8 CI jobs https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/827400 | 10:44 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_keystone stable/ussuri: Remove legacy centos-8 CI jobs https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/827475 | 10:45 |
| noonedeadpunk | doh | 11:08 |
| noonedeadpunk | (related to osa/remove-centos8 ) | 11:09 |
| *** dviroel|out is now known as dviroel | 11:15 | |
| jrosser | this would be the next to need to merge https://review.opendev.org/c/openstack/openstack-ansible-os_manila/+/827470 | 11:42 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_neutron stable/victoria: Remove legacy centos-8 jobs https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/827483 | 11:44 |
| jrosser | noonedeadpunk: fyi xena branch is broken and we need a backport of this to fix it https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/827468 | 11:52 |
| jrosser | also wallaby too | 11:53 |
| noonedeadpunk | hm, i can recall we dropped that not that long ago as nothing was installed from there? | 11:53 |
| jrosser | only on master | 11:53 |
| jrosser | the path is different | 11:53 |
| jrosser | we didnt spot 8 vs. 8-stream in the URL | 11:54 |
| noonedeadpunk | and what we install from it now ?:) | 11:54 |
| jrosser | i don't know - we see what that patch does | 11:54 |
| jrosser | but either way its broken for stable branch | 11:54 |
| noonedeadpunk | well, maybe we should backport https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/824202 instead then? | 11:55 |
| jrosser | hrrm the patch to openstack-hosts isnt going to actually install nova | 11:55 |
| jrosser | well, i don't know, as that patch doesnt clean up the old repo | 11:56 |
| noonedeadpunk | we can add dnm one if needed | 11:56 |
| noonedeadpunk | oh, you mean for upgrade, yeah, that's fair | 11:57 |
| noonedeadpunk | but still, maybe we should jsut add clean-up of it instead of re-adding repo we don't need? | 11:57 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: DNM Test advancedvirt-common repo https://review.opendev.org/c/openstack/openstack-ansible/+/827485 | 11:58 |
| jrosser | could be - this needs someone clueful from the RH side to say what the right thing to do is | 11:59 |
| noonedeadpunk | oh, well, it has libvirt 7... | 12:01 |
| noonedeadpunk | and python3-libguestfs / python3-libvirt | 12:02 |
| noonedeadpunk | which means it's not needed https://opendev.org/openstack/openstack-ansible-os_nova/src/branch/master/vars/redhat.yml#L75-L77 | 12:03 |
| jrosser | ah more tidy up we can do? | 12:05 |
| noonedeadpunk | yeah, looks like that | 12:05 |
| noonedeadpunk | So I see now why we need virt repo :) | 12:09 |
| noonedeadpunk | But tbh - I don't see why we need it outside of nova computes | 12:10 |
| jrosser | i see quite a few of these on distro/metal/focal jobs "Unable to start service libvirtd-tls.socket: Job failed. See \"journalctl -xe\" for details.\n" | 12:11 |
| jrosser | noonedeadpunk: yes, well we can always move it to the nova role? | 12:11 |
| noonedeadpunk | Um... yes, but I guess I miss why we re-adding it to openstack_hosts at first place?:) | 12:13 |
| noonedeadpunk | to backport it? | 12:13 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_gnocchi master: Remove apache to uwsgi migration tasks and variables https://review.opendev.org/c/openstack/openstack-ansible-os_gnocchi/+/827439 | 12:14 |
| jrosser | yes it was to backport it | 12:18 |
| jrosser | i was thinking that we removed it because we thought nothing was installed, but actually it was the wrong repo for centos-8-stream jobs | 12:18 |
| jrosser | and that might be why nothing was installed, that the base stuff from the distro was newer than the wrong repo stuff | 12:19 |
| noonedeadpunk | So maybe we jsut backport repo removal and re-add it to nova at once? | 12:27 |
| noonedeadpunk | (and backport this) | 12:27 |
| noonedeadpunk | and clean-up of old stuff | 12:28 |
| agemuend | I think we used to need the advanced-virtualization repo also because the qemu-kvm in core had some artificial limitations, just saying | 12:35 |
| jrosser | noonedeadpunk: that sounds reasonable | 12:36 |
| opendevreview | Andrew Bonney proposed openstack/openstack-ansible stable/xena: requirements: bump zun role version to fix upgrade CI jobs https://review.opendev.org/c/openstack/openstack-ansible/+/827489 | 12:46 |
| jrosser | mnaser: did you get an answer to your radosgw namespaces thing - it sounds like this https://docs.ceph.com/en/latest/radosgw/multisite/#realms | 12:49 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-openstack_hosts master: Add data to disable the advanced-virtualisation repository https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/827495 | 13:27 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-openstack_hosts master: Add data to disable the advanced-virtualisation repository https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/827495 | 13:31 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-openstack_hosts master: Remove configuration of advanced-virtualisation repo https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/827497 | 13:31 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_nova master: Add advancedvirt-common repository https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/827501 | 13:56 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: DNM Test advancedvirt-common repo https://review.opendev.org/c/openstack/openstack-ansible/+/827485 | 13:58 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-openstack_hosts master: Add data to disable the advanced-virtualisation repository https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/827495 | 14:03 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-openstack_hosts master: Remove configuration of advanced-virtualisation repo https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/827497 | 14:03 |
| jrosser | this is ready https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/827475 | 14:04 |
| mgariepy | jrosser, if you need another quick review you can ping me, | 14:06 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_manila stable/ussuri: Remove legacy centos-8 CI jobs. https://review.opendev.org/c/openstack/openstack-ansible-os_manila/+/827470 | 14:06 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_manila stable/ussuri: Remove legacy centos-8 CI jobs. https://review.opendev.org/c/openstack/openstack-ansible-os_manila/+/827470 | 14:09 |
| rohit02 | team,cinder v2 api is deprecated in Wallaby is that right? | 14:20 |
| opendevreview | Merged openstack/openstack-ansible-tests stable/wallaby: Remove legacy centos-8 jobs https://review.opendev.org/c/openstack/openstack-ansible-tests/+/827239 | 14:21 |
| noonedeadpunk | Yep, it is | 14:24 |
| jrosser | rohit02: from the cinder docs https://docs.openstack.org/api-ref/block-storage/v2/ | 14:24 |
| noonedeadpunk | Well it was deprecated for quite a while but it's dropped now) | 14:24 |
| rohit02 | thanx team | 14:29 |
| opendevreview | Merged openstack/openstack-ansible master: [DOC] A centos deployment host does not need the RDO repo installing https://review.opendev.org/c/openstack/openstack-ansible/+/827326 | 14:37 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-openstack_hosts master: Add centos_mirror_url variable. https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/827510 | 14:40 |
| jrosser | noonedeadpunk: mgariepy this next centos-8 one is ussuri and failing everything so i dropped all the jobs https://review.opendev.org/c/openstack/openstack-ansible-os_manila/+/827470 | 14:43 |
| spatel | look like c8 is all over the place.. yike.. | 15:01 |
| opendevreview | Merged openstack/openstack-ansible-os_manila stable/ussuri: Remove legacy centos-8 CI jobs. https://review.opendev.org/c/openstack/openstack-ansible-os_manila/+/827470 | 15:15 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_manila stable/victoria: Remove legacy centos-8 CI jobs https://review.opendev.org/c/openstack/openstack-ansible-os_manila/+/827515 | 15:23 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-openstack_hosts master: Add data to disable the advanced-virtualisation repository https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/827495 | 15:46 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-openstack_hosts master: Remove configuration of advanced-virtualisation repo https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/827497 | 15:46 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-openstack_hosts master: Add centos_mirror_url variable. https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/827510 | 15:46 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: DNM Test advancedvirt-common repo https://review.opendev.org/c/openstack/openstack-ansible/+/827485 | 15:50 |
| opendevreview | Merged openstack/openstack-ansible-os_keystone stable/ussuri: Remove legacy centos-8 CI jobs https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/827475 | 15:58 |
| *** dviroel is now known as dviroel|lunch | 16:09 | |
| jrosser | noonedeadpunk: do you remember why this is abandoned? https://review.opendev.org/c/openstack/openstack-ansible/+/824028 | 16:17 |
| jrosser | becasue https://zuul.opendev.org/t/openstack/build/40b056ad3f084520b26f4d170143e800 | 16:18 |
| noonedeadpunk | Not needed iirc as ansible-lint there don't require rich yet | 16:23 |
| jrosser | something odd going on with that build i linked then :/ | 16:24 |
| noonedeadpunk | Hm, well, I think I abandoned by mistake actually.... | 16:30 |
| noonedeadpunk | as we have ansible-lint==4.3.5 set everywhere for V, and >=4.0 requires rich | 16:31 |
| noonedeadpunk | So maybe I wanted abandon for U only, but also touched V :( | 16:31 |
| jrosser | oddly the linters job fails in that patch itself | 16:34 |
| jrosser | i really wonder if i am wasting my time with the centos advancedvirt-common repo | 16:40 |
| jrosser | it looks like packages are newer in the basic OS than in the extra repo | 16:40 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-openstack_hosts master: Add data to disable the advanced-virtualisation repository https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/827495 | 16:45 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-openstack_hosts master: Remove configuration of advanced-virtualisation repo https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/827497 | 16:45 |
| *** dviroel|lunch is now known as dviroel | 17:12 | |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-openstack_hosts master: Add data to disable the advanced-virtualisation repository https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/827495 | 17:26 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-openstack_hosts master: Remove configuration of advanced-virtualisation repo https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/827497 | 17:26 |
| jrosser | noonedeadpunk: thoughts on this? it looks pretty broken on bionic and focal quite apart from the linters trouble https://review.opendev.org/c/openstack/openstack-ansible-os_manila/+/827515/ | 19:18 |
| jrosser | i don't have much experience with manila to judge if its trivial / terrible brokenness | 19:19 |
| noonedeadpunk | I can recall pushing bunch of changes to fix manila... | 19:24 |
| noonedeadpunk | I'm not sure if it was ever passing on V tbh | 19:25 |
| jrosser | until we find the next one, that V manila patch is the next puzzle piece for centos-8 removal | 19:33 |
| mgariepy | "Version 2.65 is not supported by the API. Minimum is 2.0 and maximum is 2.57." | 19:53 |
| mgariepy | how does tempest determine the version of api to use ? | 19:53 |
| mgariepy | https://zuul.opendev.org/t/openstack/build/c64dbb1a493243ff8dbe0ff7dabd0b45/log/logs/openstack/aio1-utility/tempest_run.log.txt#172 | 19:55 |
| jrosser | strange - maybe we have a wrong version of the manila tempest plugin there | 19:56 |
| jrosser | so how confident do we feel about this https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/827495 | 19:59 |
| jrosser | i cannot think of a way to test that in zuul in an upgrade job | 20:00 |
| prometheanfire | for octavia, the lbaas network is piped in like a normal client network for neutron to access right? | 20:00 |
| jrosser | as branch N-1 is needing the same patch | 20:00 |
| jrosser | prometheanfire: from spatel is this https://satishdotpatel.github.io/openstack-ansible-octavia/ | 20:02 |
| spatel | This is how i am running octavia in Production (much easier and less complex) | 20:03 |
| prometheanfire | that's close to what I'm aiming for, using networkd for the host bridge (which is on a vlan between hosts) | 20:15 |
| prometheanfire | and veth pairs will be networkd controlled, like the normal client networks | 20:17 |
| jrosser | you can do whatever you like so long as the right tag on br-vlan on the compute node ends up untagged on br-lbaas on the controller | 20:17 |
| jrosser | noonedeadpunk: https://github.com/ansible/ansible/blob/devel/lib/ansible/module_utils/common/respawn.py#L18-L32 | 20:45 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Remove symlinking of selinux libraries into the ansible-runtime venv https://review.opendev.org/c/openstack/openstack-ansible/+/827556 | 21:05 |
| *** dviroel is now known as dviroel|out | 21:22 | |
| opendevreview | Merged openstack/openstack-ansible-os_manila master: Remove legacy policy.json cleanup handler https://review.opendev.org/c/openstack/openstack-ansible-os_manila/+/827445 | 21:24 |
| gouthamr | mgariepy jrosser: snooping here, tempest doesn't automatically do this; it relies on configuration - manila-tempest-plugin is branchless, so the latest release can be used to test an old version of manila as long as you can satisfy the plugin's own requirements | 21:25 |
| gouthamr | the tempest config opt you'll need here is this one: https://github.com/openstack/manila-tempest-plugin/blob/73e0b6fdc450b7539bf6a97349f5774177ffb24a/manila_tempest_tests/config.py#L42-L45 | 21:26 |
| gouthamr | you'll need to set this to whatever the max version supported by the cloud is (the error message tells us this is 2.57) | 21:27 |
| jrosser | is that in the [share] section of the tempest config file? | 21:28 |
| gouthamr | if this is a set and forget, i recommend grabbing the version from this version history doc: https://docs.openstack.org/manila/latest/contributor/api_microversion_history.html | 21:28 |
| gouthamr | yes | 21:28 |
| gouthamr | jrosser: tempestconf does this for automation: : https://github.com/openstack-archive/python-tempestconf/blob/505e20a4dd771536257a4c4db645da707f727668/config_tempest/services/share.py#L35 | 21:28 |
| jrosser | unfortunately tempestconf has never properly understood internal endpoint=http and external endpoint=https/self-signed | 21:29 |
| gouthamr | grr - that's the old repo, https://opendev.org/openinfra/python-tempestconf/src/commit/d630e489aa36aefa17c4ac9633af07694b825bcd/config_tempest/services/share.py#L41-L44 | 21:29 |
| jrosser | afaik the more development oriented stuff in devstack is not set up like that | 21:29 |
| gouthamr | jrosser: yeah; on devstack, we do this: https://github.com/openstack/manila/blob/master/devstack/plugin.sh#L968-L977 | 21:30 |
| jrosser | thats really useful - we can certainly fix this for our tests | 21:31 |
| admin1 | just checking here .. if there is a command that shows all ongoing migrations from all projects .. | 21:32 |
| jrosser | gouthamr: we have a dict here which fills out the [share] section of tempest.conf https://github.com/openstack/openstack-ansible/blob/master/tests/roles/bootstrap-host/templates/user_variables_manila.yml.j2#L38-L55 | 21:34 |
| jrosser | it should be very simple to get the microversion in there as well | 21:34 |
| gouthamr | jrosser++ | 21:37 |
| gouthamr | yep | 21:37 |
| jrosser | gouthamr: really appreciate the input btw - details like that are super cool to get :) | 21:39 |
| gouthamr | jrosser: you're welcome :) | 21:41 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Set minimum and maximum microversions for manila api https://review.opendev.org/c/openstack/openstack-ansible/+/827560 | 21:53 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!