Thursday, 2022-12-01

*** dviroel|afk is now known as dviroel00:08
*** dviroel is now known as dviroel|out01:33
opendevreviewJames Denton proposed openstack/openstack-ansible-os_neutron master: Install openvswitch packages in Neutron server container  https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/86624904:24
opendevreviewJames Denton proposed openstack/openstack-ansible master: Implement OVN inventory changes and deploy by default  https://review.opendev.org/c/openstack/openstack-ansible/+/86292404:25
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-tests master: Switch CentOS 8 Stream to 9 Stream  https://review.opendev.org/c/openstack/openstack-ansible-tests/+/86619709:20
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-plugins master: Allow to define condition for DB creation  https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/86605910:36
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_octavia master: Add coordination to octavia  https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/86605810:39
*** dviroel|out is now known as dviroel10:43
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Run zookeeper installation for validate job  https://review.opendev.org/c/openstack/openstack-ansible/+/86604710:46
opendevreviewMerged openstack/openstack-ansible-os_neutron master: add ovn ssl config  https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/86240311:23
opendevreviewJames Denton proposed openstack/openstack-ansible-os_neutron master: Install openvswitch packages in Neutron server container  https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/86624911:30
noonedeadpunkjamesdenton: ^ sounds like not doing what it should12:17
noonedeadpunkor doing more then it should12:18
noonedeadpunkI was jsut trying to build up aio to check12:18
noonedeadpunkas jobs reliably failing with it12:18
jrosserit probably started a bunch of services we may not want?12:25
noonedeadpunkI'm quite sure it did actually13:07
noonedeadpunkI'm trying without that patch to see what it needs and then see what we should actually do13:07
noonedeadpunkI was actually a bit distracted by https://review.opendev.org/c/openstack/openstack-ansible/+/752311 to see if we gain any performance from building wheels. And then again distracted by ara failing on our ssh plugin...13:09
noonedeadpunkAnd that we run https://opendev.org/openstack/openstack-ansible/src/branch/master/scripts/scripts-library.sh#L249-L257 after each step in gate-check-commit13:09
noonedeadpunkinterestingly, that tempest fails on teardown only13:13
*** frenzy_friday|rover is now known as frenzy_friday|rover|lunch13:33
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_neutron master: Install openvswitch packages in Neutron server container  https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/86624913:36
noonedeadpunkI guess that should do it ^13:37
noonedeadpunkjrosser: do you think I should revert this https://review.opendev.org/c/openstack/openstack-ansible/+/866047/3/playbooks/zookeeper-install.yml ? did mainly to trigger validate job, kind of don't want to waste CI resources but know that changing copyright is wrong...13:40
noonedeadpunkSo not sure 13:40
jrosserit is a bit odd to change the copyright13:44
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Define apmhora provider for tempest config  https://review.opendev.org/c/openstack/openstack-ansible/+/86633513:49
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Define apmhora provider for tempest config  https://review.opendev.org/c/openstack/openstack-ansible/+/86633513:49
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_octavia master: Add coordination to octavia  https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/86605813:49
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Run zookeeper installation for validate job  https://review.opendev.org/c/openstack/openstack-ansible/+/86604713:53
*** frenzy_friday|rover|lunch is now known as frenzy_friday|rover14:01
noonedeadpunkdamiandabrowski: can you kindly also vote on https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/86556614:04
jamesdentonnoonedeadpunk yes, i kicked off a local job, too.14:08
jamesdentonahh, perhaps14:09
jamesdentonthanks14:09
noonedeadpunkwe really need to land 865566 to unblock master jobs14:13
jamesdentondone14:18
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_ironic master: Update variables for switchport introspection during inspection  https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/86443714:18
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_ironic master: Add variable for user defined list of deploy images  https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/86531014:19
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_ironic master: Add variable for user defined list of deploy images  https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/86531014:20
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_ironic master: Ensure nginx config is present for ironic inspector  https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/86440614:22
opendevreviewMerged openstack/openstack-ansible-os_octavia master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/85765414:54
opendevreviewMerged openstack/openstack-ansible-os_designate master: Fix race condition during designate setup  https://review.opendev.org/c/openstack/openstack-ansible-os_designate/+/86570114:56
jrosseri found another bug with the parallel-git-clone module15:01
jrosser128 core machine is toooooo many threads and it breaks15:01
opendevreviewMerged openstack/openstack-ansible-os_octavia master: Enable /healthcheck for Octavia API  https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/86441915:06
opendevreviewMerged openstack/openstack-ansible-os_horizon master: Install ironic-ui from git source rather than a pip package  https://review.opendev.org/c/openstack/openstack-ansible-os_horizon/+/86392115:29
noonedeadpunkjrosser: lol15:34
noonedeadpunkcan we pass threads to the module?15:34
jrosserit takes cores * some multiplier15:35
noonedeadpunkor well, we can max out to 16 or smth reasonable15:35
jrosserbut it needs a max15:35
noonedeadpunk128core machine I assume that's arm?15:35
jrosserit is15:35
jrosseri have an AIO in an arm vm but i can't boot cirros on it15:36
jrosserso i try the same on bare metal too15:36
jrosseri am not too sure how to debug it tbh15:36
jrosseralso what is happening with interface names these days15:40
jrosserenP1p1s0f1np115:40
jrosserreally?!15:40
jamesdentondude15:40
jamesdentonit's out of hand15:40
jrosseroh yes, i'll just rememeber that and type it in somewhere - i don't think so :(15:41
opendevreviewJames Denton proposed openstack/openstack-ansible-os_neutron master: Install openvswitch packages in Neutron server container  https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/86624915:43
jamesdentona name only a machine can love15:43
opendevreviewMerged openstack/openstack-ansible master: Explicitly define neutron_plugin_base for OVS  https://review.opendev.org/c/openstack/openstack-ansible/+/86601215:53
opendevreviewMerged openstack/openstack-ansible-os_ironic stable/yoga: Pass CA pass to Swift CLI  https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/86556615:57
noonedeadpunkjamesdenton: why you said that ovs logic is broken?16:01
noonedeadpunkah16:01
noonedeadpunkI was checking old zuul logs16:01
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Disable sahara real tempest tests  https://review.opendev.org/c/openstack/openstack-ansible/+/86612416:34
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_ironic master: Enable /healtcheck for Ironic APIs  https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/86442316:34
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Define apmhora provider for tempest config  https://review.opendev.org/c/openstack/openstack-ansible/+/86633516:38
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-haproxy_server master: Fix warnings in haproxy config  https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/86478616:41
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Switch distro jobs to Jammy  https://review.opendev.org/c/openstack/openstack-ansible/+/86612616:52
noonedeadpunkso....16:53
noonedeadpunkbumping apache limits didn't help keystone16:53
noonedeadpunkwe still get plenty of failures on tempesting keystone16:53
noonedeadpunkI kind of woner if we _really_ want to run full identity api tests...16:55
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-os_tempest master: Update cirros image to 0.6.1  https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/86637016:56
noonedeadpunkwe'd better find reason why it fails ofc, but stilll16:57
jrosserit has been a long time now like that16:59
noonedeadpunkwell, true, but we create users/projects/catalog during role runtime anyway17:03
noonedeadpunkso if it's not working - we will fail early anyway17:03
noonedeadpunkwell, except infra jobs17:03
jrosseroh well i mean we don't just ignore it for all this time - there is nothing obvious wrong when keystone breaks like this17:06
jrosserother than haproxy thinks it is down?17:06
noonedeadpunkyup17:07
noonedeadpunkI haven't seen ooms either17:07
noonedeadpunkSo it's some connection limit I guess17:08
noonedeadpunkbut increasing mpm settings didn't help much 17:11
noonedeadpunkSo I'm obviously missing smth17:11
jamesdentonnoonedeadpunk yeah, i was checking logs of failed jobs before the rest were done17:22
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Do not try to install packages on each log_instance_info run  https://review.opendev.org/c/openstack/openstack-ansible/+/86637217:23
* noonedeadpunk trying to find out what during log_instance_info run causes https://paste.openstack.org/show/bkdnIOr1vL4cEJ9vDM0i/ in output17:24
opendevreviewJames Denton proposed openstack/openstack-ansible master: Remove deprecated OVN and Panko services  https://review.opendev.org/c/openstack/openstack-ansible/+/86620517:35
noonedeadpunkOk, it's `ansible -i localhost, localhost -m setup` that causes that17:35
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Use local connection while colelcting instance info  https://review.opendev.org/c/openstack/openstack-ansible/+/86637417:40
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Use local connection while colelcting instance info  https://review.opendev.org/c/openstack/openstack-ansible/+/86637417:40
noonedeadpunk^ this would save another couple of mins for each job17:41
noonedeadpunkjamesdenton: btw, do you know if there's any way to set ovn to log into journal instead of files?17:45
noonedeadpunkso keystone was "down" for 12 seconds: https://zuul.opendev.org/t/openstack/build/58473167135147028d25b73b56670861/log/logs/host/haproxy.service.journal-15-06-19.log.txt#1846-186917:47
noonedeadpunkand /var/log/messages stopps 2 mins before that time o_O17:49
noonedeadpunkbtw we obviosuly misuse dstat17:50
noonedeadpunkhttps://zuul.opendev.org/t/openstack/build/58473167135147028d25b73b56670861/log/logs/openstack/instance-info/dstat.log.txt17:50
noonedeadpunkit's even better on jammy :D https://zuul.opendev.org/t/openstack/build/b6e0acd2a4df41ad911a2b5642b28a48/log/logs/openstack/instance-info/dstat.log.txt17:53
noonedeadpunkwell, dstat has been deprecated and renamed I guess https://github.com/dstat-real/dstat/issues/17018:01
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Do not try to install packages on each log_instance_info run  https://review.opendev.org/c/openstack/openstack-ansible/+/86637218:07
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Do not try to install packages on each log_instance_info run  https://review.opendev.org/c/openstack/openstack-ansible/+/86637218:07
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Replace dstat with dool  https://review.opendev.org/c/openstack/openstack-ansible/+/86637718:13
jamesdentonnoonedeadpunk i imagine there is, i can look into it18:13
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Replace dstat with dool  https://review.opendev.org/c/openstack/openstack-ansible/+/86637718:14
noonedeadpunk866249 looks sweet btw :)18:18
noonedeadpunkand yeah, logging to journal would be very sweet!18:19
noonedeadpunkBtw regarding reviews - there're couple of patches that seems ready https://review.opendev.org/q/parentproject:openstack/openstack-ansible++NOT+label:Workflow%253C%253D-1+NOT+label:Workflow%253E%253D1+label:Verified%253E%253D1+NOT+label:Code-Review%253C%253D-1+label:Code-Review%253E%253D2+branch:master+status:open+18:20
noonedeadpunkwell, it seems that check queue in zuul is quite outstanding now :(18:23
noonedeadpunklike some checks are in for 5h :(18:24
jamesdentonyes, if we can get 866249 then i think 862924 will be good to go18:26
opendevreviewMerged openstack/openstack-ansible-os_octavia stable/yoga: Adding octavia_provider_network_mtu-parameter parameter  https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/86564320:10
opendevreviewMerged openstack/openstack-ansible-os_heat stable/xena: Install git into heat containers  https://review.opendev.org/c/openstack/openstack-ansible-os_heat/+/86556520:42
*** dviroel is now known as dviroel|out21:02
opendevreviewMerged openstack/openstack-ansible master: Allow to define user.rc file for OSA  https://review.opendev.org/c/openstack/openstack-ansible/+/86586721:34
opendevreviewMerged openstack/openstack-ansible master: Define a group for ironic_console  https://review.opendev.org/c/openstack/openstack-ansible/+/86094721:34
opendevreviewMerged openstack/openstack-ansible master: Add zookeeper deployment  https://review.opendev.org/c/openstack/openstack-ansible/+/86475021:34
opendevreviewMerged openstack/openstack-ansible-lxc_hosts master: Use correct apt repo when ubuntu architecture is not x86_64  https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/86616722:07
opendevreviewMerged openstack/openstack-ansible-os_nova master: fix: os_vif_ovs should not be in the middle of libvirt section  https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/86615923:01
* prometheanfire wonders why https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/865067 wasn't good enough23:35
opendevreviewMerged openstack/openstack-ansible-os_neutron master: Set default plugin type to OVN  https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/86596123:44

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