| *** amoralej|off is now known as amoralej | 07:00 | |
| kevko | morning \o/ | 07:20 |
|---|---|---|
| mnasiadka | morning | 07:39 |
| *** rpittau|afk is now known as rpittau | 07:55 | |
| kevko | mnasiadka: could you check again my patches ? i think all comments implemented :/ | 08:02 |
| mnasiadka | will do later today | 08:02 |
| opendevreview | Pierre Riteau proposed openstack/kayobe master: Add support for apt proxy setting https://review.opendev.org/c/openstack/kayobe/+/799629 | 08:17 |
| opendevreview | Pierre Riteau proposed openstack/kayobe master: Ubuntu: add upgrade jobs in CI https://review.opendev.org/c/openstack/kayobe/+/797626 | 08:18 |
| opendevreview | Pierre Riteau proposed openstack/kayobe master: Fix seed VM configdrive when admin network is a VLAN https://review.opendev.org/c/openstack/kayobe/+/752955 | 08:18 |
| opendevreview | Pierre Riteau proposed openstack/kayobe master: Revert to Xena IPA images https://review.opendev.org/c/openstack/kayobe/+/809644 | 08:20 |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible master: [WIP] Do not set ip_forward https://review.opendev.org/c/openstack/kolla-ansible/+/809977 | 09:00 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: WIP: Rocky Linux https://review.opendev.org/c/openstack/kolla/+/809001 | 09:28 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: WIP: Rocky Linux https://review.opendev.org/c/openstack/kolla/+/809001 | 09:52 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: WIP: Rocky Linux https://review.opendev.org/c/openstack/kolla/+/809001 | 09:55 |
| mnasiadka | hrw, yoctozepto: around kolla/Rocky Linux - I think it's in a shape I'd like to get some reviews - if you have time ^^ | 09:55 |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible master: Do not enable mariadb-clustercheck when not needed https://review.opendev.org/c/openstack/kolla-ansible/+/809990 | 09:59 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: docs: ceph: add copy-on-write for Glance https://review.opendev.org/c/openstack/kolla-ansible/+/809992 | 10:05 |
| opendevreview | Merged openstack/kayobe master: CI: Log disk usage details https://review.opendev.org/c/openstack/kayobe/+/809209 | 10:16 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: docs: ceph: add copy-on-write for Glance https://review.opendev.org/c/openstack/kolla-ansible/+/809992 | 10:22 |
| opendevreview | Pierre Riteau proposed openstack/kayobe stable/train: CI: Log disk usage details https://review.opendev.org/c/openstack/kayobe/+/809660 | 10:22 |
| opendevreview | Pierre Riteau proposed openstack/kayobe master: Add support for apt proxy setting https://review.opendev.org/c/openstack/kayobe/+/799629 | 10:40 |
| opendevreview | Pierre Riteau proposed openstack/kayobe master: Add support for apt proxy setting https://review.opendev.org/c/openstack/kayobe/+/799629 | 10:56 |
| kevko | guys, i have some testing environment where ipv6 is disabled in grub so some files (so /proc/sys/net/ipv6/neigh/default/gc_thresh3 etc also for some other options) not exists.. | 10:58 |
| kevko | i think there should be check if it is disabled in sysctl or grub :/ | 11:02 |
| opendevreview | Pierre Riteau proposed openstack/kayobe master: Add support for apt proxy setting https://review.opendev.org/c/openstack/kayobe/+/799629 | 11:15 |
| hrw | mnasiadka: I was just about to look and complain | 11:16 |
| hrw | ;D | 11:16 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: WIP: Rocky Linux https://review.opendev.org/c/openstack/kolla/+/809001 | 11:19 |
| hrw | mnasiadka: I very much dislike whole base/Dockerfile.j2 | 11:22 |
| *** amoralej is now known as amoralej|lunch | 11:25 | |
| hrw | mnasiadka: tried to talk with Rocky devs to make them provide 'openvswitch' package so we do not have to remember about adding '2.13' each time? | 11:26 |
| hrw | {% if base_package_type == 'rpm' and base_distro == 'centos' %} | 11:31 |
| hrw | my favourite part ;D | 11:31 |
| hrw | reviewed. -1 | 11:32 |
| hrw | mnasiadka: in some images you remove python3-rtslib, in some it stays so it is not that it is missing in Rocky Linux. | 11:35 |
| hrw | mnasiadka: maybe some parts of that patch needs to be split out | 11:35 |
| mnasiadka | hrw: sure, maybe something changed over the time, will look at your comments - thanks | 11:59 |
| mnasiadka | hrw: well, now RDO is providing a virtual package and choosing if it's 2.13 or 2.15, we won't be able to keep that in sync | 12:00 |
| *** amoralej|lunch is now known as amoralej | 12:21 | |
| yoctozepto | yeah, it's messy | 12:37 |
| yoctozepto | I wonder if it's not better to keep rocky as k-a target | 12:38 |
| yoctozepto | i.e. on host | 12:38 |
| yoctozepto | kevko: disabling ipv6 is known to break at runtime as well | 12:38 |
| kevko | yoctozepto: yeah, but why it is not fixed :P ? | 12:39 |
| kevko | yoctozepto: it's trivial fix | 12:39 |
| yoctozepto | kevko: because one is not supposed to disable ipv6 | 12:39 |
| yoctozepto | you can (and should) disable it per interface | 12:40 |
| yoctozepto | not disable the whole ipv6 subsystem | 12:40 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: WIP: Rocky Linux https://review.opendev.org/c/openstack/kolla/+/809001 | 12:49 |
| mnasiadka | yoctozepto: and deploy centos-stream-9 container images on Rocky Linux (so essentially centos-linux-8)? | 12:50 |
| mnasiadka | fantastic, we use binary cinderlib package on source cinder-volume builds | 12:53 |
| yoctozepto | https://review.opendev.org/c/openstack/kolla-ansible/+/809977 so green! :D | 12:58 |
| yoctozepto | mnasiadka: yeah, I was working to fix those quirks but then moved onto debian... | 12:59 |
| yoctozepto | I *really* wonder if we wouldn't be better off with one common platform for inside the images | 12:59 |
| yoctozepto | like other containerised projects | 12:59 |
| mnasiadka | for API services? sure | 13:00 |
| yoctozepto | single flavour of images, multiple hosts | 13:00 |
| mnasiadka | for the rest, I'm not so convinced - but maybe I'm just biased ;) | 13:00 |
| yoctozepto | I think it could work well | 13:01 |
| hrw | mnasiadka: https://paste.centos.org/view/4850158f for base/Dockerfile.j2 | 13:01 |
| hrw | mnasiadka: some more changes would be needed to better unify but this is example of what I would go | 13:02 |
| mnasiadka | I was thinking of that, but felt it's less readable - but from other perspective base/Dockerfile is hardly-readable without a jinja2 compiler already ;) | 13:03 |
| mnasiadka | compiler/filter | 13:03 |
| mnasiadka | I'll pursue that way tomorrow, no time today... | 13:04 |
| hrw | mnasiadka: https://paste.centos.org/view/39eda498 even (plus check are if/else/endifs are done right) | 13:07 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: WIP: Rocky Linux https://review.opendev.org/c/openstack/kolla/+/809001 | 13:07 |
| yoctozepto | hrw/mnasiadka: let's merge mgoddard's CI fix for monasca https://review.opendev.org/c/openstack/kolla-ansible/+/807689 | 13:09 |
| yoctozepto | ah no, wait :-) | 13:10 |
| mnasiadka | with most of the change being commented out? | 13:10 |
| hrw | ah. did +3 already | 13:10 |
| mnasiadka | and one-shot container that is exited? | 13:10 |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible master: CI: monasca: stop checking Elasticsearch index https://review.opendev.org/c/openstack/kolla-ansible/+/807689 | 13:10 |
| hrw | [[ a = b ]] or [[ a == b ]]? | 13:11 |
| yoctozepto | no, had to change the order to have the unhealthiness check work on that scenario | 13:12 |
| yoctozepto | mnasiadka: yeah, it's making the checks worse but lets us see other, "more real", issues in that scenario | 13:13 |
| yoctozepto | I don't know better for now ;-) | 13:13 |
| mnasiadka | I still don't know what is worse, running a CI with one broken component and claiming it's green, or leaving it as red ;) | 13:14 |
| yoctozepto | not entirely sure either, but it's a big job (5 nodes) and now always red - very sad | 13:15 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: WIP: Rocky Linux https://review.opendev.org/c/openstack/kolla/+/809001 | 13:28 |
| mnasiadka | hrw: ^^ found time | 13:28 |
| hrw | mnasiadka: more readable ;D | 13:30 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: WIP: Rocky Linux https://review.opendev.org/c/openstack/kolla/+/809001 | 13:31 |
| hrw | mnasiadka: can you split that python3-cinderlib to separate patch? it feels like bugfix we should have anyway | 13:31 |
| mnasiadka | hrw: yeah, I was thinking about it, will do | 13:31 |
| hrw | mnasiadka: let me finish one review... | 13:33 |
| hrw | mnasiadka: what would you say for "ENV openvswitch_rpm_package=openvswitch" somewhere (base?) and then reuse it? | 13:34 |
| hrw | but no, it would not cover python3 pckage. | 13:35 |
| mnasiadka | hrw: we could do ARG openvswitch_version=2.13 on rocky and empty on centos probably | 13:37 |
| mnasiadka | I'll test later | 13:37 |
| hrw | mnasiadka: commented previous version | 13:38 |
| hrw | hm. gerrit shows that current. meh | 13:39 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: cinder-volume: Install source cinderlib in source https://review.opendev.org/c/openstack/kolla/+/810016 | 13:43 |
| opendevreview | Piotr Parczewski proposed openstack/kolla-ansible master: Use friendly target names in Prometheus https://review.opendev.org/c/openstack/kolla-ansible/+/810017 | 13:45 |
| opendevreview | Merged openstack/kayobe master: kolla: Set neutron_plugin_agent to OVN when it's enabled https://review.opendev.org/c/openstack/kayobe/+/797072 | 14:08 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: WIP: Rocky Linux https://review.opendev.org/c/openstack/kolla/+/809001 | 14:15 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: WIP: Rocky Linux https://review.opendev.org/c/openstack/kolla/+/809001 | 14:26 |
| mnasiadka | hrw: no more updates today ;) | 14:27 |
| hrw | ;d | 14:31 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: WIP: Rocky Linux https://review.opendev.org/c/openstack/kolla/+/809001 | 14:40 |
| mnasiadka | ok, I lied - had to fix unit tests. | 14:40 |
| opendevreview | Michal Nasiadka proposed openstack/kayobe master: CI: Fix bashate ignores https://review.opendev.org/c/openstack/kayobe/+/810029 | 14:48 |
| yoctozepto | liar lol | 15:50 |
| *** rpittau is now known as rpittau|afk | 16:00 | |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible master: [WIP] Do not load br_netfilter https://review.opendev.org/c/openstack/kolla-ansible/+/810086 | 16:53 |
| *** amoralej is now known as amoralej|off | 17:16 | |
| mloza | hello, is it possible to unset an option via custom config? I have backup enabled but I would like to unset these lines https://github.com/openstack/kolla-ansible/blob/master/ansible/roles/cinder/templates/cinder.conf.j2#L33-L40 from DEFAULT and move them to the backend config | 17:18 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: WIP: Rocky Linux https://review.opendev.org/c/openstack/kolla/+/809001 | 17:57 |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible master: Use mariadb_tag as default for mariadb_clustercheck_tag https://review.opendev.org/c/openstack/kolla-ansible/+/809581 | 18:02 |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible master: Use mariadb_tag as default for all mariadb tags https://review.opendev.org/c/openstack/kolla-ansible/+/809581 | 18:03 |
| opendevreview | Merged openstack/kolla-ansible master: Add Alertmanger metric target(s) https://review.opendev.org/c/openstack/kolla-ansible/+/782210 | 18:08 |
| opendevreview | Merged openstack/kolla-ansible master: Remove haproxy,keepalived groups https://review.opendev.org/c/openstack/kolla-ansible/+/807344 | 18:09 |
| opendevreview | Verification of a change to openstack/kolla-ansible master failed: Fix neutron upgrade using host limit without controllers https://review.opendev.org/c/openstack/kolla-ansible/+/808515 | 18:28 |
| opendevreview | Merged openstack/kayobe master: CI: Remove IPA images from saved /etc/kolla https://review.opendev.org/c/openstack/kayobe/+/809329 | 18:41 |
| opendevreview | Merged openstack/kolla-ansible master: Add kolla-ansible gather-facts command https://review.opendev.org/c/openstack/kolla-ansible/+/794610 | 18:54 |
| yoctozepto | mloza: nope, the best you can do is to provide them in your overrides and then they will take precedence over those - what's your issue with them? | 18:58 |
| kevko | yoctozepto: you are quite active :D | 19:11 |
| kevko | this evening | 19:11 |
| opendevreview | Merged openstack/kolla-ansible master: Add disable_firewall variable https://review.opendev.org/c/openstack/kolla-ansible/+/798854 | 19:57 |
| opendevreview | Merged openstack/kolla-ansible stable/victoria: Add override timeout for openstack exporter https://review.opendev.org/c/openstack/kolla-ansible/+/808493 | 20:03 |
| opendevreview | Merged openstack/kolla-ansible master: use ironic user in ironic_neutron_agent.ini file https://review.opendev.org/c/openstack/kolla-ansible/+/808673 | 20:03 |
| *** hrww is now known as hrw | 20:26 | |
| opendevreview | Merged openstack/kolla-ansible master: CI: stop setting ceph_nova_user https://review.opendev.org/c/openstack/kolla-ansible/+/798910 | 20:47 |
| mloza | yoctozepto: I have two volume backends. I would like to move those lines under [rbd-1] so I can separate backups options for the second backend | 21:00 |
| opendevreview | Verification of a change to openstack/kolla-ansible master failed: Fix neutron upgrade using host limit without controllers https://review.opendev.org/c/openstack/kolla-ansible/+/808515 | 21:00 |
| mloza | I can have separate backup* | 21:00 |
| mloza | right now, all backups goes to the backup pool of [rbd-1] | 21:02 |
| opendevreview | Merged openstack/kolla-ansible master: Fix neutron upgrade using host limit without controllers https://review.opendev.org/c/openstack/kolla-ansible/+/808515 | 23:11 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!