| opendevreview | Merged openstack/kolla-ansible master: Add /v3 suffix to OPENSTACK_KEYSTONE_URL https://review.opendev.org/c/openstack/kolla-ansible/+/924487 | 00:20 |
|---|---|---|
| opendevreview | Jens Harbott proposed openstack/kolla-ansible master: CI: improve get_logs.sh https://review.opendev.org/c/openstack/kolla-ansible/+/870521 | 05:56 |
| mnasiadka | morning | 06:06 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible stable/2024.1: Fix post-config of OVS for manila-share servers https://review.opendev.org/c/openstack/kolla-ansible/+/926028 | 06:07 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible stable/2023.2: Fix post-config of OVS for manila-share servers https://review.opendev.org/c/openstack/kolla-ansible/+/926029 | 06:10 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible stable/2023.1: Fix post-config of OVS for manila-share servers https://review.opendev.org/c/openstack/kolla-ansible/+/926030 | 06:11 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: WIP: horizon: Copy wsgi.py to /etc/openstack-dashboard/ https://review.opendev.org/c/openstack/kolla/+/925979 | 06:13 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Add venus-dashboard into horizon https://review.opendev.org/c/openstack/kolla-ansible/+/890637 | 06:15 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Add venus-dashboard into horizon https://review.opendev.org/c/openstack/kolla-ansible/+/890637 | 06:16 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Add venus-dashboard into horizon https://review.opendev.org/c/openstack/kolla-ansible/+/890637 | 06:16 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Add venus-dashboard into horizon https://review.opendev.org/c/openstack/kolla-ansible/+/890637 | 06:16 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Add venus-dashboard into horizon https://review.opendev.org/c/openstack/kolla-ansible/+/890637 | 06:17 |
| opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: docker/podman_sdk: Install python3-docker/podman when python3.11+ https://review.opendev.org/c/openstack/ansible-collection-kolla/+/925994 | 06:19 |
| opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: docker/podman_sdk: Install python3-docker/podman when python3.11+ https://review.opendev.org/c/openstack/ansible-collection-kolla/+/925994 | 06:19 |
| opendevreview | Franciszek Przewozny proposed openstack/kolla-ansible stable/2024.1: Add /v3 suffix to OPENSTACK_KEYSTONE_URL https://review.opendev.org/c/openstack/kolla-ansible/+/926031 | 06:49 |
| opendevreview | Franciszek Przewoźny proposed openstack/kolla-ansible stable/2024.1: Add /v3 suffix to OPENSTACK_KEYSTONE_URL https://review.opendev.org/c/openstack/kolla-ansible/+/926031 | 07:05 |
| opendevreview | Franciszek Przewoźny proposed openstack/kolla-ansible stable/2023.2: Add /v3 suffix to OPENSTACK_KEYSTONE_URL https://review.opendev.org/c/openstack/kolla-ansible/+/926032 | 07:12 |
| opendevreview | Franciszek Przewoźny proposed openstack/kolla-ansible stable/2023.1: Add /v3 suffix to OPENSTACK_KEYSTONE_URL https://review.opendev.org/c/openstack/kolla-ansible/+/926033 | 07:14 |
| opendevreview | Ivan Halomi proposed openstack/kolla-ansible master: Refactor of kolla_container_facts https://review.opendev.org/c/openstack/kolla-ansible/+/911417 | 07:36 |
| opendevreview | Jens Harbott proposed openstack/kolla-ansible master: Add fluentd_enable_watch_timer https://review.opendev.org/c/openstack/kolla-ansible/+/785309 | 07:38 |
| mnasiadka | frickler: https://review.opendev.org/c/openstack/kolla-ansible/+/925980 - seems this works, are we fine with me polishing those two patches and merging it? | 08:05 |
| mnasiadka | kevko: ^^ | 08:05 |
| mnasiadka | and bbezak ^^ | 08:05 |
| *** mrunge_ is now known as mrunge | 08:09 | |
| opendevreview | Verification of a change to openstack/kolla-ansible stable/2024.1 failed: Fix barbican's configuration https://review.opendev.org/c/openstack/kolla-ansible/+/923859 | 08:10 |
| frickler | mnasiadka: ack, sounds like a good idea to me, started with two review questions ;) | 08:17 |
| mnasiadka | frickler: replied ;) | 08:23 |
| opendevreview | Sven Kieske proposed openstack/kolla-ansible stable/2024.1: CI: Fix cases where port is not available yet https://review.opendev.org/c/openstack/kolla-ansible/+/926037 | 08:26 |
| opendevreview | Bartosz Bezak proposed openstack/kolla-ansible master: Restart OVS container after hw-offload change https://review.opendev.org/c/openstack/kolla-ansible/+/925970 | 08:27 |
| SvenKieske | kevko: could you please stop the "ping" comments? at least ping please only the people you want to ping, because I always waste like half a minute looking at a patchset I already reviewed, just to see you pinged someone else. thanks. | 08:31 |
| opendevreview | Ivan Halomi proposed openstack/kolla-ansible master: Move actions to kolla_container_facts https://review.opendev.org/c/openstack/kolla-ansible/+/911505 | 08:31 |
| SvenKieske | mnasiadka: so I guess the WIP can be removed? | 08:33 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: horizon: Use /etc/openstack-dashboard/wsgi.py https://review.opendev.org/c/openstack/kolla-ansible/+/925980 | 08:33 |
| opendevreview | Ivan Halomi proposed openstack/kolla-ansible master: Merge of container_facts modules https://review.opendev.org/c/openstack/kolla-ansible/+/912460 | 08:33 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: horizon: Copy wsgi.py to /etc/openstack-dashboard/ https://review.opendev.org/c/openstack/kolla/+/925979 | 08:34 |
| mnasiadka | SvenKieske: done | 08:34 |
| SvenKieske | and I agree with the comment about /etc/ | 08:35 |
| opendevreview | Ivan Halomi proposed openstack/kolla-ansible master: Add action for getting container names list https://review.opendev.org/c/openstack/kolla-ansible/+/924389 | 08:35 |
| opendevreview | Bartosz Bezak proposed openstack/kolla-ansible master: Restart OVS container after hw-offload change https://review.opendev.org/c/openstack/kolla-ansible/+/925970 | 08:36 |
| opendevreview | Bartosz Bezak proposed openstack/kolla-ansible master: Restart OVS container after hw-offload change https://review.opendev.org/c/openstack/kolla-ansible/+/925970 | 08:39 |
| SvenKieske | if someone got time, I think this is a very promising patchset: https://review.opendev.org/c/openstack/kolla-ansible/+/745164 (even if it's around for some time, it got some recent fixes) | 08:42 |
| opendevreview | Roman Krček proposed openstack/kolla-ansible master: Performance: Don't notify handlers during config https://review.opendev.org/c/openstack/kolla-ansible/+/745164 | 08:45 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: horizon: Copy wsgi.py to /etc/openstack-dashboard/ https://review.opendev.org/c/openstack/kolla/+/925979 | 08:48 |
| SvenKieske | maybe someone can reply here as well, we're a little stuck on how to best copy certs into containers: https://review.opendev.org/c/openstack/kolla-ansible/+/924651/comments/0d3f71c1_564a6f0a | 08:56 |
| kevko | SvenKieske: regarding yesterday discussion I think consul can be also added to Kolla code | 09:05 |
| kevko | SvenKieske: I also realized that corosync used in Kolla is BSD ...so why not consul which is BSD :) | 09:06 |
| mnasiadka | HashiCorp Consul is BSD? | 09:07 |
| mnasiadka | Since when? | 09:07 |
| kevko | mnasiadka: ah sorry, you are right ... That one letter confused me probably :-D | 09:10 |
| opendevreview | Roman Krček proposed openstack/kolla-ansible master: Refactor services' check-containers and optimise https://review.opendev.org/c/openstack/kolla-ansible/+/773243 | 09:27 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: horizon: Use /etc/openstack-dashboard/wsgi.py https://review.opendev.org/c/openstack/kolla-ansible/+/925980 | 09:41 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: horizon: Use /etc/openstack-dashboard/wsgi.py https://review.opendev.org/c/openstack/kolla-ansible/+/925980 | 09:42 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: horizon: Use versionless path for wsgi.py and static https://review.opendev.org/c/openstack/kolla-ansible/+/925980 | 09:42 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: horizon: Copy wsgi.py to /etc/openstack-dashboard/ https://review.opendev.org/c/openstack/kolla/+/925979 | 09:42 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: horizon: Use versionless path for wsgi.py and static https://review.opendev.org/c/openstack/kolla-ansible/+/925980 | 09:44 |
| opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: sdk: Stop enabling osbpo on Ubuntu https://review.opendev.org/c/openstack/ansible-collection-kolla/+/926044 | 09:50 |
| opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: docker/podman_sdk: Install python3-docker/podman when python3.11+ https://review.opendev.org/c/openstack/ansible-collection-kolla/+/925994 | 09:50 |
| SvenKieske | mnasiadka: any possibilty to drop the hard requirement for the ovn-exporter to connect only to split up components? https://review.opendev.org/c/openstack/kolla-ansible/+/855498 the upstream code currently simply doesn't allow to do that. | 09:50 |
| SvenKieske | I have linked some details in aboves changesets comments | 09:51 |
| mnasiadka | SvenKieske: how the hell it doesn't support it, since we use tcp: for connection? | 09:52 |
| SvenKieske | mnasiadka: this is the code that get's called, in the underlying ovsdb lib: https://github.com/greenpau/ovsdb/blob/ba2fa6bc5e69ee5511f72847d060e6709036ae0f/ovn.go#L44 | 09:53 |
| mnasiadka | it uses local socket directly? geez | 09:54 |
| SvenKieske | and it's not configurable to say "hey, I want you to only connect to ovs" "only connect to northbound client", no iot always connects all, and if it gets any error, well it errors out | 09:55 |
| SvenKieske | I guess I can rework that logic over time, but well that will take some time (golang n00b here, and not well versed in the ovsdb internals), and I don't know if upstream will even take my patch, because upstream did no commit this year :) | 09:56 |
| SvenKieske | and I don't know I can maintain that lib myself alone, I don't know shit about it, besides what I read in the code and in the RFC itself. | 09:56 |
| SvenKieske | I try and see if I can somehow motivate some people to maybe take over maintenance of ovn-exporter and the lib, you don't happen to have some spare golang/network devs around mnasiadka? :D | 10:05 |
| opendevreview | Merged openstack/kolla-ansible stable/2023.1: Add /v3 suffix to OPENSTACK_KEYSTONE_URL https://review.opendev.org/c/openstack/kolla-ansible/+/926033 | 10:08 |
| mnasiadka | SvenKieske: not really, and my cycles are limited :) | 10:20 |
| mnasiadka | but would be happy to get any metrics out of OVS/OVN finally | 10:20 |
| SvenKieske | yes, me too. maybe I can fix upstream up in some weekends work, but I would prefer to build out simple: support ovn setup with not everything spread out, and then add more features iteratively. would also make testing upstream changes way easier for me tbh. | 10:21 |
| SvenKieske | it's already really brittle to configure know, will not get prettier I guess. | 10:24 |
| opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: docker/podman_sdk: Install python3-docker/podman when python3.11+ https://review.opendev.org/c/openstack/ansible-collection-kolla/+/925994 | 10:26 |
| opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: docker/podman_sdk: Install python3-docker/podman when python3.11+ https://review.opendev.org/c/openstack/ansible-collection-kolla/+/925994 | 10:41 |
| kevko | do you also have gerrit so slow ? | 10:49 |
| kevko | kinda unresponsive ? | 10:49 |
| SvenKieske | what I observed in the last days, is that gerrit marked files as "unreviewed" which I 100% viewed on that same patchset already. very weird. | 10:50 |
| SvenKieske | it's not slower than usual though. large patchsets take some seconds on first load, then it's okayish I would say. | 10:51 |
| SvenKieske | 2.56 seconds for a site that consists mostly of text (on first load): https://review.opendev.org/c/openstack/kolla-ansible/+/745164 | 10:52 |
| SvenKieske | one of the largest chunks of that is actually TLS configuration (519 ms) | 10:54 |
| opendevreview | Michal Arbet proposed openstack/ansible-collection-kolla master: Add installation of dbus package https://review.opendev.org/c/openstack/ansible-collection-kolla/+/922614 | 10:54 |
| opendevreview | Merged openstack/kolla-ansible stable/2023.2: Add /v3 suffix to OPENSTACK_KEYSTONE_URL https://review.opendev.org/c/openstack/kolla-ansible/+/926032 | 11:12 |
| opendevreview | Verification of a change to openstack/ansible-collection-kolla master failed: sdk: Stop enabling osbpo on Ubuntu https://review.opendev.org/c/openstack/ansible-collection-kolla/+/926044 | 11:35 |
| opendevreview | Merged openstack/kolla-ansible master: Restart OVS container after hw-offload change https://review.opendev.org/c/openstack/kolla-ansible/+/925970 | 12:43 |
| opendevreview | Bartosz Bezak proposed openstack/kolla-ansible stable/2024.1: Restart OVS container after hw-offload change https://review.opendev.org/c/openstack/kolla-ansible/+/926058 | 12:59 |
| opendevreview | Bartosz Bezak proposed openstack/kolla-ansible stable/2024.1: Restart OVS container after hw-offload change https://review.opendev.org/c/openstack/kolla-ansible/+/926058 | 13:01 |
| opendevreview | Bartosz Bezak proposed openstack/kolla-ansible stable/2023.2: Restart OVS container after hw-offload change https://review.opendev.org/c/openstack/kolla-ansible/+/926059 | 13:02 |
| opendevreview | Bartosz Bezak proposed openstack/kolla-ansible stable/2023.1: Restart OVS container after hw-offload change https://review.opendev.org/c/openstack/kolla-ansible/+/926060 | 13:03 |
| opendevreview | Bartosz Bezak proposed openstack/kolla-ansible stable/2023.1: Restart OVS container after hw-offload change https://review.opendev.org/c/openstack/kolla-ansible/+/926060 | 13:03 |
| opendevreview | Bartosz Bezak proposed openstack/kolla-ansible stable/2023.2: Restart OVS container after hw-offload change https://review.opendev.org/c/openstack/kolla-ansible/+/926059 | 13:04 |
| opendevreview | Verification of a change to openstack/kolla-ansible stable/2024.1 failed: Add /v3 suffix to OPENSTACK_KEYSTONE_URL https://review.opendev.org/c/openstack/kolla-ansible/+/926031 | 13:27 |
| opendevreview | Merged openstack/kolla-ansible stable/2024.1: Add /v3 suffix to OPENSTACK_KEYSTONE_URL https://review.opendev.org/c/openstack/kolla-ansible/+/926031 | 18:00 |
| opendevreview | Maksim Malchuk proposed openstack/kolla master: Configure Apache services to use correct locale https://review.opendev.org/c/openstack/kolla/+/926084 | 18:24 |
| opendevreview | Maksim Malchuk proposed openstack/kolla master: Configure Apache services to use correct locale https://review.opendev.org/c/openstack/kolla/+/926084 | 18:30 |
| opendevreview | Maksim Malchuk proposed openstack/kolla master: Configure Apache services to use correct locale https://review.opendev.org/c/openstack/kolla/+/926084 | 18:36 |
| opendevreview | Verification of a change to openstack/ansible-collection-kolla master failed: sdk: Stop enabling osbpo on Ubuntu https://review.opendev.org/c/openstack/ansible-collection-kolla/+/926044 | 19:59 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!