Thursday, 2025-03-27

opendevreviewAlfredo Moralejo proposed openstack/watcher-tempest-plugin master: Add real-data scenario tests  https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/94561307:55
*** sfinucan is now known as stephenfin10:23
amoralejsean-k-mooney, wrt https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/945613/comment/884cc817_cd953567/ could i use directly tempest.common.compute.create_test_server from the watcher-tempest-plugin or you mean implement something like that in our base.py ?11:39
amoralejstuff under common is expected to be usable by any external plugin?11:40
amoralejoh, sorry, i just realized create_server is a wrapper on top of create_test_server, so i guess we can pass everything we need from there11:45
sean-k-mooney you can use it directly but yes its wrapper all teh way down11:45
amoralejthx11:45
sean-k-mooneywaht im really suggesting is you do it like this https://github.com/openstack/tempest/blob/80c0477f78c71a2bd2e1a324c41cd2f50329b200/tempest/api/volume/test_volumes_backup.py#L124-L12811:46
sean-k-mooneyinstead of creating a new keypair just create teh validation resouces and pass that in11:46
sean-k-mooneyif you need to dynamicaly create a keypair for some reason that ok11:47
sean-k-mooneybut i was hopign to remvoe https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/945613/3/watcher_tempest_plugin/tests/scenario/test_execute_strategies_real.py#6711:47
mtembo#startmeeting Watcher IRC Weekly Meeting - 27 March 202512:00
opendevmeetMeeting started Thu Mar 27 12:00:07 2025 UTC and is due to finish in 60 minutes.  The chair is mtembo. 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___27_march_2025'12:00
mtemboo/ hello 12:02
mtembowho's present today ?12:02
rlandyo/12:03
jgilabero/12:03
chandankumaro/12:04
amoralejo/12:05
mtembofolks it looks like there is not much to discuss today.12:07
mtembowould anyone like to raise a topic ?12:07
mtemboalright, we shall move along ... 12:10
mtembo #topic     Bug Triage12:10
mtembo#link https://bugs.launchpad.net/watcher/+bug/210422012:10
rlandywe seems to have a few recent bugs on strategies12:13
amoralejthat's architecture issue, 12:14
amoralejthe redesign choises the reported mentioned is our ptg topic (Watcher model collector improvement ideas)12:14
amoralejin the operator we are setting default to 15 mins instead of 1h, but the reported behavior can still happen12:15
amoralejoptimal solution would be to make collection event based, but we need to make sure we don't overload nova api12:16
amoralejI can summarize this in a bug comment12:18
rlandyamoralej++12:20
mtemboamoralej++12:22
mtembo#topic: Volunteers for chair for 3rd April meeting12:22
mtemboI will take the next meeting: mtembo12:24
rlandythanks mtembo  - I can take the after PTG12:25
mtemboThank you all.12:26
mtembo#endmeeting12:26
opendevmeetMeeting ended Thu Mar 27 12:26:56 2025 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)12:26
opendevmeetMinutes:        https://meetings.opendev.org/meetings/watcher_irc_weekly_meeting___27_march_2025/2025/watcher_irc_weekly_meeting___27_march_2025.2025-03-27-12.00.html12:26
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/watcher_irc_weekly_meeting___27_march_2025/2025/watcher_irc_weekly_meeting___27_march_2025.2025-03-27-12.00.txt12:26
opendevmeetLog:            https://meetings.opendev.org/meetings/watcher_irc_weekly_meeting___27_march_2025/2025/watcher_irc_weekly_meeting___27_march_2025.2025-03-27-12.00.log.html12:26
amoralejsean-k-mooney, thanks, i will follow that approach12:28
opendevreviewAlfredo Moralejo proposed openstack/watcher-tempest-plugin master: Add real-data scenario tests  https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/94561313:18
opendevreviewAlfredo Moralejo proposed openstack/watcher-tempest-plugin master: Add real-data scenario tests  https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/94561313:20
chandankumarsean-k-mooney: amoralej Hello, please add it to your review queue https://review.opendev.org/q/topic:%22bug/2100741%22 when free, thank you! It will finish the functional tests move.13:36
opendevreviewchandan kumar proposed openstack/watcher master: [DNM] RDO Zuul check job  https://review.opendev.org/c/openstack/watcher/+/94507913:53
opendevreviewchandan kumar proposed openstack/watcher master: [DNM] RDO Zuul check job  https://review.opendev.org/c/openstack/watcher/+/94507913:54
opendevreviewchandan kumar proposed openstack/watcher master: [DNM] RDO Zuul check job  https://review.opendev.org/c/openstack/watcher/+/94507914:01
opendevreviewchandan kumar proposed openstack/watcher master: [DNM] RDO Zuul check job  https://review.opendev.org/c/openstack/watcher/+/94507914:24
amoralejsean-k-mooney++ thanks to your comments, my new test is much cleaner now14:30
*** haleyb_ is now known as haleyb15:06
sean-k-mooneyamoralej: i agree 5 min time.sleep is a problem however15:46
sean-k-mooneyunless we skip that test by default in upstream jobs15:46
opendevreviewFrancisco Seruca Salgado proposed openstack/watcher master: [DNM] RDO Zuul check job  https://review.opendev.org/c/openstack/watcher/+/94507915:52
amoralejsean-k-mooney, we are skipping it by default actually, https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/945613/5/watcher_tempest_plugin/tests/scenario/test_execute_strategies_real.py#3515:53
sean-k-mooneythat also a problem15:54
amoralejin our upstream jobs se are setting optimize.datasource to gnocchi or prometheus. Note that, that parameter is only used to set the backend where we are injecting metrics15:54
sean-k-mooneywe need to be abel to run this with promethous enabled15:54
amoralejyes, the name of the parameter is missleading15:55
amoralejthat's actually unrelated to the prometheus backend15:55
sean-k-mooneyno i mean we should not have a seprate job to run this15:55
sean-k-mooneywe shoudl be able to run it in any job15:55
amoralejmy approach was to be mutually exclusive with inject-metrics jobs, is that fine?15:56
sean-k-mooneyno i dont think thats ok15:56
sean-k-mooneywell15:56
sean-k-mooneywe are running this in perodic right15:57
amoralejyes15:57
amoralejand experimenatl15:57
sean-k-mooneyi still think that too expensive honestly 15:57
opendevreviewFrancisco Seruca Salgado proposed openstack/watcher master: [DNM] RDO Zuul check job  https://review.opendev.org/c/openstack/watcher/+/94507915:57
amoralejsee https://review.opendev.org/c/openstack/watcher/+/94562715:57
sean-k-mooneymaybe its ok but a full multi node job for one test is a bit much15:57
sean-k-mooneyi know we can run other tests that done need metrics15:58
sean-k-mooneybut even still that expensive15:58
amoralejok, i see your point15:58
sean-k-mooneyi would invert that if15:58
sean-k-mooneyhave it skip if there is no data souce15:58
sean-k-mooney CONF.optimize.datasource is ment to be the data souce that is deployed15:59
sean-k-mooneyits not intened to contole if we inject or not15:59
amoralejactually, i know we have some other tests which do not require inject-data, we may include them in the same job15:59
sean-k-mooneymost of the api tests 15:59
sean-k-mooneyand one or two of the stagies tests doen need any metrics15:59
amoralejyes, all the api ones + some scenario16:00
sean-k-mooneyi need to rebase this too https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/94017116:00
amoraleji.e. the consolidation one does work perfectly fine with no fake data iirc16:00
sean-k-mooneyso  https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/940171 removes the injection form test that done need it16:00
sean-k-mooneybut i started that ebfore we changed how they run so its in merge conflict16:01
amoralejyes, that's good16:01
amoralejand i think we should implement a mechanism to skip the tests which require injection when injection is not configured16:02
amoralejand the oposite, tests we want to skip when injection is setup16:02
amoralejwe should do a new parameter? wdyt ?16:03
amoralejI'm afraid of mixing real and injected data jobs16:03
amoraleji.e. for fake data jobs in prometheus we disable the cpu metrics from the node_exporter16:03
amoralejmixing real and injected will lead us to issues imo16:04
sean-k-mooneyi dont think we shoudl test them seperaly16:04
sean-k-mooneywhat type of issues do you see16:05
amoraleji.e. see this https://github.com/openstack/watcher/commit/539be503f066c50f83d86c4d2476da5231c823cf16:06
sean-k-mooney right so we didnt need to do that16:06
sean-k-mooneyit was not causing any actual instablity16:06
sean-k-mooneybut any one test would not have both injection and not16:06
sean-k-mooneycurrently the suite is set up to only run 1 test at a time16:07
amoralejwon't you think it will be much harder to be deterministic in the metrics we expect if we mix two sets of metrics ?16:07
sean-k-mooneyif we are using real load it wont be determinsitc16:07
sean-k-mooneywhich is why i didnt want to add tests that do that at all16:08
amoralejone saying, instance_cpu_usage is ~ 0 (real one) and fake one is 70% or whatever16:08
amoralejwell, it's less deterministic that fake data, that's right but it can be "deterministic enough" for some tests16:08
sean-k-mooneyi guess to move forward we can do the following16:09
sean-k-mooneylets add a annotation to tests that use real data.16:09
sean-k-mooneyand instead of usign config optiones to decied if they run16:09
sean-k-mooneywe can use teh regex16:09
amoralejyou mean skip annotation ?16:10
sean-k-mooneythen we can split things up in the job config rather then the plugin code16:10
sean-k-mooneytempest has teh idea of annoating tests https://github.com/openstack/tempest/blob/80c0477f78c71a2bd2e1a324c41cd2f50329b200/tempest/scenario/test_network_v6.py#L24016:11
amoralejit implies that an anmotated tests is disabled by default ?16:11
amoralejhttps://docs.openstack.org/tempest/latest/HACKING.html#test-attributes16:13
sean-k-mooneyexluded by the test regex16:13
amoraleji see16:13
amoralejwe'll need to add regex16:13
amoralejlgtm16:13
sean-k-mooneywell so im trying to avoid having multiple way to deceide if a test shoudl run16:14
sean-k-mooneyconfig option are fine but we shoudl generally use them for providing info that cna change between deployments or chagne the behavior16:14
sean-k-mooneyrather then choosing if a test runs or not16:15
sean-k-mooneyim a litel uncomfortable with test skiping when we change the datasouce we set in the config16:15
sean-k-mooneyits not terrible just harder to debug16:16
amoralejok, that works for me16:16
sean-k-mooneyso im wondering if we shoudl do somethign like this @decorators.attr(type=['slow','real_load'])16:17
amoralejI'm still thinking in one case, where we want to run real + tests which does not require injection16:17
amoralejmanaging that via regex will be harder16:17
sean-k-mooneyok for now lets just leave it as you have it now16:18
amoralejthat's where i think it'd may be good to a have a parameter to enable/disable injection, maybe a different one16:18
sean-k-mooneywe can revist this in the future16:18
amoralejbut i fully agree using CONF.optimize.datasource is missleading16:19
sean-k-mooneybut the implciation fo this is we wont be able to run thi in check in github or downstream in the componet pipeline or downstream in gitlab16:19
sean-k-mooneywe will need to put it into a sepreate perodic downstream too16:19
sean-k-mooneythat really what i wanted to avoid16:19
amoralejbut if you check the actual usage of the parameter you will see the only real use is for data injection16:20
amoralejbut yes, it's probably inadequate 16:20
sean-k-mooneybut that not what it for16:20
sean-k-mooneyits to tell you what datasouce is deploy16:20
sean-k-mooneyso that you can make any decision based on that16:20
sean-k-mooneyinjection is one thing that needs that16:21
sean-k-mooneywe may have other in the future16:21
amoralejso we should probably rework that, remove the empty option and add a new one16:21
sean-k-mooneywell i dont want ti to contol inejection16:21
amoralejok, maybe we will be able to make it work with real + injected in the same job16:22
sean-k-mooneyfor now let get it wrokign in a seperate job like you started with but we can revisit it16:22
amoralejok16:22
sean-k-mooneyi dont want ot change the requirement on you mid way true16:23
sean-k-mooneywe can get it working in a useabel way and hten evolve it later16:23
amoralejyes, i think there are more improvements to do, like https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/94017116:24
amoralejalso, https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/94532516:24
amoralejcurrently, test_execute_workload_stabilization_strategy is always creating empty action plan16:25
amoralejbut i need to refine that16:25
amoralejif you are too busy i probably can take over https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/940171 16:26
amoralejbut yeah, step by step16:26
amoralejwrt runtime for the test, 30-60secs is probably enough for upstream jobs where we  have some flexibility to setup polling interval in both prometheus and ceilometer, but in operators env, that's hardcoded to 120secs https://github.com/openstack-k8s-operators/telemetry-operator/blob/7cc05b83658fab71f30fff180b23f37983db9809/templates/ceilometercompute/config/polling.yaml#L416:30
amoralejthere will be a way to modify it soon, but not implemented yet16:30
amoralejthat's where the 300 comes from, we need at least two points16:30
amoralejI may do that time a parameter in [optimize] section16:31
sean-k-mooneyamoralej: i put it on the back burner becuase we had a plan to actully get injection workign for promethus16:42
sean-k-mooneyif you want to take it over feel free there was one test that failed because it was in a clas sthat had teest that need injection and others that dont16:42
sean-k-mooneyso i need to go back and split that into a seperat class ro rethink how i group them16:43
sean-k-mooneycurrently i was disabling it per class but you could do it per test16:43
amoralejyeah, that's what i was thinking16:43
amoralejdisabling by test16:44
opendevreviewAlfredo Moralejo proposed openstack/watcher-tempest-plugin master: Add real-data scenario tests  https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/94561316:51
opendevreviewAlfredo Moralejo proposed openstack/watcher master: Add real-data based tests to experimental pipeline  https://review.opendev.org/c/openstack/watcher/+/94562716:53
opendevreviewAlfredo Moralejo proposed openstack/watcher-tempest-plugin master: Add real-data scenario tests  https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/94561318:45

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