Thursday, 2025-09-11

« Wednesday, 2025-09-10 Index Friday, 2025-09-12 »
dviroelhi all o/ - watcher meeting will start in 10 minutes, here in this channel11:50
dviroeletherpad link https://etherpad.opendev.org/p/openstack-watcher-irc-meeting#L3311:51
dviroel#startmeeting watcher12:00
opendevmeetMeeting started Thu Sep 11 12:00:58 2025 UTC and is due to finish in 60 minutes.  The chair is dviroel. 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'12:00
dviroelhi all, lets see who is around today o/12:01
dviroelcourtesy ping: amoralej jgilaber sean-k-mooney chandankumar morenod rlandy12:02
morenodo/12:02
sean-k-mooneyo/12:02
rlandyo/12:02
dviroellet's start with today's meeting agenda12:02
dviroel#link https://etherpad.opendev.org/p/openstack-watcher-irc-meeting#L33 (Meeting agenda)12:02
dviroelfeel free to add your own topics to the agenda12:03
dviroel#topic Announcements12:03
dviroel#link https://releases.openstack.org/flamingo/schedule.html12:04
dviroelwe are at RC1 target week12:04
dviroela new branch is expected, for 2025.2 release12:05
dviroelwhich for now wil only receive release-critical issues12:05
dviroelin case a critical bug is found during these days, until de final release12:05
dviroelrc1 patch is ready, only waiting the prelude to merge12:06
dviroel#link https://review.opendev.org/c/openstack/releases/+/96012312:07
dviroelwe can cover the prelude and cycle highlight patches in the next topic.12:07
* dviroel prelude to merge, and hash to be updated in the release patch :)12:08
dviroelok, moving to the next announcement12:08
dviroelthis is a important one12:08
dviroelCore team updates - removals, reminders, and new addition12:08
dviroeljust in case you missed this email from sean-k-mooney, in openstack-discuss ml12:08
dviroel#link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/QYX6OP2MAMKZ2LFYVCIEIZFRHO5ZKSAZ/12:09
dviroelthere are some updates proposed in this thread12:09
dviroel1. Immediate removals from cores that  have been inactive for an extended period time12:10
dviroel2. a new core proposal, for watcher-core group: jgilaber 12:10
dviroeland a notice, for more cores removal in future releases12:11
sean-k-mooney ya so dviroel remieded me of the exact wording we said at the last ptg12:11
sean-k-mooneyi.e. that we woudl consider activey in the last 2 cycles as recent12:11
dviroel+112:12
sean-k-mooneydviroel: also did the work to find the contibutors activeity dates12:12
sean-k-mooneywhich helped a lot12:12
sean-k-mooneyi have not seen any objections12:12
sean-k-mooneyso unless there are any now12:13
sean-k-mooneyill implement the changes after the meeting12:13
dviroelanyone has any objections on both changes? removals and addition?12:13
dviroel#agreed on removal of core reviewers have been inactive for more than 2 cycles12:14
dviroel#agreed  on adding jgilaber to watcher-core group12:14
dviroeljgilaber is out today, it is a public holiday in his location12:15
sean-k-mooneygoing forward what i think we shoudl do is review this at the start of each slurp12:15
sean-k-mooneyand see who has not reviewd in teh prior slurp and who has but with very low activity12:16
dviroel++12:16
sean-k-mooneythis being the removals mainly12:16
sean-k-mooneywe can evaualte addtions on an ongoing basis12:16
dviroelsure12:16
dviroelsean-k-mooney: thanks for proposing these updates12:17
sean-k-mooneyno worries, i kind of have a related topic to this12:17
sean-k-mooneybut we can talk about that at the end if we have time12:17
dviroelACK12:17
dviroelsorry, ack12:17
dviroeljgilaber welcome, and thanks for the help with reviews o/12:18
dviroelmoving to the next topic12:18
dviroel#topic reviews12:18
dviroel#link https://review.opendev.org/c/openstack/watcher/+/960298 (relese note prelude)12:18
dviroelthis is the release prelude, thanks sean-k-mooney for proposing12:19
dviroelwe can see it rendered here https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_e20/openstack/e2080f4dda5044b18871da18757bb77f/docs/unreleased.html12:19
dviroellgtm12:19
dviroelwe need to merge it soon, to proceed with the rc1 release12:19
sean-k-mooney yes. again if there are no objects i wil likely merge it later today12:20
dviroelplease folks, review it now while we still have some time12:20
dviroel++12:20
sean-k-mooneyonce its merged ill update the sha in the rc1 release patch with it12:20
dviroelgreat, thanks!12:21
dviroelthe next one is the cycle highlights12:21
dviroel#link https://review.opendev.org/c/openstack/releases/+/960325 (cycle highlights)12:21
dviroelwhich should merge soon12:21
dviroelsince was W+1 minutes ago12:21
dviroelI added a comment about some possible additions, but I think that the patch covers the most important ones12:22
dviroelthanks again for proposing sean-k-mooney 12:22
sean-k-mooneyso next cycle these will fall to our new release liasons to do12:22
sean-k-mooneybut i wanted to compelt htem for this cycle12:22
sean-k-mooneythis is almost hte last of the release work for this cycle12:23
dviroel+112:23
sean-k-mooneythe last part, which was goign to be my other topic12:23
sean-k-mooneyis we need to update the specs repo, and create teh launchpad series for 2026.112:24
sean-k-mooneythe bit i wanted to dicuss was12:24
sean-k-mooneyshould i add other to the launchpad group for the seirs cration12:24
sean-k-mooneyor not12:24
sean-k-mooneycurrently its myself and checkner12:25
sean-k-mooneyhttps://launchpad.net/~watcher-drivers/+members12:25
dviroelyeah, I do think so12:25
sean-k-mooneyso if i do, shoudl it be the cores or the release liasosn12:25
sean-k-mooneyor both12:26
dviroelwe can assign the series creation task to the liasons I think12:26
sean-k-mooneywe can table this till next weeks irc meeting if we want12:26
sean-k-mooneyya its actully listed in the release liaosn guide i created12:27
dviroelwe currently have 2 active liasons, which can split the work12:27
sean-k-mooneyack12:27
rlandychandan in out until next week12:27
sean-k-mooneyya i was just going ot do it this cycle12:27
dviroeli would say that we can have at least one or 2 cores to help as admins of this group12:27
sean-k-mooneybut i can add them so they can do it going forward12:27
dviroeland the liasons to do the work12:27
sean-k-mooneyok that works for me12:28
dviroel+112:28
dviroelabout the watcher-specs updates12:28
dviroel#link https://review.opendev.org/c/openstack/watcher-specs/+/96017712:28
dviroelI proposed this patch, but needs another eyes to see if I am missing something12:29
sean-k-mooneyoh great you did have time to create it12:29
sean-k-mooneyskiming it it looks ok but ill review it properly later12:30
dviroelack, tks12:30
sean-k-mooneyi think going forward we shoudl do this in 2 steps12:30
sean-k-mooneywehn we pass spec freeze at milestone 212:30
sean-k-mooneywe shoudl create the spec dir for the next cycle12:31
sean-k-mooneyand then all that is needed at FF+1 shoudl be moving the implmented spec and redirects12:31
sean-k-mooneydoing it all in one patch is also fine12:31
dviroelmakes sense, since we can move propose specs to the next cycle, if they don't fit in the current one12:31
sean-k-mooneyexactly12:31
dviroel+112:32
sean-k-mooneyit elimiates the potital for a review gap if we have context loaded12:32
sean-k-mooneyanything else on release topics12:33
sean-k-mooneyif not we can move on i think12:33
dviroelsure, lets move12:33
dviroel#topic Bug Triage12:33
dviroelhere I added a lot of open bugs12:33
dviroelwhich we may not have time to cover today12:33
dviroelbut then I will move to next meeting agenda12:34
dviroel#link https://bugs.launchpad.net/watcher/+bug/2122347 (No CORS support)12:34
dviroeltkajinam open this one, and raised the issue here in the channel12:34
dviroelthe thing is that currently watcher does not support CORS middleware12:35
opendevreviewOpenStack Release Bot proposed openstack/watcher-dashboard stable/2025.2: Update .gitreview for stable/2025.2  https://review.opendev.org/c/openstack/watcher-dashboard/+/96052312:35
opendevreviewOpenStack Release Bot proposed openstack/watcher-dashboard stable/2025.2: Update TOX_CONSTRAINTS_FILE for stable/2025.2  https://review.opendev.org/c/openstack/watcher-dashboard/+/96052412:35
opendevreviewOpenStack Release Bot proposed openstack/watcher-dashboard master: Update master for stable/2025.2  https://review.opendev.org/c/openstack/watcher-dashboard/+/96052512:35
* dviroel bot drained my attention12:36
sean-k-mooneyya i was also lookign at those in other channels.12:36
sean-k-mooneyya so i think its imporant to add supprot for cors at least somewhat natively12:36
sean-k-mooneyits possibel to do this in teh websever but that not how all the rest of openstack works12:37
sean-k-mooneyso we shoudl try not to make watcher special12:37
sean-k-mooneyim unsure if we shoudl consdier backporting this or not.12:37
sean-k-mooneythsi si technialy a feature and woudl be an expction to the policy12:38
dviroelyeah, this is an important discussion to have (the backporting)12:38
sean-k-mooneyso my proposal is let dicuss that spereatly after we are happy with the change and the testing fo the change12:38
dviroelthe patch is proposed btw12:39
sean-k-mooneyits is yes12:39
dviroel#link https://review.opendev.org/c/openstack/watcher/+/96004412:39
* dviroel it took some time to open the LP here12:39
sean-k-mooneyit does nto tweak ci to test the chagne. 12:40
sean-k-mooneyi was undecieed if i wanted to ask takashi to do that or just do that ourselve in a follow up12:40
sean-k-mooneygiven they are busy with other things12:40
sean-k-mooneyi was leanign towrads doign it in a follow up12:40
dviroelyeah we can split the work here, and propose the ci testing 12:41
sean-k-mooneywe do actully have the tls proxy enabled12:42
sean-k-mooneyhttps://b56ae66d8d1228f47a8b-8be9667f5530aa8ec0c6e8c86da1b76d.ssl.cf1.rackcdn.com/openstack/f7310413ea7642ef81922e59abaece31/controller/logs/local_conf.txt12:42
sean-k-mooneyim just not sure if we shoudl ahve other testing and if we have cors configuredc in the job12:42
sean-k-mooneyso that woudl eb the scope fo the follow up which ties into the next bug i think12:42
dviroelack12:43
dviroelwe already have the importance set in the CORS one12:43
dviroelmoving to the next12:43
dviroel#link https://bugs.launchpad.net/watcher/+bug/2122353 (links in response contain wrong protocol when standalone api...)12:43
sean-k-mooneyso we have a few options here.12:44
sean-k-mooneyfirst this will be fixed as a side effect of the CORS patch i belive12:44
sean-k-mooneybut standaloen api here means eventlet12:44
dviroelhum12:44
sean-k-mooneywhich we have deprecated and plan to remove12:44
dviroelright, which is deprecated12:45
sean-k-mooneyso we coudl chosoe to not fix this12:45
sean-k-mooneyoh im wrong12:45
sean-k-mooneythe fix is to use  http_proxy_to_wsgi middleware12:45
dviroel#link https://review.opendev.org/c/openstack/watcher/+/96015712:46
sean-k-mooneyoh so no12:46
sean-k-mooneythat uses to be in the cors patch in v112:46
sean-k-mooneytakashi split that out so it can be backported sepetately12:46
sean-k-mooneydviroel: soim inclided to say valid and medium12:47
sean-k-mooneyand we shoudl backport that12:47
dviroelok, updating the bug then12:47
dviroelI will need to take a look with more time on that12:48
sean-k-mooneythe only issue i see is its currently  the 3rd patch in the chain12:48
sean-k-mooneyafter the cores work and the requeit id change12:48
dviroelwhich brings us to the next bug12:48
sean-k-mooney:)12:48
dviroel#link https://bugs.launchpad.net/watcher/+bug/2122350 (x-openstack-request-id header is not included in response headers)12:48
sean-k-mooneyya so this si important for debuging12:49
dviroel#link https://review.opendev.org/c/openstack/watcher/+/96015412:49
sean-k-mooneyso that you can trace calss across services12:49
dviroelyeah, this is very helpful12:49
sean-k-mooneyas the request id is the same end to end12:49
dviroelthis is really a missing thing on watcher12:49
sean-k-mooneyso agin i woudl triage as medium 12:49
sean-k-mooneyoperaationally this is imporant to debug problems in production properly 12:50
sean-k-mooneyover all im leanign towards sayign we shoudl backprot all 3 or we shoudl ask takasi to put the cors patch at teh end instead of the start12:50
dviroel++12:50
sean-k-mooneythe reaon these depend on each other today is the cors path add a depency on oslo.middleware12:51
sean-k-mooneywhich i think is why the content provider job is failign because we will need to update teh watcher spec file12:52
sean-k-mooneyadding that dep escpially since they are adding a very old verion of it as the min requireemnt (which is valid for the satble branches too) 12:53
dviroelso the idea of putting the cors at the end is that we may or may not backport it?12:53
sean-k-mooneyshodul be ok12:53
sean-k-mooneyyep12:53
sean-k-mooneyit allwos use to deferr that if we want to backprot the other two12:54
sean-k-mooneywhich are more clear cut to me12:54
dviroelack, sounds like a good plan12:54
dviroelagree12:54
sean-k-mooneylets try and capature that in the gerrti review12:54
sean-k-mooneytkajinam: ^ but if you do have time to read back hopefully that works for you too12:55
dviroelanything else to add wrt these bugs? I think we can cover at least one more12:55
dviroel#link https://blueprints.launchpad.net/watcher/+spec/instance-live-migration-timeout12:55
dviroelthis is a possible bug :) 12:56
sean-k-mooneyya so i have nto converted it yet12:56
sean-k-mooneyi went through all the old still open blueprint durign the week12:56
sean-k-mooneyand marked them obsolete12:56
sean-k-mooneybtu i found this whiel doing that12:56
sean-k-mooneyi have checked and as far as i can tell the report is still correct12:56
dviroelhum, that the timeout is 120 seconds (hard coded)12:57
sean-k-mooneywe take the time out as a keywrod arg and default to 12012:57
sean-k-mooneybut we never pass that keyword arge anywhere12:57
sean-k-mooneyso yes its effectivly ahrdcoded12:57
sean-k-mooneyunless i missed it12:57
sean-k-mooneythe question however is how to fix it12:57
sean-k-mooneyif we add a config option or more so if we add a stragey parmater 12:58
dviroelor both12:58
sean-k-mooneythis puts it into feature or master only territory12:58
sean-k-mooneyya or both a defautl in the config with an overried in teh audit12:58
sean-k-mooneyso i dont know if this shoudl eb a bug12:59
sean-k-mooney or a feature but i suspect we need a general solution12:59
sean-k-mooneyi.e. for other operations as well. so maybe somethign we need to investiage12:59
sean-k-mooneywe can revisit or dicuss outside the meeting13:00
sean-k-mooneyjust wanted to highlight it 13:00
dviroeli would say that we should at least have a proposal that we can backport, this can be a blocker for some operators that want to use watcher, since migration is the most important/used action13:01
dviroelmaybe split in two different solutions13:01
dviroelbut something that we can discuss afterwards, yes13:01
sean-k-mooneyagreed on both fronts13:02
sean-k-mooneywe are at/over time13:02
sean-k-mooneydo we want to contineu or wrap up13:02
dviroellets wrap 13:02
dviroeland move the remaining bugs to the next weel13:03
dviroeltks rlandy for volunteering to chair next week13:03
dviroeli will be also available if needed13:03
dviroelthank you all for participating13:03
dviroelwe will meet again next week o/13:04
dviroel#endmeeting13:04
opendevmeetMeeting ended Thu Sep 11 13:04:07 2025 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)13:04
opendevmeetMinutes:        https://meetings.opendev.org/meetings/watcher/2025/watcher.2025-09-11-12.00.html13:04
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/watcher/2025/watcher.2025-09-11-12.00.txt13:04
opendevmeetLog:            https://meetings.opendev.org/meetings/watcher/2025/watcher.2025-09-11-12.00.log.html13:04
morenodthanks for hosting dviroel 13:04
* tkajinam reading the log13:11
opendevreviewTakashi Kajinami proposed openstack/watcher master: Fix missing X-OpenStack-Request-ID header  https://review.opendev.org/c/openstack/watcher/+/96015413:13
opendevreviewTakashi Kajinami proposed openstack/watcher master: Add capability to parse forward headers  https://review.opendev.org/c/openstack/watcher/+/96015713:14
opendevreviewTakashi Kajinami proposed openstack/watcher master: Fix missing CORS middleware  https://review.opendev.org/c/openstack/watcher/+/96004413:16
tkajinam^^ sean-k-mooney changed the order13:16
dviroeltkajinam++ tks13:30
opendevreviewTakashi Kajinami proposed openstack/watcher master: Fix missing X-OpenStack-Request-ID header  https://review.opendev.org/c/openstack/watcher/+/96015415:18
opendevreviewTakashi Kajinami proposed openstack/watcher master: Add capability to parse forward headers  https://review.opendev.org/c/openstack/watcher/+/96015715:19
opendevreviewTakashi Kajinami proposed openstack/watcher master: Fix missing CORS middleware  https://review.opendev.org/c/openstack/watcher/+/96004415:19
opendevreviewTakashi Kajinami proposed openstack/watcher master: Fix missing X-OpenStack-Request-ID header  https://review.opendev.org/c/openstack/watcher/+/96015415:24
opendevreviewTakashi Kajinami proposed openstack/watcher master: Add capability to parse forward headers  https://review.opendev.org/c/openstack/watcher/+/96015715:24
opendevreviewTakashi Kajinami proposed openstack/watcher master: Fix missing CORS middleware  https://review.opendev.org/c/openstack/watcher/+/96004415:24
opendevreviewTakashi Kajinami proposed openstack/watcher master: Fix missing X-OpenStack-Request-ID header  https://review.opendev.org/c/openstack/watcher/+/96015415:46
opendevreviewTakashi Kajinami proposed openstack/watcher master: Fix missing X-OpenStack-Request-ID header  https://review.opendev.org/c/openstack/watcher/+/96015415:46
opendevreviewTakashi Kajinami proposed openstack/watcher master: Add capability to parse forward headers  https://review.opendev.org/c/openstack/watcher/+/96015715:50
opendevreviewTakashi Kajinami proposed openstack/watcher master: Fix missing CORS middleware  https://review.opendev.org/c/openstack/watcher/+/96004415:50
opendevreviewMerged openstack/watcher master: Add 2025.2 release notes prelude  https://review.opendev.org/c/openstack/watcher/+/96029819:08
opendevreviewMerged openstack/watcher master: Enable storage model collector by default  https://review.opendev.org/c/openstack/watcher/+/95132319:36
opendevreviewMerged openstack/watcher master: Handle missing dst_pool parameter in zone_migration  https://review.opendev.org/c/openstack/watcher/+/95014920:57
« Wednesday, 2025-09-10 Index Friday, 2025-09-12 »

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