| noonedeadpunk | mornings | 08:36 |
|---|---|---|
| noonedeadpunk | yup, that's the intention :) | 08:36 |
| noonedeadpunk | (about skyline-console) | 08:36 |
| opendevreview | Andrew Bonney proposed openstack/openstack-ansible-os_horizon master: Use overridable URL for github rather than hard-coding https://review.opendev.org/c/openstack/openstack-ansible-os_horizon/+/914615 | 09:12 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_neutron master: Create an openrc for nb/sb clients https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/913582 | 09:56 |
| opendevreview | Damian Dąbrowski proposed openstack/openstack-ansible-haproxy_server master: Do not log content of copied static files https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/914623 | 10:11 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_neutron master: [doc] Expand documentation on OVN useful commands https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/913588 | 10:21 |
| opendevreview | Merged openstack/openstack-ansible-haproxy_server unmaintained/wallaby: Update .gitreview for unmaintained/wallaby https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/913083 | 10:23 |
| opendevreview | Merged openstack/openstack-ansible-haproxy_server unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/913140 | 10:28 |
| gokhan | hello noonedeadpunk , in antelope version can l3 agents run active/active. I am using ha routers in l3 agent and 2 of them is active and one of them is passive and there is no downtime. Is it expected state ? | 10:32 |
| noonedeadpunk | they can not be active/active | 10:34 |
| noonedeadpunk | they're always active/passive | 10:34 |
| gokhan | L3 Agent | 10:35 |
| gokhan | HostIDHigh Availability Status | 10:35 |
| gokhan | dev-infra12aaf6ff6-b7b2-44df-a510-affcc0f881ddactive | 10:35 |
| gokhan | dev-infra263b8c2a9-5f08-444c-82ae-f7b3502f6213standby | 10:35 |
| gokhan | dev-infra3f91a1f40-34d5-497f-9a01-ca5f6c930520active | 10:35 |
| noonedeadpunk | well, then smth went wrong :D | 10:38 |
| noonedeadpunk | do you have router IP on interfaces inside namespaces on both? | 10:38 |
| noonedeadpunk | or that is ovn? | 10:38 |
| noonedeadpunk | (I guess you won't say l3 for ovn) | 10:38 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_neutron master: Fix multiline yaml formatting in neutron systemd services https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/914544 | 10:44 |
| gokhan | noonedeadpunk, I am using l3 with ovs | 10:46 |
| noonedeadpunk | yeah, so it can't be active/active anyway | 10:46 |
| gokhan | yes I looked at interfaces and on qg interfaces there are same ips :( | 10:46 |
| noonedeadpunk | also - have you configured to have 3 agents per router? | 10:46 |
| noonedeadpunk | I guess yes, as otherwise you shouldn't see it in agent list.... | 10:47 |
| gokhan | noonedeadpunk, in neutron conf max_l3_agents_per_router = 3 | 10:49 |
| gokhan | it is really weird and my connection to instance doesn't cut anyway | 10:51 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-hardening master: Use valid value for CREATE_HOME https://review.opendev.org/c/openstack/ansible-hardening/+/908977 | 10:52 |
| noonedeadpunk | Well, it kind of feels some sort of split-brain | 10:58 |
| noonedeadpunk | I'd tried to check for keepalived pid process and kill it on one of "active".... | 10:59 |
| opendevreview | Damian Dąbrowski proposed openstack/openstack-ansible-haproxy_server master: Do not log content of copied static files https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/914623 | 11:00 |
| opendevreview | Damian Dąbrowski proposed openstack/openstack-ansible-haproxy_server master: Do not log content of copied static files https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/914623 | 11:00 |
| noonedeadpunk | you can check with `cat /var/lib/neutron/ha_confs/<router uuid>.pid.keepalived-vrrp | 11:01 |
| noonedeadpunk | or jsut remove a router from agent trhough cli | 11:02 |
| jrosser | so we need apt 2.3.10 in order to be able to have a gpg key embedded directly into a deb822 style sources config | 11:02 |
| noonedeadpunk | and bullseye has prior verison? | 11:02 |
| jrosser | and bullseye only has 2.2.4 | 11:02 |
| jrosser | which is pretty sad | 11:03 |
| noonedeadpunk | so... we should revert then patch for openstack_hosts | 11:03 |
| noonedeadpunk | and leave that for 2024.2 | 11:03 |
| jrosser | right, yes | 11:03 |
| gokhan | noonedeadpunk, I killed it process after that it status schanged to backup but then again master https://paste.openstack.org/show/bcLEDEwHeuS7qrl4ZG7z/ | 11:03 |
| noonedeadpunk | gokhan: huh. then you'd better ask neutron folks :D | 11:03 |
| gokhan | noonedeadpunk, yes I am also asking this in neutron channel, thanks | 11:07 |
| gokhan | noonedeadpunk, I also killed second primary l3 agent process and then standby node status changed to active :) | 11:11 |
| noonedeadpunk | whaaaat | 11:13 |
| noonedeadpunk | so... I'd try to check from the namespace if keepalived can communicate | 11:13 |
| gokhan | I am checking ha ports | 11:15 |
| gokhan | noonedeadpunk, yes in infra1 namespaces I can not ping ha port ips but on infra 2 and infra3 I can ping between themselves | 11:20 |
| jrosser | ok so looks like we did not yet merge any deb822 stuff | 11:25 |
| noonedeadpunk | oh? | 11:44 |
| noonedeadpunk | aha, https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/907434? failed dramatically yesterday, I see | 11:45 |
| jrosser | yes | 11:45 |
| jrosser | i've WIP everyting on that topic for now | 11:45 |
| opendevreview | Merged openstack/openstack-ansible-os_horizon stable/2023.1: Do not change mode of files recursively https://review.opendev.org/c/openstack/openstack-ansible-os_horizon/+/914016 | 11:54 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_skyline master: Add EL distro support https://review.opendev.org/c/openstack/openstack-ansible-os_skyline/+/912370 | 12:04 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_skyline master: Install skyline-console through yarn https://review.opendev.org/c/openstack/openstack-ansible-os_skyline/+/914405 | 12:04 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_skyline master: Install skyline-console through yarn https://review.opendev.org/c/openstack/openstack-ansible-os_skyline/+/914405 | 12:04 |
| opendevreview | Merged openstack/openstack-ansible-os_heat master: Grant proper privileges to admin user for testing purposes https://review.opendev.org/c/openstack/openstack-ansible-os_heat/+/912108 | 12:51 |
| opendevreview | Merged openstack/ansible-hardening master: Use valid value for CREATE_HOME https://review.opendev.org/c/openstack/ansible-hardening/+/908977 | 13:04 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-hardening stable/2023.2: Use valid value for CREATE_HOME https://review.opendev.org/c/openstack/ansible-hardening/+/914593 | 13:05 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-hardening stable/2023.1: Use valid value for CREATE_HOME https://review.opendev.org/c/openstack/ansible-hardening/+/914594 | 13:06 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-hardening stable/zed: Use valid value for CREATE_HOME https://review.opendev.org/c/openstack/ansible-hardening/+/914595 | 13:06 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_heat stable/2023.2: Grant proper privileges to admin user for testing purposes https://review.opendev.org/c/openstack/openstack-ansible-os_heat/+/914596 | 13:18 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_heat stable/2023.1: Grant proper privileges to admin user for testing purposes https://review.opendev.org/c/openstack/openstack-ansible-os_heat/+/914597 | 13:18 |
| opendevreview | Merged openstack/openstack-ansible-os_horizon master: Reflect horizon_webroot setting in Apache vhost configuration https://review.opendev.org/c/openstack/openstack-ansible-os_horizon/+/913442 | 13:19 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_heat stable/zed: Grant proper privileges to admin user for testing purposes https://review.opendev.org/c/openstack/openstack-ansible-os_heat/+/914598 | 13:21 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_horizon stable/2023.2: Reflect horizon_webroot setting in Apache vhost configuration https://review.opendev.org/c/openstack/openstack-ansible-os_horizon/+/914599 | 13:33 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_horizon stable/2023.1: Reflect horizon_webroot setting in Apache vhost configuration https://review.opendev.org/c/openstack/openstack-ansible-os_horizon/+/914600 | 13:33 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_horizon stable/zed: Reflect horizon_webroot setting in Apache vhost configuration https://review.opendev.org/c/openstack/openstack-ansible-os_horizon/+/914661 | 13:33 |
| opendevreview | Amy Marrich proposed openstack/openstack-ansible-os_neutron master: [doc] Briefly describe VPNaaS plugin changes with OVN https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/910981 | 13:39 |
| mgariepy | spotz[m], noonedeadpunk added a comment on : https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/910981 | 13:52 |
| noonedeadpunk | fair enough | 13:53 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_neutron master: [doc] Briefly describe VPNaaS plugin changes with OVN https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/910981 | 13:53 |
| noonedeadpunk | we pretty much need to land https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/914275 to switch to tracking 2024.1 | 13:54 |
| mgariepy | done. | 13:54 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Use container setup role from plugins repo https://review.opendev.org/c/openstack/openstack-ansible/+/905004 | 13:56 |
| opendevreview | Merged openstack/openstack-ansible-os_neutron stable/2023.2: Restart OVN on certificate changes https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/914013 | 13:56 |
| opendevreview | Merged openstack/openstack-ansible-os_neutron stable/2023.2: Use ansible_facts['processor_vcpus'] instead of fact variable https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/912753 | 13:57 |
| noonedeadpunk | awesome, thanks! | 14:08 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-galera_server master: Implement installation method selection for MariaDB role https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/914530 | 14:12 |
| noonedeadpunk | ThiagoCMC: you can try this one for mariadb from distro packages ^ | 14:13 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Test distro installation method fo MariaDB/RabbitMQ for distro jobs https://review.opendev.org/c/openstack/openstack-ansible/+/914649 | 14:19 |
| noonedeadpunk | Actually, I'm pretty much happy with state of https://review.opendev.org/q/topic:%22osa/skyline%22 today.... | 14:20 |
| noonedeadpunk | will try to spawn another aio just in case..... | 14:32 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_skyline master: Install skyline-console through yarn https://review.opendev.org/c/openstack/openstack-ansible-os_skyline/+/914405 | 14:46 |
| noonedeadpunk | just in case - it passed ^ | 14:46 |
| opendevreview | James Denton proposed openstack/openstack-ansible-os_skyline master: Support large uploads via Skyline https://review.opendev.org/c/openstack/openstack-ansible-os_skyline/+/914149 | 14:53 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_skyline master: Do not define a random password for each run https://review.opendev.org/c/openstack/openstack-ansible-os_skyline/+/912332 | 14:54 |
| noonedeadpunk | jrosser: about adding unmaintained patches to the dashboard. we'd need to drop smth from it for that | 15:09 |
| noonedeadpunk | as there's a gerrit limit on 10 queries | 15:09 |
| noonedeadpunk | I'd drop either blueprints with negative feedback or just -1CR | 15:10 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Add cinderstore glance testing scenario https://review.opendev.org/c/openstack/openstack-ansible/+/901187 | 15:16 |
| opendevreview | Merged openstack/openstack-ansible-os_swift unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-os_swift/+/913173 | 15:16 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Add cinderstore glance testing scenario https://review.opendev.org/c/openstack/openstack-ansible/+/901187 | 15:17 |
| opendevreview | Merged openstack/openstack-ansible-os_masakari unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-os_masakari/+/913163 | 15:21 |
| opendevreview | Merged openstack/openstack-ansible-os_aodh unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-os_aodh/+/913148 | 15:21 |
| opendevreview | Merged openstack/openstack-ansible-os_placement unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-os_placement/+/913169 | 15:23 |
| opendevreview | Merged openstack/openstack-ansible-os_ironic unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/913159 | 15:28 |
| opendevreview | Merged openstack/openstack-ansible-os_nova unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/913167 | 15:28 |
| jrosser | noonedeadpunk: i guess i don't find the "14 days without feedback" section particularly useful | 15:29 |
| noonedeadpunk | as without negative feedback is kinda same, yeah | 15:30 |
| jrosser | oh well "without negative feedback" is sort of == things to review? | 15:33 |
| noonedeadpunk | kind of | 15:33 |
| noonedeadpunk | but with too small limit | 15:34 |
| jrosser | regardless of that we can totally lose the "Blueprints without negative feedback" | 15:35 |
| opendevreview | Andrew Bonney proposed openstack/openstack-ansible-os_horizon master: Use overridable URL for github rather than hard-coding https://review.opendev.org/c/openstack/openstack-ansible-os_horizon/+/914615 | 15:35 |
| opendevreview | Merged openstack/openstack-ansible-os_mistral unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-os_mistral/+/913164 | 15:49 |
| opendevreview | Merged openstack/openstack-ansible-repo_server unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/913180 | 15:50 |
| opendevreview | Merged openstack/openstack-ansible-rsyslog_client unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-rsyslog_client/+/913181 | 15:52 |
| opendevreview | Merged openstack/openstack-ansible-os_barbican unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-os_barbican/+/913149 | 15:54 |
| opendevreview | Merged openstack/openstack-ansible-os_trove unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-os_trove/+/913176 | 15:54 |
| opendevreview | Merged openstack/openstack-ansible-os_adjutant unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-os_adjutant/+/913146 | 15:55 |
| opendevreview | Merged openstack/ansible-hardening stable/2023.2: Use valid value for CREATE_HOME https://review.opendev.org/c/openstack/ansible-hardening/+/914593 | 15:55 |
| opendevreview | Merged openstack/openstack-ansible-os_magnum unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/913161 | 15:58 |
| jrosser | noonedeadpunk: you might come across this https://paste.opendev.org/show/bc0e8HTjOcFLk1s5qr78/ | 16:01 |
| jrosser | we have an override for `deployment_extra_facts_filter` in /etc/openstack_deploy/haproxy_all.yml and 2023.2 ignores that | 16:02 |
| jrosser | when `deployment_extra_facts_filter` is a task var it is higher precendence than host/group vars | 16:02 |
| jrosser | not sure if there is a better way to do that | 16:03 |
| opendevreview | Merged openstack/openstack-ansible-os_designate unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-os_designate/+/913154 | 16:06 |
| opendevreview | Merged openstack/openstack-ansible-os_sahara unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-os_sahara/+/913171 | 16:06 |
| noonedeadpunk | jrosser: nah, I actually haven't. We just have tiny playbook that would also include gather-hardware-facts.yml but passing it vars we need | 16:08 |
| jrosser | ah ok | 16:09 |
| opendevreview | Merged openstack/openstack-ansible-os_tempest unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/913175 | 16:09 |
| noonedeadpunk | but what are you overriding in group_vars? | 16:09 |
| jrosser | we end up with missing facts for bond0 interface | 16:09 |
| noonedeadpunk | deployment_extra_facts_filter? | 16:09 |
| jrosser | yes | 16:09 |
| jrosser | i guess we just should be very careful when using existing var names as task vars | 16:10 |
| jrosser | as they then are basically hardwired | 16:10 |
| jrosser | it was ok on 2023.1 | 16:11 |
| noonedeadpunk | until this ? https://opendev.org/openstack/openstack-ansible/commit/ba9a248550ba1a9d10d764aec38cbe67b3a11ca6 | 16:11 |
| noonedeadpunk | which is backported doooooown the road | 16:12 |
| noonedeadpunk | https://review.opendev.org/q/I4e72e81d05d8d70e76730a46687dd06f3e66ebc6 | 16:12 |
| jrosser | yeah - i suspect we froze our 2023.1 mirror pretty early and didnt have that | 16:13 |
| jrosser | looks like ultimately this is becasue we also have `haproxy_stats_bind_address: "{{ ansible_facts['bond0']['ipv4']['address'] }}"` | 16:17 |
| ThiagoCMC | noonedeadpunk, thank you! I'll give it a try this weekend. | 16:18 |
| noonedeadpunk | jrosser: so, we can move these vars to haproxy group_vars then I assume? | 16:21 |
| noonedeadpunk | but I'm not sure what exactly broke with that patch | 16:22 |
| noonedeadpunk | as default facts are even more minimalistic | 16:22 |
| noonedeadpunk | or well... | 16:22 |
| noonedeadpunk | if you overriden that in group_vars... | 16:22 |
| noonedeadpunk | so yeah - let's move https://opendev.org/openstack/openstack-ansible/src/branch/master/playbooks/haproxy-install.yml#L22-L29 to group_vars/haproxy and backport? | 16:23 |
| jrosser | it’s already defined in group_vars/all | 16:25 |
| noonedeadpunk | yeah, and then haproxy will have precedence as more specific group | 16:25 |
| jrosser | hmm ok | 16:25 |
| noonedeadpunk | and then you should be able to override in openstack_deploy/group_vars/haproxy still due to inventory load precedence | 16:25 |
| noonedeadpunk | like all is just ansible reserved thing iirc | 16:26 |
| noonedeadpunk | so not `all` having precedence over all always iirc | 16:26 |
| jrosser | aaah right yes I just checked the docs | 16:27 |
| noonedeadpunk | Can you quickly check the dashboard before I changed the topic?:) https://bit.ly/3ISUETc | 16:28 |
| jrosser | playbook group vars still win over inventory group vars though | 16:28 |
| noonedeadpunk | but we don't have playbook group vars? | 16:29 |
| noonedeadpunk | like as a concept? | 16:29 |
| jrosser | well | 16:30 |
| jrosser | we have the group_vars in the inventory directory | 16:30 |
| opendevreview | Merged openstack/openstack-ansible stable/2023.1: Determine if upgrade source branch is stable/ or unmaintained/ https://review.opendev.org/c/openstack/openstack-ansible/+/912435 | 16:30 |
| noonedeadpunk | yeah, so these are inventory ones. same as in /etc/openstack_deploy | 16:31 |
| jrosser | and then also /etc/openstack_deploy/group_vars | 16:31 |
| noonedeadpunk | we place noop inventory.ini there if group_vars exist or smth like that | 16:31 |
| jrosser | and I guess the precedence of those is not clear | 16:31 |
| noonedeadpunk | precedence according to inventory iirc | 16:31 |
| noonedeadpunk | https://opendev.org/openstack/openstack-ansible/src/branch/master/scripts/openstack-ansible.rc#L19 | 16:31 |
| noonedeadpunk | like latter overrides prior | 16:32 |
| jrosser | so is that line 19 what defines that haproxy_all group vars in /etc overrides the ones in the integrated repo? | 16:34 |
| jrosser | as a side effect of the inventory in /etc being last in the list | 16:34 |
| noonedeadpunk | yeah, kinda | 16:34 |
| noonedeadpunk | pretty much | 16:34 |
| jrosser | right that makes sense now | 16:35 |
| noonedeadpunk | It's /o\ but yes, makes sense :D | 16:35 |
| opendevreview | Merged openstack/openstack-ansible-os_glance master: Add worker_self_reference_url to glance configuration https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/914275 | 16:35 |
| opendevreview | Merged openstack/openstack-ansible-os_glance unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/913155 | 16:35 |
| opendevreview | Merged openstack/openstack-ansible-os_glance unmaintained/wallaby: Update .gitreview for unmaintained/wallaby https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/913098 | 16:35 |
| jrosser | ok so I see that yes it’s the simplest thing to move those task vars to haproxy_all group vars | 16:36 |
| noonedeadpunk | will you push smth then? | 16:36 |
| jrosser | sure, will do that later | 16:36 |
| noonedeadpunk | ++ | 16:36 |
| jrosser | we just started on bobcat upgrade in the lab yesterday/today | 16:37 |
| jrosser | so few small gotchas coming up | 16:37 |
| noonedeadpunk | so, you're not going to use just SLURPs? | 16:38 |
| jrosser | I pushed a patch for os_neutron and the linter is just /o\ | 16:38 |
| noonedeadpunk | I'd actually waited for next point release, as requirements patch now merged that bumps python3-ovs | 16:39 |
| jrosser | well we want to do cluster api | 16:39 |
| jrosser | so need to get beyond antelope to not need 100 cherry picks | 16:39 |
| noonedeadpunk | https://review.opendev.org/c/openstack/requirements/+/912071 | 16:40 |
| noonedeadpunk | I was hoping to get things merged/backported to make new point release fast | 16:40 |
| jrosser | still rocking linuxbridge here :) | 16:40 |
| noonedeadpunk | aha | 16:40 |
| noonedeadpunk | I thought you had ovs | 16:40 |
| jrosser | got to choose which fight to have tbh | 16:40 |
| * noonedeadpunk still regrets about good old days with lxb | 16:41 | |
| jrosser | for slurp the uograde pain is really not so bad | 16:42 |
| jrosser | I mean it is not so painful to upgrade openstack that we want to skip | 16:43 |
| noonedeadpunk | Yeah... And I was actually thinking of finally doing some beta release close to the coordinated one... | 16:43 |
| jrosser | operating system upgrade from focal to jammy was massively larger work | 16:43 |
| noonedeadpunk | yeah, true | 16:43 |
| noonedeadpunk | it's getting even worse with encrypted volumes or other things you can't live-migrate | 16:44 |
| jrosser | I have a feeling that we hacked that - as it is single org cloud we could give the admin user permission on the secrets | 16:45 |
| jrosser | but obv that is not cool for many use cases | 16:46 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: [Feature] Add skyline deployment capability https://review.opendev.org/c/openstack/openstack-ansible/+/859446 | 16:50 |
| noonedeadpunk | crap, one last nit ^ | 16:50 |
| noonedeadpunk | jrosser: yeah, I mean for private that is less sensitive thing potentially. Though even for private I can see how this can fail | 16:51 |
| noonedeadpunk | ie super cool and moders devs don't want to trust stupid infra | 16:51 |
| *** ChanServ changes topic to "Launchpad: https://launchpad.net/openstack-ansible || Weekly Meetings: https://wiki.openstack.org/wiki/Meetings/openstack-ansible || Review Dashboard: http://bit.ly/osa-review-board-v5" | 16:55 | |
| opendevreview | Merged openstack/openstack-ansible-os_blazar unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-os_blazar/+/913150 | 16:59 |
| opendevreview | Merged openstack/openstack-ansible-os_heat unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-os_heat/+/913157 | 17:00 |
| opendevreview | Merged openstack/openstack-ansible-os_manila unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-os_manila/+/913162 | 17:00 |
| opendevreview | Merged openstack/ansible-role-systemd_networkd unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/ansible-role-systemd_networkd/+/913132 | 17:00 |
| opendevreview | Merged openstack/openstack-ansible-os_ceilometer unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-os_ceilometer/+/913151 | 17:03 |
| opendevreview | Merged openstack/openstack-ansible-rabbitmq_server unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/913179 | 17:03 |
| opendevreview | Merged openstack/openstack-ansible-os_cloudkitty unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-os_cloudkitty/+/913153 | 17:03 |
| opendevreview | Merged openstack/openstack-ansible-galera_server unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/913139 | 17:04 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible stable/wallaby: Switch SHAs to EOM https://review.opendev.org/c/openstack/openstack-ansible/+/913414 | 17:04 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible stable/wallaby: Switch SHAs to EOM https://review.opendev.org/c/openstack/openstack-ansible/+/913414 | 17:05 |
| opendevreview | Merged openstack/ansible-hardening unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/ansible-hardening/+/913135 | 17:06 |
| opendevreview | Merged openstack/openstack-ansible-os_zun unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-os_zun/+/913177 | 17:06 |
| opendevreview | Merged openstack/openstack-ansible-openstack_openrc unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-openstack_openrc/+/913145 | 17:07 |
| opendevreview | Merged openstack/openstack-ansible-os_murano unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-os_murano/+/913165 | 17:08 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible stable/victoria: Switch SHAs to EOM https://review.opendev.org/c/openstack/openstack-ansible/+/910193 | 17:08 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible stable/victoria: Switch SHAs to EOM https://review.opendev.org/c/openstack/openstack-ansible/+/913418 | 17:09 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible stable/victoria: Switch SHAs to EOM https://review.opendev.org/c/openstack/openstack-ansible/+/913418 | 17:09 |
| opendevreview | Merged openstack/openstack-ansible-os_senlin unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-os_senlin/+/913172 | 17:10 |
| opendevreview | Merged openstack/ansible-role-systemd_mount unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/ansible-role-systemd_mount/+/913131 | 17:10 |
| opendevreview | Merged openstack/openstack-ansible-os_keystone unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/913160 | 17:12 |
| opendevreview | Merged openstack/openstack-ansible-os_gnocchi unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-os_gnocchi/+/913156 | 17:14 |
| opendevreview | Merged openstack/openstack-ansible-lxc_container_create unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-lxc_container_create/+/913141 | 17:20 |
| opendevreview | Merged openstack/openstack-ansible-lxc_hosts unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/913142 | 17:23 |
| ThiagoCMC | What do you folks think about this: https://github.com/redis/redis/pull/13157 ? | 17:25 |
| ThiagoCMC | Sorry, wrong chat... lol But still valid | 17:26 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-galera_server master: Implement installation method selection for MariaDB role https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/914530 | 17:35 |
| noonedeadpunk | ThiagoCMC: no redis - no problems? | 17:36 |
| noonedeadpunk | I hate the tendency though | 17:37 |
| noonedeadpunk | though I'm glad that mongo backed up and now allows distribution/installation of itself through external sources/tools | 17:38 |
| noonedeadpunk | lol, they've really dropped old licenses with original contributors? | 17:39 |
| ThiagoCMC | Yeah, precisely, I don't like this trend either | 17:39 |
| ThiagoCMC | Yeah! LOL | 17:39 |
| noonedeadpunk | I'm not a lawyer, but pretty sure they can not do this, and remove bsd headers like that... | 17:41 |
| ThiagoCMC | That's how I understand it too | 17:42 |
| noonedeadpunk | haha https://github.com/redis/redis/pull/13157#issuecomment-2013219637 | 17:42 |
| noonedeadpunk | yeah, that sucks ofc... but well, let's see how it will turn up | 17:44 |
| noonedeadpunk | as out of canonical taking lxd out of opensource, we now have incus that we can adopt in osa. | 17:44 |
| noonedeadpunk | as snap was pretty much a blocker | 17:45 |
| ThiagoCMC | That sounds like fun! I still use LXD but planing to move to Incus as well. It was awesome to see LXD packaged in Debian! Now Incus will be added too. Sad to see Canonical following the trend... Desperate times, it seems | 17:49 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Test distro installation method fo MariaDB/RabbitMQ for distro jobs https://review.opendev.org/c/openstack/openstack-ansible/+/914649 | 17:53 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-galera_server master: Add distro infra jobs https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/914691 | 17:55 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-rabbitmq_server master: Add distro infra jobs https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/914692 | 17:56 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Test distro installation method for MariaDB/RabbitMQ https://review.opendev.org/c/openstack/openstack-ansible/+/914649 | 17:56 |
| opendevreview | Merged openstack/ansible-role-uwsgi unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/ansible-role-uwsgi/+/913134 | 18:03 |
| opendevreview | Merged openstack/openstack-ansible-memcached_server unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-memcached_server/+/913143 | 18:04 |
| opendevreview | Merged openstack/openstack-ansible-os_tacker unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-os_tacker/+/913174 | 18:04 |
| opendevreview | Merged openstack/openstack-ansible-os_blazar unmaintained/wallaby: Update .gitreview for unmaintained/wallaby https://review.opendev.org/c/openstack/openstack-ansible-os_blazar/+/913093 | 18:05 |
| opendevreview | Merged openstack/openstack-ansible-os_tempest unmaintained/wallaby: Update .gitreview for unmaintained/wallaby https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/913119 | 18:06 |
| opendevreview | Merged openstack/openstack-ansible-os_rally unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/913170 | 18:09 |
| opendevreview | Merged openstack/openstack-ansible-os_cinder unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/openstack-ansible-os_cinder/+/913152 | 18:09 |
| opendevreview | Merged openstack/ansible-role-python_venv_build unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/913130 | 18:11 |
| opendevreview | Merged openstack/ansible-role-pki unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/ansible-role-pki/+/913129 | 18:21 |
| opendevreview | Merged openstack/openstack-ansible-os_zun unmaintained/wallaby: Update .gitreview for unmaintained/wallaby https://review.opendev.org/c/openstack/openstack-ansible-os_zun/+/913121 | 18:23 |
| opendevreview | Merged openstack/ansible-role-qdrouterd unmaintained/xena: Update .gitreview for unmaintained/xena https://review.opendev.org/c/openstack/ansible-role-qdrouterd/+/913128 | 18:24 |
| opendevreview | Merged openstack/openstack-ansible-os_masakari unmaintained/wallaby: Update .gitreview for unmaintained/wallaby https://review.opendev.org/c/openstack/openstack-ansible-os_masakari/+/913106 | 18:25 |
| opendevreview | Merged openstack/openstack-ansible-os_murano unmaintained/wallaby: Update .gitreview for unmaintained/wallaby https://review.opendev.org/c/openstack/openstack-ansible-os_murano/+/913108 | 18:25 |
| opendevreview | Merged openstack/ansible-role-uwsgi unmaintained/victoria: Update .gitreview for unmaintained/victoria https://review.opendev.org/c/openstack/ansible-role-uwsgi/+/913068 | 18:27 |
| opendevreview | Merged openstack/openstack-ansible-os_swift unmaintained/wallaby: Update .gitreview for unmaintained/wallaby https://review.opendev.org/c/openstack/openstack-ansible-os_swift/+/913117 | 18:28 |
| opendevreview | Merged openstack/openstack-ansible-os_placement unmaintained/wallaby: Update .gitreview for unmaintained/wallaby https://review.opendev.org/c/openstack/openstack-ansible-os_placement/+/913113 | 18:41 |
| opendevreview | Merged openstack/ansible-hardening stable/2023.1: Use valid value for CREATE_HOME https://review.opendev.org/c/openstack/ansible-hardening/+/914594 | 19:02 |
| opendevreview | Merged openstack/openstack-ansible master: Apply ironic virt_type only for ironic_compute group https://review.opendev.org/c/openstack/openstack-ansible/+/912173 | 19:07 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible stable/2023.2: Apply ironic virt_type only for ironic_compute group https://review.opendev.org/c/openstack/openstack-ansible/+/914664 | 20:03 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible stable/2023.1: Apply ironic virt_type only for ironic_compute group https://review.opendev.org/c/openstack/openstack-ansible/+/914665 | 20:04 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible stable/zed: Apply ironic virt_type only for ironic_compute group https://review.opendev.org/c/openstack/openstack-ansible/+/914666 | 20:04 |
| opendevreview | Merged openstack/ansible-hardening stable/zed: Use valid value for CREATE_HOME https://review.opendev.org/c/openstack/ansible-hardening/+/914595 | 20:37 |
| opendevreview | Merged openstack/openstack-ansible-os_horizon master: Use overridable URL for github rather than hard-coding https://review.opendev.org/c/openstack/openstack-ansible-os_horizon/+/914615 | 20:39 |
| jrosser | on the OEM patches why does "zuul checkout to master" rather than the unmaintained branch? | 21:19 |
| jrosser | i'm a bit confused as we managed to keep the testing basically all working for unmaintained/yoga | 21:20 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Define haproxy fact gathering vars at group rather than task level https://review.opendev.org/c/openstack/openstack-ansible/+/914722 | 22:08 |
| opendevreview | Damian Dąbrowski proposed openstack/openstack-ansible-os_horizon stable/2023.2: Use overridable URL for github rather than hard-coding https://review.opendev.org/c/openstack/openstack-ansible-os_horizon/+/914667 | 22:56 |
| opendevreview | Damian Dąbrowski proposed openstack/openstack-ansible-os_horizon stable/2023.1: Use overridable URL for github rather than hard-coding https://review.opendev.org/c/openstack/openstack-ansible-os_horizon/+/914668 | 22:56 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!