*** ysandeep|out is now known as ysandeep | 05:04 | |
*** ysandeep is now known as ysandeep|lunch | 07:24 | |
*** ysandeep|lunch is now known as ysandeep | 08:04 | |
*** ysandeep is now known as ysandeep|afk | 09:00 | |
opendevreview | Jean-Philippe Evrard proposed openstack/openstack-ansible master: Cleanup useless variables https://review.opendev.org/c/openstack/openstack-ansible/+/852563 | 09:19 |
---|---|---|
opendevreview | Jean-Philippe Evrard proposed openstack/openstack-ansible master: Cleanup useless variables https://review.opendev.org/c/openstack/openstack-ansible/+/852563 | 09:21 |
*** ysandeep|afk is now known as ysandeep | 09:36 | |
*** ysandeep is now known as ysandeep|afk | 10:26 | |
noonedeadpunk | mornings | 10:40 |
*** ysandeep|afk is now known as ysandeep | 10:58 | |
opendevreview | Jiri 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/+/858260 | 11:16 |
damiandabrowski | hi! | 11:17 |
noonedeadpunk | anskiy: I have a comment on https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/855829 | 11:25 |
noonedeadpunk | As patch does break centos ovn deployment | 11:26 |
anskiy | noonedeadpunk: thank you, I will try to take a look later this week. | 11:27 |
*** tosky is now known as Guest1012 | 12:14 | |
*** tosky__ is now known as tosky | 12:14 | |
opendevreview | Jiri Podivin proposed openstack/openstack-ansible-os_tempest master: Switching linters to pre-commit system https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/858521 | 12:15 |
*** tosky is now known as Guest1013 | 12:30 | |
*** tosky__ is now known as tosky | 12:30 | |
ThiagoCMC | Hey folks, is that "br-bmaas" required for OSA Ironic? | 13:59 |
noonedeadpunk | I'd say it's designed for Ironic. | 14:11 |
noonedeadpunk | iirc it was for accessing impi or smth like that. I bet there was some kind of doc about it | 14:11 |
*** ysandeep is now known as ysandeep|dinner | 14:12 | |
ThiagoCMC | Hmm... Ok, thanks! | 14:35 |
noonedeadpunk | ThiagoCMC: maybe jamesdenton has more info about that :) | 14:39 |
opendevreview | Jean-Philippe Evrard proposed openstack/openstack-ansible master: [wip] Cleanup useless variables https://review.opendev.org/c/openstack/openstack-ansible/+/852563 | 14:55 |
noonedeadpunk | #startmeeting openstack_ansible_meeting | 15:00 |
opendevmeet | Meeting 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 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
opendevmeet | The meeting name has been set to 'openstack_ansible_meeting' | 15:00 |
noonedeadpunk | #topic rollcall | 15:00 |
noonedeadpunk | o/ | 15:00 |
damiandabrowski | hi! | 15:00 |
mgariepy | o/ | 15:01 |
noonedeadpunk | #topic office hours | 15:04 |
noonedeadpunk | mgariepy: do you remeber if we have merged fix for ssh session that got terminated on facts gathering? | 15:05 |
noonedeadpunk | I just saw another failure in CI on master :( | 15:05 |
noonedeadpunk | https://zuul.opendev.org/t/openstack/build/72e5f4b4bdb0419baf73ac716e620cbc/log/job-output.txt#11708 | 15:05 |
mgariepy | i think we did. | 15:05 |
mgariepy | it wasn't a proper fix tho. | 15:06 |
mgariepy | just having a longer delay .. so it won't happens as often | 15:06 |
noonedeadpunk | yeah, and it doesn't :) | 15:06 |
mgariepy | https://github.com/openstack/openstack-ansible/commit/cbdba67ad0b5a3e29db390c8e6b66721719184c0 | 15:07 |
noonedeadpunk | in order to release new Y version (with fixed keepalived role SHA) we need this https://review.opendev.org/c/openstack/openstack-ansible/+/858234 | 15:07 |
noonedeadpunk | yeah, great then. well, it's not but we have what we have | 15:07 |
noonedeadpunk | and we have bunch of reviews lingering around | 15:08 |
mgariepy | i'll take a look this afternoon. | 15:10 |
noonedeadpunk | ok, awesome. | 15:12 |
noonedeadpunk | I 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-ptg | 15:13 |
noonedeadpunk | I will also take time to populate it | 15:13 |
noonedeadpunk | I have booked a slot for operator hour on Wednesday, 19 October at 14-15 UTC | 15:26 |
noonedeadpunk | #link https://etherpad.opendev.org/p/osa-antelope-operator-hours | 15:27 |
noonedeadpunk | Regarding Rocky - I did not have a chance to spawn AIO ( | 15:30 |
noonedeadpunk | damiandabrowski: 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 |
noonedeadpunk | jamesdenton: 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 args | 15:34 |
noonedeadpunk | But can't recall where that has ended up | 15:34 |
*** ysandeep|dinner is now known as ysandeep | 15:35 | |
damiandabrowski | unfortunately I didn't :/ feel quite bad about it, will try to look into this soon | 15:36 |
noonedeadpunk | ok, would be great if you could write it down and return to it once have some cycles | 15:37 |
opendevreview | Rajesh Tailor proposed openstack/openstack-ansible-ops master: Fix typos https://review.opendev.org/c/openstack/openstack-ansible-ops/+/858546 | 15:49 |
noonedeadpunk | #endmeeting | 16:01 |
opendevmeet | Meeting ended Tue Sep 20 16:01:09 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-09-20-15.00.html | 16:01 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/openstack_ansible_meeting/2022/openstack_ansible_meeting.2022-09-20-15.00.txt | 16:01 |
opendevmeet | Log: https://meetings.opendev.org/meetings/openstack_ansible_meeting/2022/openstack_ansible_meeting.2022-09-20-15.00.log.html | 16:01 |
opendevreview | Merged openstack/openstack-ansible-haproxy_server master: Add variable for setting certbot `domains` option https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/857719 | 16:03 |
opendevreview | Merged openstack/openstack-ansible-os_nova master: Isolate vif for ovs backend by default https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/857204 | 16:03 |
ThiagoCMC | jamesdenton, 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|out | 16:15 | |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Bump ansible-core version to 2.13.4 https://review.opendev.org/c/openstack/openstack-ansible/+/857506 | 16:20 |
opendevreview | Dmitriy 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/+/858555 | 16:30 |
jamesdenton_ | noonedeadpunk yes, i was able to provide extra switches to the uwsgi app, let me find that | 16: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 |
noonedeadpunk | ok, should I take over and try to make some patch out of it? | 16:54 |
noonedeadpunk | or 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 next | 16: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 it | 16:56 |
opendevreview | Merged openstack/openstack-ansible master: Switch keepalived vrrp version from 2 to 3 https://review.opendev.org/c/openstack/openstack-ansible/+/857507 | 17:04 |
noonedeadpunk | I have no idea yet | 17:07 |
noonedeadpunk | btw good news is that ansible-core 2.13.4 seems to be super close to pass CI | 17:08 |
mgariepy | if you have some patch that need more urgent attention please poke me. | 17:45 |
ThiagoCMC | jamesdenton, thank you! | 17:59 |
opendevreview | Merged openstack/openstack-ansible-os_nova master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/857636 | 19:03 |
opendevreview | Merged openstack/openstack-ansible-os_senlin master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-os_senlin/+/857672 | 19:03 |
opendevreview | Merged openstack/openstack-ansible-os_heat master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-os_heat/+/857643 | 19:05 |
opendevreview | Merged openstack/openstack-ansible-galera_server master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/857812 | 19:10 |
opendevreview | Merged openstack/openstack-ansible-haproxy_server master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/857821 | 19:11 |
opendevreview | Merged openstack/openstack-ansible-os_placement master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-os_placement/+/857637 | 19:11 |
opendevreview | Merged openstack/openstack-ansible-os_ironic master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/857639 | 19:12 |
opendevreview | Merged openstack/openstack-ansible-os_tacker master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-os_tacker/+/857833 | 19:12 |
opendevreview | Merged openstack/openstack-ansible-rsyslog_server master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-rsyslog_server/+/857846 | 19:13 |
opendevreview | Merged openstack/openstack-ansible-os_keystone master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/857521 | 19:14 |
opendevreview | Merged openstack/openstack-ansible-openstack_hosts master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/857825 | 19:14 |
opendevreview | Merged openstack/openstack-ansible-lxc_hosts master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/857822 | 19:15 |
opendevreview | Merged openstack/openstack-ansible-os_barbican master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-os_barbican/+/857642 | 19:16 |
opendevreview | Merged openstack/openstack-ansible-os_rally master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/857656 | 19:16 |
opendevreview | Merged openstack/openstack-ansible-os_gnocchi master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-os_gnocchi/+/857831 | 19:19 |
opendevreview | Merged openstack/openstack-ansible-memcached_server master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-memcached_server/+/857823 | 19:19 |
opendevreview | Merged openstack/openstack-ansible-os_designate master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-os_designate/+/857641 | 19:20 |
opendevreview | Merged openstack/openstack-ansible-os_blazar master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-os_blazar/+/857829 | 19:20 |
opendevreview | Merged openstack/openstack-ansible-os_swift master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-os_swift/+/857673 | 19:21 |
mgariepy | noonedeadpunk, https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/858555 left you a small comment on :) | 19:22 |
opendevreview | Merged openstack/openstack-ansible-os_ceilometer master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-os_ceilometer/+/857830 | 19:22 |
opendevreview | Merged openstack/openstack-ansible-os_cinder master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-os_cinder/+/857635 | 19:22 |
opendevreview | Merged openstack/openstack-ansible-os_mistral master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-os_mistral/+/857651 | 19:23 |
opendevreview | Merged openstack/ansible-role-systemd_networkd master: Remove redundant vars line https://review.opendev.org/c/openstack/ansible-role-systemd_networkd/+/857863 | 19:25 |
opendevreview | Merged openstack/openstack-ansible-os_aodh master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-os_aodh/+/857828 | 19:27 |
opendevreview | Merged openstack/openstack-ansible-os_magnum master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/857646 | 19:29 |
opendevreview | Merged openstack/openstack-ansible-ceph_client master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/857813 | 19:38 |
opendevreview | Merged openstack/openstack-ansible-os_horizon master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-os_horizon/+/857644 | 19:40 |
opendevreview | Merged openstack/openstack-ansible-os_glance master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/857634 | 19:50 |
opendevreview | Merged openstack/openstack-ansible-os_tempest master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/857674 | 19:50 |
opendevreview | Merged openstack/openstack-ansible-os_neutron master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/857638 | 19:50 |
opendevreview | Merged openstack/openstack-ansible-os_murano master: Remove redundant vars line https://review.opendev.org/c/openstack/openstack-ansible-os_murano/+/857649 | 19:58 |
opendevreview | Merged openstack/openstack-ansible stable/xena: Fix dynamic-address-fact gathering with tags https://review.opendev.org/c/openstack/openstack-ansible/+/858331 | 20:12 |
opendevreview | Merged openstack/openstack-ansible stable/yoga: Fix dynamic-address-fact gathering with tags https://review.opendev.org/c/openstack/openstack-ansible/+/858329 | 20:12 |
opendevreview | Merged openstack/ansible-role-python_venv_build master: Remove redundant vars line https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/857875 | 20:23 |
ThiagoCMC | I'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 |
opendevreview | Merged openstack/openstack-ansible-os_keystone master: Add the option to deploy keystone without apache https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/856563 | 20:52 |
ThiagoCMC | jamesdenton, 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 |
jamesdenton | ThiagoCMC IPMI just needs to be reachable from ironic-conductor IP, whatever that is | 22:14 |
jamesdenton | In my environment, it's routed upstream | 22:14 |
jamesdenton | it does not need it own bridge or to be connected to openstack infra in any way, just needs to be reachable via routing | 22:14 |
jamesdenton | if "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 node | 22:15 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!