| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: libvirt: Add role for modular libvirt daemons https://review.opendev.org/c/openstack/kolla-ansible/+/997367 | 07:58 |
|---|---|---|
| opendevreview | Michal Nasiadka proposed openstack/kolla master: Move PIP_* env variables to ARG and config https://review.opendev.org/c/openstack/kolla/+/993675 | 08:22 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: Add cleanup_repos to revert build_only overrides https://review.opendev.org/c/openstack/kolla/+/993712 | 08:23 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: Add monitoring for external component versions https://review.opendev.org/c/openstack/kolla/+/972434 | 08:31 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: Updates to sources.py https://review.opendev.org/c/openstack/kolla/+/994895 | 08:31 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: Updates to sources.py https://review.opendev.org/c/openstack/kolla/+/994895 | 08:32 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: Updates to sources.py https://review.opendev.org/c/openstack/kolla/+/994895 | 08:32 |
| mnasiadka | blanson[m]: updated https://review.opendev.org/c/openstack/kolla/+/972434/21 - would be nice to get this in (so we can have a bot updating sources.py) | 08:33 |
| opendevreview | Rafal Lewandowski proposed openstack/kayobe master: Enable systemd-resolved configuration for Debian based distros https://review.opendev.org/c/openstack/kayobe/+/997692 | 08:45 |
| kevko | morning o/ | 08:45 |
| opendevreview | OpenStack Proposal Bot proposed openstack/kolla master: Updated ansible-lint in lint-requirements.txt https://review.opendev.org/c/openstack/kolla/+/995628 | 08:45 |
| opendevreview | Rafal Lewandowski proposed openstack/kayobe master: Enable systemd-resolved configuration for Debian based distros https://review.opendev.org/c/openstack/kayobe/+/997692 | 08:46 |
| kevko | i've backported master pending patch for neutron-ovn-agent https://review.opendev.org/c/openstack/kolla-ansible/+/996078 , in log I can see problems with caps .... so i've blindly coppied cap_add from neutron-ovn-vpn-agent from default just to see if it will help ....and to my surprise, this doesn't add any CapAdd entries at all when I check docker | 08:49 |
| kevko | inspect ...how is that possible ? Am I doing something wrong ? | 08:49 |
| *** jhorstmann is now known as Guest13590 | 08:52 | |
| opendevreview | Alex Welsh proposed openstack/kolla-ansible master: Add openstack cacert to uwsgi env for Glance https://review.opendev.org/c/openstack/kolla-ansible/+/996647 | 08:53 |
| Vii | mnasiadka: blanson[m]: Hi, can we move forward with patch 993788: "Migrate internal state storage to YAML" https://review.opendev.org/c/openstack/kolla/+/993788 | 08:54 |
| Vii | 2026.2 is getting closer, so it would be great if we could get this change moving forward and have it updated in time. | 08:55 |
| kevko | Vii: why you are rewriting something you really don't need to ? | 08:58 |
| Vii | kevko: This was agreed during a previous? PTG. YAML is more readable, and currently we often hit issues with commas, tabs, and formatting errors in the existing format. | 09:01 |
| mnasiadka | kevko: we’re moving from config.json to config.yaml - makes sense to not use half json half yaml in kolla_set_configs, right? | 09:07 |
| kevko | well, yeah...on the end it's nicer ...but state default is completly hidden before end user ... | 09:15 |
| kevko | so no problems with commas, tabs, formatting errors | 09:16 |
| kevko | mnasiadka: btw ...do you know why caps for me is not working ^^ ? | 09:16 |
| mnasiadka | I have no clue, I know Vii had some weird patch for caps - but it seemed to not be required in vpn agent | 09:17 |
| kevko | mnasiadka: it's required :) | 09:18 |
| kevko | mnasiadka: that's why i coppied working tested vpn agent to OVN-agent .... but somehow internal kolla-ansible don't reflect defaults/main.yml where caps are defined ..to the final container ... | 09:19 |
| opendevreview | Alex Welsh proposed openstack/kolla-ansible master: Add openstack cacert to uwsgi env for Glance https://review.opendev.org/c/openstack/kolla-ansible/+/996647 | 09:19 |
| kevko | this is ovn-agent https://paste.openstack.org/show/bWFcwigj0X0CGFzE2dLT/ | 09:20 |
| kevko | so ... i said to myself ...ah, okay .. i will set caps for ovn-agent into default ...run reconfigure ..and that's it ... but capsadd not in final docker inspect :D ..while the same working in another container :D | 09:21 |
| opendevreview | Leonie Chamberlin-Medd proposed openstack/kayobe master: Adds missing reconfigure.yml for manage-containers https://review.opendev.org/c/openstack/kayobe/+/997502 | 09:21 |
| Vii | kevko: 2025.2 https://paste.openstack.org/show/bzJ53nqNyCLdgz68bn07/ | 09:23 |
| kevko | Vii: ovn-vpn-agent is fine as I said :D ..please read again ..carefully | 09:24 |
| Vii | ahh sry | 09:24 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Migrate to pyproject.toml https://review.opendev.org/c/openstack/kolla-ansible/+/997697 | 09:24 |
| 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 | 09:26 |
| Vii | kevko: Did you forget to change this part? handlers/main.yml ... cap_add: "{{ service.cap_add | default(omit) }}" ? | 09:34 |
| kevko | Vii: hmm, probably :D | 09:39 |
| kevko | Vii: thanks | 09:39 |
| Vii | :) | 09:39 |
| opendevreview | Piotr Milewski proposed openstack/kolla-ansible master: loadbalancer: drop privileged mode for haproxy and add capabilities https://review.opendev.org/c/openstack/kolla-ansible/+/975551 | 09:39 |
| Vii | kevko: Also check ^ because I remember there was something related to "normalize_cap_add", but I think it was only an issue with older Docker versions. It should be fine now. | 09:40 |
| Vii | Let's see what the tests say. | 09:41 |
| opendevreview | Rafal Lewandowski proposed openstack/kayobe master: Add support for QoS egress/ingress settings in systemd-networkd https://review.opendev.org/c/openstack/kayobe/+/997201 | 09:42 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: ironic: stop using driver-requirements.txt https://review.opendev.org/c/openstack/kolla/+/997702 | 10:00 |
| opendevreview | Owen Jones proposed openstack/kolla master: ironic-pxe: don't fail if symlinks already exist https://review.opendev.org/c/openstack/kolla/+/996814 | 10:14 |
| opendevreview | Merged openstack/kolla-ansible master: Migrate to pyproject.toml https://review.opendev.org/c/openstack/kolla-ansible/+/997697 | 10:18 |
| opendevreview | Merged openstack/kolla-ansible master: Updated ansible-lint j2lint in lint-requirements.txt https://review.opendev.org/c/openstack/kolla-ansible/+/995629 | 10:18 |
| opendevreview | Grzegorz Koper proposed openstack/kolla-ansible master: neutron: Split shared network agents into role https://review.opendev.org/c/openstack/kolla-ansible/+/975614 | 10:21 |
| opendevreview | Owen Jones proposed openstack/kolla master: ironic-pxe: don't fail if symlinks already exist https://review.opendev.org/c/openstack/kolla/+/996814 | 10:35 |
| opendevreview | Merged openstack/kolla master: Updated ansible-lint in lint-requirements.txt https://review.opendev.org/c/openstack/kolla/+/995628 | 10:37 |
| opendevreview | Grzegorz Koper proposed openstack/kolla-ansible master: neutron: Split shared compute agents into role https://review.opendev.org/c/openstack/kolla-ansible/+/997600 | 10:41 |
| opendevreview | Grzegorz Koper proposed openstack/kolla-ansible master: neutron: Split ML2/OVS agents into role https://review.opendev.org/c/openstack/kolla-ansible/+/997601 | 10:42 |
| opendevreview | Grzegorz Koper proposed openstack/kolla-ansible master: neutron: Split ML2/OVN agents into role https://review.opendev.org/c/openstack/kolla-ansible/+/997602 | 10:42 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: libvirt: Add role for modular libvirt daemons https://review.opendev.org/c/openstack/kolla-ansible/+/997367 | 10:52 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Convert into a bundled Ansible collection https://review.opendev.org/c/openstack/kolla-ansible/+/997710 | 11:01 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: Add cleanup_repos to revert build_only overrides https://review.opendev.org/c/openstack/kolla/+/993712 | 11:03 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Convert into a bundled Ansible collection https://review.opendev.org/c/openstack/kolla-ansible/+/997710 | 11:24 |
| opendevreview | Rafal Lewandowski proposed openstack/kayobe master: Remove leftover device mapper references from the code https://review.opendev.org/c/openstack/kayobe/+/997714 | 12:17 |
| opendevreview | Rafal Lewandowski proposed openstack/kayobe master: Enable systemd-resolved configuration for Debian based distros https://review.opendev.org/c/openstack/kayobe/+/997692 | 12:30 |
| opendevreview | Rafal Lewandowski proposed openstack/kayobe master: Remove leftover device mapper references from the code https://review.opendev.org/c/openstack/kayobe/+/997714 | 12:32 |
| opendevreview | Leonie Chamberlin-Medd proposed openstack/kayobe master: Add missing reconfigure.yml for manage-containers https://review.opendev.org/c/openstack/kayobe/+/997502 | 14:18 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Convert into a bundled Ansible collection https://review.opendev.org/c/openstack/kolla-ansible/+/997710 | 15:03 |
| opendevreview | Will Szumski proposed openstack/kayobe master: Enable systemd-resolved configuration for Debian based distros https://review.opendev.org/c/openstack/kayobe/+/997692 | 15:11 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: neutron: Run metadata extension in ovn agent https://review.opendev.org/c/openstack/kolla-ansible/+/996083 | 15:23 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: neutron: Remove OVN metadata agent https://review.opendev.org/c/openstack/kolla-ansible/+/996078 | 15:23 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Fix missing runtime configuration for neutron-ovn-agent https://review.opendev.org/c/openstack/kolla-ansible/+/997744 | 15:23 |
| opendevreview | Grzegorz Koper proposed openstack/kolla master: docs: Refresh running-in-development guide https://review.opendev.org/c/openstack/kolla/+/997745 | 15:23 |
| opendevreview | Will Szumski proposed openstack/kayobe master: Cleaner split of kolla-ansible install and configure https://review.opendev.org/c/openstack/kayobe/+/995016 | 15:24 |
| opendevreview | Grzegorz Koper proposed openstack/kolla master: docs: Refresh running-in-development guide https://review.opendev.org/c/openstack/kolla/+/997745 | 15:32 |
| opendevreview | Grzegorz Koper proposed openstack/kolla master: docs: Refresh running-in-development guide https://review.opendev.org/c/openstack/kolla/+/997745 | 15:38 |
| opendevreview | Grzegorz Koper proposed openstack/kolla master: docs: Refresh running-in-development guide https://review.opendev.org/c/openstack/kolla/+/997745 | 15:42 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Mark Debian Trixie jobs as non voting https://review.opendev.org/c/openstack/kolla-ansible/+/997750 | 15:46 |
| opendevreview | Grzegorz Koper proposed openstack/kolla master: docs: Refresh running-in-development guide https://review.opendev.org/c/openstack/kolla/+/997745 | 15:46 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Mark Debian Trixie jobs as non voting https://review.opendev.org/c/openstack/kolla-ansible/+/997750 | 15:46 |
| mnasiadka | frickler: ^^ 997750 might interest you ;-) | 15:47 |
| opendevreview | Will Szumski proposed openstack/kayobe master: Fix vault password env var validation https://review.opendev.org/c/openstack/kayobe/+/996842 | 16:25 |
| opendevreview | Verification of a change to openstack/kayobe master failed: Remove leftover device mapper references from the code https://review.opendev.org/c/openstack/kayobe/+/997714 | 16:48 |
| blanson[m] | https://review.opendev.org/c/openstack/kolla-ansible/+/986237 little review is waiting for review in the review queue (: | 17:00 |
| kevko | blanson[m]: and what about second +2 for https://review.opendev.org/c/openstack/kolla-ansible/+/989850 ? :) | 17:10 |
| kevko | blanson[m]: going to check | 17:10 |
| blanson[m] | holy +2000 | 17:10 |
| blanson[m] | I'll give it a look | 17:11 |
| kevko | blanson[m]: well, actually it's the same repeating again and again :) ...for every service ..but included in one patch for easy backport (even if downstream) | 17:14 |
| kevko | blanson[m]: doug already gave +2 | 17:14 |
| mnasiadka | Asking people to review on Friday 7pm should be banned | 17:16 |
| kevko | mnasiadka: why ? :D | 17:19 |
| kevko | mnasiadka: it's best time to look around when we are already out of office :) | 17:20 |
| kevko | * in opensource world | 17:20 |
| blanson[m] | hahahahahaha | 17:21 |
| blanson[m] | mnasiadka: well it's the first time in 3 days that I can do anything but fix broken clusters or sleep | 17:21 |
| blanson[m] | so | 17:21 |
| kevko | +1 | 17:22 |
| blanson[m] | I figured there was surely some reviews waiting for some love | 17:22 |
| mnasiadka | blanson[m]: better sleep more before reviewing 2000+ files change :) | 17:25 |
| blanson[m] | I'm scared if I sleep now I'll wake up monday morning | 17:28 |
| kevko | mnasiadka: ah :D | 17:35 |
| opendevreview | Merged openstack/kolla-ansible master: CI: Mark Debian Trixie jobs as non voting https://review.opendev.org/c/openstack/kolla-ansible/+/997750 | 18:39 |
| opendevreview | Kurt Bendl proposed openstack/kolla-ansible master: DNM: selinux security_opt and policy generation https://review.opendev.org/c/openstack/kolla-ansible/+/986399 | 21:23 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!