| opendevreview | Pierre Riteau proposed openstack/kayobe master: Test overcloud provisioning with Bifrost https://review.opendev.org/c/openstack/kayobe/+/903249 | 07:24 |
|---|---|---|
| opendevreview | Magnus Lööf proposed openstack/kolla-ansible master: Enable TLS backend for designate https://review.opendev.org/c/openstack/kolla-ansible/+/866524 | 07:25 |
| SvenKieske | morning, shouldn't we backport https://review.opendev.org/c/openstack/kolla-ansible/+/929598 ? (arm arch disabled in CI) | 07:48 |
| opendevreview | Sven Kieske proposed openstack/kolla-ansible stable/2024.1: CI: Mark aarch64 non-voting https://review.opendev.org/c/openstack/kolla-ansible/+/929737 | 07:50 |
| opendevreview | Sven Kieske proposed openstack/kolla-ansible stable/2024.1: CI: Mark aarch64 non-voting https://review.opendev.org/c/openstack/kolla-ansible/+/929737 | 07:53 |
| opendevreview | Bui Doan Dang proposed openstack/kolla stable/2023.1: Fix cinder-backup: add missing sysfsutils,lsscsi package https://review.opendev.org/c/openstack/kolla/+/927890 | 08:16 |
| opendevreview | Bui Doan Dang proposed openstack/kolla stable/2023.2: Fix cinder-backup: add missing sysfsutils,lsscsi package https://review.opendev.org/c/openstack/kolla/+/927891 | 08:16 |
| opendevreview | Bui Doan Dang proposed openstack/kolla stable/2024.1: Fix cinder-backup: add missing sysfsutils,lsscsi package https://review.opendev.org/c/openstack/kolla/+/927892 | 08:17 |
| opendevreview | Will Szumski proposed openstack/kayobe master: CI: Workaround rc:-13 issue https://review.opendev.org/c/openstack/kayobe/+/929739 | 08:47 |
| opendevreview | Victor Chembaev proposed openstack/kolla-ansible stable/2024.1: Fix os-brick multipathd del map fails for SAN multipath devices https://review.opendev.org/c/openstack/kolla-ansible/+/929745 | 09:21 |
| opendevreview | Victor Chembaev proposed openstack/kolla-ansible stable/2023.2: Fix os-brick multipathd del map fails for SAN multipath devices https://review.opendev.org/c/openstack/kolla-ansible/+/929746 | 09:21 |
| opendevreview | Victor Chembaev proposed openstack/kolla-ansible stable/2023.1: Fix os-brick multipathd del map fails for SAN multipath devices https://review.opendev.org/c/openstack/kolla-ansible/+/929747 | 09:21 |
| oli | is there a way to set the vnc keymap for console to something other than en_us ? | 09:33 |
| mnasiadka | morning | 09:36 |
| SvenKieske | o/ | 09:37 |
| SvenKieske | oli: you need at least to make sure your novnc lib is new enough, see: https://docs.openstack.org/nova/latest/admin/remote-console-access.html#novnc-based-vnc-console | 09:38 |
| SvenKieske | but this is actually more a javascript/frontend question, because afaik the keymapping is directly done in novnc/javascript | 09:38 |
| SvenKieske | oli apart from that the guest vm keymappings must of course map to your local keyboard | 09:41 |
| SvenKieske | qemu these days should just send raw keymaps | 09:41 |
| SvenKieske | see this issue for the history (it) | 09:42 |
| SvenKieske | it's from 2010: https://github.com/novnc/noVNC/issues/21 * | 09:42 |
| mnasiadka | any urgent reviews we know of? | 09:46 |
| oli | thanks SvenKieske for shedding some light on that | 09:47 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: Add support for Ubuntu Noble Numbat (24.04 LTS) https://review.opendev.org/c/openstack/kolla-ansible/+/925581 | 09:54 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: Add support for Ubuntu Noble Numbat (24.04 LTS) https://review.opendev.org/c/openstack/kolla-ansible/+/925581 | 09:55 |
| PrzemekK | We have errors like this in ceilometer : Error creating resource ac60ac0d-eb97-4891-a5a4-2175ff7d05f6-vdc: Resource type instance_disk does not exist (HTTP 404): gnocchiclient.exceptions.ResourceTypeNotFound: Resource type instance_disk does not exist (HTTP 404) | 09:55 |
| PrzemekK | I see difference in https://github.com/openstack/kolla-ansible/tree/stable/2024.1/ansible/roles/ceilometer/templates and https://github.com/openstack/ceilometer/tree/stable/2024.1/ceilometer/pipeline/data | 09:56 |
| PrzemekK | it can be related to that there is '*' and not "*" ? | 09:57 |
| PrzemekK | and /event_definitions.yaml is different in kolla-ansible and ceilometer repo | 09:57 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Add support for Ubuntu Noble Numbat (24.04 LTS) https://review.opendev.org/c/openstack/kolla-ansible/+/925581 | 09:59 |
| opendevreview | Ivan Vnučko proposed openstack/kolla-ansible master: Fix Masakari instancemonitor when libvirt TLS enabled https://review.opendev.org/c/openstack/kolla-ansible/+/929594 | 10:00 |
| opendevreview | Grzegorz Koper proposed openstack/kolla-ansible master: Fixing typo in etc/kolla/globals.yml https://review.opendev.org/c/openstack/kolla-ansible/+/929752 | 10:06 |
| opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: docker_sdk: Allow breaking system packages https://review.opendev.org/c/openstack/ansible-collection-kolla/+/929755 | 10:13 |
| opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: CI: Bump ansible-core and ansible-lint https://review.opendev.org/c/openstack/ansible-collection-kolla/+/929756 | 10:15 |
| opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: CI: Bump ansible-core and ansible-lint https://review.opendev.org/c/openstack/ansible-collection-kolla/+/929756 | 10:20 |
| opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: CI: Bump ansible-core and ansible-lint https://review.opendev.org/c/openstack/ansible-collection-kolla/+/929756 | 10:22 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Fix handling configs in base image https://review.opendev.org/c/openstack/kolla/+/915440 | 10:23 |
| opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: CI: Bump ansible-core and ansible-lint https://review.opendev.org/c/openstack/ansible-collection-kolla/+/929756 | 10:25 |
| mattcrees | Good morning all | 10:26 |
| kevko | mornign | 10:27 |
| mattcrees | Could I get a second core review here please? https://review.opendev.org/c/openstack/kolla-ansible/+/926146 SvenKieske perhaps, as you've already given +1 ;) | 10:28 |
| mnasiadka | done | 10:34 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Fix handling configs in base image https://review.opendev.org/c/openstack/kolla/+/915440 | 10:35 |
| kevko | mnasiadka, mattcrees, SvenKieske : It would be nice to review this -> https://review.opendev.org/c/openstack/kolla/+/915440 << I've added 100% coverage of unit tests for my proposed functions ... also docstrings and everything ..So, I would appreciate a review. It's another step for config and notifiers .... | 10:37 |
| kevko | https://review.opendev.org/c/openstack/kolla/+/929516 << and something similar issue ... around same topic | 10:37 |
| mattcrees | Sure, I'll add them to my list | 10:40 |
| kevko | Thank you ... | 10:40 |
| * SvenKieske writing from lunch: kevko it's already on my list, you reminded me yesterday already. but it's no small review, it's over 600 lines, so I need to make some time for that, nothing I can review in half an hour :) but I'll do it. | 10:42 | |
| kevko | it's some time i've written it ... But I am using it, and it works very nicely, but when I was writing the unit tests and docstrings ... I got a bit of a headache from the algorithm :). I really recommend testing it on any use case somewhere in a test container. | 10:43 |
| kevko | SvenKieske: well, most of that lines are docstrings and unit tests .....before it was 20 lines :D | 10:43 |
| kevko | SvenKieske: okay, 70 lines | 10:44 |
| opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: CI: Bump ansible-core and ansible-lint https://review.opendev.org/c/openstack/ansible-collection-kolla/+/929756 | 10:46 |
| SvenKieske | well I don't know how you do reviews but I review every changed line and you even have to look between diffs sometimes and for context reference other files or patchsets as well.. | 10:58 |
| kevko | SvenKieske: do you test in real ? | 10:58 |
| SvenKieske | so sometimes you also need to look at the lines that didn't change ;) | 10:58 |
| SvenKieske | depends on the change and on the area. a change can "work" in real and still be totally wrong, and the reverse can also be true. | 10:59 |
| SvenKieske | also a code review is a code review not a code "test", some things can't easily be tested locally, that's what CI is for. | 11:00 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Add support for Ubuntu Noble Numbat (24.04 LTS) https://review.opendev.org/c/openstack/kolla-ansible/+/925581 | 11:00 |
| kevko | SvenKieske: It's good that everyone does it differently :) | 11:01 |
| kevko | SvenKieske: i think everything can be tested locally except some propriertary software or hw ... openstack in HA on some VMs ...that's it | 11:03 |
| kevko | SvenKieske: but yeah, i totally understand | 11:03 |
| SvenKieske | well personally I only got my laptop locally, so I find it hard to setup e.g. galera cluster, or generally stuff that requires external networks etc. | 11:04 |
| SvenKieske | I don't have any huge lab or something like that | 11:04 |
| SvenKieske | maybe some day I get to that | 11:04 |
| mnasiadka1 | Well, nobody has a huge lab - but usually we have an option to spawn a couple of VMs | 11:16 |
| kevko | I have whole openstack cloud and several stacks on :) | 11:31 |
| kevko | Translation (Slovak to English): | 11:32 |
| kevko | But it's already an old man, the disks in Ceph are failing, sometimes the whole server, and so on... but it still serves its purpose. | 11:32 |
| kevko | ups, it seems i need to fix my chatgpt launguage helper in terminal :D :D | 11:36 |
| opendevreview | Merged openstack/kolla-ansible stable/2024.1: CI: Mark aarch64 non-voting https://review.opendev.org/c/openstack/kolla-ansible/+/929737 | 11:36 |
| opendevreview | Merged openstack/kolla-ansible master: Fixing typo in etc/kolla/globals.yml https://review.opendev.org/c/openstack/kolla-ansible/+/929752 | 11:36 |
| kevko | I'm getting lazier and lazier, but I'm kind of enjoying it | 11:37 |
| kevko | btw, i found that we don't have any CI test for backup ... shouldn't we have some ? | 11:38 |
| mnasiadka | who needs backup working ;) | 11:46 |
| kevko | mnasiadka: hahaaaa :D true :D | 11:50 |
| kevko | True, no one reported a bug that the backup actually doesn’t work ... I discovered it during the proxysql rework | 11:51 |
| opendevreview | Verification of a change to openstack/kolla-ansible master failed: Add a precheck to catch RMQ SLURP upgrades https://review.opendev.org/c/openstack/kolla-ansible/+/926146 | 11:55 |
| PrzemekK | here reports that backup not working from couple days ago - https://bugs.launchpad.net/kolla-ansible/+bug/2080818 https://bugs.launchpad.net/kolla-ansible/+bug/2080408 | 11:57 |
| kevko | PrzemekK: yeah, because I reported those bugs :D | 12:00 |
| kevko | PrzemekK: it's me | 12:00 |
| PrzemekK | ahh ok. I dont use integrated db in kolla-ansible because its hard to install backup agent to backup logs every x minutes etc | 12:02 |
| kevko | it seems nobody use it :D | 12:06 |
| mnasiadka1 | nobody use proxysql you mean? :) | 12:07 |
| kevko | backup :D | 12:13 |
| mnasiadka1 | well, second bug is fixed | 12:13 |
| mnasiadka1 | first bug says backup does not work with proxysql | 12:13 |
| kevko | mnasiadka1: there were a several patches regarding proxysql ... and also TLS from Roman ... | 12:14 |
| mnasiadka1 | is there a bug that says backup is not working at all? preferably in stable releases? | 12:14 |
| kevko | mnasiadka1: i don't know ... i work against master on my dev | 12:14 |
| kevko | mnasiadka1: Okay, now I understand your hidden implication :D | 12:16 |
| kevko | Hmm, I'm actually realizing that if I find a bug in the master branch... I probably shouldn't report it as a bug, right? | 12:18 |
| kevko | but proxysql is a bug also in stable ... | 12:19 |
| frickler | fyi I'll be a bit late for the meeting | 12:27 |
| mnasiadka1 | ah, it's Wed - right | 12:28 |
| *** mnasiadka1 is now known as mnasiadka | 12:28 | |
| opendevreview | Matt Crees proposed openstack/kolla-ansible master: Fix type in RabbitMQ versions pinning https://review.opendev.org/c/openstack/kolla-ansible/+/929768 | 12:28 |
| opendevreview | Matt Crees proposed openstack/kolla-ansible master: Fix typo in RabbitMQ versions pinning https://review.opendev.org/c/openstack/kolla-ansible/+/929768 | 12:30 |
| PrzemekK | Can You introduce this change to master / 2024.1 code ? https://review.opendev.org/q/topic:%22implement-neutron-ovn-vpn-agent%22 https://bugs.launchpad.net/kolla-ansible/+bug/2048392 | 12:35 |
| mnasiadka | PrzemekK: help us in reviewing and testing the code, then maybe it's going to be in 2024.2 - we don't backport features | 12:40 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [CI] Add mariadb backup into mariadb scenario https://review.opendev.org/c/openstack/kolla-ansible/+/929771 | 12:41 |
| mnasiadka | mgoddard mnasiadka bbezak frickler kevko SvenKieske mmalchuk gkoper jangutter jsuazo jovial osmanlicilegi mattcrees dougszu darmach | 12:51 |
| mnasiadka | - meeting in 9 minutes | 12:51 |
| kevko | cigarette time :) | 12:52 |
| mnasiadka | #startmeeting kolla | 13:00 |
| opendevmeet | Meeting started Wed Sep 18 13:00:04 2024 UTC and is due to finish in 60 minutes. The chair is mnasiadka. Information about MeetBot at http://wiki.debian.org/MeetBot. | 13:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 13:00 |
| opendevmeet | The meeting name has been set to 'kolla' | 13:00 |
| mnasiadka | #topic rollcall | 13:00 |
| SvenKieske | o/ | 13:00 |
| jovial | o/ | 13:00 |
| mnasiadka | o/ | 13:00 |
| mhiner | o/ | 13:01 |
| mattcrees | o/ | 13:01 |
| kevko | \o/ | 13:01 |
| mmalchuk | o/ | 13:01 |
| mnasiadka | #topic agenda... (full message at <https://matrix.org/oftc/media/v1/media/download/AZ8DWFB-4WLSxI_dYsDe3rGB2f2-z4LGcJb4q5sdro8hxmDJkV27TceKI56yr1eU0DklwzJ4v-E7o_sAvwpVz_9CeSBvEpawAG1hdHJpeC5vcmcvQkpIWHFGeldrbFpGS2NuWW9WV29oQlN1>) | 13:02 |
| mnasiadka | #topic CI status | 13:02 |
| mnasiadka | Anything bleeding red? | 13:02 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [CI] Add mariadb backup into mariadb scenario https://review.opendev.org/c/openstack/kolla-ansible/+/929771 | 13:02 |
| mnasiadka | I guess not | 13:02 |
| mnasiadka | #topic Release tasks | 13:03 |
| mnasiadka | It's R-2 - release in two weeks | 13:03 |
| mnasiadka | I'll try to post Highlights for Kolla, Kolla-Ansible and Kayobe so marketing team is happy | 13:03 |
| jovial | Is feature freeze this week? | 13:04 |
| mnasiadka | Yes, the docs say R-2 is Feature Freeze | 13:05 |
| mnasiadka | Although freeze exception just needs two core agree on reviewing a patch | 13:05 |
| mnasiadka | *two cores | 13:05 |
| jovial | Got you - I will start marking kayobe patches for the the release as RP+1 | 13:06 |
| mnasiadka | Since we have more or less 3 months to release - we need to agree if we're still going to push in Ubuntu 24.04 LTS support | 13:07 |
| mnasiadka | Since it's sort of in a good condition in Kolla and Kolla-Ansible - but I didn't start anything in Kayobe | 13:07 |
| mnasiadka | We don't have to do it this cycle, but I guess it would be nice to get it in | 13:08 |
| mnasiadka | Anybody any strong opinions? | 13:08 |
| SvenKieske | not for kayobe unfortunately :) | 13:09 |
| kevko | well, I've asked lastly about the k-a role patch vs kolla drop kolla-distro ...no feedback | 13:09 |
| jovial | Nice - anything still in flight for 24.04 in kolla/kolla-ansible or has everything landed? | 13:09 |
| mnasiadka | kevko: frickler seems to be a bit off, bbezak is on sick leave - let's wait another week? | 13:09 |
| mnasiadka | jovial: https://review.opendev.org/q/topic:%22kolla-ubuntu-noble%22 | 13:10 |
| mnasiadka | jovial: Jakub should be working on it, I assume we would need to at least see if it's trivial to get it in Kayobe or not | 13:10 |
| mnasiadka | So let's check and then decide next week | 13:11 |
| kevko | mnasiadka: okay | 13:11 |
| mnasiadka | We sort of already covered this release planning | 13:11 |
| mnasiadka | #topic Additional agenda (from whiteboard) | 13:11 |
| mnasiadka | jovial: Schedule data for kayobe PTG if not already discussed | 13:12 |
| jovial | date* | 13:12 |
| mnasiadka | jovial: As I mentioned last time, Kolla PTG slots will start from Tue 22nd (because I'm coming back from US on Monday) | 13:12 |
| mnasiadka | So I think we could schedule some Tue/Wed slots for Kolla and a Thu slot for Kayobe? | 13:13 |
| jovial | Yep, that works for me. Just wanted to get something in the diary | 13:13 |
| mnasiadka | 22nd October of course | 13:13 |
| mnasiadka | Ok, I'll hande reserving the bot slots and creating the Etherpad | 13:13 |
| mnasiadka | And send a mail to openstack-discuss | 13:13 |
| jovial | thanks | 13:14 |
| mnasiadka | jovial: Just to announce that I have proposed to add Matt Crees as a Kayobe core on the mailing list if no one has seen the message | 13:14 |
| mnasiadka | Yes, seen that, didn't have time to respond, but +1 from me | 13:14 |
| opendevreview | Piotr proposed openstack/kolla-ansible master: Gnocchi Configuration file - Deprecated: Option "enable_proxy_headers_parsing" https://review.opendev.org/c/openstack/kolla-ansible/+/929477 | 13:14 |
| jovial | Just raising awareness... feel free to reply to the email on openstack-discuss with any comments | 13:15 |
| mnasiadka | Thanks | 13:15 |
| mnasiadka | That's all for additional agenda | 13:15 |
| mnasiadka | #topic Open discussion | 13:15 |
| mnasiadka | Anybody, anything? | 13:15 |
| chembervint | hi, can we discuss my patch https://review.opendev.org/c/openstack/kolla-ansible/+/920377 | 13:16 |
| SvenKieske | I prepared some OIDC related small patches, if someone could look at those: https://review.opendev.org/c/openstack/kolla-ansible/+/928940 https://review.opendev.org/c/openstack/kolla-ansible/+/928949 | 13:17 |
| chembervint | It seams that the last question according 920377 is a documention - should we add something or not | 13:18 |
| mnasiadka | chembervint: sure - from my perspective the case is clear - either we're moving the octavia interface hacky feature written for CI to production status and we rework the docs for it to sound like that - or we're keeping that for Test/Dev like it is now - and then nobody should be doing big scale in Test/Dev :) | 13:18 |
| mmalchuk | please review and merge backports: https://review.opendev.org/q/(project:openstack/kolla+or+project:openstack/kolla-ansible)+(branch:unmaintained/zed+or+branch:unmaintained/yoga)+status:open | 13:20 |
| mmalchuk | yoga CI fixed | 13:20 |
| mnasiadka | SvenKieske, kevko any opinions on chembervint 's change? | 13:20 |
| chembervint | mnasiadka: our customers (our customers are operators;) ) using this feature in the prod environment with a scale up to 100computes. and actually, I think it should be prod ready feature, because not all DC are ready to spread L2 between racks | 13:20 |
| kevko | mnasiadka: just checking ... | 13:20 |
| SvenKieske | well LGTM, it's a minor detail if it's not working in all cases, we use this in prod already, documented or not :) | 13:21 |
| SvenKieske | sure would be nice to have more clear docs, as always, but not a deal breaker to me. | 13:21 |
| kevko | mnasiadka: my opinion is that octavia stuff in kolla-ansible - i mean that part with interface etc ...is not for production ... | 13:21 |
| SvenKieske | sorry, can't write that much, swamped with upgrade work | 13:22 |
| SvenKieske | so leave the docs as is, merge the fix don't advertise it for prod but we still use it in prod? fine with me :) | 13:22 |
| mnasiadka | That's also my problem - we're injecting a systemd unit for octavia-interface, which is if I remember correctly a bash script outside network manager or any other interface management subsystem | 13:22 |
| SvenKieske | but we do this already, this is not new, is it? | 13:23 |
| mnasiadka | this was meant for CI and development environments | 13:23 |
| kevko | on the other side, if everywhere it will be specified that it's not for production ..why not ... | 13:23 |
| mnasiadka | ok then, I can live with big scale under test/dev | 13:23 |
| mnasiadka | let me merge it | 13:23 |
| * frickler sneaks in and reads backlog | 13:23 | |
| mnasiadka | done | 13:24 |
| chembervint | as for me - it could be used also for production, if we understand how to tune it (exactly what I've added to the docs) | 13:25 |
| kevko | Then it's the operator's responsibility. The problem is that as soon as we start investing time into similar patches, people will adopt them, and that's a path to the hell... So once again, I don't have a problem with it, but the pitfalls of this solution must be clearly described everywhere... and that it's NOTexactly ideal to use in production. | 13:25 |
| SvenKieske | maybe frickler has to add something as I think he implemented the octavia interface stuff on our side :) | 13:26 |
| mnasiadka | From my perspective that octavia-interface thing is more geared towards Kayobe than Kolla-Ansible | 13:26 |
| mnasiadka | I mean it's easier to implement it there, maybe in NetworkManager and with static IPs | 13:26 |
| frickler | well it is a hacky solution, somehow abusing the neutron mechanism I think | 13:26 |
| mnasiadka | and then it might be something we could recommend in production | 13:26 |
| mnasiadka | But for now, it's hacky :) | 13:27 |
| chembervint | NM is a good idea, but not all linux distros support it in the same way ... ubuntu - netplan. centos - NM .... and so on | 13:27 |
| frickler | yes, switching to static configuration is on our todo list for sure | 13:27 |
| chembervint | yes, I agree, it's hacky. but as it has been discuses in the patch comments - probably we have to fix the current problem. and next think how to rework it in the master? | 13:28 |
| frickler | I also see that I had +2d the patch already, so seems to be fine | 13:29 |
| mnasiadka | Yes, I silenced my Asperger | 13:30 |
| kevko | is it already time to dicsuss patches then ? :D | 13:30 |
| mnasiadka | ok, unless frickler has anything else - I'd be closing the meeting :) | 13:30 |
| mnasiadka | ah | 13:30 |
| kevko | I have something on frickler :D | 13:30 |
| chembervint | thank you! | 13:31 |
| mnasiadka | frickler: did you have time to look into kevko 's hacky lib/python linking approach vs my service role? :) | 13:31 |
| chembervint | and one more short question - could you please push the button on the backports for merged patch? https://review.opendev.org/c/openstack/kolla-ansible/+/928077 | 13:31 |
| kevko | mnasiadka: i will still argue that's not hacky ... symlink to directory is normal in linux | 13:31 |
| kevko | example :) | 13:32 |
| kevko | (nova-compute)[root@compute0 /]# ls -la /lib | 13:32 |
| kevko | lrwxrwxrwx 1 root root 7 Apr 27 02:02 /lib -> usr/lib | 13:32 |
| kevko | var/lib/kolla/venv/lib64 -> Points to: /var/lib/kolla/venv/lib | 13:32 |
| kevko | var/spool/mail -> Points to: /var/mail | 13:32 |
| kevko | /etc/xdg/systemd/user -> Points to: /etc/systemd/user | 13:32 |
| kevko | etc etc etc | 13:32 |
| frickler | mnasiadka: kevko: yes, I looked at it, but didn't make up my mind yet. I think kevko's solution is easier and I tend to prefer that | 13:33 |
| kevko | /bin -> Points to: /usr/bin | 13:33 |
| mnasiadka | kevko: easy, not kilometer long monologue - we've been there already, everybody knows you want that merged | 13:33 |
| mnasiadka | let frickler and others make up their mind, it's not a race | 13:34 |
| kevko | but i didn't want to discuss this ... i wanted to just ask to another review for https://review.opendev.org/c/openstack/kolla/+/829295 https://review.opendev.org/c/openstack/kolla/+/915440 https://review.opendev.org/c/openstack/kolla/+/929516 | 13:34 |
| frickler | I place a vote after the meeting | 13:34 |
| frickler | mnasiadka: I also would like to see 24.04 support for 2024.2, do you think you could use help for that? | 13:35 |
| frickler | (though I'll also note that I'll be offline for some days starting tomorrow) | 13:35 |
| kevko | well, that symlink also helps me to have one patch for every distro as i need to patch for several customers and it's nice to has one header in patch file ... for this patch https://review.opendev.org/c/openstack/kolla/+/829295 | 13:36 |
| mnasiadka | geez, 211 files in one patch | 13:37 |
| kevko | mnasiadka: i squashed two commits into one ... | 13:37 |
| kevko | mnasiadka: but i can unsquash it | 13:37 |
| kevko | mnasiadka: all 208 files are same | 13:38 |
| mnasiadka | frickler: basically kolla/kolla-ansible parts are ready - there's a bug in python3-docker in Ubuntu that is getting fixed (so we'll use noble-proposed repo for some time) - I don't think I need any more help in that, and Kayobe is rather on SHPC side | 13:38 |
| mnasiadka | kevko: no need, just love going through 208 files to check if they are really the same | 13:38 |
| kevko | mnasiadka: don't click expand all :D | 13:38 |
| mnasiadka | anyway | 13:40 |
| mnasiadka | I guess we can wrap up | 13:40 |
| mnasiadka | Thanks for coming - see you next week! | 13:40 |
| mnasiadka | #endmeeting | 13:40 |
| opendevmeet | Meeting ended Wed Sep 18 13:40:53 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 13:40 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/kolla/2024/kolla.2024-09-18-13.00.html | 13:40 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/kolla/2024/kolla.2024-09-18-13.00.txt | 13:40 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/kolla/2024/kolla.2024-09-18-13.00.log.html | 13:40 |
| kevko | thx all | 13:41 |
| SvenKieske | ty | 13:45 |
| opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: CI: Bump ansible-core and ansible-lint https://review.opendev.org/c/openstack/ansible-collection-kolla/+/929756 | 13:49 |
| opendevreview | Merged openstack/kolla-ansible stable/2023.2: Add command to upgrade to a target version of RMQ https://review.opendev.org/c/openstack/kolla-ansible/+/926998 | 14:03 |
| opendevreview | Merged openstack/kolla-ansible master: Docs: point to RMQ SLURP handing in upgrade guide https://review.opendev.org/c/openstack/kolla-ansible/+/928907 | 14:03 |
| opendevreview | Merged openstack/kolla-ansible master: Be clear that rabbitmq-upgrade is not idempotent https://review.opendev.org/c/openstack/kolla-ansible/+/928906 | 14:03 |
| opendevreview | Merged openstack/kolla-ansible master: Fix typo in RabbitMQ versions pinning https://review.opendev.org/c/openstack/kolla-ansible/+/929768 | 14:03 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Switch mariadb's loadbalancer from HAProxy to ProxySQL https://review.opendev.org/c/openstack/kolla-ansible/+/913724 | 14:09 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [CI] Add mariadb backup into mariadb scenario https://review.opendev.org/c/openstack/kolla-ansible/+/929771 | 14:09 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [Trivial] Fix ironic when proxysql is enabled https://review.opendev.org/c/openstack/kolla-ansible/+/929781 | 14:09 |
| opendevreview | Merged openstack/kolla-ansible master: make OIDCClaimDelimiter configurable https://review.opendev.org/c/openstack/kolla-ansible/+/928940 | 14:33 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Switch to Dynamic Python Version Detection https://review.opendev.org/c/openstack/kolla/+/926744 | 14:38 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Add mechanism for patching files in containers https://review.opendev.org/c/openstack/kolla/+/829295 | 14:41 |
| opendevreview | Matt Crees proposed openstack/kolla-ansible stable/2024.1: Fix typo in RabbitMQ versions pinning https://review.opendev.org/c/openstack/kolla-ansible/+/929798 | 14:43 |
| opendevreview | Merged openstack/kolla-ansible master: Fix octavia-interface timeout https://review.opendev.org/c/openstack/kolla-ansible/+/920377 | 14:43 |
| opendevreview | Matt Crees proposed openstack/kolla-ansible stable/2023.2: Fix typo in RabbitMQ versions pinning https://review.opendev.org/c/openstack/kolla-ansible/+/929799 | 14:43 |
| opendevreview | Matt Crees proposed openstack/kolla-ansible stable/2023.1: Fix typo in RabbitMQ versions pinning https://review.opendev.org/c/openstack/kolla-ansible/+/929800 | 14:44 |
| opendevreview | Matt Crees proposed openstack/kolla-ansible stable/2024.1: Docs: point to RMQ SLURP handing in upgrade guide https://review.opendev.org/c/openstack/kolla-ansible/+/929803 | 14:44 |
| opendevreview | Matt Crees proposed openstack/kolla-ansible stable/2023.2: Docs: point to RMQ SLURP handing in upgrade guide https://review.opendev.org/c/openstack/kolla-ansible/+/929804 | 14:44 |
| opendevreview | Matt Crees proposed openstack/kolla-ansible stable/2023.1: Docs: point to RMQ SLURP handing in upgrade guide https://review.opendev.org/c/openstack/kolla-ansible/+/929805 | 14:44 |
| opendevreview | Matt Crees proposed openstack/kolla-ansible stable/2024.1: Be clear that rabbitmq-upgrade is not idempotent https://review.opendev.org/c/openstack/kolla-ansible/+/929806 | 14:45 |
| opendevreview | Matt Crees proposed openstack/kolla-ansible stable/2023.2: Be clear that rabbitmq-upgrade is not idempotent https://review.opendev.org/c/openstack/kolla-ansible/+/929807 | 14:45 |
| opendevreview | Matt Crees proposed openstack/kolla-ansible stable/2023.1: Be clear that rabbitmq-upgrade is not idempotent https://review.opendev.org/c/openstack/kolla-ansible/+/929808 | 14:45 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Standardize python version paths https://review.opendev.org/c/openstack/kolla-ansible/+/926745 | 14:56 |
| opendevreview | Merged openstack/kayobe master: ironic: revert to ironic's default drivers & interfaces https://review.opendev.org/c/openstack/kayobe/+/836999 | 15:08 |
| opendevreview | Martin Hiner proposed openstack/kolla-ansible master: Move to high level client in DockerWorker https://review.opendev.org/c/openstack/kolla-ansible/+/908295 | 15:10 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Standardize python version paths https://review.opendev.org/c/openstack/kolla-ansible/+/926745 | 15:13 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Standardize python version paths https://review.opendev.org/c/openstack/kolla-ansible/+/926745 | 15:16 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Standardize python version paths https://review.opendev.org/c/openstack/kolla-ansible/+/926745 | 15:19 |
| opendevreview | Piotr proposed openstack/kolla-ansible master: Move enable_proxy_headers_parsing option from [oslo_middleware] to [api] section https://review.opendev.org/c/openstack/kolla-ansible/+/929477 | 15:21 |
| opendevreview | Piotr proposed openstack/kolla-ansible master: Move enable_proxy_headers_parsing to [api] section https://review.opendev.org/c/openstack/kolla-ansible/+/929477 | 15:31 |
| opendevreview | Jens Harbott proposed openstack/kolla-ansible master: Standardize python version paths https://review.opendev.org/c/openstack/kolla-ansible/+/926745 | 16:36 |
| frickler | kevko: sorry I haven't been good enough at explaining what I meant in my comment. ^^ is what I was thinking, feel free to revert if you don't like it | 16:36 |
| SvenKieske | I'm outofoffice already, but that LGTM, I don't see python 4 on the horizon just yet ;) | 16:40 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Change from PERCONA_SCHEMA to mysql for mariabackup https://review.opendev.org/c/openstack/kolla-ansible/+/922729 | 16:47 |
| kevko | frickler: no problem ...but why i should set something to variable even if it is not needed anymore to change it dynamically in a template ? | 16:59 |
| kevko | frickler: i think the approach I've choose is for example for future ..for python 4 5 6 etc ...you just change group_vars and that's it...without sed everything match python3 ... | 17:00 |
| kevko | *is for example better for future | 17:00 |
| kevko | frickler: if you don't like ..i think it's even better to just drop variable in jinja and just rewrite the occurences to fixed string as it is not needed anymore to template it .... | 17:01 |
| kevko | frickler: what do you think | 17:01 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Fix mariadb backup unable to create history table https://review.opendev.org/c/openstack/kolla-ansible/+/929827 | 17:01 |
| kevko | frickler: also - your current approach is to somewhere leave variable set and somewhere change directly ...and from my perspective it looks like little bit chaotic | 17:03 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [Trivial] Fix ironic when proxysql is enabled https://review.opendev.org/c/openstack/kolla-ansible/+/929781 | 17:06 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Switch mariadb's loadbalancer from HAProxy to ProxySQL https://review.opendev.org/c/openstack/kolla-ansible/+/913724 | 17:06 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Change from PERCONA_SCHEMA to mysql for mariabackup https://review.opendev.org/c/openstack/kolla-ansible/+/922729 | 17:06 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Fix mariadb backup unable to create history table https://review.opendev.org/c/openstack/kolla-ansible/+/929827 | 17:06 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [CI] Add mariadb backup into mariadb scenario https://review.opendev.org/c/openstack/kolla-ansible/+/929771 | 17:06 |
| frickler | kevko: well my idea is just to avoid repeating long strings twice or more in a file by keeping the variable, some kind of deduplication. but as I said feel free to revert or amend as you like. I'll be offline now, will +2 when back either way | 17:16 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [CI] Add mariadb backup into mariadb scenario https://review.opendev.org/c/openstack/kolla-ansible/+/929771 | 17:17 |
| kevko | frickler: okay, thank you | 17:19 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Backup from active mariadb server https://review.opendev.org/c/openstack/kolla/+/928956 | 17:21 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [Trivial] Fix ironic when proxysql is enabled https://review.opendev.org/c/openstack/kolla-ansible/+/929781 | 17:25 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Switch mariadb's loadbalancer from HAProxy to ProxySQL https://review.opendev.org/c/openstack/kolla-ansible/+/913724 | 17:25 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Change from PERCONA_SCHEMA to mysql for mariabackup https://review.opendev.org/c/openstack/kolla-ansible/+/922729 | 17:25 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Fix mariadb backup unable to create history table https://review.opendev.org/c/openstack/kolla-ansible/+/929827 | 17:25 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [CI] Add mariadb backup into mariadb scenario https://review.opendev.org/c/openstack/kolla-ansible/+/929771 | 17:25 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Fix mariadb backup unable to create history table https://review.opendev.org/c/openstack/kolla-ansible/+/929827 | 17:42 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [CI] Add mariadb backup into mariadb scenario https://review.opendev.org/c/openstack/kolla-ansible/+/929771 | 17:42 |
| opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: CI: Bump ansible-core and ansible-lint https://review.opendev.org/c/openstack/ansible-collection-kolla/+/929756 | 17:49 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Standardize python version paths https://review.opendev.org/c/openstack/kolla-ansible/+/926745 | 19:36 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Switch mariadb's loadbalancer from HAProxy to ProxySQL https://review.opendev.org/c/openstack/kolla-ansible/+/913724 | 20:36 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Change from PERCONA_SCHEMA to mysql for mariabackup https://review.opendev.org/c/openstack/kolla-ansible/+/922729 | 20:36 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Fix mariadb backup unable to create history table https://review.opendev.org/c/openstack/kolla-ansible/+/929827 | 20:36 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [CI] Add mariadb backup into mariadb scenario https://review.opendev.org/c/openstack/kolla-ansible/+/929771 | 20:36 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Fix IPv6 address format in ProxySQL configuration https://review.opendev.org/c/openstack/kolla-ansible/+/929848 | 20:36 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Fix IPv6 address format in ProxySQL configuration https://review.opendev.org/c/openstack/kolla-ansible/+/929848 | 20:40 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Switch mariadb's loadbalancer from HAProxy to ProxySQL https://review.opendev.org/c/openstack/kolla-ansible/+/913724 | 20:40 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Change from PERCONA_SCHEMA to mysql for mariabackup https://review.opendev.org/c/openstack/kolla-ansible/+/922729 | 20:40 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Fix mariadb backup unable to create history table https://review.opendev.org/c/openstack/kolla-ansible/+/929827 | 20:40 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [CI] Add mariadb backup into mariadb scenario https://review.opendev.org/c/openstack/kolla-ansible/+/929771 | 20:40 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Fix IPv6 address format in ProxySQL configuration https://review.opendev.org/c/openstack/kolla-ansible/+/929848 | 22:52 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Switch mariadb's loadbalancer from HAProxy to ProxySQL https://review.opendev.org/c/openstack/kolla-ansible/+/913724 | 22:52 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Change from PERCONA_SCHEMA to mysql for mariabackup https://review.opendev.org/c/openstack/kolla-ansible/+/922729 | 22:52 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Fix mariadb backup unable to create history table https://review.opendev.org/c/openstack/kolla-ansible/+/929827 | 22:52 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [CI] Add mariadb backup into mariadb scenario https://review.opendev.org/c/openstack/kolla-ansible/+/929771 | 22:52 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Fix IPv6 address format in ProxySQL configuration https://review.opendev.org/c/openstack/kolla-ansible/+/929848 | 23:43 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Switch mariadb's loadbalancer from HAProxy to ProxySQL https://review.opendev.org/c/openstack/kolla-ansible/+/913724 | 23:43 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Change from PERCONA_SCHEMA to mysql for mariabackup https://review.opendev.org/c/openstack/kolla-ansible/+/922729 | 23:43 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Fix mariadb backup unable to create history table https://review.opendev.org/c/openstack/kolla-ansible/+/929827 | 23:43 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [CI] Add mariadb backup into mariadb scenario https://review.opendev.org/c/openstack/kolla-ansible/+/929771 | 23:43 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!