| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Introduce oneshot docker_restart_policy https://review.opendev.org/c/openstack/kolla-ansible/+/881707 | 06:56 |
|---|---|---|
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Introduce oneshot docker_restart_policy https://review.opendev.org/c/openstack/kolla-ansible/+/881707 | 06:56 |
| mmessah | Hi everyone. "fluentd crash - elasticsearch version mismatch" issue is also happened for Zed. This bug seem resolved for Yoga. Was there a plan to get the changes to zed release? | 07:11 |
| mmessah | https://bugs.launchpad.net/kolla/+bug/2018463 | 07:11 |
| mnasiadka | There is no elasticsearch role in Zed | 07:29 |
| mnasiadka | We don't support elasticsearch in Zed, only OpenSearch | 07:29 |
| mmalchuk | mnasiadka please +W https://review.opendev.org/c/openstack/kolla/+/890302 | 07:30 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: systemd: handle running container without systemd unit https://review.opendev.org/c/openstack/kolla-ansible/+/890198 | 07:30 |
| mmalchuk | there need backports | 07:30 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Introduce oneshot docker_restart_policy https://review.opendev.org/c/openstack/kolla-ansible/+/881707 | 07:30 |
| mnasiadka | mmalchuk: done | 07:31 |
| mmalchuk | thank you | 07:31 |
| mnasiadka | bbezak, frickler: https://review.opendev.org/c/openstack/kolla-ansible/+/890198 needs reviews - fixes systemd issue with MariaDB upgrade | 07:32 |
| mnasiadka | wondering if we should drop force=True instead of that loop with waiting for container to be stopped | 07:43 |
| SvenKieske | o/ back from "vacation" aka moving my household | 07:51 |
| mnasiadka | SvenKieske: we call that "polish vacation" in Poland ;) | 07:54 |
| SvenKieske | ah, really? nice to know :D :) | 07:55 |
| SvenKieske | well I'm still on a temporary desk setup until my new desk arrives next week | 07:56 |
| mmessah | @mnsiadka we are not deploying ElasticSearch with Kolla Ansible, we use external elasticsearch that we created an elasticsearch cluster outside of openstack. we defined our cluster with `elasticsearch_address` parameter. Our elasticsearch cluster version is 7.x but fluentd needs elasticsearch 8.x version. Is there any way to continue without upgrading our elasticsearch cluster? | 08:16 |
| mnasiadka | mmessah: build the fluentd image on your own and downgrade elasticsearch plugin version there (using template overrides) | 08:42 |
| opendevreview | howardlee proposed openstack/kolla master: Adding db sync script for venus https://review.opendev.org/c/openstack/kolla/+/890512 | 08:52 |
| yusufgungor | Hi everyone, https://bugs.launchpad.net/kolla-ansible/+bug/1992153 removes the show_multiple_locations parameter which kolla-ansible sets to True until Zed. On our environment this caused to creating image sourced volume duration from 5 seconds to 2 minutes or more. We have to override this config. Is this duration acceptable for others? | 09:10 |
| mmessah | @mnasiadka thanks for reply, we work on this, maybe we can use fluentd release for yoga :) | 09:31 |
| mnasiadka | mmessah: should work | 09:42 |
| mmalchuk | yusufgungor this question for business mostly. did they leverage with security issue in favor of time to market?) | 09:46 |
| mmalchuk | yusufgungor here almost the technical people only | 09:48 |
| mmalchuk | as for me 2 minutes is not so long in case of security issue) | 09:49 |
| opendevreview | Verification of a change to openstack/kolla master failed: Revert "Disable build of Gnocchi containers" https://review.opendev.org/c/openstack/kolla/+/890302 | 09:52 |
| SvenKieske | mnasiadka: _how_ was this triaged? https://bugs.launchpad.net/bugs/2027668 | 09:56 |
| mnasiadka | SvenKieske: read the end of the first line, how can we not accept this as a bug? :) | 09:56 |
| SvenKieske | ah my bad, I always mix the english meaning of triage with a different meaning in the german direct translation, which is something like "It's already dealt with" :D | 09:59 |
| mmalchuk | SvenKieske are you from german? | 10:07 |
| SvenKieske | mmalchuk: yes | 10:10 |
| opendevreview | Jan Gutter proposed openstack/kolla-ansible master: [WIP] etcd: update to v3.4 https://review.opendev.org/c/openstack/kolla-ansible/+/890465 | 10:10 |
| mmalchuk | SvenKieske thats cool. more people... from around the world | 10:11 |
| jangutter | SvenKiese: at least the term "to table" didn't become part of tech jargon: https://en.wikipedia.org/wiki/Table_(parliamentary_procedure) | 10:18 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: systemd: handle running container without systemd unit https://review.opendev.org/c/openstack/kolla-ansible/+/890198 | 10:19 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Introduce oneshot docker_restart_policy https://review.opendev.org/c/openstack/kolla-ansible/+/881707 | 10:20 |
| jangutter | (when triage's medical meaning is applied in English, it usually means people are dying... choosing it as a common term in tech is particularly ironic) | 10:22 |
| opendevreview | howardlee proposed openstack/kolla master: Adding db sync script for venus https://review.opendev.org/c/openstack/kolla/+/890512 | 10:25 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: systemd: handle running container without systemd unit https://review.opendev.org/c/openstack/kolla-ansible/+/890198 | 10:25 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Introduce oneshot docker_restart_policy https://review.opendev.org/c/openstack/kolla-ansible/+/881707 | 10:26 |
| mmalchuk | In software engineering, triaging is a process where each bug is prioritized on its severity, frequency, risk, impact on the team, etc. The goal of triage is to evaluate, prioritize, and assign the resolution of bugs or defects. | 10:27 |
| mmalchuk | why software engineering compared with medical?) | 10:28 |
| mmalchuk | triaging != sorting | 10:28 |
| mmalchuk | Triage is when hospitals sort out injured people according to their need for emergency medical attention to determine who gets care first. | 10:30 |
| mmalchuk | so in medical its kind of prioritize too | 10:30 |
| mmalchuk | may be I'm wrong? correct me | 10:31 |
| jangutter | You're not wrong: but usually in the medical sense, "triage" means making life or death decisions: "one of these folks gets a better chance to live, pick one" | 10:59 |
| jangutter | In software engineering "triage" is used because you're always operating without enough resources to fix all bugs. | 11:01 |
| jangutter | In medicine, when you're using the term "triage" it usually means something in the system has gone wrong. | 11:02 |
| opendevreview | Verification of a change to openstack/kolla master failed: Revert "Disable build of Gnocchi containers" https://review.opendev.org/c/openstack/kolla/+/890302 | 11:05 |
| jangutter | ... (in my opinion, this could have been solved by calling it "classify" rather than "triage") | 11:07 |
| yusufgungor | @mmalchuk you are right but at least can we make show_multiple_locations as a configuration parameter for kolla-ansible? I think the people who uses openstack as private cloud can take that security risk. We can also add a note about the slowness effect if not setting this variable to True. | 11:19 |
| yusufgungor | Glance config reference said that "...is deprecated for removal since Newton...Please keep an eye on the Glance release notes to stay up to date on progress in addressing this issue." | 11:19 |
| yusufgungor | since newton so there is no update since 2016 :( | 11:19 |
| yusufgungor | It is ok for us, we know the parameter, it is effects and override it but removing this parameter from kolla-ansible without any descriptive note can create bad user experience for newcomers. | 11:19 |
| mmalchuk | yusufgungor I agree with adding a note to the configuration. feel free with review | 11:21 |
| priteau | Anyone able to +2 this doc fix? https://review.opendev.org/c/openstack/kolla-ansible/+/881455 | 11:21 |
| priteau | Actually it has two +2 already | 11:22 |
| priteau | Just need the W+1 | 11:22 |
| priteau | bbezak maybe? | 11:22 |
| mnasiadka | had only one, but merging | 11:25 |
| mnasiadka | *it's merging | 11:25 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Introduce oneshot docker_restart_policy https://review.opendev.org/c/openstack/kolla-ansible/+/881707 | 11:26 |
| opendevreview | Michal Nasiadka proposed openstack/kayobe master: Fix CentOS / Rocky route options https://review.opendev.org/c/openstack/kayobe/+/889680 | 11:27 |
| yusufgungor | @mmalchuk is it ok for adding a configuration parameter to globals.yml with a note? | 11:29 |
| mmalchuk | no.. you always can use overrides. only note. | 11:30 |
| frickler | isn't that what https://review.opendev.org/c/openstack/kolla-ansible/+/881455/2/doc/source/reference/storage/external-ceph-guide.rst already has? | 11:38 |
| frickler | might get a bit more explicit about the performance effect of copy-on-write, though | 11:39 |
| mnasiadka | seems rocky9 mirror went crazy, appstream repomd is garbage | 11:40 |
| yusufgungor | @mmalchuk adding the note to which file? | 11:47 |
| opendevreview | Merged openstack/kolla-ansible master: Correct glance.conf example in external Ceph guide https://review.opendev.org/c/openstack/kolla-ansible/+/881455 | 11:49 |
| mmalchuk | yusufgungor check the code by Radosław, and do followup | 11:49 |
| mmalchuk | yusufgungor ansible/roles/glance/templates/glance-api.conf.j2 | 11:50 |
| yusufgungor | @mmalchuk ok thanks | 11:50 |
| mmalchuk | yusufgungor something like 'lets revert the change but add a note instead completely remove the option' | 11:53 |
| mmalchuk | lets operators decide | 11:54 |
| opendevreview | Verification of a change to openstack/kolla master failed: Revert "Disable build of Gnocchi containers" https://review.opendev.org/c/openstack/kolla/+/890302 | 12:39 |
| priteau | mnasiadka: there was a +2 from Mark on patch set 1, patch set 2 is just merge conflict fixed | 12:55 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: systemd: handle running container without systemd unit https://review.opendev.org/c/openstack/kolla-ansible/+/890198 | 13:15 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Introduce oneshot docker_restart_policy https://review.opendev.org/c/openstack/kolla-ansible/+/881707 | 13:15 |
| mnasiadka | priteau: well, it was not copied, so it didn't matter ;-) | 13:27 |
| opendevreview | Sven Kieske proposed openstack/kolla-ansible stable/2023.1: Fix OpenSearch Dashboards health check https://review.opendev.org/c/openstack/kolla-ansible/+/890483 | 14:15 |
| opendevreview | Sven Kieske proposed openstack/kolla-ansible stable/zed: Fix OpenSearch Dashboards health check https://review.opendev.org/c/openstack/kolla-ansible/+/890484 | 14:15 |
| opendevreview | Sven Kieske proposed openstack/kolla-ansible stable/yoga: Fix OpenSearch Dashboards health check https://review.opendev.org/c/openstack/kolla-ansible/+/890485 | 14:16 |
| jps_whitestack | Hello! was wandering if any core reviewers are available to check out my k-a proposal: https://review.opendev.org/c/openstack/kolla-ansible/+/875983 | 14:16 |
| opendevreview | Verification of a change to openstack/kolla master failed: Revert "Disable build of Gnocchi containers" https://review.opendev.org/c/openstack/kolla/+/890302 | 14:19 |
| jps_whitestack | It configures Fluentd's plugin systemd to read logs from /var/log/journal | 14:19 |
| SvenKieske | I'm no core, but I commented anyway :) | 14:26 |
| jps_whitestack | Thank you ! | 14:33 |
| opendevreview | Juan Pablo Suazo proposed openstack/kolla-ansible master: Enable the Fluentd Plugin Systemd https://review.opendev.org/c/openstack/kolla-ansible/+/875983 | 14:37 |
| jps_whitestack | I have addressed your comments, thanks ! | 14:38 |
| opendevreview | Juan Pablo Suazo proposed openstack/kolla-ansible master: Configures the tap-as-a-service neutron plugin https://review.opendev.org/c/openstack/kolla-ansible/+/885417 | 14:42 |
| jps_whitestack | Also, any core reviewers who could take a look at https://review.opendev.org/c/openstack/kolla-ansible/+/885417 would be appreciated ! It's part of a group of proposals to kolla and kolla-ansible to install and configure TAAS, a neutron plugin which allows port mirroring on openstack ports. | 14:46 |
| opendevreview | Merged openstack/kolla master: Revert "Disable build of Gnocchi containers" https://review.opendev.org/c/openstack/kolla/+/890302 | 15:52 |
| opendevreview | Maksim Malchuk proposed openstack/kolla stable/2023.1: Revert "Disable build of Gnocchi containers" https://review.opendev.org/c/openstack/kolla/+/890538 | 16:24 |
| opendevreview | Maksim Malchuk proposed openstack/kolla stable/zed: Revert "Disable build of Gnocchi containers" https://review.opendev.org/c/openstack/kolla/+/890539 | 16:25 |
| opendevreview | Maksim Malchuk proposed openstack/kolla stable/yoga: Revert "Disable build of Gnocchi containers" https://review.opendev.org/c/openstack/kolla/+/890540 | 16:27 |
| opendevreview | Maksim Malchuk proposed openstack/kolla stable/xena: Revert "Disable build of Gnocchi containers" https://review.opendev.org/c/openstack/kolla/+/890541 | 16:28 |
| opendevreview | Juan Pablo Suazo proposed openstack/kolla-ansible master: Enable the Fluentd Plugin Systemd https://review.opendev.org/c/openstack/kolla-ansible/+/875983 | 21:24 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!