Wednesday, 2021-11-17

*** brinzhang_ is now known as brinzhang07:26
mnasiadkamorning07:44
mnasiadkayoctozepto, 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/+/79820507:45
*** amoralej|off is now known as amoralej08:08
opendevreviewMark 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/+/81815509:05
holtgreweCan 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
mgoddardmnasiadka: rocky/stein???09:19
mgoddardholtgrewe: https://docs.openstack.org/kayobe/latest/configuration/reference/hosts.html#users-and-groups09:19
holtgrewemgoddard, 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
mgoddardholtgrewe: +109:23
opendevreviewMerged openstack/kolla-ansible stable/xena: docs: Parameterize kolla-ansible version and branch  https://review.opendev.org/c/openstack/kolla-ansible/+/81783909:26
opendevreviewMerged openstack/kolla-ansible stable/xena: docs: Get release name dynamically  https://review.opendev.org/c/openstack/kolla-ansible/+/81784009:26
opendevreviewMerged openstack/kolla-ansible stable/wallaby: docs: Parameterize kolla-ansible version and branch  https://review.opendev.org/c/openstack/kolla-ansible/+/81784109:26
opendevreviewMerged openstack/kolla-ansible stable/wallaby: docs: Get release name dynamically  https://review.opendev.org/c/openstack/kolla-ansible/+/81784209:26
opendevreviewMerged openstack/kolla-ansible stable/victoria: docs: Parameterize kolla-ansible version and branch  https://review.opendev.org/c/openstack/kolla-ansible/+/81784309:27
opendevreviewMerged openstack/kolla-ansible stable/victoria: docs: Get release name dynamically  https://review.opendev.org/c/openstack/kolla-ansible/+/81784409:27
mnasiadkamgoddard: http://lists.openstack.org/pipermail/openstack-discuss/2021-November/025798.html09:29
mnasiadkaand https://zuul.opendev.org/t/openstack/config-errors09:30
mnasiadkastable/rocky and stein use unallowed ansible_python_interpreter ;-)09:30
mnasiadkaso 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
opendevreviewIvan Halomi proposed openstack/kolla-ansible master: Refactor of kolla_docker into module_utils  https://review.opendev.org/c/openstack/kolla-ansible/+/81821210:27
hrwmorning10:37
hrwhttps://marcin.juszkiewicz.com.pl/2017/11/17/i-am-now-core-reviewer-in-kolla/ - 4 years passed by ;D10:38
opendevreviewIvan Halomi proposed openstack/kolla-ansible master: Refactor of kolla_docker into module_utils  https://review.opendev.org/c/openstack/kolla-ansible/+/81821410:45
opendevreviewPierre Riteau proposed openstack/kayobe stable/xena: Infra VMs: fix some playbook host patterns  https://review.opendev.org/c/openstack/kayobe/+/81821710:55
mgoddardhrw: congrats11:00
mgoddardmnasiadka: I don't mind too much either way. I don't think we've seen patches for those for a while11:01
hrwthx11:11
opendevreviewlikui proposed openstack/kolla-ansible master: Use Docker healthchecks for ironic-neutron-agent services  https://review.opendev.org/c/openstack/kolla-ansible/+/81770611:21
mnasiadkamgoddard: so I'll raise eol patches and let's get over with those, I see tripleo did eol on those already.11:33
hrwso which is the oldest we care? victoria? ussuri?11:41
mgoddardhrw: train11:49
mgoddardhrw: well, what do you mean care?11:49
hrwsay something other than 'it is no longer supported'11:49
mgoddardfor backports, ussuri will soon be EM11:49
mgoddardto backport to victoria11:49
hrwok11:49
mgoddardwe'll accept patches back to train, if CI passes11:50
mnasiadkaI don't see anybody working on fixing train CI, so I think it's next in EOL line :) (but obviously not now)11:53
opendevreviewMerged openstack/kayobe master: Infra VMs: fix some playbook host patterns  https://review.opendev.org/c/openstack/kayobe/+/81802011:53
mgoddardmnasiadka: there is a patch from priteau that gets part way11:54
mnasiadkaYes, 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
priteauSorry, got too busy with other things11:55
jingvarwhat is correct sequence to update container image and config?12:08
jingvarno worries, I've done it12:26
hrwmnasiadka: 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
hrwmnasiadka: RDO migrates from cs8 to cs9 in y cycle by providing packages for both12:50
*** amoralej is now known as amoralej|lunch13:18
mnasiadkahrw: I think it's a very good topic for todays meeting ;-)13:18
hrwyeah13:18
mnasiadkaI think if we don't want to do cs9 - Yoga is the last release for cs8 binary13:19
hrwmnasiadka: added notes to whiteboard13:21
hrwmnasiadka: btw - we still do not have patch/reno about binary deprecations iirc?13:22
mnasiadkahrw: no, we don't - I think we have a list of preparation items agreed on the PTG13:36
opendevreviewMichal Nasiadka proposed openstack/kolla master: Drop root privileges for ovn  https://review.opendev.org/c/openstack/kolla/+/66511713:37
opendevreviewMichal Nasiadka proposed openstack/kolla master: Drop root privileges for ovn  https://review.opendev.org/c/openstack/kolla/+/66511713:38
opendevreviewMichal Nasiadka proposed openstack/kolla master: Drop root privileges for ovn  https://review.opendev.org/c/openstack/kolla/+/66511713:39
hrwmeeting is in 1:08, right?13:52
*** amoralej|lunch is now known as amoralej13:55
mgoddardhrw: yes13:56
opendevreviewMichal Nasiadka proposed openstack/kolla master: Drop root privileges for ovn  https://review.opendev.org/c/openstack/kolla/+/66511714:11
opendevreviewMartin Hiner proposed openstack/kolla-ansible master: Refactor of kolla_docker into module_utils  https://review.opendev.org/c/openstack/kolla-ansible/+/81795414:22
holtgreweI 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
opendevreviewMartin Hiner proposed openstack/kolla-ansible master: Refactor of kolla_docker into module_utils  https://review.opendev.org/c/openstack/kolla-ansible/+/81795414:47
holtgreweWithout doing this, I'm getting no bridge defined, when doing this, the interface is created with "TYPE=Ethernet"14:47
holtgreweor rather MichaelRigart.interfaces complains about "Interface brbond0 is of an unexpected type"14:49
hrwmgoddard mnasiadka hrw egonzalez yoctozepto rafaelweingartne cosmicsound osmanlicilegi bbezak parallax Fl1nt frickler adrian-a: meeting in few14:56
mnasiadkahrw: thanks, was busy on another computer ;)14:56
mnasiadka#startmeeting kolla15:00
opendevmeetMeeting 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
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
opendevmeetThe meeting name has been set to 'kolla'15:00
mnasiadka#topic rollcall15:00
adrian-a\o15:01
mgoddard\o15:01
yoctozeptoo/15:01
halomivao/15:02
Martin_Hinero/15:02
hrw[°][o]15:03
mnasiadka#topic agenda15:05
mnasiadka* Announcements15:05
mnasiadka* Review action items from the last meeting15:05
mnasiadka* CI status15:05
mnasiadka* Release tasks15:05
mnasiadka* Backports & stable releases15:05
mnasiadka* Yoga cycle planning15:05
mnasiadka* [infra] Need to fix Zuul errors15:05
mnasiadka* CentOS plans15:05
mnasiadka* Open discussion15:05
mnasiadka#topic Announcements15:05
mnasiadkaNo announcements today.15:06
mnasiadka#topic Review action items from the last meeting15:06
mnasiadkaaction on mnasiadka to action security bugs - did nada - because I was on vacation :)15:08
mnasiadkaaction on yoctozepto to  hide properly init-runonce 15:08
yoctozeptoI have not done mine either15:08
mnasiadka#action mnasiadka to triage security bugs and update them with resolution plan (if needed)15:08
mnasiadka#action yoctozepto hide properly init-runonce15:08
mnasiadka#topic CI Status15:08
mnasiadkaI see all green15:09
mnasiadka(we'll cover the zuul config errors later)15:09
mnasiadka#topic Release tasks15:09
yoctozeptowe should release xena15:10
priteauThere are a few backports still waiting merge for kayobe15:10
mnasiadkaYes, but we need an RC3 for Kayobe first15:10
priteauhttps://review.opendev.org/c/openstack/kayobe/+/818217/15:10
mnasiadkawe can although release finals for Kolla and Kolla-Ansible, right?15:11
yoctozeptoyeah, that's my point15:12
yoctozeptolet's do it15:12
mnasiadkapriteau: let's get a list of Kayobe RC3 pending changes on the whiteboard15:12
mnasiadkaIt'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
hrwmnasiadka: make kaybobe-release-blockers topic?15:12
mnasiadkaSo we should be fine15:12
mnasiadkahrw: yes, that also works, we'll work it out :)15:13
hrwthat way one link to put ;D15:13
mnasiadkaok then, let's continue15:14
mnasiadka#topic Backports & stable releases15:14
mnasiadkathe whiteboard says: "We are probably overdue a backport session (see L248 for last checked commits)."15:14
mnasiadkaAfter backports merge, we should release stable branches.15:14
mnasiadkaSo - any volunteer to go through backports and make sure those are done?15:15
yoctozeptoso many backports to be seen then :D15:15
yoctozeptonot me, /me has too much on his plate atm15:15
mnasiadkaI have a lot on my plate as well, so would prefer someone else to take it ;)15:16
yoctozeptoneed more plates :D15:17
mnasiadkato break ;)15:17
yoctozeptohopefully not15:17
mnasiadkaWell, if there are no volunteers, it will just need to wait a bit15: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 planning15:19
mnasiadkaI think we need to go through the prereqs for deprecating binary - if we're going to deprecate it early this cycle.15:20
mnasiadkaL349 in https://etherpad.opendev.org/p/kolla-yoga-ptg15:21
mnasiadkaAny volunteers for the tasks there?15:23
yoctozeptotough crowd today15:25
mnasiadkaI guess15:25
mnasiadkaAnybody wants to write the reno about deprecating?15:26
hrwI want to stay in quiet corner and wait for end of the world.15:26
mnasiadkaOk then, so let's wait until binary deprecates by itself :)15:26
mnasiadkaLet's continue if there are no volunteers15:27
mnasiadka#topic [infra] Need to fix Zuul errors15:27
yoctozeptofor stein and rocky15:27
mnasiadkaseems we have bad configs in kolla-ansible stable/rocky and stable/stein15:27
yoctozeptoso I guess it's best to eol them15:28
mnasiadkaWe had a discussion earlier today about it, and I think we EOL15:28
yoctozeptoas train is still unhappy ;-)15:28
yoctozeptoall right, I agree15:28
yoctozeptoanyone disagrees?15:28
mnasiadkapriteau: is on a mission to save the train, but maybe not now ;)15:28
priteauI made some progress during train upgrades, but once all our customers are beyond train I don't know if there will be much motivation :P15:29
mnasiadkathen we can EOL Train and forget about Python2 :)15:29
hrwwhat is a name of Z cycle?15:29
priteaunot decided yet15:29
hrwok15:29
mnasiadkaok, no protests, I'll raise EOL changes for rocky and stein15:30
mnasiadkafor both k-a and kolla, right?15:30
opendevreviewMarcin Juszkiewicz proposed openstack/kolla master: reno: binary images are now deprecated  https://review.opendev.org/c/openstack/kolla/+/81824015:31
mnasiadka#action mnasiadka to EOL rocky and stein for both kolla and kolla-ansible15:31
hrwpatch landed. message needs rewriting probably but patch exists15:31
mnasiadkaok, that's a start15:31
mnasiadkathanks hrw15:32
mnasiadka#topic CentOS plans15:32
mnasiadkabinary images gets deprecated in Y, dropped in Z15:32
mnasiadkathat was the plan I guess15:32
mnasiadkabut15:32
mnasiadkaRDO moves from CS8 to CS9 in Y cycle15:32
mnasiadkapackages for both will be provided to simplify migration15:32
mnasiadkathat means Yoga is the last cycle with cs8+binary for Kolla? Since we opted-out of cs9 migration?15:33
hrwso if we decide to go CS8 to the end of CentOS in Kolla then Y is cs8/binary+source and Z is cs8/source only15:33
hrwand as we migrate to "latest binary" in R-late then it looks like sane plan15:34
mnasiadkaseems sane to me15:35
mnasiadkaanybody disagrees?15:35
yoctozeptomakes sense15:35
yoctozeptojust need to deprecate it properly15:35
yoctozeptoand glad we can support it until then15:36
hrwlooks 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 possible15:36
hrwYoga has binary deprecation, Z has binary drop and !debian deprecation, A has only one set of images15:37
mnasiadkaYes, but let's focus on getting rid of binary for now.15:37
hrwyes, just wanted to mention15:37
mnasiadkaok, no protesters again15:38
hrwso cs8 to the end15:38
mnasiadka#agreed to not pursue cs9 (as agreed on the PTG) and Z release will use cs8+source (for centos)15:39
mnasiadkaI guess that concludes the CentOS discussion15:39
mnasiadka#topic Open discussion15:39
hrwnow #action inform openstack ml about binary deprecation and centos plans15:39
hrw?15:39
mnasiadkathat's already in agreed PTG action items15:40
hrwah, right15:40
mnasiadkaSo it's on my list :)15:40
mnasiadkaso - open discussion - anybody has any points?15:40
hrwkevko goes for core - vote on ML if not done yet15:40
mgoddardhave we done the necessary user feedback gathering to start deprecating things?15:41
mnasiadkalooking at the PTG etherpad - we wanted first to deprecate binary, and then send a mail - should we do it other way around?15:42
hrwwe not merged it yet15:42
mgoddardI think we wanted to deprecate in Xena, but it didn't happen15:43
mgoddardin which case I'd say discuss first15:43
mnasiadkaWell, 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 maintained15:44
mnasiadkaIs that what we want to do?15:44
mgoddardI'd suggest including the single distro plan too15: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
hrwsorry, too much [sarcasm emoji] recently15:46
mnasiadkaI'm not in favor of sending two bombs at the same time.15:46
mgoddardyou'd rather send another one next week, saying 'oh, by the way, you can't use your current distro either?'15:47
mgoddardit's one plan15:47
mnasiadkaI don't believe it's one plan, it's not as in we do both or nothing.15:48
hrwit is more or less one plan. just split on 3-4 cycles15:49
mnasiadkaI can include both in one mail, but I think it will bring more confusion than real user feedback.15:49
hrwfrom what is now"15:49
hrwfrom "what is now" to "one set of images"15:49
mgoddardwe are reducing 6 sets of images to one15:49
mnasiadkathat's long way to go15:50
hrwbut that's one plan15:50
hrwhere is a plan. we split it to parts. this cycle gets part1, then part2, then part315:50
mgoddardyes, and people need to know how their setup will end up. Why let them think they can use centos/source?15:50
hrwespecially when they jump releases15:51
mgoddardmost people would want to switch straight to the chosen image config, rather than via another type15:51
hrwwallaby centos/binary -> xena centos/binary -> yoga centos/binary -> zeus centos/source -> athena debian/source would require planning15:52
hrwOne Set to rule them all15:53
mnasiadkaHonestly, 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
hrwok, can I get other topic?15:56
mnasiadkaSure15:56
hrwdid someone use docker squash support?15:57
headphoneJamesI have a patch to towards system scope in keystone ready for review: https://review.opendev.org/c/openstack/kolla-ansible/+/81557715:57
hrwcan we kill it?15:57
mnasiadkawhat's the problem with it?15:58
mgoddardheadphoneJames: I reviewed today15:58
headphoneJamesmgoddard: thanks!15:59
hrwmnasiadka: just wondering how much it is used. if no one uses it then we do not even know does it work15:59
mnasiadkawell, I would say it might be useful for some people, but if it's not tested - then probably it doesn't work ;)16:00
mnasiadkaWhat'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
hrwmnasiadka: test2/nova-libvirt:13.1.0 has 29 layers16:02
mnasiadkaok, we're over time, let's continue after the meeting16:02
mgoddardhrw: I don't think it's hurting anyone16:02
mnasiadka#endmeeting16:02
opendevmeetMeeting ended Wed Nov 17 16:02:53 2021 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:02
opendevmeetMinutes:        https://meetings.opendev.org/meetings/kolla/2021/kolla.2021-11-17-15.00.html16:02
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/kolla/2021/kolla.2021-11-17-15.00.txt16:02
opendevmeetLog:            https://meetings.opendev.org/meetings/kolla/2021/kolla.2021-11-17-15.00.log.html16:02
mnasiadkawell, it's not hurting, and nobody raised a bug that it doesn't work16:03
mnasiadkaat least not that I know of16:03
hrwlet me test16:04
hrwbtw - amount of changes in kolla/ to get rid of distros and binary is quite small. most will be in docker/16:05
hrwok, docker squashing still works16:05
hrwtest/nova-libvirt:13.1.0 has 3 layers16:06
halomivacould 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
hrwhalomiva: tox says "argh"16:12
halomivawe 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
halomivawe were told to split systemd integration for container manipulation into more small changes so we want to know if this one is small enough16:17
hrwI try to stay away from k-a code but so far it looks fine to me as it is just move of code between files16:17
mgoddardhalomiva: I'd like to see it be even more simple16:20
mgoddardhalomiva: I'm running git diff origin/master:ansible/library/kolla_docker.py ansible/module_utils/docker_worker.py16:21
mgoddardhalomiva: I'd like that to be small - first patch simply copies DockerWorker class to docker_worker16:21
mgoddardhalomiva: as I said before, IMO refactors should be separate from code changes16:21
halomivalook at the first patchset, is that what you mean? 16:23
halomivashould we post adding container worker as another change and not as next patchset?16:23
mgoddardholtgrewe: I'm looking at https://review.opendev.org/c/openstack/kolla-ansible/+/81795416:26
mgoddardsorry, that should have been halomiva 16:26
mgoddard+1 for separate change16:26
* hrw out16:29
opendevreviewMerged openstack/kolla-ansible stable/xena: The deprecated iscsi deploy interface has been removed since xena  https://review.opendev.org/c/openstack/kolla-ansible/+/81815516:36
opendevreviewMerged openstack/kayobe-config-dev stable/xena: Define infra VMs for testing  https://review.opendev.org/c/openstack/kayobe-config-dev/+/81767217:11
opendevreviewMerged openstack/kayobe stable/xena: CI: add Infra VM jobs  https://review.opendev.org/c/openstack/kayobe/+/81767317:36
*** amoralej is now known as amoralej|off17:55
opendevreviewMerged openstack/kayobe stable/xena: Infra VMs: fix some playbook host patterns  https://review.opendev.org/c/openstack/kayobe/+/81821718:39
opendevreviewMark Goddard proposed openstack/kayobe master: Introduce max fail percentage to playbooks  https://review.opendev.org/c/openstack/kayobe/+/81828820:46
opendevreviewMark Goddard proposed openstack/kayobe master: Refactor host configure commands to use a single playbook  https://review.opendev.org/c/openstack/kayobe/+/81828920:46
opendevreviewMark Goddard proposed openstack/kayobe master: Import kolla-ansible baremetal role  https://review.opendev.org/c/openstack/kayobe/+/81829020:46
opendevreviewMerged openstack/kayobe-config-dev stable/xena: Synchronise job config with kayobe  https://review.opendev.org/c/openstack/kayobe-config-dev/+/81767423:25

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