| dviroel | hey folks, o/ | 11:55 |
|---|---|---|
| dviroel | watcher meeting will start in 5 mins | 11:55 |
| dviroel | #startmeeting watcher | 12:00 |
| opendevmeet | Meeting started Thu Jun 18 12:00:12 2026 UTC and is due to finish in 60 minutes. The chair is dviroel. Information about MeetBot at http://wiki.debian.org/MeetBot. | 12:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 12:00 |
| opendevmeet | The meeting name has been set to 'watcher' | 12:00 |
| dviroel | hi all o/ | 12:00 |
| chandankumar | o/ | 12:00 |
| jgilaber | o/ | 12:01 |
| amoralej | o/ | 12:01 |
| dviroel | courtesy ping: sean-k-mooney morenod rlandy | 12:01 |
| rlandy | o/ | 12:01 |
| dviroel | let's start with today's meeting agenda | 12:01 |
| dviroel | #link https://etherpad.opendev.org/p/openstack-watcher-irc-meeting#L31 (Meeting agenda) | 12:02 |
| dviroel | few topics today as expected since we are having a different meeting today | 12:02 |
| dviroel | #topic Announcements | 12:02 |
| dviroel | I believe that is important to remimder about our deadlines that are close | 12:03 |
| dviroel | Spec Freeze is July 2 (14 days) | 12:03 |
| dviroel | we are 2 weeks from spec freeze | 12:03 |
| * dviroel checking specs open | 12:03 | |
| dviroel | #link https://review.opendev.org/q/project:openstack/watcher-specs+is:+open | 12:03 |
| dviroel | we have one spec open yet | 12:04 |
| dviroel | #link https://review.opendev.org/c/openstack/watcher-specs/+/987171 | 12:04 |
| dviroel | is winiciusallan[m] spec about preemtible instances | 12:04 |
| amoralej | oh, i forgot to sent my comments, i just did | 12:04 |
| dviroel | reminder for us to read and provide feedbacks | 12:04 |
| dviroel | as also a reminder for winiciusallan[m] to keep updating the spec as needed in the following weeks | 12:05 |
| dviroel | to we can get the spec in | 12:05 |
| winiciusallan[m] | o/ | 12:05 |
| dviroel | since we kind already agree that is a nice to have feature | 12:05 |
| winiciusallan[m] | sorry this week was really busy, but i'll take some time to update it this week | 12:05 |
| dviroel | so we just need to handle the details now | 12:05 |
| dviroel | winiciusallan[m]: ack thanks, lets also try to review from our side here | 12:06 |
| winiciusallan[m] | thanks for pinging, doug | 12:06 |
| dviroel | so we don't let everything for the last week | 12:06 |
| dviroel | np winiciusallan[m] | 12:06 |
| dviroel | anyone has other watcher spec to propose yet? | 12:06 |
| amoralej | i expect to send one | 12:07 |
| amoralej | related to optimization of actionplans | 12:07 |
| dviroel | i was originally thinking on proposing a spec related to filtering invalid destination migration | 12:07 |
| amoralej | if i have the time, hopefully | 12:07 |
| dviroel | i may also propose this week if I have time, but unlikely to have time to implement it this release maybe | 12:08 |
| dviroel | but it would be good to have an initial idea too | 12:08 |
| amoralej | i have similar plan | 12:08 |
| dviroel | ack, so that's what I had in mind for announcements due to the deadline | 12:09 |
| dviroel | next topic then | 12:09 |
| dviroel | #topic Stable branches review day | 12:09 |
| dviroel | so we agreed last week that we would use today's meeting to review stable patches | 12:10 |
| dviroel | the idea is to use this remaining meeting time to go through stable patches and review them... | 12:10 |
| dviroel | now, how do we want to drive this? | 12:10 |
| dviroel | ideas? | 12:10 |
| dviroel | we can go one by one in sync here, with links | 12:11 |
| dviroel | or each one goes through the patches and just comment here specific concerns? | 12:11 |
| dviroel | thoughts? | 12:11 |
| jgilaber | we might have time to go through 2026.1 and 2025.2 | 12:11 |
| sean-k-mooney | o/ | 12:11 |
| jgilaber | but I doubt we'll get through everything | 12:12 |
| amoralej | yes | 12:12 |
| dviroel | #link https://review.opendev.org/q/project:openstack/watcher+branch:stable/2026.1+status:open | 12:12 |
| amoralej | that's easy :) | 12:12 |
| dviroel | so dowe go one by one then? | 12:12 |
| dviroel | 981225: Add watcher-grenade-skip-level-always job | https://review.opendev.org/c/openstack/watcher/+/981225 | 12:13 |
| jgilaber | I'd say yes, let's see how many we can go through | 12:13 |
| amoralej | btw, last days unit jobs have been failing | 12:13 |
| amoralej | randomly | 12:13 |
| dviroel | 2026.1 is a slurp | 12:13 |
| dviroel | ack, once we approve it, we can handle zuul async later I think | 12:14 |
| dviroel | not sure about the randim unit tests failures tbh | 12:14 |
| dviroel | logs already gone for this patch | 12:14 |
| amoralej | example https://zuul.opendev.org/t/openstack/build/a9c20c282fff4291a6d3cf0cf16b6da1 | 12:15 |
| dviroel | I would just check the results, if is jsing the correct versions and approve it | 12:15 |
| amoralej | but yeah, let's discuss later | 12:15 |
| dviroel | yep | 12:15 |
| dviroel | #link https://review.opendev.org/q/project:openstack/watcher+branch:stable/2025.2+status:open | 12:15 |
| dviroel | starting 2025.2 | 12:15 |
| dviroel | 980298: Handle missing fields building storage model XML or list | https://review.opendev.org/c/openstack/watcher/+/980298 | 12:16 |
| dviroel | done already | 12:16 |
| dviroel | i mean, lgtm | 12:17 |
| dviroel | handle failing when accessing the models, some try/except to avoid error in logs and so on... | 12:17 |
| dviroel | feel free stop me in any of those... | 12:18 |
| dviroel | 979129: Fix small issues in cinder_helper unit test | https://review.opendev.org/c/openstack/watcher/+/979129 | 12:18 |
| sean-k-mooney | the only thing ill note is while i think the backprot are valid | 12:19 |
| dviroel | actually this ^depends 979128: Fix check for volume type in cinder helper migrate | https://review.opendev.org/c/openstack/watcher/+/979128 | 12:19 |
| dviroel | so this one ^ | 12:19 |
| sean-k-mooney | as we are doing this review we need to also check that there are no changes form the preir release | 12:19 |
| sean-k-mooney | unless the change is called out in the commit message | 12:19 |
| sean-k-mooney | the only reall change i expect going to 2025.2 woudl be auto pep8 vs ruff | 12:20 |
| sean-k-mooney | or was that for master to 2026.1? | 12:21 |
| jgilaber | I think most of them will be from before we made the swith to ruff | 12:21 |
| dviroel | i don't recall | 12:21 |
| sean-k-mooney | anyway the frist oen was fine | 12:22 |
| sean-k-mooney | https://review.opendev.org/c/openstack/watcher/+/979128 | 12:22 |
| sean-k-mooney | you emthion this has a depency? | 12:22 |
| sean-k-mooney | oh you just mean https://review.opendev.org/c/openstack/watcher/+/979129/ need https://review.opendev.org/c/openstack/watcher/+/979128/ | 12:22 |
| dviroel | correct | 12:22 |
| sean-k-mooney | but that is ok because they are stacked in that order | 12:22 |
| dviroel | because I started top to botton https://review.opendev.org/q/project:openstack/watcher+branch:stable/2025.2+status:open | 12:23 |
| dviroel | there is now a list of zone_migration related fixes | 12:24 |
| sean-k-mooney | ack | 12:24 |
| sean-k-mooney | so ya both of those look fine to me so +2w | 12:24 |
| amoralej | yep, zuul should know how to handle if we add +W to both, right? | 12:24 |
| amoralej | as are stacked | 12:24 |
| dviroel | yes | 12:24 |
| dviroel | 979120: Add tests for get_dst_pool_and_type method | https://review.opendev.org/c/openstack/watcher/+/979120 | 12:24 |
| dviroel | this is the first one of zone_migration fixes | 12:25 |
| amoralej | those are also properly stacked | 12:25 |
| dviroel | right, I am reviewing them in the order already | 12:26 |
| dviroel | most of them amoralej already reviewed too, tks! | 12:26 |
| sean-k-mooney | same | 12:27 |
| amoralej | i did a pass some time ago, yep | 12:27 |
| sean-k-mooney | https://review.opendev.org/c/openstack/watcher/+/979952/1 is nextg | 12:28 |
| sean-k-mooney | or rather https://review.opendev.org/c/openstack/watcher/+/979953/1 | 12:28 |
| sean-k-mooney | since doug already did the other | 12:28 |
| dviroel | i already reviewed, but if you guys spot something, just W-1 | 12:29 |
| dviroel | :) | 12:29 |
| dviroel | i am on https://review.opendev.org/c/openstack/watcher/+/979953 | 12:29 |
| sean-k-mooney | its part of a diffent bug tecnially but ya it looks fine to me | 12:30 |
| dviroel | this one mentions other patch | 12:30 |
| dviroel | which merged in 2025.2 already https://review.opendev.org/c/openstack/watcher/+/968817 | 12:30 |
| dviroel | jgilaber: correct? ^ | 12:30 |
| jgilaber | yes IIRC the patch linked introduced a regression | 12:30 |
| dviroel | ack, so we need this in | 12:31 |
| jgilaber | we merged this on in master just after feature freeze at the time I think | 12:31 |
| sean-k-mooney | we have alrady backproted that https://review.opendev.org/c/openstack/watcher/+/968817 | 12:31 |
| sean-k-mooney | so the depency is in teh 2025.2 branch | 12:31 |
| sean-k-mooney | but not yet in 2025.1 | 12:31 |
| sean-k-mooney | so we will need to becareful when we get to that point | 12:32 |
| sean-k-mooney | but its fien on the current branch to continue | 12:32 |
| jgilaber | they are also stacked in the 2025.1 branch | 12:32 |
| dviroel | right, wouldn't be the case of squashing them | 12:32 |
| jgilaber | in a long chain | 12:32 |
| dviroel | ack | 12:32 |
| sean-k-mooney | the only time we squsion is if there is a gate bug that we cant resolve otherwise | 12:33 |
| sean-k-mooney | *squash | 12:33 |
| dviroel | but yeah, i would propose both in a single change to not miss the second one | 12:33 |
| sean-k-mooney | i would not | 12:34 |
| dviroel | not because is really needed | 12:34 |
| sean-k-mooney | and ask youto back port it properly by stackign them | 12:34 |
| sean-k-mooney | i get where your comming form but we really should try to preseve teh git history | 12:34 |
| jgilaber | ^^ that was my thinking | 12:35 |
| sean-k-mooney | in general you would get a merge conflict | 12:35 |
| sean-k-mooney | if you put them in the wrong order anyway | 12:35 |
| dviroel | the chain doesn't block the first one to get merged and miss the second one, that's my point | 12:36 |
| sean-k-mooney | in anycase i think we are done with 2025.2 | 12:36 |
| dviroel | yep | 12:36 |
| sean-k-mooney | right but for chains like this the better way to do it is to hold the +w on the bottom patch until we are happy with the chain | 12:36 |
| dviroel | we are done with 2025.2 | 12:36 |
| sean-k-mooney | then send them all at once | 12:36 |
| dviroel | ack, requires more attention | 12:37 |
| sean-k-mooney | i mean there is no wrong answer here really other then not reviewing them and fixing it | 12:37 |
| dviroel | yeah, agree | 12:37 |
| sean-k-mooney | s/fixing/backporting/ | 12:37 |
| dviroel | btw, we are done with the proposed patches | 12:38 |
| dviroel | but there are probably more fixes that could be backported | 12:38 |
| sean-k-mooney | ya so im not sure i fi have the mental capasity to rapid fire review 2025.1 | 12:38 |
| jgilaber | jfyi when all the patches land I'll propose a cherry pick for https://review.opendev.org/c/openstack/watcher/+/970381 which enables some zone migration tempest tests | 12:38 |
| jgilaber | it needed some of those fixes and I forgot to propose it before | 12:38 |
| sean-k-mooney | can i suggest we do this again for 2025.1 next week | 12:38 |
| amoralej | i was checking at master to see if we want to propose anything else to 2026.1, maybe the fix for https://bugs.launchpad.net/watcher/+bug/2152645 ? | 12:39 |
| jgilaber | sean-k-mooney, I think that's ok, we did great progress today | 12:39 |
| sean-k-mooney | or async and then we can prepare release for the next milestone | 12:39 |
| dviroel | amoralej: good call, I think that worth backporting | 12:39 |
| sean-k-mooney | ya that seam valid to me as well | 12:39 |
| amoralej | other than that, i think no much to backport from master, the rlock, i wuldn't backport it at least until we run it in master for some time | 12:40 |
| dviroel | you mean the fix for launchpad.net/bugs/2152254 | 12:41 |
| amoralej | yes | 12:41 |
| sean-k-mooney | https://review.opendev.org/c/openstack/watcher/+/989395 | 12:41 |
| amoralej | i'd start backporting only the ones for #2152645 | 12:41 |
| sean-k-mooney | for https://bugs.launchpad.net/watcher/+bug/2152645 | 12:42 |
| amoralej | https://review.opendev.org/c/openstack/watcher/+/989395/1?usp=related-change + https://review.opendev.org/c/openstack/watcher/+/989396/2?usp=related-change | 12:42 |
| sean-k-mooney | ya im not sure if we want to backport the full instance locking changes yet | 12:42 |
| amoralej | yep | 12:42 |
| sean-k-mooney | im not against that but i think that large change could do with baking in ci a bit longer | 12:42 |
| dviroel | sean-k-mooney: i think that i mixed bugs in topics and i will update LP with the correct one | 12:42 |
| sean-k-mooney | the notification locking is less invasive so we can start with that | 12:43 |
| sean-k-mooney | and then perhaps near milestone 3 see if we want the per instance locking changes as well | 12:43 |
| dviroel | https://review.opendev.org/c/openstack/watcher/+/989396 | 12:44 |
| dviroel | ok, I can propose the backports for the sync | 12:44 |
| amoralej | ack | 12:45 |
| dviroel | I'm not sure that I can do this in a chain using UI | 12:45 |
| dviroel | i will do that later from my laptop | 12:45 |
| dviroel | once we get 2025.2 patches in, we can do a review in 2025.1 too, maybe async | 12:46 |
| amoralej | no rush, let me know and i will review async | 12:46 |
| dviroel | what else we have for today? | 12:47 |
| dviroel | oh, there is a bug | 12:47 |
| dviroel | #topic Bugs | 12:47 |
| dviroel | #link https://bugs.launchpad.net/watcher/+bug/2156473 (vm_workload_consolidation strategy ignores allocation limits, overpacking instances onto nodes_ | 12:48 |
| dviroel | i see the issue... | 12:48 |
| amoralej | yeah, i sent a patch already but didn't fill importance | 12:49 |
| amoralej | for me is medium or high | 12:49 |
| dviroel | #link https://review.opendev.org/c/openstack/watcher/+/993088 (993088: Add allocation-based capacity checks to vm_workload_consolidation) | 12:49 |
| amoralej | i mentioned in last meeting but then i reproduced with a real environment | 12:50 |
| dviroel | just wondering if more strategies do the same thing... | 12:50 |
| amoralej | also, note that i stacked the fix for that on top of the one for https://bugs.launchpad.net/watcher/+bug/2141939 | 12:51 |
| amoralej | dviroel, we may easily test it with functional tests https://review.opendev.org/q/topic:%22blueprint-functional-test-infrastructure%22 :) | 12:51 |
| dviroel | +1 on functional tests infra, thanks a lot | 12:52 |
| dviroel | nee to take some time to look at it | 12:52 |
| amoralej | tbh, i want to dig more into validation | 12:52 |
| dviroel | for the Bug reported, I think that is ery important to fix, and there is no way to workaround the issue probably | 12:53 |
| amoralej | i know that node_workload_consolidation does it well | 12:53 |
| amoralej | hardware_maintenance has a tricky behavior | 12:54 |
| jgilaber | the bug seems valid to me and I could see it either as high or medium | 12:54 |
| amoralej | it checks if all the instances fit in backup_node if specified, if not all the instances in maintenance_host fits into the backup one, then it creates migrations without specifying destination and leaves nova to decide | 12:55 |
| dviroel | but if check only for usage, not allocation right | 12:56 |
| amoralej | only allocation, i mean hardware_maintenance | 12:56 |
| amoralej | hardware_maintenance only checks allocations, not usage at all | 12:56 |
| dviroel | ah ok | 12:56 |
| amoralej | look for "test_backup_insufficient_capacity" in https://review.opendev.org/c/openstack/watcher/+/993352/3/watcher/tests/functional/test_host_maintenance.py#389 | 12:57 |
| amoralej | that's how i learn it | 12:57 |
| amoralej | that seems to be by-design behavior | 12:57 |
| amoralej | but yeah, that's another strategy, sorry i diverged | 12:58 |
| dviroel | yeah. as long is documented, it is a ok behavior | 12:58 |
| dviroel | ok, wrt to the bug | 12:59 |
| dviroel | i would set to medium for now | 12:59 |
| amoralej | wfm | 12:59 |
| dviroel | we could say that works by design :) | 12:59 |
| dviroel | but the design isn't good | 12:59 |
| amoralej | it could be better :) | 13:00 |
| dviroel | since there is already a fix to it, we can discuss more in gerrit | 13:00 |
| dviroel | thanks for reporting and proposing the fix amoralej | 13:00 |
| dviroel | #topic Other discussions | 13:01 |
| dviroel | we are out of time | 13:01 |
| amoralej | yep | 13:01 |
| dviroel | but amoralej wants to propose that we change or shift our meeting time | 13:01 |
| amoralej | it'd be great if we could move this mtg to a different time | 13:01 |
| amoralej | it conflicts to other mtg that i'd like to atttend | 13:02 |
| dviroel | 1h later works for me, but we need more feedback from the team | 13:02 |
| sean-k-mooney | as long as its any time after about 11:00 utc i generaly have no issue with moving it | 13:02 |
| amoralej | we can add it as first topic in next mtg | 13:02 |
| dviroel | ack amoralej | 13:02 |
| dviroel | I also added a place to vote there async | 13:02 |
| jgilaber | I'd be fine with 1h our earlier or before, but personally I'd prefer after | 13:02 |
| sean-k-mooney | or keeping it at its current slot but ya lets revisit next week or do it async | 13:03 |
| dviroel | we can cover that next meeting and take the decision | 13:03 |
| amoralej | sure | 13:03 |
| dviroel | #topic Volunteers to chair next meeting | 13:03 |
| dviroel | i will be around, but if someone want to chair | 13:03 |
| jgilaber | I can do it dviroel | 13:03 |
| dviroel | thanks jgilaber | 13:03 |
| dviroel | let's wrap up for today | 13:03 |
| dviroel | thank you all for participating | 13:04 |
| dviroel | #endmeeting | 13:04 |
| opendevmeet | Meeting ended Thu Jun 18 13:04:09 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 13:04 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/watcher/2026/watcher.2026-06-18-12.00.html | 13:04 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/watcher/2026/watcher.2026-06-18-12.00.txt | 13:04 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/watcher/2026/watcher.2026-06-18-12.00.log.html | 13:04 |
| amoralej | thanks dviroel for chairing! | 13:04 |
| dviroel | ty all for participating | 13:04 |
| opendevreview | Merged openstack/watcher stable/2026.1: Add watcher-grenade-skip-level-always job https://review.opendev.org/c/openstack/watcher/+/981225 | 13:58 |
| opendevreview | Merged openstack/watcher stable/2025.2: Handle missing fields building storage model XML or list https://review.opendev.org/c/openstack/watcher/+/980298 | 13:58 |
| opendevreview | Merged openstack/watcher stable/2025.2: Add tests for get_dst_pool_and_type method https://review.opendev.org/c/openstack/watcher/+/979120 | 13:58 |
| dviroel | sean-k-mooney: hey, so I forgot to ask you about teim-ci | 14:07 |
| dviroel | do you intend to continue maintaining it? | 14:07 |
| amoralej | yeah, i also had thought on that | 14:13 |
| opendevreview | Merged openstack/watcher stable/2025.2: Fix zone migration dst pool and type selection https://review.opendev.org/c/openstack/watcher/+/979121 | 14:21 |
| opendevreview | Merged openstack/watcher stable/2025.2: Add test for zone migration audit without dst_type https://review.opendev.org/c/openstack/watcher/+/979952 | 14:21 |
| opendevreview | Merged openstack/watcher stable/2025.2: Ensure zone migration creates the correct action https://review.opendev.org/c/openstack/watcher/+/979953 | 14:44 |
| opendevreview | Merged openstack/watcher stable/2025.2: Test zone migration audit without destination https://review.opendev.org/c/openstack/watcher/+/979954 | 14:44 |
| sean-k-mooney | dviroel: amoralej so i can | 15:10 |
| sean-k-mooney | its currntly broken | 15:10 |
| sean-k-mooney | which is why its not reporting | 15:10 |
| sean-k-mooney | teh vm it was running on was getting like 34kbps clones for some reason that i have not debugged | 15:10 |
| sean-k-mooney | so i powered it off | 15:11 |
| sean-k-mooney | dviroel: amoralej can i assume that ye found it useful and would like it to return? | 15:11 |
| sean-k-mooney | fixign it is hopefully not hard i just need to sit down and do it, or have ai do it :) | 15:13 |
| dviroel | sean-k-mooney: I think that it is useful yes | 15:14 |
| dviroel | if you want you can consider a manual trigger too, so we trigger the job when we think that is better | 15:14 |
| sean-k-mooney | it supprot both "temi-ci: manual" will triger it on watcher and cyborg | 15:15 |
| dviroel | manual trigger is another demonstration of our interest in its report | 15:15 |
| sean-k-mooney | so im happy to go fix it and or make any tweaks folks want | 15:16 |
| sean-k-mooney | i looked at it breifly a day or two ago but it was like mindnight and when i saw the issue was networkign related i just went to bed :) | 15:17 |
| dviroel | it would be great, thanks sean-k-mooney - when you have some time of course | 15:18 |
| dviroel | i know that I can get similar outputs using your repo content | 15:18 |
| dviroel | I tested once | 15:18 |
| sean-k-mooney | ya i need to complete my refactor of that to make it more trivial to do | 15:18 |
| dviroel | so we can run it locally if needed until you get some time to debug the issue | 15:19 |
| sean-k-mooney | i have mostly been focusing my side project efforts on the pci-sim work for cyborg and ARD for the last little bit | 15:19 |
| sean-k-mooney | so i just didnt get back aorund to working on it | 15:19 |
| dviroel | no worries | 15:19 |
| sean-k-mooney | i booted back up the vm https://zuul.teim.app/t/main/status and its triing to clone | 15:34 |
| sean-k-mooney | 2026-06-18 15:33:14,795 DEBUG zuul.Repo: Cloning from https://******@review.opendev.org/a/openstack/watcher to /var/lib/zuul/executor-git/review.opendev.org/openstack/openstack%2Fwatcher | 15:34 |
| sean-k-mooney | i might just need to reboot the host or my home switch/routere/ it workign fine form my laptop but not form the pod on my k8s vm | 15:35 |
| sean-k-mooney | ya its just timing out | 15:37 |
| amoralej | sean-k-mooney, yes, i think it's useful, actually i found it improved over time | 15:42 |
| amoralej | i'd vote for having it back :) | 15:42 |
| dviroel | amoralej: if you are still around, which job did you mention that was failing/ | 16:55 |
| amoralej | lemme look for it | 16:55 |
| dviroel | build-openstack-releasenotes? | 16:55 |
| dviroel | this one failed for me ^ | 16:55 |
| dviroel | but I remember a openstack-discuss thread around that | 16:55 |
| amoralej | see https://review.opendev.org/c/openstack/watcher/+/961486 | 16:55 |
| amoralej | https://zuul.opendev.org/t/openstack/build/22abb77ec9af48eaaa2ab95f4da57a68 | 16:57 |
| amoralej | https://zuul.opendev.org/t/openstack/build/1303dcfee55840688299d48e3e63b59e | 16:57 |
| amoralej | actually, i saw a different message than those | 16:57 |
| amoralej | yeah and https://zuul.opendev.org/t/openstack/build/a9c20c282fff4291a6d3cf0cf16b6da1 | 16:57 |
| amoralej | last one is "sqlalchemy.exc.OperationalError: (sqlite3.OperationalError) database is locked" but previous ones are different | 16:58 |
| amoralej | all those are unit test jobs which are usually very reliable | 16:58 |
| opendevreview | Merged openstack/watcher stable/2025.2: Fix check for volume type in cinder helper migrate https://review.opendev.org/c/openstack/watcher/+/979128 | 17:00 |
| opendevreview | Merged openstack/watcher stable/2025.2: Fix small issues in cinder_helper unit test https://review.opendev.org/c/openstack/watcher/+/979129 | 17:00 |
| dviroel | i was not expecting database lock issues anymore in master branch, there is a lock to avoid that. maybe requires an investigation in this specific test | 17:14 |
| amoralej | yeah, that's what catched my attention | 17:14 |
| amoralej | i couldn't find any relevant merge that may affect | 17:15 |
| amoralej | at least in watcher | 17:16 |
| opendevreview | Joan Gilabert proposed openstack/watcher stable/2025.2: Enable extra zone migration tests on master branch https://review.opendev.org/c/openstack/watcher/+/993976 | 17:29 |
| opendevreview | Merged openstack/watcher-dashboard master: tox: Bump min_version to 4.28.0 https://review.opendev.org/c/openstack/watcher-dashboard/+/992645 | 18:04 |
| opendevreview | Merged openstack/watcher master: Remove unused PlacementHelper methods https://review.opendev.org/c/openstack/watcher/+/991146 | 20:26 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!