| opendevreview | OpenStack Proposal Bot proposed openstack/kolla master: Updated ansible-lint in lint-requirements.txt https://review.opendev.org/c/openstack/kolla/+/995628 | 02:15 |
|---|---|---|
| opendevreview | OpenStack Proposal Bot proposed openstack/kolla-ansible master: Updated ansible-lint j2lint in lint-requirements.txt https://review.opendev.org/c/openstack/kolla-ansible/+/995629 | 02:39 |
| frickler | nice the bot worked fine. but then "*** Invalid or missing pyproject.toml!" | 06:40 |
| kevko | Morning | 08:10 |
| kevko | anybody for review - all comments resolved .. https://review.opendev.org/c/openstack/kolla/+/988849 and https://review.opendev.org/c/openstack/kolla/+/989936 | 08:10 |
| blanson[m] | kevko: +2'd both seems like very sensible changes | 09:00 |
| mikal | I was hoping to enter into a commercial arrangement, perhaps with frickler or blanson[m] where I paid them in a graph which shows we're no longer seeing libvrirtd rate limiting on master CI in return for some quick reviews on some security fixes that have been in backport statis for a little long. | 09:05 |
| mikal | Specifically, https://madebymikal.com/shee0aedieFeico7Fiey/libvirt_chart2.png I think shows a real improvement, and hopefully will continue. | 09:06 |
| mikal | And then https://review.opendev.org/c/openstack/kolla-ansible/+/994603 and https://review.opendev.org/c/openstack/kolla-ansible/+/994604 are security backports I'd love to see merged and I don't think are controversial. | 09:06 |
| mikal | Honestly https://review.opendev.org/c/openstack/kolla/+/986283 is perhaps a tiny bit more controversial but not very, and then would unlock the one remaining fix at https://review.opendev.org/c/openstack/kolla/+/993067 | 09:07 |
| *** jhorstmann is now known as Guest12403 | 09:10 | |
| blanson[m] | mikal: wait the only thing I'm getting out of that deal is a graph ? I feel ripped out | 09:31 |
| blanson[m] | (will check your patches ;) | 09:31 |
| mikal | Ummm, two graphs? | 09:32 |
| mikal | A blog post! | 09:32 |
| blanson[m] | The kolla patch for virtlogd has been backported already ? | 09:33 |
| blanson[m] | if yes I think these are no brainers indeed | 09:33 |
| blanson[m] | I mean even without, this shouldn't break anything but it's a bit pointless | 09:33 |
| blanson[m] | yup they are on all stable | 09:34 |
| blanson[m] | ok then it's +2 | 09:34 |
| mikal | Yeah sorry, was distracted by a shiny object. They should all be pure backports, with the exception that the precheck landed initially on 2026.1 not master because mnasiadka wasn't concerned about master mismatching its built images. | 09:40 |
| opendevreview | Jed Preist proposed openstack/kayobe master: Fix kolla inventory hostname formatting https://review.opendev.org/c/openstack/kayobe/+/995656 | 09:40 |
| opendevreview | Jed Preist proposed openstack/kayobe master: Fix kolla inventory hostname formatting https://review.opendev.org/c/openstack/kayobe/+/995656 | 09:46 |
| mikal | blanson[m]: sorry to be a horrendous jerk, but any chance of a review on https://review.opendev.org/c/openstack/kolla/+/986283 as well so that the virtlogd backport sitting behind it can merge too? | 09:49 |
| blanson[m] | :) | 09:50 |
| mikal | Three graphs? | 09:50 |
| blanson[m] | deal | 09:50 |
| mikal | Amazing. | 09:51 |
| mikal | You drive a hard bargain. | 09:51 |
| blanson[m] | I'll give it a look in the afternoon, right now I'm too hungry to be trusted with +2 power | 09:51 |
| mikal | Three graphs and a mars bar! | 09:51 |
| mikal | But that's fine. I'm just keen to close out the security bug in launchpad. | 09:51 |
| tafkamax | Theres a snickers commercial "you are not yourself, eat a snickers" that references an hungry stomach | 09:52 |
| mikal | I think the term "hangry" came form an advertising campaign as well? | 09:57 |
| opendevreview | Jed Preist proposed openstack/kayobe master: Fix kolla inventory hostname formatting https://review.opendev.org/c/openstack/kayobe/+/995656 | 09:57 |
| parallax | I think the two of the core reviewers are currently vacationing :) | 10:19 |
| frickler | mikal: I'm not really into reviewing rocky related changes, I fear you'll have to wait for the vacationers for that one | 10:55 |
| kevko | blanson[m]: thanks | 11:18 |
| kevko | frickler: maybe you can set second +2 and +w please ? https://review.opendev.org/c/openstack/kolla/+/988849 and https://review.opendev.org/c/openstack/kolla/+/989936 | 11:23 |
| opendevreview | Maksim Malchuk proposed openstack/kayobe stable/2025.2: Add support for url sources in kolla_sources https://review.opendev.org/c/openstack/kayobe/+/995678 | 13:19 |
| opendevreview | Maksim Malchuk proposed openstack/kayobe stable/2025.1: Add support for url sources in kolla_sources https://review.opendev.org/c/openstack/kayobe/+/995679 | 13:20 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible stable/2026.1: UnicodeDecodeError in PodmanWorker.compare_config https://review.opendev.org/c/openstack/kolla-ansible/+/995693 | 14:38 |
| opendevreview | Merged openstack/kayobe master: Fix kolla inventory hostname formatting https://review.opendev.org/c/openstack/kayobe/+/995656 | 14:56 |
| opendevreview | Jed Preist proposed openstack/kayobe stable/2026.1: Fix kolla inventory hostname formatting https://review.opendev.org/c/openstack/kayobe/+/995699 | 14:57 |
| opendevreview | Merged openstack/kolla stable/2025.1: [2025.2 and older] Create Ironic temp directory https://review.opendev.org/c/openstack/kolla/+/995011 | 15:07 |
| opendevreview | Merged openstack/kolla-ansible stable/2025.1: Set Ironic tempdir https://review.opendev.org/c/openstack/kolla-ansible/+/995012 | 15:07 |
| opendevreview | Thomas Bechtold proposed openstack/kolla-ansible master: Load iscsi_tcp on all iscsid hosts, not just Ironic https://review.opendev.org/c/openstack/kolla-ansible/+/995714 | 15:44 |
| opendevreview | Doug Szumski proposed openstack/kolla master: fluentd: add remote_syslog output plugin support https://review.opendev.org/c/openstack/kolla/+/984472 | 16:03 |
| opendevreview | Thomas Bechtold proposed openstack/kolla-ansible master: Load iscsi_tcp on all iscsid hosts https://review.opendev.org/c/openstack/kolla-ansible/+/995714 | 16:05 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Fix parsing of permissions with special bits https://review.opendev.org/c/openstack/kolla/+/988849 | 16:14 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: keystone: Setup fernet credentials encryption keys https://review.opendev.org/c/openstack/kolla-ansible/+/970861 | 19:54 |
| opendevreview | Maksim Malchuk proposed openstack/kayobe-config stable/2025.1: [2025.1-only] Synchronise with latest kayobe https://review.opendev.org/c/openstack/kayobe-config/+/995752 | 21:12 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!
