Tuesday, 2025-06-24

noonedeadpunkhey there06:07
noonedeadpunkI'm back so I guess we can do the meeting :)06:08
noonedeadpunkchannel looked quite deserted while looking through the history06:14
noonedeadpunkwow, gitea looks very different as well06:18
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible stable/2023.2: Bump SHAs for EOL-ing 2023.2  https://review.opendev.org/c/openstack/openstack-ansible/+/95089306:21
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible stable/2024.1: Bump SHAs for 2024.1  https://review.opendev.org/c/openstack/openstack-ansible/+/95088706:31
jrossergood morning06:48
opendevreviewMarcus Klein proposed openstack/openstack-ansible-ops master: grafana.grafana works without ansible fact vars  https://review.opendev.org/c/openstack/openstack-ansible-ops/+/95041107:19
noonedeadpunko/07:20
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible stable/2024.2: Bump SHAs for 2024.2  https://review.opendev.org/c/openstack/openstack-ansible/+/95089207:20
kleinigood morning07:20
opendevreviewDmitriy Chubinidze proposed openstack/openstack-ansible master: Install sudo on Debian 12  https://review.opendev.org/c/openstack/openstack-ansible/+/95306107:55
kleininoonedeadpunk, https://review.opendev.org/c/openstack/openstack-ansible-ops/+/950411 <- what is wrong about using galera_address?08:33
noonedeadpunkah, lol, ok, my bad08:34
noonedeadpunknothing is wrong, I just forgot we have this: https://opendev.org/openstack/openstack-ansible/src/commit/090ec052368f07cae932743aad4d01a510297d34/inventory/group_vars/all/infra.yml#L6108:34
opendevreviewDmitriy 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/+/95320211:35
opendevreviewDmitriy 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/+/95320211:37
opendevreviewDmitriy Chubinidze proposed openstack/openstack-ansible-openstack_hosts master: Fix minor grammar issues in task  https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/95281211:37
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-openstack_hosts master: Implement GRUB modification process  https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/95194711:39
noonedeadpunkSo 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/+/95185211:42
opendevreviewIvan Anfimov proposed openstack/openstack-ansible-galera_server master: wip  https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/95320311:44
opendevreviewIvan Anfimov proposed openstack/openstack-ansible-galera_server master: wip  https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/95320311:44
opendevreviewIvan 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/+/95320311:46
opendevreviewIvan 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/+/95320311:46
opendevreviewIvan 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/+/95320612:12
opendevreviewIvan 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/+/95320612:14
opendevreviewIvan 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/+/95320312:56
opendevreviewDmitriy 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/+/95189114:01
opendevreviewDmitriy 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/+/95195014:25
opendevreviewMerged openstack/openstack-ansible-ops master: grafana.grafana works without ansible fact vars  https://review.opendev.org/c/openstack/openstack-ansible-ops/+/95041114:59
noonedeadpunk#startmeeting openstack_ansible_meeting15:00
opendevmeetMeeting 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
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
NeilHanlono/ 15:00
NeilHanlonwelcome back noonedeadpunk :) 15:00
noonedeadpunkthanks :)15:00
DavidGomezo/15:00
NeilHanlonhope you had a good time off15:01
noonedeadpunksorry - I clean forgot to send email last week :(15:01
NeilHanlonwe relaxed like you told us to :D 15:01
noonedeadpunkbut yeah, it was quite good. Though I wish I took even slightly more time off...15:01
NeilHanlonsame, i need a real vacation soon heh15:01
jrossero/ hello15:01
noonedeadpunkfor sure, for sure15:02
damiandabrowskihi!15:03
noonedeadpunk#topic office hours15:03
noonedeadpunktoday I've updated SHA bumps, as they were not reviewed/merged for 2 weeks, so did not make sense to leave them as-is15:05
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible stable/2023.2: Bump SHAs for EOL-ing 2023.2  https://review.opendev.org/c/openstack/openstack-ansible/+/95089315:06
noonedeadpunk#link https://review.opendev.org/q/topic:%22bump_osa%22+status:open15:07
noonedeadpunkbut some are failing now :(15:07
noonedeadpunkanother thing about releases - that 2023.2 went to EOL last week IIRC.15:07
noonedeadpunkSo 2023.2-eol tag was created for all our roles, only integrated repo left to finalize this.15:08
noonedeadpunkNeilHanlon: are you aware of any progress on adding Rocky 10 as CI image?15:08
NeilHanlonnot yet, but I will start to look at that15:09
noonedeadpunkor C10S...15:09
noonedeadpunkok, gotcha15:10
noonedeadpunkalso, 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/+/95185215:10
jrosseri wonder if we need a pip/setuptools etc bump on master for the sha bump failures there15:11
noonedeadpunkso I'd ask to review  https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/951852 with some prio15:11
noonedeadpunkyou mean this? https://review.opendev.org/c/openstack/openstack-ansible/+/951288/1/global-requirement-pins.txt15:12
jrosseroh /o\15:12
noonedeadpunkthe error with this bump is `ValueError: check_hostname requires server_hostname`15:13
noonedeadpunkwhich apparently goes from pip and is quite confusing...15:13
noonedeadpunkNeilHanlon: btw right now mnasiadka has proposed Rocky 10 image: https://review.opendev.org/c/openstack/diskimage-builder/+/952548 :)15:14
NeilHanlonoh, excellent :) 15:14
mnasiadkayeah, it's a long chain of patches before we can merge this one though (switch CI approach in DIB and some others...)15:15
opendevreviewDmitriy Chubinidze proposed openstack/openstack-ansible master: Ensure the remote_tmp directory exists with correct permissions  https://review.opendev.org/c/openstack/openstack-ansible/+/95321815:16
noonedeadpunkit;'s not _that_ long, but yeah15:16
noonedeadpunkI can hopefully test this one internally, as we rely on DIB for building images locally as well.15:16
noonedeadpunkmnasiadka: I kinda wonder why we're going path of having different yum.repos.d there15:18
noonedeadpunkas this probably could/should be `metalink=https://mirrors.centos.org/metalink?repo=centos-baseos-$releasever-stream&arch=$basearch` ?15:21
noonedeadpunkanyway15:23
noonedeadpunkother then that my plan is to proceed with reviews on pki/vault topic this week15:25
mnasiadkanoonedeadpunk: I ,,inherited'' that patch after somebody else, thanks for spotting that ;-)15:28
noonedeadpunkI guess that's prety much it from my side then15:32
noonedeadpunkah. we also got a reported issue with skyline here:15:32
noonedeadpunk#link https://answers.launchpad.net/openstack-ansible/+question/82199315:33
noonedeadpunkI've tried to reproduce the issue, and have a really weird behavior on metal installation...15:33
noonedeadpunkas I can't even auth in Skyline, getting 403 while trying to auth.15:34
noonedeadpunkI think it might be related to HSTS actually, as permissions were made for Horizon, while Skyline is kinda different15:35
jrosserhmm yes i can see this being incorrect15:40
jrosseri think it shows up in the browser console? i forgot a lot about this tbh15:41
noonedeadpunkbuyt still I see 401 even in /api/v1/login for gunicorn15:41
noonedeadpunkso it's smth different it seems15:41
opendevreviewDmitriy Chubinidze proposed openstack/openstack-ansible master: Define /tmp directory as default  https://review.opendev.org/c/openstack/openstack-ansible/+/95321815:44
noonedeadpunkI'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
noonedeadpunkso potentially another thing to look at...15:46
noonedeadpunkas there are apperntly no more topics to discuss, let's conclude the meeting15:49
noonedeadpunk#endmeeting15:49
opendevmeetMeeting ended Tue Jun 24 15:49:28 2025 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:49
opendevmeetMinutes:        https://meetings.opendev.org/meetings/openstack_ansible_meeting/2025/openstack_ansible_meeting.2025-06-24-15.00.html15:49
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/openstack_ansible_meeting/2025/openstack_ansible_meeting.2025-06-24-15.00.txt15:49
opendevmeetLog:            https://meetings.opendev.org/meetings/openstack_ansible_meeting/2025/openstack_ansible_meeting.2025-06-24-15.00.log.html15:49
noonedeadpunkjrosser: I replied in https://review.opendev.org/c/openstack/ansible-role-systemd_mount/+/951889/3 and ready to discuss it if needed16:21
jrosserI think I totally missed that the bar was used in multiple places16:29
jrosser*var16:29
noonedeadpunkaha, ok, gotcha then16:30
jrosserbut also I would think the overhead of a task is >> overhead of evaluating a var?16:30
jrosseranyway, whichever makes more sense but we do often put all sorts in vars/ and defaults/ for roles16:31
jrosserhugepages interesting topic though too :)16:31
noonedeadpunkwell, this depends on the output of command16:31
noonedeadpunkso probably setting it in vars is a bit weird?16:31
jrosserits ok I understand now with the multiple uses16:32
jrosserfor hugepages we have a script that did analysis16:33
noonedeadpunkso my idea was like this: https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/951950/4/doc/source/configure-nova.rst16:33
jrosserand 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 that16:34
noonedeadpunkoh16:34
noonedeadpunkI did not dive that deep into the topic to be honest...16:35
jrosserwe had some nvidia stuff that mandated hugepages16:35
noonedeadpunkI pretty much tried to implement what is documented by Nova and realized we don't have a straight way of doing that16:35
jrosserbut eventually it worked without explicitly enabling16:36
noonedeadpunksean 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
noonedeadpunkbut dunno16:37
jrosseryes I can see that being useful for migration16:37
noonedeadpunkanyway thought we should support usecase out of the box16:38
jrosseryes it’s a good idea16:39
jrosserI should see if we can contribute our script16:39
jrosserit was very interesting to understand what was happening with THP16:39
noonedeadpunkit;'s very interesting indeed... as I have not too deep understanding so far unfortunatelly - just really basic one16:40
opendevreviewMerged openstack/openstack-ansible-openstack_hosts master: Leverage community.general.modprobe for kernel module management  https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/95185216:41
* noonedeadpunk jsut recalled that forgot to contribute network check script16:45
opendevreviewMerged openstack/openstack-ansible-openstack_hosts master: Fix minor grammar issues in task  https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/95281216:57
opendevreviewIvan 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/+/95320217:01
opendevreviewIvan 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/+/95320217:01
opendevreviewIvan 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/+/95320217:03
opendevreviewMerged openstack/openstack-ansible-openstack_hosts master: Allow to blacklist kernel modules  https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/95186517:19
opendevreviewMerged openstack/openstack-ansible master: docs: fix for issues when switching between pages using localization  https://review.opendev.org/c/openstack/openstack-ansible/+/95228617:27
opendevreviewMerged openstack/openstack-ansible-galera_server master: Revert "Temporary disable molecule jobs"  https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/95161517:39
opendevreviewDmitriy Chubinidze proposed openstack/openstack-ansible master: wip: deploy  https://review.opendev.org/c/openstack/openstack-ansible/+/95323217:47
opendevreviewDmitriy Chubinidze proposed openstack/openstack-ansible master: wip: deploy  https://review.opendev.org/c/openstack/openstack-ansible/+/95323217:50
opendevreviewDmitriy Chubinidze proposed openstack/openstack-ansible master: wip: deploy  https://review.opendev.org/c/openstack/openstack-ansible/+/95323217:54
noonedeadpunkmnasiadka: 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... dunno18:08
mnasiadkanoonedeadpunk: 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
noonedeadpunkok, that is a good advice, will do18:13
noonedeadpunkI'm building Fedora 41 even with this patch now quite successfully...18:14
noonedeadpunkso now, hopefully, dib testing will become reproducible locally...18:17
opendevreviewDmitriy Chubinidze proposed openstack/openstack-ansible master: [doc] Add console code for copying directory  https://review.opendev.org/c/openstack/openstack-ansible/+/95323218:36
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: [doc] Add documentation on how to rotate passwords  https://review.opendev.org/c/openstack/openstack-ansible/+/95145118:38
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: [doc] Add documentation on how to rotate passwords  https://review.opendev.org/c/openstack/openstack-ansible/+/95145118:42
opendevreviewDmitriy Chubinidze proposed openstack/openstack-ansible master: [doc] use mariadb instead of mysql  https://review.opendev.org/c/openstack/openstack-ansible/+/95324219:38
opendevreviewDmitriy Chubinidze proposed openstack/openstack-ansible master: [doc] use mariadb instead of mysql  https://review.opendev.org/c/openstack/openstack-ansible/+/95324219:39
opendevreviewMerged openstack/openstack-ansible-lxc_hosts master: Fix name of yum_repository task  https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/95201119:57
opendevreviewIvan Anfimov proposed openstack/openstack-ansible master: wip  https://review.opendev.org/c/openstack/openstack-ansible/+/95228921:03
opendevreviewIvan Anfimov proposed openstack/openstack-ansible master: wip  https://review.opendev.org/c/openstack/openstack-ansible/+/95228921:03
opendevreviewIvan Anfimov proposed openstack/openstack-ansible master: wip  https://review.opendev.org/c/openstack/openstack-ansible/+/95228921:11
opendevreviewIvan Anfimov proposed openstack/openstack-ansible master: Fix for deprecation warnings with gather_facts  https://review.opendev.org/c/openstack/openstack-ansible/+/95228921:12
opendevreviewMerged openstack/openstack-ansible-lxc_hosts master: Remove COPR clean-up tasks  https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/95201222:10
opendevreviewMerged openstack/openstack-ansible master: Install sudo on Debian 12  https://review.opendev.org/c/openstack/openstack-ansible/+/95306122:26

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