| *** JohnnyW557 is now known as JohnnyW55 | 04:13 | |
| *** mrunge_ is now known as mrunge | 06:39 | |
| mnasiadka | good morning | 08:13 |
|---|---|---|
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Bump ansible-core versions to 2.17 and 2.18 https://review.opendev.org/c/openstack/kolla-ansible/+/941787 | 10:12 |
| opendevreview | Verification of a change to openstack/kolla-ansible master failed: Merge of container_facts modules https://review.opendev.org/c/openstack/kolla-ansible/+/912460 | 10:15 |
| opendevreview | Jakub Darmach proposed openstack/kolla stable/2024.1: Added systemd-standalone-sysusers package for Debuntu https://review.opendev.org/c/openstack/kolla/+/944100 | 10:17 |
| opendevreview | Jakub Darmach proposed openstack/kolla stable/2024.1: Add support for Ubuntu 24.04 LTS https://review.opendev.org/c/openstack/kolla/+/932386 | 10:25 |
| frickler | hmm, no idea why https://review.opendev.org/c/openstack/kolla-ansible/+/912460 is suddenly failing on all podman jobs, doesn't really look related? | 10:28 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Bump ansible-core versions to 2.17 and 2.18 https://review.opendev.org/c/openstack/kolla-ansible/+/941787 | 10:43 |
| opendevreview | Jakub Darmach proposed openstack/kolla stable/2024.1: Add support for Ubuntu 24.04 LTS https://review.opendev.org/c/openstack/kolla/+/932386 | 11:21 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Wait until containers exit health starting state https://review.opendev.org/c/openstack/kolla-ansible/+/944104 | 11:29 |
| mnasiadka | frickler: I think it happens in all patches, maybe ^^ will help a bit | 11:34 |
| priteau | I saw some failures on podman jobs where we start rabbitmq and immediately after we run `exec rabbitmq <something>` but it fails because the container is not really started yet. | 11:36 |
| priteau | Apparently this works OK on Docker but not on Podman | 11:51 |
| frickler | sounds reasonable, but what has changed to make it failing now? | 12:02 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Wait until containers exit health starting state https://review.opendev.org/c/openstack/kolla-ansible/+/944104 | 12:07 |
| opendevreview | Jakub Darmach proposed openstack/kolla stable/2024.1: Add support for Ubuntu 24.04 LTS https://review.opendev.org/c/openstack/kolla/+/932386 | 12:31 |
| darmach | Are we on today mnasiadka? | 13:06 |
| koperg[m] | 15:00 ? :D | 13:11 |
| mnasiadka | Well, it's 14:17 | 13:17 |
| mnasiadka | Unless my jet lag time machine lies :) | 13:17 |
| darmach | Those pesky timezones! | 13:30 |
| opendevreview | Norman Ziegner proposed openstack/kolla-ansible master: Fix typo in prechecks command https://review.opendev.org/c/openstack/kolla-ansible/+/944117 | 13:41 |
| mnasiadka | mnasiadka bbezak frickler kevko SvenKieske mmalchuk gkoper jangutter jsuazo jovial osmanlicilegi mattcrees dougszu darmach pabloclsn - meeting in 9 minutes | 13:51 |
| opendevreview | Zhoneym proposed openstack/kolla-ansible master: Limit bcrypt version to <=4.0.1 for compatibility https://review.opendev.org/c/openstack/kolla-ansible/+/943184 | 13:56 |
| opendevreview | Norman Ziegner proposed openstack/kolla-ansible master: Fix typo in prechecks command https://review.opendev.org/c/openstack/kolla-ansible/+/944117 | 13:57 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Wait until containers exit health starting state https://review.opendev.org/c/openstack/kolla-ansible/+/944104 | 14:01 |
| mnasiadka | gosh | 14:02 |
| mnasiadka | #startmeeting kolla | 14:02 |
| opendevmeet | Meeting started Wed Mar 12 14:02:26 2025 UTC and is due to finish in 60 minutes. The chair is mnasiadka. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:02 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:02 |
| opendevmeet | The meeting name has been set to 'kolla' | 14:02 |
| mnasiadka | #topic rollcall | 14:02 |
| mnasiadka | o/ | 14:02 |
| frickler | \o | 14:02 |
| r-krcek | o/ | 14:02 |
| frickler | still jetlagged? ;) | 14:02 |
| mmalchuk | o/ | 14:02 |
| mnasiadka | frickler: sort of, doing a lot of things at the same time to catch up two weeks absence :) | 14:03 |
| mnasiadka | darmach koperg wake up :) | 14:04 |
| mnasiadka | #topic agenda | 14:04 |
| mnasiadka | * Announcements | 14:05 |
| mnasiadka | * CI status | 14:05 |
| mnasiadka | * Release tasks | 14:05 |
| mnasiadka | * Current cycle planning | 14:05 |
| mnasiadka | * Additional agenda (from whiteboard) | 14:05 |
| mnasiadka | * Open discussion | 14:05 |
| mnasiadka | #topic Announcements | 14:05 |
| mnasiadka | So... I'm in New Zealand time zone when the PTG happens - so PTG will be around 7AM UTC time slot - I'll post those on openstack-discuss ML once I book these and create an etherpad for Kolla | 14:06 |
| mnasiadka | or something like that | 14:06 |
| mnasiadka | #topic CI status | 14:06 |
| mnasiadka | Seems we have some neutron-metadata-agent healthchecking issues | 14:06 |
| mnasiadka | Trying to debug it in https://review.opendev.org/c/openstack/kolla-ansible/+/944104 | 14:07 |
| mnasiadka | Also add some more debug to CI logs for unhealthy containers, so we don't have to do that next time. | 14:07 |
| mnasiadka | I've seen some rabbitmq podman related container state problems, but I assume it's different | 14:07 |
| mnasiadka | (although these don't happen every time) | 14:07 |
| mnasiadka | #topic Release tasks | 14:08 |
| frickler | for metadata maybe neutron changed something? | 14:08 |
| frickler | might be related to eventlet removal and change timings somehow | 14:08 |
| mnasiadka | Might be, that's why I need to first check if for example the process name did not change or something similar - we need more debug logs | 14:08 |
| mnasiadka | And maybe better healthchecking scripts, but that's not going to happen soon ;) | 14:08 |
| frickler | ack | 14:09 |
| mnasiadka | It's R-3 | 14:09 |
| mnasiadka | We might see breakages due to some last minute patches | 14:09 |
| mnasiadka | I don't think we have any immediate work to do - koperg is working on Ansible bump | 14:09 |
| mnasiadka | #topic Current cycle planning | 14:09 |
| mnasiadka | Anybody wants to discuss any feature? any review requests? | 14:10 |
| yuval | I have this one: https://review.opendev.org/c/openstack/kolla-ansible/+/938649 | 14:10 |
| yuval | I think its merge ready | 14:10 |
| mmalchuk | yep. cinder issue in nova with tempest | 14:10 |
| mmalchuk | https://review.opendev.org/c/openstack/kolla-ansible/+/937038/6 | 14:10 |
| mmalchuk | frickler: | 14:11 |
| seunghunlee | I have https://review.opendev.org/c/openstack/kolla/+/934735 | 14:11 |
| frickler | I have a simple ironic-related patch https://review.opendev.org/c/openstack/kolla/+/942785 and bbezak had another one which we might want to combine | 14:11 |
| frickler | https://review.opendev.org/c/openstack/kolla/+/933860 | 14:11 |
| frickler | I'll add the others above to my list | 14:12 |
| mnasiadka | yuval: one last question and I'm happy to apply my +2 ;) | 14:12 |
| mmalchuk | frickler: can we go back with catalog_info in 6 patchet? | 14:13 |
| frickler | mmalchuk: I don't think we should backport in kolla anyway, but I'll update, yes | 14:14 |
| mmalchuk | frickler: but we need merge in master an I mean 6 patchset | 14:16 |
| mmalchuk | not latest | 14:16 |
| mnasiadka | ok, added the rest of the patches to my review queue | 14:16 |
| opendevreview | yuval proposed openstack/kolla-ansible master: add lightbits driver support https://review.opendev.org/c/openstack/kolla-ansible/+/938649 | 14:17 |
| seunghunlee | Thanks | 14:17 |
| yuval | mnasiadka: you right, removed :) | 14:18 |
| mnasiadka | #topic Additional agenda (from whiteboard) | 14:20 |
| mnasiadka | Nothing new there | 14:20 |
| mnasiadka | #topic Open discussion | 14:21 |
| mnasiadka | Anybody anything? | 14:21 |
| yuval | yea | 14:23 |
| yuval | I got a question | 14:23 |
| mnasiadka | shoot | 14:23 |
| yuval | I want to add an env variable to the cinder docker | 14:23 |
| yuval | currently I am forking out from the main branch to do it using the group_vars | 14:23 |
| yuval | and kolla-supported way to do it? | 14:24 |
| yuval | any* | 14:24 |
| yuval | this is the reason I need to do it: https://opendev.org/openstack/os-brick/src/commit/6e83ac6eeee8f3a4b3265a4e927dca1bc190e088/os_brick/privileged/__init__.py#L24 | 14:25 |
| mnasiadka | Not really, we have some places where we do supply environment in role defaults/ and then use it when starting the container, but not in Cinder | 14:25 |
| mnasiadka | Like https://github.com/openstack/kolla-ansible/blob/b952debacc0b6516468d6fd8121facc65eb4d9c1/ansible/roles/etcd/defaults/main.yml#L7 | 14:25 |
| frickler | to me that looks like a bad design decision in os-brick | 14:26 |
| yuval | yes.. maybe we can do some magic so we can control this in any docker we bring up? | 14:26 |
| yuval | frickler: totally agree - I uploaded a change for it but was rejected | 14:27 |
| frickler | I don't see any rejection here https://review.opendev.org/c/openstack/os-brick/+/942689 | 14:28 |
| yuval | yes... silent rejection | 14:29 |
| frickler | cinder team is just almost non-existent in my experience | 14:30 |
| yuval | anyway I will think how we can control it and bring up a patch | 14:30 |
| yuval | frickler: no, no, They are doing amazing job. it just doing this change is not trivial | 14:31 |
| mnasiadka | Ok then | 14:31 |
| mnasiadka | I guess that's it | 14:32 |
| MattCrees[m] | Sorry, missed the current cycle planning. Could we get some core reviewer eyes on the big rabbitmq patch chain again please? https://review.opendev.org/c/openstack/kolla-ansible/+/924623 and https://review.opendev.org/c/openstack/kolla-ansible/+/942867 are good to go imo. | 14:32 |
| koperg[m] | @mnasiadka: sorry missed the start of the meeting ( client work ) We can consider ansible bump technically done - thanks for your help :D | 14:32 |
| MattCrees[m] | We have quite a few to patches get through if we want RabbitMQ 4.0 in epoxy :) | 14:32 |
| mnasiadka | Well, we need more reviewers it seems. | 14:33 |
| mnasiadka | If you guys would review not only the patches you want to get in, maybe we'd get more core reviewers... | 14:33 |
| yuval | I will try | 14:35 |
| darmach | +1 | 14:37 |
| darmach | (me too) | 14:37 |
| koperg[m] | Can take a look, if you are satisfied with +1 :P | 14:37 |
| MattCrees[m] | Ok, I'll try to make time to review other patches. | 14:38 |
| mnasiadka | Reality is, that there are 4 active core reviewers, and we can't have velocity to merge everything - and the situation is getting worse, not better. | 14:38 |
| mnasiadka | If your employers value/use kolla/kolla-ansible/kayobe - they should invest more. | 14:39 |
| mnasiadka | That's my 5 cents :) | 14:39 |
| mnasiadka | Thank you for coming - see you next week :) | 14:39 |
| mnasiadka | #endmeeting | 14:39 |
| opendevmeet | Meeting ended Wed Mar 12 14:39:42 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:39 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/kolla/2025/kolla.2025-03-12-14.02.html | 14:39 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/kolla/2025/kolla.2025-03-12-14.02.txt | 14:39 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/kolla/2025/kolla.2025-03-12-14.02.log.html | 14:39 |
| koperg[m] | o/ | 14:39 |
| yuval | Thank you! | 14:39 |
| MattCrees[m] | cheers | 14:39 |
| mmalchuk | mnasiadka: thanks | 14:39 |
| darmach | o/ | 14:40 |
| seunghunlee | Thanks | 14:41 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Fix boolean representation in all configurations https://review.opendev.org/c/openstack/kolla-ansible/+/944124 | 15:00 |
| kevko | darmach: So do you belong to the 'true' club or the 'True' club of people? | 15:14 |
| darmach | I'm one of the upper case folk | 15:15 |
| kevko | darmach: me too ..as it is more Pythonic | 15:16 |
| kevko | But since I got a -1 from mnasiadka for that in a useful review… let's fix it then :D … because at least I'll feel in future reviews that it correlates with the other booleans in the code… even though it will break my heart :D | 15:18 |
| kevko | :D | 15:18 |
| darmach | Java, however, goes lowercase! At least we're fixing the worst - mixed case troughout the project ;) | 15:20 |
| mnasiadka | For now, if the majority of configs is lowercase true - then let’s not go against the majority | 15:21 |
| mnasiadka | If you want to run a full scale operation and define a documented standard and change it everywhere - fine by me | 15:21 |
| kevko | yeah | 15:21 |
| kevko | Well, there's a difference between having true in JSON, YAML, or a Python config... because true in Python is just a string, whereas True is actually a boolean. The fact that it works is probably just due to configparser or maybe oslo.config... But honestly, I don’t really care. If it’s supposed to be lowercase, then it should be lowercase | 15:25 |
| kevko | everywhere. Otherwise, someone will copy a True example, someone else will copy a true example, and the whole thing will happen again... | 15:25 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Wait until containers exit health starting state https://review.opendev.org/c/openstack/kolla-ansible/+/944104 | 17:02 |
| opendevreview | Will Szumski proposed openstack/kayobe master: Bump version of stackhpc.linux collection https://review.opendev.org/c/openstack/kayobe/+/944142 | 17:38 |
| opendevreview | Merged openstack/kayobe master: Revert "CI: Make kayobe-overcloud-upgrade-slurp-rocky9 non-voting" https://review.opendev.org/c/openstack/kayobe/+/944077 | 18:17 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Wait until containers exit health starting state https://review.opendev.org/c/openstack/kolla-ansible/+/944104 | 18:43 |
| opendevreview | Dr. Jens Harbott proposed openstack/kolla-ansible master: Cinder: move to block-storage in service catalog https://review.opendev.org/c/openstack/kolla-ansible/+/937038 | 18:44 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Wait until containers exit health starting state https://review.opendev.org/c/openstack/kolla-ansible/+/944104 | 18:47 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Wait until containers exit health starting state https://review.opendev.org/c/openstack/kolla-ansible/+/944104 | 18:55 |
| -opendevstatus- NOTICE: One of our Zuul job log storage providers is experiencing errors. We have removed that storage target from base jobs. You should be able to safely recheck changes now. | 20:23 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!