noonedeadpunk | hey there | 06:07 |
---|---|---|
noonedeadpunk | I'm back so I guess we can do the meeting :) | 06:08 |
noonedeadpunk | channel looked quite deserted while looking through the history | 06:14 |
noonedeadpunk | wow, gitea looks very different as well | 06:18 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible stable/2023.2: Bump SHAs for EOL-ing 2023.2 https://review.opendev.org/c/openstack/openstack-ansible/+/950893 | 06:21 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible stable/2024.1: Bump SHAs for 2024.1 https://review.opendev.org/c/openstack/openstack-ansible/+/950887 | 06:31 |
jrosser | good morning | 06:48 |
opendevreview | Marcus Klein proposed openstack/openstack-ansible-ops master: grafana.grafana works without ansible fact vars https://review.opendev.org/c/openstack/openstack-ansible-ops/+/950411 | 07:19 |
noonedeadpunk | o/ | 07:20 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible stable/2024.2: Bump SHAs for 2024.2 https://review.opendev.org/c/openstack/openstack-ansible/+/950892 | 07:20 |
kleini | good morning | 07:20 |
opendevreview | Dmitriy Chubinidze proposed openstack/openstack-ansible master: Install sudo on Debian 12 https://review.opendev.org/c/openstack/openstack-ansible/+/953061 | 07:55 |
kleini | noonedeadpunk, https://review.opendev.org/c/openstack/openstack-ansible-ops/+/950411 <- what is wrong about using galera_address? | 08:33 |
noonedeadpunk | ah, lol, ok, my bad | 08:34 |
noonedeadpunk | nothing is wrong, I just forgot we have this: https://opendev.org/openstack/openstack-ansible/src/commit/090ec052368f07cae932743aad4d01a510297d34/inventory/group_vars/all/infra.yml#L61 | 08:34 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-openstack_hosts master: Ensure sudo is installed before configuring it https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/953202 | 11:35 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-openstack_hosts master: Ensure sudo is installed before configuring it https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/953202 | 11:37 |
opendevreview | Dmitriy Chubinidze proposed openstack/openstack-ansible-openstack_hosts master: Fix minor grammar issues in task https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/952812 | 11:37 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-openstack_hosts master: Implement GRUB modification process https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/951947 | 11:39 |
noonedeadpunk | So some CI (specifically healthcheck/hosts) is blocked now due to merging https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/951869 but not https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/951852 | 11:42 |
opendevreview | Ivan Anfimov proposed openstack/openstack-ansible-galera_server master: wip https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/953203 | 11:44 |
opendevreview | Ivan Anfimov proposed openstack/openstack-ansible-galera_server master: wip https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/953203 | 11:44 |
opendevreview | Ivan Anfimov proposed openstack/openstack-ansible-galera_server master: Cannot set fs attributes on a non-existent symlink target. follow should be set to False to avoid this. https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/953203 | 11:46 |
opendevreview | Ivan Anfimov proposed openstack/openstack-ansible-galera_server master: Cannot set fs attributes on a non-existent symlink target https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/953203 | 11:46 |
opendevreview | Ivan Anfimov proposed openstack/openstack-ansible-os_keystone master: Ensure the remote_tmp directory exists with correct permissions https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/953206 | 12:12 |
opendevreview | Ivan Anfimov proposed openstack/openstack-ansible-os_keystone master: Ensure the remote_tmp directory exists with correct permissions https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/953206 | 12:14 |
opendevreview | Ivan Anfimov proposed openstack/openstack-ansible-galera_server master: Cannot set fs attributes on a non-existent symlink target https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/953203 | 12:56 |
opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-systemd_mount master: Allow to define only overrides for mounts https://review.opendev.org/c/openstack/ansible-role-systemd_mount/+/951891 | 14:01 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_nova master: [doc] Add description on how to use huge pages https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/951950 | 14:25 |
opendevreview | Merged openstack/openstack-ansible-ops master: grafana.grafana works without ansible fact vars https://review.opendev.org/c/openstack/openstack-ansible-ops/+/950411 | 14:59 |
noonedeadpunk | #startmeeting openstack_ansible_meeting | 15:00 |
opendevmeet | Meeting started Tue Jun 24 15:00:36 2025 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 |
NeilHanlon | o/ | 15:00 |
NeilHanlon | welcome back noonedeadpunk :) | 15:00 |
noonedeadpunk | thanks :) | 15:00 |
DavidGomez | o/ | 15:00 |
NeilHanlon | hope you had a good time off | 15:01 |
noonedeadpunk | sorry - I clean forgot to send email last week :( | 15:01 |
NeilHanlon | we relaxed like you told us to :D | 15:01 |
noonedeadpunk | but yeah, it was quite good. Though I wish I took even slightly more time off... | 15:01 |
NeilHanlon | same, i need a real vacation soon heh | 15:01 |
jrosser | o/ hello | 15:01 |
noonedeadpunk | for sure, for sure | 15:02 |
damiandabrowski | hi! | 15:03 |
noonedeadpunk | #topic office hours | 15:03 |
noonedeadpunk | today I've updated SHA bumps, as they were not reviewed/merged for 2 weeks, so did not make sense to leave them as-is | 15:05 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible stable/2023.2: Bump SHAs for EOL-ing 2023.2 https://review.opendev.org/c/openstack/openstack-ansible/+/950893 | 15:06 |
noonedeadpunk | #link https://review.opendev.org/q/topic:%22bump_osa%22+status:open | 15:07 |
noonedeadpunk | but some are failing now :( | 15:07 |
noonedeadpunk | another thing about releases - that 2023.2 went to EOL last week IIRC. | 15:07 |
noonedeadpunk | So 2023.2-eol tag was created for all our roles, only integrated repo left to finalize this. | 15:08 |
noonedeadpunk | NeilHanlon: are you aware of any progress on adding Rocky 10 as CI image? | 15:08 |
NeilHanlon | not yet, but I will start to look at that | 15:09 |
noonedeadpunk | or C10S... | 15:09 |
noonedeadpunk | ok, gotcha | 15:10 |
noonedeadpunk | also, as wrote previously today, our hosts scenario is broken now due to merging https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/951869 but not https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/951852 | 15:10 |
jrosser | i wonder if we need a pip/setuptools etc bump on master for the sha bump failures there | 15:11 |
noonedeadpunk | so I'd ask to review https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/951852 with some prio | 15:11 |
noonedeadpunk | you mean this? https://review.opendev.org/c/openstack/openstack-ansible/+/951288/1/global-requirement-pins.txt | 15:12 |
jrosser | oh /o\ | 15:12 |
noonedeadpunk | the error with this bump is `ValueError: check_hostname requires server_hostname` | 15:13 |
noonedeadpunk | which apparently goes from pip and is quite confusing... | 15:13 |
noonedeadpunk | NeilHanlon: btw right now mnasiadka has proposed Rocky 10 image: https://review.opendev.org/c/openstack/diskimage-builder/+/952548 :) | 15:14 |
NeilHanlon | oh, excellent :) | 15:14 |
mnasiadka | yeah, it's a long chain of patches before we can merge this one though (switch CI approach in DIB and some others...) | 15:15 |
opendevreview | Dmitriy Chubinidze proposed openstack/openstack-ansible master: Ensure the remote_tmp directory exists with correct permissions https://review.opendev.org/c/openstack/openstack-ansible/+/953218 | 15:16 |
noonedeadpunk | it;'s not _that_ long, but yeah | 15:16 |
noonedeadpunk | I can hopefully test this one internally, as we rely on DIB for building images locally as well. | 15:16 |
noonedeadpunk | mnasiadka: I kinda wonder why we're going path of having different yum.repos.d there | 15:18 |
noonedeadpunk | as this probably could/should be `metalink=https://mirrors.centos.org/metalink?repo=centos-baseos-$releasever-stream&arch=$basearch` ? | 15:21 |
noonedeadpunk | anyway | 15:23 |
noonedeadpunk | other then that my plan is to proceed with reviews on pki/vault topic this week | 15:25 |
mnasiadka | noonedeadpunk: I ,,inherited'' that patch after somebody else, thanks for spotting that ;-) | 15:28 |
noonedeadpunk | I guess that's prety much it from my side then | 15:32 |
noonedeadpunk | ah. we also got a reported issue with skyline here: | 15:32 |
noonedeadpunk | #link https://answers.launchpad.net/openstack-ansible/+question/821993 | 15:33 |
noonedeadpunk | I've tried to reproduce the issue, and have a really weird behavior on metal installation... | 15:33 |
noonedeadpunk | as I can't even auth in Skyline, getting 403 while trying to auth. | 15:34 |
noonedeadpunk | I think it might be related to HSTS actually, as permissions were made for Horizon, while Skyline is kinda different | 15:35 |
jrosser | hmm yes i can see this being incorrect | 15:40 |
jrosser | i think it shows up in the browser console? i forgot a lot about this tbh | 15:41 |
noonedeadpunk | buyt still I see 401 even in /api/v1/login for gunicorn | 15:41 |
noonedeadpunk | so it's smth different it seems | 15:41 |
opendevreview | Dmitriy Chubinidze proposed openstack/openstack-ansible master: Define /tmp directory as default https://review.opendev.org/c/openstack/openstack-ansible/+/953218 | 15:44 |
noonedeadpunk | I'm really not sure why I have issues with Skyline though, as I'm pretty sure it was working normally on Caracal for me before... | 15:46 |
noonedeadpunk | so potentially another thing to look at... | 15:46 |
noonedeadpunk | as there are apperntly no more topics to discuss, let's conclude the meeting | 15:49 |
noonedeadpunk | #endmeeting | 15:49 |
opendevmeet | Meeting ended Tue Jun 24 15:49:28 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:49 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/openstack_ansible_meeting/2025/openstack_ansible_meeting.2025-06-24-15.00.html | 15:49 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/openstack_ansible_meeting/2025/openstack_ansible_meeting.2025-06-24-15.00.txt | 15:49 |
opendevmeet | Log: https://meetings.opendev.org/meetings/openstack_ansible_meeting/2025/openstack_ansible_meeting.2025-06-24-15.00.log.html | 15:49 |
noonedeadpunk | jrosser: I replied in https://review.opendev.org/c/openstack/ansible-role-systemd_mount/+/951889/3 and ready to discuss it if needed | 16:21 |
jrosser | I think I totally missed that the bar was used in multiple places | 16:29 |
jrosser | *var | 16:29 |
noonedeadpunk | aha, ok, gotcha then | 16:30 |
jrosser | but also I would think the overhead of a task is >> overhead of evaluating a var? | 16:30 |
jrosser | anyway, whichever makes more sense but we do often put all sorts in vars/ and defaults/ for roles | 16:31 |
jrosser | hugepages interesting topic though too :) | 16:31 |
noonedeadpunk | well, this depends on the output of command | 16:31 |
noonedeadpunk | so probably setting it in vars is a bit weird? | 16:31 |
jrosser | its ok I understand now with the multiple uses | 16:32 |
jrosser | for hugepages we have a script that did analysis | 16:33 |
noonedeadpunk | so my idea was like this: https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/951950/4/doc/source/configure-nova.rst | 16:33 |
jrosser | and something large like 99% of memory across hypervisors was merged by transparent huge pages without needing to enable actual hugepages with all the downsides that come with that | 16:34 |
noonedeadpunk | oh | 16:34 |
noonedeadpunk | I did not dive that deep into the topic to be honest... | 16:35 |
jrosser | we had some nvidia stuff that mandated hugepages | 16:35 |
noonedeadpunk | I pretty much tried to implement what is documented by Nova and realized we don't have a straight way of doing that | 16:35 |
jrosser | but eventually it worked without explicitly enabling | 16:36 |
noonedeadpunk | sean adviced usage of huge pages to speedup VM boot time as well as improve live migrations, as then it's bigger blocks of memory which are changed and needs to be migrated, so should improve performance... | 16:37 |
noonedeadpunk | but dunno | 16:37 |
jrosser | yes I can see that being useful for migration | 16:37 |
noonedeadpunk | anyway thought we should support usecase out of the box | 16:38 |
jrosser | yes it’s a good idea | 16:39 |
jrosser | I should see if we can contribute our script | 16:39 |
jrosser | it was very interesting to understand what was happening with THP | 16:39 |
noonedeadpunk | it;'s very interesting indeed... as I have not too deep understanding so far unfortunatelly - just really basic one | 16:40 |
opendevreview | Merged openstack/openstack-ansible-openstack_hosts master: Leverage community.general.modprobe for kernel module management https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/951852 | 16:41 |
* noonedeadpunk jsut recalled that forgot to contribute network check script | 16:45 | |
opendevreview | Merged openstack/openstack-ansible-openstack_hosts master: Fix minor grammar issues in task https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/952812 | 16:57 |
opendevreview | Ivan Anfimov proposed openstack/openstack-ansible-openstack_hosts master: Ensure sudo is installed before configuring it https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/953202 | 17:01 |
opendevreview | Ivan Anfimov proposed openstack/openstack-ansible-openstack_hosts master: Ensure sudo is installed before configuring it https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/953202 | 17:01 |
opendevreview | Ivan Anfimov proposed openstack/openstack-ansible-openstack_hosts master: Ensure sudo is installed before configuring it https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/953202 | 17:03 |
opendevreview | Merged openstack/openstack-ansible-openstack_hosts master: Allow to blacklist kernel modules https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/951865 | 17:19 |
opendevreview | Merged openstack/openstack-ansible master: docs: fix for issues when switching between pages using localization https://review.opendev.org/c/openstack/openstack-ansible/+/952286 | 17:27 |
opendevreview | Merged openstack/openstack-ansible-galera_server master: Revert "Temporary disable molecule jobs" https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/951615 | 17:39 |
opendevreview | Dmitriy Chubinidze proposed openstack/openstack-ansible master: wip: deploy https://review.opendev.org/c/openstack/openstack-ansible/+/953232 | 17:47 |
opendevreview | Dmitriy Chubinidze proposed openstack/openstack-ansible master: wip: deploy https://review.opendev.org/c/openstack/openstack-ansible/+/953232 | 17:50 |
opendevreview | Dmitriy Chubinidze proposed openstack/openstack-ansible master: wip: deploy https://review.opendev.org/c/openstack/openstack-ansible/+/953232 | 17:54 |
noonedeadpunk | mnasiadka: I think you already come to this in your dib patches, but potentially you might be interested in https://review.opendev.org/c/openstack/diskimage-builder/+/922109 as well... dunno | 18:08 |
mnasiadka | noonedeadpunk: you might want to rebase on https://review.opendev.org/c/openstack/diskimage-builder/+/949942/46 or https://review.opendev.org/c/openstack/diskimage-builder/+/952953/19 and adapt to new devstack based testing - then I guess it would make sense to merge it, especially that we're removing f37 ;-) | 18:10 |
noonedeadpunk | ok, that is a good advice, will do | 18:13 |
noonedeadpunk | I'm building Fedora 41 even with this patch now quite successfully... | 18:14 |
noonedeadpunk | so now, hopefully, dib testing will become reproducible locally... | 18:17 |
opendevreview | Dmitriy Chubinidze proposed openstack/openstack-ansible master: [doc] Add console code for copying directory https://review.opendev.org/c/openstack/openstack-ansible/+/953232 | 18:36 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: [doc] Add documentation on how to rotate passwords https://review.opendev.org/c/openstack/openstack-ansible/+/951451 | 18:38 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: [doc] Add documentation on how to rotate passwords https://review.opendev.org/c/openstack/openstack-ansible/+/951451 | 18:42 |
opendevreview | Dmitriy Chubinidze proposed openstack/openstack-ansible master: [doc] use mariadb instead of mysql https://review.opendev.org/c/openstack/openstack-ansible/+/953242 | 19:38 |
opendevreview | Dmitriy Chubinidze proposed openstack/openstack-ansible master: [doc] use mariadb instead of mysql https://review.opendev.org/c/openstack/openstack-ansible/+/953242 | 19:39 |
opendevreview | Merged openstack/openstack-ansible-lxc_hosts master: Fix name of yum_repository task https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/952011 | 19:57 |
opendevreview | Ivan Anfimov proposed openstack/openstack-ansible master: wip https://review.opendev.org/c/openstack/openstack-ansible/+/952289 | 21:03 |
opendevreview | Ivan Anfimov proposed openstack/openstack-ansible master: wip https://review.opendev.org/c/openstack/openstack-ansible/+/952289 | 21:03 |
opendevreview | Ivan Anfimov proposed openstack/openstack-ansible master: wip https://review.opendev.org/c/openstack/openstack-ansible/+/952289 | 21:11 |
opendevreview | Ivan Anfimov proposed openstack/openstack-ansible master: Fix for deprecation warnings with gather_facts https://review.opendev.org/c/openstack/openstack-ansible/+/952289 | 21:12 |
opendevreview | Merged openstack/openstack-ansible-lxc_hosts master: Remove COPR clean-up tasks https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/952012 | 22:10 |
opendevreview | Merged openstack/openstack-ansible master: Install sudo on Debian 12 https://review.opendev.org/c/openstack/openstack-ansible/+/953061 | 22:26 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!