opendevreview | Maksim Malchuk proposed openstack/kayobe master: Ubuntu: add support for Apt repository configuration https://review.opendev.org/c/openstack/kayobe/+/816364 | 06:47 |
---|---|---|
opendevreview | Maksim Malchuk proposed openstack/kayobe master: Ubuntu: add support for Apt configuration https://review.opendev.org/c/openstack/kayobe/+/818068 | 06:48 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: monasca: Remove monasca-grafana leftovers https://review.opendev.org/c/openstack/kolla-ansible/+/834446 | 07:07 |
opendevreview | Merged openstack/kayobe master: CI: pin pytest-metadata<2 for molecule https://review.opendev.org/c/openstack/kayobe/+/834790 | 07:15 |
opendevreview | Pierre Riteau proposed openstack/kayobe stable/xena: CI: pin pytest-metadata<2 for molecule https://review.opendev.org/c/openstack/kayobe/+/834723 | 08:50 |
opendevreview | Jan Horstmann proposed openstack/kolla-ansible master: Fix grafana provisioning.yaml indentation https://review.opendev.org/c/openstack/kolla-ansible/+/834826 | 08:56 |
opendevreview | Pierre Riteau proposed openstack/kayobe master: CI: Restore upper constraints for molecule job https://review.opendev.org/c/openstack/kayobe/+/834827 | 09:05 |
opendevreview | Maksim Malchuk proposed openstack/kayobe stable/xena: Ubuntu: support host package update https://review.opendev.org/c/openstack/kayobe/+/834607 | 09:24 |
opendevreview | Maksim Malchuk proposed openstack/kayobe stable/xena: Add support for Rocky Linux 8 https://review.opendev.org/c/openstack/kayobe/+/834309 | 09:25 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible master: fix missing lang env in curator crontab https://review.opendev.org/c/openstack/kolla-ansible/+/781114 | 09:44 |
yoctozepto | mgoddard: hi, please let me know if the solution from https://review.opendev.org/c/openstack/kolla-ansible/+/834004 appeals to you | 10:38 |
*** mrunge_ is now known as mrunge | 10:47 | |
opendevreview | Merged openstack/kayobe stable/wallaby: Ubuntu: support host package update https://review.opendev.org/c/openstack/kayobe/+/834608 | 10:55 |
opendevreview | Merged openstack/kayobe master: Ubuntu: add support for Apt repository configuration https://review.opendev.org/c/openstack/kayobe/+/816364 | 10:55 |
opendevreview | Maksim Malchuk proposed openstack/kayobe stable/xena: Ubuntu: add support for Apt repository configuration https://review.opendev.org/c/openstack/kayobe/+/834847 | 11:04 |
opendevreview | Michal Nasiadka proposed openstack/kayobe master: Use openstack-ci-mirrors in ipa image build https://review.opendev.org/c/openstack/kayobe/+/821675 | 11:40 |
opendevreview | Merged openstack/kolla-ansible master: [CI] Add Venus scenario https://review.opendev.org/c/openstack/kolla-ansible/+/823222 | 12:28 |
opendevreview | Merged openstack/kayobe stable/xena: CI: pin pytest-metadata<2 for molecule https://review.opendev.org/c/openstack/kayobe/+/834723 | 12:50 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: designate: Allow to disable notifications https://review.opendev.org/c/openstack/kolla-ansible/+/802301 | 12:59 |
opendevreview | Merged openstack/kolla-ansible master: monasca: Remove monasca-grafana leftovers https://review.opendev.org/c/openstack/kolla-ansible/+/834446 | 13:06 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: designate: Allow to disable notifications https://review.opendev.org/c/openstack/kolla-ansible/+/802301 | 13:09 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible master: fix missing lang env in curator crontab https://review.opendev.org/c/openstack/kolla-ansible/+/781114 | 13:09 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: designate: Allow to disable notifications https://review.opendev.org/c/openstack/kolla-ansible/+/802301 | 13:10 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: designate: transform designate_ns_record into list https://review.opendev.org/c/openstack/kolla-ansible/+/802304 | 13:13 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: designate: transform designate_ns_record into list https://review.opendev.org/c/openstack/kolla-ansible/+/802304 | 13:14 |
opendevreview | Verification of a change to openstack/kayobe master failed: Ubuntu: add support for Apt configuration https://review.opendev.org/c/openstack/kayobe/+/818068 | 13:17 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: designate: transform designate_ns_record into list https://review.opendev.org/c/openstack/kolla-ansible/+/802304 | 13:28 |
opendevreview | Merged openstack/kayobe stable/xena: Add support for Rocky Linux 8 https://review.opendev.org/c/openstack/kayobe/+/834309 | 13:33 |
jingvar | Can I use bifrost.yml:{% include 'site/bifrost.yml' %} ? | 14:51 |
jingvar | same include worsk with kolla/globals.yml:{% include 'custom/kolla-ansible/ansible/roles/glance/defaults/main.yml' %} | 14:52 |
jingvar | but fails with bifrost | 14:53 |
mnasiadka | mgoddard mnasiadka hrw egonzalez yoctozepto rafaelweingartne cosmicsound osmanlicilegi bbezak parallax Fl1nt frickler adrian-a - meeting in 7 | 14:53 |
yoctozepto | mnasiadka: OK | 14:54 |
mnasiadka | then OK | 14:54 |
mnasiadka | :) | 14:54 |
yoctozepto | :-) | 14:55 |
mnasiadka | #startmeeting kolla | 15:00 |
opendevmeet | Meeting started Wed Mar 23 15:00:01 2022 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 |
mnasiadka | o/ | 15:00 |
mgoddard | \oo | 15:01 |
yoctozepto | o/ | 15:01 |
mnasiadka | Crowds today ;-) | 15:03 |
mnasiadka | #topic agenda | 15:03 |
mnasiadka | * Announcements | 15:03 |
mnasiadka | * Review action items from the last meeting | 15:03 |
mnasiadka | * CI status | 15:03 |
mnasiadka | * Release tasks | 15:03 |
mnasiadka | * Current cycle planning | 15:03 |
mnasiadka | * Additional agenda (from whiteboard) | 15:03 |
mnasiadka | * Open discussion | 15:03 |
mnasiadka | #topic Announcements | 15:03 |
mnasiadka | We've been invited to next OpenInfra Live to talk about Kolla features for Y (and planned for Z I think) - mgoddard will represent Kolla | 15:04 |
mnasiadka | (since I can't at that time) | 15:04 |
mnasiadka | #topic Review action items from the last meeting | 15:04 |
mnasiadka | mnasiadka to triage security bugs and update them with resolution plan (if needed) | 15:05 |
mnasiadka | hrw Enable osbpo in Debian APT sources, abandon extrepo command use then? | 15:05 |
mnasiadka | I haven't finished mine, hrw absent once again | 15:05 |
mnasiadka | #action mnasiadka to triage security bugs and update them with resolution plan (if needed) | 15:05 |
mnasiadka | #action hrw Enable osbpo in Debian APT sources, abandon extrepo command use then? | 15:05 |
mnasiadka | #topic CI status | 15:05 |
mnasiadka | Kolla/Kolla-Ansible seem to be green | 15:05 |
mnasiadka | Kayobe had some molecule related breakage but fixed | 15:05 |
mnasiadka | Anyone have different view? | 15:05 |
yoctozepto | nope | 15:06 |
mnasiadka | Ok then | 15:06 |
mnasiadka | #topic Release tasks | 15:06 |
mnasiadka | Feature freeze this week | 15:06 |
mnasiadka | Allegedly next week we should start preparing RC1 | 15:07 |
yoctozepto | mhm | 15:07 |
mnasiadka | #topic Current cycle planning | 15:07 |
mnasiadka | So, is there anything we need to squeeze in (as in review needed)? | 15:08 |
mgoddard | hold on | 15:08 |
mgoddard | this week is feature freeze | 15:08 |
mgoddard | oh, prepare for RC1, not cut RC1 | 15:09 |
mgoddard | carry on | 15:09 |
mnasiadka | prepare, one week later we should cut | 15:09 |
mnasiadka | at least according to the plan | 15:09 |
yoctozepto | yea | 15:09 |
yoctozepto | I want ironic patches to merge | 15:10 |
mnasiadka | let's start with priorities from the whiteboard | 15:10 |
mnasiadka | What are we doing with the systemd patch? | 15:10 |
mnasiadka | https://review.opendev.org/c/openstack/kolla-ansible/+/816724 | 15:10 |
mnasiadka | Moving to Z? | 15:10 |
mgoddard | If we're cutting RC1 in 2 weeks, then it's too late, IMO | 15:11 |
mnasiadka | ok, should we mark it with RP -1? | 15:11 |
mnasiadka | (I also feel it's too big of a change, to merge it last minute) | 15:12 |
yoctozepto | agreed | 15:12 |
mgoddard | we should aim to merge it early in Zed though | 15:13 |
mnasiadka | makes perfect sense | 15:13 |
mnasiadka | ok then, what about that Kolla collection? | 15:13 |
mnasiadka | that's a long list of patches again :) | 15:13 |
mnasiadka | 7 to be specific | 15:14 |
mgoddard | I'd like for it to land, but at this point it seems unlikely | 15:14 |
mgoddard | it needs kolla cores to land 7, then kayobe cores to land a big one | 15:14 |
mgoddard | for it to be most useful | 15:15 |
yoctozepto | hmm | 15:15 |
mgoddard | still, any that land are a step in the right direction | 15:15 |
yoctozepto | I can review again | 15:15 |
mgoddard | appreciated | 15:15 |
mnasiadka | so now Ironic | 15:16 |
yoctozepto | indeed | 15:18 |
mnasiadka | https://review.opendev.org/c/openstack/kolla/+/832163 - those last comments are funny ;) | 15:19 |
yoctozepto | funny how? | 15:20 |
mnasiadka | well, funny that we don't know why it's there, but it's needed? | 15:20 |
yoctozepto | well, I know what lines 3 and 4 are for | 15:20 |
mnasiadka | mgoddard: are we good to merge it, or do you want to test it somewhere? | 15:21 |
mnasiadka | the toolbox patch is merging | 15:22 |
mnasiadka | so we're left with this and the rename patch | 15:22 |
yoctozepto | and some other k-a patches | 15:22 |
yoctozepto | the rename patch got a proposed solution to waiting on waits in the upgrade testing patch | 15:22 |
yoctozepto | I will "backport" once mgoddard says it's ok | 15:22 |
mgoddard | backport which part? | 15:23 |
mgoddard | oh, the check | 15:23 |
yoctozepto | "backport" as in move the upgrade.yml part to the rename patch | 15:23 |
yoctozepto | yeah | 15:23 |
yoctozepto | the wait on waits | 15:23 |
mnasiadka | anyway, I'm not the resident Ironic expert, I'll leave you two to it :) | 15:23 |
mnasiadka | And then there's a whole lot in Kayobe land, but I think we need to tackle that internally in SHPC | 15:24 |
yoctozepto | mnasiadka: you mean you will rubberstamp anything ironic when one of us +2s? ;-) | 15:25 |
mnasiadka | well, you can't +2, because these are your patches :) | 15:25 |
yoctozepto | mnasiadka: I meant in general | 15:25 |
mnasiadka | in general - I'll rubberstamp what is logical and sensible, I only meant I don't have the time to test it anywhere :) | 15:26 |
mnasiadka | and the path change in tftp map file... I sense trouble with some museum-level hardware, but maybe it's nothing :) | 15:27 |
mnasiadka | nothing that can't be easily fixed | 15:27 |
mnasiadka | Ok, are we done with current cycle? | 15:28 |
mnasiadka | There are patches like ProxySQL | 15:28 |
mnasiadka | which are not added to Yoga priorities | 15:28 |
yoctozepto | it would be good to get them in but they serious reviews | 15:28 |
mnasiadka | so we move them to Z | 15:28 |
mnasiadka | So let's move on | 15:29 |
mnasiadka | #topic Additional agenda (from whiteboard) | 15:29 |
mnasiadka | There's one, since it's daylight savings change time (this weekend in Poland) | 15:29 |
yoctozepto | ah yes | 15:29 |
mnasiadka | And I'm not really a fan of having meetings at 5pm | 15:30 |
yoctozepto | me neither | 15:30 |
mnasiadka | I propose we do what Ironic did - move the meeting one hour earlier | 15:30 |
frickler | +1 | 15:30 |
yoctozepto | that's also what I've done with masakari's back then | 15:30 |
yoctozepto | +2 | 15:30 |
mnasiadka | ok | 15:30 |
mnasiadka | #agreed to move the meeting time one hour earlier (dst time change this weekend) | 15:30 |
mnasiadka | #topic Open discussion | 15:31 |
frickler | two things from me | 15:31 |
frickler | first I want to apologize for the backport mess I created | 15:31 |
frickler | I agree with yoctozepto that we should try to avoid such things in the future | 15:31 |
frickler | the second is that the libvirt sasl things seem to be broken with hosts that have long fqdns | 15:32 |
yoctozepto | argh | 15:32 |
frickler | saslpasswd creates a user entry with user@hostname | 15:32 |
mgoddard | hmm, how long? | 15:32 |
frickler | but libvirt checks for user@hostname.example.org | 15:32 |
yoctozepto | mgoddard: I guess "long" as "with domain part" | 15:33 |
frickler | any where hostname -s != hostname -f | 15:33 |
yoctozepto | guessed it | 15:33 |
frickler | at least that's what it looks like to me so far | 15:33 |
frickler | still looking at the implementations | 15:33 |
mnasiadka | let's raise a bug, target it to Yoga milestone and get it fixed? | 15:33 |
frickler | I'll create a bug at least, yes | 15:34 |
mgoddard | where exactly does it fail? | 15:34 |
frickler | I've tried to reproduce in CI where currently we only use short hostnames, but no success yet | 15:34 |
frickler | nova fails to auth against libvirt | 15:35 |
mnasiadka | great, I asked to for targeting it to milestone, so we can track such ,,last minute'' things that should get fixed before the release | 15:35 |
mgoddard | frickler: are you using libvirt TLS? | 15:35 |
frickler | no | 15:35 |
mnasiadka | I'll use the opportunity to welcome frickler amongst core reviewers, will add him rights to both kolla and kolla-ansible in a minute (and send a mail to ML) | 15:36 |
mgoddard | frickler: when not using libvirt TLS, nova should use IP to connect to libvirt | 15:36 |
mgoddard | congrats frickler :) | 15:36 |
frickler | thx | 15:36 |
mgoddard | (migration_interface_address) | 15:36 |
yoctozepto | congrats frickler (and finally!) | 15:37 |
frickler | mgoddard: nova connects to the IP, but libvirt uses the fqdn for that IP to construct the sasl_username | 15:37 |
mgoddard | hmm | 15:37 |
frickler | mgoddard: e.g. | 15:37 |
frickler | (nova-libvirt)[root@testbed-node-0 /]# sasldblistusers2 | 15:37 |
frickler | fred@testbed-node-0: userPassword | 15:37 |
frickler | but libvirt check for fred@testbed-node-0.osism.xyz | 15:38 |
mgoddard | so there is some reverse name lookup against the IP used to connect, which resolves to your FQDN? | 15:38 |
frickler | if I add an account with "-u (nova-libvirt)[root@testbed-node-0 /]# sasldblistusers2 | 15:38 |
frickler | ehm | 15:38 |
frickler | if I add an account with "-u testbed-node-0.osism.xyz" it works | 15:38 |
yoctozepto | frickler, mgoddard: does that mean that w/ TLS it would work? | 15:38 |
frickler | yes, libvirt does that lookup | 15:39 |
mgoddard | perhaps | 15:39 |
mgoddard | ok, maybe we need to add -u to the account creation then | 15:40 |
frickler | getnameinfo() | 15:40 |
yoctozepto | what does -u do? | 15:40 |
frickler | set the "domain" of the user | 15:40 |
yoctozepto | mhm | 15:41 |
frickler | I can propose a patch, I just wanted to reproduce the issue in CI first | 15:42 |
yoctozepto | would be good to | 15:44 |
yoctozepto | we can set long hostnames on the hosts | 15:44 |
yoctozepto | but note the base jobs run with tls | 15:44 |
yoctozepto | so if it helps, then they are not going to catch the issue anyway | 15:44 |
yoctozepto | good to verify though | 15:44 |
frickler | https://review.opendev.org/c/openstack/kolla-ansible/+/834662 is where I started | 15:44 |
frickler | but it didn't fail, I can try to disable tls | 15:45 |
yoctozepto | ++ | 15:46 |
hrw | o/ | 15:46 |
hrw | mnasiadka: #cancel action - we stay with extrepo | 15:47 |
mnasiadka | ok then | 15:48 |
mnasiadka | all is clear I guess | 15:49 |
mnasiadka | let's finish for today | 15:49 |
mnasiadka | thanks for attending! | 15:49 |
mnasiadka | #endmeeting | 15:49 |
opendevmeet | Meeting ended Wed Mar 23 15:49:24 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:49 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/kolla/2022/kolla.2022-03-23-15.00.html | 15:49 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/kolla/2022/kolla.2022-03-23-15.00.txt | 15:49 |
opendevmeet | Log: https://meetings.opendev.org/meetings/kolla/2022/kolla.2022-03-23-15.00.log.html | 15:49 |
yoctozepto | thanks mnasiadka for chairing | 15:50 |
opendevreview | Merged openstack/kolla master: toolbox: Add ironic client https://review.opendev.org/c/openstack/kolla/+/834692 | 15:54 |
opendevreview | Radosław Piliszek proposed openstack/ansible-collection-kolla master: baremetal: refactor kolla user & group creation into a separate role https://review.opendev.org/c/openstack/ansible-collection-kolla/+/821015 | 15:57 |
yoctozepto | mnasiadka: let's merge https://review.opendev.org/c/openstack/ansible-collection-kolla/+/821015 for mgoddard | 15:57 |
yoctozepto | the next patches in series have failures and merge conflicts though | 15:57 |
mnasiadka | yoctozepto: done | 16:02 |
yoctozepto | :-) | 16:06 |
mnasiadka | mgoddard: what about https://review.opendev.org/c/openstack/ansible-collection-kolla/+/821207/1 ? looks quite important ;-) | 16:10 |
mgoddard | mnasiadka: not really, we use the branch | 16:11 |
mnasiadka | as a final step, or do we want to do releases? | 16:11 |
yoctozepto | I prefer the branch | 16:11 |
mnasiadka | I guess it makes sense | 16:12 |
mnasiadka | ok then, enough for today :) | 16:13 |
* yoctozepto off as well | 16:14 | |
opendevreview | Merged openstack/kayobe master: Ubuntu: add support for Apt configuration https://review.opendev.org/c/openstack/kayobe/+/818068 | 17:18 |
opendevreview | Maksim Malchuk proposed openstack/kayobe stable/xena: Ubuntu: add support for Apt configuration https://review.opendev.org/c/openstack/kayobe/+/834929 | 17:20 |
opendevreview | Merged openstack/ansible-collection-kolla master: baremetal: refactor kolla user & group creation into a separate role https://review.opendev.org/c/openstack/ansible-collection-kolla/+/821015 | 18:33 |
kalle | hi I'm trying to deploy kayobe (multinode) and I have a few questions: | 21:35 |
kalle | can I deploy kayobe with neutron_plugin_agent: ovn? | 21:35 |
kalle | (If yes how do i configure this?) | 21:35 |
kalle_ | does anyone know weather kayobe con deploy anything other than openvswitch as neutron_plugin_agent? | 21:56 |
opendevreview | Dr. Jens Harbott proposed openstack/kolla-ansible master: WIP: Test setting a FQDN as hostname https://review.opendev.org/c/openstack/kolla-ansible/+/834662 | 22:23 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!