Thursday, 2025-02-06

opendevreviewMerged openstack/watcher master: Update master for stable/2024.2  https://review.opendev.org/c/openstack/watcher/+/92941408:07
opendevreviewMerged openstack/watcher-dashboard stable/2024.2: Update TOX_CONSTRAINTS_FILE for stable/2024.2  https://review.opendev.org/c/openstack/watcher-dashboard/+/92941808:07
rlandyhi all ... reminder Watcher meeting here in 30 mins11:27
rlandyplease add any topics to  https://etherpad.opendev.org/p/openstack-watcher-irc-meeting11:28
rlandy#startmeeting Watcher IRC Weekly Meeting - 6 February 202512:00
opendevmeetMeeting 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
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.12:00
opendevmeetThe meeting name has been set to 'watcher_irc_weekly_meeting___6_february_2025'12:00
rlandyo/ hi folks12:01
marioshello o/12:01
rlandylet's see how's around12:01
rlandywho is around12:01
amoralejo/12:02
mtemboo/12:02
dviroelo/12:02
sean-k-mooneyo/12:03
rlandyok ... let's begin with the listed topics12:04
rlandy#topic: (rlandy) Any more action needed on the DPL model discussed last week?12:04
rlandyAre we all set there?12:04
mariosi *think* it merged? checking12:04
mariosyeah12:04
marioswe have https://review.opendev.org/c/openstack/governance/+/940167 which establishes we use dpl for coming cycle12:05
sean-k-mooneythe first patch did yes12:05
mariosthe only outstanding (and it may stay that way) is the "add watcher project update liaison" https://review.opendev.org/c/openstack/governance/+/94016812:05
sean-k-mooneythe second patch to re-add chenker is pending12:05
rlandyok - do we need any more actions there?12:05
sean-k-mooneythey -1'd it but we are unsure if that means they dont want ot be liason or do but set the wrong flag12:06
sean-k-mooneyno12:06
mariosnot that i'm aware of 12:06
sean-k-mooneywe are just waiting for them to respond to gmann's comment12:06
rlandyok thanks sean-k-mooney and marios - moving on ...12:06
rlandy#topic: (rlandy) Patches still requiring review or merge12:06
rlandyanyone has reviews to add today?12:07
amoralejnot me, thanks marios for your reviews!12:07
mariosbeen trying to merge some of the older patches in the last few days with the new voting rights12:07
mariosthanks amoralej 12:07
mariosespecially some of the obvious ones like autogenerated/bot patches that have been sitting for weeks12:08
rlandychandan is out today but may have reviews on the testing/tempest plugin side12:08
rlandy#action: rlandy to try dig those out and post here12:08
rlandyafter meeting12:08
mariosi have an item later in the agenda12:09
mariosbut i will just add it here 12:09
marios#info prometheus data source vm metrics merged https://review.opendev.org/c/openstack/watcher/+/93889312:09
rlandywould you like to go now - since we are on the reviews topic?12:09
mariosthanks amoralej 12:09
mariosdone rlandy ;D12:09
mariosthanks12:09
rlandy:)12:09
sean-k-mooneyso while we have not fuly finished hte blueprint/spec12:10
rlandyany others?12:10
sean-k-mooneythe mvp code is now merged.12:10
mariosare you referring to the keystone integration sean-k-mooney ? or sthing else12:10
sean-k-mooneyso we just need to focus on the testing to complete the remainging work12:10
mariosah i see, you mean ci side12:10
amoralej+112:10
sean-k-mooneymarios: the tempest job12:11
mariosack12:11
sean-k-mooneyso i think we are making good progress on compelting that this cycle12:11
sean-k-mooneyso ya we can move on12:11
rlandyok - moving on ...12:12
rlandy#topic: (rlandy) reminder: PTG items list is on notes doc: https://etherpad.opendev.org/p/openstack-watcher-irc-meeting12:12
rlandyjust a reminder that we added a list there for anything that comes to mind to raise at PTG12:12
rlandymarios (I think) responded to the initial form - thank you12:13
mariosyeah last week or whenever and we mentioned it in last irc meet 12:13
mariosi'll update if/when i hear somethign further there for choosing dates/times etc12:13
rlandy+112:14
rlandyany other questions on PTG readiness?12:15
amoralejyou want us to start adding topics to discuss in the etherpad or only about organization so far?12:16
rlandyboth - we can triage what we actually bring12:17
rlandybut good for us to start gathering ideas12:17
amoralejack12:18
rlandythen we will know how many sessions to request12:18
rlandywe can review them closer to the time12:19
rlandyok ... looks like we are done with the PTG discussion12:20
amoralejwe could start by creating several sections for topicts to discuss, API, service, strategies, dashboard or something like that12:20
rlandyack - anything we want to do in that cycle 12:21
sean-k-mooneythat an option. we may want to that after we have the list of topic12:21
dviroel+112:21
amoralejmake sense12:22
rlandyanything more here?12:23
rlandyok  - 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 meeting12:26
mtembook12:26
rlandythank you mtembo for volunteering :)12:26
rlandyFed 13th is yours to chair12:26
mtemboack 12:26
rlandymoving on ...12:27
rlandy#topic: (marios): reminder coming feature freeze Feb 24 - Feb 28 https://releases.openstack.org/epoxy/schedule.html12:27
mariosjust a reminder nothing much to add here ... epoxy feature freeze coming in 3 weeks ish12:27
mariosapart from the prometheus data source work i am not aware of something else going on 12:27
marioswe have some outstanding bits for the ci side but i think we should be OK 12:28
marioswe can move on rlandy unless there is anything else here thank you12:28
rlandyand dviroel piece in nova?12:28
mariosyeah but that is not for epoxy i think?12:28
dviroelit is for epoxy in nova12:28
mariosah 12:28
mariosmy bad 12:28
dviroeli've  got some reviews in the main patch, it was also updated to solve some issues12:29
dviroelwaiting for more feedback there 12:29
dviroel#link https://review.opendev.org/q/topic:%22bp/show-scheduler-hints-in-server-details%2212:30
dviroelin case someone wants to take a look :) 12:30
rlandysean-k-mooney: are you still planning more for the apscheduler stuff? iiuc, we're leaving it as is for now12:30
rlandyeventlet12: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 ptg12:31
rlandyok - thank you12:31
sean-k-mooneyin the next 1-3 release we need to work on removing watchers use of eventlet so that might be how we resolve the remianing issues12:32
sean-k-mooneythe other possiblity is moving to usign the oslo.db engine fascade context manager 12:33
sean-k-mooneyto better handlel the session/conneciton lifetime12:33
sean-k-mooneyboth are non trivial so we shoudl dicuss that before investing mroe time12:33
rlandy+1 12:34
sean-k-mooneyi consider both to be "tech debt" that shoudl eventually be adressed but im not sure when12:34
sean-k-mooneybtu we can move on for now12:34
rlandyok - anything more on epoxy feature freeze?12:34
sean-k-mooneyi gues i have two quick points12:34
rlandypls go ahead12: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 rc112:35
mariosthanks for doing that sean-k-mooney 12:35
marioslet me know if i can help 12:35
sean-k-mooneyso once i have those created ill ask for review. this is part fo the standard release duties.12:35
mariosi did some releases but a while ago12:36
dviroeltks sean-k-mooney++12:36
sean-k-mooneyyep im sure you can espically with wording12:36
sean-k-mooneythe other item is we shoudl consdier if we want to deprecate any of the goals/stragies12:36
sean-k-mooneywe dicussed perhaps deprecating the noisy neighbour one12:37
sean-k-mooneysince the metric it depned on is not provided by any of the backend any more12:37
sean-k-mooneywhile we shoudl not remove it this cycle. if we intend to do that in the next 2 cycles we need to deprecated it this cycle12:37
amoralejyes, i think it's a good candidate to be deprecated12:38
mariossean-k-mooney: if i recall requirement is 1 cycle as deprecated before removal right?12:38
mariosor is it 2?12:38
sean-k-mooneythe actual requirement is the deprecation must ship in a slurp release before it can be removed12:38
sean-k-mooneyso in a .1 release like epoxy12:38
mariosah12:38
mariosso if we don't, its next year ten12:39
mariosthen12:39
sean-k-mooneyyep12:39
sean-k-mooneyso we can reflect on that and email the list for extra feedback12:39
mariossean-k-mooney++12:39
rlandyack - thank you for raising that12:40
sean-k-mooneyif we are going to do this i would like to do it before i create teh release prelude/cycle highlights so we can refence it12:40
sean-k-mooneythat was all i had for release topics.12:40
rlandyok - great12:40
rlandythanks,  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
rlandymarios: ^^12:41
mariosthanks rlandy ... so just an announcement that there was no negative feedback to sean-k-mooney proposal and we have new cores12:41
mariosso we can clear the backlog and avoid patches sitting for weeks12:41
rlandygood news - thank you12:41
sean-k-mooneyyep, i forgot to do it before goint on pto firday but i implemtned it at the weekend when i remmeberd12:42
sean-k-mooneyi emailed hte list on tuesday when i got back12:42
rlandysean-k-mooney++12:42
mariosthanks sean-k-mooney and thanks rlandy ... if there is no other comment we can move on 12:42
rlandycongrats to the new cores12:43
dviroelwe already used our new powers to merge some pending changes :) 12:43
marios+112:43
marios+2 even ;)12:43
sean-k-mooneyi saw :)12:43
rlandyok ... moving on12:43
rlandynext is the bug triage12:43
rlandy#topic: (mtembo) Bug list for review12:44
mtembowe spoke about this one last weeek: thats where we ended https://bugs.launchpad.net/watcher-tempest-plugin/+bug/209541612:44
rlandy#link: https://bugs.launchpad.net/watcher-tempest-plugin/+bug/209541612:45
rlandydviroel, mtembo looks like you worked on this ...12:45
rlandythe bug is still in New12:46
mtemboit is fixed from the numerous tests ran thereafter 12:46
dviroelno, i just added a comment about our last discussion on irc12:46
rlandyok - so let's close it out?12:46
mtemboWe should close it 12:46
rlandyok wait ... sorry dviroel 12:46
dviroelrlandy:  i mean that I didn't work on this bug12:47
mariosone 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 it12:47
rlandygood point12:47
mariosi 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
mariosso was there a fix for this or how did the tests pass12:48
rlandymtembo: could you update the bug with latest results and close out? 12:49
sean-k-mooneyso ya that not clear to me what is the fix?12:49
mtembonot directly as far as i understand 12:49
sean-k-mooneywe shoudl not move things to fix commeited if we do not know how it was fix. we shoudl move it to invlaid or incomplete12:49
mtembomoved it fixed commited. thanks smooney12:50
mariosno mtembo not fix committed 12:50
mtemboI did include the link the latest tests ran12:50
mtembo#link: https://review.opendev.org/c/openstack/watcher/+/94029312:51
sean-k-mooneyim saying fix commited is wrong12:51
mariosfix 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 invalid12:51
rlandybut reading back ... we still don't know what fixed the bug12:51
rlandyack12:51
mtembonoted. thanks12:51
sean-k-mooneyok so my orgianl theory was it was fixed by the orgianl asp schduler fix or dougs fix12:51
mtembomoved to invalid12:51
sean-k-mooneyif it nolonger repoduces we can move it to invlaid or incompele12:52
mtembodone 12:52
mtembomoving on: 12:52
mtembolink: https://bugs.launchpad.net/watcher/+bug/209699112:52
mariosi think this is valid but its more like a feature request/enhancement not a bug per se...?12:53
amoralejthat's mine, i reported to track something we discover 12:53
sean-k-mooneyso this is valid alhtough it really a feature not a bug IMO12:53
marios(unironically sean-k-mooney ;) )12:54
amoralejso, how should i report it?12:54
amoralejcould be a topic for PTG btw12:54
mariosi think fine to keep the bug for now and can track it here?12:54
sean-k-mooneywe can track it as a bug for now but likely we will need a spec to desgin how to enabel horizontal scaling12:54
mariosbut yeah i was going to say ptg or 'other discussion' 12:54
amoralejyeah12:54
mariosbut for triage i think yes valid, lets mark it triaged12:55
sean-k-mooneywe coudl add a new tag for this "rfe"12:55
amoralej+112:55
marioswfm12:55
dviroelmaybe also add [RFE] in the title?12:55
sean-k-mooneythat is what neturon does they file rfe bugs and if they decied to implement them move it to the approate tracker at that point12:55
sean-k-mooneysure [RFE] helps amke it clear12:56
amoralejdone12:56
mariosbug fine but will we also have a blueprint sean-k-mooney ? 12:56
mariosi mean do we *need* bp for this type of work?12:56
mariosi think we do if we have a spec12:56
sean-k-mooneyif we work on this we will likely end up closign it and moving it to a bluepirnt12:56
sean-k-mooneywith that said we can come to a decsion about that colelctively 12:56
mariosack12:56
sean-k-mooneybased on how we want to work12:57
sean-k-mooneythats also a possibel ptg topic12:57
sean-k-mooneyhow to track things like this going forward12:57
mtemboso are we moving this to triaged?12:58
mariosyes12:58
sean-k-mooneyyep already done and i set impoarntce ot medium12:58
mtemboack, done 12:58
mtembolink: https://bugs.launchpad.net/watcher/+bug/185767512:59
mariostime rlandy 12:59
rlandyack  ... mtembo can we continue next week?12:59
mtemboyes please12:59
rlandythank you ... pls pick up then12:59
mariosthanks for bringing those mtembo 12:59
mariosi think we should be in pretty good shape on untriaged12:59
rlandyok - thanks all for attending13:00
mariosthanks rlandy13:00
rlandywe will pick up on PTG topics and bugs next week13:00
rlandy#endmeeting13:00
opendevmeetMeeting ended Thu Feb  6 13:00:32 2025 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)13:00
opendevmeetMinutes:        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.html13:00
opendevmeetMinutes (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.txt13:00
opendevmeetLog:            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.html13:00
dviroeltks rlandy++13:00
amoralejthanks rlandy13:00
sean-k-mooneyfreya (dog) wants to go out side so steppign away for a few o/13:00
rlandyhappy walking freya13:01
marios:)13:01
sean-k-mooneyshe just wanted to go out side and pee so all done13:03
mtembothank 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/!