Thursday, 2025-05-29

opendevreviewJoan Gilabert proposed openstack/watcher master: Handle missing dst_pool parameter in zone_migration  https://review.opendev.org/c/openstack/watcher/+/95014907:19
opendevreviewDavid proposed openstack/watcher-tempest-plugin master: Update parameters on test_execute_workload_stabilization_strategy  https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/95096907:34
opendevreviewDavid proposed openstack/watcher-tempest-plugin master: Update tests for host maintenance strategy  https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/95085107:44
opendevreviewDavid proposed openstack/watcher-tempest-plugin master: Update tests for host maintenance strategy.  https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/95085107:54
opendevreviewAlfredo Moralejo proposed openstack/watcher-tempest-plugin master: Enforce compute host in create_one_instance_per_host  https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/95038907:56
opendevreviewJoan Gilabert proposed openstack/watcher master: Handle missing dst_pool parameter in zone_migration  https://review.opendev.org/c/openstack/watcher/+/95014907:57
opendevreviewDavid proposed openstack/watcher-tempest-plugin master: Update tests for host maintenance strategy.  https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/95085108:07
opendevreviewTakashi Kajinami proposed openstack/watcher-tempest-plugin master: Remove stable/2023.2 job  https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/95124108:30
opendevreviewchandan kumar proposed openstack/watcher master: [WIP] host maintenance restore backup node  https://review.opendev.org/c/openstack/watcher/+/95124509:22
opendevreviewAlfredo Moralejo proposed openstack/watcher-tempest-plugin master: Enforce compute host in create_one_instance_per_host  https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/95038909:30
opendevreviewDavid proposed openstack/watcher-tempest-plugin master: Change timeframe of fake metrics uploaded to gnocchi.  https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/95096909:31
opendevreviewDavid proposed openstack/watcher-tempest-plugin master: Change timeframe of fake metrics uploaded to gnocchi.  https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/95096909:32
opendevreviewDavid proposed openstack/watcher-tempest-plugin master: Add tests for workload_balance with injected data  https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/94972209:33
opendevreviewchandan kumar proposed openstack/watcher master: [WIP] host maintenance restore backup node  https://review.opendev.org/c/openstack/watcher/+/95124511:12
rlandyreminder  - watcher IRC meeting here in ~10 mins11:51
rlandypls add topics to https://etherpad.opendev.org/p/openstack-watcher-irc-meeting11:51
rlandy#startmeeting Watcher Meeting - 29 may 202512:01
opendevmeetMeeting started Thu May 29 12:01:26 2025 UTC and is due to finish in 60 minutes.  The chair is rlandy. Information about MeetBot at http://wiki.debian.org/MeetBot.12:01
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.12:01
opendevmeetThe meeting name has been set to 'watcher_meeting___29_may_2025'12:01
rlandyhi all .. who's around?12:01
dviroelo/12:01
chandankumaro/12:01
jgilabero/12:02
amoralej_o/12:02
rlandyCourtesy ping: sean-k-mooney 12:02
sean-k-mooneyo/12:02
mtemboo/12:02
rlandyok - let's get started12:02
sean-k-mooneythat working already :)12:02
rlandyTopics for today are on: 12:03
rlandy#link https://etherpad.opendev.org/p/openstack-watcher-irc-meeting#L5812:03
rlandyI start :)12:03
rlandy#topic: (rlandy) Strategies marked [POC] "This is a proof of concept that is not meant to be used in production"12:03
rlandyTwo strategies that have active work on them are still marked POC in the doc:12:04
rlandy#link https://docs.openstack.org/watcher/latest/strategies/host_maintenance.html12:04
rlandy#link: https://docs.openstack.org/watcher/latest/strategies/workload_balance.html 12:04
rlandyexamples of strategies that are NOT POC are included in the notes ...12:04
rlandy#link https://docs.openstack.org/watcher/latest/strategies/zone_migration.html12:05
rlandynot POC12:05
rlandyso ...12:05
rlandydo we want to keep that label POC  in the documentation if we have fully updated/tested/doc'ed a strategy?12:05
rlandythe doc for both host maintenance and balance is being worked on for updates 12:06
rlandyand there are open bugs12:06
sean-k-mooneythats a good question12:06
sean-k-mooneyso i want to move away form t he POC termonlogy in general and adopt experimental12:07
sean-k-mooneyso i woudl prefer to update the docs to do that and again we need to loop back to the idea of adding a supprot level matix in the docs too12:07
chandankumar+1 to experimental 12:07
sean-k-mooneyi think we can remvoe the code level comment and perhaps the doc comment 12:07
sean-k-mooneyim not sure if we shoudl graduate it to fully supprot or not yet however12:08
chandankumarHow a strategy graduate from experimental/poc to supported in upstream?12:08
rlandywhat do we want to label zone migration as?12:08
sean-k-mooneyboth are good questions.12:09
amoralej_I'd prefer a different term that supported for upstream, but i agree on the idea12:09
sean-k-mooneyin most project we do not merge the code until we think its ready for production12:09
*** amoralej_ is now known as amoralej12:09
sean-k-mooneyso we would not have merge thse poc impleation at all12:09
sean-k-mooneybut since they exist12:09
sean-k-mooneyto me its wether we think its mature enogh to run in production12:09
sean-k-mooneyand are we happy that we can continue to test and maintain it going forward12:10
chandankumarsince we have fixed the strategies, now it is working with ample test coverage but we have not data on usage Do we still want to mark them experimental?12:10
sean-k-mooneyif we are happy with the level of testing and docs and the stragey is fucntioanl we can remvoe the experimental lable12:10
jgilaberwe discussed at PTG to have an overview/status of the project, should the strategies state be documented there?12:10
sean-k-mooneyyes we agreed to create a matrics in the docs that woudl docuemtn the testing and supprot level of all stragies12:11
sean-k-mooneylike this https://docs.openstack.org/nova/latest/user/support-matrix.html12:11
rlandyso something like ...12:12
rlandyon the top level strategies page:12:12
rlandystrategy: test level experimental12:12
rlandyand we remove all references of support level in other places12:13
rlandyand we control it only there12:13
chandankumar+112:13
sean-k-mooneyyes i think so so here https://docs.openstack.org/watcher/latest/strategies/12:13
sean-k-mooneywhen we move something ebtween levels i think we shoud create a relese note for that as well12:13
rlandyas to which strategy gets what level, we can get feedback and agreement on the review12:14
rlandy+112:14
sean-k-mooneyi.e. when it goes form experiemtal to production ready we have a short relese not for that12:14
amoralej+112:14
sean-k-mooneyrlandy: yep we can do that in the review12:14
rlandyok - thank you all for the discussion12:14
rlandy#action rlandy to put up a review with a table (removed from other places in the doc) for review12:15
rlandyie: of strategies and test/experimental level12:15
chandankumarI have one more question12:15
rlandygo ahead12:15
chandankumarmany strategies are not triggered via horizon UI12:15
rlandygood point12:16
chandankumarDo we want to document that also in each strategy doc?12:16
rlandythose requiring params12:16
rlandyyes!12:16
rlandyin the table?12:16
jgilaber+1, we can have an extra column with UI support12:16
chandankumar+1 for extra column12:17
sean-k-mooneywe could12:17
chandankumarDo we want to put it in main strategy page?12:17
mtembo+1 extra column12:17
sean-k-mooneythe columns i was thinking of are as follws12:17
sean-k-mooneyname, testing level, supprot level(supproted, experimental, deprecation, removed) 12:18
sean-k-mooneywe can add horizon supprot12:18
rlandyremoved?12:18
sean-k-mooneyor we can encode that in a similar table in teh watcher-dashbaord repo12:18
sean-k-mooneyand cross linked12:18
sean-k-mooneyrlandy: so sometiems we docuemnt the release in which a feature is remvoed12:19
sean-k-mooneyim not sure if we need that or not12:19
sean-k-mooneybut that more for upgrades 12:19
sean-k-mooneywhen you are upgrding to a new release you can review the matics to understand if all the stragies you use are still suprpoted. this would be captrued in an upgrades and deprecation release note as well12:20
sean-k-mooneyso its not required12:20
sean-k-mooneywe can defer the detail i think to the review12:20
rlandyagreed - let's start with the review and continue the conversation there. thank you all for your input12:20
chandankumarrlandy: thank you for bringing it!12:20
rlandyanything else on this topic?12:21
rlandymoving on ...12:21
rlandy#topic (dviroel) Eventlet Removal12:21
dviroelo/12:22
rlandydviroel, do you want to take this?12:22
dviroelyep, first, wrt to the email sent last week, the call for MAAS maintainers, no answers so far12:22
rlandynow what do we do?12:22
dviroelso  I am planning to proceed with a patch to mark both MAAS and Ironic as experimental features, since they lack support, testing and docs12:22
dviroelthe question is12:23
dviroelis enough just to update the docs (the plugins part)12:23
dviroelor should we also consider adding some log messages?12:23
sean-k-mooneywe should likely add a log message and release note + a doc12:24
sean-k-mooneythe log message shoudl only log on startup IMO12:25
dviroelack, yeah, log once12:25
sean-k-mooneyneutron did that when they made linux bridge experimental12:25
dviroelright, so I will proceed with a patch for that12:25
dviroeltks for the inputs12:26
sean-k-mooneyjust for consitency we likely shoudl havea support/testing matrics for datasouces and integrations as well12:26
sean-k-mooneylooking at https://docs.openstack.org/watcher/latest/admin/index.html12:26
sean-k-mooneythere is a high level page for datasouces https://docs.openstack.org/watcher/latest/datasources/index.html so that is simple12:26
sean-k-mooneybut im not seeing anythin for mass/ironic there12:26
sean-k-mooneywe might want to add an "services" section to the doc12:26
sean-k-mooneywehre we list integration with nova cinder ironic mass ectra12:27
sean-k-mooneyi.e. the external services that watcher can interact with12:27
sean-k-mooneythere we can follow the same patteern as we jsut dicssed for rlandy topic12:28
dviroelyep, i found mention to ironic only on change_node_power_state action doc12:28
sean-k-mooneyya so the name of that is a littel hard "services" works but it could also be "integrations" or similar12:29
sean-k-mooneymaas makes it a littel werid because its not part of openstack12:29
sean-k-mooneythe complete lack of docs about ironic and mass is part of why they are experimental so we probly shoudl have a "docs level" column on these as well12:31
dviroelyeah12:31
dviroelany other ideas? 12:32
rlandysounds right for now12:32
dviroeli will take a look on docs and see what we can do12:32
dviroeland we can chat more12:32
sean-k-mooneydviroel: back to your question i think we can proceed with marking them as experimental and i would suggest startign a patch to do that and we can revisit next week.12:32
dviroeland we can chat more12:32
rlandydviroel: thank you for raising this ... anything more on the topic?12:33
dviroelnext thing is about changes in BackgroundSchedulerService and Watcher Services12:33
dviroelstill in progress but, the good news is that the patch with the new threading backend for oslo.service merged recently12:33
dviroelthread in ML:12:33
dviroel#link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/33EPOBRCGEKYUVDPFUZJZ2HENGDXVRE4/12:33
dviroeli have a open pacth for initial evenlet move, but I still need to updated it from comments12:34
dviroeland that't what I have for this week12:35
rlandyok- thank you - any more comments on anything dviroel raised?12:36
dviroelno, thanks12:36
rlandy#action dviroel to look at doc additions12:36
rlandymoving on ...12:37
rlandy#topic (sean): just an fyi we are going to use https://etherpad.opendev.org/p/grian-ui-status to track/drive developemt of the grian-ui plugin12:37
rlandysean-k-mooney: ^^ would you like to take that?12:37
sean-k-mooneyam the short verson is as follows12:38
sean-k-mooneydurign the ptg we agreed with the telemery team to create a new horizon plugin for tenant metrics vizualisation12:38
sean-k-mooneyover the last 2-3 weeks we have created the repo and started bootstraping it12:39
sean-k-mooneycurrently we are in the forming state choosing wehre we will host meeting, how we will track work ectra12:39
sean-k-mooneywe have decied to use that ether pad to drive that for now simialr to howe we are suign the etherpad for this meeting12:39
sean-k-mooneywe likely will use #openstack-telemetry to dicussion this topic going forward12:40
rlandy+1 do you need any additional support/resources?12:40
sean-k-mooneythats about it. for now no, we are hopign to have some intiall demos around m2 perhaps a littel later12:41
sean-k-mooneywe will likely starte with create 2 example staic dashbaord 1 an admin only dashboard12:41
sean-k-mooneyand a seocn that is tenatn facing to prove out both types12:42
sean-k-mooneythose will evlove over time12:42
sean-k-mooneybtu once we get to that initall pont we will likely ask for some user feedback12:42
sean-k-mooneyi think that is where the wider team can help12:42
sean-k-mooneyif you want to be more involed before that that fine too12:43
rlandythank you sean-k-mooney12:43
sean-k-mooneyanyway i think thats it unless there are any other questions?12:43
amoralejlet us known where you have something that we can install and test12:44
amoraleji'd like to check the dashboards once ready12:44
chandankumarDo we need anything from devstack-plugin-prometheus for grian-ui?12:45
sean-k-mooneyyep we will12:45
sean-k-mooneychandankumar: not currently12:45
chandankumarok12:45
sean-k-mooneyi have a basic devstack job ready 12:45
sean-k-mooneyit currently fails becasue we dont have a functial plugin yet12:45
sean-k-mooneyso the install fails as a result12:45
sean-k-mooneybut i have focused sofar on setign up the testing and jobs for the repo12:46
sean-k-mooneyso now that that is mostly done we can start on buildign out actual fucntionaliy12:46
rlandyok - anything more here?12:47
chandankumarthank you sean-k-mooney for sharing the progress here!12:48
rlandymoving on to give the other topics some time ...12:48
rlandy#topic: (jgilaber) Updated zone migration strategy etherpad with what I think are the relevant use cases12:48
rlandy#link https://etherpad.opendev.org/p/zone_migration_volume_migration_parameters12:49
rlandyjgilaber: ^^ do you want to take this?12:49
jgilaberyep12:49
jgilaberI have added a section on the same etherpad that I brought last week12:49
jgilaberwith the few use cases for volume migration that I think the zone migration can support12:49
jgilaberwith the help of amoralej 12:50
jgilaberwe came up with 3 scenarios12:50
jgilaber1 that should work currently, and 2 other that will require some changes to the volume migration action12:50
jgilaberI also have a small sections with questions that I think we'll need to ask to cinder experts12:51
jgilaberlet me know if you see anything that you think is wrong or is not clear12:51
sean-k-mooneywhat is the status of the vm migration part of this. is that still pending or did we merge the first two patches12:52
jgilabervm migration seems to work well12:52
jgilaberwe merged the path for the dst_node that I think is the only bug that affects that part12:53
sean-k-mooneyi think its good to continue working on the cinder part in followups but we can likely merge the intall partcial fix if we ahve nto already done so12:53
sean-k-mooneyok good12:53
sean-k-mooneyi jsut didnt recall if that part was merged or not12:53
jgilaberI've also changed the storage follow-up to that patch https://review.opendev.org/c/openstack/watcher/+/95014912:54
jgilaberafter the use cases review, I don't think it's great to have the dst_pool parameter mandatory12:54
sean-k-mooneyso the 3 types fo volume migratoin are retype, migration between host in a pool and migration between backend host in different pools correct?12:54
sean-k-mooneyjgilaber: yes i agree i talks to gmaan about this a litttle offline12:55
jgilaberI think that's right sean-k-mooney 12:55
sean-k-mooneyi think the way forward is to remvoe the required fields form the schema12:55
sean-k-mooneyand move the validation to the api where we can encode the logic of which combintions of optiosn are valid more cleanly12:55
sean-k-mooneyfor example for retry neither the souce or dest pool shoudl eb requried as far as i am aware12:56
sean-k-mooney*retype12:56
amoralejwrt the cases, the one that should work currently, which is the retype (dst_pool empty) has some known bugs that we need to fix. There is a list at the end of the etherpad12:56
jgilabersrc_pool is required currently as it is the only way it has to filter which volumes to migrate12:57
jgilaberthat is a bug, because src_type is ignored12:57
sean-k-mooneyhum ok12:57
sean-k-mooneythat is perhaps unfortunate. it would be nice in the case wehre you are moving vms to be able to retype the volumes assocated with those vms and only those volumes12:58
jgilaberbtw there is also a list on known bugs at the bottom of the etherpad12:58
jgilaberdefinitely we should fix that, we should be able to work with just the src_type12:59
sean-k-mooneyi.e. please mvoe the vms on host A in az 1 to AZ2 and retype there voluems form mass_storage_az1 to mass_storage_az212:59
amoralejsean-k-mooney, actually there is the opossite, migrate or retype the volumes in a pool and the vms with volumes attached to them13:00
amoralejthat exists already13:00
amoralej(with certain bugs)13:01
sean-k-mooneyi htink oen of the bugs that is more impoant to fix is the lack of documenation13:01
amoralejbut not the "migrate the VMs and their attached volumes"13:01
sean-k-mooneyfor such a complex stagy i think we will need to build out a set of usecase driven example13:02
amoralejyep13:02
rlandy(we are overtime but we can continue this discussion - will move the rest of the topics to next week)13:02
jgilaber+1, that's what we set out to do in the etherpad13:02
sean-k-mooneyack13:02
jgilaberthose use cases can then serve as the backbone of the documentation13:02
sean-k-mooneyack13:03
sean-k-mooneyill try and fined time to review that with more detail over the next week13:03
sean-k-mooneythanks for writing this up13:03
jgilaberthanks!13:03
rlandyjgilaber: anything more on this topic?13:04
jgilabernop, that is all from me13:04
rlandyok - thank you13:04
rlandyI will move the bug triage to next week or we can take that on line ...13:04
rlandypls note:13:04
rlandyRequest for review: https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/94955713:05
rlandychandankumar: ^^ any comments on that for the reviewers?13:05
chandankumarIt contains improvements from previous host maintenance tempest plugin patches 13:05
chandankumarPlease have a look, thank you!13:05
rlandythank you chandankumar13:06
rlandymoving on ...13:06
rlandyvolunteers to chair next week's meeting?13:06
chandankumarcount me in for next week13:06
rlandythank you chandankumar 13:07
rlandythanks all for attending and the good discussions13:07
chandankumarthank you rlandy !13:07
rlandypls take any topics we ran out of time for on to the channel or next week's agenda13:08
rlandy#endmeeting13:08
opendevmeetMeeting ended Thu May 29 13:08:20 2025 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)13:08
opendevmeetMinutes:        https://meetings.opendev.org/meetings/watcher_meeting___29_may_2025/2025/watcher_meeting___29_may_2025.2025-05-29-12.01.html13:08
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/watcher_meeting___29_may_2025/2025/watcher_meeting___29_may_2025.2025-05-29-12.01.txt13:08
opendevmeetLog:            https://meetings.opendev.org/meetings/watcher_meeting___29_may_2025/2025/watcher_meeting___29_may_2025.2025-05-29-12.01.log.html13:08
opendevreviewAlfredo Moralejo proposed openstack/watcher-tempest-plugin master: Enforce compute host in create_one_instance_per_host  https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/95038916:28
opendevreviewJoan Gilabert proposed openstack/watcher master: Enable storage model collector by default  https://review.opendev.org/c/openstack/watcher/+/95132316:45
opendevreviewRonelle Landy proposed openstack/watcher master: Update Workload Balance strategy documentation  https://review.opendev.org/c/openstack/watcher/+/95102519:04

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