12:01:04 #startmeeting watcher 12:01:04 Meeting started Thu Dec 19 12:01:04 2024 UTC and is due to finish in 60 minutes. The chair is rlandy. Information about MeetBot at http://wiki.debian.org/MeetBot. 12:01:04 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 12:01:04 The meeting name has been set to 'watcher' 12:01:18 hello all o/ who is around today? 12:01:22 o/ 12:01:25 o/ 12:03:20 #link https://etherpad.opendev.org/p/openstack-watcher-irc-meeting meeting agenda 12:03:35 ok lets get going on the agenda items we will take them in order given 12:04:21 #topic (rlandy) Patches requiring reviews / open reviews of note since last meeting 12:04:54 note this is not a full list of reviews - some new and active ones since the last meeting ... feel free to add any more for discussion 12:05:23 ok lets take them one at a time, give folks chance for any concerns or comments or status 12:05:40 #link https://review.opendev.org/c/openstack/watcher/+/934423 Add prometheus data source for watcher decision engine 12:05:56 marios: ^^ any comments or requests on that one? 12:06:57 looks like amoralej_ +1'ed the last revision 12:07:31 i'm half around :) 12:07:57 i think current state is good 12:08:16 this is also lgtm, as first version of prometheus datasource 12:08:33 thank you amoralej_ dviroel 12:09:04 ok, moving on to the next review ... 12:09:16 #link https://review.opendev.org/c/openstack/watcher/+/937625 Remove ceilometer datasource 12:10:02 looks like chandankumar and dviroel reviewed that 12:10:48 noting marios comment: "thanks for working on that Takashi. from a quick review i don't think there will be any conflict with the new prometheus data source" 12:11:44 any comments or concerns on that one? 12:12:08 right, didn't check why CI is not happy with that 12:13:08 ok, moving on to the next review ... 12:13:21 the next two are backports 12:13:33 #link https://review.opendev.org/c/openstack/watcher/+/937825 Backport Omnibus Fixes and Associated Tests 12:13:40 #link https://review.opendev.org/c/openstack/watcher/+/937823 Backport Omnibus Fixes and Associated Tests 12:14:22 marios added sean to those to review 12:15:54 any comments or concerns on those? 12:16:39 need to take a look but, commit message should at least specify which commits we are cherry picking there 12:17:19 request to add comments to the review - thank you 12:18:06 ok, moving on to the next review ... 12:18:17 #link https://review.opendev.org/c/openstack/watcher-dashboard/+/935043 Support multiple global_efficacy indicators in action plans table 12:19:20 has +1's from 3 reviewers 12:19:27 looks like awaiting cores 12:20:38 any comments or concerns on that one? 12:21:58 ok, moving on to the last review ... 12:22:12 #link https://review.opendev.org/c/openstack/watcher/+/822559 Fix incorrect logging format 12:22:36 this has a +1 from jgilaber 12:22:39 thank you 12:23:37 any comments or concerns on that one? 12:25:00 ok - any more reviews to highlight for today? 12:26:35 ok, moving on .. 12:26:52 #topic (rlandy) plan for 2025 to change watcher-tempest-plugin cores 12:27:30 kopecmartin sent a mailing list notification about changing focus in the new year 12:28:03 chandankumar will be coming on as core - thank you chandankumar 12:28:23 chandankumar has significant former tempest experience 12:29:14 any questions or concerns here? 12:29:20 +1 12:29:35 no concerns, +1 12:30:23 thank you 12:30:34 ok moving on ... 12:31:00 #topic (rlandy) reminder this meeting is canceled over the next two weeks (happy holidays) - next meeting is 9th January 2025 12:31:21 thank you amoralej_ for volunteering to chair that meeting 12:32:11 that is the end of the topics on the etherpad. is there any topic anyone else wants to raise? 12:32:23 #topic open discussion 12:33:53 anything anyone would like to bring up here before we close out today? 12:35:20 ok then thanks all for participating o/ 12:35:38 #endmeeting