yoctozepto | mgoddard: day to merge the ironic patches :-) | 06:47 |
---|---|---|
opendevreview | Merged openstack/kolla master: CI: remove binary jobs https://review.opendev.org/c/openstack/kolla/+/836572 | 06:56 |
hrw | ho ho. no binary world started | 07:02 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: tests: drop testing for binary install type https://review.opendev.org/c/openstack/kolla/+/836573 | 07:05 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: kolla: drop 'binary' install type https://review.opendev.org/c/openstack/kolla/+/836574 | 07:06 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: tests: drop testing for binary install type https://review.opendev.org/c/openstack/kolla/+/836573 | 07:06 |
*** amoralej|off is now known as amoralej | 07:08 | |
hrw | INFO:kolla.common.utils.base:Successfully tagged short-name-1/debian-base:14.1.0 | 07:18 |
hrw | world is not yet ready for such names | 07:18 |
hrw | short-name-1/debian-elasticsearch-curator:14.1.0 etc | 07:19 |
frickler | hrw: 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 missing | 07:28 |
opendevreview | Mark Goddard proposed openstack/kayobe stable/xena: Fix Ansible inventory generation when reusing group names https://review.opendev.org/c/openstack/kayobe/+/837056 | 08:37 |
opendevreview | Mark Goddard proposed openstack/kayobe stable/wallaby: Fix Ansible inventory generation when reusing group names https://review.opendev.org/c/openstack/kayobe/+/837057 | 08:38 |
opendevreview | Mark Goddard proposed openstack/kayobe stable/victoria: Fix Ansible inventory generation when reusing group names https://review.opendev.org/c/openstack/kayobe/+/837058 | 08:38 |
opendevreview | Merged openstack/kolla-ansible master: CI: remove binary jobs https://review.opendev.org/c/openstack/kolla-ansible/+/836580 | 08:55 |
opendevreview | Mark Goddard proposed openstack/kayobe master: ironic: revert to ironic's default drivers & interfaces https://review.opendev.org/c/openstack/kayobe/+/836999 | 09:07 |
opendevreview | Radosław Piliszek proposed openstack/kolla master: kolla: drop 'binary' install type https://review.opendev.org/c/openstack/kolla/+/836574 | 09:45 |
opendevreview | Radosław Piliszek proposed openstack/kolla master: drop add_binary_source_envs() macro https://review.opendev.org/c/openstack/kolla/+/836575 | 09:47 |
opendevreview | Radosław Piliszek proposed openstack/kolla master: drop infra-rename support https://review.opendev.org/c/openstack/kolla/+/836576 | 09:47 |
opendevreview | Radosław Piliszek proposed openstack/kolla master: docker: drop binary parts https://review.opendev.org/c/openstack/kolla/+/836577 | 09:47 |
yoctozepto | frickler: revote https://review.opendev.org/c/openstack/kolla/+/836574/3 ? :-) | 09:48 |
frickler | yoctozepto squashmaster ;) | 09:53 |
yoctozepto | frickler: :-) | 09:53 |
opendevreview | Radosław Piliszek proposed openstack/kolla master: drop infra-rename support https://review.opendev.org/c/openstack/kolla/+/836576 | 09:54 |
opendevreview | Radosław Piliszek proposed openstack/kolla master: docker: drop binary parts https://review.opendev.org/c/openstack/kolla/+/836577 | 09:55 |
opendevreview | Radosław Piliszek proposed openstack/kolla master: drop add_binary_source_envs() macro https://review.opendev.org/c/openstack/kolla/+/836575 | 09:55 |
opendevreview | Radosław Piliszek proposed openstack/kolla master: docker: drop binary parts https://review.opendev.org/c/openstack/kolla/+/836577 | 09:58 |
opendevreview | Radosław Piliszek proposed openstack/kolla master: drop add_binary_source_envs() macro https://review.opendev.org/c/openstack/kolla/+/836575 | 09:58 |
yoctozepto | the order should be fine now | 09:59 |
yoctozepto | I reviewed all but the biggest one | 09:59 |
opendevreview | Mark Goddard proposed openstack/kayobe master: ironic: default to ipxe booting https://review.opendev.org/c/openstack/kayobe/+/836203 | 10:01 |
hrw | morning | 10:14 |
hrw | debian-nova-libvirt - I like it. but patch to have shorter names will wait | 10:17 |
opendevreview | Merged openstack/kayobe master: Restore forgotten linuxbridge-agent container https://review.opendev.org/c/openstack/kayobe/+/837033 | 10:30 |
hrw | migration to centos stream 9 will take a while ;( | 10:34 |
hrw | https://github.com/fluent/fluentd/discussions/3704 | 10:34 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: drop infra_image_prefix https://review.opendev.org/c/openstack/kolla/+/837114 | 10:39 |
hrw | yoctozepto: here you have | 10:39 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: add releasenote about dropping binary images support https://review.opendev.org/c/openstack/kolla/+/837115 | 10:43 |
frickler | time 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 |
frickler | current usage in k-a seems to be wildly mixed | 11:39 |
yoctozepto | that'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 order | 11:40 |
frickler | (we just had this discussion downstream and I'm now seeking for general wisdom) | 11:40 |
hrw | I usually use it as first | 11:58 |
*** amoralej is now known as amoralej|lunch | 12:41 | |
opendevreview | Merged openstack/kolla master: kolla: drop 'binary' install type https://review.opendev.org/c/openstack/kolla/+/836574 | 13:09 |
*** amoralej|lunch is now known as amoralej | 13:24 | |
hrw | yay | 13:26 |
hrw | no more binary possible | 13:26 |
kevko | :P | 13:29 |
hrw | https://review.opendev.org/q/topic:zed-remove-support-for-binary-images waits for reviews | 13:30 |
mnasiadka | hrw: did some ;-) | 14:31 |
hrw | replied | 14:31 |
hrw | fun to see how they go in ;d | 14:32 |
mnasiadka | but that's some incosistency on our side, we allow users to override aodh_base_packages but not aodh_api_packages | 14:32 |
hrw | sent patch | 14:33 |
hrw | ;D | 14:34 |
mnasiadka | hrw: that td-agent thing is a bit worrying - I assume gem install fluentd would cause the same? | 14:34 |
mnasiadka | I mean hit the same problem, with old Ruby and openssl | 14:34 |
yoctozepto | what td-agent thing? | 14:35 |
hrw | https://github.com/fluent/fluentd/discussions/3704 | 14:36 |
hrw | centos stream 9 migration is on hold due to lack of many things | 14:37 |
hrw | no elasticsearch, no grafana, no fluentd and some others | 14:37 |
hrw | I updated patch adding opensearch image to cover x86-64/aarch64 but that needs k-a side too | 14:40 |
hrw | mnasiadka: https://review.opendev.org/c/openstack/kolla/+/837115 | 14:41 |
yoctozepto | all the more reason to just go debian-only and stop mentioning irrelevant issues :-) | 14:42 |
yoctozepto | anyhow, do any of you remember why we finally have not switched to fluent bit? | 14:42 |
mnasiadka | elasticsearch is moving to opensearch, so maybe that's a dependency | 14:42 |
mnasiadka | yoctozepto: I think because we would need to rewrite the config or something similar | 14:43 |
mnasiadka | and there's no Monasca something for fluent bit | 14:43 |
yoctozepto | mnasiadka: but you don't remember exactly? we have cleaned up the config greatly recently | 14:43 |
mnasiadka | I remember exactly we have no Monasca output plugin for fluent bit | 14:44 |
yoctozepto | mnasiadka: I figured :D | 14:45 |
mnasiadka | but we might have to go for fluent bit before fluentd gets in a working shape on EL9/Ubuntu 22.04 | 14:45 |
mnasiadka | :( | 14:45 |
mnasiadka | (if fluent bit works there) | 14:45 |
mnasiadka | no grafana? huh | 14:46 |
yoctozepto | grafana should be easy | 14:46 |
yoctozepto | it's go | 14:46 |
yoctozepto | maybe it's just not packaged | 14:46 |
hrw | for grafana look https://github.com/grafana/grafana/issues/41036#event-6392386171 | 14:46 |
hrw | it is packaged but gpg key uses sha1 which rhel9 dropped | 14:47 |
yoctozepto | lol | 14:47 |
yoctozepto | what a pity | 14:47 |
hrw | same with elasticsearch and some other keys | 14:47 |
mnasiadka | that's it for simple bump :D | 14:47 |
yoctozepto | indeed | 14:47 |
hrw | it is listed in 'migrate to cs9' patch ;d | 14:48 |
yoctozepto | but it's an issue on their side, and easy to fix | 14:48 |
yoctozepto | fluentd seems worse | 14:48 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: Drop qdrouterd image https://review.opendev.org/c/openstack/kolla/+/837141 | 14:48 |
hrw | dropping it from k-a is more work | 14:49 |
yoctozepto | yeah, similarly for binary/source image split | 14:51 |
yoctozepto | a lot of cleaning up awaits us | 14:51 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla-ansible master: drop qdrouterd support https://review.opendev.org/c/openstack/kolla-ansible/+/837142 | 14:52 |
hrw | and I have a patch in a queue to drop 'source-' from image names | 14:53 |
hrw | but that maybe for next cycle | 14:53 |
yoctozepto | no, it sounds good for this cycle | 14:58 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: pin MarkupSafe to keep older Jinja working https://review.opendev.org/c/openstack/kolla/+/836587 | 14:58 |
hrw | instead of bumping jinja | 14:59 |
hrw | yoctozepto: will wait for current queue to merge | 15:00 |
yoctozepto | that makes sense | 15:00 |
hrw | markupsafe/jinja will be a problem then for centos ;( | 15:01 |
hrw | yoctozepto: 'drop binary' queue has too complicated order to put patch on top ;d | 15:02 |
yoctozepto | ack | 15:03 |
hrw | CS9 patch can be reviewed despite merge conflict. It is on zuul files only. | 15:04 |
hrw | after 'docker: drop binary parts' merge we can cleanup it to make less layers in images by concat RUN lines | 15:07 |
yoctozepto | yeah, I feel the files are getting much more readable as well | 15:09 |
opendevreview | Mark Goddard proposed openstack/kayobe master: ironic: default to ipxe booting https://review.opendev.org/c/openstack/kayobe/+/836203 | 15:10 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: build watcher images https://review.opendev.org/c/openstack/kolla/+/837145 | 15:10 |
hrw | and another two patches abandoned | 15:17 |
yoctozepto | where? | 15:20 |
hrw | tripleo | 15:21 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla-ansible master: drop qdrouterd support https://review.opendev.org/c/openstack/kolla-ansible/+/837142 | 15:21 |
hrw | no response since november so I do not care either | 15:22 |
hrw | yoctozepto: I hope to cover all now | 15:22 |
yoctozepto | thanks | 15:22 |
hrw | and markupsafe abandoned as well | 15:24 |
hrw | move to cs9 is the only solution | 15:25 |
hrw | and for this we have patch | 15:25 |
* hrw out | 15:26 | |
hrw | have a nice weekend everyone | 15:26 |
yoctozepto | have a nice weekend | 15:36 |
opendevreview | Merged openstack/kolla-ansible master: tests: use source instead of binary https://review.opendev.org/c/openstack/kolla-ansible/+/836652 | 15:45 |
opendevreview | Merged openstack/kolla-ansible master: drop binary install type from templates and config https://review.opendev.org/c/openstack/kolla-ansible/+/836653 | 15:45 |
opendevreview | Merged openstack/kolla master: add releasenote about dropping binary images support https://review.opendev.org/c/openstack/kolla/+/837115 | 15:45 |
mgoddard | no messing around with this binary drop | 15:48 |
yoctozepto | yup | 15:48 |
yoctozepto | seems many cores are interested in seeing them gone | 15:48 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: drop infra-rename support https://review.opendev.org/c/openstack/kolla/+/836576 | 15:55 |
yoctozepto | mgoddard: I hope you came to approve ironic changes ;p | 15:57 |
mgoddard | yoctozepto: once we have a passing, correct kayobe change | 15:57 |
mgoddard | because otherwise it breaks us | 15:57 |
yoctozepto | mgoddard: which one you mean? | 15:57 |
mgoddard | the one you have reviewed | 15:58 |
yoctozepto | mgoddard: ah; well, now you know legacy pxe works | 15:58 |
mgoddard | indeed | 15:58 |
yoctozepto | just need to wait for ipxe results | 15:58 |
yoctozepto | ok | 15:58 |
*** amoralej is now known as amoralej|off | 16:43 | |
yoctozepto | mgoddard: it | 16:56 |
yoctozepto | mgoddard: is | 16:57 |
yoctozepto | mgoddard: green | 16:57 |
yoctozepto | mgoddard: now you merge :D | 16:57 |
opendevreview | Merged openstack/kayobe stable/xena: Fix Ansible inventory generation when reusing group names https://review.opendev.org/c/openstack/kayobe/+/837056 | 17:23 |
opendevreview | Merged openstack/kayobe stable/wallaby: Fix Ansible inventory generation when reusing group names https://review.opendev.org/c/openstack/kayobe/+/837057 | 17:24 |
opendevreview | Merged openstack/kayobe stable/victoria: Fix Ansible inventory generation when reusing group names https://review.opendev.org/c/openstack/kayobe/+/837058 | 17:24 |
opendevreview | James Kirsch proposed openstack/kolla master: Replace Certbot with Lego for Let's Encrypt container https://review.opendev.org/c/openstack/kolla/+/834211 | 19:12 |
opendevreview | Tim Shearer proposed openstack/kolla master: Adjust file permissions on _extend_start files. https://review.opendev.org/c/openstack/kolla/+/836212 | 20:48 |
opendevreview | Tim Shearer proposed openstack/kolla master: Adjust permissions on _extend_start files. https://review.opendev.org/c/openstack/kolla/+/836212 | 20:49 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!