| sondx | Hi mnasiadka, | 07:10 |
|---|---|---|
| sondx | https://github.com/openstack/kolla-ansible/blob/master/ansible/roles/ovn/templates/ovn-controller.json.j2#L2 | 07:10 |
| sondx | Why use "tcp:127.0.0.1:{{ ovsdb_port }}" to deploy ovn-controller service, but don't use "unix:/run/openvswitch/db.sock". | 07:10 |
| mnasiadka | sondx: because at the time of adding support to that, the reviewers felt that it's a better solution that writing to a socket. | 07:45 |
| mnasiadka | feel free to propose a patch | 07:45 |
| mnasiadka | (or find the old one with comments) | 07:46 |
| sondx | mnasiadka: | 09:12 |
| sondx | ovn-controller use "tcp:127.0.0.1:{{ ovsdb_port }}", it generate a error "reconnect|ERR|tcp:127.0.0.1:26762: no response to inactivity probe after 5 seconds, disconnecting" in "/var/log/kolla/openvswitch/ovsdb-server.log" log file . | 09:12 |
| sondx | If ovn-controller use "unix:/run/openvswitch/db.sock", that error will be solved. | 09:12 |
| mnasiadka | sondx: as I said - feel free to raise a bug and propose a patch | 09:13 |
| sondx | mnasiadka: Ok, thank you very much! | 09:15 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: haproxy: support single external frontend https://review.opendev.org/c/openstack/kolla-ansible/+/823395 | 10:13 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Test haproxy single external frontend https://review.opendev.org/c/openstack/kolla-ansible/+/841239 | 10:17 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Test haproxy single external frontend https://review.opendev.org/c/openstack/kolla-ansible/+/841239 | 10:19 |
| opendevreview | Piotr Parczewski proposed openstack/kolla-ansible master: Add support for deploying prometheus-msteams https://review.opendev.org/c/openstack/kolla-ansible/+/812679 | 10:58 |
| opendevreview | Piotr Parczewski proposed openstack/kolla-ansible master: Add support for deploying prometheus-msteams https://review.opendev.org/c/openstack/kolla-ansible/+/812679 | 11:21 |
| opendevreview | Merged openstack/kolla-ansible master: Edit services roles to support database sharding https://review.opendev.org/c/openstack/kolla-ansible/+/770216 | 12:15 |
| opendevreview | Merged openstack/kolla-ansible master: Add loadbalancer-config role and wrap haproxy-config role inside https://review.opendev.org/c/openstack/kolla-ansible/+/851004 | 12:15 |
| opendevreview | Merged openstack/kolla-ansible master: Add proxysql-config role and wrap it in loadbalancer-config role https://review.opendev.org/c/openstack/kolla-ansible/+/851442 | 12:22 |
| opendevreview | Merged openstack/kolla-ansible master: [CI] Test ProxySQL with shards in the nova cells scenario https://review.opendev.org/c/openstack/kolla-ansible/+/770621 | 12:22 |
| *** amoralej is now known as amoralej|lunch | 12:28 | |
| opendevreview | Lin PeiWen proposed openstack/kolla-ansible stable/yoga: keystone: Remove redundant 'run_once' https://review.opendev.org/c/openstack/kolla-ansible/+/852736 | 12:52 |
| opendevreview | Lin PeiWen proposed openstack/kolla-ansible master: keystone: Remove redundant 'run_once' https://review.opendev.org/c/openstack/kolla-ansible/+/852739 | 13:02 |
| *** amoralej|lunch is now known as amoralej | 13:20 | |
| mnasiadka | mgoddard mnasiadka hrw yoctozepto bbezak parallax Fl1nt frickler adrian-a - meeting in 8 | 13:52 |
| TimBeermann[m] | o/ | 13:55 |
| RamonaRautenberg[m] | o/ | 13:57 |
| mnasiadka | not yet | 13:58 |
| opendevreview | Merged openstack/kolla master: Remove Keystone admin endpoint https://review.opendev.org/c/openstack/kolla/+/843751 | 13:58 |
| mnasiadka | #startmeeting kolla | 14:00 |
| opendevmeet | Meeting started Wed Aug 10 14:00:57 2022 UTC and is due to finish in 60 minutes. The chair is mnasiadka. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| opendevmeet | The meeting name has been set to 'kolla' | 14:00 |
| mnasiadka | #topic rollcall | 14:01 |
| mnasiadka | o/ | 14:01 |
| frickler | now \o/ | 14:01 |
| matfechner | o/ | 14:01 |
| mmalchuk | hi | 14:01 |
| mnasiadka | #topic agenda | 14:03 |
| mnasiadka | * Announcements | 14:03 |
| mnasiadka | * Review action items from the last meeting | 14:03 |
| mnasiadka | * CI status | 14:03 |
| mnasiadka | * Release tasks | 14:03 |
| mnasiadka | * Current cycle planning | 14:03 |
| mnasiadka | * Additional agenda (from whiteboard) | 14:03 |
| mnasiadka | * Open discussion | 14:03 |
| mnasiadka | #topic announcements | 14:03 |
| mnasiadka | PTG in October is virtual - https://lists.openstack.org/pipermail/openstack-discuss/2022-August/029879.html | 14:03 |
| mnasiadka | #topic Review action items from the last meeting | 14:03 |
| mnasiadka | frickler check availability of zed UCA for Jammy | 14:03 |
| frickler | hrw answered that immediately | 14:04 |
| mnasiadka | ok then | 14:04 |
| frickler | so next up is creating the actual patch | 14:04 |
| mnasiadka | frickler: are you planning to do that? | 14:04 |
| frickler | I can do that, yes | 14:05 |
| mnasiadka | thanks | 14:06 |
| mnasiadka | #topic CI status | 14:06 |
| mnasiadka | seems mainly green | 14:06 |
| mnasiadka | #topic Release tasks | 14:07 |
| mnasiadka | It's R-8 - Switch images to current release¶ | 14:07 |
| mnasiadka | Ubuntu UCA is part of that | 14:08 |
| mnasiadka | CentOS Stream 9 is not really working IIRC | 14:08 |
| mnasiadka | And we should really be working on Rocky Linux 9 | 14:08 |
| mnasiadka | I'll take care of that based on CS9 patches | 14:08 |
| mnasiadka | #topic Additional agenda (from whiteboard) | 14:08 |
| mnasiadka | (headphoneJames) please review let's encrypt patch: https://review.opendev.org/c/openstack/kolla-ansible/+/741340 | 14:09 |
| mnasiadka | Is there any core reviewer interested in testing and reviewing let's encrypt patch? Or even a non-core? | 14:09 |
| mnasiadka | Ok, we're a bit limited on the meeting today - I'll re-ask that next week. | 14:10 |
| mnasiadka | (frickler) Using octavia_network_type: "tenant" in production | 14:11 |
| mnasiadka | Why are our docs telling people not to do this? https://docs.openstack.org/kolla-ansible/wallaby/reference/networking/octavia.html#development-or-testing | 14:11 |
| mnasiadka | Introduced in https://review.opendev.org/c/openstack/kolla-ansible/+/783876 | 14:11 |
| mnasiadka | Octavia proposes the same setup in their install guide without any warning https://docs.openstack.org/octavia/latest/install/install-ubuntu.html#prerequisites | 14:11 |
| mgoddard | I would like to test LE, one day | 14:11 |
| mnasiadka | I think at the time of merging that feature - we did anticipate that the systemd unit with a shell script might just fail sometimes, and we did not want users coming back and complaining | 14:11 |
| mnasiadka | If somebody wants to raise a patch to change the ,,recommendation'' - please do - I guess it should be ok. | 14:12 |
| mgoddard | indeed | 14:12 |
| frickler | so for me, the thing one could consider not fit for production is using dhcp for the interface | 14:12 |
| frickler | a static configuration would be more stable | 14:12 |
| frickler | but the current wording suggests that tenant networks are somewhat unstable | 14:12 |
| frickler | so I'll make a patch to update the doc, fine | 14:13 |
| frickler | just wanted to hear whether there's any hidden reasoning I'm not aware of | 14:14 |
| mnasiadka | frickler: are you planning to add static ip configuration as well? | 14:14 |
| frickler | mnasiadka: maybe later, not now | 14:16 |
| mnasiadka | ok | 14:16 |
| mnasiadka | next one | 14:16 |
| mnasiadka | (frickler) ceph-ansible has been un-deprecated (kind of) | 14:16 |
| mnasiadka | https://github.com/ceph/ceph-ansible/commit/77fc0e105efaaa8a47e0d6f5676c01b2f674230f | 14:16 |
| mnasiadka | should we consider switching the CI back to it? | 14:16 |
| mnasiadka | not an urgent topic, maybe punt to PTG discussion | 14:16 |
| frickler | yeah, we still use ceph-ansible for osism and CI for cephadm seems to be failing a lot | 14:17 |
| mnasiadka | When it comes to me - I'm not a fan of ceph-ansible, and I can't see a reason why we would be going back to use it in CI instead of cephadm (especially that ceph-ansible readme points to use cephadm). | 14:17 |
| mnasiadka | I think me and Radek put enough hours, to make cephadm green again. | 14:17 |
| frickler | well currently I'm seeing a lot of failures for it | 14:18 |
| frickler | but if you prefer to stick to it, I won't object | 14:18 |
| frickler | just won't be able to help as much in maintaining it | 14:18 |
| frickler | anyway, we can discuss again when yoctozepto is back possibly | 14:20 |
| mnasiadka | Today it seems green, yesterday there were some failures - but it's not always cephadm fault I guess. | 14:20 |
| mnasiadka | I'll try to take a look later. | 14:20 |
| mnasiadka | next one | 14:21 |
| mnasiadka | (i.halomi) Podman and systemd support status | 14:21 |
| mnasiadka | review of systemd with docker a current status on podman version : https://review.opendev.org/c/openstack/kolla-ansible/+/816724 | 14:21 |
| mnasiadka | problems with using podman-py | 14:21 |
| mnasiadka | review status of kolla-build with podman support : https://review.opendev.org/c/openstack/kolla/+/828716/ | 14:21 |
| mnasiadka | future testing of Podman functionality in Zuul | 14:21 |
| ihalomi | our podman version is almost ready | 14:22 |
| ihalomi | but we have one problem with podman-py module | 14:22 |
| ihalomi | it doesnt accept empty environment variables and it deletes them | 14:22 |
| ihalomi | it creates problem in creating haproxy monitor user in mariadb where he has no password at default, can we add there some dummy password to solve this? | 14:23 |
| mnasiadka | Well, that would be a separate change not only affecting podman | 14:25 |
| mnasiadka | mgoddard: ideas? | 14:25 |
| ihalomi | i created issue in podman-py community so there is chance they will solve it but they didnt respond yet | 14:26 |
| mnasiadka | well, the alternative to wait is to set dummy password for mariadb monitor user - are you planning to send that patch for review? | 14:29 |
| mgoddard | can we translate them to empty strings? | 14:29 |
| mnasiadka | it might be we could alter mariadb-clustercheck script to send empty password if variable is unset? | 14:30 |
| mnasiadka | but we would need to move the code into kolla repository | 14:31 |
| ihalomi | so should i create patch for only this password change? | 14:31 |
| mnasiadka | that's the workaround I guess, unless you're willing to patch that in podman-py and there's a chance it will be merged there soon | 14:32 |
| ihalomi | i am waiting for their response but i dont think i will have time to patch it myself | 14:33 |
| mnasiadka | well, you know what are the options now :) | 14:34 |
| mnasiadka | ok, let's move on | 14:35 |
| mnasiadka | #topic Open discussion | 14:35 |
| mnasiadka | (I skipped the Current Cycle Planning, but there were two features to discuss from the Additional Agenda - and I don't think we have the audience to discuss more of them today) | 14:36 |
| mnasiadka | Anybody has any additional topics/queries? | 14:36 |
| m_hiner | Did anyone take a look at Systemd patch failures, and if yes, can give some hints about what's wrong? | 14:37 |
| mmalchuk | any chance to look at my reviews? orpphan interfaces cleanup for example | 14:37 |
| m_hiner | One failed at cache fail and other at couldn't had connection closed | 14:38 |
| m_hiner | So I suspect the problem might not be on the patch side | 14:39 |
| mmalchuk | or repos.yaml for Kayobe? | 14:39 |
| frickler | links would be helpful | 14:40 |
| mmalchuk | https://review.opendev.org/c/openstack/kayobe/+/845338 | 14:40 |
| mmalchuk | https://review.opendev.org/c/openstack/kayobe/+/840033 | 14:40 |
| m_hiner | Systemd: https://review.opendev.org/c/openstack/kolla-ansible/+/816724 | 14:40 |
| mmalchuk | or even old TSL sync K-A to Kayobe | 14:41 |
| mmalchuk | https://review.opendev.org/c/openstack/kayobe/+/793697 | 14:41 |
| frickler | oh, that's all kayobe, I don't do that, but will look at the other one later | 14:42 |
| mmalchuk | thanks a lot | 14:43 |
| mnasiadka | I'll try to look at Kayobe ones next week most probably | 14:43 |
| mmalchuk | mgoddard could You please look? You should be familar with issues | 14:43 |
| mmalchuk | mnasiadka thanks too | 14:44 |
| mnasiadka | m_hiner: on 5th Aug we merged some cephadm related patches, rebase/recheck so we have fresh results | 14:44 |
| m_hiner | Oh okay, thanks | 14:45 |
| mnasiadka | And I'll look into the results once they show up | 14:45 |
| mnasiadka | ok, I think that's enough for today | 14:45 |
| mnasiadka | Thanks! | 14:45 |
| mnasiadka | #endmeeting | 14:45 |
| opendevmeet | Meeting ended Wed Aug 10 14:45:49 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:45 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/kolla/2022/kolla.2022-08-10-14.00.html | 14:45 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/kolla/2022/kolla.2022-08-10-14.00.txt | 14:45 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/kolla/2022/kolla.2022-08-10-14.00.log.html | 14:45 |
| frickler | thx mnasiadka | 14:46 |
| mmalchuk | thanks | 14:49 |
| mmalchuk | bb | 14:49 |
| mmalchuk | \q | 14:49 |
| *** amoralej is now known as amoralej|off | 16:08 | |
| opendevreview | Merged openstack/kolla-ansible master: Add support for deploying prometheus-msteams https://review.opendev.org/c/openstack/kolla-ansible/+/812679 | 17:14 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!