| mnasiadka | morning | 06:11 |
|---|---|---|
| opendevreview | renliang proposed openstack/kolla master: Fixed an issue with zun-cin-daemon building images in aarch64. https://review.opendev.org/c/openstack/kolla/+/885729 | 06:44 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: Fix an issue with zun-cni-daemon building images in aarch64 https://review.opendev.org/c/openstack/kolla/+/885729 | 06:45 |
| SvenKieske | morning | 07:50 |
| SvenKieske | congratz for being voted as PTL again, mnasiadka ;) | 07:53 |
| mnasiadka | thanks ;) | 07:59 |
| hrw | mnasiadka: gratki! | 07:59 |
| mnasiadka | I forgot which cycle in a row is that ;) | 07:59 |
| mnasiadka | hrw: do we really need https://review.opendev.org/c/openstack/kolla/+/886086 ? it's failing on SHA-1, and the currently used key should be the new one, based on their website | 08:00 |
| hrw | let me abandon | 08:06 |
| kevko | morning \o/ | 08:09 |
| kevko | mnasiadka, hrw : can we remove https://github.com/openstack/kolla/blob/3d703b60e6fb1062313b57293696f1c34151c27f/docker/macros.j2#L64 ulimit from this command ? podman failing on it ...this is the last issue | 08:11 |
| hrw | kevko: 'git blame' pointed at me? | 08:12 |
| kevko | hrw: i've tried to use git blame ..and went through 4 5 patches ... :D but i didn't find the commit which proposed | 08:12 |
| hrw | kevko: I would say 'go for it and check is someone complain' | 08:13 |
| mnasiadka | kevko: and second question is why ulimit fails on podman? | 08:15 |
| mnasiadka | default ulimit is lower? | 08:15 |
| hrw | podman does not run as root | 08:16 |
| mnasiadka | we use become for all container related tasks, so I guess it does | 08:16 |
| kevko | I don't think there is a sudo kolla-build | 08:31 |
| kevko | In Kolla ci | 08:31 |
| mnasiadka | well, we didn't - because we used docker engine to do all the work | 08:37 |
| mnasiadka | now podman is running the build as a user | 08:37 |
| mnasiadka | podman build supports --ulimit, maybe the default one is too small | 08:39 |
| mnasiadka | I'm not saying we really need that ulimit in td-agent-gem install | 08:39 |
| mnasiadka | but removing it might bring some issues on docker side of things | 08:39 |
| mnasiadka | because docker default is 1048576 | 08:41 |
| mnasiadka | so maybe the easiest approach is to set to 1024 - just like Neutron does (https://review.opendev.org/c/openstack/tripleo-heat-templates/+/559268) | 08:41 |
| mnasiadka | and 1024 is the podman default | 08:41 |
| kevko | so change 65536 -> 1024 ? | 08:42 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Add podman for image building https://review.opendev.org/c/openstack/kolla/+/825791 | 09:01 |
| opendevreview | Michal Arbet proposed openstack/kolla master: [CI] Add podman build https://review.opendev.org/c/openstack/kolla/+/895817 | 09:01 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Lower limit for open files when installing fluent gem https://review.opendev.org/c/openstack/kolla/+/896007 | 09:01 |
| opendevreview | Bartosz Bezak proposed openstack/kolla-ansible master: Add option for extra kernel modules in neutron role https://review.opendev.org/c/openstack/kolla-ansible/+/894735 | 09:32 |
| opendevreview | Will Szumski proposed openstack/kolla-ansible master: Support exposing prometheus_server externally https://review.opendev.org/c/openstack/kolla-ansible/+/828695 | 09:48 |
| opendevreview | Will Szumski proposed openstack/kolla-ansible master: Support exposing prometheus_server externally https://review.opendev.org/c/openstack/kolla-ansible/+/828695 | 09:50 |
| opendevreview | Bartosz Bezak proposed openstack/kayobe master: CI: Use nested nodepools for jobs that are spawning vms https://review.opendev.org/c/openstack/kayobe/+/872647 | 09:55 |
| opendevreview | Michal Nasiadka proposed openstack/kayobe-config master: kayobe-env: Unstick KOLLA_SOURCE_PATH and KOLLA_VENV_PATH https://review.opendev.org/c/openstack/kayobe-config/+/895888 | 10:01 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: Drop openvswitch-netcontrold https://review.opendev.org/c/openstack/kolla/+/891655 | 10:35 |
| kevko | mnasiadka, hrw: Podman passing now , can u review ? | 10:39 |
| mnasiadka | kevko: can you tell me why podman build is so slow? :) | 10:51 |
| mnasiadka | 23m 02s docker vs 1h 16m 00s podman | 10:51 |
| kevko | haha :D i don't know :D | 10:56 |
| kevko | mnasiadka: Podman provides faster startup thanks to its daemon-less structure. At any point in time, the host system stays protected. However, when it comes to overall building, Docker is a winner. The building part relies primarily on image building. Since Docker has native image building, it can build at least six times faster than Podman. | 10:57 |
| kevko | mnasiadka: https://www.solutelabs.com/blog/podman-vs-docker#:~:text=The%20building%20part%20relies%20primarily,six%20times%20faster%20than%20Podman. | 10:57 |
| mnasiadka | should we add that to docs, so users don't ask that? | 10:57 |
| kevko | mnasiadka: hmm, where it should go ? | 10:59 |
| mnasiadka | https://docs.openstack.org/kolla/latest/admin/index.html#building-container-images - somewhere here? | 10:59 |
| kevko | SvenKieske: do you want to help me with docs :D ? I hate it and i know you like docs :) | 11:00 |
| mnasiadka | probably here https://docs.openstack.org/kolla/latest/admin/image-building.html#building-kolla-images - e.g. that since Bobcat we support building with Podman (Buildah) and you use it like that, and expect it being slow? :) | 11:00 |
| opendevreview | Merged openstack/kayobe stable/zed: Limit interface names to 15 characters https://review.opendev.org/c/openstack/kayobe/+/884540 | 11:06 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible master: prometheus: Fix etcd scrape when etcd uses TLS https://review.opendev.org/c/openstack/kolla-ansible/+/896052 | 11:28 |
| SvenKieske | kevko: where exactly do you need assistance? be warned though that I hate restructured text syntax :D | 11:32 |
| kevko | SvenKieske ^^ just a note that we supports podman and it's slow 😂 | 11:35 |
| SvenKieske | interesting that it's slow; I skimmed the blog you linked and googled a bit around, no real conclusion though (I don't know what "native" in this context should mean, written in C?). | 11:36 |
| SvenKieske | so that would be kolla docs, I suppose? | 11:37 |
| SvenKieske | But what should it contain, besides what is already covered here? https://review.opendev.org/c/openstack/kolla/+/825791/34/releasenotes/notes/add-docker-engine-7402d3fe7349f1f3.yaml Let me take a look.. | 11:38 |
| SvenKieske | should we maybe add a note to the central README.rst that we have podman support? I don't know if it is worth it to explicitly mention docker|podman everywhere? | 11:41 |
| opendevreview | Verification of a change to openstack/kayobe stable/yoga failed: Limit interface names to 15 characters https://review.opendev.org/c/openstack/kayobe/+/884541 | 11:41 |
| mnasiadka | SvenKieske: not saying everywhere, just in https://docs.openstack.org/kolla/latest/admin/image-building.html#building-kolla-images - should be easy to do | 11:43 |
| mnasiadka | this is how you use podman, by default we use docker, end of story | 11:43 |
| opendevreview | Sven Kieske proposed openstack/kolla master: [docs] fix docker best practice link https://review.opendev.org/c/openstack/kolla/+/896054 | 11:45 |
| SvenKieske | mnasiadka: ok | 11:45 |
| SvenKieske | how do I build podman images? xD | 11:46 |
| SvenKieske | let me look up the option name.. | 11:46 |
| SvenKieske | mhm | 11:51 |
| SvenKieske | do we really want to make users install the podman-py manually? why not add it to requirements.txt? | 11:51 |
| mnasiadka | it should be added | 11:52 |
| mnasiadka | since we have docker-py there | 11:52 |
| SvenKieske | yep | 11:52 |
| mnasiadka | but it might be too late for this cycle | 11:52 |
| mnasiadka | anybody want to pursue that with the requirements team? | 11:53 |
| mnasiadka | (it needs to be in requirements repo in upper-constraints.txt) | 11:53 |
| SvenKieske | mhm | 11:53 |
| opendevreview | Bartosz Bezak proposed openstack/kolla-ansible master: Add option for extra kernel modules in neutron role https://review.opendev.org/c/openstack/kolla-ansible/+/894735 | 11:54 |
| SvenKieske | can't it be just in requirements.txt in kolla repo? | 11:54 |
| SvenKieske | I'm still a noob when it comes to know when something is in upper constraints.. | 11:54 |
| SvenKieske | I really don't understand this, it's not like every project is running in the same venv and thus must share the same deps? | 11:55 |
| frickler | they must all be co-installable, yes | 11:56 |
| frickler | not about venv mainly, but distro pkgs | 11:56 |
| frickler | but why should kolla require both docker-py and podman-py if only one of them is needed? | 11:57 |
| SvenKieske | ah, the good old dpkg way | 11:57 |
| kevko | Hmm, podman and docker were in requirements .. but I think frickler suggested to remove ? | 11:57 |
| frickler | yes, because having both installed at the same time makes no sense to me | 11:57 |
| SvenKieske | if we don't want to be one of them to be second class citizen they must be? | 11:57 |
| kevko | i don't think it's needed to have it in requirements ..as it is not hard requirement ...it's just optional | 11:58 |
| frickler | we should drop both from being required, ack | 11:58 |
| kevko | +1 | 11:58 |
| SvenKieske | ok so second class citizen and support requests go to frickler and kevko :) | 11:58 |
| SvenKieske | both? | 11:58 |
| SvenKieske | can kolla-build work at all without them? | 11:58 |
| frickler | no | 11:58 |
| frickler | but that's the price of having a choice: you actually have to make one | 11:59 |
| SvenKieske | so, I don't know... the file is called "_requirements_.txt" you know? | 11:59 |
| SvenKieske | defaults must work | 11:59 |
| frickler | you could add extra sections for both | 11:59 |
| SvenKieske | yeah, and make the installation docs even longer to..save 5mb disk spaceß | 11:59 |
| mnasiadka | makes sense for me, remove docker from requirements.txt | 12:00 |
| mnasiadka | are the errors in kolla-build enough for users to know how to install docker/podman-py? :) | 12:00 |
| kevko | it is i think | 12:00 |
| SvenKieske | why do you want to make your product less userfriendly? why make it harder to setup for beginners? | 12:00 |
| mnasiadka | it's not a product, it's open source software | 12:01 |
| SvenKieske | can someone please provide some justification that is not "well we could user either lib a or lib b, thus none is _technically_ required"= | 12:01 |
| SvenKieske | yeah it might as well be closed source or deleted from the universe without users. | 12:01 |
| mnasiadka | well, none is technically required, you can template out dockerfiles and use whatever you want to build them | 12:01 |
| kevko | you can fork and do whatewer you want :D | 12:02 |
| mnasiadka | yes | 12:02 |
| mnasiadka | fine by me | 12:02 |
| SvenKieske | the man who does not read has no advantage over the man who can't read - Mark Twain | 12:02 |
| mnasiadka | kolla-ansible doesn't have docker-py in requirements, you want to tell us it's also not user friendly? | 12:03 |
| SvenKieske | uhm, when does kolla-ansible need to interact with docker-py? | 12:04 |
| opendevreview | Maksim Malchuk proposed openstack/kolla master: Drop openvswitch-netcontrold https://review.opendev.org/c/openstack/kolla/+/891655 | 12:04 |
| kevko | kolla-docker module ? :D | 12:04 |
| SvenKieske | I'm not aware I installed that manually, like, ever? | 12:04 |
| mnasiadka | kolla_docker | 12:04 |
| mnasiadka | because ansible-collection-kolla baremetal role did install that for you when you run "kolla-ansible bootstrap-servers" | 12:05 |
| SvenKieske | okay? so you agree it's good to not need to install it manually? or should we remove that as well? | 12:05 |
| SvenKieske | I don't yet see any point being made, quite the contrary. | 12:05 |
| mnasiadka | situation is the same, we deliver software without required dependency, and you need to go the extra mile of installing it | 12:05 |
| mnasiadka | if removing docker from requirements.txt in kolla is seen as not being user friendly, then I probably need to stop being the PTL | 12:06 |
| SvenKieske | as I said, I'm pretty sure I never did type "pip install docker-py" into any terminal, so I beg to differ that I installed something manually :) | 12:06 |
| mnasiadka | I did that too many times with kolla-ansible, but whatever ;) | 12:07 |
| mnasiadka | so it's either we install both, or none | 12:07 |
| kevko | SvenKieske: do you think that people using kolla and kolla-ansible don't know how to install docker or podman library if they know how to build images ? | 12:07 |
| SvenKieske | I don't know, there are dozens of users who just install releases from pypi instead from, so there you are.. | 12:08 |
| SvenKieske | *git | 12:08 |
| mnasiadka | well, even if they get docker-py installed | 12:08 |
| mnasiadka | it doesn't mean they have Docker Engine installed | 12:08 |
| mnasiadka | so it won't work | 12:08 |
| kevko | indeed | 12:08 |
| mnasiadka | I'm pretty sure we have conscious users that know how to install a python package and Docker Engine | 12:09 |
| SvenKieske | I'm also not claiming it's "hard" or "users won't understand it" just "it's not necessary" and annoying | 12:09 |
| mnasiadka | well, if we want to install both (add them to requirements.txt) - then we need to add podman-py to openstack/requirements and get it managed by the requirements team | 12:09 |
| mnasiadka | otherwise requirements-check CI job is going to fail | 12:09 |
| SvenKieske | the ideal software installation should imho be a one liner like "apt install $foo" or "pip install $bar" and not a book full of instructions | 12:10 |
| mnasiadka | unless we don't care and remove the requirements-check job | 12:10 |
| kevko | +1 ...so podman feature which is ready to merge will stay unmerged for next half year :D | 12:10 |
| mnasiadka | yeah, let's wait 2 more years | 12:10 |
| kevko | because of one line in separate requirements repo | 12:10 |
| kevko | SvenKieske: haha, have you ever install openstack manually ? :D | 12:11 |
| mnasiadka | and then we'll get a bug report, that podman-py is a useless dependency, because the user doesn't want that package :) | 12:11 |
| SvenKieske | then delete that from the requirements.txt, but then also add all the extra steps needed to the docs, that's why I brought this topic up to begin with, currently, it's a nice oneliner to install kolla: "python3 -m pip install kolla" | 12:12 |
| kevko | SvenKieske: you can propose dockerfile and add installation steps inside dockerfile ...then it will not be the pip but docker ...solved | 12:13 |
| mnasiadka | isn't that what we wanted to do before this discussion? | 12:13 |
| mnasiadka | remove docker from requirements.txt and amend the docs? | 12:13 |
| SvenKieske | kevko: why would I fix things you break? do it yourself, ty. | 12:13 |
| SvenKieske | before this discussion nobody seemed to be aware that we either need instructions to install podman-py or include it in requirements.txt | 12:14 |
| opendevreview | Bartosz Bezak proposed openstack/kayobe master: CI: Use nested nodepools for jobs that are spawning vms https://review.opendev.org/c/openstack/kayobe/+/872647 | 12:14 |
| SvenKieske | there was only an error logged in kolla-build if the lib is not there. | 12:14 |
| mnasiadka | SvenKieske: I was aware, frickler was aware because we discussed that at least two times :) | 12:14 |
| SvenKieske | technically that's also sufficient? who needs docs anyway? openstack users now to read the sourcecode anyway | 12:14 |
| kevko | SvenKieske: I didn't break anything ...I didn't touch requirements repo ... i've just invested my time to get podman patch merged ...and it's optional | 12:15 |
| SvenKieske | but you didn't raise this topics in the code review | 12:15 |
| kevko | i am ok to not touch anything ...for me help is enough :D | 12:15 |
| SvenKieske | we could also get rid of the error log, it's technically not required :) | 12:15 |
| SvenKieske | I'm sure we only want the smartest users | 12:15 |
| mnasiadka | I love those discussions that don't lead to anything | 12:16 |
| mnasiadka | can we focus on THE problem? | 12:16 |
| SvenKieske | yes, they lead to me not patching the docs :) | 12:16 |
| mnasiadka | ah, that's your plan | 12:16 |
| SvenKieske | you said there is no problem? so discussion is over. I'm just replying because you keep typing stuff into keyboards. | 12:16 |
| mnasiadka | you could have responded to kevko that you will not help him - case solved | 12:16 |
| mnasiadka | instead of wasting bandwidth | 12:17 |
| kevko | okay, i will probably not touch podman from this moment .. tired fs rom thi | 12:17 |
| mnasiadka | lol | 12:17 |
| mnasiadka | SvenKieske: see what you've done? | 12:17 |
| SvenKieske | okay, let me restate what happened, just to correct it for historical purposes: I tried to help with the docs. I noticed that we have no requirements for podman-py, I asked what's to be done if it should be added to requirements.txt | 12:18 |
| mnasiadka | and then frickler proposed we don't need to add it, we can remove docker - so the user can choose whatever he should install based on docs or the wind direction | 12:18 |
| SvenKieske | this let to neither podman-py being added to requirements, nor to put it in docs, but to docker-py to also be removed, making more work for the docs. | 12:18 |
| mnasiadka | I can do that work, if you're complaining on the amount of work that needs to be done - no problem | 12:20 |
| SvenKieske | and I frankly don't agree with this, it's user unfriendly. it's technically strictly worse to have to type "pip install kolla podman-py" than to omit the last part. | 12:20 |
| mnasiadka | we just need to agree what do we want to do :) | 12:20 |
| mnasiadka | well, we shouldn't even direct users to do pip install kolla | 12:20 |
| SvenKieske | you also need to list in the docs now the lower and upper tested versions of podman-py and docker-py | 12:20 |
| SvenKieske | now you lost me, how are users supposed to install kolla if not by pip? | 12:21 |
| mnasiadka | I'm saying that by pip, but not use the version in pypi - because we're crappy at doing point releases | 12:21 |
| mnasiadka | (but that's another topic) | 12:22 |
| SvenKieske | that's another can of worms I don't want to talk about now, can we maybe stick to one topic, that already got expanded to two topicsß | 12:22 |
| mnasiadka | So, I understand that SvenKieske opts for adding podman-py to requirements.txt | 12:23 |
| SvenKieske | yeah, that way we don't even need to change the docs, at least for the installation part | 12:23 |
| frickler | but these are all interrelated. like for my downstream, we wrap all this in a container build, no issue at all | 12:23 |
| mnasiadka | kevko, frickler, bbezak - what do you choose - adding podman-py to requirements.txt or dropping docker from requirements.txt and asking users in docs to install docker-py or podman-py? | 12:23 |
| SvenKieske | I'd even volunteer to provide a patch for u-c, if that is needed | 12:23 |
| SvenKieske | frickler: sure, but we should not make upstream more complicated to sell our wrapper, no? ;) | 12:24 |
| SvenKieske | the optimal solution would be if pip would understand optional dependencies, but I guess such a thing doesn't exist? | 12:25 |
| mnasiadka | well, there's option number three - we install podman or docker based on what user wants to do using subprocess ;-) | 12:25 |
| SvenKieske | ah, that seems to exists, called "extras" in pip | 12:25 |
| frickler | well that's the extra sections I mentioned. you would "pip install kolla[podman]" or kolla[docker] | 12:25 |
| mnasiadka | ah, you mean pip install kolla[podman]? | 12:25 |
| SvenKieske | that's actually nice, no? is that something that's accepted in openstack? | 12:26 |
| mnasiadka | well, what happens if user does pip install kolla without extras? | 12:26 |
| mnasiadka | no docker and no podman? :) | 12:26 |
| SvenKieske | borked kolla? :D but then you can point at docs? or install docker if it's omitted? | 12:27 |
| frickler | they'll get an error when they try to run it, telling them they have to make a choice | 12:27 |
| SvenKieske | I mean docker is historically the default | 12:27 |
| SvenKieske | I like the extras stuff, if that works and has no weird problems with u-c and pbr and stuff | 12:28 |
| SvenKieske | which I really don't know, not a pip expert. | 12:28 |
| frickler | but it was you who demanded equal treatment for podman | 12:28 |
| frickler | I'd need to check the u-c situation for extras | 12:29 |
| SvenKieske | yeah I still think it would not hurt to just install it. but I can do compromises, actually :) | 12:29 |
| SvenKieske | and it's technically better to install something only if it's needed I get that, e.g. for security reasons. | 12:29 |
| * SvenKieske did run gentoo for far too long | 12:30 | |
| mnasiadka | I still think an error message saying "Package podman-py not present, please install it by running: python -m pip install podman-py" would be sufficient for majority of the population | 12:30 |
| SvenKieske | for kolla-build that might be true, but on the other hand, many people just run this stuff in CI pipelines, I guess they need to dig that out of the logs than, but I'm agreeing..somewhat | 12:31 |
| SvenKieske | I don't want to waste your time anymore on this, whatever. | 12:32 |
| mnasiadka | CI pipelines fail every week because some external dependency went crazy | 12:32 |
| mnasiadka | and we're not backporting this approach, so I don't see a problem | 12:32 |
| SvenKieske | yeah, let's add one more to the list ;) | 12:32 |
| SvenKieske | sure | 12:32 |
| mnasiadka | well, I can see it, but it's easily fixable from a downstream CI perspective ;) | 12:33 |
| SvenKieske | as I got the file already open I'll sketch a doc patch with rudimentary instructions | 12:33 |
| mnasiadka | the extras might be nice, but we would need to drop docker from requirements.txt | 12:34 |
| mnasiadka | so the effect is most probably the same | 12:34 |
| mnasiadka | just we give a convenient "pip3 install kolla[docker]" or "pip3 install kolla[podman]" | 12:34 |
| mnasiadka | or we go the requirements route and add podman-py there :D | 12:35 |
| opendevreview | Merged openstack/kayobe master: CI: Migrate RabbitMQ queues during upgrade tasks https://review.opendev.org/c/openstack/kayobe/+/893137 | 12:36 |
| opendevreview | Merged openstack/kayobe master: Add python 3.10 to setup.cfg metadata https://review.opendev.org/c/openstack/kayobe/+/886546 | 12:36 |
| SvenKieske | I'll just go with your original suggestion - for the docs - to add the instructions there. | 12:36 |
| SvenKieske | imho requirements.txt changes should go into the kolla build patch | 12:36 |
| opendevreview | Merged openstack/kayobe master: Remove firewall-debian role https://review.opendev.org/c/openstack/kayobe/+/883858 | 12:39 |
| opendevreview | Merged openstack/kayobe-config-dev master: Rename disable_selinux_do_reboot https://review.opendev.org/c/openstack/kayobe-config-dev/+/883368 | 12:39 |
| opendevreview | Sven Kieske proposed openstack/kolla master: [docs] add instructions for optional dependencies https://review.opendev.org/c/openstack/kolla/+/896081 | 12:39 |
| opendevreview | Sven Kieske proposed openstack/kolla master: [docs] add instructions for optional dependencies https://review.opendev.org/c/openstack/kolla/+/896081 | 12:41 |
| opendevreview | Sven Kieske proposed openstack/kolla master: [docs] add instructions for optional dependencies https://review.opendev.org/c/openstack/kolla/+/896081 | 12:42 |
| opendevreview | Merged openstack/kayobe master: Fix ipa_build_dib_elements_default comments https://review.opendev.org/c/openstack/kayobe/+/894765 | 12:58 |
| opendevreview | Merged openstack/kayobe-config master: kayobe-env: Unstick KOLLA_SOURCE_PATH and KOLLA_VENV_PATH https://review.opendev.org/c/openstack/kayobe-config/+/895888 | 12:58 |
| opendevreview | Pierre Riteau proposed openstack/kayobe-config-dev stable/2023.1: Rename disable_selinux_do_reboot https://review.opendev.org/c/openstack/kayobe-config-dev/+/896105 | 13:32 |
| opendevreview | Pierre Riteau proposed openstack/kayobe-config-dev stable/zed: Rename disable_selinux_do_reboot https://review.opendev.org/c/openstack/kayobe-config-dev/+/896106 | 13:32 |
| opendevreview | Will Szumski proposed openstack/kolla-ansible master: Support exposing prometheus_server externally https://review.opendev.org/c/openstack/kolla-ansible/+/828695 | 14:57 |
| opendevreview | Merged openstack/kayobe stable/yoga: Limit interface names to 15 characters https://review.opendev.org/c/openstack/kayobe/+/884541 | 15:19 |
| -opendevstatus- NOTICE: The lists.openinfra.dev and lists.starlingx.io sites will be offline briefly for migration to a new server | 15:31 | |
| SvenKieske | mhm, would it be possible to drop "kolla-ansible-ubuntu-binary-upgrade" job for zed release? is that even still supported? | 15:45 |
| SvenKieske | I assume it's testing binary releases? | 15:45 |
| SvenKieske | just noticed because it failed here: https://review.opendev.org/c/openstack/kolla-ansible/+/895380 | 15:46 |
| SvenKieske | ah, is that needed for yoga upgrade? because binary is not in zed support matrix: https://docs.openstack.org/kolla/zed/support_matrix | 15:47 |
| SvenKieske | I never did run binary containers, how is the upgrade story there, going from binary to source release? | 15:48 |
| SvenKieske | ah I see, this _is_ the binary->source upgrade test: https://review.opendev.org/c/openstack/kolla-ansible/+/839047 | 15:52 |
| opendevreview | Merged openstack/kayobe-config-dev stable/2023.1: Rename disable_selinux_do_reboot https://review.opendev.org/c/openstack/kayobe-config-dev/+/896105 | 16:27 |
| opendevreview | Pierre Riteau proposed openstack/kayobe master: Fix data file path detection with new pip https://review.opendev.org/c/openstack/kayobe/+/894831 | 16:29 |
| opendevreview | Merged openstack/kayobe-config-dev stable/zed: Rename disable_selinux_do_reboot https://review.opendev.org/c/openstack/kayobe-config-dev/+/896106 | 16:57 |
| opendevreview | Maksim Malchuk proposed openstack/kayobe-config stable/2023.1: kayobe-env: Unstick KOLLA_SOURCE_PATH and KOLLA_VENV_PATH https://review.opendev.org/c/openstack/kayobe-config/+/896114 | 17:52 |
| opendevreview | Maksim Malchuk proposed openstack/kayobe-config stable/zed: kayobe-env: Unstick KOLLA_SOURCE_PATH and KOLLA_VENV_PATH https://review.opendev.org/c/openstack/kayobe-config/+/896115 | 17:52 |
| opendevreview | Maksim Malchuk proposed openstack/kayobe-config stable/yoga: kayobe-env: Unstick KOLLA_SOURCE_PATH and KOLLA_VENV_PATH https://review.opendev.org/c/openstack/kayobe-config/+/896116 | 17:53 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!