| opendevreview | Roman Krček proposed openstack/kolla-ansible master: Refactoring: restructure swift role https://review.opendev.org/c/openstack/kolla-ansible/+/918040 | 07:11 |
|---|---|---|
| opendevreview | Roman Krček proposed openstack/kolla-ansible master: Refactoring: restructure swift role https://review.opendev.org/c/openstack/kolla-ansible/+/918040 | 07:34 |
| r-krcek | Hi. I was going through blueprints for kolla-ansible and I have noticed seemingly abandoned patchset for check-containers: https://review.opendev.org/c/openstack/kolla-ansible/+/599735 Any idea why that could be? The change looks that it could bring a new useful feature. | 07:52 |
| mnasiadka | we don't do blueprints for a long time | 07:52 |
| mnasiadka | if you want to pick this up - sure, but it's the end of C cycle - so don't expect it's going to be merged for Caracal | 07:53 |
| r-krcek | Yeah, if its not a problem, Id like to take a look at it. No problem if it doesnt make it to the release. | 07:55 |
| mnasiadka | sure, go for it | 07:57 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: kolla_url: port is a string https://review.opendev.org/c/openstack/kolla-ansible/+/916979 | 08:00 |
| r-krcek | And if I would like to look for new work in the future, the place to go is the launchpad wishlist since you dont do blueprints anymore? | 08:01 |
| opendevreview | Bartosz Bezak proposed openstack/kolla master: Refine release notes before kolla 2024.1 release https://review.opendev.org/c/openstack/kolla/+/918416 | 08:07 |
| opendevreview | Bartosz Bezak proposed openstack/ansible-collection-kolla master: Refine release notes before kolla 2024.1 release https://review.opendev.org/c/openstack/ansible-collection-kolla/+/918417 | 08:15 |
| opendevreview | Bartosz Bezak proposed openstack/ansible-collection-kolla master: Refine release notes before kolla 2024.1 release https://review.opendev.org/c/openstack/ansible-collection-kolla/+/918417 | 08:32 |
| mnasiadka | r-krcek: yes, you can create launchpad bug with [RFE] in the name, if you really want to - but it's not required, when you feel it's ready - feel free to come to the meeting, or add the patch for ,,expedited review'' in the whiteboard | 08:38 |
| Alex-Welsh | kevko & frickler, could you please take a look at this change: https://review.opendev.org/c/openstack/kolla-ansible/+/912420 I need a non-stackhpc opinion (ideally to merge into caracal) | 09:02 |
| opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: Refine release notes before kolla 2024.1 release https://review.opendev.org/c/openstack/ansible-collection-kolla/+/918417 | 09:38 |
| mattcrees | Morning all, I'm looking for a second core for this patch https://review.opendev.org/c/openstack/kolla-ansible/+/906766 . Should be trivial ;) | 09:52 |
| opendevreview | Bartosz Bezak proposed openstack/kolla-ansible master: Refine release notes before kolla 2024.1 release https://review.opendev.org/c/openstack/kolla-ansible/+/918425 | 10:02 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: kolla_url: port is a string https://review.opendev.org/c/openstack/kolla-ansible/+/916979 | 10:06 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: kolla_url: port is a string https://review.opendev.org/c/openstack/kolla-ansible/+/916979 | 10:06 |
| opendevreview | Uwe Jäger proposed openstack/kolla-ansible master: Fix link in reno https://review.opendev.org/c/openstack/kolla-ansible/+/916978 | 10:13 |
| opendevreview | Bartosz Bezak proposed openstack/kolla-ansible master: Refine release notes before kolla 2024.1 release https://review.opendev.org/c/openstack/kolla-ansible/+/918425 | 10:13 |
| opendevreview | Bartosz Bezak proposed openstack/kolla-ansible master: Refine release notes before kolla 2024.1 release https://review.opendev.org/c/openstack/kolla-ansible/+/918425 | 10:14 |
| IvanVnuko[m] | Hi, I'm working on adding TLS encryption of RabbitMQ management plugin (UI) communication to HAproxy. Are blueprints used? Or do I just create a change when ready? | 10:17 |
| mnasiadka | Just create a change | 10:18 |
| opendevreview | Bartosz Bezak proposed openstack/kolla-ansible master: Refine release notes before kolla 2024.1 release https://review.opendev.org/c/openstack/kolla-ansible/+/918425 | 10:44 |
| stan | I see kolla deploys ha proxy with L4 healtchecks, but it does not remove my unhealthy nova-api container from its load balancing pool, have you ever had this problem? | 10:50 |
| stan | mnasiadka any idea why it might be the case | 10:51 |
| mnasiadka | no | 10:52 |
| stan | Switching to L7 health check fixes it, but still unsure why the original issue occured | 10:53 |
| opendevreview | Bartosz Bezak proposed openstack/kolla-ansible master: Refine release notes before kolla 2024.1 release https://review.opendev.org/c/openstack/kolla-ansible/+/918425 | 10:58 |
| SvenKieske | stan: well depending on why it went unhealthy possibly it's still responsive on L4 and thus won't be removed from the lb pool, no? | 11:06 |
| opendevreview | Bartosz Bezak proposed openstack/kolla-ansible master: Refine release notes before kolla 2024.1 release https://review.opendev.org/c/openstack/kolla-ansible/+/918425 | 11:07 |
| SvenKieske | also normal tcp timeout is like 180 seconds, I'm not sure if we change this somewhere in kolla though. | 11:07 |
| *** parallax is now known as Guest4529 | 11:10 | |
| opendevreview | Merged openstack/kolla-ansible master: Configure Blazar with custom CA file https://review.opendev.org/c/openstack/kolla-ansible/+/906766 | 11:21 |
| opendevreview | Merged openstack/kolla-ansible master: Fix link in reno https://review.opendev.org/c/openstack/kolla-ansible/+/916978 | 11:21 |
| stan | SvenKieske Yes you're right, it still connects with TCP, but the api servers are not responding, I would assume it's best to do L7 checks to avoid these scenarios? No? | 11:24 |
| opendevreview | Pierre Riteau proposed openstack/kolla-ansible stable/2023.2: Configure Blazar with custom CA file https://review.opendev.org/c/openstack/kolla-ansible/+/918175 | 11:35 |
| opendevreview | Pierre Riteau proposed openstack/kolla-ansible stable/2023.1: Configure Blazar with custom CA file https://review.opendev.org/c/openstack/kolla-ansible/+/918176 | 11:36 |
| SvenKieske | stan: seems about right, maybe there is even already a patch floating around for this, I think I remember something in that direction at least.. | 11:37 |
| SvenKieske | mhm I don't find any change related to this, seems I'm misremembering | 11:39 |
| stan | SvenKieske I would like to contribute to the project as it has helped me immensely. Any guidance for a first timer? | 11:41 |
| SvenKieske | stan: sure: I assume you already know your way around git? do you also know gerrit? | 11:43 |
| stan | Git yes, but gerrit no | 11:43 |
| SvenKieske | the general kolla contribution guide is here, it has a lot of topics, so you might want to skip some: https://docs.openstack.org/kolla/latest/contributor/index.html | 11:43 |
| stan | SvenKieske Ah thanks, I will go through the docs and see if I can find my way around.. | 11:44 |
| SvenKieske | I'd suggest then to start with the basics of gerrit, because it's quite a different workflow than github etc. https://docs.openstack.org/contributors/code-and-documentation/using-gerrit.html | 11:44 |
| SvenKieske | this guides you through account setup: https://docs.openstack.org/contributors/common/setup-gerrit.html | 11:45 |
| SvenKieske | you can ping me here or via Mailing List for specific questions. | 11:45 |
| stan | Thank you very much 😊 sure will check with you for any assistance | 11:46 |
| SvenKieske | you're welcome :) always good to have new contributors. | 11:47 |
| opendevreview | Aravindh Murugesan proposed openstack/kolla-ansible master: HAProxy: Switch to L7 Healthchecks https://review.opendev.org/c/openstack/kolla-ansible/+/918437 | 12:21 |
| stan | SvenKieske it was easier to set up than I expected. | 12:21 |
| stan | thanks for the links. really helped | 12:22 |
| stan | SvenKieske Zuul review and +2 from core maintainers to merge? | 12:26 |
| SvenKieske | yes | 12:26 |
| stan | 😊👍 | 12:26 |
| SvenKieske | it would be nice if you could file a bug for the current behaviour at https://bugs.launchpad.net/kolla-ansible/ and link the bug number in the commit message. | 12:28 |
| stan | sure thing | 12:29 |
| SvenKieske | Like this: "Closes-Bug: #123456 (your bug id)" https://docs.openstack.org/contributors/common/git.html#footers | 12:30 |
| opendevreview | Aravindh Murugesan proposed openstack/kolla-ansible master: HAProxy: Switch to L7 Healthchecks https://review.opendev.org/c/openstack/kolla-ansible/+/918437 | 12:40 |
| opendevreview | Aravindh Murugesan proposed openstack/kolla-ansible master: HAProxy: Switch to L7 Healthchecks https://review.opendev.org/c/openstack/kolla-ansible/+/918437 | 12:40 |
| SvenKieske | thank you | 12:41 |
| stan | SvenKieske added it as you suggested | 12:41 |
| SvenKieske | yes, could you maybe add to the bugreport if this change fixes the issue for you, so did you already test it? Thanks! | 12:42 |
| opendevreview | Aravindh Murugesan proposed openstack/kolla-ansible master: HAProxy: Switch to L7 Healthchecks https://review.opendev.org/c/openstack/kolla-ansible/+/918437 | 12:47 |
| opendevreview | Roman Krček proposed openstack/kolla-ansible master: Check containers during kolla-ansible check https://review.opendev.org/c/openstack/kolla-ansible/+/599735 | 12:47 |
| stan | SvenKieske reviewed you suggestion on typo. | 12:50 |
| SvenKieske | thanks, already saw it :) | 12:52 |
| stan | SvenKieske I tested this change in my lab, no problem for the past week. I intentional brought down some container and haproxy marked it as unhealthy as expected.. | 12:53 |
| SvenKieske | nice! that's very valuable information, you could add it either to the bugreport or the commit message, or both :) | 12:56 |
| opendevreview | Aravindh Murugesan proposed openstack/kolla-ansible master: HAProxy: Switch to L7 Healthchecks https://review.opendev.org/c/openstack/kolla-ansible/+/918437 | 12:58 |
| stan | I added it to both now | 12:58 |
| SvenKieske | awesome, thanks | 12:59 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: Refine release notes before kolla 2024.1 release https://review.opendev.org/c/openstack/kolla/+/918416 | 14:45 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: Refine release notes before kolla 2024.1 release https://review.opendev.org/c/openstack/kolla/+/918416 | 14:57 |
| opendevreview | Pierre Riteau proposed openstack/kolla-ansible master: heat: set backups_enabled=False when cinder-backup is disabled https://review.opendev.org/c/openstack/kolla-ansible/+/918455 | 15:01 |
| opendevreview | Merged openstack/ansible-collection-kolla master: Refine release notes before kolla 2024.1 release https://review.opendev.org/c/openstack/ansible-collection-kolla/+/918417 | 15:03 |
| opendevreview | Merged openstack/kolla-ansible master: Refine release notes before kolla 2024.1 release https://review.opendev.org/c/openstack/kolla-ansible/+/918425 | 15:10 |
| PrzemekK | This will be fixed in 2024.1 ? https://bugs.launchpad.net/kolla-ansible/+bug/1989791 | 15:11 |
| SvenKieske | it's not merged, but as it is a bugfix I suppose it will be backported if it doesn't make the release | 15:13 |
| mnasiadka | nothing that is not merged right now will make the C release | 15:16 |
| mnasiadka | bbezak: is raising rc1 changes today | 15:17 |
| opendevreview | Merged openstack/kolla master: Refine release notes before kolla 2024.1 release https://review.opendev.org/c/openstack/kolla/+/918416 | 15:17 |
| mnasiadka | bbezak: ^^ fire off ;) | 15:17 |
| mnasiadka | we have less than one month until the whole Kolla projects family release, and I'd prefer to have it done as soon as possible so we can go back to merging features for D. | 15:17 |
| opendevreview | Pierre Riteau proposed openstack/kolla-ansible master: heat: set backups_enabled=False when cinder-backup is disabled https://review.opendev.org/c/openstack/kolla-ansible/+/918455 | 19:39 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!