| opendevreview | Khoi proposed openstack/kolla-ansible master: Add ceph_volume_backend_name It is useful for multi AZ and each AZ has same volume backend type. https://review.opendev.org/c/openstack/kolla-ansible/+/940228 | 05:31 |
|---|---|---|
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: placement: Add support for using uWSGI https://review.opendev.org/c/openstack/kolla-ansible/+/937448 | 06:28 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: cinder: Add support for using uWSGI https://review.opendev.org/c/openstack/kolla-ansible/+/937449 | 06:28 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: proxysql: Set threads to 4 at minimum https://review.opendev.org/c/openstack/kolla-ansible/+/940290 | 09:46 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: proxysql: Set threads to 4 at minimum https://review.opendev.org/c/openstack/kolla-ansible/+/940290 | 09:48 |
| mnasiadka | frickler: https://review.opendev.org/c/openstack/kolla-ansible/+/937448 - placement uwsgi ready | 09:51 |
| opendevreview | Pablo Colson proposed openstack/kolla-ansible master: Add ovn-bgp-agent and FRR roles https://review.opendev.org/c/openstack/kolla-ansible/+/937066 | 10:01 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: proxysql: Set threads to 4 at minimum https://review.opendev.org/c/openstack/kolla-ansible/+/940290 | 10:22 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: proxysql: Set threads to 4 https://review.opendev.org/c/openstack/kolla-ansible/+/940290 | 10:23 |
| frickler | mnasiadka: so this weird log file name only happened for placement and no other service? | 10:42 |
| mnasiadka | frickler: not in Nova surely and not in cinder - but I don't know if we don't set log_file in other services | 10:45 |
| opendevreview | Merged openstack/kolla-ansible master: Reintroduce kolla-ansible check https://review.opendev.org/c/openstack/kolla-ansible/+/599735 | 10:51 |
| opendevreview | Merged openstack/kolla-ansible stable/2024.1: CI: Fix cases where port is not available yet https://review.opendev.org/c/openstack/kolla-ansible/+/926037 | 10:58 |
| opendevreview | Merged openstack/kolla-ansible stable/2024.1: CI: fix lint requirements for python 3.12 https://review.opendev.org/c/openstack/kolla-ansible/+/926181 | 10:58 |
| opendevreview | Pablo Colson proposed openstack/kolla-ansible master: feat: added some docs for ovn-bgp-agent in Networking https://review.opendev.org/c/openstack/kolla-ansible/+/940295 | 11:05 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: placement: Add support for using uWSGI https://review.opendev.org/c/openstack/kolla-ansible/+/937448 | 11:14 |
| opendevreview | Merged openstack/kolla-ansible stable/2023.2: Fix prometheus.yml templating https://review.opendev.org/c/openstack/kolla-ansible/+/926624 | 11:22 |
| opendevreview | Verification of a change to openstack/kolla-ansible stable/2024.1 failed: CI: Fix variable name for Nova noVNC FQDN https://review.opendev.org/c/openstack/kolla-ansible/+/928502 | 11:22 |
| opendevreview | Merged openstack/kolla-ansible stable/2023.2: [CI] Force config drive for ipv6 jobs https://review.opendev.org/c/openstack/kolla-ansible/+/924231 | 11:22 |
| pabloclsn | Hi, I just did a merge requests on openstack kolla but I think I made a mistake, I wanted to add a commit (some docs) to a MR (https://review.opendev.org/c/openstack/kolla-ansible/+/937066) of an other Author but when I use git review it created an other MR linked to the first one (https://review.opendev.org/c/openstack/kolla-ansible/+/940295) is it OK ? | 12:08 |
| frickler | pabloclsn: yes, that's how gerrit works, each commit gets reviewed on its own, which is why we also talk about patches or patchsets instead of MR | 12:11 |
| opendevreview | Pablo Colson proposed openstack/kolla-ansible master: feat: added some docs for ovn-bgp-agent in Networking https://review.opendev.org/c/openstack/kolla-ansible/+/940295 | 12:13 |
| pabloclsn | Oh ok so I did nothing wrong | 12:15 |
| opendevreview | Pablo Colson proposed openstack/kolla-ansible master: Add ovn-bgp-agent and FRR roles https://review.opendev.org/c/openstack/kolla-ansible/+/937066 | 12:16 |
| opendevreview | Pablo Colson proposed openstack/kolla-ansible master: feat: added some docs for ovn-bgp-agent in Networking https://review.opendev.org/c/openstack/kolla-ansible/+/940295 | 12:16 |
| pabloclsn | should I manually reviewer to the patches ? | 12:18 |
| frickler | pabloclsn: no, give it a couple of days assuming the CI passes, if no reviews happen then you can ask here | 12:38 |
| opendevreview | Merged openstack/kolla-ansible master: swift: Deprecate for removal in 2025.2 https://review.opendev.org/c/openstack/kolla-ansible/+/939968 | 12:49 |
| pabloclsn | Ok thanks frickler | 13:25 |
| opendevreview | Pablo Colson proposed openstack/kolla-ansible master: feat: added some docs for ovn-bgp-agent in Networking https://review.opendev.org/c/openstack/kolla-ansible/+/940295 | 13:38 |
| pabloclsn | for example, if I want to contribute to this patch and fix the code to resolve the comments, how can i add a patchset to the patch without creating a new patch ? | 14:10 |
| pabloclsn | this patch https://review.opendev.org/c/openstack/kolla/+/937057 | 14:14 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: proxysql: Set threads to 4 https://review.opendev.org/c/openstack/kolla-ansible/+/940290 | 14:55 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: proxysql: Add ability to configure watchdog https://review.opendev.org/c/openstack/kolla-ansible/+/940290 | 14:56 |
| mnasiadka | pabloclsn: https://docs.openstack.org/contributors/code-and-documentation/using-gerrit.html#pushing-a-change | 14:57 |
| pabloclsn | I can amend on someone else commit ?? | 15:02 |
| opendevreview | Pablo Colson proposed openstack/kolla-ansible master: Implement neutron-ovn-vpn-agent https://review.opendev.org/c/openstack/kolla-ansible/+/924575 | 15:11 |
| mnasiadka | pabloclsn: yes | 15:44 |
| mnasiadka | pabloclsn: although it's nice to ask if he doesn't mind | 15:45 |
| pabloclsn | Ok thanks mnasiadka :) | 16:17 |
| *** priteau is now known as Guest7267 | 17:04 | |
| *** priteau2 is now known as priteau | 17:04 | |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: CI: Simply globals-default by moving to globals.d https://review.opendev.org/c/openstack/kolla-ansible/+/940320 | 17:07 |
| opendevreview | Verification of a change to openstack/kolla-ansible stable/2024.1 failed: [CI] Force config drive for ipv6 jobs https://review.opendev.org/c/openstack/kolla-ansible/+/924230 | 18:13 |
| pabloclsn | I'd like to write some CI for this. I suppose I should add some test in the test-neutron.sh file, but I'd like to actually test this like in zuul but I have no idea how it's working, does someone have any clue ton run this localy ? | 18:45 |
| pabloclsn | this -> https://review.opendev.org/c/openstack/kolla-ansible/+/924575 | 18:50 |
| opendevreview | Merged openstack/kolla-ansible master: placement: Add support for using uWSGI https://review.opendev.org/c/openstack/kolla-ansible/+/937448 | 19:12 |
| opendevreview | Maksim Malchuk proposed openstack/kolla-ansible master: Add ovn-bgp-agent and FRR roles https://review.opendev.org/c/openstack/kolla-ansible/+/937066 | 20:20 |
| pabloclsn | The vagrant method to deploy localy with kolla-ansible is not up to date at all right ? | 21:28 |
| opendevreview | Verification of a change to openstack/kolla-ansible stable/2024.1 failed: CI: Fix variable name for Nova noVNC FQDN https://review.opendev.org/c/openstack/kolla-ansible/+/928502 | 22:04 |
| opendevreview | Michal Arbet proposed openstack/kolla master: [letsencrypt] Add ProxySQL support for certificate renewal https://review.opendev.org/c/openstack/kolla/+/936920 | 23:18 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Add TLS support for MariaDB Connections https://review.opendev.org/c/openstack/kolla-ansible/+/933924 | 23:24 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Enhance TLS configuration and certificate handling in Kolla https://review.opendev.org/c/openstack/kolla-ansible/+/937108 | 23:24 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Add let's encrypt support for ProxySQL https://review.opendev.org/c/openstack/kolla-ansible/+/936922 | 23:24 |
| opendevreview | Merged openstack/kolla-ansible stable/2024.1: CI: Fix variable name for Nova noVNC FQDN https://review.opendev.org/c/openstack/kolla-ansible/+/928502 | 23:37 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!