| opendevreview | Merged openstack/kolla-ansible stable/2024.1: [CI] Fix OVN jobs - add required neutron_dns_domain https://review.opendev.org/c/openstack/kolla-ansible/+/931617 | 07:43 |
|---|---|---|
| opendevreview | Merged openstack/kolla-ansible stable/2023.1: CI: improve get_logs.sh https://review.opendev.org/c/openstack/kolla-ansible/+/927091 | 07:43 |
| opendevreview | Merged openstack/kolla-ansible stable/2024.1: fact gathering: Remove gathering of package facts https://review.opendev.org/c/openstack/kolla-ansible/+/926746 | 07:43 |
| opendevreview | Merged openstack/kolla-ansible stable/2024.1: fact gathering: Group hosts before gathering facts https://review.opendev.org/c/openstack/kolla-ansible/+/926751 | 07:46 |
| opendevreview | Merged openstack/ansible-collection-kolla stable/2023.2: Add installation of dbus package https://review.opendev.org/c/openstack/ansible-collection-kolla/+/930485 | 07:46 |
| opendevreview | Merged openstack/kolla-ansible stable/2023.2: Fixing typo in etc/kolla/globals.yml https://review.opendev.org/c/openstack/kolla-ansible/+/930467 | 07:46 |
| opendevreview | Merged openstack/kolla-ansible stable/2023.1: Fixing typo in etc/kolla/globals.yml https://review.opendev.org/c/openstack/kolla-ansible/+/930466 | 07:46 |
| opendevreview | Merged openstack/kolla-ansible stable/2024.1: Fixing typo in etc/kolla/globals.yml https://review.opendev.org/c/openstack/kolla-ansible/+/930468 | 07:46 |
| opendevreview | Merged openstack/kolla-ansible stable/2023.2: Fix typo in RabbitMQ versions pinning https://review.opendev.org/c/openstack/kolla-ansible/+/929799 | 07:46 |
| opendevreview | Merged openstack/kolla-ansible stable/2024.1: Fix typo in RabbitMQ versions pinning https://review.opendev.org/c/openstack/kolla-ansible/+/929798 | 07:46 |
| opendevreview | Merged openstack/kolla-ansible stable/2023.1: Fix typo in RabbitMQ versions pinning https://review.opendev.org/c/openstack/kolla-ansible/+/929800 | 07:46 |
| opendevreview | Merged openstack/kolla-ansible stable/2024.1: Docs: point to RMQ SLURP handing in upgrade guide https://review.opendev.org/c/openstack/kolla-ansible/+/929803 | 07:47 |
| opendevreview | Merged openstack/kolla-ansible stable/2023.1: Docs: point to RMQ SLURP handing in upgrade guide https://review.opendev.org/c/openstack/kolla-ansible/+/929805 | 07:47 |
| opendevreview | Merged openstack/kolla-ansible stable/2023.2: Docs: point to RMQ SLURP handing in upgrade guide https://review.opendev.org/c/openstack/kolla-ansible/+/929804 | 07:47 |
| opendevreview | Merged openstack/kolla-ansible stable/2023.2: Add NET_RAW to dnsmasq container https://review.opendev.org/c/openstack/kolla-ansible/+/929470 | 07:47 |
| opendevreview | Merged openstack/kolla-ansible stable/2024.1: Add NET_RAW to dnsmasq container https://review.opendev.org/c/openstack/kolla-ansible/+/929469 | 07:47 |
| opendevreview | Merged openstack/kolla-ansible stable/2023.1: Revert "Allow setting any_errors_fatal true for gather-facts" https://review.opendev.org/c/openstack/kolla-ansible/+/928595 | 07:47 |
| opendevreview | Merged openstack/kolla-ansible stable/2024.1: Revert "Allow setting any_errors_fatal true for gather-facts" https://review.opendev.org/c/openstack/kolla-ansible/+/928593 | 07:49 |
| opendevreview | Merged openstack/kolla-ansible stable/2023.2: Revert "Allow setting any_errors_fatal true for gather-facts" https://review.opendev.org/c/openstack/kolla-ansible/+/928594 | 07:54 |
| opendevreview | Merged openstack/ansible-collection-kolla master: Drop OpenEuler support https://review.opendev.org/c/openstack/ansible-collection-kolla/+/930475 | 08:17 |
| opendevreview | Matt Crees proposed openstack/kayobe stable/2024.1: Adds support for using Cumulus switches with NGS https://review.opendev.org/c/openstack/kayobe/+/931883 | 08:23 |
| opendevreview | Matt Crees proposed openstack/kayobe stable/2023.2: Adds support for using Cumulus switches with NGS https://review.opendev.org/c/openstack/kayobe/+/931884 | 08:24 |
| opendevreview | Matt Crees proposed openstack/kayobe stable/2023.1: Adds support for using Cumulus switches with NGS https://review.opendev.org/c/openstack/kayobe/+/931885 | 08:24 |
| opendevreview | Matt Crees proposed openstack/kayobe stable/2023.1: Adds support for using Cumulus switches with NGS https://review.opendev.org/c/openstack/kayobe/+/931885 | 08:32 |
| opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: Rename True/False to true/false in preparation for ansible-lint bump https://review.opendev.org/c/openstack/ansible-collection-kolla/+/929756 | 08:55 |
| opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: Rename True/False to true/false in preparation for ansible-lint bump https://review.opendev.org/c/openstack/ansible-collection-kolla/+/929756 | 08:55 |
| kevko | mnasiadka: around ? | 08:56 |
| opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: Adapt code in preparation for ansible-lint bump https://review.opendev.org/c/openstack/ansible-collection-kolla/+/929756 | 08:56 |
| mnasiadka | kevko: yup | 09:00 |
| kevko | mnasiadka: i switched xbstream to mbstream and proxysql is not affected anymore :D | 09:00 |
| mnasiadka | lol | 09:00 |
| kevko | mnasiadka: well ...it's because xbstream has bad check of version ... | 09:00 |
| kevko | mnasiadka: xbstream is checking what is reported by client - the binary version ...even if select @@version is reporting Mariadb ...that's the reason why it's failing ... | 09:01 |
| kevko | mnasiadka: so what now ? :D | 09:01 |
| kevko | mnasiadka: because i think we should use mbstream if we are also using mariabackup tool | 09:02 |
| mnasiadka | well, let's change to mbstream - question as always how it affects existing users - as in existing backups restoration and so on | 09:03 |
| kevko | mnasiadka: because xtrabackup was for percona xtrabackup ...mbstream is recommended | 09:04 |
| kevko | mnasiadka: well I can also add bash parseargs which will accept stream option ...and if not specified ..it will be same as now ... | 09:06 |
| kevko | mnasiadka: but mbstream is backward compatible of course ......it's drop-in replacement | 09:07 |
| mnasiadka | well, if it works for restoring old backups made with xbstream (compression or whatever it's used for) I'm fine with changing that and backporting even | 09:08 |
| kevko | mnasiadka: hmm, wait ..maybe i am mistificating :D ...still old script in my image ... | 09:10 |
| kevko | mnasiadka: let me check again and better :D | 09:10 |
| opendevreview | Merged openstack/kolla-ansible stable/2024.1: Add a precheck to catch RMQ SLURP upgrades https://review.opendev.org/c/openstack/kolla-ansible/+/918977 | 09:10 |
| kevko | mnasiadka: nah, it's the same ...okay ..let me solve as we discussed in review | 09:14 |
| 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 | 09:25 |
| opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: Adapt code in preparation for ansible-lint bump https://review.opendev.org/c/openstack/ansible-collection-kolla/+/931891 | 09:27 |
| 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 | 09:27 |
| opendevreview | Merged openstack/kolla-ansible stable/2024.1: Support mode in named volumes when using podman https://review.opendev.org/c/openstack/kolla-ansible/+/922953 | 09:38 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Add way to backup from MariaDB replica server https://review.opendev.org/c/openstack/kolla/+/928956 | 09:53 |
| 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 | 09:57 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Add backend TLS between MariaDB and ProxySQL https://review.opendev.org/c/openstack/kolla-ansible/+/909912 | 09:57 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Add frontend TLS ability to ProxySQL https://review.opendev.org/c/openstack/kolla-ansible/+/925500 | 09:57 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Add frontend database TLS for Keystone https://review.opendev.org/c/openstack/kolla-ansible/+/925507 | 09:57 |
| 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 | 09:57 |
| 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 | 09:57 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [CI] Fix kolla-ansible-rocky9-kvm https://review.opendev.org/c/openstack/kolla-ansible/+/930975 | 09:57 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [CI] Add mariadb backup into mariadb scenario https://review.opendev.org/c/openstack/kolla-ansible/+/929771 | 09:57 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Update MySQL monitor user privileges for ProxySQL monitoring https://review.opendev.org/c/openstack/kolla-ansible/+/930979 | 09:57 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [DNM] [CI] Just let's test backup few times https://review.opendev.org/c/openstack/kolla-ansible/+/931144 | 09:57 |
| kevko | mnasiadka: what now ? better ? i didn't touch script | 09:57 |
| opendevreview | Rafal Lewandowski proposed openstack/kolla-ansible master: [WIP] Enable ML2/OVN and distributed FIP by default. https://review.opendev.org/c/openstack/kolla-ansible/+/904959 | 10:37 |
| opendevreview | Roman Krček proposed openstack/kolla-ansible master: Move actions to kolla_container_facts https://review.opendev.org/c/openstack/kolla-ansible/+/911505 | 11:21 |
| opendevreview | Roman Krček proposed openstack/kolla-ansible master: Move actions to kolla_container_facts https://review.opendev.org/c/openstack/kolla-ansible/+/911505 | 11:45 |
| opendevreview | Merged openstack/ansible-collection-kolla master: Adapt code in preparation for ansible-lint bump https://review.opendev.org/c/openstack/ansible-collection-kolla/+/931891 | 12:17 |
| opendevreview | Merged openstack/ansible-collection-kolla master: CI: Bump ansible-core and ansible-lint https://review.opendev.org/c/openstack/ansible-collection-kolla/+/929756 | 12:17 |
| mnasiadka | mnasiadka bbezak frickler kevko SvenKieske mmalchuk gkoper jangutter jsuazo jovial osmanlicilegi mattcrees dougszu darmach - meeting in 10 minutes | 12:50 |
| opendevreview | Roman Krček proposed openstack/kolla-ansible master: Rewrite kolla-ansible CLI to python https://review.opendev.org/c/openstack/kolla-ansible/+/923110 | 12:51 |
| opendevreview | Roman Krček proposed openstack/kolla-ansible master: Fix unintentional trigger of ansible handlers https://review.opendev.org/c/openstack/kolla-ansible/+/924145 | 12:55 |
| 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 | 12:55 |
| mnasiadka | #startmeeting kolla | 13:01 |
| opendevmeet | Meeting started Wed Oct 9 13:01:08 2024 UTC and is due to finish in 60 minutes. The chair is mnasiadka. Information about MeetBot at http://wiki.debian.org/MeetBot. | 13:01 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 13:01 |
| opendevmeet | The meeting name has been set to 'kolla' | 13:01 |
| mnasiadka | #topic rollcall | 13:01 |
| mnasiadka | o/ | 13:01 |
| SvenKieske | o/ | 13:01 |
| r-krcek | \o/ | 13:01 |
| mmalchuk | o/ | 13:01 |
| bbezak | o/ | 13:02 |
| mnasiadka | #topic agenda | 13:02 |
| mnasiadka | * CI status | 13:02 |
| mnasiadka | * Release tasks | 13:02 |
| mnasiadka | * Current cycle planning | 13:02 |
| mnasiadka | * Additional agenda (from whiteboard) | 13:02 |
| mnasiadka | * Open discussion | 13:03 |
| mnasiadka | #topic CI status | 13:03 |
| mnasiadka | Overall green, apart the Ansible locale on CentOS 9 - but should be fixed now with new DIB release | 13:03 |
| mnasiadka | #topic Release tasks | 13:03 |
| mnasiadka | So, we're two weeks into cycle trailing if I can count | 13:03 |
| mnasiadka | Areas of focus are on whiteboard line 226 | 13:04 |
| mnasiadka | Noble Kolla-Ansible support is nearly done - need to fight cephadm and Ubuntu 24.04 oddities | 13:04 |
| SvenKieske | #link https://etherpad.opendev.org/p/KollaWhiteBoard#L226 | 13:04 |
| mnasiadka | Ubuntu 24.04 ships with an apparmor policy that has a space in the name | 13:05 |
| mnasiadka | and cephadm doesn't support it yet :) | 13:05 |
| mnasiadka | https://review.opendev.org/c/openstack/kolla-ansible/+/869252 | 13:05 |
| SvenKieske | the upstream patch is at least merged, it seems? :) | 13:05 |
| mnasiadka | That's the awful Huawei stuff, but we agreed to merge it | 13:05 |
| mnasiadka | SvenKieske: in master, not in reef, and before it gets released it will be Christmas period | 13:05 |
| mnasiadka | I've added my +2 to Huawei patch, if any other core can review and merge - would be nice | 13:06 |
| mnasiadka | https://review.opendev.org/q/topic:%22mariadb-proxysql%22 - I'm progressing through it slowly | 13:07 |
| mnasiadka | And would be nice to get a second assignee | 13:08 |
| mnasiadka | Basically I can't review everything so bbezak SvenKieske and frickler if you could assign yourself to any reviews in the whiteboard it would be nice | 13:09 |
| mnasiadka | I'm aware we're not going to merge everything this cycle, but let's try | 13:09 |
| mnasiadka | And apart the kevko's big chains - we have the kolla-ansible python CLI https://review.opendev.org/c/openstack/kolla-ansible/+/923110 | 13:09 |
| SvenKieske | I'll look into it in the evening, I already did some reviews of previous iterations. In general it seems I don't have the time anymore to almost dedicate my complete working hours towards this, unfortunately, so I'll shift stuff in my private time, where possible. | 13:09 |
| mnasiadka | SvenKieske: I've seen you have some reservations for bandit stuff | 13:10 |
| mnasiadka | I haven't had a deep dive yet, but will do this week | 13:10 |
| SvenKieske | well the bandit stuff have some meaning, it's a security check, maybe we can just omit the function, as it's afaik only called once, so maybe no need for an abstraction layer? that's what I remember from memory. | 13:10 |
| kevko | \o | 13:10 |
| mnasiadka | I'll have a look and try to drive to completion this cycle. | 13:11 |
| SvenKieske | I'm still slightly -1 on just silencing the security check, if we don't find a proper solution we can still do it, of course. | 13:11 |
| mnasiadka | Yeah, understood | 13:11 |
| mnasiadka | Additional agenda is also only about Kevko nicely asking for reviews | 13:12 |
| mnasiadka | so I'll go to Open discussion directly - so we can all use remaining time for reviews | 13:12 |
| mnasiadka | #topic Open discussion | 13:12 |
| mnasiadka | Anybody anything? | 13:12 |
| kevko | yeah, please | 13:12 |
| mmalchuk | backport of the https://review.opendev.org/c/openstack/kolla-ansible/+/930170 | 13:12 |
| mmalchuk | its trivial solves issues early on inspection of baremetal | 13:13 |
| mmalchuk | needed in yoga asap | 13:13 |
| mmalchuk | and zed | 13:13 |
| mmalchuk | thanks | 13:13 |
| mmalchuk | is it possible? any objections? | 13:14 |
| SvenKieske | can you file a bug that describes what issue it solves? we can then link that and do the proper backports imho? | 13:14 |
| mnasiadka | I'm fine with that, should be helpful for users | 13:15 |
| mmalchuk | I was so busy to create bugreport to attach it | 13:15 |
| SvenKieske | :D | 13:15 |
| mnasiadka | It's not really a bug, it's a feature - but it improves operator's lives | 13:15 |
| mmalchuk | another one core could be fine and I'll prepare backports | 13:15 |
| mnasiadka | So as long as it doesn't bring any inconvenience to users - we could do the backports? | 13:15 |
| mnasiadka | Ok, do the backports mmalchuk - I don't see any objections | 13:17 |
| mmalchuk | ok. thanks | 13:17 |
| mmalchuk | will do today | 13:17 |
| mnasiadka | Anything else? | 13:17 |
| mmalchuk | https://review.opendev.org/c/openstack/kolla-ansible/+/931200 | 13:17 |
| mmalchuk | trivial backport | 13:17 |
| mmalchuk | cider part merged | 13:17 |
| mmalchuk | cinder* | 13:17 |
| mmalchuk | so please merge it | 13:17 |
| mnasiadka | bbezak: can you? | 13:18 |
| bbezak | done | 13:18 |
| mnasiadka | yay | 13:18 |
| mnasiadka | I guess that's it | 13:19 |
| jhorstmann | I am lookiung for some feedback on https://review.opendev.org/c/openstack/kolla-ansible/+/921097 . | 13:19 |
| mnasiadka | will wait another couple of minutes and close the meeting | 13:19 |
| mmalchuk | thats all from my side | 13:19 |
| mmalchuk | thanks | 13:19 |
| mnasiadka | Can anyone with Grafana knowledge have a look in jhorstmann 's patch? | 13:19 |
| mnasiadka | I'm monitoring ignorant. | 13:20 |
| mnasiadka | Ah, I have one - https://review.opendev.org/c/openstack/kolla-ansible/+/931738 | 13:20 |
| mnasiadka | Tested in https://review.opendev.org/c/openstack/kolla-ansible/+/931776 | 13:21 |
| SvenKieske | well jhorstmann's change is more about avoiding container restarts by not always copying all files, should be possible to review without extensive grafana knowledge ;) | 13:21 |
| SvenKieske | the ovn stuff is on my todo list as well | 13:22 |
| mnasiadka | I have some other things on my radar, but let me add it to my list | 13:22 |
| mnasiadka | Ok, enough for today. | 13:23 |
| mmalchuk | SvenKieske custom dashboards | 13:23 |
| mnasiadka | Thanks for coming! | 13:23 |
| mnasiadka | And please review those patches - so we can release sooner than on christmas | 13:23 |
| mmalchuk | SvenKieske need test to see restarts | 13:23 |
| SvenKieske | anybody want's to pay for reviews during non payed time by employers? (only half joking) :D | 13:23 |
| kevko | :D :'( | 13:23 |
| SvenKieske | thanks for running the meeting mnasiadka | 13:24 |
| mmalchuk | SvenKieske isn't this an opensource? feel open to review) | 13:24 |
| mmalchuk | mnasiadka thanks | 13:24 |
| mnasiadka | #endmeeting | 13:25 |
| opendevmeet | Meeting ended Wed Oct 9 13:25:03 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 13:25 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/kolla/2024/kolla.2024-10-09-13.01.html | 13:25 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/kolla/2024/kolla.2024-10-09-13.01.txt | 13:25 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/kolla/2024/kolla.2024-10-09-13.01.log.html | 13:25 |
| mnasiadka | #endmeeting | 13:25 |
| mnasiadka | what the heck | 13:25 |
| mmalchuk | ? | 13:25 |
| kevko | What is hard to review one line ? here for example ? https://review.opendev.org/c/openstack/kolla-ansible/+/930979 | 13:26 |
| kevko | ONE line | 13:27 |
| mmalchuk | no so hard, but it chained, so need to review other changes in the chain) | 13:29 |
| SvenKieske | two lines, and I see verified -1 ? | 13:29 |
| SvenKieske | ;) | 13:29 |
| kevko | SvenKieske: yes ..because of CI ? retry limit ? | 13:29 |
| kevko | SvenKieske: two same lines ... | 13:30 |
| kevko | SvenKieske: for you I've also prepared a python test https://github.com/keuko/sven-kolla/blob/master/test.py | 13:30 |
| SvenKieske | okay, well when I have limited time I don't check every CI result (if it's -1), only if I want to debug why it's failing :) | 13:30 |
| kevko | i don't know what else to do ... feeling frustrated more and more | 13:30 |
| SvenKieske | yes I know, it's really appreciated, I just need to find the time to review :-( | 13:31 |
| SvenKieske | so you really did nothing wrong there, quite the opposite | 13:31 |
| kevko | well, my oldest patch here is from january 2024 | 13:33 |
| kevko | Very important patches that, for example, fix configs injected into the container with unit tests and with docstrings requested from you, have been lying there since April | 13:34 |
| kevko | The only thing left for me to do is just spam repeatedly again and again | 13:35 |
| SvenKieske | I understand, but please, I hope you also understand that I can't keep up the review counts as I did in the last year when I can't put in the same hours, as these are not gonna get payed. Of course I do stuff in my free time, but certainly not 8 hours a day. THanks. | 13:41 |
| kevko | By the way, since I'm constantly dealing with MariaDB in CI, I've noticed that reconfiguring or upgrading MariaDB often fails—not because of my patches, but due to WSREP synchronization issues. Shouldn't we increase the timeout in CI to give mariadbs enough time to sync up? | 13:45 |
| kevko | because then I am ending with several rechecks per day only because of this for example | 13:46 |
| kevko | but also other reviews | 13:48 |
| SvenKieske | sounds like a good idea, is it really just timeouts? I got a wsrep sync issue in a deployment currently, but seems more related to network issues. | 13:55 |
| kevko | SvenKieske: whatever it is ...longer wait can fix the issue no ? | 14:01 |
| kevko | https://9e1d865d639286d1a07a-b0014becc5640974ea321328f0a307e3.ssl.cf5.rackcdn.com/929827/21/check/kolla-ansible-rocky9-mariadb/79f6558/primary/logs/ansible/reconfigure | 14:01 |
| kevko | but yeah, this looks like network issue or something | 14:02 |
| mnasiadka | kevko: got a minute for https://review.opendev.org/c/openstack/kolla-ansible/+/931738 ? Would be nice to get that merged this week | 14:09 |
| kevko | mnasiadka: done | 14:17 |
| kevko | mnasiadka: In return, you could take another look at that replica backup review, as I restored the script there, and I didn’t even touch it as you wanted | 14:22 |
| mnasiadka | Done - you need to bug another core now ;-) | 14:23 |
| kevko | mnasiadka: Thank God in heaven | 14:27 |
| opendevreview | Maksim Malchuk proposed openstack/kolla-ansible stable/2024.1: Add configuration for NTP server in Ironic Inspector DHCP server https://review.opendev.org/c/openstack/kolla-ansible/+/931955 | 14:32 |
| opendevreview | Maksim Malchuk proposed openstack/kolla-ansible stable/2023.2: Add configuration for NTP server in Ironic Inspector DHCP server https://review.opendev.org/c/openstack/kolla-ansible/+/931956 | 14:33 |
| opendevreview | Maksim Malchuk proposed openstack/kolla-ansible stable/2023.1: Add configuration for NTP server in Ironic Inspector DHCP server https://review.opendev.org/c/openstack/kolla-ansible/+/931957 | 14:33 |
| opendevreview | Merged openstack/kolla-ansible master: Add support to configure Huawei backends in Cinder https://review.opendev.org/c/openstack/kolla-ansible/+/869252 | 14: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 | 15:19 |
| 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 | 15:19 |
| 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 | 15:19 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [CI] Add mariadb backup into mariadb scenario https://review.opendev.org/c/openstack/kolla-ansible/+/929771 | 15:19 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Update MySQL monitor user privileges for ProxySQL monitoring https://review.opendev.org/c/openstack/kolla-ansible/+/930979 | 15:19 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [DNM] [CI] Just let's test backup few times https://review.opendev.org/c/openstack/kolla-ansible/+/931144 | 15:19 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Add backend TLS between MariaDB and ProxySQL https://review.opendev.org/c/openstack/kolla-ansible/+/909912 | 15:19 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Add frontend TLS ability to ProxySQL https://review.opendev.org/c/openstack/kolla-ansible/+/925500 | 15:19 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Add frontend database TLS for Keystone https://review.opendev.org/c/openstack/kolla-ansible/+/925507 | 15:19 |
| kevko | SvenKieske: it will be very cool if you will find the time for this simple script here https://review.opendev.org/c/openstack/kolla/+/928956 ...it's requirement for everything ... mnasiadka already gave +2 | 15:21 |
| opendevreview | Merged openstack/kolla-ansible master: ovn: Fix FQDN-based deployments https://review.opendev.org/c/openstack/kolla-ansible/+/931738 | 15:34 |
| 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 | 16:39 |
| opendevreview | Rafal Lewandowski proposed openstack/kolla-ansible master: [WIP] Enable ML2/OVN and distributed FIP by default. https://review.opendev.org/c/openstack/kolla-ansible/+/904959 | 17:22 |
| opendevreview | Merged openstack/kolla-ansible stable/2023.1: fact gathering: Remove gathering of package facts https://review.opendev.org/c/openstack/kolla-ansible/+/926750 | 18:03 |
| opendevreview | Merged openstack/kolla-ansible stable/2023.1: fact gathering: Group hosts before gathering facts https://review.opendev.org/c/openstack/kolla-ansible/+/926753 | 18:15 |
| SvenK | kevko: currently looking at https://review.opendev.org/c/openstack/kolla-ansible/+/922729 | 18:46 |
| SvenK | anything more urgent? most of todays list from the meeting is already merged or has my +2 | 18:46 |
| kevko | SvenK: well this ...w8 a minute | 19:31 |
| kevko | SvenK: https://review.opendev.org/c/openstack/kolla/+/928956 | 19:32 |
| kevko | SvenK: https://review.opendev.org/c/openstack/kolla-ansible/+/913724 | 19:33 |
| SvenK | ah ty, will have a look | 19:36 |
| SvenK | just reviewed the huge set config stuff, finally | 19:36 |
| kevko | SvenK: very thanks ! | 19:43 |
| kevko | SvenK: it's really simple function ... | 19:43 |
| kevko | SvenK: also covered by unit tests | 19:43 |
| SvenK | yeah well, the tests are the long part :D | 19:47 |
| SvenK | I guess we should do backports for https://review.opendev.org/c/openstack/kolla/+/928956 then? | 19:47 |
| kevko | SvenK: well we can, but you know ...it's individual always what to backport and what not ... | 19:53 |
| SvenK | yep | 19:54 |
| SvenK | mhm | 19:54 |
| SvenK | kevko: you have seen this error? this is the second time today I see it, it made your proxysql change fail: https://zuul.opendev.org/t/openstack/build/d468a5aa61f949cbadfec9cafd3f0693/log/primary/logs/kolla/all-CRITICAL.txt | 19:54 |
| SvenK | 2024-10-09 16:26:37.432 1038 CRITICAL neutron [None req-e59c7358-0f0b-446f-8f68-fbde0c1f6a15 - - - - - -] Unhandled error: sqlalchemy.exc.OperationalError: (pymysql.err.OperationalError) (9001, 'Max connect timeout reached while reaching hostgroup 0 after 10503ms') | 19:55 |
| SvenK | this seems suspicious | 19:55 |
| SvenK | but I guess unrelated | 19:57 |
| kevko | SvenK: this is when proxysql can't set active host ...if it's not pinggable or if wsrep status is somehow broken ...if it is not ok in 10sec ...it's reporting this ..because cant reach healthy server | 20:03 |
| kevko | SvenK: but it's expected ...as you don't want to write to unhealthy cluster | 20:03 |
| kevko | SvenK: and CI has sometimes just slow disk or network issue | 20:03 |
| kevko | SvenK: same symptom as when its failing during reconfigure | 20:06 |
| SvenK | mhm pretty sure this was single host deployment? but whatever..see you tomorrow :) | 21:11 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!