| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Add centos10s jobs for both x86 and aarch64 https://review.opendev.org/c/openstack/kolla-ansible/+/951751 | 07:08 |
|---|---|---|
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Bump instance creation timeout to 5 minutes https://review.opendev.org/c/openstack/kolla-ansible/+/952273 | 07:10 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Bump instance creation timeout to 5 minutes https://review.opendev.org/c/openstack/kolla-ansible/+/952273 | 07:11 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: tls: use ca-bundle.crt instead of ca-bundle.trust.crt https://review.opendev.org/c/openstack/kolla-ansible/+/952280 | 07:11 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: tls: use ca-bundle.crt instead of ca-bundle.trust.crt https://review.opendev.org/c/openstack/kolla-ansible/+/952280 | 07:11 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Add centos10s jobs for both x86 and aarch64 https://review.opendev.org/c/openstack/kolla-ansible/+/951751 | 07:11 |
| opendevreview | Pierre Riteau proposed openstack/kayobe stable/2025.1: CI: Remove installation of catatonit https://review.opendev.org/c/openstack/kayobe/+/954133 | 07:18 |
| opendevreview | Alex Welsh proposed openstack/kayobe master: Enable building of neutron-bgp-dragent container https://review.opendev.org/c/openstack/kayobe/+/954352 | 07:58 |
| opendevreview | Amir Hossein Ahmadi proposed openstack/kolla-ansible master: swift: reintroduce Swift role and restore deployment https://review.opendev.org/c/openstack/kolla-ansible/+/953576 | 08:04 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Add centos10s jobs for both x86 and aarch64 https://review.opendev.org/c/openstack/kolla-ansible/+/951751 | 08:59 |
| opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: redhat: Remove dbus-glib-devel from packages https://review.opendev.org/c/openstack/ansible-collection-kolla/+/954443 | 10:35 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Add centos10s jobs for both x86 and aarch64 https://review.opendev.org/c/openstack/kolla-ansible/+/951751 | 10:35 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Add TLS support for MariaDB Connections https://review.opendev.org/c/openstack/kolla-ansible/+/933924 | 11:38 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Add TLS support for MariaDB Connections https://review.opendev.org/c/openstack/kolla-ansible/+/933924 | 11:41 |
| kevko | bbezak: replied to your comments in https://review.opendev.org/c/openstack/kolla-ansible/+/933924 << | 11:44 |
| opendevreview | Amir Hossein Ahmadi proposed openstack/kolla-ansible master: swift: reintroduce Swift role and restore deployment https://review.opendev.org/c/openstack/kolla-ansible/+/953576 | 12:48 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Add centos10s jobs for both x86 and aarch64 https://review.opendev.org/c/openstack/kolla-ansible/+/951751 | 12:54 |
| frickler | mnasiadka: bbezak: no meeting today? (not that I'd complain) | 13:10 |
| amir58118 | Seems like it! Maybe a quiet week | 13:12 |
| kevko | vhi | 13:12 |
| kevko | hi | 13:13 |
| amir58118 | I've been working on the following topic over the past month. (( https://review.opendev.org/q/topic:"dev-swift-modernization" )) | 13:14 |
| amir58118 | kevko: I hope it will be ready for review in the coming days. | 13:14 |
| mnasiadka | frickler: I basically totally forgot | 13:15 |
| mnasiadka | If there's nobody really needing a meeting, we can just skip ;-) | 13:15 |
| amir58118 | +1 | 13:15 |
| kevko | i want revies :) | 13:43 |
| kevko | w | 13:43 |
| mnasiadka | kevko: for? | 14:04 |
| kevko | mainly https://review.opendev.org/c/openstack/kolla-ansible/+/933924 and https://review.opendev.org/c/openstack/kolla-ansible/+/952700 | 14:09 |
| Vii | https://review.opendev.org/c/openstack/kolla-ansible/+/953458 | 14:11 |
| Vii | Hey, is there a chance this will get approved? I think it's a great option for large environments to send logs to the nearest OpenSearch node instead of through Haproxy. | 14:11 |
| opendevreview | Merged openstack/kolla-ansible master: CI: Bump instance creation timeout to 5 minutes https://review.opendev.org/c/openstack/kolla-ansible/+/952273 | 14:15 |
| kevko | Vii: is it sending logs through lb haproxy ? | 14:21 |
| Vii | kevko: I've added a new option that lets you decide how you want to send logs. | 14:25 |
| Vii | The default is through haproxy, but you can set "fluentd_opensearch_destination: direct" to use OpenSearch nodes on FluentD agents instead of the haproxy LB address. I've included a full description in: doc/source/reference/logging-and-monitoring/central-logging-guide.rst | 14:25 |
| Vii | In large environments, it doesn't make sense to push 10GB of logs through Haproxy. If Haproxy fails, we don't have up-to-date logs. | 14:27 |
| Vii | See the Haproxy statistics. In my test environment where I do almost nothing, I have IN: 42GB OUTL: 17GB over the last 7 days. | 14:30 |
| Vii | https://imgur.com/a/hLMhsoc | 14:31 |
| kevko | Vii: It definitely cannot be the default load balancer ... and maybe not used at all ... because 'unnecessary data' like logs cannot affect the functionality of the service. On a large scale, this would kill OpenStack. | 14:34 |
| Vii | For me, there shouldn't be logs sent by LB either. But I don't want to change the current logic. | 14:36 |
| Vii | I would be happy if it went through, I would just set what I'm interested in in globals.yaml permanently :) | 14:37 |
| Vii | I would be happy if it went through, I would just set what I'm interested in in globals.yaml permanently :) | 14:37 |
| kevko | Vii: Haha, sorry .. I only looked at it on my mobile ... your patch is actually exactly what I'm talking about .. you're providing the option to influence this stupid behavior of Kolla by a variable | 14:39 |
| kevko | Vii: need to go ..but LGTM for now ... | 14:39 |
| Vii | or give me info in the comments to skip LB as default and I will make a correction :) | 14:40 |
| Vii | and completely delete the variable | 14:40 |
| kevko | Vii: Add Signed-off-by , and maybe change the behaviour and add upgrade section into reno that it was changed and old behaviour can be fullfilled by ..... | 14:41 |
| kevko | Vii: for me it sounds better to have default setting to send it directly instead of LB | 14:41 |
| mnasiadka | If shipping logs via LB is not the thing we want, why do we want to keep the old behaviour? | 14:42 |
| kevko | mnasiadka: as I said ... i would change it to direct ...but add reno with upgrade that there is new var and behaviour changed ... | 14:42 |
| mnasiadka | well, I wouldn't hide anything behind a variable, let's just change approach and that's it. | 14:43 |
| kevko | and action is required from user if he want to preserve LB path ... for whatever reason is .. | 14:43 |
| mnasiadka | If something doesn't need an LB - that's even better. | 14:43 |
| kevko | mnasiadka: why not ... I just thought that you will have different opinion | 14:44 |
| mnasiadka | Well, usually big changes need some alternatives, but this looks relatively small and non-problematic (unless there's a problem I'm not seeing) | 14:44 |
| kevko | ok ..need to go .. | 14:45 |
| kevko | i am on mobile | 14:45 |
| Vii | I'll send the changes tomorrow | 14:45 |
| Vii | thx | 14:45 |
| Vii | https://review.opendev.org/c/openstack/kolla-ansible/+/951127 Cherry pick of950311,5 - It would be good to accept this for the older version as well, because when users start updating, there will be a lot of reports | 14:47 |
| opendevreview | Verification of a change to openstack/kolla-ansible master failed: tls: use ca-bundle.crt instead of ca-bundle.trust.crt https://review.opendev.org/c/openstack/kolla-ansible/+/952280 | 14:59 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible stable/2025.1: CI: Bump instance creation timeout to 5 minutes https://review.opendev.org/c/openstack/kolla-ansible/+/954515 | 15:05 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible stable/2024.2: CI: Bump instance creation timeout to 5 minutes https://review.opendev.org/c/openstack/kolla-ansible/+/954516 | 15:05 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible stable/2024.1: CI: Bump instance creation timeout to 5 minutes https://review.opendev.org/c/openstack/kolla-ansible/+/954517 | 15:06 |
| kevko | mnasiadka: btw ..flavor change helped with upgrade jobs, right ? | 15:07 |
| mnasiadka | seems so | 15:07 |
| kevko | Cool | 15:07 |
| kevko | From time to time I checked and I had that feel ...so wanted to ask | 15:08 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: gnocchi: Remove from openstack fluentd services https://review.opendev.org/c/openstack/kolla-ansible/+/954127 | 15:21 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Add ceilometer scenario https://review.opendev.org/c/openstack/kolla-ansible/+/944276 | 15:22 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: gnocchi: Add support for using uWSGI https://review.opendev.org/c/openstack/kolla-ansible/+/940101 | 15:22 |
| opendevreview | Verification of a change to openstack/kayobe master failed: Enable building of neutron-bgp-dragent container https://review.opendev.org/c/openstack/kayobe/+/954352 | 16:39 |
| opendevreview | Verification of a change to openstack/kolla-ansible master failed: tls: use ca-bundle.crt instead of ca-bundle.trust.crt https://review.opendev.org/c/openstack/kolla-ansible/+/952280 | 16:51 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Add ceilometer scenario https://review.opendev.org/c/openstack/kolla-ansible/+/944276 | 19:28 |
| opendevreview | Verification of a change to openstack/kolla-ansible master failed: tls: use ca-bundle.crt instead of ca-bundle.trust.crt https://review.opendev.org/c/openstack/kolla-ansible/+/952280 | 19:35 |
| opendevreview | Verification of a change to openstack/kolla-ansible master failed: tls: use ca-bundle.crt instead of ca-bundle.trust.crt https://review.opendev.org/c/openstack/kolla-ansible/+/952280 | 22:41 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!