opendevreview | Jay Jahns proposed openstack/kolla-ansible master: Ensure backend defaults are present in template https://review.opendev.org/c/openstack/kolla-ansible/+/933696 | 02:00 |
---|---|---|
opendevreview | Merged openstack/kolla-ansible stable/2024.2: Update .gitreview for stable/2024.2 https://review.opendev.org/c/openstack/kolla-ansible/+/933582 | 07:44 |
opendevreview | Merged openstack/kolla-ansible stable/2024.2: Update TOX_CONSTRAINTS_FILE for stable/2024.2 https://review.opendev.org/c/openstack/kolla-ansible/+/933583 | 07:46 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Fix TLS settings when letsencrypt turned on https://review.opendev.org/c/openstack/kolla-ansible/+/925971 | 10:13 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Don't notify handlers during copy-cert https://review.opendev.org/c/openstack/kolla-ansible/+/924651 | 10:14 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Performance: Don't notify handlers during config https://review.opendev.org/c/openstack/kolla-ansible/+/745164 | 10:14 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Refactor services' check-containers and optimise https://review.opendev.org/c/openstack/kolla-ansible/+/773243 | 10:14 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Fix unintentional trigger of ansible handlers https://review.opendev.org/c/openstack/kolla-ansible/+/924145 | 10:14 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [DNM] Just test custom handlers https://review.opendev.org/c/openstack/kolla-ansible/+/931761 | 10:14 |
opendevreview | Pierre Riteau proposed openstack/kayobe master: CI: Skip test_ntp_clock_synchronized on Ubuntu 24.04 https://review.opendev.org/c/openstack/kayobe/+/933463 | 10:19 |
frickler | koalas: reminder that we've moved to meeting back 1h in order to keep the same slot in european timezones (so starting 3h from now, not 2h) https://review.opendev.org/c/opendev/irc-meetings/+/933556 | 11:00 |
opendevreview | Matt Crees proposed openstack/kolla-ansible master: Fix cinder etcd3gw backend_url https://review.opendev.org/c/openstack/kolla-ansible/+/933726 | 11:02 |
frickler | mnasiadka: bbezak: SvenKieske: I've updated the bug template at https://bugs.launchpad.net/kolla-ansible/+configure-bugtracker to drop the source/binary question which seems obsolete and added a reference to quay.io, please check whether that looks reasonable to you | 11:09 |
frickler | (noticed that when looking at https://bugs.launchpad.net/kolla/+bug/2084983) | 11:10 |
opendevreview | Verification of a change to openstack/kayobe master failed: Replace use of kolla-ansible --key argument https://review.opendev.org/c/openstack/kayobe/+/933595 | 11:11 |
SvenKieske | frickler: that language regarding "official images or selfbuilt" can be read as us preferring users to use "official" images, where in reality we advise users to do the reverse: self building. maybe we can make that more clear, that the quay.io images are discouraged? | 11:11 |
SvenKieske | the rest LGTM I think, good to remove the "binary" option. | 11:13 |
kevko | anybody for second +2 and +w https://review.opendev.org/c/openstack/kolla/+/915440 ? SvenKieske already voted | 11:38 |
luca_ | hello, not sure if this is the right place for this question, sorry in advance if it's not. I installed and trying to make ironic work with kolla-ansible, i managed to start the inspection of a machine but i get an authorization error Access was denied to the following resource: baremetal:port:create, there is already a bug opened https://bugs.launchpad.net/kolla-ansible/+bug/2064655. | 12:06 |
luca_ | I was wondering if you have any hints on how to fix it, i know ironic-inspector is deprecated and i should use the agent, but as far as i understand kolla is not really supporting it. I tried adding system_scope=all to the ironic-inpector.conf, but this resulted in another error (Failed to start inspection: The baremetal-introspection service for 10.48.10.254:us-hou-01 exists but does not have any supported versions.) | 12:07 |
luca_ | should i change the policy of ironic in order to have baremetal ports created by the inspector user or is there an easier way? Thanks | 12:07 |
kevko | luca_: feel free to provide your custom policy file | 12:16 |
*** 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:36 | |
frickler | one day I'll also get rid of those ugly shortener links | 12:37 |
frickler | SvenKieske: yes, I stumbled about that, too, but not sure how to best phrase it? maybe s/official/upstream/ for a start? | 12:39 |
kevko | community ? | 12:45 |
frickler | kevko: that might also be an option, yes | 13:08 |
kevko | nice, we merged python kolla-ansible and -i as inventory is not working ? :D | 13:49 |
kevko | ah, ok ..working ..but command needs to be first | 13:49 |
kevko | then inventory | 13:49 |
bbezak | Hi | 13:56 |
bbezak | meeting in 4 | 13:56 |
bbezak | mnasiadka bbezak frickler kevko SvenKieske mmalchuk gkoper jangutter jsuazo jovial osmanlicilegi mattcrees dougszu darmach | 13:56 |
frickler | kevko: yes, that's due to how cliff works, gets mentioned in the release notes, though | 13:59 |
bbezak | #startmeeting kolla | 14:00 |
opendevmeet | Meeting started Wed Oct 30 14:00:07 2024 UTC and is due to finish in 60 minutes. The chair is bbezak. 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 |
bbezak | #topic rollcall | 14:00 |
SvenKieske | o/ | 14:00 |
frickler | \o | 14:00 |
Pcmalih_ | Having hard time booting node using ironic (redfish) | 14:00 |
mattcrees[m] | o/ | 14:00 |
kevko | \o/ | 14:00 |
mnasiadka | o/ | 14:01 |
luca_ | kevko: i did not yet tried a custom policy file, my issue is that with default ironic policy ironic-inspector service created by kolla is not able to create port, the policy in question is baremetal:port:create (role:admin and system_scope:all) or (role:service and system_scope:all) and i am not sure on how to fix it | 14:01 |
Pcmalih_ | Wondering if ironic really works with baremetal when deployed using kolla-Ansible? | 14:01 |
mnasiadka | Guys, there's a meeting | 14:02 |
bbezak | luca_, Pcmalih_ let's discuss that after the meeting | 14:02 |
mnasiadka | Questions after the meeting | 14:02 |
luca_ | ok | 14:02 |
Pcmalih_ | Appreciate if anyone share their experience. | 14:02 |
mnasiadka | #topic agenda | 14:02 |
mnasiadka | * CI status | 14:02 |
mnasiadka | * Release tasks | 14:03 |
mnasiadka | * Open discussion | 14:03 |
mnasiadka | #topic CI status | 14:03 |
mnasiadka | Seems it's sort of green, Kayobe has some clock sync issues, but apart of that looks ok | 14:03 |
mnasiadka | #topic Release tasks | 14:03 |
mnasiadka | the only task now is to get from rc1 to final for Dalmatian | 14:03 |
mnasiadka | there are some Kayobe related patches that we merged, and once Kayobe is all green - I'll post rc2 and then final | 14:04 |
mnasiadka | Once we have final, we need to remember about the deploy guide patch | 14:04 |
mnasiadka | Anything I missed regarding making final release? | 14:04 |
kevko | Well, I don't even want to reach out anymore. :) | 14:05 |
mnasiadka | reach out? | 14:05 |
kevko | https://review.opendev.org/c/openstack/kolla/+/915440 < second +2 missing | 14:06 |
mnasiadka | Well, this needs to wait for Epoxy | 14:06 |
kevko | okay, no problem | 14:06 |
mnasiadka | ok then | 14:07 |
mnasiadka | Next week probably we can look at those unmerged patches by kevko and others - once we have final release | 14:08 |
mnasiadka | #topic Open discussion | 14:08 |
mnasiadka | Anybody anything? | 14:08 |
chembervint | Hi. yes, I have one question ... just a second :) | 14:09 |
mattcrees[m] | Could we priorities some reviews for this patch please? Upgrades to Caracal break on Octavia without it right now: https://review.opendev.org/c/openstack/kolla-ansible/+/932408 | 14:09 |
mnasiadka | kevko frickler bbezak can you have a look? ^^ | 14:10 |
jovial | I have a couple of topics after we've finished with this one | 14:11 |
frickler | ack, on it | 14:11 |
chembervint | we are currently looking for a ability to control OOM score for kolla containers. we are ready to add it to kolla. couple of questions: 1) is there difference to put in into systemd units or into docker\podman-worker modeule 2) do we need only OOM-score or OOM-disable also? 3) do we need ability to adjust OOM-score for each container, or kolla-wide? | 14:11 |
kevko | yep | 14:12 |
mnasiadka | chembervint: you're being extremely vague, but I assume you want to put OOMScoreAdjust=xxx in systemd unit? | 14:13 |
mnasiadka | I mean probably nobody currently has insight what is possible in docker/podman, and what is possible in systemd apart that variable | 14:14 |
mnasiadka | So you'd need to tell us about pros and cons | 14:14 |
chembervint | mnasiadka: either OOMScoreAdjust=xxx in systemd unit or oom_score_adj into kolla_dcocker.py (docker run option) | 14:14 |
mnasiadka | does podman have the same option? | 14:15 |
chembervint | yes, like a docker - the same | 14:15 |
mnasiadka | so which one is a better solution? | 14:15 |
SvenKieske | :) | 14:17 |
kevko | mnasiadka: hmm, shouldn't we move creation of database to some separate task and call in upgrade and also bootstrap ? | 14:17 |
mnasiadka | kevko: this is a one off, it's needed only in that stable branch | 14:17 |
frickler | kevko: this is a one off only in 2023.2 and 2024.1, so worth no extra effort IMO | 14:17 |
kevko | mnasiadka: because if we merge that https://review.opendev.org/c/openstack/kolla-ansible/+/932408 ...we will have same tasks in two files ... in bootstrap.yaml and upgrade.yaml | 14:17 |
mnasiadka | I'm fine if anybody wants to go the extra mile on this, but I don't :) | 14:17 |
chembervint | I'm not sure for the moment. I've wanted to discuss it with you :) but ok - I will research more on systemd vs docker - where we have to apply this. And what do you think - do we need ability to adjust it for each kolla container, or kolla-wide setting will be enough? | 14:17 |
mnasiadka | chembervint: let's start with kolla-wide, no need to overcomplicate if there's no requirement | 14:18 |
frickler | chembervint: what's your use case for this in general? don't you have enough memory on your controllers? | 14:18 |
kevko | mnasiadka: well, what I am trying to say ...we can call creation of DB and user everytime ...during the config phase ...or somewhere around ...as that ansible module works fine ...create if db not exist ..and just touch if already exist | 14:18 |
chembervint | ok! thank you! will do my best and bring the patch soon | 14:19 |
mnasiadka | kevko: if you want to rework all roles in master - I'm fine with that, but then we won't backport this - this is a one off for upgrade from Antelope to Bobcat/Caracal, and it's perfectly fine it's not perfect | 14:20 |
kevko | i mean ..every time in context deploy, config, upgrade ... (in deploy phase before boostrap-service) | 14:20 |
kevko | hmm, okay ..i am just trying to review every time :D | 14:21 |
SvenKieske | mhm | 14:21 |
mnasiadka | Added +w | 14:22 |
mnasiadka | jovial: your turn? | 14:22 |
jovial | thanks. Firstly just an update from the PTG. I emailed openstack-dicusss about the recommended tooz driver: https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/JTYWG37I4OS7QSVIPY5DPMF5SFTVMY5I/ | 14:23 |
mnasiadka | No answers at all :) | 14:23 |
jovial | Indeed | 14:23 |
SvenKieske | why is this octavia thing only a single release thing though? I mean I believe you, but I don't understand the issue it seems :D | 14:23 |
jovial | I didn't miss any important tags in the subject did I? | 14:24 |
SvenKieske | re: tooz: it might be better to ping individual contributors to that codebase? :) | 14:24 |
mnasiadka | we introduced a new database in MariaDB just for Octavia, we didn't add creating it to upgrade, just to deploy | 14:24 |
kevko | SvenKieske: persistence_database (jobboard) was added in some release ... don't remember which | 14:24 |
chembervint | frickler: we have a lot of memory on controllers. but, you know, different services could waste a huge amount of memory esecially if they experiencing the problems. bugs, memory leaks ... we've faced such problems even with a ethernet kernel module how eat all memory .. | 14:24 |
mnasiadka | jovial: no, I'll try to ask Oslo folks, maybe we'll get an answer | 14:25 |
mnasiadka | jovial: anything else? | 14:25 |
SvenKieske | jovial: well you could have added some context to your mail: without it, it just reads like some curiosity, not that you really need the information urgently :) | 14:25 |
jovial | Just a request to get this patch merged for 2024.2: https://review.opendev.org/c/openstack/kolla-ansible/+/920294 (might be a long shot) | 14:25 |
SvenKieske | but also might be oslo people didn't read it, maybe tagging it with oslo might move more eyes | 14:25 |
mnasiadka | well, posted something on oslo channel, let's see | 14:26 |
jovial | thanks mnasiadka - it would be good to get an answer :) | 14:26 |
frickler | the oslo team is very small, similar to reqs team | 14:26 |
chembervint | jovial: our experience is to use tooz + redis - redis is much more faster than a etcd. kolla os already ready for redis. it works really fine. and also we've tested tooz+consul - also works fine, if you already using the consul. | 14:27 |
kevko | +1 | 14:27 |
mnasiadka | jovial: I'm afraid it's rc1, I wouldn't like to make the release process more complicated - can we come back to it in 2025.1 (and if it's a bug - we could backport) | 14:27 |
mnasiadka | I think so many people use Redis, and I think Octavia jobboard (at least in kolla-ansible) requires redis, so we shouldn't really overcomplicate things | 14:28 |
jovial | Yeah, fair enough. I'm calling it a bug, but its a matter of interpretation there :laughs: | 14:28 |
kevko | if not all people :D | 14:28 |
mnasiadka | Once valkey is in Debian packages - we'll move to valkey in 2025.1 and that's it | 14:28 |
kevko | Hmm, maybe I should create and build valkey for debian :) | 14:29 |
mnasiadka | ok, anything else? :) | 14:29 |
mnasiadka | kevko: it's built for unstable, but seems it's not on the FTP - I have no clue why it takes so long | 14:29 |
kevko | mnasiadka: it's in trixie already | 14:30 |
mnasiadka | ok, it wasn't there like two weeks ago | 14:30 |
kevko | mnasiadka: yeah, it's debian policy to have some package in unstable for some time ..then it's automatically moved | 14:30 |
mnasiadka | Do we need to think about moving to Trixie in 2025.1? When is it due for releasing? | 14:31 |
kevko | 2025 i think | 14:32 |
mnasiadka | Ok, then maybe we need to jump in 2025.1, we'll see | 14:32 |
kevko | june maybe :) | 14:32 |
mnasiadka | or in 2025.2 | 14:32 |
mnasiadka | ok then | 14:32 |
mnasiadka | I see no more topics | 14:32 |
mnasiadka | Thanks for coming guys | 14:33 |
mnasiadka | jovial: let's focus on kayobe rc1 and kolla-ansible rc2 | 14:33 |
mnasiadka | See you next week | 14:33 |
mnasiadka | #endmeeting | 14:33 |
frickler | bbezak needs to end it | 14:34 |
bbezak | #endmeeting | 14:34 |
opendevmeet | Meeting ended Wed Oct 30 14:34:12 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:34 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/kolla/2024/kolla.2024-10-30-14.00.html | 14:34 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/kolla/2024/kolla.2024-10-30-14.00.txt | 14:34 |
opendevmeet | Log: https://meetings.opendev.org/meetings/kolla/2024/kolla.2024-10-30-14.00.log.html | 14:34 |
SvenKieske | :D | 14:34 |
SvenKieske | thank you | 14:34 |
Pcmalih_ | Hi #mnasiadka | 14:34 |
Pcmalih_ | Can we talk about ironic issues for few minutes? | 14:35 |
SvenKieske | it seems no oO | 14:37 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Fix TLS settings when letsencrypt turned on https://review.opendev.org/c/openstack/kolla-ansible/+/925971 | 15:35 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Don't notify handlers during copy-cert https://review.opendev.org/c/openstack/kolla-ansible/+/924651 | 15:35 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Performance: Don't notify handlers during config https://review.opendev.org/c/openstack/kolla-ansible/+/745164 | 15:35 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Refactor services' check-containers and optimise https://review.opendev.org/c/openstack/kolla-ansible/+/773243 | 15:35 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Fix unintentional trigger of ansible handlers https://review.opendev.org/c/openstack/kolla-ansible/+/924145 | 15:35 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [DNM] Just test custom handlers https://review.opendev.org/c/openstack/kolla-ansible/+/931761 | 15:35 |
kevko | SvenKieske, mnasiadka: i forgot ..this should be merged before dalmatian I would say https://review.opendev.org/c/openstack/kolla-ansible/+/925971 | 15:38 |
mnasiadka | that's a bug, can be backported :) | 15:39 |
opendevreview | Merged openstack/kolla-ansible stable/2024.1: Fix Octavia service upgrade failure from 2023.1 https://review.opendev.org/c/openstack/kolla-ansible/+/932408 | 15:39 |
kevko | mnasiadka: okay, nevermind ... | 15:42 |
opendevreview | Verification of a change to openstack/kayobe master failed: Replace use of kolla-ansible --key argument https://review.opendev.org/c/openstack/kayobe/+/933595 | 15:57 |
opendevreview | Matt Crees proposed openstack/kolla-ansible stable/2023.2: Fix Octavia service upgrade failure from 2023.1 https://review.opendev.org/c/openstack/kolla-ansible/+/933757 | 16:00 |
opendevreview | Jake Hutchinson proposed openstack/kayobe master: Add loop control to custom DNF repos https://review.opendev.org/c/openstack/kayobe/+/933761 | 16:37 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible stable/2024.2: DNM: Run all jobs https://review.opendev.org/c/openstack/kolla-ansible/+/933762 | 16:41 |
opendevreview | Will Szumski proposed openstack/kayobe master: Aligns Kayobe Ansible version with Kolla Ansible https://review.opendev.org/c/openstack/kayobe/+/926639 | 16:46 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Fix TLS settings when letsencrypt turned on https://review.opendev.org/c/openstack/kolla-ansible/+/925971 | 16:47 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Don't notify handlers during copy-cert https://review.opendev.org/c/openstack/kolla-ansible/+/924651 | 16:47 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Performance: Don't notify handlers during config https://review.opendev.org/c/openstack/kolla-ansible/+/745164 | 16:47 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Refactor services' check-containers and optimise https://review.opendev.org/c/openstack/kolla-ansible/+/773243 | 16:47 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Fix unintentional trigger of ansible handlers https://review.opendev.org/c/openstack/kolla-ansible/+/924145 | 16:47 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [DNM] Just test custom handlers https://review.opendev.org/c/openstack/kolla-ansible/+/931761 | 16:47 |
opendevreview | Will Szumski proposed openstack/kayobe master: Support running without root privileges https://review.opendev.org/c/openstack/kayobe/+/923430 | 16:51 |
opendevreview | Will Szumski proposed openstack/kayobe master: Support running without root privileges https://review.opendev.org/c/openstack/kayobe/+/923430 | 16:52 |
*** amorin_ is now known as amorin | 17:35 | |
opendevreview | Merged openstack/kolla-ansible master: Prevent accidental downgrades of RabbitMQ https://review.opendev.org/c/openstack/kolla-ansible/+/929912 | 17:59 |
-opendevstatus- NOTICE: The Gerrit service on review.opendev.org will be offline momentarily at 20:30 utc (half an hour from now) to apply a configuration change | 20:02 | |
-opendevstatus- NOTICE: The Gerrit service on review.opendev.org will be offline momentarily to apply a configuration change | 20:31 | |
opendevreview | Verification of a change to openstack/kayobe master failed: Replace use of kolla-ansible --key argument https://review.opendev.org/c/openstack/kayobe/+/933595 | 20:39 |
opendevreview | Merged openstack/kayobe master: Replace use of kolla-ansible --key argument https://review.opendev.org/c/openstack/kayobe/+/933595 | 23:56 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!