Wednesday, 2023-11-15

opendevreviewPetr Slavchenkov proposed openstack/kolla-ansible master: Fix proxysql get mariadb_backup_database_password  https://review.opendev.org/c/openstack/kolla-ansible/+/90097907:58
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: WIP: Run Tempest in test-core-openstack  https://review.opendev.org/c/openstack/kolla-ansible/+/87882608:42
opendevreviewPetr Slavchenkov proposed openstack/kolla-ansible master: Fix proxysql get mariadb_backup_database_password  https://review.opendev.org/c/openstack/kolla-ansible/+/90097908:55
opendevreviewPetr Slavchenkov proposed openstack/kolla-ansible master: Fix proxysql get mariadb_backup_database_password  https://review.opendev.org/c/openstack/kolla-ansible/+/90097908:59
opendevreviewMichal Nasiadka proposed openstack/ansible-collection-kolla master: CI: add podman jobs  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/89946812:00
opendevreviewMichal Nasiadka proposed openstack/ansible-collection-kolla master: CI: add podman jobs  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/89946812:01
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: WIP: Run Tempest in test-core-openstack  https://review.opendev.org/c/openstack/kolla-ansible/+/87882612:10
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: WIP: Run Tempest in test-core-openstack  https://review.opendev.org/c/openstack/kolla-ansible/+/87882612:10
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: WIP: Run Tempest in test-core-openstack  https://review.opendev.org/c/openstack/kolla-ansible/+/87882612:31
mnasiadkafrickler: is TC thinking what to do with Masakari, or are we just waiting for the next cycle and lack of PTL?12:41
fricklermnasiadka: I'm not sure, gmann is currently handling all the pending PTL things. if it is currently working for kolla as far as CI coverage goes, I'm fine with deferring to next cycle12:50
mnasiadkaWell, bobcat is failing, but since there are users - I'm not going to raise the issue now :)12:50
mnasiadkaso let's wait if the situation will resolve12:51
mnasiadkahttps://review.opendev.org/q/project:openstack/masakari+is:open+branch:stable/2023.212:51
mnasiadkathis looks just fantastic12:51
fricklerwell since it is failing in kolla since over a month, we could add a deprecation warning in bobcat, that would make it easier to drop next cycle and also alarm possible consumers12:53
frickler"because CI jobs are failing and no contributors are around to fix it" something like that?12:53
fricklerhttps://zuul.opendev.org/t/openstack/builds?job_name=kolla-ansible-ubuntu-masakari&project=openstack/kolla-ansible12:54
SvenKieskeseems reasonable12:54
SvenKieskebtw should we backport https://review.opendev.org/c/openstack/kolla-ansible/+/881707 ?12:54
fricklerthen if someone cares about the support in kolla, they have a chance to show up12:55
opendevreviewMartin Hiner proposed openstack/kolla-ansible master: Rename kolla_docker to kolla_container  https://review.opendev.org/c/openstack/kolla-ansible/+/90041212:55
mnasiadkaSvenKieske: seems like it would make sense backporting back to the first release when we had systemd12:56
fricklerha, jinx, I was just about to ask mhiner if they are around to update ^^ :-)12:56
mnasiadkawhich is Antelope I guess?12:56
mnasiadkayes, antelope12:56
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible stable/2023.1: Introduce oneshot docker_restart_policy  https://review.opendev.org/c/openstack/kolla-ansible/+/90098512:56
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible stable/2023.1: Introduce oneshot docker_restart_policy  https://review.opendev.org/c/openstack/kolla-ansible/+/90098512:57
fricklerhmm, where do the conflicts come from, do we need more backports?12:57
mnasiadkano podman in antelope12:58
mnasiadkaso basically two files do not needed ;-)12:58
mnasiadka*not needed12:58
fricklerah, ack. you also need to unwip the patch then12:59
mnasiadkaalready done12:59
fricklerhmm, seems gerrit is slow with updating things, then. thx13:00
opendevreviewMerged openstack/kolla-ansible master: Adds condition to handle QoS extension in sriov_agent.ini template  https://review.opendev.org/c/openstack/kolla-ansible/+/89962913:14
opendevreviewMerged openstack/ansible-collection-kolla master: podman: Support registry mirror  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/89965713:14
mnasiadkahttps://review.opendev.org/q/project:openstack/ansible-collection-kolla+is:open+label:Review-Priority - these two would like reviews, so we could standardise podman/docker settings in kolla-ansible CI13:17
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible stable/2023.1: Adds condition to handle QoS extension in sriov_agent.ini template  https://review.opendev.org/c/openstack/kolla-ansible/+/90098613:17
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible stable/zed: Adds condition to handle QoS extension in sriov_agent.ini template  https://review.opendev.org/c/openstack/kolla-ansible/+/90098713:17
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible stable/yoga: Adds condition to handle QoS extension in sriov_agent.ini template  https://review.opendev.org/c/openstack/kolla-ansible/+/90098813:17
opendevreviewMichal Nasiadka proposed openstack/ansible-collection-kolla master: CI: add podman jobs  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/89946813:40
opendevreviewMerged openstack/kayobe master: ipa-images: use changed openstack.cloud output  https://review.opendev.org/c/openstack/kayobe/+/90075313:49
mnasiadkamgoddard mnasiadka bbezak frickler kevko SvenKieske mmalchuk gkoper jangutter jsuazo jovial - meeting in 8 minutes13:52
mnasiadka#startmeeting kolla14:01
opendevmeetMeeting started Wed Nov 15 14:01:24 2023 UTC and is due to finish in 60 minutes.  The chair is mnasiadka. Information about MeetBot at http://wiki.debian.org/MeetBot.14:01
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:01
opendevmeetThe meeting name has been set to 'kolla'14:01
mnasiadka#topic rollcall14:01
mnasiadkao/14:01
jovial[m]0/14:01
mmalchuko/14:01
janguttero/14:01
mhinero/14:01
frickler\o14:02
kevko\o/14:03
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
mnasiadka#topic CI Status14:03
mnasiadkaUpdated all CI statuses, marked Masakari as broken14:03
jovial[m]Did the Ironic job get fixed in the end?14:04
mnasiadkafrickler: just as we discussed - do we want to mark Masakari as deprecated and fire off the mail to openstack-ml?14:04
SvenKieskeo/14:04
mnasiadkajovial[m]: yes, by https://review.opendev.org/c/openstack/kolla-ansible/+/900985 (well, the master patch, not the backport)14:05
fricklermnasiadka: I'd say yes, I can create a patch if you take care of the mail in return14:05
mnasiadkafrickler: we have a deal14:05
mnasiadkaok, let's continue14:06
mnasiadka#topic Release tasks14:06
mnasiadkaI've marked most of the patches we should try to squeeze in Bobcat as RP+114:07
mnasiadkaPlease try to review them, ask those that are missing - but no more major features please14:07
mnasiadkaadd those that are missing14:08
mnasiadkanot ask14:08
SvenKieskewould quorum queues be considered a major feature? :)14:08
fricklerlet's get the cat out of the house14:08
fricklergood question. "maybe"?14:09
mnasiadkaI think we agreed we should do it this cycle14:09
mnasiadkaso - SvenKieske - what do you need to get it merged? :)14:09
jovial[m]I also added RP+1 to a few kayobe patches: https://review.opendev.org/q/project:openstack/kayobe+label:Review-Priority%253D1+status:open to try and get them into the release14:10
SvenKieskehopefully just a new CI run and then inspect the logs again, there was a recent fix in oslo.messaging that was missing, I hope that's all at least14:11
mnasiadkaSvenKieske: would be good14:11
mnasiadkaSvenKieske: can you post a link so we can RP+1?14:11
SvenKieskemight still need to disect more of the upstream docs - which are cluttered - and revisit what to do for upgrades, but I guess as a first try it's good to support new deployments/go with downtime?14:12
SvenKieske#link https://review.opendev.org/c/openstack/kolla-ansible/+/89854314:12
SvenKieskethe oslo change got merged: https://review.opendev.org/c/openstack/oslo.messaging/+/88931314:12
jovial[m]SvenKieske: Are you going to add some docs explaining how to migrate from the old style?14:13
fricklerso we still need that in bobcat and a new release https://review.opendev.org/c/openstack/oslo.messaging/+/90089114:13
jovial[m]sorry, missed your earlier message14:13
SvenKieskefrickler: right, but as it's a bugfix I think that should be hopefully backported quickly? maybe too optimistic :)14:14
mnasiadkaSvenKieske: we should be fine backporting the upgrade/migration docs later on14:15
mnasiadkaif that requires a lot more work and we can't get it out through the door now14:15
SvenKieskeok14:16
mnasiadkaok then let's go forward14:16
mnasiadka#topic Current cycle planning14:16
mnasiadkaCurrent means sort of Bobcat, I've tidied up the Bobcat priorities on the Whiteboard14:17
mnasiadkaWill organise the Caracal priorities and send out a PTG summary14:17
mnasiadka#topic Additional agenda14:18
mnasiadkanone14:18
mnasiadka#topic Open discussion14:18
mnasiadkaso, we've got 40 minutes for open discussion ;)14:19
mhinerI want to ask if you're interested in the migration patch https://review.opendev.org/c/openstack/kolla-ansible/+/83694114:19
SvenKieskecertainly, as there are a lot of docker deployments out there :)14:19
mhinerIt's goal is to migrate from Docker to Podman and vice versa within the same version14:19
SvenKieskesame version meaning openstack release, or..?14:20
mhineryes, that14:20
SvenKieskethat would be good information to add to the commit message imho :)14:20
jovial[m]mhiner: very nice - 'migrate' as a verb seems a bit generic though14:21
mnasiadkaWell, we probably are interested, but I think main case is that there is a CI job that tests that14:21
mnasiadkaSecond of all, ansible/migrate.yml is too generic, we have ansible/opensearch-migration.yml - so we should have ansible/container-engine-migration.yml and ideally support both directions? ;-)14:22
mnasiadkaon other front, shouldn't it be that simple as changing kolla_container_runtime to podman?14:22
mnasiadkaah, volumes migration14:23
mnasiadkaok then, we need a command for the migration surely14:23
mhinermnasiadka: both directions are supported and yeah ansible/migrate.yml can be renamed14:23
jovial[m]how close is it to working?14:24
SvenKieskeack, nice :)14:24
mnasiadkaand then we need CI jobs testing that14:24
mnasiadkadon't assume reviewers will go the extra mile and test that on their env14:24
mhinerjovial[m]: I worked on it relatively long time ago but iirc it should be functional14:25
mhinermnasiadka: am I able to create that CI job?14:25
SvenKieskeyeah, might be tricky to test14:25
mnasiadkamhiner: sure, just add it in .zuul.d/ - copy cat the opensearch migration jobs and you'll be fine14:26
mhinerokay, will look into it14:26
mnasiadkasorry, zuul.d (not a hidden one)14:26
mnasiadkaif you need help, just reach out :)14:26
mhineralright, thanks14:27
janguttermhiner, you don't need special permissions to run and test a zuul job in the CI, and while developing it you can temporarily disable the other jobs till you've got it ironed out.14:27
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: Add keystone_federation_oidc_additional_options variable for additional options  https://review.opendev.org/c/openstack/kolla-ansible/+/89283114:27
SvenKieskethat commit message looks..interesting :D14:28
SvenKieskeI heard you like additional options, so I put addtional options in your additional options.14:28
mnasiadkayeah, the author got bored, I'll refresh it14:29
mnasiadkaok, anybody has anything else or should we wrap up?14:32
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: keystone: Add support for OIDC options  https://review.opendev.org/c/openstack/kolla-ansible/+/89283114:35
mnasiadkaSvenKieske: better? :)14:35
jovial[m]just needs a reno ;-)14:36
mnasiadkamakes sense14:36
SvenKieskevery nice :)14:37
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: keystone: Add support for OIDC options  https://review.opendev.org/c/openstack/kolla-ansible/+/89283114:40
mnasiadkaand there's the reno14:40
mnasiadkaok, I guess nothing more14:41
mnasiadkathanks for coming!14:41
mnasiadka#endmeeting14:41
opendevmeetMeeting ended Wed Nov 15 14:41:08 2023 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:41
opendevmeetMinutes:        https://meetings.opendev.org/meetings/kolla/2023/kolla.2023-11-15-14.01.html14:41
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/kolla/2023/kolla.2023-11-15-14.01.txt14:41
opendevmeetLog:            https://meetings.opendev.org/meetings/kolla/2023/kolla.2023-11-15-14.01.log.html14:41
opendevreviewMerged openstack/kolla-ansible master: CI: Log details about failed containers  https://review.opendev.org/c/openstack/kolla-ansible/+/89248814:41
mmalchukmnasiadka thanks14:43
SvenKieskemhm thanks14:43
SvenKieskeseems gerrit is really slow I don't see the reno14:43
SvenKieskeeven force reloading the page doesn't help. did notice slow gerrit beginning yesterday in the evening (took ages to load)14:44
greatgatsbyMind if I just ask for quick clarification on masakari being deprecated?  So it's going to be removed entirely from future kolla releases?  I'd just like to be able to pass this info on to my team.14:45
SvenKieskethat depends on masakari: it's currently not well developed afaik, so if that doesn't change it will get dropped I guess?14:47
SvenKieskeso if you depend on it that is a good point to get involved upstream :)14:47
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: keystone: Add support for OIDC options  https://review.opendev.org/c/openstack/kolla-ansible/+/89283114:49
mnasiadkaSvenKieske: now it's there, sorry ;)14:49
mnasiadkagreatgatsby: it's going to be deprecated in Bobcat, if it won't shape up in Caracal - then we'll drop it14:50
greatgatsbyinteresting, thanks.  I'm not sure we "depend" on it, but we do currently enable it.  Good to know there's a chance it might be removed.14:51
mnasiadkajangutter: so now Ubuntu podman has been building over 3 hours - sure it doesn't need your patch? :)14:53
jangutter:o14:53
mnasiadkahttps://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_7a8/899231/4/check/kolla-build-ubuntu-podman/7a80523/job-output.txt14:53
jangutterWe can check, the podman info should tell us.14:53
jangutter    Native Overlay Diff: "true"  <--- It doesn't look like the same issue (from: https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_7a8/899231/4/check/kolla-build-ubuntu-podman/7a80523/kolla/podman-info.txt )14:54
mnasiadkaok then14:54
jangutterfrom reading around, _weird_ things can cause slowdowns that snowball. One thing a friend hit was old versions of rpm needed a limit to open files.14:57
jangutterBut this isn't rpm based containers, so https://www.linuxquestions.org/questions/linux-software-2/docker-fedora-31-installing-rpms-for-redhat-variants-slow-at-scriptlet-start-4175667314/ shouldn't apply.14:58
janguttermnasiadka: do you have a link to the zuul page?15:01
janguttersomething stalled on that, the build was progressing fine then, no output and it timed out.15:03
jangutterThis one just hung.... https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_7a8/899231/4/check/kolla-build-ubuntu-podman/7a80523/kolla/build/ironic-prometheus-exporter.log15:06
kevkodoes anyone using pdf generation from kolla-ansible ? :) 15:06
kevkoi mean tox ...15:06
opendevreviewMerged openstack/kolla-ansible master: Fix OpenStack exporter scrape with internal TLS & FQDN  https://review.opendev.org/c/openstack/kolla-ansible/+/87748315:10
SvenKieskekevko: I never used that, didn't even know it is a thing, why do you ask?15:24
opendevreviewPedro Alvarez proposed openstack/kolla-ansible stable/zed: Fix OpenStack exporter scrape with internal TLS & FQDN  https://review.opendev.org/c/openstack/kolla-ansible/+/90099015:33
opendevreviewPedro Alvarez proposed openstack/kolla-ansible stable/2023.1: Fix OpenStack exporter scrape with internal TLS & FQDN  https://review.opendev.org/c/openstack/kolla-ansible/+/90099215:35
opendevreviewMerged openstack/kolla-ansible master: ovn: Fix wording in limit check  https://review.opendev.org/c/openstack/kolla-ansible/+/89891015:52
SvenKieskemhm, just glancing at https://etherpad.opendev.org/p/kolla-stable-yoga-eom should we maybe merge this for yoga? https://review.opendev.org/c/openstack/kolla/+/880799 (rmq 3.10)15:55
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible stable/2023.1: ovn: Fix wording in limit check  https://review.opendev.org/c/openstack/kolla-ansible/+/90099415:57
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible stable/zed: ovn: Fix wording in limit check  https://review.opendev.org/c/openstack/kolla-ansible/+/90099515:58
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible stable/yoga: ovn: Fix wording in limit check  https://review.opendev.org/c/openstack/kolla-ansible/+/90099615:58
opendevreviewVerification of a change to openstack/kayobe stable/yoga failed: Generate local Kolla Ansible config in check mode  https://review.opendev.org/c/openstack/kayobe/+/90058916:19
opendevreviewMark Goddard proposed openstack/kolla-ansible master: fact gathering: Group hosts before gathering facts  https://review.opendev.org/c/openstack/kolla-ansible/+/89959217:09
opendevreviewMark Goddard proposed openstack/kolla-ansible master: fact gathering: Remove gathering of package facts  https://review.opendev.org/c/openstack/kolla-ansible/+/90105617:09
mnasiadkajangutter: zuul.openstack.org ?17:15
janguttermnasiadka: i mean the specific zuul result page for the log you sent.17:15
mnasiadkajangutter: https://review.opendev.org/c/openstack/kolla/+/899231?tab=change-view-tab-header-zuul-results-summary17:16
jangutterthanks!17:17
jangutteroh yeah, this is a Zuul timeout, raw logs are needed. But regardless, I think the build process hung on one job rather than being generally slow.17:18
jangutterYou can see https://zuul.opendev.org/t/openstack/build/7a80523666914aa9a625d7b5f178d114/log/job-output.txt#1001 <--- it hung here for hours and didn't progress17:19
mnasiadkahttps://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_7a8/899231/4/check/kolla-build-ubuntu-podman/7a80523/kolla/build/ironic-prometheus-exporter.log17:20
mnasiadkait didn't even properly start building anything17:20
opendevreviewVerification of a change to openstack/kayobe stable/yoga failed: Fix setting kolla_admin_openrc_cacert  https://review.opendev.org/c/openstack/kayobe/+/90058617:44
opendevreviewJuan Pablo Suazo proposed openstack/kolla-ansible master: Enable the Fluentd Plugin Systemd  https://review.opendev.org/c/openstack/kolla-ansible/+/87598317:50
opendevreviewMichal Nasiadka proposed openstack/kayobe master: Remove Monasca and co remnants  https://review.opendev.org/c/openstack/kayobe/+/90105917:53
opendevreviewMichal Nasiadka proposed openstack/kayobe master: Drop not used grafana-conf from reqs  https://review.opendev.org/c/openstack/kayobe/+/90106017:56
opendevreviewMerged openstack/kayobe stable/zed: Generate local Kolla Ansible config in check mode  https://review.opendev.org/c/openstack/kayobe/+/90058817:58
opendevreviewMerged openstack/kayobe stable/yoga: Fix an issue when user forgot combine custom passwords  https://review.opendev.org/c/openstack/kayobe/+/90045517:58
opendevreviewVerification of a change to openstack/kayobe stable/2023.1 failed: Fix an issue when user forgot combine custom passwords  https://review.opendev.org/c/openstack/kayobe/+/90046118:11
opendevreviewMerged openstack/kayobe stable/zed: Fix an issue when user forgot combine custom passwords  https://review.opendev.org/c/openstack/kayobe/+/90046218:12
opendevreviewMerged openstack/kolla-ansible stable/2023.1: Introduce oneshot docker_restart_policy  https://review.opendev.org/c/openstack/kolla-ansible/+/90098518:36

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