| opendevreview | Merged openstack/openstack-ansible master: Remove test_volumes_backup tests https://review.opendev.org/c/openstack/openstack-ansible/+/825165 | 00:15 |
|---|---|---|
| opendevreview | Merged openstack/openstack-ansible master: Remove tempest.api.volume.admin.test_multi_backend test https://review.opendev.org/c/openstack/openstack-ansible/+/825166 | 00:15 |
| *** dviroel|rover|afk is now known as dviroel|out | 00:16 | |
| *** ysandeep|out is now known as ysandeep | 02:12 | |
| *** ysandeep is now known as ysandeep|afk | 04:18 | |
| *** ysandeep|afk is now known as ysandeep | 06:13 | |
| *** ysandeep is now known as ysandeep|brb | 07:25 | |
| *** ysandeep|brb is now known as ysandeep | 07:30 | |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Use glusterfs to synchronise repo server contents https://review.opendev.org/c/openstack/openstack-ansible/+/837589 | 07:48 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Bump ansible-core version to 2.12.5 https://review.opendev.org/c/openstack/openstack-ansible/+/839312 | 07:51 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Use glusterfs to synchronise repo server contents https://review.opendev.org/c/openstack/openstack-ansible/+/837589 | 09:14 |
| damiandabrowski[m] | hmmm do You have any idea what happened here? https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/825164/9 | 09:53 |
| damiandabrowski[m] | https://paste.openstack.org/raw/bG3RDKhuTg1tBf1Uw8eI/ | 09:53 |
| damiandabrowski[m] | I can checkout 59a3cb190f7913e32b044dc2733cfe7e46c4c99a without any issues | 09:54 |
| opendevreview | Siavash Sardari proposed openstack/openstack-ansible-haproxy_server master: set haproxy repository version. https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/839351 | 10:07 |
| jrosser | damiandabrowski[m]: is it the upgrade job you're asking about? | 10:11 |
| damiandabrowski[m] | yeah | 10:11 |
| jrosser | this will fix it | 10:12 |
| jrosser | https://review.opendev.org/c/openstack/openstack-ansible/+/838762 | 10:12 |
| jrosser | what seems to happen is we pick the most recent SHA for things like ansible-sshd and ceph-ansible and put them in ansible-role-requirements.yml | 10:12 |
| jrosser | then $something happens to those repos, maybe a force push or a wierd merge commit and that SHA is no longer usable for cloning | 10:13 |
| jrosser | recently ansible-sshd pushed a whole pile of changes and that broke the SHA we used to use | 10:13 |
| damiandabrowski[m] | ahh thanks, but I assume it was only temporary, as I can perform `git checkout 59a3cb190f7913e32b044dc2733cfe7e46c4c99a` locally without any issues, right? | 10:19 |
| damiandabrowski[m] | anyway, I'll trigger recheck, thanks! | 10:19 |
| jrosser | damiandabrowski[m]: yes, this seems to be something to do with how the ansible git module works | 10:20 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-repo_server master: Clean up legacy lsycnd, rsync and ssh key config https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/837859 | 10:35 |
| *** ysandeep is now known as ysandeep|coffee | 10:37 | |
| *** dviroel|out is now known as dviroel|rover | 11:09 | |
| *** ysandeep|coffee is now known as ysandeep | 11:28 | |
| opendevreview | Siavash Sardari proposed openstack/openstack-ansible-os_octavia master: Add flexability for octavia cinder variable. https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/839373 | 13:00 |
| opendevreview | Merged openstack/openstack-ansible-os_cinder master: Setup db/mq/service during scheduler run https://review.opendev.org/c/openstack/openstack-ansible-os_cinder/+/824553 | 13:09 |
| opendevreview | Merged openstack/openstack-ansible-lxc_hosts master: systemd-resolved package is not available for redhat https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/835594 | 13:11 |
| opendevreview | Merged openstack/openstack-ansible-os_nova master: Implement nova direct RBD image retrieve https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/828897 | 13:14 |
| mgariepy | we should really cleanup the review queue at some points. there are quite a lot of old patches there.. | 13:30 |
| jrosser | yes, it's been all a bit wedged up for the last week with ceph and upgrade jobs broken | 13:37 |
| jrosser | really needs daily attention to try to free things up | 13:38 |
| opendevreview | Siavash Sardari proposed openstack/openstack-ansible-os_octavia master: Add support for system scope openrc https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/839381 | 13:39 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_keystone master: Handle host with unset ansible_host https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/804206 | 13:39 |
| mgariepy | hmm. old 2020..2017 patches.. .. | 13:40 |
| opendevreview | Merged openstack/openstack-ansible-os_neutron master: Do not deploy api-paste for CentOS distro deployment https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/838729 | 13:44 |
| opendevreview | Merged openstack/openstack-ansible-os_keystone master: Migrate ssl certificate generation to the PKI role https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/830179 | 14:32 |
| *** ysandeep is now known as ysandeep|afk | 14:36 | |
| opendevreview | Merged openstack/openstack-ansible-os_keystone master: Tidy IDP setup task files https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/830260 | 14:44 |
| *** ysandeep|afk is now known as ysandeep | 14:59 | |
| noonedeadpunk | #startmeeting openstack_ansible_meeting | 15:05 |
| opendevmeet | Meeting started Tue Apr 26 15:05:49 2022 UTC and is due to finish in 60 minutes. The chair is noonedeadpunk. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:05 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:05 |
| opendevmeet | The meeting name has been set to 'openstack_ansible_meeting' | 15:05 |
| noonedeadpunk | #topic rollcall | 15:05 |
| mgariepy | hey | 15:06 |
| noonedeadpunk | hey | 15:06 |
| jamesdenton | hi | 15:06 |
| noonedeadpunk | sorry, I tried to run from my phone first but seems it doesn't work :( | 15:06 |
| noonedeadpunk | or I connected to freenode there..... | 15:07 |
| noonedeadpunk | anyway) | 15:08 |
| damiandabrowski[m] | hey! | 15:09 |
| noonedeadpunk | #topic office hours | 15:10 |
| noonedeadpunk | first of all I want to congrat damiandabrowski[m] in our core team! | 15:11 |
| jrosser | o/ hello | 15:11 |
| noonedeadpunk | I just added him to the group, haven't written ML yet :) | 15:11 |
| jamesdenton | congrats! | 15:11 |
| mgariepy | congrats damiandabrowski[m] ! :D | 15:11 |
| NeilHanlon | congrats damiandabrowski[m]! | 15:12 |
| damiandabrowski[m] | thanks guys! | 15:12 |
| noonedeadpunk | Then just wanted to say - I'm not near IRC this week so can't really check on things :( | 15:12 |
| noonedeadpunk | but since we have one more reviewer now we likely covered :p | 15:12 |
| opendevreview | Merged openstack/openstack-ansible-haproxy_server master: Add functionality to accept both HTTP and HTTPS during upgrade https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/829899 | 15:13 |
| mgariepy | damiandabrowski[m], so you only need to cover for noonedeadpunk work this week ;p no pressure ! :P | 15:13 |
| noonedeadpunk | yeah, exactly :D | 15:13 |
| damiandabrowski[m] | hahha :D I just want to kindly ask for few more reviews in my tempest patches https://review.opendev.org/q/topic:tempest-damian-2021-12+AND+-status:merged | 15:15 |
| NeilHanlon | i'm always happy to provide a blind +1 ;) | 15:17 |
| NeilHanlon | will take a look at those today out of interest damiandabrowski[m] | 15:17 |
| damiandabrowski[m] | thanks! | 15:18 |
| jrosser | damiandabrowski[m]: i did look but i think there is a *lot* of rebasing needed? it was quite confusing | 15:18 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_octavia master: Use PKI role for certificate generation https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/839068 | 15:19 |
| damiandabrowski[m] | yeah the plan was to gather initial reviews -> apply corrections(if required) -> merge | 15:19 |
| damiandabrowski[m] | i planned to do it like this to avoid unnecessary zuul jobs, but i can rebase them now if You think it's better idea | 15:20 |
| noonedeadpunk | damiandabrowski[m]: I think you can do rebase now... | 15:23 |
| noonedeadpunk | It was a plan to wait for a week and 2 passed till now ;) | 15:23 |
| noonedeadpunk | To have that said, I pushed WIP patch regarding Octavia using PKI role | 15:24 |
| noonedeadpunk | https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/839068 | 15:24 |
| jrosser | we have the upgrade case to handle yet? | 15:25 |
| noonedeadpunk | It doesn't have upgrade path and my best thought to handle upgrade is to create a playbook to integrated repo | 15:25 |
| noonedeadpunk | yeah | 15:25 |
| noonedeadpunk | As I actually don't see better way then just to rename/move existing certs on the octavia_cert_setup_host | 15:26 |
| noonedeadpunk | Btw have we checked 22.04 availability in CI? | 15:27 |
| noonedeadpunk | Seems it's present?:) https://opendev.org/openstack/project-config/src/branch/master/nodepool/nodepool.yaml#L42 | 15:28 |
| noonedeadpunk | we should try adding this out:) | 15:30 |
| noonedeadpunk | anything I missed and requires attention? | 15:31 |
| mgariepy | should we try to clenaup the review queue ? there is patches there since 2017.. | 15:32 |
| noonedeadpunk | I believe some of them could be still valid though.... | 15:32 |
| noonedeadpunk | I can totally recall one that I'm not dropping on purpose not to forget to revise one day - about how we're handling limiting across hosts for nova/cinder | 15:33 |
| mgariepy | yeah some of them are probably still good but there are a couple of them that i suspect not needed anymore.. | 15:35 |
| mgariepy | like one of compatibility matrix update.. from 2018.. | 15:36 |
| noonedeadpunk | Yeah..... | 15:37 |
| noonedeadpunk | well.... | 15:37 |
| mgariepy | we have about 240 patches on master.. | 15:37 |
| mgariepy | but yeah we should try to clean up the queue a bit/ reorganize them | 15:37 |
| jrosser | I think that we should use the dashboard more effectively | 15:38 |
| jrosser | working on fixing the things that need reverify, actively reviewing the ones with +1 | 15:39 |
| noonedeadpunk | Do you see any changes to it? | 15:39 |
| jrosser | i think i mean that the information is pretty clear in the dashboard as it is | 15:39 |
| mgariepy | not all the patches show up there tho. | 15:40 |
| jrosser | but without actively working on the patches to get them moved through the stages and merged, they will just pile up | 15:40 |
| mgariepy | i think | 15:40 |
| noonedeadpunk | mgariepy: I _think_ this should be arranged now when I parented all projects under openstack-ansible group | 15:42 |
| opendevreview | Merged openstack/openstack-ansible-os_tempest master: do not include [*-feature-enabled] sections in tempest.conf https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/825164 | 15:42 |
| noonedeadpunk | But if you found any patch missing from it - just ping me and we will fix that :) | 15:42 |
| mgariepy | https://review.opendev.org/q/parentproject:openstack/openstack-ansible+branch:master+status:open | 15:43 |
| mgariepy | 10 pages. | 15:44 |
| jrosser | we miss the ones which are +2+W but blocked? | 15:44 |
| mgariepy | ho. | 15:44 |
| jrosser | mgariepy: you can click the headings in the dashboards and get the full lists for each category? | 15:45 |
| mgariepy | not sure. | 15:46 |
| noonedeadpunk | I bet it's possible.... | 15:46 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-galera_server stable/xena: Use separate tmp directory https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/839338 | 15:47 |
| noonedeadpunk | jrosser: you mean we miss a category for that? | 15:48 |
| noonedeadpunk | basically having +V+2+W ? | 15:48 |
| jrosser | i think there is an "approved but cannot merge" category | 15:48 |
| jrosser | like if there is an outstanding depends-on | 15:49 |
| noonedeadpunk | iirc there was no such catregory | 15:49 |
| noonedeadpunk | But I think I can try to add one | 15:50 |
| jrosser | i do remember we mentioned this before, maybe there was also a zuul mergability flag that was disabled, and may now be returned | 15:50 |
| noonedeadpunk | Oh, yes, we likely discussed that but I never happen to follow up on that... | 15:51 |
| opendevreview | Merged openstack/openstack-ansible master: In AIO and CI set the internal vip to https only for 'deploy' actions. https://review.opendev.org/c/openstack/openstack-ansible/+/830431 | 15:51 |
| noonedeadpunk | Basically I still haven't added queues... | 15:51 |
| jrosser | hmm | 15:51 |
| noonedeadpunk | But I kind of tend to add queues after branching at least to experiment when we're done with main bunch of things we want to land before experimenting | 15:52 |
| noonedeadpunk | Regarding adding such category to dashboard - I think it's usefull and will write down to take a look there one more time | 15:53 |
| jrosser | i am just working on upgrade path for lsyncd -> glusterfs | 15:54 |
| mgariepy | yay no more lsyncd soon !:D | 15:56 |
| noonedeadpunk | hurray! really great to hear! | 16:01 |
| noonedeadpunk | I will continue looking in octavia and stuff only next week :( | 16:01 |
| noonedeadpunk | #endmeeting | 16:01 |
| opendevmeet | Meeting ended Tue Apr 26 16:01:58 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:01 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/openstack_ansible_meeting/2022/openstack_ansible_meeting.2022-04-26-15.05.html | 16:01 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/openstack_ansible_meeting/2022/openstack_ansible_meeting.2022-04-26-15.05.txt | 16:01 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/openstack_ansible_meeting/2022/openstack_ansible_meeting.2022-04-26-15.05.log.html | 16:01 |
| jrosser | do we currently support a case with multiple repo servers used for wheel builds | 16:03 |
| jrosser | i.e multiple architecture or OS? | 16:04 |
| mgariepy | it used to work-ish mostly when upgrading distro on some release. | 16:04 |
| jrosser | i am needing to decide if on each repo server i copy the contents of /var/www into the new filesystem | 16:04 |
| mgariepy | like xenial > focal upgrade | 16:04 |
| jrosser | or if i do it on only one | 16:05 |
| jrosser | during upgrade | 16:05 |
| damiandabrowski[m] | when it comes to multiple OS, i tested it during ubutnu 18.04->20.04 upgrade | 16:05 |
| damiandabrowski[m] | and it's quite messy :/ i have created a bug report recently, let me find it | 16:05 |
| mgariepy | oops. xenial > bionic > focal ;) haha | 16:05 |
| damiandabrowski[m] | https://bugs.launchpad.net/openstack-ansible/+bug/1964535 | 16:05 |
| damiandabrowski[m] | but regarding to copying contents of /var/www, i think that even You have multi-distro envrionment, all repo servers should still have the same content, right? | 16:08 |
| jrosser | damiandabrowski[m]: well i don't know, lsyncd only runs on one of them | 16:09 |
| jrosser | so currently i feel we have kind of asymmetric distribution of wheels | 16:10 |
| damiandabrowski[m] | ouh, You're right. From this perspective i think we can't say we "support" multi-distro envs | 16:10 |
| jrosser | particularly if you have 3 x86_64 controllers and an addtional arm64 wheel build host | 16:10 |
| damiandabrowski[m] | however, gluster should make things a bit better :D | 16:11 |
| jrosser | oh absolutely yes | 16:11 |
| jrosser | all of this trouble should go away | 16:11 |
| *** dviroel|rover is now known as dviroel|rover|lunch | 16:11 | |
| *** ysandeep is now known as ysandeep|out | 16:17 | |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-repo_server master: Clean up legacy lsycnd, rsync and ssh key config https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/837859 | 16:33 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-repo_server master: Add upgrade path from lsyncd to shared filesystem. https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/839411 | 16:33 |
| damiandabrowski[m] | rebasing tempest patches | 16:40 |
| opendevreview | Damian Dąbrowski proposed openstack/openstack-ansible-os_tempest master: Allow to create only specific tempest resources. https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/803477 | 16:40 |
| opendevreview | Damian Dąbrowski proposed openstack/openstack-ansible-os_tempest master: Remove unused variables https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/825405 | 16:40 |
| opendevreview | Damian Dąbrowski proposed openstack/openstack-ansible-os_tempest master: Do not store unnecessary sections in tempest.conf https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/825407 | 16:40 |
| opendevreview | Damian Dąbrowski proposed openstack/openstack-ansible-os_tempest master: Fix hardcoded instance_type in [heat_plugin] section https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/825408 | 16:41 |
| opendevreview | Damian Dąbrowski proposed openstack/openstack-ansible-os_tempest master: Improve domain support https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/836833 | 16:41 |
| opendevreview | Damian Dąbrowski proposed openstack/openstack-ansible-os_tempest master: Allow to create only specific tempest resources. https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/803477 | 16:44 |
| opendevreview | Damian Dąbrowski proposed openstack/openstack-ansible-os_tempest master: Fix hardcoded flavor_ref and flavor_ref_alt https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/803492 | 16:44 |
| opendevreview | Damian Dąbrowski proposed openstack/openstack-ansible-os_tempest master: Add support for both Credential Provider Mechanisms https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/825403 | 16:45 |
| opendevreview | Damian Dąbrowski proposed openstack/openstack-ansible-os_tempest master: Remove unused variables https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/825405 | 16:45 |
| opendevreview | Damian Dąbrowski proposed openstack/openstack-ansible-os_tempest master: Do not store unnecessary sections in tempest.conf https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/825407 | 16:45 |
| opendevreview | Damian Dąbrowski proposed openstack/openstack-ansible-os_tempest master: Fix hardcoded instance_type in [heat_plugin] section https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/825408 | 16:45 |
| opendevreview | Damian Dąbrowski proposed openstack/openstack-ansible-os_tempest master: Improve domain support https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/836833 | 16:45 |
| *** dviroel|rover|lunch is now known as dviroel|rover | 17:02 | |
| mgariepy | noonedeadpunk, jrosser the patch all appears when i expand the 14 days w/o feedback | 17:15 |
| *** dviroel|rover is now known as dviroel|rover|out | 22:56 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!