opendevreview | Merged openstack/watcher master: Update master for stable/2024.2 https://review.opendev.org/c/openstack/watcher/+/929414 | 08:07 |
---|---|---|
opendevreview | Merged openstack/watcher-dashboard stable/2024.2: Update TOX_CONSTRAINTS_FILE for stable/2024.2 https://review.opendev.org/c/openstack/watcher-dashboard/+/929418 | 08:07 |
rlandy | hi all ... reminder Watcher meeting here in 30 mins | 11:27 |
rlandy | please add any topics to https://etherpad.opendev.org/p/openstack-watcher-irc-meeting | 11:28 |
rlandy | #startmeeting Watcher IRC Weekly Meeting - 6 February 2025 | 12:00 |
opendevmeet | Meeting started Thu Feb 6 12:00:52 2025 UTC and is due to finish in 60 minutes. The chair is rlandy. 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_irc_weekly_meeting___6_february_2025' | 12:00 |
rlandy | o/ hi folks | 12:01 |
marios | hello o/ | 12:01 |
rlandy | let's see how's around | 12:01 |
rlandy | who is around | 12:01 |
amoralej | o/ | 12:02 |
mtembo | o/ | 12:02 |
dviroel | o/ | 12:02 |
sean-k-mooney | o/ | 12:03 |
rlandy | ok ... let's begin with the listed topics | 12:04 |
rlandy | #topic: (rlandy) Any more action needed on the DPL model discussed last week? | 12:04 |
rlandy | Are we all set there? | 12:04 |
marios | i *think* it merged? checking | 12:04 |
marios | yeah | 12:04 |
marios | we have https://review.opendev.org/c/openstack/governance/+/940167 which establishes we use dpl for coming cycle | 12:05 |
sean-k-mooney | the first patch did yes | 12:05 |
marios | the only outstanding (and it may stay that way) is the "add watcher project update liaison" https://review.opendev.org/c/openstack/governance/+/940168 | 12:05 |
sean-k-mooney | the second patch to re-add chenker is pending | 12:05 |
rlandy | ok - do we need any more actions there? | 12:05 |
sean-k-mooney | they -1'd it but we are unsure if that means they dont want ot be liason or do but set the wrong flag | 12:06 |
sean-k-mooney | no | 12:06 |
marios | not that i'm aware of | 12:06 |
sean-k-mooney | we are just waiting for them to respond to gmann's comment | 12:06 |
rlandy | ok thanks sean-k-mooney and marios - moving on ... | 12:06 |
rlandy | #topic: (rlandy) Patches still requiring review or merge | 12:06 |
rlandy | anyone has reviews to add today? | 12:07 |
amoralej | not me, thanks marios for your reviews! | 12:07 |
marios | been trying to merge some of the older patches in the last few days with the new voting rights | 12:07 |
marios | thanks amoralej | 12:07 |
marios | especially some of the obvious ones like autogenerated/bot patches that have been sitting for weeks | 12:08 |
rlandy | chandan is out today but may have reviews on the testing/tempest plugin side | 12:08 |
rlandy | #action: rlandy to try dig those out and post here | 12:08 |
rlandy | after meeting | 12:08 |
marios | i have an item later in the agenda | 12:09 |
marios | but i will just add it here | 12:09 |
marios | #info prometheus data source vm metrics merged https://review.opendev.org/c/openstack/watcher/+/938893 | 12:09 |
rlandy | would you like to go now - since we are on the reviews topic? | 12:09 |
marios | thanks amoralej | 12:09 |
marios | done rlandy ;D | 12:09 |
marios | thanks | 12:09 |
rlandy | :) | 12:09 |
sean-k-mooney | so while we have not fuly finished hte blueprint/spec | 12:10 |
rlandy | any others? | 12:10 |
sean-k-mooney | the mvp code is now merged. | 12:10 |
marios | are you referring to the keystone integration sean-k-mooney ? or sthing else | 12:10 |
sean-k-mooney | so we just need to focus on the testing to complete the remainging work | 12:10 |
marios | ah i see, you mean ci side | 12:10 |
amoralej | +1 | 12:10 |
sean-k-mooney | marios: the tempest job | 12:11 |
marios | ack | 12:11 |
sean-k-mooney | so i think we are making good progress on compelting that this cycle | 12:11 |
sean-k-mooney | so ya we can move on | 12:11 |
rlandy | ok - moving on ... | 12:12 |
rlandy | #topic: (rlandy) reminder: PTG items list is on notes doc: https://etherpad.opendev.org/p/openstack-watcher-irc-meeting | 12:12 |
rlandy | just a reminder that we added a list there for anything that comes to mind to raise at PTG | 12:12 |
rlandy | marios (I think) responded to the initial form - thank you | 12:13 |
marios | yeah last week or whenever and we mentioned it in last irc meet | 12:13 |
marios | i'll update if/when i hear somethign further there for choosing dates/times etc | 12:13 |
rlandy | +1 | 12:14 |
rlandy | any other questions on PTG readiness? | 12:15 |
amoralej | you want us to start adding topics to discuss in the etherpad or only about organization so far? | 12:16 |
rlandy | both - we can triage what we actually bring | 12:17 |
rlandy | but good for us to start gathering ideas | 12:17 |
amoralej | ack | 12:18 |
rlandy | then we will know how many sessions to request | 12:18 |
rlandy | we can review them closer to the time | 12:19 |
rlandy | ok ... looks like we are done with the PTG discussion | 12:20 |
amoralej | we could start by creating several sections for topicts to discuss, API, service, strategies, dashboard or something like that | 12:20 |
rlandy | ack - anything we want to do in that cycle | 12:21 |
sean-k-mooney | that an option. we may want to that after we have the list of topic | 12:21 |
dviroel | +1 | 12:21 |
amoralej | make sense | 12:22 |
rlandy | anything more here? | 12:23 |
rlandy | ok - let's continue ... the bug list is next on the etherpad - thank you mtembo for adding it - would like to cover that last as it takes time | 12:25 |
rlandy | #topic: (rlandy) Volunteers for chair for Feb 13 meeting | 12:26 |
mtembo | ok | 12:26 |
rlandy | thank you mtembo for volunteering :) | 12:26 |
rlandy | Fed 13th is yours to chair | 12:26 |
mtembo | ack | 12:26 |
rlandy | moving on ... | 12:27 |
rlandy | #topic: (marios): reminder coming feature freeze Feb 24 - Feb 28 https://releases.openstack.org/epoxy/schedule.html | 12:27 |
marios | just a reminder nothing much to add here ... epoxy feature freeze coming in 3 weeks ish | 12:27 |
marios | apart from the prometheus data source work i am not aware of something else going on | 12:27 |
marios | we have some outstanding bits for the ci side but i think we should be OK | 12:28 |
marios | we can move on rlandy unless there is anything else here thank you | 12:28 |
rlandy | and dviroel piece in nova? | 12:28 |
marios | yeah but that is not for epoxy i think? | 12:28 |
dviroel | it is for epoxy in nova | 12:28 |
marios | ah | 12:28 |
marios | my bad | 12:28 |
dviroel | i've got some reviews in the main patch, it was also updated to solve some issues | 12:29 |
dviroel | waiting for more feedback there | 12:29 |
dviroel | #link https://review.opendev.org/q/topic:%22bp/show-scheduler-hints-in-server-details%22 | 12:30 |
dviroel | in case someone wants to take a look :) | 12:30 |
rlandy | sean-k-mooney: are you still planning more for the apscheduler stuff? iiuc, we're leaving it as is for now | 12:30 |
rlandy | eventlet | 12:30 |
sean-k-mooney | correct. i do not have a good repoducer. so for now i think we shoudl put that on hold but disucss the path forward at the ptg | 12:31 |
rlandy | ok - thank you | 12:31 |
sean-k-mooney | in the next 1-3 release we need to work on removing watchers use of eventlet so that might be how we resolve the remianing issues | 12:32 |
sean-k-mooney | the other possiblity is moving to usign the oslo.db engine fascade context manager | 12:33 |
sean-k-mooney | to better handlel the session/conneciton lifetime | 12:33 |
sean-k-mooney | both are non trivial so we shoudl dicuss that before investing mroe time | 12:33 |
rlandy | +1 | 12:34 |
sean-k-mooney | i consider both to be "tech debt" that shoudl eventually be adressed but im not sure when | 12:34 |
sean-k-mooney | btu we can move on for now | 12:34 |
rlandy | ok - anything more on epoxy feature freeze? | 12:34 |
sean-k-mooney | i gues i have two quick points | 12:34 |
rlandy | pls go ahead | 12:35 |
sean-k-mooney | i will proably start workign on the release prelude adn summary patches in the next week or two in advace of rc1 | 12:35 |
marios | thanks for doing that sean-k-mooney | 12:35 |
marios | let me know if i can help | 12:35 |
sean-k-mooney | so once i have those created ill ask for review. this is part fo the standard release duties. | 12:35 |
marios | i did some releases but a while ago | 12:36 |
dviroel | tks sean-k-mooney++ | 12:36 |
sean-k-mooney | yep im sure you can espically with wording | 12:36 |
sean-k-mooney | the other item is we shoudl consdier if we want to deprecate any of the goals/stragies | 12:36 |
sean-k-mooney | we dicussed perhaps deprecating the noisy neighbour one | 12:37 |
sean-k-mooney | since the metric it depned on is not provided by any of the backend any more | 12:37 |
sean-k-mooney | while we shoudl not remove it this cycle. if we intend to do that in the next 2 cycles we need to deprecated it this cycle | 12:37 |
amoralej | yes, i think it's a good candidate to be deprecated | 12:38 |
marios | sean-k-mooney: if i recall requirement is 1 cycle as deprecated before removal right? | 12:38 |
marios | or is it 2? | 12:38 |
sean-k-mooney | the actual requirement is the deprecation must ship in a slurp release before it can be removed | 12:38 |
sean-k-mooney | so in a .1 release like epoxy | 12:38 |
marios | ah | 12:38 |
marios | so if we don't, its next year ten | 12:39 |
marios | then | 12:39 |
sean-k-mooney | yep | 12:39 |
sean-k-mooney | so we can reflect on that and email the list for extra feedback | 12:39 |
marios | sean-k-mooney++ | 12:39 |
rlandy | ack - thank you for raising that | 12:40 |
sean-k-mooney | if we are going to do this i would like to do it before i create teh release prelude/cycle highlights so we can refence it | 12:40 |
sean-k-mooney | that was all i had for release topics. | 12:40 |
rlandy | ok - great | 12:40 |
rlandy | thanks, marios ... you have the next topic as well ... | 12:41 |
rlandy | #topic: new cores per proposal by smooney https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/message/UHTZVEZEKE37NYWNGXXUK5G4SHQH7TDS/ | 12:41 |
rlandy | marios: ^^ | 12:41 |
marios | thanks rlandy ... so just an announcement that there was no negative feedback to sean-k-mooney proposal and we have new cores | 12:41 |
marios | so we can clear the backlog and avoid patches sitting for weeks | 12:41 |
rlandy | good news - thank you | 12:41 |
sean-k-mooney | yep, i forgot to do it before goint on pto firday but i implemtned it at the weekend when i remmeberd | 12:42 |
sean-k-mooney | i emailed hte list on tuesday when i got back | 12:42 |
rlandy | sean-k-mooney++ | 12:42 |
marios | thanks sean-k-mooney and thanks rlandy ... if there is no other comment we can move on | 12:42 |
rlandy | congrats to the new cores | 12:43 |
dviroel | we already used our new powers to merge some pending changes :) | 12:43 |
marios | +1 | 12:43 |
marios | +2 even ;) | 12:43 |
sean-k-mooney | i saw :) | 12:43 |
rlandy | ok ... moving on | 12:43 |
rlandy | next is the bug triage | 12:43 |
rlandy | #topic: (mtembo) Bug list for review | 12:44 |
mtembo | we spoke about this one last weeek: thats where we ended https://bugs.launchpad.net/watcher-tempest-plugin/+bug/2095416 | 12:44 |
rlandy | #link: https://bugs.launchpad.net/watcher-tempest-plugin/+bug/2095416 | 12:45 |
rlandy | dviroel, mtembo looks like you worked on this ... | 12:45 |
rlandy | the bug is still in New | 12:46 |
mtembo | it is fixed from the numerous tests ran thereafter | 12:46 |
dviroel | no, i just added a comment about our last discussion on irc | 12:46 |
rlandy | ok - so let's close it out? | 12:46 |
mtembo | We should close it | 12:46 |
rlandy | ok wait ... sorry dviroel | 12:46 |
dviroel | rlandy: i mean that I didn't work on this bug | 12:47 |
marios | one comment for future reference, please include a link to job logs in the bug? i cannot see it in the description or did i miss it | 12:47 |
rlandy | good point | 12:47 |
marios | i mean the trace in the description looks like it could be from a ci job if that is the case please include the link | 12:47 |
marios | so was there a fix for this or how did the tests pass | 12:48 |
rlandy | mtembo: could you update the bug with latest results and close out? | 12:49 |
sean-k-mooney | so ya that not clear to me what is the fix? | 12:49 |
mtembo | not directly as far as i understand | 12:49 |
sean-k-mooney | we shoudl not move things to fix commeited if we do not know how it was fix. we shoudl move it to invlaid or incomplete | 12:49 |
mtembo | moved it fixed commited. thanks smooney | 12:50 |
marios | no mtembo not fix committed | 12:50 |
mtembo | I did include the link the latest tests ran | 12:50 |
mtembo | #link: https://review.opendev.org/c/openstack/watcher/+/940293 | 12:51 |
sean-k-mooney | im saying fix commited is wrong | 12:51 |
marios | fix committed means we committed some fix for that but i don't think that is the case here so as sean-k-mooney said above we should move it to incomplete or invalid | 12:51 |
rlandy | but reading back ... we still don't know what fixed the bug | 12:51 |
rlandy | ack | 12:51 |
mtembo | noted. thanks | 12:51 |
sean-k-mooney | ok so my orgianl theory was it was fixed by the orgianl asp schduler fix or dougs fix | 12:51 |
mtembo | moved to invalid | 12:51 |
sean-k-mooney | if it nolonger repoduces we can move it to invlaid or incompele | 12:52 |
mtembo | done | 12:52 |
mtembo | moving on: | 12:52 |
mtembo | link: https://bugs.launchpad.net/watcher/+bug/2096991 | 12:52 |
marios | i think this is valid but its more like a feature request/enhancement not a bug per se...? | 12:53 |
amoralej | that's mine, i reported to track something we discover | 12:53 |
sean-k-mooney | so this is valid alhtough it really a feature not a bug IMO | 12:53 |
marios | (unironically sean-k-mooney ;) ) | 12:54 |
amoralej | so, how should i report it? | 12:54 |
amoralej | could be a topic for PTG btw | 12:54 |
marios | i think fine to keep the bug for now and can track it here? | 12:54 |
sean-k-mooney | we can track it as a bug for now but likely we will need a spec to desgin how to enabel horizontal scaling | 12:54 |
marios | but yeah i was going to say ptg or 'other discussion' | 12:54 |
amoralej | yeah | 12:54 |
marios | but for triage i think yes valid, lets mark it triaged | 12:55 |
sean-k-mooney | we coudl add a new tag for this "rfe" | 12:55 |
amoralej | +1 | 12:55 |
marios | wfm | 12:55 |
dviroel | maybe also add [RFE] in the title? | 12:55 |
sean-k-mooney | that is what neturon does they file rfe bugs and if they decied to implement them move it to the approate tracker at that point | 12:55 |
sean-k-mooney | sure [RFE] helps amke it clear | 12:56 |
amoralej | done | 12:56 |
marios | bug fine but will we also have a blueprint sean-k-mooney ? | 12:56 |
marios | i mean do we *need* bp for this type of work? | 12:56 |
marios | i think we do if we have a spec | 12:56 |
sean-k-mooney | if we work on this we will likely end up closign it and moving it to a bluepirnt | 12:56 |
sean-k-mooney | with that said we can come to a decsion about that colelctively | 12:56 |
marios | ack | 12:56 |
sean-k-mooney | based on how we want to work | 12:57 |
sean-k-mooney | thats also a possibel ptg topic | 12:57 |
sean-k-mooney | how to track things like this going forward | 12:57 |
mtembo | so are we moving this to triaged? | 12:58 |
marios | yes | 12:58 |
sean-k-mooney | yep already done and i set impoarntce ot medium | 12:58 |
mtembo | ack, done | 12:58 |
mtembo | link: https://bugs.launchpad.net/watcher/+bug/1857675 | 12:59 |
marios | time rlandy | 12:59 |
rlandy | ack ... mtembo can we continue next week? | 12:59 |
mtembo | yes please | 12:59 |
rlandy | thank you ... pls pick up then | 12:59 |
marios | thanks for bringing those mtembo | 12:59 |
marios | i think we should be in pretty good shape on untriaged | 12:59 |
rlandy | ok - thanks all for attending | 13:00 |
marios | thanks rlandy | 13:00 |
rlandy | we will pick up on PTG topics and bugs next week | 13:00 |
rlandy | #endmeeting | 13:00 |
opendevmeet | Meeting ended Thu Feb 6 13:00:32 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 13:00 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/watcher_irc_weekly_meeting___6_february_2025/2025/watcher_irc_weekly_meeting___6_february_2025.2025-02-06-12.00.html | 13:00 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/watcher_irc_weekly_meeting___6_february_2025/2025/watcher_irc_weekly_meeting___6_february_2025.2025-02-06-12.00.txt | 13:00 |
opendevmeet | Log: https://meetings.opendev.org/meetings/watcher_irc_weekly_meeting___6_february_2025/2025/watcher_irc_weekly_meeting___6_february_2025.2025-02-06-12.00.log.html | 13:00 |
dviroel | tks rlandy++ | 13:00 |
amoralej | thanks rlandy | 13:00 |
sean-k-mooney | freya (dog) wants to go out side so steppign away for a few o/ | 13:00 |
rlandy | happy walking freya | 13:01 |
marios | :) | 13:01 |
sean-k-mooney | she just wanted to go out side and pee so all done | 13:03 |
mtembo | thank you all. enjoy the walk | 13:03 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!