*** mnasiadka_ is now known as mnasiadka | 06:29 | |
*** parallax is now known as Guest227 | 06:42 | |
opendevreview | Merged openstack/kolla-ansible master: Fix ironic interface option for ironic-inspector https://review.opendev.org/c/openstack/kolla-ansible/+/863026 | 08:04 |
---|---|---|
hkominos | Morning all. Is there anybody online from the kayobe project? | 08:21 |
frickler | hkominos: best just ask your real question and wait for someone to answer | 08:55 |
hkominos | frickler: Good point | 08:58 |
hkominos | When 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 |
hkominos | editing them in /inventory/overcloud has not had the desired effect | 09:01 |
*** parallax is now known as Guest233 | 09:23 | |
mnasiadka | hkominos: use overcloud_group_hosts_map in etc/kayobe/overcloud.yml to map hosts to groups | 09:41 |
hkominos | mnasiadka: thx! | 09:53 |
opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/yoga: Fix ironic interface option for ironic-inspector https://review.opendev.org/c/openstack/kolla-ansible/+/863248 | 10:39 |
opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/xena: Fix ironic interface option for ironic-inspector https://review.opendev.org/c/openstack/kolla-ansible/+/863249 | 10:39 |
opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/wallaby: Fix ironic interface option for ironic-inspector https://review.opendev.org/c/openstack/kolla-ansible/+/863270 | 10:39 |
thelounge44 | seems that skyline changes are waiting for review https://review.opendev.org/c/openstack/kolla-ansible/+/828464 , https://review.opendev.org/c/openstack/kolla/+/826948 | 12:31 |
opendevreview | Ivan Halomi proposed openstack/kolla-ansible master: Adding container engine to kolla_container_facts https://review.opendev.org/c/openstack/kolla-ansible/+/862883 | 12:47 |
*** Guest233 is now known as parallax | 13:15 | |
mnasiadka | oh boy, DST changes again | 13:50 |
mnasiadka | hrw, mgoddard, frickler, kevko: do we want the meeting in 10 minutes, or at the usual time (in 1hr 10 min)? | 13:51 |
kevko | i am here in both times | 13:52 |
mnasiadka | let's maybe use the usual time in 1hr, since one one reply :) | 14:00 |
mgoddard | I don't mind | 14:02 |
frickler | I would prefer now, if we move to later we will have conflict with largescale sig | 14:03 |
mnasiadka | ok, let's then just start, and agree on meeting time from next week onwards ;-) | 14:06 |
mnasiadka | #startmeeting kolla | 14:06 |
opendevmeet | Meeting 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 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:06 |
opendevmeet | The meeting name has been set to 'kolla' | 14:06 |
mnasiadka | #topic rollcall | 14:06 |
mnasiadka | o/ | 14:06 |
frickler | \o | 14:07 |
mnasiadka | kevko, mgoddard, hrw | 14:08 |
TimBeermann[m] | O/ | 14:08 |
ihalomi | \o | 14:08 |
m_hiner | o/ | 14:08 |
mgoddard | \o | 14:10 |
mnasiadka | #topic agenda | 14:12 |
mnasiadka | * CI status | 14:12 |
mnasiadka | * Release tasks | 14:12 |
mnasiadka | * Regular stable releases (first meeting in a month) | 14:12 |
mnasiadka | * Current cycle planning | 14:12 |
mnasiadka | * Additional agenda (from whiteboard) | 14:12 |
mnasiadka | * Open discussion | 14:12 |
mnasiadka | #topic CI status | 14:12 |
mnasiadka | whiteboard says green | 14:12 |
mnasiadka | I haven't seen any spectacular failures today, so let's assume it is | 14:13 |
mnasiadka | #topic Regular stable releases (first meeting in a month) | 14:13 |
mnasiadka | I think Wallaby went EM, so no more stable releases there | 14:13 |
mgoddard | did we have a final release? | 14:14 |
mnasiadka | hmm, seems not since 6th Oct | 14:15 |
mnasiadka | I'll speak with the release team later if we can still do that - I was pretty sure we did | 14:16 |
mnasiadka | Argh, I missed one topic | 14:16 |
mnasiadka | #topic Release tasks | 14:16 |
mnasiadka | We can't branch yet, still kolla/kolla-ansible/kayobe needs opensearch | 14:16 |
mnasiadka | #topic Current cycle planning | 14:17 |
frickler | what about other stable branch releases? | 14:17 |
frickler | I can create a patch unless someone else wants to | 14:17 |
mnasiadka | Ah right, we should do other stable branch releases as we do every month | 14:17 |
frickler | ok, so I'll do that | 14:18 |
mnasiadka | thanks | 14:18 |
mnasiadka | Ok, so current cycle for us is still Zed | 14:19 |
mnasiadka | We have opensearch to review and test - I'll try to do that this week | 14:19 |
mnasiadka | There's skyline that probably could get in | 14:19 |
mnasiadka | And the systemd patch in kolla-ansible | 14:19 |
mnasiadka | Anyone willing to review skyline or systemd patch thoroughly? | 14:19 |
frickler | I will do skyline, but no timeline yet | 14:20 |
mnasiadka | ok then | 14:22 |
mnasiadka | systemd also waiting for somebody :) | 14:22 |
mnasiadka | I'll try to review all of them, but surely not everything at once | 14:22 |
mnasiadka | #topic Additional agenda (from whiteboard) | 14:23 |
mnasiadka | the 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 |
mnasiadka | m_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 depth | 14:25 |
mnasiadka | mgoddard: comfortable to talk about the systemd patch, or should we move it to next week? | 14:25 |
m_hiner | I think the templating could be discussed right now | 14:26 |
mnasiadka | If 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 |
kevko | i wanted to review skyline also ...but don't know if i will have a time :( | 14:28 |
m_hiner | I also think so, as it wouldn't have any significant use cases (at least as far as I can think of) | 14:30 |
m_hiner | and also to when customizing the template file, the users would also have to edit the kolla_systemd_worker.py which uses that template | 14:31 |
mnasiadka | So let's leave it out, we can revisit if we can find a case for it | 14:32 |
mnasiadka | Seems 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 week | 14:33 |
mnasiadka | m_hiner: is that ok? | 14:33 |
m_hiner | yes, but please do | 14:34 |
m_hiner | we would really like to have it merged already | 14:34 |
mnasiadka | I understand, but now there's less reviewers than last cycle, so it will be harder... | 14:35 |
mnasiadka | anyway, let's talk about it next week | 14:35 |
m_hiner | alright | 14:35 |
mnasiadka | #topic Open discussion | 14:35 |
mnasiadka | Now the question is - should we have the meeting now at 3pm CET (which is 2pm UK time and 2pm UTC it seems)? | 14:36 |
mnasiadka | Or should I move it in the repository for an hour later? | 14:37 |
mnasiadka | (at least until next daylight saving changes) | 14:37 |
frickler | I would like to stay at this time, since the largescale sig meeting is immediately following | 14:37 |
ihalomi | i 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 |
mnasiadka | mgoddard, kevko: anyone opposing? | 14:38 |
mnasiadka | ihalomi: looks huge, would be nice | 14:38 |
mgoddard | as for meeting time, I don't mind | 14:39 |
mnasiadka | Ok then, let's try this for now | 14:40 |
mnasiadka | Any other open discussion topics? | 14:40 |
frickler | just a question, did anyone ever discuss the ubuntu license terms? | 14:41 |
ihalomi | these two patchest are ready to merge https://review.opendev.org/c/openstack/kolla-ansible/+/862883 , https://review.opendev.org/c/openstack/kolla-ansible/+/862751 | 14:41 |
frickler | if I read them correctly, Ubuntu may only be distributed unmodified, not patched like in our containers | 14:41 |
frickler | https://ubuntu.com/legal/intellectual-property-policy | 14:42 |
mnasiadka | that's interesting... | 14:44 |
mnasiadka | Does it mean we need to stop publishing Ubuntu images? ;-) | 14:44 |
frickler | that might be the result, but ianal and so on | 14:45 |
mnasiadka | Should we ask somebody? e.g. foundation? | 14:46 |
frickler | that page claims it was last updated in 2015, so not a new situation | 14:46 |
frickler | so my first thinking was surely this has somehow been discussed long ago | 14:46 |
frickler | but if it hasn't, maybe TC first and have them talk to foundation if needed might be good | 14:47 |
mnasiadka | Ok, how do we contact the TC? a mail to the usual openstack-discuss ML will suffice? | 14:48 |
frickler | I would usually just ask in #openstack-tc | 14:48 |
frickler | highlight "tc-members" | 14:48 |
kevko | frickler: is it "modified" if we just adding packages and code ? | 14:49 |
frickler | kevko: that's my understanding, yes | 14:49 |
mnasiadka | well, we use the base Ubuntu image, we're not modifying it in theory - but let's ask just to be sure | 14:49 |
kevko | i 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 question | 14:50 |
kevko | we are not modifing the image ..we are just adding layers with changes ..not modifiing the first layer which is image ..vanilla image | 14:51 |
kevko | but i don't care ubuntu :D | 14:51 |
frickler | hmm, the layers argument might work, but might be good to have this checked by someone with more legal experience | 14:52 |
mnasiadka | ok then, I'll pick it up in some ,,spare'' time | 14:52 |
mnasiadka | #action mnasiadka ask TC for guidance regarding Ubuntu distribution policy (https://ubuntu.com/legal/intellectual-property-policy) | 14:53 |
mnasiadka | I guess that's all for today | 14:53 |
mnasiadka | thanks for coming! | 14:53 |
ihalomi | mnasiadka: about your comment in patchset, where should i add it? is site.yml best options? | 14:53 |
mnasiadka | #endmeeting | 14:53 |
opendevmeet | Meeting ended Wed Nov 2 14:53:37 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:53 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/kolla/2022/kolla.2022-11-02-14.06.html | 14:53 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/kolla/2022/kolla.2022-11-02-14.06.txt | 14:53 |
opendevmeet | Log: https://meetings.opendev.org/meetings/kolla/2022/kolla.2022-11-02-14.06.log.html | 14:53 |
frickler | thx mnasiadka | 14:53 |
mnasiadka | ihalomi: 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 probably | 14:54 |
ihalomi | okay, so I dont have to do anything about those patchsets now? | 14:55 |
mnasiadka | for now not, let's do a series of reviews | 14:55 |
opendevreview | wu.chunyang proposed openstack/kolla-ansible master: Implement automatic deploy of trove https://review.opendev.org/c/openstack/kolla-ansible/+/863321 | 15:22 |
opendevreview | Ivan Halomi proposed openstack/kolla-ansible master: Adding container_engine to kolla_toolbox module https://review.opendev.org/c/openstack/kolla-ansible/+/862751 | 16:37 |
opendevreview | Ivan Halomi proposed openstack/kolla-ansible master: Add systemd container control https://review.opendev.org/c/openstack/kolla-ansible/+/816724 | 16:37 |
opendevreview | Ivan Halomi proposed openstack/kolla-ansible master: Splitting docker_worker into container_worker https://review.opendev.org/c/openstack/kolla-ansible/+/863414 | 16:37 |
opendevreview | Will Szumski proposed openstack/kolla-ansible master: Adds mechanism to configure additional libvirt secrets https://review.opendev.org/c/openstack/kolla-ansible/+/863421 | 17:01 |
opendevreview | Ivan Halomi proposed openstack/kolla-ansible master: Splitting docker_worker into container_worker https://review.opendev.org/c/openstack/kolla-ansible/+/863414 | 18:18 |
*** darmach8 is now known as darmach | 22:35 | |
*** darmach is now known as Guest287 | 22:36 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!