08:00:28 <licanwei> #startmeeting Watcher
08:00:29 <openstack> Meeting started Wed Jun 19 08:00:28 2019 UTC and is due to finish in 60 minutes.  The chair is licanwei. Information about MeetBot at http://wiki.debian.org/MeetBot.
08:00:30 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
08:00:32 <openstack> The meeting name has been set to 'watcher'
08:00:42 <Dantalion> hello
08:01:01 <licanwei> Dantalion: hi~
08:02:12 <licanwei> #topic Announcements
08:02:32 <licanwei> Ceilometer is planning to reintroduce cpu_util metric in T2/T3
08:02:57 <licanwei> Dantalion: where did you get the news?
08:03:08 <Dantalion> Let me look for it one sec
08:04:10 <chenke> I'm coming.
08:06:25 <Dantalion> The have hidden it well
08:08:05 <licanwei> Dantalion: I confirmed it with one Ceilometer core member
08:08:12 <chenke> It will be a rewarding meeting.
08:08:48 <licanwei> Dantalion: They plan to restore it ,but the main problem is no one have time to do it
08:08:54 <Dantalion> Okaj, good enough. It is hard to find ceilometer meeting notes.
08:09:18 <Dantalion> It must also be hard to reimplement because cpu_util used to work with rate_of_transformer feature which has been removed
08:09:18 <chenke> It will be a rewarding meeting.
08:09:24 <licanwei> Dantalion: We can wait until the end of Train
08:09:55 <licanwei> Dantalion: yes
08:10:04 <Dantalion> Yes I agree if at the end of Train cpu_util is still not available we will need to compute it ourself in Watcher
08:10:21 <Dantalion> cpu_util can be derived from cpu metric
08:10:34 <licanwei> #topic reviews
08:10:58 <licanwei> https://review.opendev.org/#/c/664613/ Documentation configuring grafana datasource needs review
08:11:20 <Dantalion> This is done I hope it is easy to understand
08:11:47 <licanwei> Dantalion: I'll review
08:12:03 <chenke> +1
08:12:33 <licanwei> https://review.opendev.org/#/c/663489/ check instance state for instance.update needs review
08:13:23 <Dantalion> This is related to notications bug right?
08:13:31 <Dantalion> I can look at this week
08:14:49 <licanwei> Dantalion: Thanks
08:15:23 <chenke> our internet speed  is very slow now.
08:15:53 <licanwei> https://review.opendev.org/#/c/649341/ Grafana proxy datasource to retrieve metrics
08:16:01 <licanwei> Dantalion: Is it ready for review?
08:16:19 <Dantalion> I need to do two small changes before review. I have commented details and put workflow -1 in meantime
08:16:34 <chenke> I see
08:16:57 <chenke> After your remove workflow, I will learn your code.
08:17:06 <Dantalion> Okaj thanks
08:17:31 <licanwei> https://review.opendev.org/#/c/661301/ Implement the configuration for Grafana datasource needs review
08:17:46 <licanwei> +1 for review
08:17:57 <chenke> +1
08:18:46 <licanwei> https://review.opendev.org/#/c/660011/ Add resource_name in action input parameter field
08:19:06 <licanwei> chenke:
08:19:38 <licanwei> chenke: you need to submit a new patch
08:19:45 <chenke> Ok
08:20:12 <chenke> After add name, I will update it.
08:20:36 <licanwei> #topic Open Discussion
08:21:16 <licanwei> Update launchpad Watcher core driver team and members
08:21:44 <licanwei> Dantalion: Thank you, I have the admin right now~
08:22:07 <Dantalion> Yes now we can manage Watcher core driver team and I can edit blueprints on launchpad :)
08:22:39 <Dantalion> I think core driver team poorly reflects current contributors maybe we can ask them if they still want to contribute and remove otherwise
08:23:06 <Dantalion> Asking them provides oppertunity to respark their interest in Watcher as well
08:23:30 <chenke> Good idea.
08:24:30 <chenke> Watcher is a project worth investing in.
08:24:45 <licanwei> Dantalion: through maillist?
08:25:22 <chenke> I  don't have mail outgoing permissions
08:25:48 <Dantalion> licanwei: I was thinking using email registered on Launchpad since they likely no longer read the watcher mailinglist
08:26:14 <Dantalion> For instance on userpage you can see email, example: https://launchpad.net/~xlei-boll
08:27:36 <licanwei> Dantalion: yes
08:28:08 <licanwei> Should we remove inactive Watcher core reviewers?
08:28:34 <Dantalion> I suggest the same procedure, email and ask if they still want to contribute if not we can remove them
08:29:47 <licanwei> Dantalion: Actually we had removed some inactive core last year
08:30:35 <Dantalion> Than maybe this one can still wait but I know at least one who can no longer contribute to Watcher because of new job
08:32:00 <licanwei> Dantalion: ok , I'll send a maillist
08:32:40 <licanwei> whether we need to rename the field 'human_id' of the data model
08:33:00 <licanwei> Let's continue this problem~
08:33:11 <Dantalion> I now fully understand difference between human_id and name
08:33:35 <Dantalion> human_id is equal to name but with strutils.to_slug(name)
08:33:43 <chenke> yes
08:33:46 <Dantalion> to_slug does the following: Convert to lowercase, remove non-word characters, and convert spaces to hyphens.
08:34:07 <chenke> congratulations
08:34:34 <chenke> I suggest we use original name
08:35:02 <licanwei> chenke: +1
08:35:04 <Dantalion> I think replacing human_id with name is fine
08:36:27 <licanwei> ok, last one
08:36:30 <licanwei> whether we need to receive and process the Nova unversioned notifications
08:36:50 <licanwei> as mentioned in https://review.opendev.org/#/c/663332
08:37:02 <Dantalion> Is nova unversioned old and versioned new?
08:37:49 <licanwei> Dantalion: yes
08:38:28 <Dantalion> Do we want Watcher to listen for nova unversioned notifcations or is this about the Nova configuration in devstack?
08:39:03 <Dantalion> I think the devstack configuration is fine to prevent the Ceilometer bug
08:39:16 <Dantalion> But what would be the benefit of Watcher supporting unversioned?
08:40:27 <licanwei> Dantalion: Do you know that Nova emit unversioned or versioned notification or both in CERN infrastructure?
08:41:32 <Dantalion> licanwei: I will ask takes only 30 seconds
08:42:12 <licanwei> Dantalion: ok :)
08:43:18 <Dantalion> We only use versioned notifications from Nova and are moving away from unversioned in other services as well
08:44:14 <licanwei> Dantalion: OK
08:45:35 <chenke> So we should continue to use version
08:46:01 <Dantalion> Yes I think so but the devstack seems to need Nova to emit both because of Ceilometer bug
08:46:12 <Dantalion> And I think that is fine until the bug is resolved
08:46:24 <licanwei> Dantalion: User need to change the nova config to enable versioned notification if they want to use Watcher
08:46:51 <licanwei> Dantalion: I think we should add a comment
08:47:18 <chenke> I agree.
08:47:22 <Dantalion> licanweI: Maybe something to put in releasenotes that Watcher requires versioned notifications from Nova
08:47:54 <licanwei> Dantalion: good idea~
08:48:20 <chenke> We should remind users if they are not using devstack.
08:49:59 <licanwei> any question?
08:50:09 <Dantalion> I have no questions
08:50:17 <chenke> Me too.
08:50:34 <licanwei> OK, I'll end the meeting
08:50:47 <licanwei> thank you all~
08:50:59 <Dantalion> Thanks everyone
08:51:01 <chenke> thanks everyone.
08:51:08 <chenke> bye~~
08:51:19 <licanwei> Have a good day~
08:51:28 <licanwei> #endmeeting