Tuesday, 2022-09-20

*** ysandeep|out is now known as ysandeep05:04
*** ysandeep is now known as ysandeep|lunch07:24
*** ysandeep|lunch is now known as ysandeep08:04
*** ysandeep is now known as ysandeep|afk09:00
opendevreviewJean-Philippe Evrard proposed openstack/openstack-ansible master: Cleanup useless variables  https://review.opendev.org/c/openstack/openstack-ansible/+/85256309:19
opendevreviewJean-Philippe Evrard proposed openstack/openstack-ansible master: Cleanup useless variables  https://review.opendev.org/c/openstack/openstack-ansible/+/85256309:21
*** ysandeep|afk is now known as ysandeep09:36
*** ysandeep is now known as ysandeep|afk10:26
noonedeadpunkmornings10:40
*** ysandeep|afk is now known as ysandeep10:58
opendevreviewJiri Podivin proposed openstack/openstack-ansible-os_tempest master: Reordering variable definitions to prevent empty includelist edge case  https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/85826011:16
damiandabrowskihi!11:17
noonedeadpunkanskiy: I have a comment on https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/85582911:25
noonedeadpunkAs patch does break centos ovn deployment11:26
anskiynoonedeadpunk: thank you, I will try to take a look later this week.11:27
*** tosky is now known as Guest101212:14
*** tosky__ is now known as tosky12:14
opendevreviewJiri Podivin proposed openstack/openstack-ansible-os_tempest master: Switching linters to pre-commit system  https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/85852112:15
*** tosky is now known as Guest101312:30
*** tosky__ is now known as tosky12:30
ThiagoCMCHey folks, is that "br-bmaas" required for OSA Ironic?13:59
noonedeadpunkI'd say it's designed for Ironic. 14:11
noonedeadpunkiirc it was for accessing impi or smth like that. I bet there was some kind of doc about it14:11
*** ysandeep is now known as ysandeep|dinner14:12
ThiagoCMCHmm... Ok, thanks!14:35
noonedeadpunkThiagoCMC: maybe jamesdenton has  more info about that :)14:39
opendevreviewJean-Philippe Evrard proposed openstack/openstack-ansible master: [wip] Cleanup useless variables  https://review.opendev.org/c/openstack/openstack-ansible/+/85256314:55
noonedeadpunk#startmeeting openstack_ansible_meeting15:00
opendevmeetMeeting started Tue Sep 20 15:00:20 2022 UTC and is due to finish in 60 minutes.  The chair is noonedeadpunk. Information about MeetBot at http://wiki.debian.org/MeetBot.15:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
opendevmeetThe meeting name has been set to 'openstack_ansible_meeting'15:00
noonedeadpunk#topic rollcall15:00
noonedeadpunko/15:00
damiandabrowskihi!15:00
mgariepyo/15:01
noonedeadpunk#topic office hours15:04
noonedeadpunkmgariepy: do you remeber if we have merged fix for ssh session that got terminated on facts gathering?15:05
noonedeadpunkI just saw another failure in CI on master :(15:05
noonedeadpunkhttps://zuul.opendev.org/t/openstack/build/72e5f4b4bdb0419baf73ac716e620cbc/log/job-output.txt#1170815:05
mgariepyi think we did.15:05
mgariepyit wasn't a proper fix tho.15:06
mgariepyjust having a longer delay .. so it won't happens as often15:06
noonedeadpunkyeah, and it doesn't :)15:06
mgariepyhttps://github.com/openstack/openstack-ansible/commit/cbdba67ad0b5a3e29db390c8e6b66721719184c015:07
noonedeadpunkin order to release new Y version (with fixed keepalived role SHA) we need this https://review.opendev.org/c/openstack/openstack-ansible/+/85823415:07
noonedeadpunkyeah, great then. well, it's not but we have what we have15:07
noonedeadpunkand we have bunch of reviews lingering around15:08
mgariepyi'll take a look this afternoon.15:10
noonedeadpunkok, awesome. 15:12
noonedeadpunkI haven't fill in any real topics (-ENOTIME) into our PTG etherpad. But everybody welcome to contribute to it!15:12
noonedeadpunk#link https://etherpad.opendev.org/p/osa-antelope-ptg15:13
noonedeadpunkI will also take time to populate it15:13
noonedeadpunkI have booked a slot for operator hour on Wednesday, 19 October at 14-15 UTC15:26
noonedeadpunk#link https://etherpad.opendev.org/p/osa-antelope-operator-hours15:27
noonedeadpunkRegarding Rocky - I did not have a chance to spawn AIO (15:30
noonedeadpunkdamiandabrowski: I know you have a lot now, but I should ask - did you have a chance to look on https://bugs.launchpad.net/openstack-ansible/+bug/1981463 previously?15:31
noonedeadpunkjamesdenton: I do recall you looked into https://bugs.launchpad.net/openstack-ansible/+bug/1987405 ? or was that somebody else? And as a result it was posibee to provide extra files to uwsgi with args15:34
noonedeadpunkBut can't recall where that has ended up15:34
*** ysandeep|dinner is now known as ysandeep15:35
damiandabrowskiunfortunately I didn't :/ feel quite bad about it, will try to look into this soon15:36
noonedeadpunkok, would be great if you could write it down and return to it once have some cycles15:37
opendevreviewRajesh Tailor proposed openstack/openstack-ansible-ops master: Fix typos  https://review.opendev.org/c/openstack/openstack-ansible-ops/+/85854615:49
noonedeadpunk#endmeeting16:01
opendevmeetMeeting ended Tue Sep 20 16:01:09 2022 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:01
opendevmeetMinutes:        https://meetings.opendev.org/meetings/openstack_ansible_meeting/2022/openstack_ansible_meeting.2022-09-20-15.00.html16:01
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/openstack_ansible_meeting/2022/openstack_ansible_meeting.2022-09-20-15.00.txt16:01
opendevmeetLog:            https://meetings.opendev.org/meetings/openstack_ansible_meeting/2022/openstack_ansible_meeting.2022-09-20-15.00.log.html16:01
opendevreviewMerged openstack/openstack-ansible-haproxy_server master: Add variable for setting certbot `domains` option  https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/85771916:03
opendevreviewMerged openstack/openstack-ansible-os_nova master: Isolate vif for ovs backend by default  https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/85720416:03
ThiagoCMCjamesdenton, hello! Do you know if that "br-bmaas" required for OSA Ironic? If yes, what's it for (PXE or IPMI)?16:10
*** ysandeep is now known as ysandeep|out16:15
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Bump ansible-core version to 2.13.4  https://review.opendev.org/c/openstack/openstack-ansible/+/85750616:20
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-galera_server master: Bump mariadb version to 10.6.9  https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/85855516:30
jamesdenton_noonedeadpunk yes, i was able to provide extra switches to the uwsgi app, let me find that16:43
jamesdenton_ThiagoCMC br-bmaas is intended for pxe (and probably the flat network for neutron, too)16:44
jamesdenton_noonedeadpunk here's the modified execstart: https://paste.opendev.org/show/b0fX1XztEn69eLHj3Av9/16:52
noonedeadpunkok, should I take over and try to make some patch out of it?16:54
noonedeadpunkor you want/have time for that :)16:54
jamesdenton_if you could, that'd be great. but i can give it a go maybe later this week or next16:55
jamesdenton_maybe adding a "wsgi_execstarts" to the service definitions in https://github.com/openstack/openstack-ansible-os_neutron/blob/master/vars/main.yml? not sure how you want to approach it16:56
opendevreviewMerged openstack/openstack-ansible master: Switch keepalived vrrp version from 2 to 3  https://review.opendev.org/c/openstack/openstack-ansible/+/85750717:04
noonedeadpunkI have no idea yet17:07
noonedeadpunkbtw good news is that ansible-core 2.13.4 seems to be super close to pass CI17:08
mgariepyif you have some patch that need more urgent attention please poke me.17:45
ThiagoCMCjamesdenton, thank you!17:59
opendevreviewMerged openstack/openstack-ansible-os_nova master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/85763619:03
opendevreviewMerged openstack/openstack-ansible-os_senlin master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-os_senlin/+/85767219:03
opendevreviewMerged openstack/openstack-ansible-os_heat master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-os_heat/+/85764319:05
opendevreviewMerged openstack/openstack-ansible-galera_server master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/85781219:10
opendevreviewMerged openstack/openstack-ansible-haproxy_server master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/85782119:11
opendevreviewMerged openstack/openstack-ansible-os_placement master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-os_placement/+/85763719:11
opendevreviewMerged openstack/openstack-ansible-os_ironic master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/85763919:12
opendevreviewMerged openstack/openstack-ansible-os_tacker master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-os_tacker/+/85783319:12
opendevreviewMerged openstack/openstack-ansible-rsyslog_server master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-rsyslog_server/+/85784619:13
opendevreviewMerged openstack/openstack-ansible-os_keystone master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/85752119:14
opendevreviewMerged openstack/openstack-ansible-openstack_hosts master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/85782519:14
opendevreviewMerged openstack/openstack-ansible-lxc_hosts master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/85782219:15
opendevreviewMerged openstack/openstack-ansible-os_barbican master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-os_barbican/+/85764219:16
opendevreviewMerged openstack/openstack-ansible-os_rally master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/85765619:16
opendevreviewMerged openstack/openstack-ansible-os_gnocchi master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-os_gnocchi/+/85783119:19
opendevreviewMerged openstack/openstack-ansible-memcached_server master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-memcached_server/+/85782319:19
opendevreviewMerged openstack/openstack-ansible-os_designate master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-os_designate/+/85764119:20
opendevreviewMerged openstack/openstack-ansible-os_blazar master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-os_blazar/+/85782919:20
opendevreviewMerged openstack/openstack-ansible-os_swift master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-os_swift/+/85767319:21
mgariepynoonedeadpunk, https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/858555 left you a small comment on :)19:22
opendevreviewMerged openstack/openstack-ansible-os_ceilometer master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-os_ceilometer/+/85783019:22
opendevreviewMerged openstack/openstack-ansible-os_cinder master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-os_cinder/+/85763519:22
opendevreviewMerged openstack/openstack-ansible-os_mistral master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-os_mistral/+/85765119:23
opendevreviewMerged openstack/ansible-role-systemd_networkd master: Remove redundant vars line  https://review.opendev.org/c/openstack/ansible-role-systemd_networkd/+/85786319:25
opendevreviewMerged openstack/openstack-ansible-os_aodh master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-os_aodh/+/85782819:27
opendevreviewMerged openstack/openstack-ansible-os_magnum master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/85764619:29
opendevreviewMerged openstack/openstack-ansible-ceph_client master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/85781319:38
opendevreviewMerged openstack/openstack-ansible-os_horizon master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-os_horizon/+/85764419:40
opendevreviewMerged openstack/openstack-ansible-os_glance master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/85763419:50
opendevreviewMerged openstack/openstack-ansible-os_tempest master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/85767419:50
opendevreviewMerged openstack/openstack-ansible-os_neutron master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/85763819:50
opendevreviewMerged openstack/openstack-ansible-os_murano master: Remove redundant vars line  https://review.opendev.org/c/openstack/openstack-ansible-os_murano/+/85764919:58
opendevreviewMerged openstack/openstack-ansible stable/xena: Fix dynamic-address-fact gathering with tags  https://review.opendev.org/c/openstack/openstack-ansible/+/85833120:12
opendevreviewMerged openstack/openstack-ansible stable/yoga: Fix dynamic-address-fact gathering with tags  https://review.opendev.org/c/openstack/openstack-ansible/+/85832920:12
opendevreviewMerged openstack/ansible-role-python_venv_build master: Remove redundant vars line  https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/85787520:23
ThiagoCMCI'm seeing some examples here about OSA Ironic, related to "nova_virt_type: ironic", if I set in globally, isn't going to affect my regular compute nodes for KVM VMs?20:52
opendevreviewMerged openstack/openstack-ansible-os_keystone master: Add the option to deploy keystone without apache  https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/85656320:52
ThiagoCMCjamesdenton, one more question!  lol  Which network is used in OSA Ironic for BMC/IPMI connections? br-bmaas is intended for PXE, not sure where to put bmc traffic...21:23
jamesdentonThiagoCMC IPMI just needs to be reachable from ironic-conductor IP, whatever that is22:14
jamesdentonIn my environment, it's routed upstream22:14
jamesdentonit does not need it own bridge or to be connected to openstack infra in any way, just needs to be reachable via routing22:14
jamesdentonif "nova_virt_type: ironic" is global, you're gonna have a bad time. it just needs to be set for the ironic compute nodes, meaning wherever you run nova compute for ironic purposes. Typically, a controller node22:15

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