| gongysh_ | hi | 02:23 |
|---|---|---|
| jeffrey4l_ | gongysh_ welcome | 02:23 |
| gongysh_ | jeffrey4l_: test | 02:23 |
| opendevreview | wu.chunyang proposed openstack/kolla-ansible master: Fix trove guest agent failed to connect rabbitmq https://review.opendev.org/c/openstack/kolla-ansible/+/904963 | 03:02 |
| opendevreview | wu.chunyang proposed openstack/kolla-ansible master: Fix trove guest agent failed to connect rabbitmq https://review.opendev.org/c/openstack/kolla-ansible/+/904963 | 03:11 |
| opendevreview | wu.chunyang proposed openstack/kolla-ansible master: Fix trove failed to discover swift endpoint https://review.opendev.org/c/openstack/kolla-ansible/+/904968 | 03:49 |
| opendevreview | Dr. Jens Harbott proposed openstack/kolla-ansible master: CI: improve get_logs.sh https://review.opendev.org/c/openstack/kolla-ansible/+/870521 | 06:23 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: openvswitch: Add ovs_wrapper for handling TERM https://review.opendev.org/c/openstack/kolla/+/905189 | 06:59 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: openvswitch: Stop using intermediate scripts https://review.opendev.org/c/openstack/kolla-ansible/+/905117 | 07:00 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: openvswitch: Stop using intermediate scripts https://review.opendev.org/c/openstack/kolla-ansible/+/905117 | 07:00 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: openvswitch: Add ovs_wrapper for handling TERM https://review.opendev.org/c/openstack/kolla/+/905189 | 07:05 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: openvswitch: Add ovs_wrapper for handling TERM https://review.opendev.org/c/openstack/kolla/+/905189 | 08:09 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: fail when systemd unit fails to stop https://review.opendev.org/c/openstack/kolla-ansible/+/891781 | 08:10 |
| opendevreview | Martin Hiner proposed openstack/kolla-ansible master: Add container engine migration scenario https://review.opendev.org/c/openstack/kolla-ansible/+/836941 | 10:02 |
| opendevreview | Merged openstack/kolla-ansible stable/2023.2: Fix Nova scp failures on Debian Bookworm https://review.opendev.org/c/openstack/kolla-ansible/+/905084 | 10:02 |
| *** ironfoot_ is now known as ironfoot | 11:04 | |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: neutron: Add higher graceful timeout for server and ovs agent https://review.opendev.org/c/openstack/kolla-ansible/+/905208 | 11:11 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: haproxy: Use -f configdir instead of for and xargs https://review.opendev.org/c/openstack/kolla-ansible/+/905121 | 11:20 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: mariadb-clustercheck: Use socat wrapper https://review.opendev.org/c/openstack/kolla-ansible/+/905131 | 11:21 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: openvswitch: Stop using intermediate scripts https://review.opendev.org/c/openstack/kolla-ansible/+/905117 | 11:21 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: fail when systemd unit fails to stop https://review.opendev.org/c/openstack/kolla-ansible/+/891781 | 11:21 |
| opendevreview | Jake Hutchinson proposed openstack/kolla-ansible master: Ironic parameter rework and default NTP server https://review.opendev.org/c/openstack/kolla-ansible/+/893031 | 13:21 |
| SvenKieske | o/ | 14:01 |
| SvenKieske | um | 14:01 |
| mnasiadka | #startmeeting kolla | 14:01 |
| opendevmeet | Meeting started Wed Jan 10 14:01:22 2024 UTC and is due to finish in 60 minutes. The chair is mnasiadka. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:01 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:01 |
| opendevmeet | The meeting name has been set to 'kolla' | 14:01 |
| mnasiadka | #topic rollcall | 14:01 |
| mnasiadka | o/ | 14:01 |
| SvenKieske | o/ | 14:01 |
| SvenKieske | :D | 14:01 |
| mmalchuk | o/ | 14:02 |
| mhiner | o/ | 14:02 |
| bbezak | o/ | 14:02 |
| jangutter | o/ | 14:03 |
| mnasiadka | #topic agenda | 14:03 |
| mnasiadka | * Review action items from the last meeting | 14:03 |
| mnasiadka | * CI status | 14:03 |
| mnasiadka | * Release tasks | 14:03 |
| mnasiadka | * Regular stable releases (first meeting in a month) | 14:03 |
| mnasiadka | * Current cycle planning | 14:03 |
| mnasiadka | * Additional agenda (from whiteboard) | 14:03 |
| mnasiadka | * Open discussion | 14:03 |
| mnasiadka | #topic CI status | 14:03 |
| mnasiadka | I think the CI is overall green, with some amber here and there | 14:03 |
| mnasiadka | but haven't seen any total breakage | 14:04 |
| mnasiadka | #topic Release tasks | 14:04 |
| mnasiadka | We've reverted the branch changes - so we're building from master now and running master in kolla-ansible | 14:04 |
| mnasiadka | What we omitted, is that we use master a-c-k on stable/2023.2 branch | 14:04 |
| mnasiadka | any volunteers to fix this? | 14:05 |
| mmalchuk | I can try | 14:06 |
| mnasiadka | ok then | 14:06 |
| mnasiadka | #topic Regular stable releases (first meeting in a month) | 14:07 |
| mnasiadka | Anybody happy to follow the regular stable releases procedure after mmalchuk fixes a-c-k in 2023.2? | 14:07 |
| bbezak | sure | 14:07 |
| bbezak | can do | 14:08 |
| mnasiadka | fantastic | 14:10 |
| mnasiadka | #topic Current cycle planning | 14:10 |
| mnasiadka | Let me check if we have Caracal priorities populated on the whiteboard | 14:10 |
| mnasiadka | https://etherpad.opendev.org/p/KollaWhiteBoard#L207 | 14:10 |
| mnasiadka | nope, only Kayobe it seems | 14:10 |
| SvenKieske | afaik there should be some stuff there, I fuzzily remember I wanted to pester the infra team with stuff to mirror? | 14:11 |
| SvenKieske | should we add stuff from https://etherpad.opendev.org/p/kolla-caracal-ptg#L111 ? not sure about the process here | 14:12 |
| mnasiadka | Yes, I'll populate it | 14:12 |
| mnasiadka | But not now, for the sake of the meeting | 14:12 |
| mnasiadka | Anything anybody is working on? | 14:12 |
| SvenKieske | okay, I see it. awesome :) | 14:12 |
| mnasiadka | Maybe the quorum queues backports? | 14:12 |
| SvenKieske | yeah I hope I get to that in the next weeks, I also have read your suggestions on changing things a bit | 14:13 |
| SvenKieske | just currently swamped with back-from-vacation and other stuff | 14:13 |
| mnasiadka | Sure :) | 14:13 |
| mnasiadka | Ok, I see nothing fancy | 14:15 |
| mnasiadka | let's go forward then | 14:15 |
| mnasiadka | #topic Additional agenda (from whiteboard) | 14:16 |
| mnasiadka | frickler is not around, but some topics from him | 14:16 |
| mnasiadka | I guess we'll talk about them next week | 14:16 |
| mnasiadka | seems nothing else on the whiteboard topics | 14:17 |
| mnasiadka | #topic Open discussion | 14:17 |
| mnasiadka | Anybody anything? | 14:17 |
| SvenKieske | I worked on shellcheck end of last year, if someone is also interested | 14:18 |
| mmalchuk | simple question | 14:18 |
| mmalchuk | can we create backports along with master change | 14:18 |
| mmalchuk | why cores show bad practice? | 14:19 |
| mmalchuk | kevko what do you think? | 14:19 |
| mnasiadka | you can create backports when you want, but to merge them we need the commit id from master | 14:20 |
| mmalchuk | so creating bunch of reviews is fine? | 14:20 |
| mmalchuk | ok then, thats all from me | 14:21 |
| mnasiadka | mmalchuk: what do you mean by creating bunch of reviews? | 14:24 |
| mnasiadka | creating backports when master is not merged yet? | 14:24 |
| mnasiadka | I agree it's an antipattern, and you need to refresh those backports after master gets merged | 14:24 |
| mnasiadka | And probably it creates some mess, but core's should always check if change is merged in master already | 14:25 |
| mnasiadka | seems we're done with the meeting | 14:26 |
| mmalchuk | mnasiadka too many reviews in gerrit without WIP | 14:26 |
| kevko | Hi | 14:26 |
| mnasiadka | Ah, just last but not least - I will be off for the next two weeks - bbezak gladly agreed to manage the meetings | 14:26 |
| mnasiadka | so please behave :) | 14:26 |
| mmalchuk | after master changes the same count of reviews in stable branches again and again | 14:26 |
| mmalchuk | with only source commit hash changed | 14:27 |
| mmalchuk | thats all | 14:28 |
| mmalchuk | mnasiadka thanks and good weekend | 14:28 |
| mnasiadka | kevko: the main ask is to create backports AFTER the master patch is merged | 14:29 |
| mnasiadka | I agree sometimes we backport something to check if it passes upgrade jobs or whatever | 14:29 |
| mmalchuk | yep | 14:29 |
| SvenKieske | yeah, sometimes it would be nice to be able to merge backports faster. well it is what it is. | 14:29 |
| mnasiadka | but that should be rather exception | 14:29 |
| mmalchuk | SvenKieske not in all cases) | 14:29 |
| mnasiadka | yes, a nice reminder to core reviewers to have a look on backports section of the dashboard | 14:29 |
| mnasiadka | ok, enough for today | 14:30 |
| mnasiadka | Thank you all for coming! | 14:30 |
| mnasiadka | #endmeeting | 14:30 |
| opendevmeet | Meeting ended Wed Jan 10 14:30:41 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:30 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/kolla/2024/kolla.2024-01-10-14.01.html | 14:30 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/kolla/2024/kolla.2024-01-10-14.01.txt | 14:30 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/kolla/2024/kolla.2024-01-10-14.01.log.html | 14:30 |
| wuchunyang | hi, cores, could you add the trove patch to review list ? | 14:31 |
| mnasiadka | can you post a link? | 14:31 |
| mmalchuk | wuchunyang it on review | 14:32 |
| wuchunyang | mmalchuk ok, thanks. | 14:32 |
| opendevreview | wu.chunyang proposed openstack/kolla-ansible master: Fix trove failed to discover swift endpoint https://review.opendev.org/c/openstack/kolla-ansible/+/904968 | 14:35 |
| mmalchuk | wuchunyang please post the link here as mnasiadka say to speedup review a bit | 14:36 |
| wuchunyang | ok no problem. | 14:36 |
| wuchunyang | https://review.opendev.org/c/openstack/kolla-ansible/+/904968 https://review.opendev.org/c/openstack/kolla-ansible/+/904963/4 | 14:37 |
| wuchunyang | these two patches are easy to review. we should merged it and back-port to stable branches. | 14:37 |
| mnasiadka | I see logging fix is sneaked in the first one, but I guess we can survive ;) | 14:38 |
| mmalchuk | I think it can be easily added back) | 14:39 |
| mnasiadka | wuchunyang: in the second one, we only have quorum queues in master and 2023.2, can you split this into two patches - with quorum queues (so L13 to L15 from https://review.opendev.org/c/openstack/kolla-ansible/+/904963/4/ansible/roles/trove/templates/trove-guestagent.conf.j2) in a separate patch stacked on top? so it's easier to backport the first one all the way to yoga, and quorum only to 2023.2? | 14:40 |
| wuchunyang | yes, logging fix also is needed to back-port, so i put them together | 14:40 |
| wuchunyang | mnasiadka ok, i will check it on yoga. then divide them to two patches. | 14:42 |
| opendevreview | Maksim Malchuk proposed openstack/kolla-ansible stable/2023.2: Switch openstack.kolla to correct branch https://review.opendev.org/c/openstack/kolla-ansible/+/905227 | 14:44 |
| mmalchuk | mnasiadka done ^^^ | 14:44 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible stable/2023.2: [2023.2 only] Switch openstack.kolla to correct branch https://review.opendev.org/c/openstack/kolla-ansible/+/905227 | 14:44 |
| mnasiadka | thanks | 14:45 |
| mnasiadka | bbezak: can you have a look at first wuchunyang's patch? | 14:46 |
| bbezak | willdo | 14:48 |
| bbezak | in spare moment :) | 14:48 |
| wuchunyang | hello cores. another question. i saw kolla pushes images to "kolla" organization in docker hub, how do we apply a "trove" organization in docker hub ? | 14:52 |
| opendevreview | wu.chunyang proposed openstack/kolla-ansible master: Fix trove guest agent failed to connect rabbitmq[1] https://review.opendev.org/c/openstack/kolla-ansible/+/904963 | 14:53 |
| opendevreview | wu.chunyang proposed openstack/kolla-ansible master: Fix trove guest agent failed to connect rabbitmq https://review.opendev.org/c/openstack/kolla-ansible/+/904963 | 14:58 |
| opendevreview | wu.chunyang proposed openstack/kolla-ansible master: Fix trove guest agent failed to connect rabbitmq https://review.opendev.org/c/openstack/kolla-ansible/+/904963 | 15:00 |
| opendevreview | wu.chunyang proposed openstack/kolla-ansible master: Fix trove guest agent failed to connect rabbitmq https://review.opendev.org/c/openstack/kolla-ansible/+/904972 | 15:00 |
| wuchunyang | mnasiadka bbezak hi, I divided into two patches. https://review.opendev.org/c/openstack/kolla-ansible/+/904963/7 https://review.opendev.org/c/openstack/kolla-ansible/+/904972/1 | 15:02 |
| mmalchuk | wuchunyang dockerhub not used anymore | 15:14 |
| mmalchuk | https://quay.io/ | 15:14 |
| mmalchuk | https://quay.io/organization/openstack.kolla | 15:15 |
| mmalchuk | is the right place for images | 15:15 |
| wuchunyang | mmalchuk i saw kolla ci uses both of dockerhub and quay.io. | 15:15 |
| bbezak | wuchunyang: maybe alter the name of second commit, as it is the same as previous one | 15:15 |
| mmalchuk | wuchunyang dockerhub used for old images | 15:16 |
| wuchunyang | bbezak can i just add 1 and 2 suffix ? | 15:16 |
| wuchunyang | mmalchuk how do we apply a quay.io organization? in trove project, we also need to build 4 docker images. but i don't know where to push. | 15:17 |
| mmalchuk | its better add something like (for stable branches) | 15:17 |
| wuchunyang | mmalchuk ok, no problem | 15:17 |
| mnasiadka | well, docker hub publishes seem to work now, but we publish weekly | 15:18 |
| mmalchuk | wuchunyang just use it as the namespace | 15:18 |
| mnasiadka | while we publish daily to quay.io | 15:18 |
| wuchunyang | i want to know who is the owner of the kolla docker hub organization ? | 15:18 |
| mmalchuk | kolla project | 15:19 |
| mnasiadka | me | 15:19 |
| mnasiadka | :) | 15:19 |
| mmalchuk | wuchunyang you shouldn't push images to quay.io yourself. just create right commits in the kolla project | 15:20 |
| mnasiadka | wuchunyang: basically register an account and open a free public project | 15:21 |
| wuchunyang | mnasiadka you use your own docker account to apply a docker-sponsored organization? right ? | 15:21 |
| mnasiadka | there is something called free team on Docker Hub | 15:22 |
| wuchunyang | mmalchuk not related to kolla images. just trove itself. as i am also trove PTL, i'd like to make it more easy to use. | 15:22 |
| mnasiadka | but Docker Hub has pull limits | 15:22 |
| mnasiadka | wuchunyang: https://www.docker.com/developers/free-team-faq/ | 15:22 |
| mnasiadka | so you can't sign up now for a free team | 15:23 |
| wuchunyang | we only build 4-8 images in trove project, doesn't the pull limits will affect us? | 15:23 |
| mmalchuk | wuchunyang youre interesting how to create your own CI and push images? ok then | 15:24 |
| mnasiadka | wuchunyang: no, pull limit is only for pulling, not pushing ;) | 15:24 |
| wuchunyang | mmalchuk yes, i don't know how to do, so i read the kolla ci code .Lol | 15:24 |
| mmalchuk | wuchunyang you can ask help in #opendev channel | 15:25 |
| mmalchuk | there are more CI involved peoples | 15:25 |
| mmalchuk | or even #opendev-infra | 15:26 |
| wuchunyang | ok, i will ask them for help. | 15:27 |
| opendevreview | wu.chunyang proposed openstack/kolla-ansible master: Fix trove failed to connect rabbitmq(stable/branch) https://review.opendev.org/c/openstack/kolla-ansible/+/904963 | 15:29 |
| wuchunyang | mnasiadka hi https://github.com/openstack/kolla/blob/master/.zuul.d/base.yaml#L19 this is your individual account ? | 15:32 |
| mnasiadka | wuchunyang: that is a robot account token tied to the free team | 15:32 |
| opendevreview | wu.chunyang proposed openstack/kolla-ansible master: Fix trove failed to connect rabbitmq(2023.2 branch) https://review.opendev.org/c/openstack/kolla-ansible/+/904972 | 15:33 |
| wuchunyang | mnasiadka I thought about it for a while, and I thought it would be more appropriate to use quay.io | 15:34 |
| mnasiadka | wuchunyang: especially that it seems you can't apply for Docker Free Team anymore | 15:34 |
| wuchunyang | because quay.io doesn't have pull limit problem | 15:34 |
| mnasiadka | wuchunyang: register an account, create an organisation, create a robot account inside that org for CI pushes - add secrets to zuul.yaml - and should work ;) | 15:35 |
| wuchunyang | you mean quay.io ? | 15:36 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: WIP: fail when systemd unit fails to stop https://review.opendev.org/c/openstack/kolla-ansible/+/891781 | 15:47 |
| opendevreview | Merged openstack/kolla-ansible master: Enable the Fluentd Plugin Systemd https://review.opendev.org/c/openstack/kolla-ansible/+/875983 | 16:00 |
| opendevreview | Jake Hutchinson proposed openstack/kolla-ansible master: Ironic parameter rework and default NTP server https://review.opendev.org/c/openstack/kolla-ansible/+/893031 | 17:02 |
| opendevreview | Merged openstack/kolla-ansible stable/2023.2: [2023.2 only] Switch openstack.kolla to correct branch https://review.opendev.org/c/openstack/kolla-ansible/+/905227 | 18:52 |
| greatgatsby | has anyone experienced crashing computes with lots of soft-lockups? We're running xena via kolla-ansible. We've seen this across multiple deployments and hardware profiles. | 18:54 |
| greatgatsby | sorry, we're on yoga, brain fart | 19:32 |
| *** jph4 is now known as jph | 23:48 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!