opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: systemd: wait for stopping container https://review.opendev.org/c/openstack/kolla-ansible/+/890198 | 06:33 |
---|---|---|
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: systemd: wait for stopping container https://review.opendev.org/c/openstack/kolla-ansible/+/890198 | 06:34 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: systemd: wait for stopping container https://review.opendev.org/c/openstack/kolla-ansible/+/890198 | 06:35 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Introduce oneshot docker_restart_policy https://review.opendev.org/c/openstack/kolla-ansible/+/881707 | 06:36 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: systemd: wait for stopping container https://review.opendev.org/c/openstack/kolla-ansible/+/890198 | 06:44 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Introduce oneshot docker_restart_policy https://review.opendev.org/c/openstack/kolla-ansible/+/881707 | 06:44 |
opendevreview | Maksim Malchuk proposed openstack/kolla master: Revert "Disable build of Gnocchi containers" https://review.opendev.org/c/openstack/kolla/+/890302 | 07:52 |
opendevreview | Maksim Malchuk proposed openstack/kolla master: Revert "Disable build of Gnocchi containers" https://review.opendev.org/c/openstack/kolla/+/890302 | 08:38 |
kevko | hmm, i think we have bad configuration for service_user in kolla-ansible | 08:58 |
mmalchuk | kevko more details? | 09:10 |
alistarle | Hello guys :) We discover an hardcoded value in ironic tftp who can break some setup, we variabilize it in this very small patch if its look good to you : https://review.opendev.org/c/openstack/kolla-ansible/+/886747 | 09:12 |
opendevreview | howardlee proposed openstack/kolla-ansible master: Fix an undefined variable bug for venus deployment https://review.opendev.org/c/openstack/kolla-ansible/+/890321 | 09:31 |
opendevreview | Will Szumski proposed openstack/kayobe master: Adds initial support for vGPUs https://review.opendev.org/c/openstack/kayobe/+/887200 | 09:33 |
opendevreview | Will Szumski proposed openstack/kayobe master: Adds initial support for vGPUs https://review.opendev.org/c/openstack/kayobe/+/887200 | 09:40 |
opendevreview | Merged openstack/kolla-ansible master: Fix L3 agent docs typo https://review.opendev.org/c/openstack/kolla-ansible/+/889938 | 09:48 |
opendevreview | howardlee proposed openstack/kolla-ansible master: Fix an undefined variable bug for venus deployment https://review.opendev.org/c/openstack/kolla-ansible/+/890321 | 10:17 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: systemd: wait for stopping container https://review.opendev.org/c/openstack/kolla-ansible/+/890198 | 11:13 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Introduce oneshot docker_restart_policy https://review.opendev.org/c/openstack/kolla-ansible/+/881707 | 11:13 |
kevko | mmalchuk: this cant pass on my environment tempest.scenario.test_server_volume_attachment.TestServerVolumeAttachmentScenario.test_server_detach_rules | 11:34 |
kevko | https://paste.openstack.org/show/bYVAmLV2BQqoxE9LuRGa/ | 11:34 |
mmalchuk | forgot what you are talking about sorry | 11:34 |
kevko | service_user in nova and service_token_roles | 11:35 |
kevko | attach/detach | 11:35 |
mmalchuk | ah... understand | 11:35 |
mmalchuk | issue in cinder client? | 11:36 |
kevko | i can find a patch ....it was some CVE in cinder/nova ..so service_user configuration was implemented to address this issue ... service token is sent with user token also ... | 11:37 |
kevko | no | 11:37 |
kevko | btw, do we test rally/tempest somewhere ? | 11:37 |
kevko | do you know where i can check if it is passing somewhere in gerrit/zuul | 11:37 |
kevko | i mean that tempest test | 11:38 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: systemd: wait for stopping container https://review.opendev.org/c/openstack/kolla-ansible/+/890198 | 11:39 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Introduce oneshot docker_restart_policy https://review.opendev.org/c/openstack/kolla-ansible/+/881707 | 11:39 |
mmalchuk | we use tempest and in xena all fine | 11:39 |
mmalchuk | whre in gerrit I don't know | 11:40 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Introduce oneshot docker_restart_policy https://review.opendev.org/c/openstack/kolla-ansible/+/881707 | 11:40 |
mmalchuk | may be in tempest project? | 11:40 |
mnasiadka | kevko: this test doesn't fail in any of my environments | 11:43 |
mnasiadka | kevko: and we test attach/detach in CI, just we don't use tempest for that | 11:47 |
opendevreview | Dr. Jens Harbott proposed openstack/kolla-ansible master: octavia: enable jobboard https://review.opendev.org/c/openstack/kolla-ansible/+/888588 | 11:49 |
kevko | mnasiadka: hmmm, can u share your tempest.conf ? | 11:50 |
kevko | mnasiadka: attach/detach is working normally ...but that test is also testing valid/invalid tokens and proper return code ... | 11:50 |
kevko | mnasiadka: if i remove this from test ..it pass https://github.com/openstack/tempest/blob/4025fa3c8e5ba8de3f2cb60f4a225f1bc550b5e4/tempest/scenario/test_server_volume_attachment.py#L98-L158 | 11:52 |
mnasiadka | The test was merged in May, might be that my tempest is older than that ;) | 11:55 |
kevko | probably :D :D | 11:56 |
mnasiadka | we could move to tempest tests, I had an approach but something was wrong ;) | 11:58 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: Run Tempest in test-core-openstack https://review.opendev.org/c/openstack/kolla-ansible/+/878826 | 11:59 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: Run Tempest in test-core-openstack https://review.opendev.org/c/openstack/kolla-ansible/+/878826 | 11:59 |
opendevreview | Dr. Jens Harbott proposed openstack/kolla-ansible master: ironic: add ironic_agent_files_directory parameter https://review.opendev.org/c/openstack/kolla-ansible/+/871054 | 12:15 |
frickler | mnasiadka: regarding the octavia-redis dependency, I looked a bit for prior art, but didn't find any. like we also don't check whether neutron is enabled for octavia, or do we? so I'm actually wondering whether this is necessary | 12:24 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: systemd: wait for stopping container https://review.opendev.org/c/openstack/kolla-ansible/+/890198 | 12:30 |
mnasiadka | frickler: we usually did something like we do in cinder - https://github.com/openstack/kolla-ansible/blob/e0b212d72de8aac1d5fbbc6562f69ef0190a80b9/ansible/group_vars/all.yml#L1057 | 12:32 |
mnasiadka | frickler: so we could enable jobboard when redis is enabled, and/or have a precheck that if you don't have redis enabled and you enable jobboard - it's not going to work ;) | 12:33 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Introduce oneshot docker_restart_policy https://review.opendev.org/c/openstack/kolla-ansible/+/881707 | 12:34 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: Run Tempest in test-core-openstack https://review.opendev.org/c/openstack/kolla-ansible/+/878826 | 12:42 |
mnasiadka | mgoddard mnasiadka hrw bbezak frickler kevko SvenKieske mmalchuk gkoper jangutter - meeting in 10 minutes | 12:49 |
frickler | jobboard is more than a coordination backend, there's also a database involved, so making it optional would be a lot more complicated | 12:50 |
mnasiadka | but it only supports redis? | 12:51 |
frickler | mnasiadka: or zookeeper, which afaict we don't support? | 12:53 |
yusufgungor | Hi, is it possible to upgrade from yoga to zed without os upgrade? Yoga release installed on ubuntu 20.04 hosts as docker containers. Do we have to upgrade hosts to ubuntu 22.04 before upgrade? | 12:55 |
mmalchuk | yusufgungor os not related in most cases, kolla use containers | 12:55 |
frickler | some review suggestions while waiting for the meeting: https://review.opendev.org/c/openstack/kolla-ansible/+/876650 https://review.opendev.org/c/openstack/kolla/+/888219 | 12:56 |
yusufgungor | @mmalchuk thansk for reply, nice to hear that. kolla-ansible zed prechecks playbook also shows that ubuntu focal and jammy is ok | 12:58 |
yusufgungor | host_os_distributions: | 12:58 |
yusufgungor | CentOS: | 12:58 |
yusufgungor | - "9" | 12:58 |
yusufgungor | Debian: | 12:58 |
yusufgungor | - "bullseye" | 12:58 |
yusufgungor | openEuler: | 12:58 |
yusufgungor | - "22.03" | 12:58 |
yusufgungor | Rocky: | 12:58 |
yusufgungor | - "9" | 12:58 |
yusufgungor | Ubuntu: | 12:58 |
yusufgungor | - "focal" | 12:58 |
yusufgungor | - "jammy" | 12:58 |
mmalchuk | frickler reviewed | 12:59 |
mnasiadka | frickler: yeah, so redis it is ;) | 12:59 |
mmalchuk | yusufgungor right | 12:59 |
mmalchuk | yusufgungor you can upgrade os later for example | 13:00 |
mnasiadka | #startmeeting kolla | 13:00 |
opendevmeet | Meeting started Wed Aug 2 13:00:50 2023 UTC and is due to finish in 60 minutes. The chair is mnasiadka. Information about MeetBot at http://wiki.debian.org/MeetBot. | 13:00 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 13:00 |
opendevmeet | The meeting name has been set to 'kolla' | 13:00 |
mnasiadka | #topic rollcall | 13:00 |
mnasiadka | o/ | 13:00 |
mmalchuk | \o | 13:00 |
jangutter | o/ | 13:01 |
frickler | \o | 13:01 |
mattcrees | o/ | 13:01 |
* frickler likes how that all lined up | 13:01 | |
mmalchuk | lol | 13:03 |
mnasiadka | #topic agenda | 13:03 |
mnasiadka | * CI status | 13:03 |
mnasiadka | * Release tasks | 13:03 |
mnasiadka | * Regular stable releases (first meeting in a month) | 13:03 |
mnasiadka | * Current cycle planning | 13:03 |
mnasiadka | * Additional agenda (from whiteboard) | 13:03 |
mnasiadka | * Open discussion | 13:03 |
mnasiadka | #topic CI Status | 13:03 |
mnasiadka | Basically master Kayobe is RED, because of a bug in Kolla-Ansible's systemd stuff | 13:03 |
mnasiadka | And we haven't seen that in Kolla-Ansible CI, because we don't use systemd at all (docker_restart_policy = no) | 13:04 |
mnasiadka | working on a fix, but basically on upgrade we end up with removing containers before they are stopped (so docker engine sends SIGKILL instead of SIGTERM) - and guess what - MariaDB doesn't like it very much. | 13:04 |
mmalchuk | thats predictable) | 13:05 |
mnasiadka | thank you captain obvious :) | 13:06 |
mnasiadka | ok, let's move on | 13:06 |
mnasiadka | #topic Release tasks | 13:06 |
mnasiadka | none this week | 13:06 |
mnasiadka | #topic Regular stable releases (first meeting in a month) | 13:06 |
mnasiadka | frickler: should we wait for the systemd fix with stable releases? | 13:06 |
frickler | which releases are affected? | 13:06 |
mnasiadka | Antelope only | 13:07 |
frickler | let's wait then I'd say | 13:08 |
mnasiadka | Yoga is going EM I think, so we'd need to tidy up patches there and submit last release | 13:08 |
frickler | also please review https://review.opendev.org/c/openstack/kolla/+/888219 that updates the instructions, I used the updated docs last time already | 13:08 |
frickler | oh, only just EMed xena | 13:09 |
frickler | but I don't mind, we don't use yoga anymore downstream | 13:09 |
mnasiadka | well, other thing is that seems EM is being changed to unmaintained (naming wise) | 13:09 |
mnasiadka | so those that use EM can expect a branch name change if I remember correctly to unmaintained/xena and unmaintained/yoga | 13:10 |
frickler | and becoming opt-in rather than opt-out going forward | 13:10 |
mnasiadka | yup, I don't mind - EM was like an unwanted child in Kolla landscape ;) | 13:11 |
mnasiadka | SHPC uses Yoga for now, but Xena is not something we need | 13:11 |
mnasiadka | So agreed - when systemd will be fixed in k-a and backported to Antelope - we should do releases | 13:12 |
frickler | I was about to ask if we can just eol yoga instead, but seems you would not agree? | 13:12 |
mnasiadka | For now I think it would be better to keep the branch open, but I'll discuss that internally and come back to it on next weekly meeting | 13:13 |
mnasiadka | Anybody needs Xena? | 13:13 |
frickler | cool | 13:13 |
mmalchuk | me | 13:13 |
mnasiadka | No patches for stable/xena open | 13:14 |
mnasiadka | mmalchuk: how long do you think you need stable/xena open? | 13:14 |
mmalchuk | couple of week | 13:14 |
mmalchuk | please | 13:14 |
mnasiadka | ok, let's reassess in September | 13:15 |
mmalchuk | thanks | 13:15 |
mnasiadka | #topic Current cycle planning | 13:15 |
mnasiadka | Anybody had cycles to review Let's Encrypt? | 13:15 |
mmalchuk | I did review | 13:15 |
opendevreview | Merged openstack/kolla master: Update stable branch management doc https://review.opendev.org/c/openstack/kolla/+/888219 | 13:16 |
mnasiadka | I don't have time this week, but I'll try next | 13:16 |
mnasiadka | Ok, let's go to agenda from whiteboard, seems we have some | 13:17 |
mnasiadka | #topic Additional agenda (from whiteboard) | 13:17 |
mnasiadka | ihalomi - failing podman test on rocky9 https://zuul.opendev.org/t/openstack/build/bfc1625b4e0f4788be21f3bf5454ba4d | 13:18 |
mnasiadka | I promised to have a look, but didn't have time - should have some after the systemd bug | 13:18 |
mnasiadka | frickler - octavia jobboard | 13:18 |
mnasiadka | https://review.opendev.org/c/openstack/kolla/+/888587 | 13:18 |
mnasiadka | https://review.opendev.org/c/openstack/kolla-ansible/+/888588 | 13:18 |
frickler | yes, took me until today to update, sorry for that | 13:18 |
frickler | first patch is merged, second we talked about earlier | 13:19 |
mnasiadka | no problems | 13:19 |
mnasiadka | kolla side is merged | 13:19 |
mnasiadka | I understand jobboard won't work without redis - so we'd need some precheck? | 13:19 |
frickler | yes, I think so, I will try to build that | 13:20 |
mnasiadka | Great, anything else on that topic? | 13:20 |
frickler | not from me | 13:20 |
frickler | except ... | 13:20 |
mnasiadka | ok, next one | 13:20 |
mnasiadka | frickler - debian bookworm support | 13:20 |
mnasiadka | https://review.opendev.org/c/openstack/kolla/+/886088 seems mostly ready | 13:20 |
mnasiadka | https://review.opendev.org/c/openstack/kolla-ansible/+/888034 need to check the haproxy config update | 13:20 |
frickler | yes, the latter is marked as wip still because I wasn't sure whether we can drop that old haproxy option globally | 13:21 |
mnasiadka | kolla side looks ok, I can fix the netcontrold image | 13:21 |
frickler | does someone know haproxy better than me? | 13:21 |
mmalchuk | haproxy authorr?) | 13:22 |
mmalchuk | what the problem? | 13:22 |
mnasiadka | https://www.haproxy.com/blog/multithreading-in-haproxy#from-multiprocess-to-multithreading | 13:22 |
mnasiadka | nbproc is not supported, and we should move to multithreading | 13:22 |
mmalchuk | but there was reviews | 13:23 |
mmalchuk | and CI passd | 13:23 |
mnasiadka | where? | 13:24 |
mmalchuk | https://review.opendev.org/c/openstack/kolla-ansible/+/883560 | 13:24 |
mmalchuk | and | 13:25 |
mmalchuk | https://review.opendev.org/c/openstack/kolla-ansible/+/883559 | 13:25 |
mnasiadka | frickler: is nbproc dropped or do we need now to do nbthread and nbproc? | 13:25 |
mmalchuk | both need reviewers | 13:25 |
mnasiadka | I haven't seen that | 13:25 |
mmalchuk | vacation) | 13:25 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: loadbalancer: support setting nbthread with variable haproxy_threads https://review.opendev.org/c/openstack/kolla-ansible/+/883559 | 13:25 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: loadbalancer: support cpu-map for threads https://review.opendev.org/c/openstack/kolla-ansible/+/883560 | 13:25 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: loadbalancer: remove support for haproxy_processes https://review.opendev.org/c/openstack/kolla-ansible/+/883982 | 13:26 |
mnasiadka | rebased, let's review | 13:26 |
frickler | me neither, I'll check those patches | 13:26 |
mnasiadka | added RP+1 to them | 13:26 |
frickler | the other thing is how to bump the debian python version in k and k-a at the same time | 13:26 |
frickler | make debian n-v in k-a temporarily? | 13:27 |
mnasiadka | no, we've been passing a variable in the past | 13:27 |
mnasiadka | but I had an idea to link the python version so that wsgi configs don't need to know the python version | 13:27 |
mnasiadka | or we could have a task to check python version in the image | 13:27 |
mnasiadka | https://review.opendev.org/c/openstack/kolla-ansible/+/874990 | 13:28 |
mnasiadka | one attempt | 13:28 |
mnasiadka | I can have a look again, so we stop doing weird dances every time | 13:28 |
mnasiadka | ok, next one | 13:28 |
mnasiadka | jangutter - etcd support | 13:28 |
mnasiadka | https://review.opendev.org/c/openstack/kolla-ansible/+/888012 (adding/removing nodes to a cluster) | 13:28 |
mnasiadka | general question: any users of etcd? etcd-3.3 might no longer be supported | 13:28 |
mnasiadka | I saw that patch, will have a look next week | 13:29 |
jangutter | thanks! no rush! | 13:29 |
mnasiadka | around upgrading etcd - we could bump to 3.4 I guess, because etcd can't do skip level upgrades, right? | 13:29 |
jangutter | That's what I read too.. | 13:29 |
mnasiadka | so let's do that | 13:30 |
jangutter | It's getting close though, if 3.6 releases then 3.4 goes out of maintenance.... so I guess sooner is better than later. | 13:30 |
mnasiadka | jangutter: happy to raise a patch to go to 3.4? | 13:30 |
mnasiadka | and then we could go to 3.5 in Caracal | 13:30 |
jangutter | try and stop me :-p | 13:30 |
mnasiadka | ok, thanks | 13:30 |
mnasiadka | I unfortunately need to run... sorry for the short meeting today | 13:31 |
mnasiadka | feel free to continue the discussion after the meeting | 13:31 |
mmalchuk | Gnocchi is back, fix - https://review.opendev.org/c/openstack/kolla/+/890302 | 13:31 |
mnasiadka | #endmeeting | 13:31 |
opendevmeet | Meeting ended Wed Aug 2 13:31:23 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 13:31 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/kolla/2023/kolla.2023-08-02-13.00.html | 13:31 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/kolla/2023/kolla.2023-08-02-13.00.txt | 13:31 |
opendevmeet | Log: https://meetings.opendev.org/meetings/kolla/2023/kolla.2023-08-02-13.00.log.html | 13:31 |
mnasiadka | mmalchuk: thanks for handling that | 13:31 |
mmalchuk | Can be merged and need backports | 13:31 |
mmalchuk | mnasiadka thanks | 13:31 |
mmalchuk | cores, review and merge please: https://review.opendev.org/c/openstack/kolla/+/890302 | 13:34 |
opendevreview | Gaƫl THEROND proposed openstack/kolla-ansible master: Improve designate role support. https://review.opendev.org/c/openstack/kolla-ansible/+/878270 | 14:37 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: systemd: wait for stopping container https://review.opendev.org/c/openstack/kolla-ansible/+/890198 | 14:40 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: systemd: wait for stopping container https://review.opendev.org/c/openstack/kolla-ansible/+/890198 | 14:40 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Introduce oneshot docker_restart_policy https://review.opendev.org/c/openstack/kolla-ansible/+/881707 | 14:41 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Introduce oneshot docker_restart_policy https://review.opendev.org/c/openstack/kolla-ansible/+/881707 | 14:41 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: Link lib/python version https://review.opendev.org/c/openstack/kolla/+/874989 | 14:43 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: Link lib/python version https://review.opendev.org/c/openstack/kolla/+/874989 | 14:44 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Drop distro_python_version https://review.opendev.org/c/openstack/kolla-ansible/+/874990 | 14:45 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Drop distro_python_version https://review.opendev.org/c/openstack/kolla-ansible/+/874990 | 14:45 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: Run Tempest in test-core-openstack https://review.opendev.org/c/openstack/kolla-ansible/+/878826 | 14:49 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: Run Tempest in test-core-openstack https://review.opendev.org/c/openstack/kolla-ansible/+/878826 | 14:49 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: systemd: wait for stopping container https://review.opendev.org/c/openstack/kolla-ansible/+/890198 | 15:01 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Introduce oneshot docker_restart_policy https://review.opendev.org/c/openstack/kolla-ansible/+/881707 | 15:01 |
opendevreview | Merged openstack/kayobe stable/xena: Limit interface names to 15 characters https://review.opendev.org/c/openstack/kayobe/+/884542 | 15:02 |
spatel | My two cents on HAproxy, nbproc is deprecated after 2.x release of haproxy and it has been replace with nbthread | 17:09 |
johnsom | That is correct nbproc is going away, it had a ton of tricky issues to work around. The new threads model is much better. | 17:46 |
guesswhat[m] | By any chance, would be possible to backport this https://review.opendev.org/c/openstack/skyline-apiserver/+/879465 ? | 20:37 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!