Wednesday, 2022-11-02

*** mnasiadka_ is now known as mnasiadka06:29
*** parallax is now known as Guest22706:42
opendevreviewMerged openstack/kolla-ansible master: Fix ironic interface option for ironic-inspector  https://review.opendev.org/c/openstack/kolla-ansible/+/86302608:04
hkominosMorning all. Is there anybody online from the kayobe project?08:21
fricklerhkominos: best just ask your real question and wait for someone to answer08:55
hkominosfrickler: Good point08:58
hkominosWhen kayobe autodiscovers the nodes in the environment, (with inventory discover) the nodes are automatically assigned in the controller group Even if their name in ironic is compute. Despite my efforts I could not find a way with kayobe to tag the nodes as computes . Any ideas ?09:01
hkominosediting them in /inventory/overcloud has not had the desired effect09:01
*** parallax is now known as Guest23309:23
mnasiadkahkominos: use overcloud_group_hosts_map in etc/kayobe/overcloud.yml to map hosts to groups09:41
hkominosmnasiadka: thx!09:53
opendevreviewMark Goddard proposed openstack/kolla-ansible stable/yoga: Fix ironic interface option for ironic-inspector  https://review.opendev.org/c/openstack/kolla-ansible/+/86324810:39
opendevreviewMark Goddard proposed openstack/kolla-ansible stable/xena: Fix ironic interface option for ironic-inspector  https://review.opendev.org/c/openstack/kolla-ansible/+/86324910:39
opendevreviewMark Goddard proposed openstack/kolla-ansible stable/wallaby: Fix ironic interface option for ironic-inspector  https://review.opendev.org/c/openstack/kolla-ansible/+/86327010:39
thelounge44seems that skyline changes are waiting for review https://review.opendev.org/c/openstack/kolla-ansible/+/828464 , https://review.opendev.org/c/openstack/kolla/+/82694812:31
opendevreviewIvan Halomi proposed openstack/kolla-ansible master: Adding container engine to kolla_container_facts  https://review.opendev.org/c/openstack/kolla-ansible/+/86288312:47
*** Guest233 is now known as parallax13:15
mnasiadkaoh boy, DST changes again13:50
mnasiadkahrw, mgoddard, frickler, kevko: do we want the meeting in 10 minutes, or at the usual time (in 1hr 10 min)?13:51
kevkoi am here in both times13:52
mnasiadkalet's maybe use the usual time in 1hr, since one one reply :)14:00
mgoddardI don't mind14:02
fricklerI would prefer now, if we move to later we will have conflict with largescale sig14:03
mnasiadkaok, let's then just start, and agree on meeting time from next week onwards ;-)14:06
mnasiadka#startmeeting kolla14:06
opendevmeetMeeting started Wed Nov  2 14:06:35 2022 UTC and is due to finish in 60 minutes.  The chair is mnasiadka. Information about MeetBot at http://wiki.debian.org/MeetBot.14:06
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:06
opendevmeetThe meeting name has been set to 'kolla'14:06
mnasiadka#topic rollcall14:06
mnasiadkao/14:06
frickler\o14:07
mnasiadkakevko, mgoddard, hrw14:08
TimBeermann[m]O/14:08
ihalomi\o14:08
m_hinero/14:08
mgoddard\o14:10
mnasiadka#topic agenda14:12
mnasiadka* CI status14:12
mnasiadka* Release tasks14:12
mnasiadka* Regular stable releases (first meeting in a month)14:12
mnasiadka* Current cycle planning14:12
mnasiadka* Additional agenda (from whiteboard)14:12
mnasiadka* Open discussion14:12
mnasiadka#topic CI status14:12
mnasiadkawhiteboard says green14:12
mnasiadkaI haven't seen any spectacular failures today, so let's assume it is14:13
mnasiadka#topic Regular stable releases (first meeting in a month)14:13
mnasiadkaI think Wallaby went EM, so no more stable releases there14:13
mgoddarddid we have a final release?14:14
mnasiadkahmm, seems not since 6th Oct14:15
mnasiadkaI'll speak with the release team later if we can still do that - I was pretty sure we did14:16
mnasiadkaArgh, I missed one topic14:16
mnasiadka#topic Release tasks14:16
mnasiadkaWe can't branch yet, still kolla/kolla-ansible/kayobe needs opensearch14:16
mnasiadka#topic Current cycle planning14:17
fricklerwhat about other stable branch releases?14:17
fricklerI can create a patch unless someone else wants to14:17
mnasiadkaAh right, we should do other stable branch releases as we do every month14:17
fricklerok, so I'll do that14:18
mnasiadkathanks14:18
mnasiadkaOk, so current cycle for us is still Zed14:19
mnasiadkaWe have opensearch to review and test - I'll try to do that this week14:19
mnasiadkaThere's skyline that probably could get in14:19
mnasiadkaAnd the systemd patch in kolla-ansible14:19
mnasiadkaAnyone willing to review skyline or systemd patch thoroughly?14:19
fricklerI will do skyline, but no timeline yet14:20
mnasiadkaok then14:22
mnasiadkasystemd also waiting for somebody :)14:22
mnasiadkaI'll try to review all of them, but surely not everything at once14:22
mnasiadka#topic Additional agenda (from whiteboard)14:23
mnasiadkathe first two were covered last week, left them to make sure jhorstmann adds Loki to the PTG etherpad, so I can groom that and paste as priorities in the whiteboard (will check later today)14:24
mnasiadkam_hiner: I know you wanted to discuss systemd patch, but I don't know if we don't need to postpone that next week - usually it was mgoddard and yoctozepto reviewing your patch when they could - but with Radek not spending time here anymore - it seems I need to look into that in more depth14:25
mnasiadkamgoddard: comfortable to talk about the systemd patch, or should we move it to next week?14:25
m_hinerI think the templating could be discussed right now14:26
mnasiadkaIf it comes to me - I don't think allowing users to modify the template is required, what would be the case for it?14:27
kevkoi wanted to review skyline also ...but don't know if i will have a time :( 14:28
m_hinerI also think so, as it wouldn't have any significant use cases (at least as far as I can think of)14:30
m_hinerand also to when customizing the template file, the users would also have to edit the kolla_systemd_worker.py which uses that template14:31
mnasiadkaSo let's leave it out, we can revisit if we can find a case for it14:32
mnasiadkaSeems the remaining review comments clarification - we need to move for the next meeting - I'll look at the patch and let's discuss it next week14:33
mnasiadkam_hiner: is that ok?14:33
m_hineryes, but please do14:34
m_hinerwe would really like to have it merged already14:34
mnasiadkaI understand, but now there's less reviewers than last cycle, so it will be harder...14:35
mnasiadkaanyway, let's talk about it next week14:35
m_hineralright 14:35
mnasiadka#topic Open discussion14:35
mnasiadkaNow the question is - should we have the meeting now at 3pm CET (which is 2pm UK time and 2pm UTC it seems)?14:36
mnasiadkaOr should I move it in the repository for an hour later?14:37
mnasiadka(at least until next daylight saving changes)14:37
fricklerI would like to stay at this time, since the largescale sig meeting is immediately following14:37
ihalomii want to ask about splitting https://review.opendev.org/c/openstack/kolla-ansible/+/799229 , should I also create smaller patchest introducing container_worker?14:38
mnasiadkamgoddard, kevko: anyone opposing?14:38
mnasiadkaihalomi: looks huge, would be nice14:38
mgoddardas for meeting time, I don't mind14:39
mnasiadkaOk then, let's try this for now14:40
mnasiadkaAny other open discussion topics?14:40
fricklerjust a question, did anyone ever discuss the ubuntu license terms?14:41
ihalomithese two patchest are ready to merge https://review.opendev.org/c/openstack/kolla-ansible/+/862883 , https://review.opendev.org/c/openstack/kolla-ansible/+/86275114:41
fricklerif I read them correctly, Ubuntu may only be distributed unmodified, not patched like in our containers14:41
fricklerhttps://ubuntu.com/legal/intellectual-property-policy14:42
mnasiadkathat's interesting...14:44
mnasiadkaDoes it mean we need to stop publishing Ubuntu images? ;-)14:44
fricklerthat might be the result, but ianal and so on14:45
mnasiadkaShould we ask somebody? e.g. foundation?14:46
fricklerthat page claims it was last updated in 2015, so not a new situation14:46
fricklerso my first thinking was surely this has somehow been discussed long ago14:46
fricklerbut if it hasn't, maybe TC first and have them talk to foundation if needed might be good14:47
mnasiadkaOk, how do we contact the TC? a mail to the usual openstack-discuss ML will suffice?14:48
fricklerI would usually just ask in #openstack-tc14:48
fricklerhighlight "tc-members"14:48
kevkofrickler: is it "modified" if we just adding packages and code ? 14:49
fricklerkevko: that's my understanding, yes14:49
mnasiadkawell, we use the base Ubuntu image, we're not modifying it in theory - but let's ask just to be sure14:49
kevkoi would say that if you change some code already landed in ubuntu or patch it somehow ...maybe yes ..but i don't know if it is our case ...hard question14:50
kevkowe are not modifing the image ..we are just adding layers with changes ..not modifiing the first layer which is image ..vanilla image14:51
kevkobut i don't  care ubuntu :D 14:51
fricklerhmm, the layers argument might work, but might be good to have this checked by someone with more legal experience14:52
mnasiadkaok then, I'll pick it up in some ,,spare'' time14:52
mnasiadka#action mnasiadka ask TC for guidance regarding Ubuntu distribution policy (https://ubuntu.com/legal/intellectual-property-policy)14:53
mnasiadkaI guess that's all for today14:53
mnasiadkathanks for coming!14:53
ihalomimnasiadka: about your comment in patchset, where should i add it? is site.yml best options?14:53
mnasiadka#endmeeting14:53
opendevmeetMeeting ended Wed Nov  2 14:53:37 2022 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:53
opendevmeetMinutes:        https://meetings.opendev.org/meetings/kolla/2022/kolla.2022-11-02-14.06.html14:53
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/kolla/2022/kolla.2022-11-02-14.06.txt14:53
opendevmeetLog:            https://meetings.opendev.org/meetings/kolla/2022/kolla.2022-11-02-14.06.log.html14:53
fricklerthx mnasiadka 14:53
mnasiadkaihalomi: probably yes, but I don't know if we want to use module defaults - maybe there are some cons, let's leave it as is most probably14:54
ihalomiokay, so I dont have to do anything about those patchsets now?14:55
mnasiadkafor now not, let's do a series of reviews14:55
opendevreviewwu.chunyang proposed openstack/kolla-ansible master: Implement automatic deploy of trove  https://review.opendev.org/c/openstack/kolla-ansible/+/86332115:22
opendevreviewIvan Halomi proposed openstack/kolla-ansible master: Adding container_engine to kolla_toolbox module  https://review.opendev.org/c/openstack/kolla-ansible/+/86275116:37
opendevreviewIvan Halomi proposed openstack/kolla-ansible master: Add systemd container control  https://review.opendev.org/c/openstack/kolla-ansible/+/81672416:37
opendevreviewIvan Halomi proposed openstack/kolla-ansible master: Splitting docker_worker into container_worker  https://review.opendev.org/c/openstack/kolla-ansible/+/86341416:37
opendevreviewWill Szumski proposed openstack/kolla-ansible master: Adds mechanism to configure additional libvirt secrets  https://review.opendev.org/c/openstack/kolla-ansible/+/86342117:01
opendevreviewIvan Halomi proposed openstack/kolla-ansible master: Splitting docker_worker into container_worker  https://review.opendev.org/c/openstack/kolla-ansible/+/86341418:18
*** darmach8 is now known as darmach22:35
*** darmach is now known as Guest28722:36

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