Wednesday, 2023-10-11

SvenKieskeo/06:31
SvenKieskeERROR:kolla.common.utils.gnocchi-base:The conflict is caused by:     gnocchi[ceph,file,keystone,mysql,s3] 4.6.0 depends on pyparsing<3.1.0 and >=2.2.0     The user requested (constraint) pyparsing===3.1.106:55
SvenKieskehttps://zuul.opendev.org/t/openstack/build/d97e12c90954419e80f44240af0595da/log/kolla/build/000_FAILED_gnocchi-base.log06:55
SvenKieskechecking if this is a gnocchi or mirror thing07:13
SvenKieskeI get the same error when building gnocchi-base locally07:32
SvenKieskereported upstream here: https://github.com/gnocchixyz/gnocchi/issues/134507:39
SvenKieskemaybe we need to fix this in global-requirements, but I hope not :)07:40
SvenKieskemnasiadka: should I mark gnocchi-base as unbuildable as a workaround? it doesn't look like this is gonna be fixable before the weekend at the very least, and I even doubt that.07:52
mnasiadkayes, sure07:52
SvenKieskeok, great :) at least one issue "solved" then..07:53
opendevreviewSven Kieske proposed openstack/kolla master: mark gnocchi-base as unbuildable  https://review.opendev.org/c/openstack/kolla/+/89792407:59
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: CI: Pin ansible-core due to breakage  https://review.opendev.org/c/openstack/kolla-ansible/+/89792608:26
SvenKieskemnasiadka: thanks for pinning in the right place, I was confused08:37
SvenKieskeabandoned my try: https://review.opendev.org/c/openstack/kolla/+/89784308:39
SvenKieskedid you test it locally?08:39
mnasiadkaSvenKieske: not yet, cephadm gave me a heart attack so I'm recovering :)08:50
SvenKieskemy heartfelt condolences :)08:57
mnasiadkaSvenKieske: looks like it works09:18
SvenKieskeI only see tox|linting jobs so far? or are you testing locally? https://zuul.opendev.org/t/openstack/builds?change=897926&skip=009:19
SvenKieskethat looks good :) https://zuul.opendev.org/t/openstack/build/4f3db520229a4520a78d540c79850ba309:24
mnasiadkaSvenKieske: you can see jobs in progress on status tab, there's a filter field - I usually put kolla in there to see all the jobs in progress for kolla ;)09:39
SvenKieskeah I knew it was somewhere but didn't find it anymore..thx09:40
SvenKieskeI'd never click on "status" I would expect to find something about..well the zuul status there :D09:41
SvenKieskemnasiadka: gnocchi fails also on this ansible bug: https://review.opendev.org/c/openstack/kolla/+/897924 so I'd like to merge your stuff asap, if possible. :)10:02
mnasiadkaSvenKieske: yup, I'll merge it once it passes Zuul to unbreak other breakage fixes :)10:07
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: CI: Break OVN cluster before reconfigure  https://review.opendev.org/c/openstack/kolla-ansible/+/89793510:50
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: CI: Break OVN cluster before reconfigure  https://review.opendev.org/c/openstack/kolla-ansible/+/89793510:53
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: CI: Break OVN cluster before reconfigure  https://review.opendev.org/c/openstack/kolla-ansible/+/89793510:55
SvenKieskemnasiadka: should I reopen https://review.opendev.org/c/openstack/kolla/+/897843 ?11:33
mnasiadkaSvenKieske: I'm not convinced anything we use in kolla_toolbox got broken11:34
mnasiadkaonly one filter, I'll have a look on it today/tomorrow, still don't understand why it's broken :)11:35
SvenKieskeyeah, I was just wondering if we could maybe assist users somehow in enforcing a certain ansible-core version, but we don't have it in requirements.txt or anywhere similar.11:35
* SvenKieske still gets lost daily in the maze that is python packaging.11:35
mnasiadkaI think it should be solvable soon, so let's not get into crazy things11:37
mnasiadkaquestion if Antelope is broken as well11:37
SvenKieskeokay, I like your optimism :D11:37
SvenKieskeI didn't look at any backport builds just yet11:37
SvenKieskeah, frickler did break gnocchi :D https://review.opendev.org/c/openstack/requirements/+/88456411:42
SvenKieskemhm, I just wonder why this wasn't detected?11:43
SvenKieskeah this was only merged 5 days ago11:43
opendevreviewPierre Riteau proposed openstack/kayobe stable/zed: Revert "CI: Disable bare metal testing on RL9/c9s"  https://review.opendev.org/c/openstack/kayobe/+/89782012:01
opendevreviewPierre Riteau proposed openstack/kayobe stable/zed: Revert "CI: Disable bare metal testing on RL9/c9s"  https://review.opendev.org/c/openstack/kayobe/+/89782012:03
ravlewHi12:06
ravlewI'm looking for a bugfix review: https://review.opendev.org/c/openstack/kolla-ansible/+/89741912:06
SvenKieskeravlew: done, but I guess you are after some precious +2 review? :D12:09
kevkoravlew: approved 12:14
opendevreviewMerged openstack/kolla-ansible master: CI: Pin ansible-core due to breakage  https://review.opendev.org/c/openstack/kolla-ansible/+/89792612:43
SvenKieske\o/12:45
mnasiadkayay12:46
opendevreviewJuan Pablo Suazo proposed openstack/kolla-ansible master: Configures the tap-as-a-service neutron plugin  https://review.opendev.org/c/openstack/kolla-ansible/+/88541712:56
mnasiadkacrap, nearly forgot about the meeting12:59
SvenKieske:D12:59
SvenKieskeseems it's a rather lonely day in here today12:59
mnasiadkamgoddard mnasiadka bbezak frickler kevko SvenKieske mmalchuk gkoper jangutter jsuazo.- meeting NOW13:00
mnasiadka#startmeeting kolla13:00
opendevmeetMeeting started Wed Oct 11 13:00:04 2023 UTC and is due to finish in 60 minutes.  The chair is mnasiadka. Information about MeetBot at http://wiki.debian.org/MeetBot.13:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.13:00
opendevmeetThe meeting name has been set to 'kolla'13:00
mnasiadka#topic rollcall13:00
mnasiadkao/13:00
janguttero/13:00
SvenKieskeo/13:00
ihalomi\o13:00
mnasiadka#topic agenda13:02
mnasiadka* CI status13:02
mnasiadka* Release tasks13:02
mnasiadka* Regular stable releases (first meeting in a month)13:02
mnasiadka* Current cycle planning13:02
mnasiadka* Additional agenda (from whiteboard)13:02
mnasiadka* Open discussion13:02
mnasiadka#topic CI status13:02
mnasiadkaAnsible broke us again, currently we pinned to previous minor releases of ansible-core13:03
mnasiadkaI think requirements broke gnocchi in Kolla, right SvenKieske ?13:03
SvenKieskegnocchi-base seems unbuildable, yes, workaround here: https://review.opendev.org/c/openstack/kolla/+/89792413:03
SvenKieskebroken 5 days ago by: https://review.opendev.org/c/openstack/requirements/+/88456413:03
SvenKieskeI also linked a github issue I created upstream at gnocchi in the commit message, there are all the details13:04
SvenKieskeI'm not sure if our mirror issues from yesterday are really fixed. I will test after gnocchi unblocks :)13:05
SvenKieskethat's my first time marking something unbuildable, so maybe someone can check if I didn't forget anything13:05
SvenKieskethere's an upstream ansible issue about our breakage as well, created by a helpful user(?)13:07
mnasiadkareviewed13:07
SvenKieske#link https://github.com/ansible/ansible/issues/8194513:07
mnasiadkayeah, checking why only nova filters have a problem13:07
SvenKieskeI mentioned a possible culprit in 2.14.11 in above linked github issue, but I didn't check all commits so might very well be wrong13:08
SvenKieske#link https://github.com/ansible/ansible/pull/8164413:08
SvenKieskethat's the one13:08
opendevreviewJuan Pablo Suazo proposed openstack/kolla-ansible master: Enable the Fluentd Plugin Systemd  https://review.opendev.org/c/openstack/kolla-ansible/+/87598313:10
mnasiadkaok then13:16
mnasiadka#topic Current cycle planning13:17
mnasiadkaI think we need to first unblock the CI in both Kolla and Kolla-Ansible13:17
mnasiadkaso let's progress on the gnocchi patch13:17
mnasiadkathen we should be able to start reviewing anything13:17
mnasiadkaanything else of very high priority right now?13:18
mnasiadkajangutter: anything etcd related?13:20
opendevreviewSven Kieske proposed openstack/kolla master: mark gnocchi-base as unbuildable  https://review.opendev.org/c/openstack/kolla/+/89792413:20
jangutternope, would be nice if we can get the version bumped this cycle, but nothing burning.13:21
jangutterAlso, feel free to be mean on the reviews :-p13:22
mnasiadkahaha13:22
mnasiadkawe're not mean, just sincere and caring :)13:22
jangutteryeah, I'm looking for feedback, not approval, so thanks for folks taking the time to take a look13:23
mnasiadkawill try, but $work is making me busy on other things now, should get better next week13:25
mnasiadka#topic Additional agenda (from whiteboard)13:25
SvenKieskethat 79 char limit is annoying :D13:26
mnasiadka885151: Adds TAAS Neutron plugin to support OVS port mirrors | https://review.opendev.org/c/openstack/kolla/+/88515113:26
mnasiadka885417: Configures the tap-as-a-service neutron plugin | https://review.opendev.org/c/openstack/kolla-ansible/+/88541713:26
mnasiadka844614: Deploy Glance and Cinder Backup with S3 backend support | https://review.opendev.org/c/openstack/kolla-ansible/+/844614 (missing a single +2)13:26
mnasiadka885417: Configures the tap-as-a-service neutron plugin | https://review.opendev.org/c/openstack/kolla-ansible/+/88541713:26
mnasiadka(by jsuazo)13:26
mnasiadkaSvenKieske: so just do noqa in the end13:26
SvenKieskeit's okay, can fix it :) I used to use good old IBM AIX terminals, so I know where it's from :)13:26
opendevreviewSven Kieske proposed openstack/kolla master: mark gnocchi-base as unbuildable  https://review.opendev.org/c/openstack/kolla/+/89792413:27
jsuazo    Hello all, i believe all comments on those proposal have been addressed. 13:27
mnasiadkataas looks ok13:27
mnasiadkawell, reno missing on the Kolla patch13:28
mnasiadkaand needs a second core reviewer (unfortunately no one else present here today)13:28
jsuazowill update today !13:28
mnasiadkaglance the same - needs somebody else than me :)13:28
SvenKieskepromote me :P ;)13:29
kevkoi will review13:29
mnasiadkawell, let's discuss that amongst core reviewers :)13:29
jsuazoAny core i could ping through mail ?13:29
SvenKieskeI guess I can use some more experience before becoming core13:29
kevkobut another meeting in progress ..sorry 13:29
mnasiadkajsuazo: kevko is here ;)13:30
SvenKieskefrickler is on PTO today at least13:30
kevkoglance s3 looks fine ..but i need to test something 13:31
kevkoglance_store_backends -> why only the first one ? 13:31
kevkoaaa ...sorry it's default backend 13:32
mnasiadkaok then let's move on13:32
mnasiadka#topic Open discussion13:32
jsuazokevko: thanks !13:32
mnasiadkaAnybody anything?13:32
mnasiadkayup13:32
SvenKieskeI'm fine if pipelines are green again I guess :)13:32
mnasiadkaok then13:33
mnasiadkaseems no open discussion13:33
mnasiadkalet's make this meeting short, due to low attendance ;-)13:33
mnasiadkathanks for coming :)13:33
mnasiadka#endmeeting13:33
opendevmeetMeeting ended Wed Oct 11 13:33:36 2023 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)13:33
opendevmeetMinutes:        https://meetings.opendev.org/meetings/kolla/2023/kolla.2023-10-11-13.00.html13:33
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/kolla/2023/kolla.2023-10-11-13.00.txt13:33
opendevmeetLog:            https://meetings.opendev.org/meetings/kolla/2023/kolla.2023-10-11-13.00.log.html13:33
mnasiadka(feel free to discuss anything outside of the meeting of course)13:33
SvenKieskeI noticed I should maybe tweak our docs to tell users to install "kolla" from git and not from pypi, what do you think?13:34
kevkolast days i am playing (stressing) with tempests and rally 13:35
kevkowhat about to allow in kolla ? :P 13:35
kevkoSvenKieske: +1+1+1+113:36
kevkoI think we should stop release via pip and release through the docker repository 13:36
kevkoas kolla images ...same for kolla-ansible image 13:36
kevko(i am using it this way for several releases )13:37
mnasiadkakevko: for now we're not forcing users to have docker on deployment host, let's not get into that - but building a container and pushing that to quay sounds like a nice idea for those that would like to use it that way13:38
kevkook, agree13:38
opendevreviewJuan Pablo Suazo proposed openstack/kolla master: Adds TAAS Neutron plugin to support OVS port mirrors  https://review.opendev.org/c/openstack/kolla/+/88515113:38
opendevreviewMerged openstack/kolla-ansible master: Add a separate interface address for tgtd  https://review.opendev.org/c/openstack/kolla-ansible/+/89741913:38
SvenKieskeI just realized this yesterday, I was just stupidly following our docs when trying local builds (I try to not do stuff locally) and I couldn't use bookworm as a result, because pypi release is old.13:39
SvenKieskeso if I fall into this trap I think other users will end there as well, possibly :)13:39
jsuazomnasiadka already added the reno btw13:39
SvenKieskejsuazo: nice :)13:40
mnasiadkajsuazo: thanks13:40
ihalomiwhat about podman collection patchset? should we remove commented variables or it is not decided yet? 13:42
mnasiadkaWell, I would probably prefer that we remove commented out variables and do everything in the same way we have for docker_sdk13:44
mnasiadkaThen we can have a discussion on the PTG about how we want it in the next cycle - and maybe have some ,,rules''13:44
opendevreviewPedro Henrique Pereira Martins proposed openstack/kolla-ansible master: Fix the docker container dimensions comparison for short notation  https://review.opendev.org/c/openstack/kolla-ansible/+/88650013:48
opendevreviewMartin Hiner proposed openstack/kolla-ansible master: Add support of podman deployment  https://review.opendev.org/c/openstack/kolla-ansible/+/79922913:54
opendevreviewMerged openstack/kayobe master: Add introspection rule to update deploy kernel location  https://review.opendev.org/c/openstack/kayobe/+/80811114:07
opendevreviewMartin Hiner proposed openstack/ansible-collection-kolla master: Add Podman support  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/85224014:31
SvenKieskezuul is pleased with gnocchi: https://review.opendev.org/c/openstack/kolla/+/89792415:21
opendevreviewMerged openstack/kolla master: mark gnocchi-base as unbuildable  https://review.opendev.org/c/openstack/kolla/+/89792418:31
-opendevstatus- NOTICE: Another short Gerrit outage for updates on review.opendev.org. This update ensures we are using the current versions of all Gerrit plugins.23:46

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