*** brinzhang_ is now known as brinzhang | 07:26 | |
mnasiadka | morning | 07:44 |
---|---|---|
mnasiadka | yoctozepto, mgoddard: regarding the Zuul config issue in rocky/stein - are we backporting the train CI change, or just abandoning CI there? https://review.opendev.org/c/openstack/kolla-ansible/+/798205 | 07:45 |
*** amoralej|off is now known as amoralej | 08:08 | |
opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/xena: The deprecated iscsi deploy interface has been removed since xena https://review.opendev.org/c/openstack/kolla-ansible/+/818155 | 09:05 |
holtgrewe | Can I make kayobe add additional keys to the centos user on the deployed controllers? Or would I simply re-define the centos user? | 09:13 |
mgoddard | mnasiadka: rocky/stein??? | 09:19 |
mgoddard | holtgrewe: https://docs.openstack.org/kayobe/latest/configuration/reference/hosts.html#users-and-groups | 09:19 |
holtgrewe | mgoddard, thanks, so I just add user centos there and put the list of additional keys that I want. That appears to work and is the recommended way as I understand by the documentation. | 09:23 |
mgoddard | holtgrewe: +1 | 09:23 |
opendevreview | Merged openstack/kolla-ansible stable/xena: docs: Parameterize kolla-ansible version and branch https://review.opendev.org/c/openstack/kolla-ansible/+/817839 | 09:26 |
opendevreview | Merged openstack/kolla-ansible stable/xena: docs: Get release name dynamically https://review.opendev.org/c/openstack/kolla-ansible/+/817840 | 09:26 |
opendevreview | Merged openstack/kolla-ansible stable/wallaby: docs: Parameterize kolla-ansible version and branch https://review.opendev.org/c/openstack/kolla-ansible/+/817841 | 09:26 |
opendevreview | Merged openstack/kolla-ansible stable/wallaby: docs: Get release name dynamically https://review.opendev.org/c/openstack/kolla-ansible/+/817842 | 09:26 |
opendevreview | Merged openstack/kolla-ansible stable/victoria: docs: Parameterize kolla-ansible version and branch https://review.opendev.org/c/openstack/kolla-ansible/+/817843 | 09:27 |
opendevreview | Merged openstack/kolla-ansible stable/victoria: docs: Get release name dynamically https://review.opendev.org/c/openstack/kolla-ansible/+/817844 | 09:27 |
mnasiadka | mgoddard: http://lists.openstack.org/pipermail/openstack-discuss/2021-November/025798.html | 09:29 |
mnasiadka | and https://zuul.opendev.org/t/openstack/config-errors | 09:30 |
mnasiadka | stable/rocky and stein use unallowed ansible_python_interpreter ;-) | 09:30 |
mnasiadka | so either we stop CI there (which probably means marking them as EOL) - or we work towards making them work (which I doubt anybody wants) | 09:30 |
opendevreview | Ivan Halomi proposed openstack/kolla-ansible master: Refactor of kolla_docker into module_utils https://review.opendev.org/c/openstack/kolla-ansible/+/818212 | 10:27 |
hrw | morning | 10:37 |
hrw | https://marcin.juszkiewicz.com.pl/2017/11/17/i-am-now-core-reviewer-in-kolla/ - 4 years passed by ;D | 10:38 |
opendevreview | Ivan Halomi proposed openstack/kolla-ansible master: Refactor of kolla_docker into module_utils https://review.opendev.org/c/openstack/kolla-ansible/+/818214 | 10:45 |
opendevreview | Pierre Riteau proposed openstack/kayobe stable/xena: Infra VMs: fix some playbook host patterns https://review.opendev.org/c/openstack/kayobe/+/818217 | 10:55 |
mgoddard | hrw: congrats | 11:00 |
mgoddard | mnasiadka: I don't mind too much either way. I don't think we've seen patches for those for a while | 11:01 |
hrw | thx | 11:11 |
opendevreview | likui proposed openstack/kolla-ansible master: Use Docker healthchecks for ironic-neutron-agent services https://review.opendev.org/c/openstack/kolla-ansible/+/817706 | 11:21 |
mnasiadka | mgoddard: so I'll raise eol patches and let's get over with those, I see tripleo did eol on those already. | 11:33 |
hrw | so which is the oldest we care? victoria? ussuri? | 11:41 |
mgoddard | hrw: train | 11:49 |
mgoddard | hrw: well, what do you mean care? | 11:49 |
hrw | say something other than 'it is no longer supported' | 11:49 |
mgoddard | for backports, ussuri will soon be EM | 11:49 |
mgoddard | to backport to victoria | 11:49 |
hrw | ok | 11:49 |
mgoddard | we'll accept patches back to train, if CI passes | 11:50 |
mnasiadka | I don't see anybody working on fixing train CI, so I think it's next in EOL line :) (but obviously not now) | 11:53 |
opendevreview | Merged openstack/kayobe master: Infra VMs: fix some playbook host patterns https://review.opendev.org/c/openstack/kayobe/+/818020 | 11:53 |
mgoddard | mnasiadka: there is a patch from priteau that gets part way | 11:54 |
mnasiadka | Yes, but I don't think it's been updated recently, that's what I meant - let's see if we get anywhere with it before years end. | 11:55 |
priteau | Sorry, got too busy with other things | 11:55 |
jingvar | what is correct sequence to update container image and config? | 12:08 |
jingvar | no worries, I've done it | 12:26 |
hrw | mnasiadka: what is a plan for centos in y/z? we deprecate binaries in y and centos in z == we need binary for y/z and centos for y/z/a, right? | 12:49 |
hrw | mnasiadka: RDO migrates from cs8 to cs9 in y cycle by providing packages for both | 12:50 |
*** amoralej is now known as amoralej|lunch | 13:18 | |
mnasiadka | hrw: I think it's a very good topic for todays meeting ;-) | 13:18 |
hrw | yeah | 13:18 |
mnasiadka | I think if we don't want to do cs9 - Yoga is the last release for cs8 binary | 13:19 |
hrw | mnasiadka: added notes to whiteboard | 13:21 |
hrw | mnasiadka: btw - we still do not have patch/reno about binary deprecations iirc? | 13:22 |
mnasiadka | hrw: no, we don't - I think we have a list of preparation items agreed on the PTG | 13:36 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: Drop root privileges for ovn https://review.opendev.org/c/openstack/kolla/+/665117 | 13:37 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: Drop root privileges for ovn https://review.opendev.org/c/openstack/kolla/+/665117 | 13:38 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: Drop root privileges for ovn https://review.opendev.org/c/openstack/kolla/+/665117 | 13:39 |
hrw | meeting is in 1:08, right? | 13:52 |
*** amoralej|lunch is now known as amoralej | 13:55 | |
mgoddard | hrw: yes | 13:56 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: Drop root privileges for ovn https://review.opendev.org/c/openstack/kolla/+/665117 | 14:11 |
opendevreview | Martin Hiner proposed openstack/kolla-ansible master: Refactor of kolla_docker into module_utils https://review.opendev.org/c/openstack/kolla-ansible/+/817954 | 14:22 |
holtgrewe | I am getting stuck on defining a network interface as a bridge on a bond on controllers. Do I have to add the bridge interface to controller_extra_network_interfaces? | 14:47 |
opendevreview | Martin Hiner proposed openstack/kolla-ansible master: Refactor of kolla_docker into module_utils https://review.opendev.org/c/openstack/kolla-ansible/+/817954 | 14:47 |
holtgrewe | Without doing this, I'm getting no bridge defined, when doing this, the interface is created with "TYPE=Ethernet" | 14:47 |
holtgrewe | or rather MichaelRigart.interfaces complains about "Interface brbond0 is of an unexpected type" | 14:49 |
hrw | mgoddard mnasiadka hrw egonzalez yoctozepto rafaelweingartne cosmicsound osmanlicilegi bbezak parallax Fl1nt frickler adrian-a: meeting in few | 14:56 |
mnasiadka | hrw: thanks, was busy on another computer ;) | 14:56 |
mnasiadka | #startmeeting kolla | 15:00 |
opendevmeet | Meeting started Wed Nov 17 15:00:11 2021 UTC and is due to finish in 60 minutes. The chair is mnasiadka. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
opendevmeet | The meeting name has been set to 'kolla' | 15:00 |
mnasiadka | #topic rollcall | 15:00 |
adrian-a | \o | 15:01 |
mgoddard | \o | 15:01 |
yoctozepto | o/ | 15:01 |
halomiva | o/ | 15:02 |
Martin_Hiner | o/ | 15:02 |
hrw | [°][o] | 15:03 |
mnasiadka | #topic agenda | 15:05 |
mnasiadka | * Announcements | 15:05 |
mnasiadka | * Review action items from the last meeting | 15:05 |
mnasiadka | * CI status | 15:05 |
mnasiadka | * Release tasks | 15:05 |
mnasiadka | * Backports & stable releases | 15:05 |
mnasiadka | * Yoga cycle planning | 15:05 |
mnasiadka | * [infra] Need to fix Zuul errors | 15:05 |
mnasiadka | * CentOS plans | 15:05 |
mnasiadka | * Open discussion | 15:05 |
mnasiadka | #topic Announcements | 15:05 |
mnasiadka | No announcements today. | 15:06 |
mnasiadka | #topic Review action items from the last meeting | 15:06 |
mnasiadka | action on mnasiadka to action security bugs - did nada - because I was on vacation :) | 15:08 |
mnasiadka | action on yoctozepto to hide properly init-runonce | 15:08 |
yoctozepto | I have not done mine either | 15:08 |
mnasiadka | #action mnasiadka to triage security bugs and update them with resolution plan (if needed) | 15:08 |
mnasiadka | #action yoctozepto hide properly init-runonce | 15:08 |
mnasiadka | #topic CI Status | 15:08 |
mnasiadka | I see all green | 15:09 |
mnasiadka | (we'll cover the zuul config errors later) | 15:09 |
mnasiadka | #topic Release tasks | 15:09 |
yoctozepto | we should release xena | 15:10 |
priteau | There are a few backports still waiting merge for kayobe | 15:10 |
mnasiadka | Yes, but we need an RC3 for Kayobe first | 15:10 |
priteau | https://review.opendev.org/c/openstack/kayobe/+/818217/ | 15:10 |
mnasiadka | we can although release finals for Kolla and Kolla-Ansible, right? | 15:11 |
yoctozepto | yeah, that's my point | 15:12 |
yoctozepto | let's do it | 15:12 |
mnasiadka | priteau: let's get a list of Kayobe RC3 pending changes on the whiteboard | 15:12 |
mnasiadka | It's R-19 week for Yoga, we have some R-17 items on our list - but I think there are some changes raised by mgoddard (looking at previous meeting logs) | 15:12 |
hrw | mnasiadka: make kaybobe-release-blockers topic? | 15:12 |
mnasiadka | So we should be fine | 15:12 |
mnasiadka | hrw: yes, that also works, we'll work it out :) | 15:13 |
hrw | that way one link to put ;D | 15:13 |
mnasiadka | ok then, let's continue | 15:14 |
mnasiadka | #topic Backports & stable releases | 15:14 |
mnasiadka | the whiteboard says: "We are probably overdue a backport session (see L248 for last checked commits)." | 15:14 |
mnasiadka | After backports merge, we should release stable branches. | 15:14 |
mnasiadka | So - any volunteer to go through backports and make sure those are done? | 15:15 |
yoctozepto | so many backports to be seen then :D | 15:15 |
yoctozepto | not me, /me has too much on his plate atm | 15:15 |
mnasiadka | I have a lot on my plate as well, so would prefer someone else to take it ;) | 15:16 |
yoctozepto | need more plates :D | 15:17 |
mnasiadka | to break ;) | 15:17 |
yoctozepto | hopefully not | 15:17 |
mnasiadka | Well, if there are no volunteers, it will just need to wait a bit | 15:18 |
mnasiadka | #action not forget to go through backports for stable branches (L248 on Whiteboard) and do stable releases afterwards. | 15:19 |
mnasiadka | #topic Yoga cycle planning | 15:19 |
mnasiadka | I think we need to go through the prereqs for deprecating binary - if we're going to deprecate it early this cycle. | 15:20 |
mnasiadka | L349 in https://etherpad.opendev.org/p/kolla-yoga-ptg | 15:21 |
mnasiadka | Any volunteers for the tasks there? | 15:23 |
yoctozepto | tough crowd today | 15:25 |
mnasiadka | I guess | 15:25 |
mnasiadka | Anybody wants to write the reno about deprecating? | 15:26 |
hrw | I want to stay in quiet corner and wait for end of the world. | 15:26 |
mnasiadka | Ok then, so let's wait until binary deprecates by itself :) | 15:26 |
mnasiadka | Let's continue if there are no volunteers | 15:27 |
mnasiadka | #topic [infra] Need to fix Zuul errors | 15:27 |
yoctozepto | for stein and rocky | 15:27 |
mnasiadka | seems we have bad configs in kolla-ansible stable/rocky and stable/stein | 15:27 |
yoctozepto | so I guess it's best to eol them | 15:28 |
mnasiadka | We had a discussion earlier today about it, and I think we EOL | 15:28 |
yoctozepto | as train is still unhappy ;-) | 15:28 |
yoctozepto | all right, I agree | 15:28 |
yoctozepto | anyone disagrees? | 15:28 |
mnasiadka | priteau: is on a mission to save the train, but maybe not now ;) | 15:28 |
priteau | I made some progress during train upgrades, but once all our customers are beyond train I don't know if there will be much motivation :P | 15:29 |
mnasiadka | then we can EOL Train and forget about Python2 :) | 15:29 |
hrw | what is a name of Z cycle? | 15:29 |
priteau | not decided yet | 15:29 |
hrw | ok | 15:29 |
mnasiadka | ok, no protests, I'll raise EOL changes for rocky and stein | 15:30 |
mnasiadka | for both k-a and kolla, right? | 15:30 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: reno: binary images are now deprecated https://review.opendev.org/c/openstack/kolla/+/818240 | 15:31 |
mnasiadka | #action mnasiadka to EOL rocky and stein for both kolla and kolla-ansible | 15:31 |
hrw | patch landed. message needs rewriting probably but patch exists | 15:31 |
mnasiadka | ok, that's a start | 15:31 |
mnasiadka | thanks hrw | 15:32 |
mnasiadka | #topic CentOS plans | 15:32 |
mnasiadka | binary images gets deprecated in Y, dropped in Z | 15:32 |
mnasiadka | that was the plan I guess | 15:32 |
mnasiadka | but | 15:32 |
mnasiadka | RDO moves from CS8 to CS9 in Y cycle | 15:32 |
mnasiadka | packages for both will be provided to simplify migration | 15:32 |
mnasiadka | that means Yoga is the last cycle with cs8+binary for Kolla? Since we opted-out of cs9 migration? | 15:33 |
hrw | so if we decide to go CS8 to the end of CentOS in Kolla then Y is cs8/binary+source and Z is cs8/source only | 15:33 |
hrw | and as we migrate to "latest binary" in R-late then it looks like sane plan | 15:34 |
mnasiadka | seems sane to me | 15:35 |
mnasiadka | anybody disagrees? | 15:35 |
yoctozepto | makes sense | 15:35 |
yoctozepto | just need to deprecate it properly | 15:35 |
yoctozepto | and glad we can support it until then | 15:36 |
hrw | looks like quick track: Yoga: cs8/binary+source, u2004/binary+source, d11/binary+source. Z: cs8/source, u2004/source, d11/source, A: d11/source is still possible | 15:36 |
hrw | Yoga has binary deprecation, Z has binary drop and !debian deprecation, A has only one set of images | 15:37 |
mnasiadka | Yes, but let's focus on getting rid of binary for now. | 15:37 |
hrw | yes, just wanted to mention | 15:37 |
mnasiadka | ok, no protesters again | 15:38 |
hrw | so cs8 to the end | 15:38 |
mnasiadka | #agreed to not pursue cs9 (as agreed on the PTG) and Z release will use cs8+source (for centos) | 15:39 |
mnasiadka | I guess that concludes the CentOS discussion | 15:39 |
mnasiadka | #topic Open discussion | 15:39 |
hrw | now #action inform openstack ml about binary deprecation and centos plans | 15:39 |
hrw | ? | 15:39 |
mnasiadka | that's already in agreed PTG action items | 15:40 |
hrw | ah, right | 15:40 |
mnasiadka | So it's on my list :) | 15:40 |
mnasiadka | so - open discussion - anybody has any points? | 15:40 |
hrw | kevko goes for core - vote on ML if not done yet | 15:40 |
mgoddard | have we done the necessary user feedback gathering to start deprecating things? | 15:41 |
mnasiadka | looking at the PTG etherpad - we wanted first to deprecate binary, and then send a mail - should we do it other way around? | 15:42 |
hrw | we not merged it yet | 15:42 |
mgoddard | I think we wanted to deprecate in Xena, but it didn't happen | 15:43 |
mgoddard | in which case I'd say discuss first | 15:43 |
mnasiadka | Well, I think it will finish like the kolla-ceph code removal, but I can send a mail to openstack-discuss and kolla-klub around a plan to deprecate binary and if anyone wants to step up and keep it maintained | 15:44 |
mnasiadka | Is that what we want to do? | 15:44 |
mgoddard | I'd suggest including the single distro plan too | 15:46 |
hrw | "Dear (potential and present) users. Due to size of our team we decided to drop support for building binary images. What do you think about it? Your beloved Kolla team" | 15:46 |
hrw | sorry, too much [sarcasm emoji] recently | 15:46 |
mnasiadka | I'm not in favor of sending two bombs at the same time. | 15:46 |
mgoddard | you'd rather send another one next week, saying 'oh, by the way, you can't use your current distro either?' | 15:47 |
mgoddard | it's one plan | 15:47 |
mnasiadka | I don't believe it's one plan, it's not as in we do both or nothing. | 15:48 |
hrw | it is more or less one plan. just split on 3-4 cycles | 15:49 |
mnasiadka | I can include both in one mail, but I think it will bring more confusion than real user feedback. | 15:49 |
hrw | from what is now" | 15:49 |
hrw | from "what is now" to "one set of images" | 15:49 |
mgoddard | we are reducing 6 sets of images to one | 15:49 |
mnasiadka | that's long way to go | 15:50 |
hrw | but that's one plan | 15:50 |
hrw | here is a plan. we split it to parts. this cycle gets part1, then part2, then part3 | 15:50 |
mgoddard | yes, and people need to know how their setup will end up. Why let them think they can use centos/source? | 15:50 |
hrw | especially when they jump releases | 15:51 |
mgoddard | most people would want to switch straight to the chosen image config, rather than via another type | 15:51 |
hrw | wallaby centos/binary -> xena centos/binary -> yoga centos/binary -> zeus centos/source -> athena debian/source would require planning | 15:52 |
hrw | One Set to rule them all | 15:53 |
mnasiadka | Honestly, we all know it's going to be shooting contest in the user feedback - but let me agree to send it out in one mail. | 15:55 |
hrw | ok, can I get other topic? | 15:56 |
mnasiadka | Sure | 15:56 |
hrw | did someone use docker squash support? | 15:57 |
headphoneJames | I have a patch to towards system scope in keystone ready for review: https://review.opendev.org/c/openstack/kolla-ansible/+/815577 | 15:57 |
hrw | can we kill it? | 15:57 |
mnasiadka | what's the problem with it? | 15:58 |
mgoddard | headphoneJames: I reviewed today | 15:58 |
headphoneJames | mgoddard: thanks! | 15:59 |
hrw | mnasiadka: just wondering how much it is used. if no one uses it then we do not even know does it work | 15:59 |
mnasiadka | well, I would say it might be useful for some people, but if it's not tested - then probably it doesn't work ;) | 16:00 |
mnasiadka | What's the usual number of Kolla image layers nowadays? Does it make sense to squash them if you add some things on top? | 16:01 |
hrw | mnasiadka: test2/nova-libvirt:13.1.0 has 29 layers | 16:02 |
mnasiadka | ok, we're over time, let's continue after the meeting | 16:02 |
mgoddard | hrw: I don't think it's hurting anyone | 16:02 |
mnasiadka | #endmeeting | 16:02 |
opendevmeet | Meeting ended Wed Nov 17 16:02:53 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:02 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/kolla/2021/kolla.2021-11-17-15.00.html | 16:02 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/kolla/2021/kolla.2021-11-17-15.00.txt | 16:02 |
opendevmeet | Log: https://meetings.opendev.org/meetings/kolla/2021/kolla.2021-11-17-15.00.log.html | 16:02 |
mnasiadka | well, it's not hurting, and nobody raised a bug that it doesn't work | 16:03 |
mnasiadka | at least not that I know of | 16:03 |
hrw | let me test | 16:04 |
hrw | btw - amount of changes in kolla/ to get rid of distros and binary is quite small. most will be in docker/ | 16:05 |
hrw | ok, docker squashing still works | 16:05 |
hrw | test/nova-libvirt:13.1.0 has 3 layers | 16:06 |
halomiva | could you please give us feedback on this refactor of kolla_docker https://review.opendev.org/c/openstack/kolla-ansible/+/817954 , so we can move on to adding podman same way? | 16:07 |
hrw | halomiva: tox says "argh" | 16:12 |
halomiva | we fixed tox in this patchset but we are looking more for feedback if we have right approach so we can move on for unit tests and systemd | 16:14 |
halomiva | we were told to split systemd integration for container manipulation into more small changes so we want to know if this one is small enough | 16:17 |
hrw | I try to stay away from k-a code but so far it looks fine to me as it is just move of code between files | 16:17 |
mgoddard | halomiva: I'd like to see it be even more simple | 16:20 |
mgoddard | halomiva: I'm running git diff origin/master:ansible/library/kolla_docker.py ansible/module_utils/docker_worker.py | 16:21 |
mgoddard | halomiva: I'd like that to be small - first patch simply copies DockerWorker class to docker_worker | 16:21 |
mgoddard | halomiva: as I said before, IMO refactors should be separate from code changes | 16:21 |
halomiva | look at the first patchset, is that what you mean? | 16:23 |
halomiva | should we post adding container worker as another change and not as next patchset? | 16:23 |
mgoddard | holtgrewe: I'm looking at https://review.opendev.org/c/openstack/kolla-ansible/+/817954 | 16:26 |
mgoddard | sorry, that should have been halomiva | 16:26 |
mgoddard | +1 for separate change | 16:26 |
* hrw out | 16:29 | |
opendevreview | Merged openstack/kolla-ansible stable/xena: The deprecated iscsi deploy interface has been removed since xena https://review.opendev.org/c/openstack/kolla-ansible/+/818155 | 16:36 |
opendevreview | Merged openstack/kayobe-config-dev stable/xena: Define infra VMs for testing https://review.opendev.org/c/openstack/kayobe-config-dev/+/817672 | 17:11 |
opendevreview | Merged openstack/kayobe stable/xena: CI: add Infra VM jobs https://review.opendev.org/c/openstack/kayobe/+/817673 | 17:36 |
*** amoralej is now known as amoralej|off | 17:55 | |
opendevreview | Merged openstack/kayobe stable/xena: Infra VMs: fix some playbook host patterns https://review.opendev.org/c/openstack/kayobe/+/818217 | 18:39 |
opendevreview | Mark Goddard proposed openstack/kayobe master: Introduce max fail percentage to playbooks https://review.opendev.org/c/openstack/kayobe/+/818288 | 20:46 |
opendevreview | Mark Goddard proposed openstack/kayobe master: Refactor host configure commands to use a single playbook https://review.opendev.org/c/openstack/kayobe/+/818289 | 20:46 |
opendevreview | Mark Goddard proposed openstack/kayobe master: Import kolla-ansible baremetal role https://review.opendev.org/c/openstack/kayobe/+/818290 | 20:46 |
opendevreview | Merged openstack/kayobe-config-dev stable/xena: Synchronise job config with kayobe https://review.opendev.org/c/openstack/kayobe-config-dev/+/817674 | 23:25 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!