| tafkamax | Good mornin | 07:43 |
|---|---|---|
| tafkamax | Any thoughts on file structure here: https://review.opendev.org/c/openstack/kolla-ansible/+/975901/comment/1bf50476_0af8947e/ ? | 07:45 |
| tafkamax | The KTBX (my new name for kolla-toolbox) is not very friendly with list datatypes. | 07:46 |
| tafkamax | In my latest iteration I tried the same that I did in the rules section previously: remote_ids: "{{ '{{ keystone_identity_provider_remote_ids }}' }}" | 07:47 |
| tafkamax | Maybe I need to try "from_json" again? | 07:47 |
| opendevreview | Taavi Ansper proposed openstack/kolla-ansible master: Fix keystone with IDP configured. https://review.opendev.org/c/openstack/kolla-ansible/+/975901 | 07:53 |
| tafkamax | My other thought would be to also try from_yaml. And then maybe try some kind of bracket destruction technique. As it is a string, then remove the brackets using some string feature. | 08:09 |
| thelounge87 | hi | 08:31 |
| thelounge87 | I propose this feature... https://review.opendev.org/c/openstack/kolla-ansible/+/977760?tab=comments | 08:32 |
| thelounge87 | The breif is that: | 08:35 |
| thelounge87 | 1. Currently the multiregion capability of the kolla-ansible is limited. In the sense it requires a single database and I believe the core team expects the deployers to design an out of kolla database which is available to all the regions. | 08:35 |
| thelounge87 | 2. And then modify kolla to use that cluster for all the regions thereby providing reliability etc. | 08:35 |
| tafkamax | I am interested in this feature an saw your reddit post. But yes it is good to discuss in the weekly meetings. | 08:36 |
| thelounge87 | 3. The main issue is that, this depends heavily on an external service -> Can cause breakage to the whole cluster in case of network failure and generally stuffy? | 08:37 |
| thelounge87 | Yep, took me long enough to find the IRC channel :) but now finally here | 08:37 |
| thelounge87 | I personally deal with a client whose cluster keeps breaking due to gigantic rtts of a "badly" designed deployment. And yeah, depending so heavily on VPN is kinda bad. | 08:39 |
| opendevreview | Seunghun Lee proposed openstack/kolla-ansible stable/2025.1: [2025.1 only] Support ProxySQL 3.0.x on 2025.1 https://review.opendev.org/c/openstack/kolla-ansible/+/974712 | 08:54 |
| thelounge87 | bbezakVishwa Mithra can we discuss the multregion feature thing now? | 09:22 |
| tafkamax | Hmm seems from_json isnt the answer aswell "msg": "Task failed: Finalization of task args for 'openstack.cloud.federation_idp' failed: Error while resolving value for 'remote_ids': The filter plugin 'ansible.builtin.from_json' failed: Expecting value: line 1 column 2 (char 1)" | 10:05 |
| *** jhorstmann is now known as Guest3831 | 10:06 | |
| tafkamax | Let's try something else :) | 10:14 |
| opendevreview | Grzegorz Koper proposed openstack/kayobe master: Add nmstate engine https://review.opendev.org/c/openstack/kayobe/+/974745 | 10:22 |
| opendevreview | Will Szumski proposed openstack/kayobe master: Adds kayobe_path variable https://review.opendev.org/c/openstack/kayobe/+/977845 | 10:37 |
| opendevreview | Will Szumski proposed openstack/kayobe master: Fix lookup plugins for external playbooks https://review.opendev.org/c/openstack/kayobe/+/978058 | 10:37 |
| opendevreview | Taavi Ansper proposed openstack/kolla-ansible master: Fix keystone with IDP configured. https://review.opendev.org/c/openstack/kolla-ansible/+/975901 | 11:27 |
| opendevreview | Seunghun Lee proposed openstack/kolla stable/2025.1: [2025.1 only] Build ProxySQL 3.0.x in 2025.1 https://review.opendev.org/c/openstack/kolla/+/974429 | 12:07 |
| opendevreview | Seunghun Lee proposed openstack/kolla-ansible stable/2025.1: [2025.1 only] Support ProxySQL 3.0.x on 2025.1 https://review.opendev.org/c/openstack/kolla-ansible/+/974712 | 12:15 |
| opendevreview | Taavi Ansper proposed openstack/kolla-ansible master: Fix keystone with IDP configured. https://review.opendev.org/c/openstack/kolla-ansible/+/975901 | 12:34 |
| tafkamax | OK | 13:43 |
| tafkamax | I have mastered KTBX syntax | 13:43 |
| tafkamax | for lists | 13:44 |
| tafkamax | you need to wrap them as json | 13:44 |
| tafkamax | and then json64decode them after the fact | 13:44 |
| bbezak | :) | 13:44 |
| tafkamax | Is it ok if I add this in the kolla-toolbox.py comments? | 13:46 |
| tafkamax | Will add an example under kolla-toolbox. | 13:51 |
| opendevreview | Taavi Ansper proposed openstack/kolla-ansible master: Fix keystone with IDP configured. https://review.opendev.org/c/openstack/kolla-ansible/+/975901 | 13:57 |
| tafkamax | Let's see if selenium test is finally happy. | 14:01 |
| tafkamax | I see there are still issues regarding the versions of some of the tools, but that is outside the scope of my patch. | 14:02 |
| tafkamax | If selenium is happy then I can reapply for review. | 14:02 |
| tafkamax | Lot's of linter errors as per usual. I am not very good with long lines and how to split them tbh. Lets see if it is okay to split them like that. | 14:24 |
| opendevreview | Taavi Ansper proposed openstack/kolla-ansible master: Fix keystone with IDP configured. https://review.opendev.org/c/openstack/kolla-ansible/+/975901 | 14:42 |
| opendevreview | Taavi Ansper proposed openstack/kolla-ansible master: Fix keystone with IDP configured. https://review.opendev.org/c/openstack/kolla-ansible/+/975901 | 17:05 |
| tafkamax | Seems federation test are passing. You can take a look again bbezak and others who have volunteered to take a peek. | 17:58 |
| tafkamax | Also I suppose release notes might be needed? | 18:55 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!