Wednesday, 2023-12-06

opendevreviewMerged openstack/kayobe master: Pin ansible-core at 2.15.6  https://review.opendev.org/c/openstack/kayobe/+/90261401:21
opendevreviewwu.chunyang proposed openstack/kolla-ansible master: CI: Trove: Create a database instance  https://review.opendev.org/c/openstack/kolla-ansible/+/86352102:33
opendevreviewwu.chunyang proposed openstack/kolla-ansible master: CI: Trove: Create a database instance  https://review.opendev.org/c/openstack/kolla-ansible/+/86352106:01
opendevreviewwu.chunyang proposed openstack/kolla-ansible master: CI: Trove: Create a database instance  https://review.opendev.org/c/openstack/kolla-ansible/+/86352107:44
opendevreviewPierre Riteau proposed openstack/kayobe master: Clean up release notes for Bobcat  https://review.opendev.org/c/openstack/kayobe/+/90275008:38
SvenKieskedcapone2004: interesting, maybe we could add a precheck if there are already certs in place not managed by kolla? ask if the user is sure they want to override them?09:06
SvenKieskedcapone2004: maybe worth it to file an issue09:36
SvenKieskemnasiadka, frickler: is this a kayobe only issue with ansible-core 2.15.7? https://review.opendev.org/c/openstack/kayobe/+/902614 should we pin in kolla{-ansible} as well?09:38
mnasiadkaI haven't seen any breakage in kolla-ansible - so why pin?09:41
opendevreviewMaksim Malchuk proposed openstack/kayobe stable/yoga: docs: add information about enabling TLS to automated guide  https://review.opendev.org/c/openstack/kayobe/+/90222809:43
opendevreviewMaksim Malchuk proposed openstack/kayobe stable/zed: docs: add information about enabling TLS to automated guide  https://review.opendev.org/c/openstack/kayobe/+/90222709:49
opendevreviewwu.chunyang proposed openstack/kolla-ansible master: CI: Trove: Create a database instance  https://review.opendev.org/c/openstack/kolla-ansible/+/86352109:59
opendevreviewMerged openstack/kayobe master: Use OpenStack 2023.2 release  https://review.opendev.org/c/openstack/kayobe/+/90242210:59
opendevreviewMerged openstack/kayobe master: Clean up release notes for Bobcat  https://review.opendev.org/c/openstack/kayobe/+/90275010:59
SvenKieskeI guess I'm just not seeing it, but can someone spot the actual error here? https://zuul.opendev.org/t/openstack/build/6ded685cf6ab4dce8816a77155b463be/log/primary/logs/ansible/test-core-openstack?severity=012:49
mnasiadkahttps://8aae34bf932c24fa9223-2b94732b4d1115584c793b46f27b760e.ssl.cf5.rackcdn.com/902699/1/check/kolla-ansible-rocky9-upgrade-cephadm/6ded685/primary/logs/ansible/test-core-openstack-upgrade12:57
mnasiadkahere12:57
mnasiadkawe could be better on saying it's on upgrade :D12:58
SvenKieskeare the logs in the zuul UI truncated? Because I don't have that failure when looking at the log via the above link..weird13:02
SvenKieskeit ends on line 2400 for me, can't scroll further13:02
jangutterno that file really ends there13:08
jangutterbut yeah, the actual error isn't in that file. the breadcrumbs are here to the real log: https://zuul.opendev.org/t/openstack/build/6ded685cf6ab4dce8816a77155b463be/console#2/3/14/primary 13:11
jangutterhttps://zuul.opendev.org/t/openstack/build/6ded685cf6ab4dce8816a77155b463be/log/primary/logs/ansible/test-core-openstack-upgrade#412 volume was still in status `creating`13:12
mnasiadkaand cinder-volume probably again has problems with RMQ13:21
mnasiadkaso we're doing something wrong in the upgrade RMQ handling - or cinder is special :D13:21
mnasiadka2023-12-05 19:15:23.760 1088 WARNING oslo.messaging._drivers.impl_rabbit [None req-cbce7ede-4fdc-41ca-829c-7e9fa1a3b044 - - - - - -] Force creating a non durable exchange.: amqp.exceptions.PreconditionFailed: Exchange.declare: (406) PRECONDITION_FAILED - inequivalent arg 'durable' for exchange 'openstack' in vhost '/': received 'true' but current is 'false'13:39
mnasiadkalooks like something is wrong on rmq side13:40
opendevreviewRafal Lewandowski proposed openstack/kayobe master: Add Redfish rules to Ironic introspection  https://review.opendev.org/c/openstack/kayobe/+/90277213:42
mnasiadkajangutter, SvenKieske: for some reason kolla-ansible stop does not work for cinder role13:52
mnasiadkalook at https://8aae34bf932c24fa9223-2b94732b4d1115584c793b46f27b760e.ssl.cf5.rackcdn.com/902699/1/check/kolla-ansible-rocky9-upgrade-cephadm/6ded685/primary/logs/ansible/stop - search "apply role cinder"13:52
mnasiadkamgoddard mnasiadka bbezak frickler kevko SvenKieske mmalchuk gkoper jangutter jsuazo jovial osmanlicilegi - meeting in 7 minutes13:53
SvenKieskeinteresting13:55
janguttermnasiadka skipped because of tags or something else? Thats site.yaml right?13:56
janguttermnasiadka: ah  --tags heat,keystone,neutron,nova <--- need to add cinder there?13:56
mnasiadkaah, so upgrade.sh is borked13:57
mnasiadkahttps://github.com/openstack/kolla-ansible/blob/af6e1ca4fd3366c1ae936d225b2835e95a4191fc/tests/upgrade.sh#L2113:57
mnasiadkain theory it should add cinder13:57
mnasiadkabut it's not13:57
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: CI: Add SCENARIO env var to upgrade.sh  https://review.opendev.org/c/openstack/kolla-ansible/+/90277613:59
mnasiadka^^ that should fix it13:59
mnasiadkathanks jangutter :)13:59
mnasiadka#startmeeting kolla14:00
opendevmeetMeeting started Wed Dec  6 14:00:00 2023 UTC and is due to finish in 60 minutes.  The chair is mnasiadka. Information about MeetBot at http://wiki.debian.org/MeetBot.14:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:00
opendevmeetThe meeting name has been set to 'kolla'14:00
mnasiadka#topic rollcall14:00
frickler\o14:00
mnasiadkao/14:00
SvenKieskeo/14:00
osmanlicilegio/14:00
janguttero/14:01
jovialo/14:01
mnasiadka#topic agenda14:03
mnasiadka* CI status14:03
mnasiadka* Release tasks14:03
mnasiadka* Regular stable releases (first meeting in a month)14:03
mnasiadka* Current cycle planning14:03
mnasiadka* Additional agenda (from whiteboard)14:03
mnasiadka* Open discussion14:03
mnasiadkaosmanlicilegi: welcome back ;-)14:03
mnasiadka#topic CI status14:04
osmanlicilegimnasiadka: thanks :) I will be back after long and tiring years.14:04
mnasiadkaI think it's overall green, with the cephadm-upgrade jobs failing because of the RMQ switch to Quorum queues14:04
mnasiadkabut that's being fixed with 90277614:04
mnasiadka#link https://review.opendev.org/c/openstack/kolla-ansible/+/90277614:04
mnasiadka#topic Release tasks14:05
mnasiadkaKolla, Kolla-Ansible and ansible-collection-kolla going for a final Bobcat release14:05
mnasiadka#link https://review.opendev.org/c/openstack/releases/+/90271814:05
mnasiadkaKayobe still waiting for rc1 - last patch merging14:06
mnasiadkajovial: will you create a patch for kayobe repos rc1?14:06
mnasiadkawell, I guess he has no other choice - worst case scenario I'll do it14:07
mnasiadkalet's move forward14:07
mnasiadka#topic Current cycle planning14:07
mnasiadkaSo, usually we did have a stability phase - so we did stay with building latest stable release (Bobcat) in Kolla14:08
jovialmnasiadka: sure, I can try after the meeting14:08
mnasiadkajust to have some calm weeks14:08
mnasiadkaBut I'm fine with us reverting to master faster to have a look if this breaks us badly14:09
mnasiadkaAny opinions?14:09
fricklerI'd be fine with waiting until January14:09
mnasiadka#link https://docs.openstack.org/kolla/latest/contributor/release-management.html#early-cycle-stability14:09
mnasiadkamakes sense, let's at least have stable gate over christmas period14:09
mnasiadkashould we prepare patches that need reverting in a gerrit topic - so it's easy to revisit them in a month time?14:10
SvenKieskeI'm fine with either. a topic seems nice14:11
mnasiadkaI think so - if there's a volunteer - I would be happy ;)14:11
mnasiadkaok then14:12
mnasiadkaAny other obvious issues that we need to fix in freshly released Bobcat?14:12
mnasiadkaAh, the horizon policy thing?14:12
SvenKieskeyeah14:13
mnasiadka#link https://review.opendev.org/q/topic:%22fix-horizon-role%2214:13
mnasiadkahttps://review.opendev.org/c/openstack/kolla-ansible/+/901761 - seems upgrade jobs are failing here14:13
fricklermnasiadka: yes, the fix needs to be backported first14:14
mnasiadkamakes sense14:14
mnasiadka#link https://review.opendev.org/c/openstack/kolla-ansible/+/90265914:14
mnasiadkaLooking for another +214:14
mnasiadkaupgrade-cephadm is known for failing now14:15
mnasiadkaok then14:15
mnasiadkathose two on the radar and let's see if anything else shows up14:16
mnasiadkaI'll finally populate Whiteboard with cycle priorities - so we can discuss next week14:16
fricklerI'll be testing things downstream, image builds are fine already14:16
mnasiadkaI'd also like to have a go at trying to look through all kolla-ansible reviews and abandon very old reviews and try to review things that would be useful14:17
mnasiadkakolla looks quite nice with only a couple of patches in progress14:18
fricklersad thing is a lot of things likely are indeed still useful and relevant. but yes, that would be a good task14:18
mnasiadkathere's a lot that are ready or nearly ready - we just lack capacity to review them14:18
mnasiadkaso I'll try to make them tidy and maybe RP+1 those that are easy to get in14:19
mnasiadkaand fairly small14:19
SvenKieskehappy to do some additional reviews14:19
mnasiadkaok then, let's move on14:19
mnasiadka#topic Additional agenda (from whiteboard)14:19
mnasiadkamattcrees: https://etherpad.opendev.org/p/kolla-ansible-rmq-quorum-queues-proposal14:20
mnasiadkaI think we already violated that proposal with turning quorum queues as default in B14:20
mnasiadkaquestion if we need more prechecks14:20
mnasiadkamaybe such prechecks, that would check if a queue is non-durable (if it exists)14:20
mnasiadkabut mattc is not here14:20
mnasiadkaI'll leave that for next time14:21
mnasiadkajsuazo also is reminding us about two reviews - but I would prefer to wait until we get final Bobcat release out until we merge new features14:21
mnasiadkaso both topics are left for next week14:22
jovialmattc reckons we need the same style of precheck that we added previously for HA queues. He will try an propose something this week.14:22
mnasiadkaWell, basically I would be happy if we could have prechecks that really stop kolla-ansible break services - so happy to review once it gets into Gerrit14:23
SvenKieskeseems you have summoned someone :)14:24
mattcreeso/ 14:24
mnasiadkawell, that's the PTL role I guess14:24
mnasiadkamattcrees: so, do your planned prechecks plan to check the ,,durability'' of the existing queues before starting services?14:25
mnasiadkacurrently our cephadm-upgrade jobs are broken, because we're not stopping cinder, and it tends to complain about non-durable queues that were left behind14:25
mattcreesYes, the plan is to have a check similar to this: https://github.com/openstack/kolla-ansible/blob/af6e1ca4fd3366c1ae936d225b2835e95a4191fc/ansible/roles/rabbitmq/tasks/precheck.yml#L21114:26
mattcreesOnly for quorum queue type instead of durability type14:26
mnasiadkaok, let's see how that works14:27
mnasiadkaSo let's move to open discussion14:27
mnasiadka#topic Open discussion14:27
mattcreesWe should be stopping cinder though... https://github.com/openstack/kolla-ansible/blob/af6e1ca4fd3366c1ae936d225b2835e95a4191fc/tests/upgrade.sh#L22C20-L22C2014:27
mnasiadkaosmanlicilegi: you can now share what made you miss us ;-)14:27
mnasiadkamattcrees: we are not, because SCENARIO env var is not passed to upgrade.sh14:27
mnasiadkabut it will be - https://review.opendev.org/c/openstack/kolla-ansible/+/90277614:28
mattcreesah ok makes sense14:28
fricklerthe wsgi scripts discussion is worth watching, but I think we can sit and wait how it works out14:28
frickler#link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/message/5RQGZZMCKKHAVH4OLSJCLBQNIHYDPGZD/14:28
mnasiadkayeah, seen that14:28
mnasiadkaI think we still have some services to move to wsgi14:29
mnasiadkastarting with Neutron14:29
osmanlicilegimnasiadka: very long story :) I will share it via dm.14:29
mnasiadkaosmanlicilegi: ok then :)14:29
fricklerthe eventlet issue is also interesting, but even less we can do about it14:29
SvenKieskedo you mean wsgi or uWSGI?14:29
mnasiadkabut we could use some more review cycles - so osmanlicilegi feel free to do reviews if you have time :)14:30
fricklerSvenKieske: it affects both in a way14:30
mnasiadkabut let's not merge any features in master until next weekly meeting please14:30
osmanlicilegimnasiadka: I will start soon reviewing :)14:32
SvenKieskecool :)14:32
mnasiadkafrickler: ubuntu 24.04 is going to be released in April, we might need to add support for it last minute in C...14:33
mnasiadkabut I hope not :)14:33
SvenKieske:D14:34
fricklerhmm, I'd rather backport that later14:34
SvenKieskeis it possible to test pre releases?14:34
fricklernot in the opendev ci14:34
mnasiadkafrickler: we'll still have time from April + 3 months, so... I reckon last minute addition14:36
fricklerok, but first we need to see if ubuntu keeps releasing on time ;)14:37
mnasiadkayup14:37
mnasiadkaI'll add that to the priorities list with a note to "look out for it"14:38
mnasiadkaOk then, let's finish for today14:38
mnasiadkaAh, one last thing14:38
mnasiadkawhen do we want to stop meetings for this year?14:38
mnasiadkaI planned to cancel the 20th and 27th of Dec and 3rd Jan14:39
fricklerI'll be around anyway14:39
mnasiadkaI'll be around as well, but I think majority of people will be off14:39
mnasiadkaSo I'll cancel those three and we can just raise things on the channel outside of the meeting14:39
fricklerack14:40
mnasiadkaThanks for coming, talk to you next week :)14:40
mnasiadka#endmeeting14:40
opendevmeetMeeting ended Wed Dec  6 14:40:08 2023 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:40
opendevmeetMinutes:        https://meetings.opendev.org/meetings/kolla/2023/kolla.2023-12-06-14.00.html14:40
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/kolla/2023/kolla.2023-12-06-14.00.txt14:40
opendevmeetLog:            https://meetings.opendev.org/meetings/kolla/2023/kolla.2023-12-06-14.00.log.html14:40
SvenKieskeI'll be away from roughly 24th of Dec til 8th of Jan.14:40
fricklerthx mnasiadka 14:40
SvenKieskethanks mnasiadka14:40
opendevreviewMerged openstack/kayobe master: Remove unused kolla bootstrap-servers variables  https://review.opendev.org/c/openstack/kayobe/+/90237915:31
opendevreviewMerged openstack/kolla-ansible master: Fix broken list concatenation in horizon role  https://review.opendev.org/c/openstack/kolla-ansible/+/90265915:58
opendevreviewMatt Crees proposed openstack/kolla-ansible master: Add precheck for RabbitMQ quorum queues  https://review.opendev.org/c/openstack/kolla-ansible/+/90278716:09
opendevreviewMatt Crees proposed openstack/kolla-ansible master: RabbitMQ: correct docs on Quorum Queue migrations  https://review.opendev.org/c/openstack/kolla-ansible/+/90278916:11
jovialmnasiadka, Does this look OK: https://review.opendev.org/c/openstack/releases/+/901950?16:13
suloHi, new to Kolla release cycle, so quick question, is Kolla skipping Antelope and going straight to Bobcat. I cant find any 2023.1 tags on docker hub. 16:17
opendevreviewMatt Crees proposed openstack/kolla-ansible master: Add precheck for RabbitMQ quorum queues  https://review.opendev.org/c/openstack/kolla-ansible/+/90278716:25
SvenKieskesulo_: no we are not skipping releases, afaik we stopped publishing to dockerhub, but my memory is fuzzy there16:32
sulo_ok, where can i find the images ?16:33
sulo_i see 2023.2 tags in docker hub https://hub.docker.com/r/kolla/nova-libvirt/tags?page=1&name=202316:33
SvenKieskegood question, it was discussed during last PTG: https://etherpad.opendev.org/p/kolla-caracal-ptg#L14616:36
mnasiadkajovial: sure16:36
SvenKieskemost people here are building their own images, that's why I'm actually not that familiar with the current status. I thought we push to quay.io, but I can't find newer release tags there either16:37
SvenKieskemaybe mnasiadka nows ;)16:37
mnasiadkaWe push to quay.io/openstack.kolla/16:38
SvenKieskeyeah, but I'm blind or I dont see any 2023.1|antelope tag there?16:38
SvenKieskemhm, seems I got bad luck: here are newer tags: https://quay.io/repository/openstack.kolla/nova-compute?tab=tags16:39
SvenKieskebut this one only has old tags? https://quay.io/repository/openstack.kolla/ubuntu-source-nova-libvirt16:40
SvenKieskeah I guess we omit the "-source" part now, don't we?16:40
SvenKieskemhm no, we don't16:42
mattcreesSvenKieske: we drop the ubuntu-source now :)  https://quay.io/repository/openstack.kolla/nova-libvirt?tab=tags16:44
SvenKieskeah right, it was this way around16:45
sulo_@SvenKieske thank you.16:48
sulo_Is there somewhere i can check to see what the current stable release branch is ?16:49
SvenKieskeno problem, thank mattcrees and mnasiadka :)16:49
sulo_yes thanks all :)16:49
SvenKieskegit ls-remote --refs https://opendev.org/openstack/kolla-ansible.git "refs/heads/stable/*"  16:53
SvenKieskegives you all stable branches without cloning the repo first16:54
SvenKieskenewest would be newest numeric one under current release naming rules16:54
SvenKieskeEOL/unmaintained releases don't show up there.16:55
sulo_perfect. thanks.17:20
opendevreviewMatt Crees proposed openstack/kolla-ansible master: Add precheck for RabbitMQ quorum queues  https://review.opendev.org/c/openstack/kolla-ansible/+/90278717:23
opendevreviewDr. Jens Harbott proposed openstack/kolla-ansible stable/2023.2: Fix broken list concatenation in horizon role  https://review.opendev.org/c/openstack/kolla-ansible/+/90282117:34
opendevreviewDr. Jens Harbott proposed openstack/kolla-ansible stable/2023.1: Fix broken list concatenation in horizon role  https://review.opendev.org/c/openstack/kolla-ansible/+/90282217:35
fricklertechnically 2023.1 is still the latest stable release, even though the stable/2023.2 branch already exists17:38
fricklermnasiadka: do we want to get the horizon fix backport right into the kolla-ansible final release? or do a .1 release right afterwards?17:39
fricklerwe don't have any +2 on the release patch yet, so we could still update17:40
SvenKieskere: "latest" stable release: I mean, we tell people all the time to use the git branches and not pay attention to pypi releases, so isn't this a little bit contradictionary?17:42
fricklerwell that's only after the initial release has been made. it is also not about pypi, but about having a non-rc tag on the branch17:45
SvenKieskemhm, can't reliably list the tags via git ls-remote in a way that would make this easily scriptable, anyway.17:49
mnasiadkafrickler: so lets merge horizon and update18:47
mnasiadkaI think we should do monthly releases without thinking too much about it, e.g. don’t really care if something is broken or not, we can always raise bugfix release between monthly releases18:50
mnasiadkaBut I also think release team review capacity is a bit… not what it used to be.18:51
mnasiadkafrickler: coming back to wsgi - seems we should start thinking of ditching mod_wsgi…20:30
-opendevstatus- NOTICE: The Gerrit service on review.opendev.org will be offline momentarily to restart it onto an updated replication key21:36
opendevreviewMerged openstack/kolla-ansible stable/2023.2: Fix broken list concatenation in horizon role  https://review.opendev.org/c/openstack/kolla-ansible/+/90282121:52

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