Friday, 2022-04-08

yoctozeptomgoddard: day to merge the ironic patches :-)06:47
opendevreviewMerged openstack/kolla master: CI: remove binary jobs  https://review.opendev.org/c/openstack/kolla/+/83657206:56
hrwho ho. no binary world started07:02
opendevreviewMarcin Juszkiewicz proposed openstack/kolla master: tests: drop testing for binary install type  https://review.opendev.org/c/openstack/kolla/+/83657307:05
opendevreviewMarcin Juszkiewicz proposed openstack/kolla master: kolla: drop 'binary' install type  https://review.opendev.org/c/openstack/kolla/+/83657407:06
opendevreviewMarcin Juszkiewicz proposed openstack/kolla master: tests: drop testing for binary install type  https://review.opendev.org/c/openstack/kolla/+/83657307:06
*** amoralej|off is now known as amoralej07:08
hrwINFO:kolla.common.utils.base:Successfully tagged short-name-1/debian-base:14.1.007:18
hrwworld is not yet ready for such names07:18
hrwshort-name-1/debian-elasticsearch-curator:14.1.0 etc07:19
fricklerhrw: I try to review all those rm-bin patches, but I'm not sure in which order they need to be merged, feel free to approve yourself or ping me if some review is missing07:28
opendevreviewMark Goddard proposed openstack/kayobe stable/xena: Fix Ansible inventory generation when reusing group names  https://review.opendev.org/c/openstack/kayobe/+/83705608:37
opendevreviewMark Goddard proposed openstack/kayobe stable/wallaby: Fix Ansible inventory generation when reusing group names  https://review.opendev.org/c/openstack/kayobe/+/83705708:38
opendevreviewMark Goddard proposed openstack/kayobe stable/victoria: Fix Ansible inventory generation when reusing group names  https://review.opendev.org/c/openstack/kayobe/+/83705808:38
opendevreviewMerged openstack/kolla-ansible master: CI: remove binary jobs  https://review.opendev.org/c/openstack/kolla-ansible/+/83658008:55
opendevreviewMark Goddard proposed openstack/kayobe master: ironic: revert to ironic's default drivers & interfaces  https://review.opendev.org/c/openstack/kayobe/+/83699909:07
opendevreviewRadosław Piliszek proposed openstack/kolla master: kolla: drop 'binary' install type  https://review.opendev.org/c/openstack/kolla/+/83657409:45
opendevreviewRadosław Piliszek proposed openstack/kolla master: drop add_binary_source_envs() macro  https://review.opendev.org/c/openstack/kolla/+/83657509:47
opendevreviewRadosław Piliszek proposed openstack/kolla master: drop infra-rename support  https://review.opendev.org/c/openstack/kolla/+/83657609:47
opendevreviewRadosław Piliszek proposed openstack/kolla master: docker: drop binary parts  https://review.opendev.org/c/openstack/kolla/+/83657709:47
yoctozeptofrickler: revote https://review.opendev.org/c/openstack/kolla/+/836574/3 ? :-)09:48
frickleryoctozepto squashmaster ;)09:53
yoctozeptofrickler: :-)09:53
opendevreviewRadosław Piliszek proposed openstack/kolla master: drop infra-rename support  https://review.opendev.org/c/openstack/kolla/+/83657609:54
opendevreviewRadosław Piliszek proposed openstack/kolla master: docker: drop binary parts  https://review.opendev.org/c/openstack/kolla/+/83657709:55
opendevreviewRadosław Piliszek proposed openstack/kolla master: drop add_binary_source_envs() macro  https://review.opendev.org/c/openstack/kolla/+/83657509:55
opendevreviewRadosław Piliszek proposed openstack/kolla master: docker: drop binary parts  https://review.opendev.org/c/openstack/kolla/+/83657709:58
opendevreviewRadosław Piliszek proposed openstack/kolla master: drop add_binary_source_envs() macro  https://review.opendev.org/c/openstack/kolla/+/83657509:58
yoctozeptothe order should be fine now09:59
yoctozeptoI reviewed all but the biggest one09:59
opendevreviewMark Goddard proposed openstack/kayobe master: ironic: default to ipxe booting  https://review.opendev.org/c/openstack/kayobe/+/83620310:01
hrwmorning10:14
hrwdebian-nova-libvirt - I like it. but patch to have shorter names will wait10:17
opendevreviewMerged openstack/kayobe master: Restore forgotten linuxbridge-agent container  https://review.opendev.org/c/openstack/kayobe/+/83703310:30
hrwmigration to centos stream 9 will take a while ;(10:34
hrwhttps://github.com/fluent/fluentd/discussions/370410:34
opendevreviewMarcin Juszkiewicz proposed openstack/kolla master: drop infra_image_prefix  https://review.opendev.org/c/openstack/kolla/+/83711410:39
hrwyoctozepto: here you have10:39
opendevreviewMarcin Juszkiewicz proposed openstack/kolla master: add releasenote about dropping binary images support  https://review.opendev.org/c/openstack/kolla/+/83711510:43
fricklertime for a friday question: should "become: true" be listed in front of the action (because it is like "sudo cmd ...") or afterwards (since it is a modifier)? does it make sense to strive for consistency one way or the other? ansible-lint doesn't seem to care, is there some other style tool that would cover such things?11:38
fricklercurrent usage in k-a seems to be wildly mixed11:39
yoctozeptothat's a good question; I would love something that tells the canonical order so that we don't have to care and just train our eyes to catch all details in the preset order11:40
frickler(we just had this discussion downstream and I'm now seeking for general wisdom)11:40
hrwI usually use it as first11:58
*** amoralej is now known as amoralej|lunch12:41
opendevreviewMerged openstack/kolla master: kolla: drop 'binary' install type  https://review.opendev.org/c/openstack/kolla/+/83657413:09
*** amoralej|lunch is now known as amoralej13:24
hrwyay13:26
hrwno more binary possible13:26
kevko:P13:29
hrwhttps://review.opendev.org/q/topic:zed-remove-support-for-binary-images waits for reviews13:30
mnasiadkahrw: did some ;-)14:31
hrwreplied14:31
hrwfun to see how they go in ;d14:32
mnasiadkabut that's some incosistency on our side, we allow users to override aodh_base_packages but not aodh_api_packages14:32
hrwsent patch14:33
hrw;D14:34
mnasiadkahrw: that td-agent thing is a bit worrying - I assume gem install fluentd would cause the same?14:34
mnasiadkaI mean hit the same problem, with old Ruby and openssl14:34
yoctozeptowhat td-agent thing?14:35
hrwhttps://github.com/fluent/fluentd/discussions/370414:36
hrwcentos stream 9 migration is on hold due to lack of many things14:37
hrwno elasticsearch, no grafana, no fluentd and some others14:37
hrwI updated patch adding opensearch image to cover x86-64/aarch64 but that needs k-a side too14:40
hrwmnasiadka: https://review.opendev.org/c/openstack/kolla/+/83711514:41
yoctozeptoall the more reason to just go debian-only and stop mentioning irrelevant issues :-)14:42
yoctozeptoanyhow, do any of you remember why we finally have not switched to fluent bit?14:42
mnasiadkaelasticsearch is moving to opensearch, so maybe that's a dependency14:42
mnasiadkayoctozepto: I think because we would need to rewrite the config or something similar14:43
mnasiadkaand there's no Monasca something for fluent bit14:43
yoctozeptomnasiadka: but you don't remember exactly? we have cleaned up the config greatly recently14:43
mnasiadkaI remember exactly we have no Monasca output plugin for fluent bit14:44
yoctozeptomnasiadka: I figured :D14:45
mnasiadkabut we might have to go for fluent bit before fluentd gets in a working shape on EL9/Ubuntu 22.0414:45
mnasiadka:(14:45
mnasiadka(if fluent bit works there)14:45
mnasiadkano grafana? huh14:46
yoctozeptografana should be easy14:46
yoctozeptoit's go14:46
yoctozeptomaybe it's just not packaged14:46
hrwfor grafana look https://github.com/grafana/grafana/issues/41036#event-639238617114:46
hrwit is packaged but gpg key uses sha1 which rhel9 dropped14:47
yoctozeptolol14:47
yoctozeptowhat a pity14:47
hrwsame with elasticsearch and some other keys14:47
mnasiadkathat's it for simple bump :D14:47
yoctozeptoindeed14:47
hrwit is listed in 'migrate to cs9' patch ;d14:48
yoctozeptobut it's an issue on their side, and easy to fix14:48
yoctozeptofluentd seems worse14:48
opendevreviewMarcin Juszkiewicz proposed openstack/kolla master: Drop qdrouterd image  https://review.opendev.org/c/openstack/kolla/+/83714114:48
hrwdropping it from k-a is more work14:49
yoctozeptoyeah, similarly for binary/source image split14:51
yoctozeptoa lot of cleaning up awaits us14:51
opendevreviewMarcin Juszkiewicz proposed openstack/kolla-ansible master: drop qdrouterd support  https://review.opendev.org/c/openstack/kolla-ansible/+/83714214:52
hrwand I have a patch in a queue to drop 'source-' from image names14:53
hrwbut that maybe for next cycle14:53
yoctozeptono, it sounds good for this cycle14:58
opendevreviewMarcin Juszkiewicz proposed openstack/kolla master: pin MarkupSafe to keep older Jinja working  https://review.opendev.org/c/openstack/kolla/+/83658714:58
hrwinstead of bumping jinja14:59
hrwyoctozepto: will wait for current queue to merge15:00
yoctozeptothat makes sense15:00
hrwmarkupsafe/jinja will be a problem then for centos ;(15:01
hrwyoctozepto: 'drop binary' queue has too complicated order to put patch on top ;d15:02
yoctozeptoack15:03
hrwCS9 patch can be reviewed despite merge conflict. It is on zuul files only.15:04
hrwafter 'docker: drop binary parts' merge we can cleanup it to make less layers in images by concat RUN lines15:07
yoctozeptoyeah, I feel the files are getting much more readable as well15:09
opendevreviewMark Goddard proposed openstack/kayobe master: ironic: default to ipxe booting  https://review.opendev.org/c/openstack/kayobe/+/83620315:10
opendevreviewMarcin Juszkiewicz proposed openstack/kolla master: build watcher images  https://review.opendev.org/c/openstack/kolla/+/83714515:10
hrwand another two patches abandoned15:17
yoctozeptowhere?15:20
hrwtripleo15:21
opendevreviewMarcin Juszkiewicz proposed openstack/kolla-ansible master: drop qdrouterd support  https://review.opendev.org/c/openstack/kolla-ansible/+/83714215:21
hrwno response since november so I do not care either15:22
hrwyoctozepto: I hope to cover all now15:22
yoctozeptothanks15:22
hrwand markupsafe abandoned as well15:24
hrwmove to cs9 is the only solution15:25
hrwand for this we have patch15:25
* hrw out15:26
hrwhave a nice weekend everyone15:26
yoctozeptohave a nice weekend15:36
opendevreviewMerged openstack/kolla-ansible master: tests: use source instead of binary  https://review.opendev.org/c/openstack/kolla-ansible/+/83665215:45
opendevreviewMerged openstack/kolla-ansible master: drop binary install type from templates and config  https://review.opendev.org/c/openstack/kolla-ansible/+/83665315:45
opendevreviewMerged openstack/kolla master: add releasenote about dropping binary images support  https://review.opendev.org/c/openstack/kolla/+/83711515:45
mgoddardno messing around with this binary drop15:48
yoctozeptoyup15:48
yoctozeptoseems many cores are interested in seeing them gone15:48
opendevreviewMarcin Juszkiewicz proposed openstack/kolla master: drop infra-rename support  https://review.opendev.org/c/openstack/kolla/+/83657615:55
yoctozeptomgoddard: I hope you came to approve ironic changes ;p15:57
mgoddardyoctozepto: once we have a passing, correct kayobe change15:57
mgoddardbecause otherwise it breaks us15:57
yoctozeptomgoddard: which one you mean?15:57
mgoddardthe one you have reviewed15:58
yoctozeptomgoddard: ah; well, now you know legacy pxe works15:58
mgoddardindeed15:58
yoctozeptojust need to wait for ipxe results15:58
yoctozeptook15:58
*** amoralej is now known as amoralej|off16:43
yoctozeptomgoddard: it16:56
yoctozeptomgoddard: is16:57
yoctozeptomgoddard: green16:57
yoctozeptomgoddard: now you merge :D16:57
opendevreviewMerged openstack/kayobe stable/xena: Fix Ansible inventory generation when reusing group names  https://review.opendev.org/c/openstack/kayobe/+/83705617:23
opendevreviewMerged openstack/kayobe stable/wallaby: Fix Ansible inventory generation when reusing group names  https://review.opendev.org/c/openstack/kayobe/+/83705717:24
opendevreviewMerged openstack/kayobe stable/victoria: Fix Ansible inventory generation when reusing group names  https://review.opendev.org/c/openstack/kayobe/+/83705817:24
opendevreviewJames Kirsch proposed openstack/kolla master: Replace Certbot with Lego for Let's Encrypt container  https://review.opendev.org/c/openstack/kolla/+/83421119:12
opendevreviewTim Shearer proposed openstack/kolla master: Adjust file permissions on _extend_start files.  https://review.opendev.org/c/openstack/kolla/+/83621220:48
opendevreviewTim Shearer proposed openstack/kolla master: Adjust permissions on _extend_start files.  https://review.opendev.org/c/openstack/kolla/+/83621220:49

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!