Tuesday, 2023-09-19

opendevreviewlikui proposed openstack/openstack-ansible-plugins master: Replace base64.encodestring with base64.encodebytes  https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/89575503:16
semanticnoonedeadpunk, thanks for supporting the feature! Still, we have to evaluate if the option would be really needed in production. Overall this cpu utilization based on busy waiting seems like BEAM responsivness optimization and it tends to lower with lowering available cpu cores by the way.06:28
semanticjrosser, yes, I did check dashboard, as I said this is pretty much almost empty openstack deployment and there's no backlog of messages, so rabbitmq is close to idling06:29
noonedeadpunkmornings07:05
noonedeadpunkoh, jrosser, you have already started looking into debian 12? :) 07:06
jrossernoonedeadpunk: yes! up to the poo t it’s broken…..07:06
jrosserit delays and tempest fails with my patches07:07
noonedeadpunk`Please, ensure that you have at least 1 host in repo_all group with the following distribution: debian 12.1 x86_64`07:07
jrosserno idea why yet07:07
noonedeadpunk12.1 sounds wrong for debian....07:07
jrosserargh autocorrect07:08
jrosseryes - I didn’t find why it does that07:08
noonedeadpunk`Found venv_build_targets: {'debian-12-x86_64': 'aio1_repo_container-9e6db7be'`07:08
jrosserthe version values come up different in the repo vs. the host07:08
noonedeadpunkand target looks correct....07:08
jrosserlook back in irc history, there was some chatter last week about this07:09
noonedeadpunkmaybe we have some condition somewhere for debian 1107:09
jrosserI have a vm somewhere I think here07:09
noonedeadpunkI was just going to have a look at debian 12 during the week as well :)07:09
jrosserif you’re going to make an AIO it would be good to get more eyes on this version wierdness07:11
jrosserand I didn’t yet figure out why tempest fails07:12
jrosseralso version of some repos we use might be wrong, perhaps07:12
noonedeadpunk++07:12
jrosserwe are lucky that mariadb already have bookworm packages though07:12
noonedeadpunkwe also kinda need to close that before merging anything else I guess https://review.opendev.org/q/topic:osa/core-2.15+status:open07:13
noonedeadpunkas we keep breaking linters with new patches...07:14
noonedeadpunks/we/I/07:14
jrosserhmm some wierd sphinx thing too07:32
noonedeadpunkYeah, I'm on it07:33
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: [doc] Fix deployment guide renderring  https://review.opendev.org/c/openstack/openstack-ansible/+/89576907:54
noonedeadpunkthis should fix it ^07:54
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Bump ansible-core to 2.15.3 and ansible-lint  https://review.opendev.org/c/openstack/openstack-ansible/+/89237107:55
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Bump ansible collection versions  https://review.opendev.org/c/openstack/openstack-ansible/+/89237308:11
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Bump ansible collection versions  https://review.opendev.org/c/openstack/openstack-ansible/+/89237308:12
jrossernoonedeadpunk: yes https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/894740 is really wierd failure09:01
jrosseri did the same and looked in the apt history and it makes no sense09:02
noonedeadpunkIt looked like there's simply no 10.11.5 in repos09:02
noonedeadpunkand then apt jsut installs from the default repo09:02
noonedeadpunkas we're using infra cache09:03
noonedeadpunkbut it's there: http://mirror.iad3.inmotion.opendev.org:8080/MariaDB/mariadb-10.11.5/repo/ubuntu/dists/jammy/main/binary-amd64/09:04
noonedeadpunkand then - it's not failing for non-upgrade....09:05
noonedeadpunkAlso regarding mariadb - we've spotted some difference in how incremental backups are made - while it was couple of seconds previously when backup made member to have syncing state (and causing haproxy to mark backend as down), now it's couple of minutes09:07
noonedeadpunkAlso according to incremental filesize, it looks like incremental backup is made from last full always, rather then from previous incremental... But it could be some change to the script actually, rather then mariabackup itself09:08
jrosserthe version string is `1:10.11.5+maria~ubu2204`09:20
jrosseri guess 10.6. beats 10.11. in alphanumeric sort09:22
jrosserif there is a broken version compare somewhere (/me looks at mad code in ansible apt module)09:23
noonedeadpunkMaybe that won't be a thing with core 2.15?:)09:24
noonedeadpunkbut that would be.... doh09:24
noonedeadpunkI believe they use python3-apt for comparing versions09:25
jrosserperhaps this needs running locally and just seeing what apt-policy says09:26
noonedeadpunkyeah09:26
noonedeadpunkif that is possible to reproduce at all09:26
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: [doc] Fix deployment guide renderring  https://review.opendev.org/c/openstack/openstack-ansible/+/89576909:45
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: [doc] Update deploy guide with relevant roles  https://review.opendev.org/c/openstack/openstack-ansible/+/89577909:55
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Bump ansible-core to 2.15.3 and ansible-lint  https://review.opendev.org/c/openstack/openstack-ansible/+/89237110:11
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Bump ansible collection versions  https://review.opendev.org/c/openstack/openstack-ansible/+/89237310:11
noonedeadpunkdeploy guide should be fixed10:39
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-rabbitmq_server master: Update RabbitMQ to 3.12 series  https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/89580410:59
noonedeadpunkfwiw, it seems I don't catch same issue with debian 12 as we see in CI11:12
noonedeadpunkwheels are built and stored properly11:12
noonedeadpunkalso weird, that wheels build has failed only on Cinder but not before in CI11:13
jrossernoonedeadpunk:  you mean the ansible version number thing?11:15
noonedeadpunkyup11:15
jrosseri think i have a held node for this11:15
jrosserthough i have forgotton.......11:16
jrosserfrickler: you did a held node for me last week - can you remind me what it's set on?11:16
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Switch classic queues to version 2  https://review.opendev.org/c/openstack/openstack-ansible/+/89580611:17
jrosseroh hmm i wonder if thats no longer existing https://zuul.opendev.org/t/openstack/autoholds11:17
noonedeadpunkoh... there's a new authold tab in zuull... that's nice11:17
noonedeadpunkso you alse didn't saw anything weird wrt debian 12 locally?11:19
noonedeadpunk* also didn't see11:19
fricklerjrosser: iirc that was for a patch which then stopped failing? anyway, let me know if you need anything new set up11:19
noonedeadpunkshould ave been for this one I guess https://review.opendev.org/c/openstack/openstack-ansible/+/89456111:20
noonedeadpunkmmmm... regarding tempest...11:24
noonedeadpunkI _think_ Debian still has unpatches version of OVS that doesn't respect hostname11:24
noonedeadpunkopenvswitch-common=3.1.0-211:24
noonedeadpunkI mean this one https://bugs.launchpad.net/cloud-archive/+bug/201775711:26
noonedeadpunkzigo: hey! do you know if it's possible to minorly update ovs on bookworm to include bugfix of this ^ ?11:26
noonedeadpunkbasically that's the patch https://mail.openvswitch.org/pipermail/ovs-dev/2023-March/403255.html11:27
noonedeadpunkAs I would kinda hate to return workaround specifically for debian 1211:28
noonedeadpunk(https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/891453)11:29
zigonoonedeadpunk: I can backport patches, but hardly can package a new version. What patch do you need in?11:30
zigoI can easily put a patched version in my unofficial debian.net backport repos though...11:31
noonedeadpunkbasically this one https://github.com/openvswitch/ovs/commit/037e2d916146140d14f12dbd34070a07755eaa3e11:31
noonedeadpunkit's already in ovs 3.1.1 iirc11:32
noonedeadpunkyup, it's in 3.1.111:32
noonedeadpunkbecause it makes append command to overwrite things11:33
noonedeadpunkwhich is really nasty, especially when it resets node hostnames on service restart, for example11:34
noonedeadpunkBut I'm pretty sure there're more unexpected behaviour11:34
noonedeadpunkboth ubuntu and redhat already patched/backported fix for that just in case :)11:37
jrossernoonedeadpunk: locally my debian12 got as far as tempest11:40
* jrosser looks for the vm11:40
jrossernoonedeadpunk: https://paste.opendev.org/show/b212irj0QBWH1qyBoPGw/11:44
noonedeadpunkyup, same for me. And on tempest it fails due to buggy ovs11:50
noonedeadpunksame bug that was on ubuntu and centos11:50
noonedeadpunkthat's really weird. I wonder if dib has smth to do with that11:51
noonedeadpunkThough I've reproduced mariadb upgrade failure11:55
noonedeadpunkhttps://paste.openstack.org/show/bGr77uEc35xBCmfumSny/11:56
noonedeadpunkAnd I'm pretty much confused when/how 10.6 got installed11:56
jrosseri wonder what `TASK [galera_server : Add galera repo]` actually did11:57
jamesdentongood morning13:03
teresaHi13:07
jrossero/ hello13:07
teresaIs there any way to get rid of 'macaddress' in the instances launched on ubuntu cloud image? It makes difficulty when you go to exchange interfaces on the customer instance13:08
teresaI mean: https://i.imgur.com/KkCrH64.png13:08
teresathe "match" section is generated by cloud-init and if you manually delete it, it will work correctly.13:09
jamesdentonthat mac address corresponds to the neutron port assigned to the VIF13:13
jamesdentoni imagine if you're swapping interfaces/ports out that you'll need to somehow update the netplan file accordingly13:13
teresaIt makes difficulty when you go to exchange interfaces on the customer instance "as the macaddress is changed and then you lost the connection to the server!"13:14
teresajamesdenton: but the server continue to work well if I delete this lines: match, macaddress, set_name13:15
jamesdentonwhat's the reason for changing the interface of the instance? just curious13:15
jamesdentonyou can try creating user-data for cloud-init that changes the netplan template to not include the mac address. for a single-homed instance it's probably OK. 13:17
teresato change the network attached to instance. We purchacse a tenant to the customer. They change interfaces (deleteing and adding interfaces) time to time for different reason, especially in their dev env13:17
teresajamesdenton: "mac address corresponds to the neutron port assigned to the VIF" --> does it have any security reason? I mean is it ok to remove macaddress from the point of security?13:19
jamesdentoni only ask because there may be better ways of accomplishing the end goal. But if that is the way they want to go about it, you'll likely want to launch the instance with custom network definition via metadata. See https://cloudinit.readthedocs.io/en/22.3/topics/network-config-format-v2.html on the format13:20
jamesdenton"I mean is it ok to remove macaddress from the point of security?" -> Yes, it's OK to remove the MAC address inside the netplan config. The purpose is to match the MAC address to that particular interface name (ie. eno3). If you had multiple interfaces it would make more sense, for consistency. For single NIC instance it's not really necessary13:21
jamesdentonIn terms of security, the instance is limited to sending traffic as the MAC address of the port/VIF connected to it. This is called 'port security' and there are default rules13:22
teresaThanks a lot13:25
jamesdentonanyone here using the elk_metrics_7x repo?13:53
andrewbonneyWe are, although we tend to update it very infrequently13:59
jrosserwe tend to have a huge bash at it every time we do some upgrade work on our elk stuff14:02
andrewbonneyYes, bash is probably a good way to describe it...14:03
jrosserthough i think we did get someone else here in irc using it fairly recently, i think i patched some stuff which had rotted a bit14:03
jrosserlike versions of things to download being a bit 404 and the like14:03
jrosserjamesdenton: to clarify, we use it for logs specifically14:04
jrosserthe rest, not so much14:04
jrosseras much as elastic would tell you a document database can store a time series, reality says a bit different14:04
jamesdentoni've seen the commits to it over time but wasn't sure how it was used. thanbks14:12
noonedeadpunkI got my laptop dead. Can somebody start the meeting?:)15:00
noonedeadpunk#startmeeting openstack_ansible_meeting15:05
opendevmeetMeeting started Tue Sep 19 15:05:06 2023 UTC and is due to finish in 60 minutes.  The chair is noonedeadpunk. Information about MeetBot at http://wiki.debian.org/MeetBot.15:05
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:05
opendevmeetThe meeting name has been set to 'openstack_ansible_meeting'15:05
noonedeadpunk#topic rollcall 15:05
noonedeadpunko/15:05
mgariepyhey15:05
mgariepyhalf there.15:05
noonedeadpunkSorry folks, I got my laptop dead couple of hours ago, so will be pretty much away until will get a new one15:05
noonedeadpunkIn the meanwhile - we can catch up on what's going on15:06
jamesdentono/15:07
jrossero/ hello15:07
noonedeadpunk# topic office hours15:09
noonedeadpunkUgh15:09
noonedeadpunk#topic office hours 15:09
noonedeadpunkSo, we had good progress on merging ansible-core 2.15, but I've pushed patches that break linters again15:10
noonedeadpunkSo some more reviews are needed15:10
noonedeadpunkAnd maybe we should prevent from merging other things until then?15:10
noonedeadpunkAs it starts to be annoying...15:11
jrosserit is these https://review.opendev.org/q/topic:osa%252Fcore-2.1515:13
noonedeadpunkYes15:13
noonedeadpunkWe also have deploy guide job broken now15:13
noonedeadpunkBut, seems the fix works15:14
noonedeadpunkAha, it's already in gates15:15
noonedeadpunkThen we need to decide on PTG time15:15
noonedeadpunkI'd suggest October 2415:16
noonedeadpunk(it's Tuesday)15:17
mgariepywould work for me.15:18
noonedeadpunkSmth like 14-17 UTC?15:18
noonedeadpunkOr 15-1815:18
jrosseroct 24 looks ok15:18
mgariepyeither of the time are ok with me.15:19
noonedeadpunkAwesome 15:21
noonedeadpunkI will book a time slot then15:21
noonedeadpunkDo we want to have an operator hour? As this seems to not work in fact15:21
noonedeadpunkWill also create an etherpad for topics15:22
noonedeadpunk#action noodeadpunk to book ptg slot and create an etherpad 15:22
noonedeadpunkRight before my laptop went down, I was trying to come up with cycle highlights15:23
noonedeadpunkI was about to mention: Debian 12, Ansible core 2.15, rabbitmq 3.12 and quorum queues 15:24
noonedeadpunkWas thinking about skyline, but I put zero effort there as of today15:25
noonedeadpunkAnything else worth to mention?15:25
noonedeadpunkI hate timing when we should push these, but we have what we have...15:25
jrosserfeels somehow like we spend most of this cycle stabilising 2023.115:26
noonedeadpunkYeah15:26
noonedeadpunkAnd it was only partially our fault15:26
opendevreviewMerged openstack/openstack-ansible master: [doc] Fix deployment guide renderring  https://review.opendev.org/c/openstack/openstack-ansible/+/89576915:27
noonedeadpunkA lot of keystone regressions as well15:27
noonedeadpunkThough, might be this time we release in time...15:27
jrosserquorum queues is probably the big thing to get completed15:28
jrosserthere is also some test improvements, i fixed a load of mess in os_tempest for more complex scenarios15:29
jrossernot sure that is all merged15:29
noonedeadpunkYeah, not sure it fits in highlights...15:29
noonedeadpunkAs not sure plenty of ppl care about that15:29
jrosserlikley not15:30
noonedeadpunkWill add skyline I guess... still plenty of time to work on that 😅15:30
jrossergood luck15:32
jrosseri think there is 'interesting' stuff to unpick there15:32
noonedeadpunkYeah... I can imagine...15:32
noonedeadpunkBut it was asked quite a lot15:32
noonedeadpunkSo worth trying15:33
jrosserthough actually we might have made it all a bit easier with haproxy maps now15:33
noonedeadpunkAh. Also, do we want to enable quorum queues by default now?15:33
noonedeadpunkAs this is smth that teases me15:33
jrosserif we can make that work on 3.x then the upgrade to 4.x would be smooth15:33
jrosserlike switch to quorum independantly of the major version upgrade of rabbitmq15:34
noonedeadpunkYeah, that's true. I guess it works, in general, but not sure about defaults 15:34
noonedeadpunkThey're quite some improvements to oslo regarding that from OVH15:34
noonedeadpunkThat are not merged yet15:34
noonedeadpunkhttps://review.opendev.org/q/topic:bug-203149715:36
noonedeadpunkAnd they should make quite a difference imo15:37
noonedeadpunkSo maybe we should disable quorum by default once we merge the topic and add another job to check that?15:37
noonedeadpunkOr dunno15:38
noonedeadpunkReally afraid after OVN 15:38
jrosserthat would be lower risk15:39
jrosserwe could also switch to quorum pretty quick after we release15:40
noonedeadpunkYeah, that as well15:40
jrosserlike get the code and default setting merged before release15:40
jrosserthen switch the default afterwards15:40
noonedeadpunkWell, not sure how much lower, given that rabbit is a nasty thing on it's own15:40
noonedeadpunkAnd you can understand if it's working or not only at scale at day2 ops15:41
noonedeadpunkOk, sounds like a plan then15:42
noonedeadpunkThat's all for me, does anybody has smth to discuss?15:42
jrosserwe have some bugs15:45
jrosserlike https://bugs.launchpad.net/openstack-ansible/+bug/203471415:46
jrosseri can only think that they have overridden `memcached_servers` there to point to the internal VIP15:50
noonedeadpunkYeah, we have doc for that for "balancing" memcached through haproxy15:53
noonedeadpunkNo idea why that would cause such issue..15:54
noonedeadpunk But maybe it relies on a cache in a way...15:54
noonedeadpunkWorth checking that for sure15:54
noonedeadpunkOr well, asking about that at least15:54
noonedeadpunk#endmeeting15:59
opendevmeetMeeting ended Tue Sep 19 15:59:22 2023 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:59
opendevmeetMinutes:        https://meetings.opendev.org/meetings/openstack_ansible_meeting/2023/openstack_ansible_meeting.2023-09-19-15.05.html15:59
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/openstack_ansible_meeting/2023/openstack_ansible_meeting.2023-09-19-15.05.txt15:59
opendevmeetLog:            https://meetings.opendev.org/meetings/openstack_ansible_meeting/2023/openstack_ansible_meeting.2023-09-19-15.05.log.html15:59
opendevreviewMerged openstack/openstack-ansible-openstack_hosts master: Define mode for journald configuration file  https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/89568117:31
Karni"As default behaviour is that traffic is being processed by 1 core, Multi-queue virtio-net do release this. It provides an approach that scales the network performance as the number of vCPUs increases, by allowing them to transfer packets through more than one virtqueue pair at a time. (Ref: RedHat Docs)" 21:22
KarniBy setting either hw_vif_multiqueue_enabled per image in its properties or hw:vif_multiqueue_enabled per flavor (Yoga+). But is there any way to apply on an existing instanc?.21:22

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