| chandankumar | sean-k-mooney: dviroel Hello, Please have a look at https://review.opendev.org/c/openstack/watcher/+/963857, 3 watcher-tempest-plugin patches related to zone migrations depends on it. thank you! | 07:46 |
|---|---|---|
| dviroel | hi all, watcher meeting will start in 15m, pls add your topics to our meeting agenda: | 11:45 |
| dviroel | https://etherpad.opendev.org/p/openstack-watcher-irc-meeting#L29 | 11:46 |
| dviroel | #startmeeting watcher | 12:00 |
| opendevmeet | Meeting started Thu Mar 5 12:00:10 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 o/ - who's around for today's meeting? | 12:00 |
| jgilaber | o/ | 12:00 |
| rlandy | o/ | 12:01 |
| amoralej | o/ | 12:01 |
| dviroel | courtesy ping: sean-k-mooney chandankumar | 12:02 |
| dviroel | so let's start with today's meeting agenda | 12:02 |
| dviroel | #link https://etherpad.opendev.org/p/openstack-watcher-irc-meeting#L29 | 12:02 |
| dviroel | as usual, feel free to add your own topics to the agenda | 12:02 |
| dviroel | there is a Review topic to place your changes that requires attention | 12:02 |
| dviroel | there is also a topic for Bugs, if you want to discuss any | 12:03 |
| dviroel | lets start from the top | 12:03 |
| dviroel | #Announcements | 12:03 |
| dviroel | #undo | 12:03 |
| opendevmeet | Removing item from minutes: #link https://etherpad.opendev.org/p/openstack-watcher-irc-meeting#L29 | 12:03 |
| dviroel | ah | 12:03 |
| dviroel | #topic Announcements | 12:03 |
| dviroel | ok, so here is again the meeting agenda: | 12:04 |
| dviroel | #link https://etherpad.opendev.org/p/openstack-watcher-irc-meeting#L29 | 12:04 |
| dviroel | first announcement | 12:04 |
| dviroel | lets look at Gazpacho schedule | 12:04 |
| dviroel | #link https://releases.openstack.org/gazpacho/schedule.html | 12:04 |
| dviroel | we should be in preparation for RC1, which is target to next week | 12:05 |
| chandankumar | o/ | 12:06 |
| dviroel | so for now we should focus on reviewing and merging important/critical fixes that we want include in this release | 12:06 |
| dviroel | it seems that we already have a list of bugs in the Review topic, so we can look them in a few | 12:06 |
| dviroel | s/bugs/patches | 12:07 |
| dviroel | It is also the week to get our release highlights merged in releases repo | 12:07 |
| dviroel | #link https://review.opendev.org/c/openstack/releases/+/978878 | 12:07 |
| dviroel | that not the right one | 12:08 |
| dviroel | #undo | 12:08 |
| opendevmeet | Removing item from minutes: #link https://review.opendev.org/c/openstack/releases/+/978878 | 12:08 |
| jgilaber | #link https://review.opendev.org/c/openstack/releases/+/976016 | 12:08 |
| jgilaber | ^^ that is the watcher patch | 12:08 |
| dviroel | ah thanks jgilaber | 12:08 |
| dviroel | please take a look and review, we need this merge this week | 12:09 |
| dviroel | i would ask our release liasions to vote once they agree | 12:09 |
| dviroel | it lgtm | 12:09 |
| dviroel | and for the recors, we also merge our release prelude | 12:10 |
| dviroel | #link https://docs.openstack.org/releasenotes/watcher/unreleased.html#prelude | 12:10 |
| dviroel | thanks rlandy for proposing both patches | 12:11 |
| amoralej | would https://blueprints.launchpad.net/watcher/+spec/skip-actions-in-pre-condition deserves a mention? | 12:11 |
| dviroel | amoralej: i think that is valid yes, it is a important feature | 12:11 |
| amoralej | that's user visible change | 12:11 |
| amoralej | also, i'd mention, now we can safely run multiple instances of the decision-engine and applier (because of the service monitors) | 12:12 |
| amoralej | I'll comment | 12:13 |
| dviroel | amoralej: yeah, please add a comment in the patch | 12:13 |
| dviroel | we can see if rlandy can update that for us | 12:13 |
| dviroel | other we can prpose a new PS | 12:13 |
| dviroel | thank amoralej++ | 12:13 |
| dviroel | one more announcement in the list | 12:14 |
| rlandy | will do | 12:14 |
| dviroel | PTG will be handled virtually on Apr 20-24 | 12:15 |
| dviroel | make your registration if you plan to join us: | 12:15 |
| dviroel | #link https://openinfra.org/ptg/ | 12:15 |
| dviroel | I also created a new etherpad for the PTG, so we can start adding topics that we plan to discuss | 12:16 |
| dviroel | #link https://etherpad.opendev.org/p/watcher-2026.2-ptg | 12:16 |
| dviroel | in the following weeks we can start proposing the dates that we want for watcher, and we may want to avoid conflicts with other projects (if possible) like cyborg and nova | 12:17 |
| dviroel | any other announcement before we continue? | 12:17 |
| sean-k-mooney | cyborg will have ptg session too but im expecting 1 cross project sesssion with nova | 12:17 |
| sean-k-mooney | i dotn know if we need one with watcher | 12:18 |
| sean-k-mooney | btu we can seee if there are any topics that cross over | 12:18 |
| sean-k-mooney | at some point watcher will need awarenes of instnace with cybrog devices | 12:18 |
| sean-k-mooney | as you canot currently live migrate tehm | 12:18 |
| dviroel | i was more worried about conflict since most of the people would like to join both | 12:18 |
| sean-k-mooney | so it will have to fall back to cold migration | 12:18 |
| rlandy | are there any other cross over teams? | 12:19 |
| rlandy | observability? | 12:19 |
| rlandy | where we want a combined session | 12:19 |
| sean-k-mooney | the telemetry team tend to have a very short ptg | 12:19 |
| sean-k-mooney | usually on one day for 2-4 hours | 12:20 |
| dviroel | yeah, maybe with observability to discuss about the future of datasources and possibly new metrics for watcher | 12:20 |
| sean-k-mooney | but yes they reached out | 12:20 |
| dviroel | but I don't see yet a cross session with cyborg, i don't want to overlap only | 12:20 |
| dviroel | both project sholdn't need the full week :) | 12:21 |
| dviroel | fow now, we can start adding the topic to the etherpad and see how much we plan to discuss too | 12:21 |
| dviroel | any other comment or annoucement? | 12:22 |
| dviroel | #topic Reviews | 12:23 |
| dviroel | I see that we have some reviews open that are candidates for this releae | 12:23 |
| dviroel | want to highlight them jgilaber ? | 12:23 |
| dviroel | #link https://review.opendev.org/c/openstack/watcher/+/958766 | 12:23 |
| jgilaber | yes, thanks dviroel | 12:24 |
| dviroel | ( Remove watcher_notifications from default value) | 12:24 |
| jgilaber | I think the first one is a good candidate since it's a trivial fix,although not terribly important | 12:24 |
| dviroel | the 'openstack.notifications' was added as default in previous release right | 12:25 |
| jgilaber | yes | 12:25 |
| amoralej | yes, it should be safe | 12:26 |
| jgilaber | the next three I would be ok with posponing them until we get the stable branch created, but I would like to get them in soon to enable volume migration testing | 12:26 |
| dviroel | i think that we are fine with the first one too | 12:26 |
| amoralej | It'd be good to the zone_migration ones in this cycle imo, otherwise, we should backport them as those are bug fixes | 12:26 |
| jgilaber | and the last one | 12:27 |
| jgilaber | #link https://review.opendev.org/c/openstack/watcher/+/978077 | 12:28 |
| jgilaber | I discovered the bug while working on the openstacksdk changes for nova_helper | 12:28 |
| jgilaber | it's a pre-existing bug but I think it would be good to get the fix | 12:28 |
| dviroel | the zone_migrations one I need to revisit, but it is a good fix to get into the release yes, and it open for a while now, but I want to look with more details | 12:29 |
| dviroel | the last one is an important fix too | 12:29 |
| dviroel | I will take a look too | 12:30 |
| jgilaber | thanks dviroel! that all from me | 12:30 |
| dviroel | and there is one more | 12:31 |
| dviroel | #link https://review.opendev.org/c/openstack/watcher/+/978077 (Handle missing fields building storage model XML or list) | 12:31 |
| * dviroel is slow today, maybe needs more coffee | 12:31 | |
| amoralej | yes, i added that, it fixes low bug, we can defer to next release although it's pretty trivial change | 12:32 |
| * dviroel fixing link again | 12:32 | |
| dviroel | #undo | 12:32 |
| opendevmeet | Removing item from minutes: #link https://review.opendev.org/c/openstack/watcher/+/978077 | 12:32 |
| dviroel | #link https://review.opendev.org/c/openstack/watcher/+/960265 Handle missing fields building storage model XML or list) | 12:32 |
| dviroel | yeah, we can focus in the other from the list for now | 12:33 |
| dviroel | thanks for proposing this one amoralej | 12:33 |
| amoralej | sure, np | 12:33 |
| dviroel | I will also take a look on https://review.opendev.org/q/project:openstack/watcher+status:open+branch:+master | 12:34 |
| dviroel | to see if we are missing something, but I think that's it | 12:35 |
| dviroel | any other reviews that you folks want to cover? | 12:35 |
| dviroel | #topic Bugs | 12:36 |
| dviroel | quick looking at LP bugs, there is one missing triage | 12:36 |
| dviroel | #link https://bugs.launchpad.net/watcher/+bug/2142219 | 12:37 |
| jgilaber | yes I opened this one recently | 12:37 |
| dviroel | as mentioned in the bug, same issue from zone_migration right | 12:37 |
| jgilaber | exactly | 12:37 |
| dviroel | when dealing with many instances/volumes this can turn to an additional problem | 12:37 |
| jgilaber | the storage capacity balance strategy has been tested quite less than zone migration | 12:38 |
| jgilaber | so we missed the problem | 12:38 |
| dviroel | the zone_migration related bug is | 12:38 |
| amoralej | did you check if we have what we need in the storagemodel to leverage on it instead of cinder api? | 12:38 |
| dviroel | #link https://bugs.launchpad.net/watcher/+bug/2119957 | 12:38 |
| jgilaber | amoralej, I did not check that | 12:38 |
| jgilaber | I submitted the report quickly so we have a tracker | 12:39 |
| dviroel | it is important to check that before removing yes | 12:39 |
| amoralej | just in case we have to extend the model for the migration | 12:39 |
| sean-k-mooney | we are currently not reporting the voluem info in the api | 12:39 |
| sean-k-mooney | so addign info for volumes wont affect it | 12:40 |
| jgilaber | it gets pools, snapshots and volume types | 12:40 |
| sean-k-mooney | as i noted before i dont think we shoudl continue to expose teh model in teh api | 12:40 |
| jgilaber | I don't think we have information about snapshots and volume types in the model | 12:40 |
| sean-k-mooney | so even if we add new values for the ocmptue side im not sure we shoudl include them in teh api respocne | 12:40 |
| dviroel | in last ptg we decided to freeze the datamodel api | 12:40 |
| sean-k-mooney | yep | 12:40 |
| amoralej | i didn't mean to get the info in the api but in the model itself | 12:41 |
| jgilaber | note that in this strategy it only uses the storage model | 12:41 |
| dviroel | but I didn't get a chance to move forward with this, but we should before adding any new attribute in compute model | 12:41 |
| amoralej | so that the strategy has all the info required | 12:41 |
| dviroel | jgilaber: yeah, it a problem only in compute model | 12:41 |
| sean-k-mooney | i still also belvie we shoudl be moving the model out of memory and into a graph db or sql db sonner rather then later | 12:41 |
| sean-k-mooney | we likely can add these fiels btu as it stands each desiion engione will have its own model in memory | 12:42 |
| sean-k-mooney | so they can have diffent views of the cloud | 12:42 |
| amoralej | yep, that's a good discussion, but a different one :) | 12:42 |
| dviroel | correct, maybe a good discussion for the next PTg | 12:42 |
| sean-k-mooney | well i bring it up mainly becasue fo size. i think we can likely add the reqruied fiels in thsi case | 12:43 |
| dviroel | +1 | 12:43 |
| sean-k-mooney | but we will evenutlly have issues with the size of hte model in the curernt pattern | 12:43 |
| amoralej | i finally could get real size data for the model, so that we can discuss in the ptg | 12:43 |
| sean-k-mooney | ack | 12:43 |
| dviroel | amoralej: nice thanks | 12:44 |
| dviroel | so for the bug, i would set to medium, as we did for the zone_migration one | 12:44 |
| amoralej | +1 | 12:44 |
| jgilaber | +1 | 12:44 |
| amoralej | and tags: decision-engine | 12:45 |
| jgilaber | and strategies | 12:45 |
| amoralej | right | 12:46 |
| dviroel | good point :) | 12:46 |
| dviroel | anything else to cove in this topic? | 12:47 |
| jgilaber | not from me | 12:47 |
| dviroel | ok, so we can move | 12:48 |
| dviroel | #topic Open Discussions | 12:48 |
| dviroel | anything that yo folks want to bring? | 12:48 |
| dviroel | I just want to highlight that 2024.2 jobs are failings | 12:48 |
| dviroel | i rechecked a 2024.2 ptach yesterday | 12:49 |
| dviroel | https://review.opendev.org/c/openstack/watcher/+/950264/1#message-af40543ff5aef0f527d1ec235f85368ab8dd236a | 12:49 |
| dviroel | it seems that most of them are failing with "No module named 'pkg_resources'" | 12:49 |
| jgilaber | the problem seems to be coming from keystonemiddleware | 12:50 |
| dviroel | so we would have to manually pin setuptools like we did for 2025.2? | 12:50 |
| sean-k-mooney | no | 12:51 |
| sean-k-mooney | we can do that for the docs job but not the rest in general | 12:51 |
| sean-k-mooney | this need to be fixed in the midelware package with an actully release on stable 2024.2 | 12:52 |
| sean-k-mooney | and htat need to happen vai the requiremetn repo with a uc bump | 12:52 |
| amoralej | 2024.2 is moving to EoL next month btw | 12:52 |
| sean-k-mooney | yep but there will be final release before then | 12:53 |
| dviroel | sean-k-mooney: is there someone working on that, do you know? | 12:53 |
| amoralej | there are ongoing fixes in requirements and middleware repo? | 12:53 |
| amoralej | yeah, that was my question actually | 12:53 |
| sean-k-mooney | dviroel: im not sure we were backporting the tempest pin but im unclear about the tox jobs | 12:53 |
| sean-k-mooney | amoralej: yes there are | 12:53 |
| sean-k-mooney | we have been trying to fix this in the sable btrnaches for 4-6 weeks | 12:54 |
| sean-k-mooney | we shoudl follow up in the qa or requirement channel | 12:54 |
| dviroel | ack, we have some backports in 2024.2 that we could merge before the final release.. | 12:55 |
| dviroel | I can follow up there and see where we are | 12:55 |
| amoralej | yep, that'd be good | 12:55 |
| sean-k-mooney | https://review.opendev.org/c/openstack/requirements/+/976602 might fix it | 12:55 |
| dviroel | ack, still not passing: https://review.opendev.org/c/openstack/requirements/+/976602?tab=change-view-tab-header-zuul-results-summary | 12:57 |
| dviroel | but lets see how it goes | 12:57 |
| dviroel | thanks for the updates sean-k-mooney | 12:57 |
| dviroel | anything else that you folks want to discuss in the last 3 min? | 12:58 |
| sean-k-mooney | https://review.opendev.org/c/openstack/requirements/+/976903 need to be cherry picked first i think to 2024.2 | 12:58 |
| dviroel | oh, interesting | 12:58 |
| sean-k-mooney | ya we need to fix the requriement repo to not depend on pkg_resouce first | 12:58 |
| sean-k-mooney | lets follow up with the requirements folks in there channel | 12:59 |
| sean-k-mooney | and see what the plan is | 12:59 |
| sean-k-mooney | we can do that seperatly | 12:59 |
| dviroel | sean-k-mooney++ | 12:59 |
| dviroel | oh last minute | 12:59 |
| dviroel | #topic Volunteers to chair next meeting | 12:59 |
| dviroel | someone want to chair next week? | 13:00 |
| dviroel | otherwise I can take it | 13:00 |
| jgilaber | I can do it dviroel | 13:00 |
| dviroel | nice, thanks jgilaber | 13:00 |
| dviroel | ok let's wrap up for today | 13:01 |
| dviroel | we will meet again next week | 13:01 |
| dviroel | thank you all for participating | 13:01 |
| dviroel | #endmeeting | 13:01 |
| opendevmeet | Meeting ended Thu Mar 5 13:01:20 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 13:01 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/watcher/2026/watcher.2026-03-05-12.00.html | 13:01 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/watcher/2026/watcher.2026-03-05-12.00.txt | 13:01 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/watcher/2026/watcher.2026-03-05-12.00.log.html | 13:01 |
| chandankumar | thanks dviroel ! | 13:01 |
| amoralej | thanks dviroel++ | 13:02 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Fix check for volume type in cinder helper migrate https://review.opendev.org/c/openstack/watcher/+/963857 | 16:15 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Fix small issues in cinder_helper unit test https://review.opendev.org/c/openstack/watcher/+/979029 | 16:15 |
| opendevreview | Merged openstack/watcher master: Add tests for get_dst_pool_and_type method https://review.opendev.org/c/openstack/watcher/+/964717 | 16:18 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Fix zone migration dst pool and type selection https://review.opendev.org/c/openstack/watcher/+/964718 | 16:28 |
| jgilaber | dviroel, sean-k-mooney thanks for the reviews! I've respined the zone_migration fix to address your comments | 16:30 |
| sean-k-mooney | https://review.opendev.org/c/openstack/watcher/+/964718/9..10/watcher/tests/unit/decision_engine/strategy/strategies/test_zone_migration.py#1571 | 16:31 |
| sean-k-mooney | shoudl that not have been self.assertEquial as you had in the regession test? | 16:32 |
| jgilaber | no that is correct. In the patch I changed a bit some of the tests and that one covers a case with two inputs for the same pool but different destinations | 16:38 |
| jgilaber | and the matching input does not specify the dst_type so it's None | 16:38 |
| sean-k-mooney | ack | 16:38 |
| rlandy | https://review.opendev.org/c/openstack/releases/+/976016 - updated | 16:50 |
| opendevreview | Merged openstack/watcher master: Remove watcher_notifications from default value https://review.opendev.org/c/openstack/watcher/+/958766 | 18:52 |
| opendevreview | Merged openstack/watcher master: Fix check for volume type in cinder helper migrate https://review.opendev.org/c/openstack/watcher/+/963857 | 19:30 |
| opendevreview | Merged openstack/watcher master: Fix zone migration dst pool and type selection https://review.opendev.org/c/openstack/watcher/+/964718 | 19:57 |
| opendevreview | Merged openstack/watcher master: Fix small issues in cinder_helper unit test https://review.opendev.org/c/openstack/watcher/+/979029 | 19:57 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!