Monday, 2024-08-05

noonedeadpunkmornings06:58
noonedeadpunkf0o: yeah, it should be master as it's been overriden in group_vars here: https://opendev.org/openstack/openstack-ansible/src/branch/stable/2024.1/inventory/group_vars/cinder_all/source_git.yml#L2006:59
noonedeadpunkwe need to land https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/925571 to unblock integrated repo (tls jobs)07:07
f0onoonedeadpunk: ty!07:29
jrossermorning07:29
opendevreviewMerged openstack/openstack-ansible-tests master: Remove the get_md5 parameter from ansible stat tasks  https://review.opendev.org/c/openstack/openstack-ansible-tests/+/92556508:10
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_keystone master: Ensure correct order for credential rotate/migrate  https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/92567309:13
opendevreviewDmitriy Rabotyagov proposed openstack/ansible-role-uwsgi stable/2023.2: Ensure uWSGI is built with pcre support  https://review.opendev.org/c/openstack/ansible-role-uwsgi/+/92568210:00
opendevreviewDmitriy Rabotyagov proposed openstack/ansible-role-uwsgi stable/2023.1: Ensure uWSGI is built with pcre support  https://review.opendev.org/c/openstack/ansible-role-uwsgi/+/92568310:00
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-os_keystone master: Use platform dependant sshd service name in restart handler  https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/92471911:20
opendevreviewJonathan Rosser proposed openstack/openstack-ansible master: Permit Ubuntu Noble for deploy host and targets in requirements checks  https://review.opendev.org/c/openstack/openstack-ansible/+/92431111:20
opendevreviewJonathan Rosser proposed openstack/openstack-ansible master: Use the 'generated' apparmor profile for all containers  https://review.opendev.org/c/openstack/openstack-ansible/+/92466111:21
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-os_magnum master: Add test for high-availability mcapi control plane  https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/92317411:21
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-os_magnum master: Add test for high availability k8s control plane on ubuntu noble  https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/92470211:22
jrosserdo we need to backport this? https://review.opendev.org/c/openstack/openstack-ansible-tests/+/92556511:49
jrosserzuul ansible version is the same regardless of branch i expect?11:49
noonedeadpunkyeah, we need12:04
opendevreviewMerged openstack/openstack-ansible-os_cinder stable/2024.1: Disable heartbeat_in_pthread by default  https://review.opendev.org/c/openstack/openstack-ansible-os_cinder/+/92483112:23
gokhanhello folks, after upgrade from victoria to antelope, I run openstack loadbalancer failover <idoflb> and then there was a problem with creating vms and loadbalancers stuck in pending update. I can't failover again because of pending update status. what can be done in these situations?  12:30
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-tests stable/2024.1: Remove the get_md5 parameter from ansible stat tasks  https://review.opendev.org/c/openstack/openstack-ansible-tests/+/92569312:41
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-tests stable/2023.2: Remove the get_md5 parameter from ansible stat tasks  https://review.opendev.org/c/openstack/openstack-ansible-tests/+/92569412:42
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-tests stable/2023.1: Remove the get_md5 parameter from ansible stat tasks  https://review.opendev.org/c/openstack/openstack-ansible-tests/+/92569512:42
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-tests unmaintained/zed: Remove the get_md5 parameter from ansible stat tasks  https://review.opendev.org/c/openstack/openstack-ansible-tests/+/92569612:43
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-tests unmaintained/yoga: Remove the get_md5 parameter from ansible stat tasks  https://review.opendev.org/c/openstack/openstack-ansible-tests/+/92569712:43
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-tests unmaintained/xena: Remove the get_md5 parameter from ansible stat tasks  https://review.opendev.org/c/openstack/openstack-ansible-tests/+/92569812:44
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-tests unmaintained/wallaby: Remove the get_md5 parameter from ansible stat tasks  https://review.opendev.org/c/openstack/openstack-ansible-tests/+/92570012:44
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-tests unmaintained/zed: Remove legacy centos-8-stream job definition  https://review.opendev.org/c/openstack/openstack-ansible-tests/+/92570112:47
jrossernoonedeadpunk: we have two .gitreview to merge for openstack-ansible-tests on xena and wallaby - should we force merge these as we can't use git-review without them12:58
noonedeadpunkwell, technically there's a way, but yeah, would make sense to12:59
jrosserit's https://review.opendev.org/c/openstack/openstack-ansible-tests/+/91312713:03
jrosserand https://review.opendev.org/c/openstack/openstack-ansible-tests/+/91318313:03
jrosserfrickler: would you be able to help with these ^ (fixing a couple of zuul errors and unblocking ability to submit reviews on those branches)13:06
fricklerjrosser: the first one depends-on https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/91627713:24
fricklerit does seem to be passing otherwise13:25
jrosseri just rechecked the depends-on to get some results again13:26
jrosserbut thats looks spectacularly broken13:26
fricklerI can merge the second one, maybe though you want to amend the commit message to mention the zuul cleanup?13:26
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-tests unmaintained/xena: Update .gitreview for unmaintained/xena  https://review.opendev.org/c/openstack/openstack-ansible-tests/+/91318313:27
jrosser^ sure13:27
opendevreviewMerged openstack/openstack-ansible-tests unmaintained/xena: Update .gitreview for unmaintained/xena  https://review.opendev.org/c/openstack/openstack-ansible-tests/+/91318313:31
fricklerand done13:31
jrosserexcellent thanks13:32
opendevreviewMerged openstack/openstack-ansible-repo_server master: Ensure default SSL config is removed for EL  https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/92557113:32
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-tests unmaintained/xena: Remove the get_md5 parameter from ansible stat tasks  https://review.opendev.org/c/openstack/openstack-ansible-tests/+/92569813:33
NeilHanlono/ quick note that I'll probably be absent tomorrow--driving out to Fedora's Flock conference in Rochester NY13:46
noonedeadpunkhave a safe drive!13:53
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-plugins master: Ensure haproxy_service_config targets right host group  https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/92434114:10
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-tests unmaintained/wallaby: Update .gitreview for unmaintained/wallaby  https://review.opendev.org/c/openstack/openstack-ansible-tests/+/91312714:16
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-tests unmaintained/yoga: Update .gitreview for unmaintained/yoga  https://review.opendev.org/c/openstack/openstack-ansible-tests/+/90849714:35
opendevreviewMerged openstack/openstack-ansible-tests stable/2023.1: Remove the get_md5 parameter from ansible stat tasks  https://review.opendev.org/c/openstack/openstack-ansible-tests/+/92569514:39
opendevreviewMerged openstack/openstack-ansible-tests stable/2023.2: Remove the get_md5 parameter from ansible stat tasks  https://review.opendev.org/c/openstack/openstack-ansible-tests/+/92569414:43
opendevreviewMerged openstack/openstack-ansible-tests stable/2024.1: Remove the get_md5 parameter from ansible stat tasks  https://review.opendev.org/c/openstack/openstack-ansible-tests/+/92569314:43
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-tests unmaintained/yoga: Update .gitreview for unmaintained/yoga  https://review.opendev.org/c/openstack/openstack-ansible-tests/+/90849714:44
opendevreviewMerged openstack/openstack-ansible-tests unmaintained/zed: Remove legacy centos-8-stream job definition  https://review.opendev.org/c/openstack/openstack-ansible-tests/+/92570114:46
jrossernoonedeadpunk: frickler similar .gitreview patch here https://review.opendev.org/c/openstack/openstack-ansible-tests/+/908497 - except this one is also using a job from wallaby and blocking that too15:05
fricklerjrosser: merging that now. any idea yet what to do with https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/916277 ? 15:42
opendevreviewMerged openstack/openstack-ansible-tests unmaintained/yoga: Update .gitreview for unmaintained/yoga  https://review.opendev.org/c/openstack/openstack-ansible-tests/+/90849715:42
jrosserthat was a circular dependancy, it was reading the branch to use from .gitreview so was completely broken15:42
jrosserthis would be way way easier in a monorepo really15:43
jrosser*from .gitreview of the openstack-ansible-tests repo15:43
frickler.gitreview should have no effect on what zuul does15:45
jrosserout test script in openstack-ansible-tests needs to know which branch it is on15:46
jrosserin a way that works both inside and outside of zuul15:46
jrosserso you end up here https://github.com/openstack/openstack-ansible-tests/blob/master/test-ansible-env-prep.sh#L5415:48
fricklerouch, that seems to be a wild hack. but ok, then we'll just need to force merge https://review.opendev.org/c/openstack/openstack-ansible-tests/+/913127 once it has some +2s I guess16:00
jrosserthat one might be ok with a recheck now - the yoga zuul error should be fixed16:01
opendevreviewMerged openstack/openstack-ansible-plugins master: Updated from OpenStack Ansible Tests  https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/92499516:01
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-tests unmaintained/zed: Remove use of debian-buster job  https://review.opendev.org/c/openstack/openstack-ansible-tests/+/92571916:15
opendevreviewMerged openstack/openstack-ansible-os_cinder stable/2023.2: Disable heartbeat_in_pthread by default  https://review.opendev.org/c/openstack/openstack-ansible-os_cinder/+/92483216:34
opendevreviewMerged openstack/ansible-config_template master: Bump version for bugfix release  https://review.opendev.org/c/openstack/ansible-config_template/+/92499817:55
opendevreviewMerged openstack/openstack-ansible-os_skyline stable/2024.1: Updated from OpenStack Ansible Tests  https://review.opendev.org/c/openstack/openstack-ansible-os_skyline/+/92500418:05
opendevreviewMerged openstack/openstack-ansible-os_neutron master: Correct 'neutron-policy-override' tag  https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/92468118:33
opendevreviewMerged openstack/openstack-ansible master: Improve regex matching for infra_lxc_validate job  https://review.opendev.org/c/openstack/openstack-ansible/+/92415518:50
opendevreviewJonathan Rosser proposed openstack/openstack-ansible master: Test on Ubuntu Noble  https://review.opendev.org/c/openstack/openstack-ansible/+/92434218:53
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: [doc] Add support for Ubuntu 24.04 to docs  https://review.opendev.org/c/openstack/openstack-ansible/+/92482918:53
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-os_magnum master: Define lock directory for oslo_concurrency  https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/92169019:02
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_magnum master: Move insecure param to keystone_auth section  https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/90511019:03
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-plugins master: Combine vars files for debian/ubuntu and ensure setuptools is present  https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/92469219:10
opendevreviewMerged openstack/openstack-ansible master: Delete glusterd data along with container data  https://review.opendev.org/c/openstack/openstack-ansible/+/92491020:39
opendevreviewMerged openstack/openstack-ansible-os_keystone master: Use platform dependant sshd service name in restart handler  https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/92471920:48

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!