| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/wallaby: Check config when checking the containers https://review.opendev.org/c/openstack/kolla-ansible/+/803671 | 06:04 |
|---|---|---|
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/victoria: Check config when checking the containers https://review.opendev.org/c/openstack/kolla-ansible/+/803672 | 06:05 |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/ussuri: Check config when checking the containers https://review.opendev.org/c/openstack/kolla-ansible/+/803673 | 06:06 |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/ussuri: Check config when checking the containers https://review.opendev.org/c/openstack/kolla-ansible/+/803673 | 06:15 |
| *** rpittau|afk is now known as rpittau | 07:23 | |
| parallax | Morning Koalas | 08:29 |
| parallax | mgoddard: I think Launchpad is missing Xena milestone | 08:29 |
| kevko | hi ! | 08:29 |
| mgoddard | parallax: added some | 08:30 |
| parallax | awesome | 08:31 |
| mgoddard | mnasiadka: how about this one? Seems quite important | 09:25 |
| mgoddard | https://review.opendev.org/c/openstack/kolla-ansible/+/798851 | 09:25 |
| hrw | morning | 09:26 |
| hrw | mgoddard, mnasiadka: https://review.opendev.org/c/openstack/kolla/+/784923 maybe? | 09:26 |
| mgoddard | busy this morning | 09:26 |
| hrw | ok | 09:28 |
| opendevreview | Skylar Tristan Kelty proposed openstack/kolla-ansible master: Update Manila deploy steps for Wallaby https://review.opendev.org/c/openstack/kolla-ansible/+/802743 | 10:10 |
| mnasiadka | mgoddard: makes sense | 10:13 |
| kevko | hrw: do you know why mongodbuser in kolla is in nobody group ? | 10:37 |
| kevko | also, can it be removed if it was dropped ? | 10:37 |
| hrw | kevko: prepare a patch which removes all removed users? just leave gaps in numbering | 10:46 |
| kevko | hrw: no, i'm just asking why mongodb has 65534 gid | 10:47 |
| kevko | hrw: because on top of my stack yoctozepto proposed a patch to avoid --force from groupadd ..https://review.opendev.org/c/openstack/kolla/+/803617/3 << | 10:48 |
| kevko | hrw: and actually that patch revealed hidden bug in kolla | 10:48 |
| kevko | hrw: 65534:65534 is nobody:nogroup in linux ...so i am asking why mongodb user has that gid set ? | 10:49 |
| kevko | i was git-blaming that file and found that it was added with other users ..but really don't know what is a reason to override nogroup group with mongodb group with same ID ..and don't know why it is not same as mongodb user uid -> so 42432 | 10:51 |
| kevko | do you have any idea ? | 10:51 |
| hrw | nope | 10:53 |
| hrw | kevko: it was before me even | 10:53 |
| kevko | hrw: do you mean it can be changed to 42432 ? because i don't think there is a reason to override nogroup group | 10:57 |
| kevko | and not to have same uuid:gid for mongodbuser (moreover if it is non-used) | 10:58 |
| hrw | kevko: mongodb was dropped. drop user, stop worrying? | 11:08 |
| * hrw -> out. back in 2-3h | 11:09 | |
| opendevreview | Michal Arbet proposed openstack/kolla master: Avoid the use of --force in groupadd https://review.opendev.org/c/openstack/kolla/+/803617 | 11:16 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Avoid the use of --force in groupadd https://review.opendev.org/c/openstack/kolla/+/803617 | 12:06 |
| yoctozepto | mgoddard, mnasiadka: replied on https://review.opendev.org/c/openstack/kolla-ansible/+/798851/2..4 | 12:10 |
| opendevreview | Piotr Parczewski proposed openstack/kolla master: prometheus: Add OVN exporter https://review.opendev.org/c/openstack/kolla/+/762986 | 12:12 |
| mgoddard | yoctozepto: replied | 12:33 |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible master: Elevated privileges required to set owner/group/mode by ansible https://review.opendev.org/c/openstack/kolla-ansible/+/775752 | 12:39 |
| yoctozepto | mgoddard: approved | 12:41 |
| mgoddard | yoctozepto: very kind | 12:41 |
| kevko | yoctozepto: your yesterday proposal -> https://zuul.opendev.org/t/openstack/build/e571601a52464f58b3f67adf333a018d/log/kolla/build/000_FAILED_masakari-monitors.log << how can it be possible if LAST_SYSTEM_GID was bumped ? | 13:16 |
| kevko | yoctozepto: shouldn't sed command be executed before any adduser,addgroup command in base image ? ..because it is not :/ | 13:17 |
| kevko | yoctozepto: or add --system to both commands ? | 13:18 |
| opendevreview | Piotr Parczewski proposed openstack/kolla-ansible master: Remove deprecated Designate option https://review.opendev.org/c/openstack/kolla-ansible/+/802450 | 13:31 |
| yoctozepto | kevko: look at the change; you rebased badly | 13:35 |
| opendevreview | Ilya Popov proposed openstack/kolla-ansible master: Extra var ironic_enable_keystone_integration added. https://review.opendev.org/c/openstack/kolla-ansible/+/793664 | 13:42 |
| opendevreview | Merged openstack/kolla-ansible master: nova: Use cinder user for Ceph https://review.opendev.org/c/openstack/kolla-ansible/+/798851 | 13:58 |
| opendevreview | Merged openstack/kolla-ansible master: Use Docker healthchecks for ceilometer services https://review.opendev.org/c/openstack/kolla-ansible/+/780597 | 13:58 |
| opendevreview | Merged openstack/kolla-ansible master: Use Docker healthchecks for zun services https://review.opendev.org/c/openstack/kolla-ansible/+/779826 | 14:04 |
| opendevreview | Merged openstack/kolla-ansible master: Use Docker healthchecks for kuryr services https://review.opendev.org/c/openstack/kolla-ansible/+/779998 | 14:04 |
| opendevreview | Merged openstack/kolla-ansible master: Remove tempest role https://review.opendev.org/c/openstack/kolla-ansible/+/797792 | 14:04 |
| opendevreview | Merged openstack/kolla-ansible master: [manila] Drop tenant_id templating from v2 endpoint https://review.opendev.org/c/openstack/kolla-ansible/+/802657 | 14:12 |
| opendevreview | Merged openstack/kolla-ansible master: Remove support for Prometheus v1 https://review.opendev.org/c/openstack/kolla-ansible/+/803644 | 14:12 |
| opendevreview | Merged openstack/kolla-ansible master: Blazar: Fix support for external keystone in multiregion deploy https://review.opendev.org/c/openstack/kolla-ansible/+/797359 | 14:12 |
| opendevreview | Verification of a change to openstack/kolla-ansible stable/ussuri failed: watcher: add missing become for copying configs https://review.opendev.org/c/openstack/kolla-ansible/+/803289 | 14:12 |
| opendevreview | Skylar Tristan Kelty proposed openstack/kolla-ansible master: Update Manila deploy steps for Wallaby https://review.opendev.org/c/openstack/kolla-ansible/+/802743 | 14:23 |
| opendevreview | Merged openstack/kolla-ansible master: Fix config action when OVN is enabled https://review.opendev.org/c/openstack/kolla-ansible/+/799839 | 14:25 |
| opendevreview | Skylar Tristan Kelty proposed openstack/kolla-ansible master: Update Manila deploy steps for Wallaby https://review.opendev.org/c/openstack/kolla-ansible/+/802743 | 14:31 |
| opendevreview | Skylar Tristan Kelty proposed openstack/kolla-ansible master: Update Manila deploy steps for Wallaby https://review.opendev.org/c/openstack/kolla-ansible/+/802743 | 14:33 |
| opendevreview | Merged openstack/kolla-ansible stable/victoria: Do not run timesync checks on deployment host https://review.opendev.org/c/openstack/kolla-ansible/+/803555 | 14:35 |
| *** rpittau is now known as rpittau|afk | 14:43 | |
| opendevreview | Merged openstack/kolla-ansible master: Reduce container metrics cardinality https://review.opendev.org/c/openstack/kolla-ansible/+/800068 | 14:47 |
| * yoctozepto 's review rush merged | 14:48 | |
| opendevreview | Ilya Popov proposed openstack/kolla-ansible master: Extra var ironic_enable_keystone_integration added. https://review.opendev.org/c/openstack/kolla-ansible/+/793664 | 14:59 |
| kevko | yoctozepto: what do you mean badly ? | 15:06 |
| kevko | what is bad ? | 15:11 |
| opendevreview | Merged openstack/kolla-ansible master: Elevated privileges required to set owner/group/mode by ansible https://review.opendev.org/c/openstack/kolla-ansible/+/775752 | 15:12 |
| yoctozepto | kevko: sounds like a philosophical question :-) | 15:25 |
| yoctozepto | I meant you have literally undone the previous changes when rebasing | 15:26 |
| yoctozepto | hence why it fails | 15:26 |
| yoctozepto | no surprise ;-) | 15:26 |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/wallaby: Fix config action when OVN is enabled https://review.opendev.org/c/openstack/kolla-ansible/+/803684 | 15:26 |
| kevko | hmm | 15:30 |
| kevko | :( ..i can't see the issue | 15:30 |
| opendevreview | Merged openstack/kolla-ansible stable/ussuri: watcher: add missing become for copying configs https://review.opendev.org/c/openstack/kolla-ansible/+/803289 | 15:52 |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/victoria: Fix config action when OVN is enabled https://review.opendev.org/c/openstack/kolla-ansible/+/803685 | 16:01 |
| yoctozepto | kevko: have you looked at the diff of https://review.opendev.org/c/openstack/kolla/+/803617/ ? | 16:06 |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/ussuri: Fix config action when OVN is enabled https://review.opendev.org/c/openstack/kolla-ansible/+/803686 | 16:10 |
| kevko | yoctozepto: but what do you mean undone previous changes ..there couldnt't be {{ name }} .. there has to be {{ user.group }} | 16:31 |
| kevko | because of haclient | 16:34 |
| opendevreview | Victor Morales proposed openstack/kolla-ansible master: Remove unused imports in merge_yaml https://review.opendev.org/c/openstack/kolla-ansible/+/803806 | 21:50 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!