| opendevreview | Pierre Riteau proposed openstack/kayobe master: Add missing font for PDF generation https://review.opendev.org/c/openstack/kayobe/+/802424 | 05:35 |
|---|---|---|
| opendevreview | Michal Nasiadka proposed openstack/kayobe master: kolla: Set neutron_plugin_agent to OVN when it's enabled https://review.opendev.org/c/openstack/kayobe/+/797072 | 06:36 |
| opendevreview | Ole Kleinschmidt proposed openstack/kolla master: Adds Adjutant support https://review.opendev.org/c/openstack/kolla/+/801231 | 07:08 |
| *** rpittau|afk is now known as rpittau | 07:29 | |
| opendevreview | Merged openstack/kolla stable/ussuri: Fix prometheus-elasticsearch-exporter container. https://review.opendev.org/c/openstack/kolla/+/802208 | 07:52 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible master: cyborg: add missing become for api-paste.ini https://review.opendev.org/c/openstack/kolla-ansible/+/802430 | 07:53 |
| opendevreview | Mark Goddard proposed openstack/kolla stable/wallaby: Fix missing templating block for kolla-toolbox. https://review.opendev.org/c/openstack/kolla/+/802352 | 07:59 |
| opendevreview | Mark Goddard proposed openstack/kolla stable/victoria: Fix missing templating block for kolla-toolbox. https://review.opendev.org/c/openstack/kolla/+/802353 | 08:00 |
| opendevreview | Mark Goddard proposed openstack/kolla stable/ussuri: Fix missing templating block for kolla-toolbox. https://review.opendev.org/c/openstack/kolla/+/802354 | 08:00 |
| opendevreview | Merged openstack/kolla stable/wallaby: Fix prometheus-elasticsearch-exporter container. https://review.opendev.org/c/openstack/kolla/+/802206 | 08:02 |
| opendevreview | Verification of a change to openstack/kolla failed: Fix prometheus-elasticsearch-exporter container. https://review.opendev.org/c/openstack/kolla/+/802207 | 08:02 |
| opendevreview | Mark Goddard proposed openstack/kayobe master: Ubuntu: add upgrade jobs in CI https://review.opendev.org/c/openstack/kayobe/+/797626 | 08:05 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible master: DNM: test ubuntu-binary-ironic https://review.opendev.org/c/openstack/kolla-ansible/+/802433 | 08:08 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible master: Add support for Ceph RadosGW integration https://review.opendev.org/c/openstack/kolla-ansible/+/776662 | 08:13 |
| opendevreview | Verification of a change to openstack/kolla failed: Add masakari-dashboard to debian binary horizon image https://review.opendev.org/c/openstack/kolla/+/799348 | 08:15 |
| opendevreview | Mark Goddard proposed openstack/kayobe master: Support configuration of firewalld https://review.opendev.org/c/openstack/kayobe/+/797068 | 08:16 |
| opendevreview | Mark Goddard proposed openstack/kayobe master: Support Ansible collections https://review.opendev.org/c/openstack/kayobe/+/797497 | 08:17 |
| opendevreview | Skylar Tristan Kelty proposed openstack/kayobe master: Add support for apt proxy setting https://review.opendev.org/c/openstack/kayobe/+/799629 | 08:38 |
| opendevreview | Mark Goddard proposed openstack/kolla master: ironic-neutron-agent: enable for Debuntu binary https://review.opendev.org/c/openstack/kolla/+/799611 | 08:57 |
| opendevreview | Mark Goddard proposed openstack/kolla master: ironic-neutron-agent: enable for Debuntu binary https://review.opendev.org/c/openstack/kolla/+/799611 | 08:58 |
| opendevreview | wu.chunyang proposed openstack/kolla-ansible master: Fix incorrect config of linuxbridge multiple external networks https://review.opendev.org/c/openstack/kolla-ansible/+/787996 | 09:11 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible master: Add support for Ceph RadosGW integration https://review.opendev.org/c/openstack/kolla-ansible/+/776662 | 09:20 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible master: Add support for Ceph RadosGW integration https://review.opendev.org/c/openstack/kolla-ansible/+/776662 | 09:23 |
| opendevreview | Merged openstack/kayobe stable/wallaby: Ubuntu: CI: rely on os_distribution, remove interfaces pause https://review.opendev.org/c/openstack/kayobe/+/799245 | 09:25 |
| opendevreview | Gaël THEROND proposed openstack/kolla stable/ussuri: Improve offline build scenario. https://review.opendev.org/c/openstack/kolla/+/802355 | 09:34 |
| opendevreview | Gaël THEROND proposed openstack/kolla stable/train: Improve offline build scenario. https://review.opendev.org/c/openstack/kolla/+/802356 | 09:36 |
| opendevreview | Gaël THEROND proposed openstack/kolla stable/victoria: Improve offline build scenario. https://review.opendev.org/c/openstack/kolla/+/802357 | 09:37 |
| opendevreview | Gaël THEROND proposed openstack/kolla stable/wallaby: Improve offline build scenario. https://review.opendev.org/c/openstack/kolla/+/802458 | 09:37 |
| opendevreview | Verification of a change to openstack/kayobe failed: Remove stale DIB_DISABLE_KERNEL_CLEANUP references https://review.opendev.org/c/openstack/kayobe/+/799853 | 09:39 |
| opendevreview | Piotr Parczewski proposed openstack/kolla-ansible master: Remove deprecated Designate option https://review.opendev.org/c/openstack/kolla-ansible/+/802450 | 09:59 |
| opendevreview | Mark Goddard proposed openstack/kayobe master: Ubuntu: add upgrade jobs in CI https://review.opendev.org/c/openstack/kayobe/+/797626 | 10:00 |
| opendevreview | Mark Goddard proposed openstack/kayobe master: Ubuntu: add upgrade jobs in CI https://review.opendev.org/c/openstack/kayobe/+/797626 | 10:01 |
| kevko | yoctozepto: hi, could you switch masakari CI to debian binary ? | 10:06 |
| yoctozepto | kevko: hi, hmm? | 10:07 |
| kevko | yoctozepto: regarding this -> https://bugs.launchpad.net/kolla/+bug/1934753 | 10:08 |
| kevko | yoctozepto: i've read an email with update that you didn't reproduce bug in CI ..i think it could be caused by source variant of image (don't know but just wondering that in binary image there could be more packages installed as dependenies ..so more users created by postinsts ..so incopalibity in UUIDs ) | 10:10 |
| kevko | but i still think it is a bug ..because two images are inherited from two different base images ..and that base images can have differents sets of user/group uuids/gids even if it was good now (which is not currently i think) | 10:12 |
| kevko | because of debian/ubuntu dynamic uuid/gids allocating | 10:12 |
| haerwu | kevko: and we create users before packages are installed | 10:35 |
| haerwu | kevko: and there were 1-2 packages which complained. we had workarounds for them, one got fixed in Debian quickly after reporting, second (haproxy?) stayed with issue | 10:36 |
| kevko | are u talking about my above bugreport ? | 10:38 |
| yoctozepto | kevko: ah, try proposing a change over https://review.opendev.org/c/openstack/kolla-ansible/+/802099 | 10:39 |
| yoctozepto | it will be DNM but you can test and show me the results | 10:39 |
| kevko | yoctozepto: hmm, i'm not good in CI configs ... can I just change source to binary ? | 10:40 |
| kevko | it should work right ? | 10:40 |
| haerwu | kevko: add creation of hacluster:haclient user into base? | 10:48 |
| yoctozepto | kevko: yeah, just in a new change | 10:49 |
| yoctozepto | (i.e., don't overwrite this one) | 10:49 |
| haerwu | kevko: if any image requires user then this user needs to be created in base image | 10:49 |
| haerwu | we already create ~80 of them so one more does not make a difference | 10:51 |
| yoctozepto | haerwu: so far I have not been able to reproduce kevko's issue so I'm unsure how that problem got into his image; we'll see after he tests binary instead | 10:52 |
| haerwu | still, if user is created then it needs to be done in base | 10:52 |
| haerwu | 42486 uid waits... | 10:53 |
| kevko | yoctozepto: so i copy your review and change source->binary and post as new review ? | 10:53 |
| yoctozepto | kevko: yeah, that works for me | 10:53 |
| yoctozepto | haerwu: +1 | 10:53 |
| haerwu | and Debian and since 109706aa8b26f6f915e1cadae89de8135c3b68bf our users are treated as 'system users' so haproxy does not complain anymore | 10:56 |
| haerwu | s/^and Debian// | 10:57 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible master: Add support for Ceph RadosGW integration https://review.opendev.org/c/openstack/kolla-ansible/+/776662 | 10:59 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [CI] Test Masakari on Debian Binary https://review.opendev.org/c/openstack/kolla-ansible/+/802456 | 11:02 |
| kevko | https://review.opendev.org/c/openstack/kolla-ansible/+/802456 | 11:04 |
| haerwu | kevko: why move of ubuntu to source was not mentioned? | 11:10 |
| kevko | i've changed it just for test because i think it will be the same issue | 11:10 |
| kevko | but i tried to replicate only on debian images | 11:11 |
| haerwu | then add [DNM], W-1 etc markers | 11:11 |
| haerwu | I added CR-1 marker instead | 11:11 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [DNM] Test Masakari on Debian Binary https://review.opendev.org/c/openstack/kolla-ansible/+/802456 | 11:12 |
| kevko | haerwu: better ? | 11:12 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [DNM] Test Masakari on Debian/Ubuntu Binary https://review.opendev.org/c/openstack/kolla-ansible/+/802456 | 11:12 |
| haerwu | better | 11:13 |
| kevko | :) thanks | 11:13 |
| opendevreview | Merged openstack/kolla master: Add masakari-dashboard to debian binary horizon image https://review.opendev.org/c/openstack/kolla/+/799348 | 11:19 |
| opendevreview | Verification of a change to openstack/kayobe failed: Add missing font for PDF generation https://review.opendev.org/c/openstack/kayobe/+/802424 | 11:25 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible master: Add support for Ceph RadosGW integration https://review.opendev.org/c/openstack/kolla-ansible/+/776662 | 12:45 |
| opendevreview | Bharat Kunwar proposed openstack/kolla-ansible master: Send Barbican API logs verbatim https://review.opendev.org/c/openstack/kolla-ansible/+/802492 | 13:06 |
| kevko | haerwu: failing neutron openvswitch agent :( ..will try to switch to source again :/ | 13:36 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [DNM] Test Masakari on Debian/Ubuntu Binary https://review.opendev.org/c/openstack/kolla-ansible/+/802456 | 13:38 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [DNM] Test Masakari on Debian/Ubuntu https://review.opendev.org/c/openstack/kolla-ansible/+/802456 | 13:44 |
| opendevreview | Michal Arbet proposed openstack/kolla stable/wallaby: Add masakari-dashboard to debian binary horizon image https://review.opendev.org/c/openstack/kolla/+/802459 | 13:46 |
| opendevreview | wu.chunyang proposed openstack/kolla-ansible master: Fix nova deployment failure when rabbitmq is disabled https://review.opendev.org/c/openstack/kolla-ansible/+/799857 | 14:07 |
| yoctozepto | kevko: notice hostmonitor uses this command so it just works (TM) in the current form | 14:08 |
| yoctozepto | have you checked locally whether source works and binary not? | 14:08 |
| yoctozepto | what was wrong with ovs agent in binary? | 14:08 |
| kevko | yoctozepto: didn't check | 14:08 |
| kevko | yoctozepto: not yet | 14:08 |
| yoctozepto | eh | 14:08 |
| kevko | i have debian binary deployed ..tempest checked and all passing | 14:09 |
| yoctozepto | masakari is not tested by tempest | 14:10 |
| yoctozepto | I mean you wanted to prove masakari/hacluster images are broken | 14:10 |
| yoctozepto | but so far that's not happening | 14:11 |
| kevko | (i meant neutron openvswitch agent ) | 14:23 |
| kevko | yoctozepto: let me build source version | 14:23 |
| yoctozepto | ok | 14:24 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [DNM] Test Masakari on Debian/Ubuntu https://review.opendev.org/c/openstack/kolla-ansible/+/802456 | 15:00 |
| kevko | yoctozepto: masakari source images working as expected ... so issue is only debian-binary | 15:42 |
| kevko | yoctozepto: https://paste.opendev.org/show/807732/ | 15:42 |
| yoctozepto | kevko_: interesting! I wonder why | 16:08 |
| kevko_ | yoctozepto: give me 5 minutes | 16:13 |
| yoctozepto | kevko_: you have been credited 5 minutes | 16:13 |
| kevko_ | how much I have ? :P | 16:17 |
| kevko_ | yoctozepto: ok, done ... here it is :) https://paste.opendev.org/show/807733/ | 16:21 |
| opendevreview | Merged openstack/kayobe master: Add missing font for PDF generation https://review.opendev.org/c/openstack/kayobe/+/802424 | 16:32 |
| kevko_ | yoctozepto: so ? | 16:33 |
| yoctozepto | kevko: seems nothing relevant | 16:39 |
| yoctozepto | ah, I see | 16:40 |
| kevko | hacluster_pacemaker and masakari_hostmonitor has different uid | 16:40 |
| kevko | !! | 16:40 |
| opendevmeet | kevko: Error: "!" is not a valid command. | 16:40 |
| kevko | -hacluster:x:101:105::/var/lib/pacemaker:/usr/sbin/nologin | 16:40 |
| kevko | +hacluster:x:102:106::/var/lib/pacemaker:/usr/sbin/nologin | 16:40 |
| yoctozepto | yeah | 16:40 |
| kevko | which is whole thing which i am describing in bugreport | 16:40 |
| yoctozepto | super odd | 16:40 |
| *** rpittau is now known as rpittau|afk | 16:40 | |
| yoctozepto | so it stays the same with source images | 16:41 |
| yoctozepto | and differs in binary | 16:41 |
| kevko | yeah | 16:41 |
| yoctozepto | as if in binary we installed something more to bump the uids | 16:41 |
| yoctozepto | but the diff does not show other additions | 16:41 |
| yoctozepto | which would differ too | 16:41 |
| yoctozepto | weird | 16:41 |
| yoctozepto | ah no | 16:42 |
| yoctozepto | I see | 16:42 |
| kevko | -messagebus:x:102:106::/nonexistent:/usr/sbin/nologin | 16:42 |
| kevko | +messagebus:x:101:105::/nonexistent:/usr/sbin/nologin | 16:42 |
| yoctozepto | messagebus | 16:42 |
| yoctozepto | something pulled in dbus things | 16:42 |
| yoctozepto | ok, makes sense to stabilise this UID | 16:43 |
| yoctozepto | though it could be tricky now with existing deployments | 16:43 |
| yoctozepto | arar | 16:43 |
| kevko | also, i think macro should be reimplemented to allow defining home directory.. login command .. and allow to add user to different named group | 16:43 |
| kevko | hacluster -> haclient | 16:43 |
| kevko | and set proper home directories for example /var/lib/hacluster (deb/ubuntu) and /home/hacluster (centos) | 16:44 |
| yoctozepto | it's /home/hacluster on centos? oh my, what a nice mess | 16:44 |
| kevko | yeah it is | 16:44 |
| kevko | yoctozepto: also you can see the mess between base and openstack-base | 16:45 |
| kevko | yoctozepto: -masakari:x:42485:42485::/home/masakari:/usr/sbin/nologin | 16:45 |
| kevko | +masakari:x:42485:42485::/var/lib/masakari:/bin/false | 16:45 |
| yoctozepto | so what we need is | 16:49 |
| yoctozepto | to add a hacluster user to kolla preset users | 16:49 |
| yoctozepto | and add a logic to extend_start to chown /var/lib/corosync and /var/lib/pacemaker appropriately | 16:49 |
| kevko | yeah | 16:50 |
| yoctozepto | hmm, in fact I think corosync uses root so it should be irrelevant, though still worth checking | 16:50 |
| kevko | one thing I don't know if it is important or not is hacluster user in haclient group | 16:50 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible master: Add support for Ceph RadosGW integration https://review.opendev.org/c/openstack/kolla-ansible/+/776662 | 16:51 |
| kevko | if I know there is a macro which is creating user->group with same name | 16:51 |
| kevko | masakari-user { uid: NUMBER, gid: NUMBER } which produces command to create masakari user and masakari group after some python mumbo-jumbo magic | 16:52 |
| kevko | so hacluster-user { uuid: number, gid: number } will produce hacluster user and hacluster group .. | 16:53 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible master: Add support for Ceph RadosGW integration https://review.opendev.org/c/openstack/kolla-ansible/+/776662 | 16:53 |
| kevko | how to get haclient group ? | 16:53 |
| kevko | yoctozepto: have to go now ...but let me know :) to bugreport ..or to review .. i would like to test it finally :( | 16:56 |
| yoctozepto | extend the logic to include groupname | 16:56 |
| yoctozepto | now it defaults to name of the user | 16:56 |
| yoctozepto | add extra key to allow to define group name | 16:57 |
| yoctozepto | and we are golden | 16:57 |
| opendevreview | David Moreau Simard proposed openstack/kolla-ansible master: DNM: Test ara 1.5.7rc1 with kolla-ansible https://review.opendev.org/c/openstack/kolla-ansible/+/720902 | 18:17 |
| opendevreview | Maksim Malchuk proposed openstack/kolla master: Add boto3 as s3 dependency for Glance container https://review.opendev.org/c/openstack/kolla/+/797355 | 22:03 |
| opendevreview | Maksim Malchuk proposed openstack/kolla master: Add boto3 as s3 dependency for Glance container https://review.opendev.org/c/openstack/kolla/+/797355 | 22:05 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!