Thursday, 2025-07-31

opendevreviewDavid proposed openstack/watcher master: Disable real metrics on devstack injected data jobs  https://review.opendev.org/c/openstack/watcher/+/95528111:36
opendevreviewMerged openstack/watcher-specs master: Add status_message field to the Audits  https://review.opendev.org/c/openstack/watcher-specs/+/95471811:54
chandankumarwho all are here12:03
rlandyo/12:03
chandankumartime for irc meeting12:03
chandankumar#startmeeting12:03
opendevmeetchandankumar: Error: A meeting name is required, e.g., '#startmeeting Marketing Committee'12:03
dviroelo/ 12:04
chandankumar#startmeeting watcher12:04
opendevmeetMeeting started Thu Jul 31 12:04:30 2025 UTC and is due to finish in 60 minutes.  The chair is chandankumar. Information about MeetBot at http://wiki.debian.org/MeetBot.12:04
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.12:04
opendevmeetThe meeting name has been set to 'watcher'12:04
dviroelthe name is required, only the name is enough12:04
dviroel:) 12:04
chandankumarcourtesy ping: sean-k-mooney chandankumar morenod rlandy12:04
sean-k-mooneyo/12:04
rlandyI'm here :)12:04
chandankumaro/12:05
morenodo/12:05
chandankumarlet's start with today's meeting agenda12:05
chandankumar#link https://etherpad.opendev.org/p/openstack-watcher-irc-meeting#L21 (Meeting agenda)12:05
chandankumarfeel free to add your own topics to the agenda12:05
chandankumarStarting with the first one12:05
chandankumar#topic Eventlet Removal12:06
dviroelo/12:06
dviroelas usual, the etherpad link12:06
dviroel#link https://etherpad.opendev.org/p/watcher-eventlet-removal (watcher evenlet removal etherpad)12:06
chandankumar    #link https://etherpad.opendev.org/p/watcher-eventlet-removal (watcher evenlet removal etherpad)12:06
dviroel:)12:06
dviroelsome minor changes this week12:06
dviroeli removed the depends-on changes from the main dec-engine patch12:07
dviroel#link https://review.opendev.org/c/openstack/watcher/+/952257 (Extend decision engine to support threading mode)12:07
dviroelthe devstack one merged, the other one was the tempest-plugin change, which is not required to merge the main one12:07
dviroelbut there is another DNM change just to test the new continous audit test:12:08
dviroel#link https://review.opendev.org/c/openstack/watcher/+/956199 12:08
opendevreviewDavid proposed openstack/watcher master: Disable real metrics on devstack injected data jobs  https://review.opendev.org/c/openstack/watcher/+/95528112:09
dviroelnot that we discussed about replacing te continuous audit test wit a unit or functional test12:09
sean-k-mooneyyep devstack change merged yesterday so that unblocks that patch12:09
sean-k-mooneywe can have both12:10
dviroelit turns that I couldn't find a way yet of mocking everything needed to simulate the bahavior found with continuous audit thread12:10
sean-k-mooneyack12:10
dviroelI updated instead the tempest-plugin change12:10
dviroel#link https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/95426412:10
dviroelto use only one audit as Alfredo suggested12:11
dviroeland turns that I hit another bug12:11
dviroelone from zone_migration that I filed in the past12:11
* dviroel find the link12:12
dviroel#link https://bugs.launchpad.net/watcher/+bug/209898412:12
dviroelso i started to hit this issue with continuous audit, with a 10s interval12:12
dviroelCI also hit that issue12:12
sean-k-mooneythat the isse with not sharing the same model?12:13
dviroelnot, another one12:13
sean-k-mooneyoh ok12:13
dviroelzone_migration gets instances/volumes from nova/cinder but while they aren't yet in the model12:13
dviroelit raises an exception, since it is not properly handled12:13
sean-k-mooneyoh didnt we fix that before12:13
sean-k-mooneyfor other stragies12:13
sean-k-mooneywe added a polling loop or somethign liek that to make sure the model was synced12:14
dviroelthis is specific for zone_migration implementation, not all strategies use clients to get info about instances/volumes12:14
dviroelthe proposed fix:12:14
dviroel#link https://review.opendev.org/c/openstack/watcher/+/956198/1/watcher/decision_engine/strategy/strategies/zone_migration.py12:14
sean-k-mooneyoh i see12:15
sean-k-mooneyyour fixing this from the watcher size not the test side12:15
dviroelanother patch to add a unit test for this scenario:12:15
dviroel#link https://review.opendev.org/c/openstack/watcher/+/95619712:15
sean-k-mooneyya this feels like it a real watcher bug12:15
sean-k-mooneyhum 12:16
sean-k-mooneyso you are plannign to fix this by filtering to only the ones in the model12:16
dviroelsean-k-mooney: it was doing this already12:16
dviroelbut not treating the exception12:16
sean-k-mooneyah your right12:16
sean-k-mooneyso the ohter way to adress this is to updte teh model with the missing isntance12:17
sean-k-mooneyi guess we can consider that as a latter enhancment and fix the expction handelign first12:17
sean-k-mooneyok i think just handelign the excption is more backportable anyway12:18
dviroelright, we can further discuss that, even if strategies should be getting info directly from the services..12:18
dviroelbut yes, we should backport this one12:19
dviroelin the etherpad there is a link to the error in CI, if someone wants to take a look12:19
dviroelalright, this bug is not eventlet related12:20
dviroelbut one change take to another12:20
dviroeland I ended fixing this bug12:20
dviroelinteresting that the continous audit test was useful for cathing it12:20
sean-k-mooneyya so we wont backport any of the eventlet change bu tthis is a ligitmate bug in its own right12:20
sean-k-mooneyand we likely shoudl backprot that12:21
dviroel+112:21
sean-k-mooneyso thatnk for filing a seperate tracker and spliting it out12:21
chandankumar#link https://bugs.launchpad.net/watcher/+bug/209898412:21
dviroelsure np12:21
chandankumarand fix https://review.opendev.org/c/openstack/watcher/+/95619812:21
sean-k-mooneychandankumar: yep dviroel linked thosse above12:22
chandankumaryup12:22
dviroelalright, if nobody has any questions, that's cover my eventlet part12:22
sean-k-mooneyone12:23
chandankumarthank you dviroel for sharing the update :-)12:23
sean-k-mooneybut slightly unerelated12:23
sean-k-mooneythe content provider job failed to build https://softwarefactory-project.io/zuul/t/rdoproject.org/build/6a8fe1f8aa174887803d784ec9cebdc412:23
chandankumarsean-k-mooney: the fix merged, few hours back12:23
dviroelyeah, it is failing in lot of jobs, but I still didn't start the investigation12:23
sean-k-mooneyhave we seen that on other patches ro do folks knwo why12:23
dviroelchandankumar: oh, good to know, i was about to ask you12:24
sean-k-mooneyoh cool12:24
sean-k-mooneyall good then12:24
chandankumarthanks sean-k-mooney for bringing that one12:24
dviroeli will recheck the patches afterwards then12:24
sean-k-mooney""" The task includes an option with an undefined variable. The error was: {{ ansible_user }}: 'ansible_user' is undefined. 'ansible_user' is undefined. {{ ansible_user }}: 'ansible_user' is undefined. 'ansible_user' is undefined"""12:24
sean-k-mooneyi think perhaps ansible_user was missing :)12:25
sean-k-mooneyansibel can be a bit verbose ocationally12:25
chandankumarhttps://github.com/openstack-k8s-operators/ci-framework/commit/225d9d2f4b38a8d8e7e56bd431bb056462aab8c612:25
dviroelyeah right, it was podman role 12:25
rlandyshowed up late yesterday12:26
rlandychandankumar, fixed it today12:26
dviroelchandankumar++12:26
chandankumarSince no further question, moving now to next topic12:26
dviroelchandankumar: we can move to next topic12:26
chandankumar#topic Croniter swap with appscheduler12:26
chandankumarI was working on above topic and we had a long discussion for the same here https://review.opendev.org/c/openstack/watcher/+/955459/5#message-191158289ed45d4824525724dc38d247c0e8d4bc12:27
chandankumar#link https://review.opendev.org/c/openstack/watcher/+/955459/5#message-191158289ed45d4824525724dc38d247c0e8d4bc12:27
chandankumarI tried to summarize notes here https://etherpad.opendev.org/p/watcher-croniter-swap, But I will drop here also12:27
chandankumarThe review discussed about migrating from croniter to appscheduler crontigger library. 12:28
chandankumarCroniter supports 7 field format (with years and seconds as optional field) while appscheduler supports 5 field format.12:28
chandankumarThe watcher continous audit specs does not provide any info about supporting 5 or 7 field format.12:28
chandankumarSince we are going to swap croniter usage with appscheduler. We saw few issues/concerns.12:28
chandankumarUpgrade Impact: Existing scheduled jobs (continuous audits) using croniter-specific syntax(which becomes invalid format) will fail after the migration.12:28
chandankumarCritical Failure: ongoing continuous audit created after the "bad-formatted" one, will also fail to schedule next runs as the worker responsible of scheduling fails with uncatched exception.12:28
chandankumarThank you sean-k-mooney and Alfredo for actively reviewing and providing feedback on this12:29
chandankumarIn order to mitigate these whole issues, the following plan is suggested:12:29
chandankumar1. We need add watcher status check to detect if any audits are using an incomparable interval format.12:29
chandankumar2. we need to deprecate the use of 6/7 column format and log a warning when its used. we can do that by trying to use aspschduler then fallback to using cronitoer if apscheduler cannot parse it. 12:29
chandankumar3. do the migration automatically on load from the db.12:30
chandankumar4. provide a CLI tool to do an online migration of the data via watcher-manage to convert from 6/7 format to 5 format12:30
chandankumar5. document a manual procedure to do the conversation via the api12:30
chandankumar6. Finally by 2026.2 we will drop the fallback and only use apscheduler.12:30
chandankumarwe also need to add proper exception handling and api validaitons for these formats.12:31
dviroelso we will call the 6/7 format as invalid already? we will just accept its input and do the conversion12:31
chandankumarThe main thing we wanted to discuss about support 5 field or 7 field format12:31
sean-k-mooney the api validation can basiclly just be "parse it with aspchdluer or cronitor"12:31
sean-k-mooneydviroel: so i coudl not find anything to say it was ever offically supproted12:31
dviroelack, we can justify that was never supported12:32
sean-k-mooneythe plan above is the most conservitive option12:32
dviroeland will be an invalid input the future releases12:32
dviroelyeah12:32
chandankumarwe went over code and specs, there is no mention of formats12:32
chandankumarthe test uses 5 field format12:32
dviroelyeah, I saw your comments about specs/releasenotes12:32
sean-k-mooneythe agressive option is say no it was never supprote we only supprot 5 colume format. but even if we did that i think the watcher-status command and posibly a helper command to do the converton woudl be good to have12:33
chandankumaryup12:33
sean-k-mooneygiven someone has taken over maintance of it again12:33
dviroelyes, since there wasn't anything blocking it before12:33
sean-k-mooneyi think we are ok to take the concerviitve one12:33
chandankumarok12:34
dviroelyeah, looks a good approach12:34
chandankumarone more question, since we have a plan in place, Do we want to document the plan in spec or existing bug would be fine to track?12:35
sean-k-mooneywe have one other option by the way, we could vendor a 7 colum parser in watcher. i woudl prefer not to but that is an option if we relaly need that in the future.12:35
sean-k-mooneythat a good question12:36
sean-k-mooneyi think we can use the exisitng bug12:36
sean-k-mooneywe may want to have a bluepirnt or a seocnd bug to track the followup work12:37
dviroelor even create more bugs, like the missing API validation, or for the missing doc12:37
dviroeletc12:37
sean-k-mooneyfor next cycel and the one after. this does nto feel like it need a spec but im not oppsoed. ya the validation exctra can be tracked seperatly12:38
chandankumarmore bugs sounds good.12:38
chandankumarI will add these info the bugs and will update the review based on the plan.12:39
dviroelack chandankumar 12:40
chandankumarThat's it I wanted to discuss on croniter swap.12:40
chandankumarAny questions or concerns on this topic before moving to next one.12:40
dviroeltks chandankumar 12:40
dviroelwe can move, lot to cover yet12:41
chandankumarthank you sean-k-mooney dviroel for the discussion!12:41
chandankumar#topic Open Reviews12:41
chandankumar#link     https://review.opendev.org/c/openstack/watcher/+/955711 (Fix api-ref doc for GET /infra-optim/v1/data_model)12:41
dviroeli have a few to request attention12:41
dviroelnot going to spend too much time on them12:41
chandankumardviroel: go ahead12:42
dviroelthere is a doc update, pls check the related bug12:42
dviroel#link https://bugs.launchpad.net/watcher/+bug/211772612:42
dviroelwe can further discuss in the bug 12:42
dviroelbut the api-ref wasn't reflecting all the fields12:43
dviroeland looking at the code, it seems that they were they since the beginning12:43
dviroelI also added a few unit tests to validate the response: 12:44
dviroel#link https://review.opendev.org/c/openstack/watcher/+/95582012:44
dviroelmaybe not the best way to do that, but I accept reviews or proposals for enhancements12:44
dviroeland finally, a small update in the extend compute model attributes spec12:45
dviroel#link https://review.opendev.org/c/openstack/watcher-specs/+/95592112:45
sean-k-mooneywe have api sample tests12:45
sean-k-mooneyso we may want to enhace those too12:45
dviroelto also incluse the flavor extra_specs in compute model12:45
dviroelsean-k-mooney: right12:46
sean-k-mooneyyou still have that last one marked as WIP in geerit12:47
sean-k-mooneymost project dont use that feature  form my expeirnce but is there a specific reason?12:47
dviroelsean-k-mooney: you are talking about:12:47
dviroel#link https://review.opendev.org/c/openstack/watcher/+/955827 ?12:47
dviroelI found a issue and marked as WIP again, but I can W-1 too12:48
sean-k-mooneyack normally we use -w instaead12:48
dviroelyep12:49
sean-k-mooneypart of the reason i prefer that12:49
sean-k-mooneyother then avoidign change :)12:49
sean-k-mooneyis i likel to leave a commetn why12:49
dviroeldone12:49
sean-k-mooneyi.e so reviewers knwo what the issue you found is if its not obvious12:49
dviroelyeah, i can will add more details about it in a few12:50
dviroeltks12:50
sean-k-mooneyno worreis you mentioend it was an issue with notificatons12:50
sean-k-mooneythat basiclly enough to let ohter know "oh this will get revised again"12:50
dviroel++12:50
chandankumarthere are few more reviews from quangngo in the bottom I am going to cover in this section. If ok?12:51
dviroelchandankumar: sure, pls go ahead, i will get back to extend-compute-model next week12:51
chandankumarReviews related to Add options to disable migration in host maintenance12:51
chandankumar#link https://review.opendev.org/c/openstack/watcher/+/95253812:52
chandankumar#link Add tests for disable migration in host maintenance https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/95421412:52
chandankumarPlease take a look at these reviews.12:52
sean-k-mooneythat was getting pretty close i think. i looked at much fo the code but not the etsting in detail12:52
chandankumarthere are some questions from author on etherpad, let me bring one by one12:52
dviroeli still own reviews there, but it is on my list12:53
chandankumarIs it possible for this feature to appear in 2025.02 release?12:53
dviroel2025.2 yes right12:53
dviroelwe are 4 weeks from the feature freeze12:54
sean-k-mooneyyes this will likely be in 2025.212:54
sean-k-mooneyubutnu are freee to backport this downstream only to thre distro12:54
dviroelbut if the question was 2025.1, that's a no12:54
sean-k-mooneybut we wont be backproting this upstream12:54
chandankumarthere was one follow up questions also Question for Ubuntu SRU: backportability this feature to any current stable branches? (A no expected, Ubuntu SRU decision just requires upstream confirmation)12:54
sean-k-mooneywe also are unlikely to backprot this to our donstream12:54
sean-k-mooneyfeature are not allowed to be backpaorted understable policy12:55
sean-k-mooneyso this was never a backport candiate12:55
chandankumarquangngo: I hope it answers the your queries.12:56
sean-k-mooneyhttps://docs.openstack.org/project-team-guide/stable-branches.html#appropriate-fixes12:56
dviroel++12:56
quangngoyes, we expect that, ack!12:56
dviroelquangngo: tks for proposing the patches, I will take a look on those12:56
chandankumarSince we have 4 mins left. I am going to move over to next topic12:57
dviroelsure12:57
sean-k-mooneyquangngo: in this particalar case canonical likely coudl backprot that enhancement downstream safely12:57
sean-k-mooneybut its more risk then we woudl normally take upstream12:57
chandankumar#topic monasca retirement and sdk adoption12:58
sean-k-mooneyya so i added that12:58
sean-k-mooneytl;dr12:58
sean-k-mooneythe tc has resolved to continue with the retirement process for monsasca12:58
sean-k-mooneyson in the next few weeks the git repos will be retired and there will be no future releases of monasca12:59
dviroelrip monasca12:59
sean-k-mooney5 months ago we deprecated support12:59
sean-k-mooneyand we had planned to remove it in 2026.212:59
sean-k-mooneyto mitigate the impact of the retirement 12:59
sean-k-mooneyi plan to work on some targeted patches to make it an optional depency for this cycle12:59
sean-k-mooneywe can dicusss for next cycle if we want to acclerate the removal13:00
dviroel+113:00
sean-k-mooneyor not13:00
sean-k-mooneywe have no tempest test or docs so iw as going to propsoe droping it at the start of 2026.113:00
dviroelmake the conditional import would be great13:00
sean-k-mooneyso the follow up to that is we shoudl do the same with all the datasocue and openstack project clients13:01
sean-k-mooneyand ideally replace the proejct client with the openstack sdk13:01
dviroel+113:01
chandankumar+113:01
sean-k-mooneythat is work for next cycle13:01
chandankumarthank you sean-k-mooney for bring that up.13:01
sean-k-mooneyi will likely draw up a propsoal for that prior to the ptg and either create a spec or blueprint13:01
sean-k-mooneythat basicly all i had. 13:02
chandankumarSince we are running out of time, I will go with last topic13:02
dviroelsean-k-mooney: thanks for that 13:02
chandankumar#topic volunteer to chair for next week meeting13:02
chandankumarAnyone would like to take it?13:03
dviroeli can chair, since I will be out on 14th13:03
chandankumarthanks dviroel 13:03
chandankumartime to wrap up13:03
dviroel:) 13:03
chandankumarthank you all for attending13:03
chandankumar#endmeeting13:03
opendevmeetMeeting ended Thu Jul 31 13:03:54 2025 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)13:03
opendevmeetMinutes:        https://meetings.opendev.org/meetings/watcher/2025/watcher.2025-07-31-12.04.html13:03
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/watcher/2025/watcher.2025-07-31-12.04.txt13:03
opendevmeetLog:            https://meetings.opendev.org/meetings/watcher/2025/watcher.2025-07-31-12.04.log.html13:03
dviroelthanks chandankumar++13:03
morenodchandankumar++13:04
sean-k-mooneydviroel: how do you feel about me tracking making monasca optional as an rfe bug?13:49
dviroelsean-k-mooney: looks fine, there isn't too much details to track as bp or spec I think13:50
sean-k-mooneyack ill file one then to track this and give a breif summary in the descripition13:51
sean-k-mooneythe client -> sdk change shoudl likely be a spec or something more formal13:52
sean-k-mooneybut we can dicuss that in the future13:52
dviroelagree, the sdk change will be a bigger effort13:53
opendevreviewDavid proposed openstack/watcher-tempest-plugin master: Add custom flavor and dynamic threshold to workload_balance tests  https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/95385314:05

Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!