| opendevreview | Joan Gilabert proposed openstack/watcher master: Fix Hypervisor attribute access in saving_energy https://review.opendev.org/c/openstack/watcher/+/999387 | 09:57 |
|---|---|---|
| opendevreview | Joan Gilabert proposed openstack/watcher master: Add Ironic emulator and saving_energy functional tests https://review.opendev.org/c/openstack/watcher/+/999388 | 09:57 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Add IronicNode wrapper dataclass https://review.opendev.org/c/openstack/watcher/+/999389 | 09:57 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Migrate ironicclient to openstacksdk https://review.opendev.org/c/openstack/watcher/+/999390 | 09:57 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Remove python-ironicclient dependency https://review.opendev.org/c/openstack/watcher/+/999391 | 09:57 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Add Ironic emulator and saving_energy functional tests https://review.opendev.org/c/openstack/watcher/+/999388 | 10:04 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Add IronicNode wrapper dataclass https://review.opendev.org/c/openstack/watcher/+/999389 | 10:04 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Migrate ironicclient to openstacksdk https://review.opendev.org/c/openstack/watcher/+/999390 | 10:04 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Remove python-ironicclient dependency https://review.opendev.org/c/openstack/watcher/+/999391 | 10:04 |
| opendevreview | Ivan Anfimov proposed openstack/watcher-dashboard master: Drop Python 3.10 https://review.opendev.org/c/openstack/watcher-dashboard/+/986989 | 12:40 |
| jgilaber | #startmeeting watcher | 13:00 |
| opendevmeet | Meeting started Thu Sep 17 13:00:12 2026 UTC and is due to finish in 60 minutes. The chair is jgilaber. 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 |
| jgilaber | Hi! Who is around today? | 13:00 |
| jgilaber | while we gather feel free to add topics to the agenda https://etherpad.opendev.org/p/openstack-watcher-irc-meeting#L30 | 13:00 |
| jgilaber | courtesy ping: dviroel amoralej sean-k-mooney chandankumar morenod rlandy | 13:01 |
| dviroel | o/ | 13:01 |
| chandankumar | o/ | 13:01 |
| amoralej | o/ | 13:01 |
| morenod | o/ | 13:01 |
| jgilaber | let's start with the first topci | 13:03 |
| jgilaber | topic, sorry | 13:03 |
| jgilaber | #topic 2027.1 vPTG | 13:03 |
| rlandy | o/ | 13:04 |
| jgilaber | we have an etherpad with some topics already | 13:04 |
| jgilaber | #link https://etherpad.opendev.org/p/watcher-2027.1-ptg | 13:04 |
| jgilaber | thanks dviroel ++ | 13:04 |
| dviroel | i started adding some and populating them | 13:04 |
| dviroel | yeah, please add your topics there | 13:04 |
| dviroel | so we can better organise the sections that we plan to cover | 13:04 |
| dviroel | even without too many topics there | 13:05 |
| dviroel | i am proposing some slots for us | 13:05 |
| dviroel | past ptg we reserved Wed, Thu and Fri (part of it) | 13:05 |
| dviroel | i was thinking on taking Wed and Thu for Watcher | 13:06 |
| dviroel | and proposing 13 to 18 UTC time | 13:06 |
| dviroel | it is 5h long, but does't mean that we would need to use all | 13:06 |
| dviroel | we may wrap up earlier | 13:06 |
| dviroel | but I guess that this time we would not need more than that | 13:07 |
| jgilaber | I think that might be longer than we need, but we can alwasy decide to stop early | 13:07 |
| dviroel | correc | 13:07 |
| dviroel | I think only one day may not be enough | 13:07 |
| jgilaber | this would not clash with cyborg, which is good, not sure about nova | 13:07 |
| dviroel | so 2 days should be fine | 13:07 |
| dviroel | nova usually gets from Tue-Fri :) | 13:07 |
| jgilaber | ack, let's see if we have any conflicts, but for now this sounds good to me | 13:08 |
| dviroel | if we have a conflict on a topic that is interesting for Cyborg/Nova, we can decide to block the time slot in our agenda | 13:08 |
| dviroel | so we can join other sessions | 13:08 |
| jgilaber | +1, any other comment or question for PTG? | 13:08 |
| amoralej | 2 days lgtm | 13:09 |
| dviroel | I will work on getting us both days | 13:09 |
| dviroel | i don't think that the bot is ready yet | 13:09 |
| dviroel | since is still pointing to previous prg | 13:10 |
| dviroel | #linkhttps://ptg.opendev.org/ptg.html | 13:10 |
| dviroel | #undo | 13:10 |
| dviroel | #link https://ptg.opendev.org/ptg.html | 13:10 |
| jgilaber | probably, since we're almost a month out | 13:10 |
| amoralej | we still have work to be done from discussions in previous ptg, it may be good to do a recap on what we already did and what is still pending | 13:10 |
| dviroel | correct ^ | 13:11 |
| dviroel | there was lot of AIs | 13:11 |
| dviroel | jgilaber: ok, I think that all for vPTG for now | 13:13 |
| jgilaber | good point amoralej | 13:13 |
| jgilaber | ack, moving to the next topic | 13:13 |
| jgilaber | #topic New status etherpad | 13:13 |
| jgilaber | we have the status etherpad ready | 13:13 |
| dviroel | amoralej: you mean as a ptg topic? the recap? not now, right? | 13:13 |
| jgilaber | #link https://etherpad.opendev.org/p/watcher-2027.1-status | 13:13 |
| dviroel | jgilaber: yeah, I created a new one | 13:13 |
| dviroel | and added some open reviews that were in my radar | 13:14 |
| jgilaber | thanks! I see there are a few patches in there already | 13:14 |
| jgilaber | nice | 13:14 |
| jgilaber | any comment on this? | 13:15 |
| dviroel | i think that we can move on | 13:15 |
| jgilaber | #topic Small core-team adjustment | 13:15 |
| jgilaber | also from dviroel | 13:15 |
| dviroel | yeah | 13:16 |
| dviroel | i would like to propose an small adjustment | 13:16 |
| dviroel | since not all watcher-core members are cores in all watcher-repos | 13:16 |
| dviroel | i mean, some repos does not inherit from watcher-core | 13:16 |
| dviroel | i think that in this moment, based on current number and quality of the reviews from all watcher-core members | 13:16 |
| dviroel | make sense to also add all watcher-core members to watcher-tempest-plugin and watcher-dashboard | 13:17 |
| dviroel | which means adding jgilaber and amoralej for these, if not already there | 13:17 |
| jgilaber | +1 from me, it would help with reviews | 13:18 |
| morenod | +1 | 13:18 |
| amoralej | +1 from me too | 13:18 |
| dviroel | only sean-k-mooney seems to not be around | 13:18 |
| dviroel | but i already had a chat with them and we agreed on that too | 13:18 |
| chandankumar | +1 , it will be helpful for dashboard and tempest plugin reviews | 13:18 |
| dviroel | ack, watcher-specs already inherit from watcher-core, so no changes | 13:19 |
| dviroel | ack, I may sent an update email and update these soon | 13:19 |
| dviroel | thanks for the feedback | 13:19 |
| jgilaber | looks like we're in agreement, thanks for the proposal | 13:19 |
| jgilaber | if there is nothing else on this topic, moving to reviews | 13:20 |
| dviroel | sure | 13:21 |
| jgilaber | #topic Revies | 13:21 |
| jgilaber | #undo | 13:21 |
| opendevmeet | Removing item from minutes: #topic Revies | 13:21 |
| jgilaber | #topic Reviews | 13:21 |
| jgilaber | we have a few patches today | 13:21 |
| jgilaber | first from dviroel | 13:21 |
| * dviroel undo works for you :) | 13:21 | |
| jgilaber | #link https://review.opendev.org/c/openstack/watcher/+/973995 | 13:21 |
| dviroel | yeah | 13:21 |
| dviroel | there is actually 3 changes | 13:22 |
| dviroel | 1005317: Remove MAAS integration | 13:22 |
| dviroel | #link https://review.opendev.org/c/openstack/watcher/+/1005317 | 13:22 |
| dviroel | 1005575: Remove the eventlet based watcher-api server | 13:22 |
| dviroel | #link https://review.opendev.org/c/openstack/watcher/+/1005575 | 13:22 |
| dviroel | 973995: Remove eventlet support | 13:22 |
| dviroel | #link https://review.opendev.org/c/openstack/watcher/+/973995 | 13:22 |
| dviroel | all related with eventlet code removal | 13:22 |
| sean-k-mooney | o/ | 13:22 |
| dviroel | I splitted into 3 changes | 13:22 |
| sean-k-mooney | and yes im on board with the propsoed core team chagnes | 13:22 |
| dviroel | sean-k-mooney: ack, thanks for the feedback | 13:23 |
| dviroel | note that the second patch removes the eventlet based watcher-api server | 13:23 |
| jgilaber | I reviewed the first 2, lgtm, I need to go through the last one | 13:23 |
| sean-k-mooney | ill see if i can look at them tomorrow as well | 13:23 |
| dviroel | we may want to do more removals in api, that are not related to eventlet itself | 13:24 |
| dviroel | but removals related to api script generation | 13:25 |
| amoralej | those are for indri only, right? | 13:25 |
| dviroel | yes | 13:25 |
| dviroel | master is indri now | 13:25 |
| amoralej | thanks that is really "early in the cycle" :) | 13:25 |
| sean-k-mooney | yes we should be removing [project.entry-points."wsgi_scripts"] | 13:25 |
| dviroel | XD | 13:25 |
| sean-k-mooney | with no replacemnt as you can just directly point at the wsgi module path | 13:26 |
| dviroel | sean-k-mooney: yes, that's the missing point from api | 13:26 |
| dviroel | I can propose another patch for that too | 13:26 |
| sean-k-mooney | ack | 13:26 |
| sean-k-mooney | that will eventully be removed form pbr and stop working entrily anyway | 13:27 |
| sean-k-mooney | wsgi_scripts has been deprecatged for a a few releases now | 13:27 |
| dviroel | ++ | 13:27 |
| dviroel | any question about the eventlet removal ones? | 13:28 |
| dviroel | jgilaber: if not, we can move to the next one | 13:28 |
| jgilaber | ack, moving on | 13:28 |
| jgilaber | #link https://review.opendev.org/c/openstack/watcher-specs/+/1005099 | 13:28 |
| dviroel | ah | 13:29 |
| dviroel | based on previous meeting discussions | 13:29 |
| dviroel | I moved and updated the audit pipeline for this cycle | 13:29 |
| dviroel | there were some changes from previous merged one | 13:29 |
| sean-k-mooney | normally you shoudl not move it | 13:29 |
| sean-k-mooney | you could copy it | 13:29 |
| dviroel | yeah | 13:30 |
| sean-k-mooney | we shoudl preserve the orgianl one unchanged in the approved folder for the previosu cycle | 13:30 |
| dviroel | i didn't | 13:30 |
| sean-k-mooney | ack | 13:30 |
| dviroel | just used the wrong wording :) | 13:30 |
| sean-k-mooney | :) | 13:30 |
| * dviroel went to the patch to check | 13:30 | |
| jgilaber | yep, the previous one is not removed in the patch | 13:31 |
| dviroel | so yeah, it has some updates based on what has been discussed in the proposed patches | 13:31 |
| sean-k-mooney | same. i guess unless there is a substative change you want to dicuss we can proceed with the review | 13:31 |
| sean-k-mooney | i dont think this requires ptg dicussion correct | 13:31 |
| dviroel | i don't think that it needs | 13:31 |
| dviroel | small changes in some section to be more aligned with the implemention | 13:31 |
| sean-k-mooney | then this could be a nice candiate to try and get merged before then | 13:32 |
| dviroel | the hint is that when reviewing, you can do a diff from the 2026.2 version :) | 13:32 |
| dviroel | sean-k-mooney: ideally yeah, so we can focus in the new work too | 13:33 |
| sean-k-mooney | exactly | 13:33 |
| dviroel | and get the feature properly teste in CI for more time | 13:33 |
| dviroel | btw, audit pipeline is in the 2027.1 review etherpad | 13:34 |
| sean-k-mooney | the implemation will likely not merge before the ptg but i woudl hope we can complet eit by m1 or years end | 13:34 |
| dviroel | so you folks can get links there | 13:34 |
| dviroel | ack! | 13:34 |
| dviroel | if we don't have any more question, we can move to the next one | 13:34 |
| jgilaber | moving on | 13:35 |
| jgilaber | #link https://review.opendev.org/c/openstack/watcher/+/988348 | 13:35 |
| dviroel | ah, this is a improvement proposed in the past | 13:35 |
| dviroel | I updated with a release notes | 13:35 |
| dviroel | and some improvements mentioned by teim-cu | 13:35 |
| dviroel | s/teim-cu/teim-ci | 13:36 |
| sean-k-mooney | adoptign the native table supprot | 13:36 |
| dviroel | not sure if you folks remember previous discussions about that patch | 13:36 |
| dviroel | i couldn't find which meeting we discussed it last time | 13:37 |
| sean-k-mooney | im alwasy confilcite on that its better in the long term but its a litel harder to read in raw form but it does improve maintainablety so +1 | 13:37 |
| amoralej | we discussed in the past about implementing a generator for that or something like that | 13:37 |
| dviroel | i remember the generator part as a possible improvement yeah | 13:37 |
| amoralej | but would be follow-up, right? | 13:37 |
| sean-k-mooney | to lazy load the metrics as needed | 13:38 |
| dviroel | but we didn't disagree also on getting this improvement for leat_loaded | 13:38 |
| sean-k-mooney | but ya htat generaotr refactor can be a followup | 13:38 |
| sean-k-mooney | i think its till proably worth it | 13:38 |
| amoralej | yes, correct | 13:38 |
| dviroel | ack - we also know that part of the improvement in strategies came from the lock improvemente already merged | 13:38 |
| sean-k-mooney | skimming it i think its reasonable to proceed in its current form | 13:39 |
| amoralej | the generator approach didn't collide to that one | 13:39 |
| sean-k-mooney | but i would have to review it properly | 13:39 |
| dviroel | but still, there is space for improving more this strategy | 13:39 |
| dviroel | ack, so feel free to review and add comments there | 13:39 |
| dviroel | any other concerns? otherwise we can move to next ones | 13:40 |
| jgilaber | next up are two tempest patches | 13:41 |
| jgilaber | #link https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/993672 | 13:41 |
| jgilaber | #link https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/1002188 | 13:41 |
| dviroel | these 2 tempest-plugins changes are for both functionalities added in hibiscus | 13:41 |
| dviroel | so would be good to get some reviews and merge them when possible | 13:41 |
| dviroel | Add API tests for audit template default_parameters | 13:42 |
| dviroel | Add tests for delete and shelve actions | 13:42 |
| dviroel | yeah, that's it | 13:42 |
| dviroel | jgilaber: next one is a small one | 13:43 |
| dviroel | #link https://review.opendev.org/c/openstack/watcher-dashboard/+/986989 | 13:43 |
| dviroel | going to merge this one, was eaiting for ci results | 13:44 |
| jgilaber | thanks dviroel | 13:44 |
| jgilaber | that was the last of it, any last minute review to mention? | 13:44 |
| jgilaber | moving to bugs | 13:45 |
| jgilaber | #topic Bugs | 13:45 |
| jgilaber | I recently opened two | 13:45 |
| jgilaber | #link https://bugs.launchpad.net/watcher/+bug/2167513 | 13:45 |
| jgilaber | both are related to the ironic integration in some way | 13:45 |
| jgilaber | the first one is related to the migration of nova helper to use the sdk | 13:46 |
| dviroel | oh | 13:46 |
| dviroel | It doesn't surprise me that this is broken | 13:46 |
| jgilaber | because we don't have much testing we introduced the bug in the saveing energy strategy | 13:46 |
| dviroel | since we don't test it | 13:46 |
| jgilaber | yes, I would consider this low given it's been there for a couple of cycles now | 13:47 |
| dviroel | +1 | 13:47 |
| jgilaber | after the PTG dicussion we can decide if we fix it or not | 13:47 |
| dviroel | sure thing | 13:47 |
| jgilaber | any objections to set it to triaged and low? | 13:47 |
| dviroel | it is already IN progress | 13:48 |
| jgilaber | oh right, I forgot | 13:48 |
| dviroel | but not assigned to you | 13:48 |
| jgilaber | I pushed a patch for it as part of the ironic functional tests and it set it to in progress | 13:49 |
| dviroel | i wasn't automatically | 13:49 |
| jgilaber | I'll assign it to myself | 13:49 |
| dviroel | ack | 13:49 |
| dviroel | Low is fine I think | 13:49 |
| jgilaber | done | 13:49 |
| amoralej | +1 | 13:49 |
| jgilaber | second one | 13:49 |
| jgilaber | #link https://bugs.launchpad.net/watcher/+bug/2167461 | 13:49 |
| jgilaber | this is also been there for a while but is not caused by the sdk migration | 13:50 |
| jgilaber | it's quite a simple bug that was not found for lack of testing | 13:50 |
| dviroel | right | 13:50 |
| jgilaber | I think low is right here as well | 13:51 |
| dviroel | +1 on low | 13:51 |
| jgilaber | ack done | 13:51 |
| jgilaber | #topic Volunteers to chair next week | 13:52 |
| dviroel | thanks for working on those jgilaber | 13:52 |
| jgilaber | any volunteer? | 13:52 |
| jgilaber | I will miss the meeting next week | 13:52 |
| dviroel | the 24th works for me | 13:52 |
| dviroel | o/ | 13:52 |
| jgilaber | thanks dviroel | 13:52 |
| amoralej | dviroel++ | 13:52 |
| jgilaber | #topic Open Discussions | 13:52 |
| jgilaber | any last minute topic before we close? | 13:52 |
| dviroel | hum, i saw some tests failing in the ci recently | 13:53 |
| dviroel | but I didn't debug more than one job | 13:53 |
| dviroel | but I may file a bug if happens more often | 13:53 |
| dviroel | the one that I investigate | 13:53 |
| dviroel | it was a migration timeout (at least from my initial investigation) | 13:54 |
| dviroel | since the related migration didn't change from MIGRATING | 13:54 |
| dviroel | before the failure | 13:54 |
| dviroel | if you see something similar, we may need to open a bug to investigate and fix | 13:55 |
| sean-k-mooney | that may be load related | 13:55 |
| sean-k-mooney | but ya goot do monitor | 13:55 |
| sean-k-mooney | *good | 13:55 |
| dviroel | yeah | 13:55 |
| dviroel | ok, i don't have any other topic | 13:56 |
| sean-k-mooney | if it was in the ipv6 job | 13:56 |
| sean-k-mooney | then it may be an knwo now fixed issue in os-vif | 13:56 |
| sean-k-mooney | but not yet released | 13:56 |
| dviroel | sean-k-mooney: it wasn't the ipv6 | 13:56 |
| sean-k-mooney | ack | 13:56 |
| dviroel | was the aetos or prometheus one | 13:56 |
| jgilaber | thanks dviroel, anything else on this topic or any other? | 13:58 |
| jgilaber | #endmeeting | 13:59 |
| opendevmeet | Meeting ended Thu Sep 17 13:59:32 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 13:59 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/watcher/2026/watcher.2026-09-17-13.00.html | 13:59 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/watcher/2026/watcher.2026-09-17-13.00.txt | 13:59 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/watcher/2026/watcher.2026-09-17-13.00.log.html | 13:59 |
| amoralej | Thanks dviroel! | 13:59 |
| dviroel | thanks jgilaber++ | 13:59 |
| amoralej | thanks jgilaber! | 14:01 |
| amoralej | i messed up with chairs today :) | 14:01 |
| opendevreview | Merged openstack/watcher-dashboard master: Drop Python 3.10 https://review.opendev.org/c/openstack/watcher-dashboard/+/986989 | 14:52 |
| opendevreview | Merged openstack/watcher master: Remove MAAS integration https://review.opendev.org/c/openstack/watcher/+/1005317 | 16:47 |
| opendevreview | Douglas Viroel proposed openstack/watcher master: Remove the eventlet based watcher-api server https://review.opendev.org/c/openstack/watcher/+/1005575 | 20:35 |
| opendevreview | Douglas Viroel proposed openstack/watcher master: Remove eventlet support https://review.opendev.org/c/openstack/watcher/+/973995 | 20:35 |
| opendevreview | Douglas Viroel proposed openstack/watcher master: Remove the deprecated WSGI scripts https://review.opendev.org/c/openstack/watcher/+/1006113 | 20:35 |
| opendevreview | Douglas Viroel proposed openstack/watcher master: Remove eventlet support https://review.opendev.org/c/openstack/watcher/+/973995 | 20:52 |
| opendevreview | Douglas Viroel proposed openstack/watcher master: Remove the deprecated WSGI scripts https://review.opendev.org/c/openstack/watcher/+/1006113 | 20:52 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!