*** jph5 is now known as jph1 | 08:19 | |
*** jph1 is now known as jph | 08:20 | |
opendevreview | Bartosz Bezak proposed openstack/kolla stable/2023.1: Pin docker.py to <7 https://review.opendev.org/c/openstack/kolla/+/903583 | 09:48 |
---|---|---|
opendevreview | Alex Welsh proposed openstack/kolla-ansible master: Update keystone service user passwords https://review.opendev.org/c/openstack/kolla-ansible/+/903178 | 11:34 |
opendevreview | Alex Welsh proposed openstack/kolla-ansible master: Update keystone service user passwords https://review.opendev.org/c/openstack/kolla-ansible/+/903178 | 12:00 |
mnasiadka | mgoddard mnasiadka bbezak frickler kevko SvenKieske mmalchuk gkoper jangutter jsuazo jovial osmanlicilegi - meeting in 8 minutes | 13:52 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Stop running pip install container_engine https://review.opendev.org/c/openstack/kolla-ansible/+/903364 | 13:55 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Stop running pip install container_engine https://review.opendev.org/c/openstack/kolla-ansible/+/903364 | 13:56 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Stop running pip install container_engine https://review.opendev.org/c/openstack/kolla-ansible/+/903364 | 13:59 |
mnasiadka | #startmeeting kolla | 14:00 |
opendevmeet | Meeting started Wed Dec 13 14:00:22 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 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
opendevmeet | The meeting name has been set to 'kolla' | 14:00 |
mnasiadka | #topic rollcall | 14:00 |
mnasiadka | o/ | 14:00 |
jovial | \o | 14:00 |
bbezak | o/ | 14:00 |
mattcrees | o/ | 14:00 |
jsuazo | \o | 14:00 |
frickler | o/ | 14:01 |
SvenKieske | o/ | 14:01 |
mmalchuk | \o | 14:01 |
mnasiadka | #topic agenda | 14:02 |
mnasiadka | * CI status | 14:02 |
mnasiadka | * Release tasks | 14:02 |
mnasiadka | * Regular stable releases (first meeting in a month) | 14:02 |
mnasiadka | * Current cycle planning | 14:02 |
mnasiadka | * Additional agenda (from whiteboard) | 14:02 |
mnasiadka | * Open discussion | 14:02 |
mnasiadka | #topic CI status | 14:02 |
mnasiadka | bleeding red | 14:02 |
mnasiadka | docker 7.0.0 broke docker builds | 14:02 |
SvenKieske | :( | 14:02 |
mnasiadka | we're trying to pin, but that's a funny task | 14:02 |
mnasiadka | hopefully we'll resolve that until the end of the week | 14:02 |
bbezak | for stable branches as well | 14:02 |
bbezak | as we probably need to ditch requirements CI check to implement it - https://review.opendev.org/c/openstack/kolla/+/903583 | 14:03 |
bbezak | https://zuul.opendev.org/t/openstack/build/342916f25b0646e491c1fec2707fe110 | 14:03 |
mnasiadka | seems that's the only sensible options | 14:03 |
mnasiadka | *option | 14:04 |
mnasiadka | anybody sees any problems in ditching requirements-check job? | 14:04 |
frickler | no, this has come up earlier already, I'm +2 to that | 14:05 |
SvenKieske | temporarily to fix this I guess? | 14:05 |
mmalchuk | why not propose change to requiremnts? | 14:05 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Pin docker to <7 in setup_gate.sh https://review.opendev.org/c/openstack/kolla-ansible/+/903364 | 14:05 |
frickler | kolla isn't to be co-installed with anything else from openstack | 14:05 |
mnasiadka | and we never documented that users should use upper-constraints | 14:05 |
mnasiadka | bbezak: just update the change with removal of reqs-check and let's fix stable branches | 14:06 |
mnasiadka | #topic Release tasks | 14:06 |
bbezak | ok | 14:06 |
mnasiadka | frickler: remember the deploy guide thing? we should raise some patch, right? | 14:06 |
frickler | already done and merged yesterday | 14:07 |
mnasiadka | fantastic | 14:07 |
mnasiadka | thanks | 14:07 |
frickler | https://review.opendev.org/c/openstack/openstack-manuals/+/903512 | 14:07 |
frickler | merged this morning, but yeah | 14:07 |
jovial | We do install kolla-ansible with constraints in kayobe. Will we need to stop doing that? | 14:07 |
mnasiadka | it's about kolla, not kolla-ansible | 14:07 |
jovial | ahh, OK | 14:08 |
mnasiadka | and we hope that once docker 7.0.1 is out with the fix for regex - we can unpin master | 14:08 |
jovial | Sounds like a plan | 14:08 |
frickler | we could also use !=7.0.0, that might pass the reqs check. though not sure about that | 14:08 |
mmalchuk | imho, both K and K-A are Openstack Python projects, so should use constraints | 14:08 |
bbezak | frickler: I can try that | 14:09 |
mnasiadka | #topic Current cycle planning | 14:10 |
mnasiadka | We agreed to hold with building Caracal in master until January | 14:10 |
mnasiadka | Any patches that should be reviewed before then? (not counting the docker7 drama) | 14:10 |
mnasiadka | mattcrees: how's the RMQ precheck going? | 14:11 |
mattcrees | Would be good to get the quorum queue precheck in, so it can be backported to Bobcat https://review.opendev.org/c/openstack/kolla-ansible/+/902787 | 14:11 |
mattcrees | There's discussion to be had on if this should actually be present in Caracal, I think it should :) | 14:11 |
opendevreview | Bartosz Bezak proposed openstack/kolla stable/2023.1: Pin docker.py to <7 https://review.opendev.org/c/openstack/kolla/+/903583 | 14:11 |
mnasiadka | it should be there always, because people might decide to stay on HA queues for another cycle and migrate somewhere in future | 14:12 |
SvenKieske | +1 on that | 14:12 |
mnasiadka | frickler: do you still need answers from mattcrees ? | 14:13 |
SvenKieske | but I already commented on the patch | 14:13 |
frickler | no, I'm fine with just removing the "drop in C" comment | 14:13 |
mnasiadka | mattcrees: drop that comment please ;-) | 14:13 |
mattcrees | Yeah will do :P | 14:13 |
mnasiadka | ok | 14:14 |
mnasiadka | don't see anything new in additional topics | 14:14 |
mnasiadka | let's go to open discussion | 14:14 |
mnasiadka | #topic Open discussion | 14:14 |
SvenKieske | side note: the docker-py 7.0.0 stuff will be needed afaik for python 3.12 support :) so hoping for 7.0.1 :) | 14:15 |
mnasiadka | frickler: just thinking, everybody is claiming mod_wsgi+apache is to be not recommended - do you know why? | 14:15 |
mnasiadka | (I started a poc on switching Nova to gunicorn, I'll probably work on this later - when we fix the docker7 drama) | 14:15 |
SvenKieske | wsgi is "Old fashioned" afaik and not that well maintained? | 14:15 |
jsuazo | Hi, need some cores for reviews | 14:16 |
jsuazo | #link https://review.opendev.org/c/openstack/kolla-ansible/+/885417 | 14:16 |
jsuazo | #link https://review.opendev.org/c/openstack/kolla-ansible/+/875983 | 14:16 |
mnasiadka | yeah, that is obvious - was thinking about any known flaw beyond that | 14:16 |
frickler | mnasiadka: no, I would like more background there, too | 14:16 |
mnasiadka | we use uwsgi in devstack and some other projects, and it's also unmaintained :) | 14:16 |
SvenKieske | mnasiadka: regarding the poc: all gunicorn docs mention you should run it behind e.g. nginx reverse proxy in prod. I think it's not that hardened as a webserver. maybe not a problem with haproxy in front of it thoguh? | 14:16 |
mmalchuk | afaik its about pbr issues becouse it was used in python2 | 14:17 |
frickler | haproxy is as good as nginx for that IMO | 14:17 |
SvenKieske | yes | 14:17 |
mnasiadka | SvenKieske: That's what I was thinking - OSA is running that behind haproxy (only for keystone/horizon SSO support they use Apache in front of uwsgi) | 14:17 |
opendevreview | Matt Crees proposed openstack/kolla-ansible master: Add precheck for RabbitMQ quorum queues https://review.opendev.org/c/openstack/kolla-ansible/+/902787 | 14:17 |
SvenKieske | fine | 14:17 |
mnasiadka | Anyway, once I get to working gunicorn - we can have a discussion if we want to go in that direction | 14:18 |
frickler | gunicorn seems like a good idea, hoping that it will be better supported | 14:18 |
SvenKieske | I would also be interested in more information regarding wsgi, I'm personally not a fan of EOL software, being burned in the past too much.. | 14:18 |
mnasiadka | surely better than uwsgi | 14:18 |
mnasiadka | well, asgi seems to be the way | 14:18 |
mnasiadka | forward | 14:18 |
mnasiadka | but surely most openstack services are not there | 14:19 |
SvenKieske | yep | 14:19 |
frickler | will also be interesting to see what becomes of this whole eventlet thing, might be slightly related | 14:20 |
mnasiadka | jsuazo: seems you need some other core's attention - I've looked into those in the past | 14:20 |
mnasiadka | frickler, bbezak - willing to help jsuazo? | 14:20 |
mnasiadka | or maybe osmanlicilegi | 14:21 |
* frickler won't get to that this year at least, sorry | 14:21 | |
mnasiadka | ok | 14:22 |
mnasiadka | any more topics? | 14:22 |
jsuazo | mnasiadka 875983 has no core reviews yet | 14:22 |
jovial | I was just goin to mention that the patch to fix: https://bugs.launchpad.net/kayobe/+bug/2028744 has merged now. Was going to check it is in the latest kolla images. | 14:23 |
jsuazo | or at least +2 | 14:23 |
mnasiadka | jovial: fix on bifrost side? | 14:24 |
jovial | indeed | 14:24 |
mnasiadka | has it been backported to 2023.1? | 14:25 |
mnasiadka | We're not building Caracal images yet, so if it's only in master - then not yet | 14:25 |
jovial | yep, it is backported to 2023.1. | 14:25 |
mnasiadka | so then should be there | 14:25 |
mnasiadka | ok then, I see no other topics | 14:26 |
bbezak | frickler: ERROR: Requirement for package docker excludes a version not excluded in the global list. | 14:26 |
bbezak | Local settings : {'!=7.0.0'} | 14:26 |
bbezak | Global settings: set() | 14:26 |
bbezak | Unexpected : {'!=7.0.0'} | 14:26 |
bbezak | didn't help | 14:26 |
bbezak | so removing the check | 14:26 |
mnasiadka | I forgot to cancel meetings during christmas/new year period | 14:27 |
mmalchuk | bbezak lets propose a change to the requremets repo? | 14:27 |
mnasiadka | I'll do that today - I mean send out a mail | 14:27 |
mnasiadka | I'm going to cancel next week, week after next and then first week in January - so the next three | 14:28 |
mnasiadka | we meet again on 10th of January | 14:28 |
mnasiadka | Happy festive period - and if there are any topics before the next meeting - feel free to raise them in the channel | 14:29 |
SvenKieske | alright, fits my schedule :) | 14:29 |
SvenKieske | you too | 14:29 |
mnasiadka | Thanks for coming | 14:29 |
mnasiadka | #endmeeting | 14:29 |
opendevmeet | Meeting ended Wed Dec 13 14:29:21 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:29 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/kolla/2023/kolla.2023-12-13-14.00.html | 14:29 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/kolla/2023/kolla.2023-12-13-14.00.txt | 14:29 |
opendevmeet | Log: https://meetings.opendev.org/meetings/kolla/2023/kolla.2023-12-13-14.00.log.html | 14:29 |
frickler | reqs CI is broken for 2023.1, so that would take some time to merge | 14:29 |
mmalchuk | mnasiadka thanks | 14:30 |
mnasiadka | frickler: not counting that upper-constraints is probably pinning to a lot lower version | 14:31 |
opendevreview | Bartosz Bezak proposed openstack/kolla stable/2023.1: Pin docker.py to <7 https://review.opendev.org/c/openstack/kolla/+/903583 | 14:36 |
opendevreview | Bartosz Bezak proposed openstack/kolla stable/2023.1: Pin docker.py to <7 https://review.opendev.org/c/openstack/kolla/+/903583 | 14:45 |
opendevreview | Matt Crees proposed openstack/kolla-ansible master: Add precheck for RabbitMQ quorum queues https://review.opendev.org/c/openstack/kolla-ansible/+/902787 | 14:49 |
frickler | mnasiadka: oh, good point, so maybe instead we can actually use u-c for that? | 14:52 |
frickler | docker===6.0.0 in 2023.1, 6.0.1 in 2023.2, 6.1.3 in master | 14:53 |
mmalchuk | imho we should | 15:11 |
mmalchuk | it also will install oslo.config the right version, not the latest one | 15:12 |
SvenKieske | fine for me | 15:27 |
mnasiadka | frickler: we would need to add a check in the code for users - they don't use u-c | 15:33 |
SvenKieske | mhm | 15:54 |
opendevreview | Merged openstack/kolla-ansible master: Configures the tap-as-a-service neutron plugin https://review.opendev.org/c/openstack/kolla-ansible/+/885417 | 16:11 |
frickler | well I think we need to make a choice here. either we do not use u-c, then the reqs check is bogus and we should drop it, but we might get into conflict with mmalchuk and also the TC. or we use it, then we also need to make it part of our documentation | 16:19 |
mnasiadka | well, isn't u-c only for testing? | 16:22 |
mnasiadka | we use it in building images and in kayobe - but still it's only for testing and making sure testing is sane | 16:25 |
mnasiadka | if we want to direct users to use u-c - then we need that in the docs | 16:25 |
mnasiadka | I would probably go for testing for docker-py <7 in the code for stable releases, we don't need to support python3.12 there | 16:26 |
mnasiadka | and maybe amend the docs that best results are with using u-c | 16:27 |
frickler | hmm, good points, I'll need to think about that | 16:31 |
asrael | good evening! i have a interesting problem with my ansible-kolla deployment. we integrated most of the kolla-collection plays into our normal pxe/awx/callback pipeline to install hypervisors, now when i try deploying the openstack with kolla-ansible, i get random problems with containers. I can see in the host journal, that the containers crashing due to not able to find stuff. | 19:57 |
asrael | Example the cron container exits with: docker[953706]: /usr/local/bin/kolla_start: line 24: exec: cron: not found | 19:57 |
asrael | has someone a advice for me? | 19:58 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!