14:00:04 <alexchadin> #startmeeting watcher
14:00:05 <openstack> Meeting started Wed Dec 13 14:00:04 2017 UTC and is due to finish in 60 minutes.  The chair is alexchadin. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:00:06 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:00:08 <openstack> The meeting name has been set to 'watcher'
14:00:45 <alexchadin> Hi!
14:01:10 <suzhengwei> hi
14:01:46 <alexchadin> Our agenda: https://wiki.openstack.org/wiki/Watcher_Meeting_Agenda#12.2F13.2F2017
14:02:51 <alexchadin> #topic Announcements
14:03:07 <alexchadin> We have released watcher on queens-2 milestone
14:03:53 <alexchadin> #topic Review Action Items
14:04:00 <alexchadin> Watcher specs
14:04:04 <hidekazu> hi
14:04:10 <alexchadin> exclude locked instances we should wait for an answer from Nova. #link https://review.openstack.org/#/c/506078/
14:04:17 <alexchadin> hidekazu: hello again
14:04:49 <hidekazu> alexchadin: :)
14:05:03 <suzhengwei> I am draft the spec for nova
14:05:15 <alexchadin> suzhengwei: cool
14:05:38 <alexchadin> suzhengwei: don't forget, this spec won't be merged before Rocky cycle
14:05:57 <suzhengwei> en
14:06:20 <suzhengwei> yes
14:06:58 <alexchadin> suzhengwei: oh, seems that you don't need to write spec
14:07:15 <alexchadin> suzhengwei: Matt: "I agree with gibi that we should have a specless blueprint for this, which can be addressed in the Rocky release at this point in the Queens schedule."
14:08:11 <alexchadin> suzhengwei: just register blueprint with detailed description and assign your commit to it
14:09:04 <suzhengwei> ok
14:09:09 <alexchadin> Add storage capacity balance Strategy needs reviews #link https://review.openstack.org/#/c/501659/
14:10:39 <alexchadin> hidekazu: you don't have enough to review it on this week, right?
14:10:49 <alexchadin> enough time*
14:11:11 <hidekazu> alexchadin: i am reviewing code first for storage capacity balance strategy.
14:12:10 <alexchadin> ok
14:12:45 <alexchadin> Watcher
14:12:56 <alexchadin> improve 'Restrict existing strategies to their default scope' needs reviews #link https://review.openstack.org/#/c/504857/
14:15:59 <alexchadin> hidekazu: why do you think it isn't related to extend-node-status bp? he uses disabled_reason in this commit
14:18:07 <hidekazu> alexchadin: this patch does not meet use case in the spec.
14:20:02 <alexchadin> hidekazu: yes, you are right
14:20:09 <alexchadin> suzhengwei: i'll review it
14:20:20 <alexchadin> #action alexchadin review https://review.openstack.org/#/c/504857/
14:20:34 <hidekazu> alexchadin: scope issue should be discussed separately.
14:20:56 <hidekazu> alexchadin: next ptg (if exists) will be better.
14:21:45 <hidekazu> alexchadin: anyway, please go ahead.
14:22:01 <alexchadin> hidekazu: will you go to PTG?
14:22:15 <alexchadin> hidekazu: I will try
14:22:18 <hidekazu> alexchadin: not sure.
14:22:31 <hidekazu> alexchadin: great
14:22:55 <alexchadin> Add Datasource Abstraction needs final core review #link https://review.openstack.org/#/c/518852/
14:24:41 <alexchadin> who would like to review it?
14:25:55 <suzhengwei> I think we should deprecate clielometer in the bp
14:27:07 <alexchadin> suzhengwei: Yumeng said ZTE still use it, but they are going to change datasource to gnocchi
14:28:33 <suzhengwei> we should keep the step for Rocky
14:28:42 <alexchadin> hidekazu: could you please review https://review.openstack.org/#/c/518852/ ?
14:28:59 <hidekazu> alexchadin: ok
14:29:44 <alexchadin> hidekazu: thanks
14:29:54 <alexchadin> #action hidekazu review https://review.openstack.org/#/c/518852/
14:30:59 <alexchadin> 'get_volume_type_by_backendname' returns a list needs reviews #link https://review.openstack.org/#/c/523040/
14:33:42 <alexchadin> hidekazu: I suppose this bug fix is for you ;)
14:34:11 <hidekazu> alexchadin: yes, code seems fine. i want to try in my env.
14:34:31 <hidekazu> alexchadin: assign me.
14:36:15 <alexchadin> #action hidekazu review https://review.openstack.org/#/c/523040/
14:36:18 <alexchadin> hidekazu: done
14:36:45 <alexchadin> option to rollback action_plan when it fails needs new PS #link https://review.openstack.org/#/c/517179/
14:36:53 <alexchadin> suzhengwei: take a look once you have some time
14:37:09 <suzhengwei> yes
14:38:29 <alexchadin> filter exclude instances during migration zuul jobs should be rechecked #link https://review.openstack.org/#/c/508108/
14:38:51 <alexchadin> suzhengwei: this one too
14:39:01 <suzhengwei> ok
14:39:06 <alexchadin> Add zone migration strategy needs new PS #link https://review.openstack.org/#/c/521691/
14:43:01 <alexchadin> suzhengwei: I can still retrieve services by name
14:47:32 <alexchadin> suzhengwei: please, add link to this fix to your patch set
14:48:40 <alexchadin> well, that's all
14:48:44 <alexchadin> thank you for coming!
14:49:29 <alexchadin> bye!
14:49:52 <suzhengwei> bye
14:50:28 <alexchadin> #endmeeting