| opendevreview | Merged openstack/devstack-plugin-prometheus master: Add shebang to mark bash files https://review.opendev.org/c/openstack/devstack-plugin-prometheus/+/995207 | 05:00 |
|---|---|---|
| opendevreview | Merged openstack/watcher master: Replace old operating system versions in install guide https://review.opendev.org/c/openstack/watcher/+/1000683 | 12:12 |
| dviroel | Hello folks, the watcher meeting will start in a few, in 4min o/ | 12:56 |
| dviroel | #startmeeting watcher | 13:00 |
| opendevmeet | Meeting started Thu Aug 13 13:00:35 2026 UTC and is due to finish in 60 minutes. The chair is dviroel. 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 'watcher' | 13:00 |
| dviroel | hi all o/ | 13:00 |
| winiciusallan[m] | o/ | 13:00 |
| dviroel | who is around today? | 13:00 |
| * dviroel just noticed that the meeting agenda had the old meeting time | 13:01 | |
| rlandy | o/ | 13:02 |
| dviroel | courtesy ping list: amoralej sean-k-mooney chandankumar morenod | 13:02 |
| dviroel | oh, i think that morenod is out too | 13:02 |
| amoralej_ | o/ | 13:02 |
| dviroel | let's start with today's meeting agenda while people join | 13:02 |
| dviroel | #link https://etherpad.opendev.org/p/openstack-watcher-irc-meeting#L33 (Meeting agenda) | 13:03 |
| dviroel | feel free to add your own topics to the agenda | 13:03 |
| dviroel | include reviews to the list, bugs if you want to discuss any | 13:03 |
| dviroel | #topic Announcements | 13:03 |
| dviroel | Feature Freeze in 2 weeks | 13:04 |
| dviroel | it is getting closer | 13:04 |
| dviroel | #link https://releases.openstack.org/hibiscus/schedule.html | 13:04 |
| dviroel | for now we can go through the patches open | 13:04 |
| dviroel | but lets do this in the review topic | 13:04 |
| dviroel | anyone has any other announcement to make? | 13:05 |
| dviroel | ok | 13:05 |
| dviroel | #topic Reviews | 13:05 |
| dviroel | the list starts with morenod patches | 13:05 |
| dviroel | the first one merged | 13:05 |
| dviroel | #link https://review.opendev.org/c/openstack/watcher-dashboard/ | 13:06 |
| dviroel | improve test coverage in dashboard | 13:06 |
| dviroel | which is awesome | 13:06 |
| dviroel | we should get some time reviewing it | 13:06 |
| dviroel | and | 13:06 |
| dviroel | #link https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/999254 | 13:06 |
| dviroel | "Skip tests with explicit destination host in multi-cell deployments" | 13:07 |
| dviroel | which is a good patch to get in | 13:07 |
| dviroel | we reviewed it a bit, and made some suggestions | 13:07 |
| dviroel | with the current state, there are still some things that can be improved in this patch, or maybe in a follow up | 13:07 |
| amoralej_ | i didn't have time to review that, sorry | 13:07 |
| amoralej_ | i see it has RP +2 | 13:08 |
| dviroel | yeah, I added | 13:08 |
| dviroel | I mentioned that would be good to add "can_migrate_between_any_hosts" validation to any other strategy, even if the strategy itseld defined the destination host | 13:08 |
| dviroel | since the test could fail in a multi-cell environment | 13:09 |
| dviroel | but we can also defer to a follow up and focus on the tests with explicit destination | 13:09 |
| dviroel | I think that is close to merge, but it would also be interesting to have one ci job testing the usage of CONF.compute.migration_dest_host and CONF.compute.migration_source_host | 13:10 |
| dviroel | I think that we should be able to get hostnames from zuul and set in one of the jobs too | 13:11 |
| sean-k-mooney | o/ | 13:11 |
| dviroel | I will let you folks comment there too | 13:11 |
| dviroel | maybe is more a RP+1 than +2 i think | 13:11 |
| amoralej_ | i think there was some pressure to get the fix | 13:12 |
| amoralej_ | so +2 can be justified | 13:12 |
| dviroel | ack, so please then review and we can sync on what is missing and what can be added in a follow up too | 13:13 |
| sean-k-mooney | has it been fixed to do ti properly | 13:13 |
| sean-k-mooney | using both source and dest | 13:13 |
| sean-k-mooney | i dont see a reason to rush this | 13:13 |
| dviroel | not yet I guess | 13:13 |
| dviroel | I think that he is on pto since our last review | 13:14 |
| dviroel | so we have to wait | 13:14 |
| sean-k-mooney | ill try and do a proper review after the meeitn and see | 13:14 |
| sean-k-mooney | https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/999254/5/watcher_tempest_plugin/tests/scenario/test_execute_host_maintenance.py#59 | 13:14 |
| sean-k-mooney | maybe | 13:14 |
| sean-k-mooney | i am seing it use the souce | 13:15 |
| sean-k-mooney | it just not skipping when it not set | 13:15 |
| dviroel | right, it uses when available, but in case destination is set, we also need the source, otherwise we may choose the same | 13:15 |
| sean-k-mooney | or doing the other fallback whcih is to select a host that is differnt form the dest | 13:15 |
| dviroel | rigth | 13:15 |
| sean-k-mooney | down stream job failures is not a resaon to expidite this when we coudl have temporlly skip the job downstream weeks ago | 13:16 |
| dviroel | but may not work when can_migrate_between_any_hosts is False | 13:16 |
| sean-k-mooney | *skipped the test | 13:16 |
| dviroel | correct, is more just a request to review other than "is needed" | 13:17 |
| sean-k-mooney | we could write a followup ourselve i may take a look at tat after i finsih reviwing | 13:17 |
| dviroel | ack, | 13:18 |
| sean-k-mooney | do we know when we expect morenod to be back | 13:18 |
| sean-k-mooney | if its later in the week we can wait | 13:18 |
| sean-k-mooney | well i guess if its monday | 13:18 |
| amoralej_ | i think next week | 13:18 |
| dviroel | next monday I think | 13:18 |
| amoralej_ | but i'm not sure | 13:18 |
| sean-k-mooney | ack | 13:18 |
| dviroel | ok, lets move on just reviewing for now | 13:19 |
| dviroel | next changes | 13:19 |
| dviroel | #link https://etherpad.opendev.org/p/watcher-2026.2-status | 13:19 |
| dviroel | lets move to our etherpad and focus on features, since our deadline is approaching | 13:19 |
| dviroel | for Openstacksdk topic | 13:19 |
| dviroel | I think that we kind of merged everything that was planned | 13:20 |
| dviroel | we moved ironic migration to next release | 13:20 |
| dviroel | only https://review.opendev.org/c/openstack/watcher/+/975498 stands | 13:20 |
| sean-k-mooney | ack, i didnt really follow that progress this release | 13:20 |
| sean-k-mooney | ah the retry helper | 13:20 |
| dviroel | but it is an improvement for the current configurations.. | 13:20 |
| dviroel | and can review that when time permits | 13:21 |
| sean-k-mooney | that potically coudl merge after FF | 13:21 |
| dviroel | right | 13:21 |
| dviroel | Audit Pipeline feature | 13:22 |
| dviroel | I proposed a *few* patches | 13:22 |
| dviroel | sometimes i mess with the topic name, sorry | 13:22 |
| dviroel | #link https://review.opendev.org/q/topic:%22bp/audit-pipeline%22+is:+open | 13:22 |
| amoralej_ | you proposed the entire feature ? | 13:22 |
| sean-k-mooney | i started on the first patch | 13:23 |
| dviroel | amoralej_: the feature is functional, but I still miss some details | 13:23 |
| sean-k-mooney | so i thikn we can merge https://review.opendev.org/c/openstack/watcher/+/995119 likely today | 13:23 |
| dviroel | the order would be | 13:23 |
| dviroel | #link https://review.opendev.org/c/openstack/watcher/+/995119 | Add MetricDataCache and integrate it into DataSourceBase | 13:23 |
| dviroel | #link https://review.opendev.org/c/openstack/watcher/+/999716 | Add metric query methods to BaseStrategy | 13:23 |
| dviroel | #link https://review.opendev.org/c/openstack/watcher/+/996849 | Audit Pipeline Object and DBAPI | 13:23 |
| dviroel | #link https://review.opendev.org/c/openstack/watcher/+/999891 | Add AuditPipeline lifecycle notifications | 13:23 |
| dviroel | #link https://review.opendev.org/c/openstack/watcher/+/1000031 | Add AuditPipeline execution handler and cascade execution mode | 13:23 |
| sean-k-mooney | i just started reading it before the meeting | 13:23 |
| dviroel | #link https://review.opendev.org/c/openstack/watcher/+/997427 | Add Audit Pipeline REST API endpoints | 13:23 |
| dviroel | not sure if we want to merge already, or wait for a review in all, you folks can decide | 13:24 |
| amoralej_ | sean-k-mooney, we were discussing how to merge it, should we review the entire chain before mergint the bottom ones? | 13:24 |
| dviroel | additionally we have | 13:24 |
| dviroel | #link https://review.opendev.org/c/openstack/python-watcherclient/+/1000709 | Add Audit Pipeline client support (microversion 1.8) | 13:24 |
| dviroel | #link https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/1000787 | Add scenario test support for the Audit Pipeline feature | 13:24 |
| sean-k-mooney | am we dont neesially have to waith for the entire chain dependign on how it broken up | 13:24 |
| sean-k-mooney | the first 2 are not dead code right | 13:25 |
| sean-k-mooney | they will take effect for normal audits | 13:25 |
| sean-k-mooney | and are tested via tempest as a reasult | 13:25 |
| dviroel | the tempest change is WIP, still need to add more test, but wanted to see it running on CI already | 13:25 |
| sean-k-mooney | so those 2 can merge | 13:25 |
| dviroel | so people can check service logs too | 13:25 |
| dviroel | right, the metric cache itself is a feature | 13:26 |
| dviroel | that does not depend on the audit pipeline | 13:26 |
| sean-k-mooney | i think we shoudl be ok with how you have broken it up to be honest | 13:26 |
| sean-k-mooney | i.e to merge it one by one | 13:26 |
| dviroel | the "Audit Pipeline Object and DBAPI" we could consider reviewing all first | 13:26 |
| dviroel | ack | 13:26 |
| sean-k-mooney | i was just gong to ask where the db change was | 13:26 |
| sean-k-mooney | it wraped on my client | 13:27 |
| sean-k-mooney | so ya we cna merge everything up to that | 13:27 |
| sean-k-mooney | i twould be good to review the rest of the chain at least minimally before doing the object/dbchange | 13:27 |
| dviroel | +! | 13:27 |
| dviroel | +1 | 13:27 |
| sean-k-mooney | and if you have a wip tempest to show it end ot end we can decied how to proceed based on that and our intiall pass | 13:27 |
| amoralej_ | ack, i think it's good to take the metric cache by itself and get those it | 13:27 |
| dviroel | yes, thre are 2 scenario tests working | 13:28 |
| sean-k-mooney | ok so conreate propsoal | 13:28 |
| dviroel | a 3 stage dummy, and a zone_migration + workload_balance | 13:28 |
| sean-k-mooney | if they are ready lest try and merge the first 2 patches this week and try and do a full pass before the next meeting | 13:28 |
| dviroel | nice | 13:29 |
| sean-k-mooney | i would propose holding +w on the object/db change until at least 1 +2 is on all the other patches at which point we can merge/review as normal | 13:29 |
| dviroel | makes sense to mee | 13:29 |
| dviroel | i think that amoralej_ also agree | 13:30 |
| amoralej_ | i do | 13:30 |
| dviroel | i will be updating some of the latest commits, based on teim analysis, but the core is proposed | 13:31 |
| dviroel | thanks for reviewing | 13:31 |
| dviroel | lets move to the next one so we can cover more features | 13:31 |
| dviroel | feel free to ping me for any questions | 13:31 |
| dviroel | - Functional Tests Infrastructure | 13:32 |
| dviroel | this is another blueprint | 13:32 |
| amoralej_ | yep | 13:32 |
| dviroel | first question | 13:32 |
| dviroel | should we go with | 13:32 |
| dviroel | #link https://review.opendev.org/c/openstack/watcher/+/994845 | 13:32 |
| dviroel | sean-k-mooney: wdyt? | 13:32 |
| sean-k-mooney | sorry got distracted | 13:33 |
| sean-k-mooney | reading back | 13:33 |
| dviroel | I am also happy with the initial test framework patch | 13:33 |
| sean-k-mooney | oh functional test | 13:33 |
| dviroel | #link https://review.opendev.org/c/openstack/watcher/+/988389 | 13:33 |
| sean-k-mooney | yes i think we can proceed | 13:33 |
| sean-k-mooney | ill also look at that after the meeting | 13:33 |
| dviroel | i am reviewing the emulators now | 13:34 |
| dviroel | +1 | 13:34 |
| dviroel | if we think that is in a good shape, follow up can be proposed if needed | 13:34 |
| dviroel | amoralej_: anything to add here? | 13:34 |
| sean-k-mooney | the reload on registration is still kind of a bug | 13:34 |
| sean-k-mooney | but we can adres that later | 13:34 |
| dviroel | ack, alfredo commented about that in the patch too | 13:35 |
| amoralej_ | I thiong we are like last week | 13:35 |
| dviroel | we can file a bug for it | 13:35 |
| amoralej_ | right? | 13:35 |
| dviroel | amoralej_: i believe so | 13:35 |
| amoralej_ | https://review.opendev.org/c/openstack/watcher/+/994845/4?usp=related-change and https://review.opendev.org/c/openstack/watcher/+/988389/11?usp=related-change should be fine to merge? | 13:35 |
| dviroel | so, any other question or comment on that? | 13:35 |
| dviroel | amoralej_: yeah, we should merge, also sean-k-mooney will take a quick look after the meeting | 13:36 |
| amoralej_ | ack | 13:36 |
| dviroel | i can W+1 in the end of the day today in case there is no -1 | 13:36 |
| sean-k-mooney | yes the first 3 we can proceed with | 13:36 |
| sean-k-mooney | ill +w them shortly | 13:36 |
| amoralej_ | once we get the nova emulators merged, i expect will be easy to apply same approach to the rest | 13:36 |
| dviroel | thanks sean-k-mooney | 13:36 |
| dviroel | nice, thanks amoralej_ | 13:36 |
| dviroel | next one in the list is | 13:37 |
| dviroel | - Preemtible Instances | 13:37 |
| dviroel | we have winiciusallan[m] around I think | 13:37 |
| winiciusallan[m] | o/ | 13:37 |
| dviroel | winiciusallan[m]: propose 3 patches | 13:37 |
| sean-k-mooney | ok cool so it has been split | 13:38 |
| dviroel | #link https://review.opendev.org/c/openstack/watcher/+/1000168 | 13:38 |
| dviroel | #link https://review.opendev.org/c/openstack/watcher/+/1000169 | 13:38 |
| dviroel | #link https://review.opendev.org/c/openstack/watcher/+/1000170 | 13:38 |
| dviroel | the first two are the actions | 13:38 |
| dviroel | which could be combined :) | 13:38 |
| dviroel | I added some comments about things that I see missing | 13:38 |
| sean-k-mooney | oh delete and shelve as sperate patchs | 13:39 |
| sean-k-mooney | yes but its fine | 13:39 |
| dviroel | we could have tempest-plugin testing those in actuator strategy | 13:39 |
| winiciusallan[m] | i saw your comments, doug. thanks for you feedback | 13:39 |
| sean-k-mooney | yep | 13:39 |
| dviroel | I linked the tempest for you to take a look | 13:39 |
| winiciusallan[m] | i splitted the actions in two separated patches based on meeting for some weeks ago | 13:39 |
| dviroel | should be feasible to get it working I think | 13:39 |
| winiciusallan[m] | if you guys prefer I can squash them if this will make the review easier | 13:40 |
| dviroel | winiciusallan[m]: if you want to run on a local devstack, I can help you | 13:40 |
| dviroel | the tempest test ^ | 13:40 |
| winiciusallan[m] | that would be great | 13:40 |
| dviroel | winiciusallan[m]: the actions also need a release notes, since they are a isolated feature | 13:40 |
| dviroel | but thats simple to fix | 13:40 |
| winiciusallan[m] | one release note for the two, right? | 13:41 |
| dviroel | winiciusallan[m]: sure, ping me if needed | 13:41 |
| dviroel | winiciusallan[m]: yes, thats why may be easier to combine, you can create a single one | 13:41 |
| sean-k-mooney | you can udpate date a common one | 13:41 |
| dviroel | ^ true | 13:41 |
| sean-k-mooney | you can even extend it with the final premept feature if you like although that can have its own too | 13:41 |
| sean-k-mooney | which ever you perfer | 13:42 |
| sean-k-mooney | they will all get complied into a singel doc anyway | 13:42 |
| dviroel | ack, we just need to have the feature note | 13:42 |
| winiciusallan[m] | ack | 13:42 |
| winiciusallan[m] | dviroel: i've addressed your comments and the changes are stashed on my laptop. i'll work today on the tempest changes to push them | 13:42 |
| dviroel | ack | 13:43 |
| dviroel | i also review the strategy | 13:43 |
| winiciusallan[m] | i decided to remove the retry logic that I added and use the existing wait_for_instance_state func | 13:43 |
| dviroel | s/review/reviewing | 13:43 |
| winiciusallan[m] | I'll push the change and you can check if it makes more sense | 13:43 |
| dviroel | ah ok, yeah, we have other cases of wait/sleep/retry in the helper | 13:44 |
| dviroel | we can see if any other configuration would be needed too, but yeah, propose it when ready | 13:44 |
| winiciusallan[m] | gotcha | 13:45 |
| dviroel | I would get the tempest-plugin also in hand, will help you to validate and reviewers to check it | 13:45 |
| dviroel | any other question folks? for the preemptible instances? | 13:45 |
| dviroel | please also include your comments to the gerrit review | 13:45 |
| dviroel | that was our last feature from the list | 13:46 |
| dviroel | there are some other improvements listed in the etherpad | 13:47 |
| dviroel | we already covered them last week | 13:47 |
| dviroel | some of them are pending updates | 13:47 |
| dviroel | anyone wants to bring any other review to the meeting? | 13:47 |
| dviroel | #topic Bugs | 13:48 |
| dviroel | no new bugs | 13:48 |
| dviroel | i also didn't planned to retriage any during this meeting | 13:48 |
| dviroel | since we would focus more on feature freeze for now | 13:49 |
| dviroel | #topic Volunteers to chair next week | 13:49 |
| dviroel | if someone wants to chair | 13:49 |
| dviroel | otherwise I can be the chair without problem | 13:49 |
| * dviroel will chair | 13:50 | |
| dviroel | #topic Open Discussions | 13:50 |
| dviroel | ^ i don't remember last time that we had time for that topic | 13:50 |
| amoralej_ | :) | 13:50 |
| dviroel | we have 10 min left, someone wants to bring any topic to discussion? | 13:51 |
| dviroel | other we can have those min back | 13:51 |
| dviroel | ok, i will take silence as a no | 13:51 |
| dviroel | let's wrap up for today | 13:51 |
| sean-k-mooney | cool | 13:52 |
| dviroel | let's wrap up for today | 13:52 |
| amoralej_ | thanks dviroel ! | 13:52 |
| dviroel | we will meet again next week | 13:52 |
| dviroel | thank you all for participating | 13:52 |
| dviroel | #endmeeting | 13:52 |
| opendevmeet | Meeting ended Thu Aug 13 13:52:20 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 13:52 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/watcher/2026/watcher.2026-08-13-13.00.html | 13:52 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/watcher/2026/watcher.2026-08-13-13.00.txt | 13:52 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/watcher/2026/watcher.2026-08-13-13.00.log.html | 13:52 |
| sean-k-mooney | i may have missed it but did we mention ptl election | 13:52 |
| sean-k-mooney | that the only topic i guess that could have been good to note | 13:53 |
| sean-k-mooney | amoralej_: dviroel since i have reviewd the funcitonl test already im going to review those first hten loop back to the caching change | 13:54 |
| amoralej_ | ack, thanks sean-k-mooney | 13:54 |
| dviroel | sean-k-mooney: ok | 13:54 |
| dviroel | sean-k-mooney: i didn't mention today yeah, I mentions in previous meetings when elections were officially open | 13:54 |
| dviroel | but true, deadline is next Wed | 13:55 |
| opendevreview | Winicius Allan Bezerra da Silva proposed openstack/watcher master: Add new delete and shelve actions https://review.opendev.org/c/openstack/watcher/+/1000168 | 15:20 |
| opendevreview | Winicius Allan Bezerra da Silva proposed openstack/watcher master: Add Workload Optimization goal and Workload Preemption strategy https://review.opendev.org/c/openstack/watcher/+/1000170 | 15:20 |
| dviroel | winiciusallan[m]: so you would abandon https://review.opendev.org/c/openstack/watcher/+/1000169 ? | 15:22 |
| winiciusallan[m] | dviroel: oh, thanks for the remainder | 15:23 |
| winiciusallan[m] | done | 15:23 |
| dviroel | thanks | 15:28 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!