| opendevreview | Boxiang Zhu proposed openstack/kolla-ansible master: [CI] Add skyline scenario https://review.opendev.org/c/openstack/kolla-ansible/+/861687 | 01:29 |
|---|---|---|
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: kolla_docker: Add container_engine as required https://review.opendev.org/c/openstack/kolla-ansible/+/900278 | 06:36 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: kolla_docker: Add container_engine as required https://review.opendev.org/c/openstack/kolla-ansible/+/900278 | 06:36 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Fix kolla_docker call https://review.opendev.org/c/openstack/kolla-ansible/+/900284 | 07:23 |
| mnasiadka | kevko: abandoned mine and fixed your patch ^^ | 07:23 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Add support for ansible-core https://review.opendev.org/c/openstack/kolla-ansible/+/896406 | 07:31 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Add support for ansible-core https://review.opendev.org/c/openstack/kolla-ansible/+/896406 | 07:32 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Add support for ansible-core https://review.opendev.org/c/openstack/kolla-ansible/+/896406 | 07:33 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Add support for ansible-core https://review.opendev.org/c/openstack/kolla-ansible/+/896406 | 07:34 |
| mnasiadka | kevko, frickler: ^^ would like to get the ansible-core patch merged, if you have time for reviewing it | 07:46 |
| frickler | I much prefer to review patches which have current CI results | 07:47 |
| mnasiadka | ok, I'll ping you then :) | 07:47 |
| frickler | I was able to find a -1 reason even without it, so we're fine ;) | 07:56 |
| mnasiadka | haha | 08:02 |
| mnasiadka | will update after the CI run finishes | 08:02 |
| opendevreview | Merged openstack/kolla-ansible master: [CI] Add skyline scenario https://review.opendev.org/c/openstack/kolla-ansible/+/861687 | 08:53 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Add support for ansible-core https://review.opendev.org/c/openstack/kolla-ansible/+/896406 | 09:52 |
| mnasiadka | frickler: all green, updated quickstart ^^ | 09:52 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: CI: Move ephemeral disk config to a role https://review.opendev.org/c/openstack/kolla/+/899465 | 09:57 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Test haproxy single external frontend https://review.opendev.org/c/openstack/kolla-ansible/+/841239 | 10:01 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Test haproxy single external frontend https://review.opendev.org/c/openstack/kolla-ansible/+/841239 | 10:03 |
| frickler | mnasiadka: ack, ty | 10:04 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Test haproxy single external frontend https://review.opendev.org/c/openstack/kolla-ansible/+/841239 | 10:05 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Test haproxy single external frontend https://review.opendev.org/c/openstack/kolla-ansible/+/841239 | 10:06 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Test haproxy single external frontend https://review.opendev.org/c/openstack/kolla-ansible/+/841239 | 10:06 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Fix kolla_docker call https://review.opendev.org/c/openstack/kolla-ansible/+/900284 | 10:43 |
| kevko | mnasiadka: done | 11:02 |
| mmalchuk | mnasiadka topic of the channel says wrong meeting time | 11:03 |
| mmalchuk | afaik changed to 14:00 UTC ? | 11:04 |
| mnasiadka | frickler: can you help in this? I don't have rights ;-) | 11:06 |
| mmalchuk | agenda URL also outdated | 11:10 |
| opendevreview | Will Szumski proposed openstack/kayobe master: Fallback to templating when only one source exists https://review.opendev.org/c/openstack/kayobe/+/900060 | 11:17 |
| opendevreview | Will Szumski proposed openstack/kayobe master: Fallback to templating when only one source exists https://review.opendev.org/c/openstack/kayobe/+/900060 | 11:18 |
| frickler | topic: IRC meetings on Wednesdays @ 14:00 UTC - https://docs.openstack.org/kolla/latest/contributor/meeting.html | Whiteboard: https://bit.ly/2MM7mWF | IRC channel is *LOGGED* @ http://goo.gl/3mzZ7b | 12:03 |
| frickler | like that? or expand the other shortener URLs, too? | 12:03 |
| frickler | I could also offer even shorter and completely untracked short URLs on https://od42.de | 12:04 |
| mnasiadka | whatever you feel necessary, I don't look at the topic :) | 12:08 |
| *** ChanServ changes topic to "IRC meetings on Wednesdays @ 14:00 UTC - https://docs.openstack.org/kolla/latest/contributor/meeting.html | Whiteboard: https://bit.ly/2MM7mWF | IRC channel is *LOGGED* @ http://goo.gl/3mzZ7b" | 12:13 | |
| *** Continuity__ is now known as Continuity | 12:19 | |
| opendevreview | Will Szumski proposed openstack/kayobe master: Fallback to templating when only one source exists https://review.opendev.org/c/openstack/kayobe/+/900060 | 12:33 |
| kevko | what is -em in a tag ...example wallaby-em wallaby-eol is end of life | 12:36 |
| kevko | aaaand ..which one is "latest" :) | 12:36 |
| kevko | eol is latest right ? | 12:41 |
| kevko | anyone here ? :D | 12:56 |
| jovial | Kevko: That is correct. A release goes into extended maintenance and then finally end-of-life. | 12:58 |
| kevko | jovial: thanks | 13:14 |
| kevko | mnasiadka: frickler: what about leftovers LE ? can we merge now ? :) | 13:24 |
| opendevreview | Martin Hiner proposed openstack/kolla-ansible master: Generalize Docker specific names https://review.opendev.org/c/openstack/kolla-ansible/+/900412 | 13:28 |
| sylvr | Hello! When configuring the seed host (kayobe) is it normal that the playbook fail when ensuring the docker registry is running while I did not configure kayobe to deploy any registry ? | 13:34 |
| opendevreview | Merged openstack/kolla-ansible master: Set HORIZON_IMAGES_UPLOAD_MODE to direct by default https://review.opendev.org/c/openstack/kolla-ansible/+/900056 | 13:41 |
| jovial | slyvr: It should be disabled by default: https://github.com/openstack/kayobe/blob/6e404d1fe49699cb6e20ece67a0cdb1a38e8937f/ansible/inventory/group_vars/all/docker-registry#L6. Don't suppose you could put the ansible output in a paste bin? | 13:41 |
| sylvr | @jovial : like this ? https://pastebin.com/iuG3DMbP | 13:44 |
| jovial | sylvr: perfect, thanks. What version of kayobe are you using? And you haven't overridden the variable: `docker_registry_enabled` in kayobe config at all? | 13:45 |
| sylvr | I'm running kayobe 14.1.0 (so latest stable ?) | 13:47 |
| sylvr | and I didn't, but I tried with overriding the variable to see if it changes something | 13:48 |
| sylvr | overriding the variable (inside kayobe-config/etc/kayobe/docker-registry.yml ) with 'False' didn't change anything | 13:50 |
| mnasiadka | mgoddard mnasiadka bbezak frickler kevko SvenKieske mmalchuk gkoper jangutter jsuazo jovial - meeting in 8 minutes | 13:52 |
| jovial | So it looks like it will still try and connect to the local docker daemon even with: `docker_registry_enabled: False`. Is docker is installed on the seed? and does your ansible user have read/write permission on the docker socket? | 13:52 |
| priteau | Yes, looks like a docker install/config error or permission error | 13:54 |
| opendevreview | Jake Hutchinson proposed openstack/kolla-ansible master: Ironic parameter rework and default NTP server https://review.opendev.org/c/openstack/kolla-ansible/+/893031 | 13:55 |
| sylvr | just added the kolla user to the group sudo and wheel, and see if it changes anything ! | 13:56 |
| sylvr | (maybe I should've added it to 'docker' group) | 13:56 |
| sylvr | but yeah, it was because I don't use the same user as kayobe on the seed, misconfiguration on my part I think | 13:57 |
| jovial | I think this task will be using the stack user unless you customised the ansible user in some way | 13:58 |
| jovial | I normally crank up the verbosity to verify that in the debug output | 13:59 |
| mnasiadka | Weekly meeting time now ;-) | 13:59 |
| sylvr | I'm going to test that, thanks a lot for you help ! | 13:59 |
| mnasiadka | #startmeeting kolla | 14:00 |
| opendevmeet | Meeting started Wed Nov 8 14:00:01 2023 UTC and is due to finish in 60 minutes. The chair is mnasiadka. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| opendevmeet | The meeting name has been set to 'kolla' | 14:00 |
| mnasiadka | #topic rollcall | 14:00 |
| mnasiadka | o/ | 14:00 |
| mmalchuk | o/ | 14:00 |
| jangutter | o/ | 14:00 |
| jovial | \o | 14:00 |
| mhiner | o/ | 14:00 |
| frickler | \o | 14:02 |
| bbezak | o/ | 14:02 |
| mnasiadka | #topic agenda | 14:03 |
| mnasiadka | * CI status | 14:03 |
| mnasiadka | * Release tasks | 14:03 |
| mnasiadka | * Regular stable releases (first meeting in a month) | 14:03 |
| mnasiadka | * Current cycle planning | 14:03 |
| mnasiadka | * Additional agenda (from whiteboard) | 14:03 |
| mnasiadka | * Open discussion | 14:03 |
| mnasiadka | #topic CI status | 14:03 |
| mnasiadka | Looks good overall, no current weird breakages | 14:03 |
| mnasiadka | Although if anybody is willing - rocky/debian ironic CI jobs have been failing long time | 14:03 |
| mnasiadka | any volunteers to have a look? | 14:03 |
| jangutter | I can spend 1 hour or so this week to take a look, I'm not familiar with them though. | 14:04 |
| mnasiadka | ok then, thanks - if you need any help - shout :) | 14:05 |
| mnasiadka | #topic Release tasks | 14:05 |
| mnasiadka | So, in order to make a release - we need to merge Let's Encrypt followups - kevko do they have RP+1? | 14:05 |
| mnasiadka | Basically - cores please add RP+1 to things we NEED to merge before branching | 14:06 |
| mnasiadka | So I'm looking for reviewers on ephemeral disk usage in CI in Kolla-Ansible - https://review.opendev.org/c/openstack/kolla/+/899465 | 14:09 |
| mnasiadka | https://review.opendev.org/c/openstack/kolla-ansible/+/899466 | 14:09 |
| jovial | Dumb question... once the branch is cut are we not allowed to add features anymore? | 14:09 |
| mnasiadka | basically that's the rule we've been adhering to - although sometimes we backport something that would not break existing users | 14:09 |
| mnasiadka | if that improves user experience | 14:10 |
| jovial | cool, thanks for the clarification | 14:10 |
| mnasiadka | ok, kevko is not here, but let's try to find the LE followups | 14:10 |
| mnasiadka | I started a list on the Whiteboard - https://etherpad.opendev.org/p/KollaWhiteBoard#L209 | 14:11 |
| mmalchuk | added one | 14:12 |
| mnasiadka | Sven is not here, but I guess td-agent bump would be useful | 14:12 |
| kevko | now | 14:13 |
| mnasiadka | jangutter: etcd bump? | 14:13 |
| kevko | sorry :/ | 14:13 |
| mnasiadka | kevko: no problem ;) | 14:14 |
| jangutter | yeah, I just queued up that one for a recheck. It's safe to go in, if folks are happy with it. | 14:14 |
| jovial | For kayobe, I'm hoping to get the vGPU support in, but I need to address some review comments. Will try and do that today. | 14:14 |
| mmalchuk | added 2nd review | 14:14 |
| mmalchuk | #link https://review.opendev.org/c/openstack/kolla/+/899773 | 14:15 |
| mmalchuk | #link https://review.opendev.org/c/openstack/kolla-ansible/+/899895 | 14:15 |
| mmalchuk | LE reviews ^^^ | 14:15 |
| kevko | thanks mmalchuk | 14:15 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: CentOS/Rocky: use CentOS Cloud SIG repo instead of Delorean (Bobcat) https://review.opendev.org/c/openstack/kolla/+/899231 | 14:15 |
| mmalchuk | #link https://review.opendev.org/c/openstack/kolla/+/894948 | 14:16 |
| kevko | btw, we should also merge this ... https://review.opendev.org/c/openstack/kolla-ansible/+/900284 | 14:16 |
| mmalchuk | td in merge conflict | 14:16 |
| mnasiadka | kevko: reviewed | 14:17 |
| mnasiadka | so means td needs a lot of love | 14:17 |
| kevko | ^^ it's fixing kolla_docker which is actually not working properly without container engine | 14:17 |
| mnasiadka | it's not required to bump it this cycle | 14:17 |
| kevko | and because we have common_options in group_vars I've also added common options to all kolla_docker occurences | 14:18 |
| opendevreview | Mark Goddard proposed openstack/kayobe stable/2023.1: Stop merging config when symlink points to same file https://review.opendev.org/c/openstack/kayobe/+/900317 | 14:19 |
| mnasiadka | ok then, seems we have enough to review for the next couple of days | 14:19 |
| mnasiadka | #topic Current cycle planning | 14:20 |
| mnasiadka | I'll populate the whiteboard with priorities from the PTG | 14:20 |
| mnasiadka | But for now nothing - let's focus on Bobcat for now | 14:20 |
| mnasiadka | #topic Additional agenda (from whiteboard) | 14:20 |
| mnasiadka | Anybody up for reviewing jsuazo's patches? | 14:21 |
| mnasiadka | jsuazo: These proposals should be ready for merge / +2 uness reviewers have further comments: | 14:21 |
| mnasiadka | 875983: Enable the Fluentd Plugin Systemd | https://review.opendev.org/c/openstack/kolla-ansible/+/875983 (ready for cores) | 14:21 |
| mnasiadka | 885417: Configures the tap-as-a-service neutron plugin | https://review.opendev.org/c/openstack/kolla-ansible/+/885417 (missing a single +2) | 14:21 |
| mnasiadka | 844614: Deploy Glance and Cinder Backup with S3 backend support | https://review.opendev.org/c/openstack/kolla-ansible/+/844614 (ready for cores) | 14:21 |
| mnasiadka | Well, any core reviewer | 14:22 |
| mnasiadka | kevko, bbezak, frickler, mgoddard: would be nice to get some reviews on those soon :) | 14:22 |
| mnasiadka | let's go for the next one | 14:22 |
| mnasiadka | mhiner - Generalize Docker specific names | https://review.opendev.org/c/openstack/kolla-ansible/+/900412 | 14:22 |
| mmalchuk | we need more cores) | 14:22 |
| mnasiadka | I sort of agree that we could do that (rename docker to container) | 14:23 |
| mnasiadka | But that will make backports a bit complicated | 14:23 |
| mnasiadka | So probably we could pass for now | 14:23 |
| mnasiadka | any other comments? | 14:24 |
| mhiner | not really | 14:24 |
| kevko | probably not | 14:24 |
| kevko | actually i left a comment on mhiner patch | 14:24 |
| mnasiadka | ok then | 14:24 |
| mnasiadka | #topic Open discussion | 14:24 |
| kevko | it's good idea ..but what about users who overriden defaults ? | 14:25 |
| mmalchuk | its a bad idea | 14:25 |
| mnasiadka | well, we should have at least a cycle with supporting both variables | 14:25 |
| mnasiadka | to ask users to move | 14:25 |
| kevko | even worse idea | 14:26 |
| mnasiadka | basically we could get away with changing module name | 14:26 |
| mnasiadka | but this is changing too much | 14:26 |
| mhiner | okay, i will revert it to just change module name and we'll see ow that looks | 14:27 |
| mnasiadka | thanks | 14:27 |
| mnasiadka | anything else for discussion? | 14:28 |
| mmalchuk | this will bring probles for users with overrides, for developers to backport stuff, for almost all peoples | 14:28 |
| mmalchuk | *problems | 14:28 |
| kevko | agree | 14:28 |
| kevko | i am not saying that we shouldn't do it in some point ...but it's massive change for now i think | 14:29 |
| jsuazo | hello, had to step away for a minute so i don't know if you already discussed it, by I left some patches on the board that are ready for +2's (had them but were wiped by an update) | 14:30 |
| Vii | 1. Will multi-ceph be available in Bobcat? | 14:31 |
| Vii | 2. can you straighten the permissions for the folders located in /var/log/kolla/ | 14:31 |
| Vii | currently, for example, adding an additional filter to fluentd (td-aget) to collect logs from /var/log/kolla/openvswitch/ does not work because the "fluentd" user cannot read the files | 14:31 |
| mnasiadka | Vii: 1. if a patch is merged today in master, it will be available in Bobcat | 14:32 |
| mnasiadka | I think we merged a patch to fix openvswitch dir permissions, but maybe not backported | 14:32 |
| mnasiadka | let me see | 14:32 |
| kevko | Vii: Glad to hear that someone using my multi ceph patch :P | 14:33 |
| mnasiadka | gerrit barfed on me | 14:33 |
| mnasiadka | so no searching now | 14:33 |
| jovial | What do people think of using capabilities to allow fluentd to read log files where we can't set the ownership easily? See: https://docs.fluentd.org/deployment/linux-capability#using-cap_dac_read_search | 14:34 |
| kevko | mnasiadka: we have zed deployed somewhere if i remember ..and zed has this bug ..i am sure | 14:34 |
| Vii | kevko: I'm very glad you did it :) | 14:34 |
| mnasiadka | kevko: not saying that it doesn't, but it seems Gerrit and Github are not liking me - so can't find if we really patched it :) | 14:34 |
| kevko | (we as Ultimum) .. | 14:34 |
| kevko | mnasiadka: maybe didn't :D | 14:35 |
| mnasiadka | oh, opendev works | 14:35 |
| jovial | One example where you can't easily set permissions is the qemu instance logs in the libvirt container (we don't currently collect these) | 14:35 |
| mnasiadka | well, libvirt container should get some revamp and we should support virtlogd | 14:36 |
| mnasiadka | we could switch from monolithic to the per service approach | 14:37 |
| mnasiadka | #link https://libvirt.org/daemons.html#monolithic-vs-modular-daemons | 14:37 |
| jovial | interesting - I hadn't heard of virtlogd before | 14:37 |
| jovial | looks promising - thanks for the link | 14:38 |
| mnasiadka | ok then | 14:39 |
| mnasiadka | jsuazo: I pasted your patches asking for reviews, so yes - we covered that | 14:39 |
| mnasiadka | seems no more topics | 14:40 |
| WJeffs35 | Hey all, I was wondering if anyone is running AMD Genoa that I could pick their brains on something to compare outside of the meeting. | 14:40 |
| mnasiadka | So let's finish the meeting | 14:42 |
| mnasiadka | #endmeeting | 14:42 |
| opendevmeet | Meeting ended Wed Nov 8 14:42:12 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:42 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/kolla/2023/kolla.2023-11-08-14.00.html | 14:42 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/kolla/2023/kolla.2023-11-08-14.00.txt | 14:42 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/kolla/2023/kolla.2023-11-08-14.00.log.html | 14:42 |
| mnasiadka | bbezak: do we run Genoa anywhere, or only Milan? | 14:42 |
| Vii | thank you all for the work you put into this project :) | 14:43 |
| mnasiadka | Vii: happy to hear that ;) | 14:46 |
| kevko | Vii: Thank you | 14:47 |
| opendevreview | Will Szumski proposed openstack/kayobe master: Respect comments about inspector_keep_ports https://review.opendev.org/c/openstack/kayobe/+/829743 | 14:58 |
| opendevreview | Merged openstack/kolla-ansible master: Add support for ansible-core https://review.opendev.org/c/openstack/kolla-ansible/+/896406 | 15:01 |
| opendevreview | Merged openstack/kayobe master: Bump up Ansible supported versions to 7.x/8.x https://review.opendev.org/c/openstack/kayobe/+/898089 | 16:15 |
| opendevreview | Merged openstack/kayobe stable/2023.1: Fix last return value in _get_base_path https://review.opendev.org/c/openstack/kayobe/+/898564 | 16:15 |
| opendevreview | Will Szumski proposed openstack/kayobe master: Respect comments about inspector_keep_ports https://review.opendev.org/c/openstack/kayobe/+/829743 | 16:27 |
| opendevreview | Merged openstack/kayobe master: Improve neutron images regex https://review.opendev.org/c/openstack/kayobe/+/899915 | 16:33 |
| opendevreview | Merged openstack/kayobe master: Fix an issue when user forgot combine custom passwords https://review.opendev.org/c/openstack/kayobe/+/861397 | 16:55 |
| opendevreview | Merged openstack/kayobe stable/2023.1: Remove unsupported kolla_docker_registry_insecure variable https://review.opendev.org/c/openstack/kayobe/+/899723 | 16:55 |
| opendevreview | Merged openstack/kayobe stable/2023.1: Add custom user_data content configuration for Bifrost https://review.opendev.org/c/openstack/kayobe/+/899329 | 16:55 |
| opendevreview | Merged openstack/kayobe master: dev: Improve error checking for config check functions https://review.opendev.org/c/openstack/kayobe/+/884202 | 16:56 |
| opendevreview | Verification of a change to openstack/kayobe master failed: dnf: Remove workaround for uppercase repo files https://review.opendev.org/c/openstack/kayobe/+/899937 | 17:17 |
| opendevreview | Merged openstack/kayobe master: Generate local Kolla Ansible config in check mode https://review.opendev.org/c/openstack/kayobe/+/869986 | 18:26 |
| spatel | I have question related docker libvirt container vs metal | 19:21 |
| spatel | are there any performance difference? | 19:21 |
| opendevreview | Merged openstack/kayobe master: Fallback to templating when only one source exists https://review.opendev.org/c/openstack/kayobe/+/900060 | 19:52 |
| opendevreview | Verification of a change to openstack/kayobe master failed: Fix setting kolla_admin_openrc_cacert https://review.opendev.org/c/openstack/kayobe/+/868199 | 19:52 |
| opendevreview | Maksim Malchuk proposed openstack/kayobe stable/2023.1: Fix an issue when user forgot combine custom passwords https://review.opendev.org/c/openstack/kayobe/+/900461 | 20:20 |
| opendevreview | Maksim Malchuk proposed openstack/kayobe stable/zed: Fix an issue when user forgot combine custom passwords https://review.opendev.org/c/openstack/kayobe/+/900462 | 20:20 |
| opendevreview | Maksim Malchuk proposed openstack/kayobe stable/yoga: Fix an issue when user forgot combine custom passwords https://review.opendev.org/c/openstack/kayobe/+/900455 | 20:48 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!