Monday, 2017-07-24

*** thorst has joined #openstack-watcher00:31
*** thorst has quit IRC00:36
*** thorst has joined #openstack-watcher01:03
*** thorst has quit IRC01:03
openstackgerritMerged openstack/watcher-dashboard master: Add DS_Store to gitignore  https://review.openstack.org/48336902:25
*** thorst has joined #openstack-watcher03:04
*** thorst has quit IRC03:10
*** Yumeng has joined #openstack-watcher03:11
*** Yumeng has quit IRC03:14
*** Yumeng has joined #openstack-watcher03:14
openstackgerritsuzhengwei proposed openstack/watcher-specs master: add name for audit  https://review.openstack.org/47486503:53
*** adisky__ has joined #openstack-watcher04:35
*** thorst has joined #openstack-watcher05:05
*** thorst has quit IRC05:10
openstackgerritYumengBao proposed openstack/watcher-specs master: Specs for Saving Energy Strategy  https://review.openstack.org/47641605:24
openstackgerritHangdong Zhang proposed openstack/python-watcherclient master: Update the documentation link for doc migration  https://review.openstack.org/48649405:56
*** hidekazu has left #openstack-watcher06:11
*** hidekazu has joined #openstack-watcher06:16
hidekazuYumeng: hi06:18
Yumenghidekazu:hi06:18
hidekazuYumeng: i have a question: why you develop ironic data model??06:19
hidekazuYumeng: energy saving strategy does not use ironic data model.. so i have a question.06:20
Yumenghidekazu:The reason why we need a new bare metal data model is that we just want to find a place to record both the hypervisor id and node uuid.06:23
Yumenghidekazu:We have three options to implement this: Option 1: adding hypervisor id and node uuid to current data model in watcher06:23
Yumenghidekazu:Option 2: Store hypervisor id and node uuid in Ironic and watcher just retrieves the info from Ironic where needed.06:23
Yumenghidekazu:Option 3: Create new data model in watcher.06:24
YumengOriginally we want to choose option2, and in the last openstack PTG, we discussed this and agreed that watcher should keep a copy of this info. So option 2 was passed. Thinking with N different CDMs, if one goes wrong, it doesn't break everything. That's why finally we go for option with new bare metal mode.06:24
YumengHere is a discussion about why we prefer Option 3 other than Option 1. http://eavesdrop.openstack.org/irclogs/%23openstack-watcher/%23openstack-watcher.2017-03-20.log.html06:26
hidekazuYumeng: last ptg, i thought energy saving strategy will use the model.06:26
hidekazuYumeng: who will use the model other than energy saving?06:27
*** thorst has joined #openstack-watcher06:27
hidekazuYumeng: for our future?06:29
*** thorst has quit IRC06:31
hidekazui found in energy saving strategy spec: This "energy saving" goal can be fulfilled by adding four features in Watcher: #. Build a new baremetal data model within current Watcher Cluster Data Model06:35
hidekazubut energy saving will not use the model.06:36
*** nicolasbock has joined #openstack-watcher07:03
Yumenghidekazu: hi07:25
Yumenghidekazu: You are right, the current energy saving code didn't use the model.07:25
YumengThe code is written before PTG at which time option 2 is chosen as a solution.07:26
Yumenghidekazu:Current energy saving strategy is simple,so there is not much difference between using the model or directly retrieving the info from Ironic in the strategy.07:26
Yumenghidekazu:But one thing is certain, the data model is useful when retrieving info inside a strategy is not that simple.07:26
*** vincentfrancoise has joined #openstack-watcher07:26
hidekazuYumeng: so you do not use the model. and will not have plan.07:27
hidekazuYumeng: will you update Problem description section in energy saving spec ?07:29
hidekazucurrent spec says we will implement 'update the bare metal data model by ironic notifications' also.07:31
Yumenghidekazu: I can make modification to use it. But only worry about the pike-3 milestone. Should we finish the feature before pike-3, and then make modification later?07:31
hidekazuYumeng: i am concerning today is deadline for spec :)07:32
Yumenghidekazu: really? not July 30?07:32
hidekazuspec freeze on July 24 and feature freeze on July 3107:33
Yumenghidekazu: ok. so what would you suggest?07:33
Yumenghidekazu: I have to update this afternoon....07:34
hidekazuYumeng: if you will use model, you may propose feature freeze exception.07:35
hidekazuYumeng: because currently your spec says energy saving uses model07:35
hidekazuYumeng: option one: current spec merge: you will use model. option two: update current spec and merge: you will not use model.07:36
hidekazuYumeng: which do you want?07:37
*** vincentfrancoise has quit IRC07:39
openstackgerritMerged openstack/watcher-specs master: update action description  https://review.openstack.org/45952907:48
Yumenghidekazu: I just discussed with licanwei, we would follow option two now: update current spec and merge: you will not use model.07:48
hidekazuYumeng:  ok, i see.07:49
*** alexchadin has joined #openstack-watcher07:54
openstackgerritsuzhengwei proposed openstack/watcher-specs master: cluster-maintenance-strategy  https://review.openstack.org/46460207:55
hidekazuYumeng: i added -1 for spec now, please update.08:01
Yumenghidekazu: Thanks!  will update!08:01
openstackgerritAlexander Chadin proposed openstack/watcher-dashboard master: Update docs to the new standards  https://review.openstack.org/48336308:03
openstackgerritsuzhengwei proposed openstack/python-watcherclient master: add name for audit, changes for python-watcherclient  https://review.openstack.org/47532408:05
*** danpawlik has quit IRC08:17
*** danpawlik has joined #openstack-watcher08:17
*** thorst has joined #openstack-watcher08:27
*** thorst has quit IRC08:32
*** hidekazu has left #openstack-watcher09:04
openstackgerritYumengBao proposed openstack/watcher-specs master: Specs for Saving Energy Strategy  https://review.openstack.org/47641609:21
Yumengalexchadin: hi09:33
openstackgerritOpenStack Proposal Bot proposed openstack/watcher-dashboard master: Imported Translations from Zanata  https://review.openstack.org/48248709:33
Yumengalexchadin: I have just had a discussion with hidekazu about this patch  https://review.openstack.org/#/c/476416/09:34
Yumengalexchadin: discussion can be found here http://eavesdrop.openstack.org/irclogs/%23openstack-watcher/%23openstack-watcher.2017-07-24.log.html09:35
Yumengalexchadin: I have updated the patch according to his suggestions. But looks like hidekazu did get off work . So if you have time, can you take a look?09:37
Yumengalexchadin: I am not sure if hidekazu will review again today. he told me that today is the spec freeze day,09:38
Yumengalexchadin: hidekazu suggested me update current spec and then merge. I need your help if he is not here.09:41
alexchadinhi Yumeng09:44
alexchadinYumeng: I have a lot of work with release preparation and doc migration. If I have some time, I'll do my best09:45
Yumengalexchadin:ok. Thanks!10:26
*** thorst has joined #openstack-watcher10:29
*** thorst has quit IRC10:33
*** jwcroppe has quit IRC11:29
*** jwcroppe has joined #openstack-watcher11:43
*** Yumeng has quit IRC11:45
*** thorst has joined #openstack-watcher11:54
*** thorst_ has joined #openstack-watcher11:56
*** thorst_ has quit IRC11:56
*** thorst_ has joined #openstack-watcher11:57
*** thorst has quit IRC11:58
*** jwcroppe has quit IRC12:37
openstackgerritMerged openstack/watcher-specs master: Specs for Saving Energy Strategy  https://review.openstack.org/47641613:35
*** jwcroppe has joined #openstack-watcher13:54
openstackgerritAlexander Chadin proposed openstack/watcher master: Remove all warnings  https://review.openstack.org/48661913:56
*** efoley has joined #openstack-watcher14:04
openstackgerritAlexander Chadin proposed openstack/watcher master: Remove all sphinx warnings  https://review.openstack.org/48661914:05
*** alexchadin has quit IRC14:06
*** nicolasbock has quit IRC15:57
*** wootehfoot has joined #openstack-watcher16:14
*** harlowja has joined #openstack-watcher17:09
*** efoley has quit IRC17:15
*** thorst_ has quit IRC17:38
*** thorst has joined #openstack-watcher17:44
*** thorst has quit IRC17:49
*** edleafe- has joined #openstack-watcher17:52
*** edleafe has quit IRC17:55
*** edleafe- is now known as edleafe17:56
*** thorst has joined #openstack-watcher18:12
*** jwcroppe has quit IRC18:58
*** jwcroppe has joined #openstack-watcher18:59
*** jwcroppe_ has joined #openstack-watcher19:01
*** jwcroppe has quit IRC19:03
*** thorst is now known as thorst_afk19:15
*** jwcroppe_ has quit IRC19:45
*** jwcroppe has joined #openstack-watcher19:45
*** danpawlik has quit IRC19:48
*** jwcroppe has quit IRC19:50
*** danpawlik has joined #openstack-watcher19:50
*** jwcroppe has joined #openstack-watcher19:53
*** jwcroppe has quit IRC20:05
*** jwcroppe has joined #openstack-watcher20:24
openstackgerritOpenStack Proposal Bot proposed openstack/watcher-dashboard master: Updated from global requirements  https://review.openstack.org/47056720:39
*** wootehfoot has quit IRC20:46
*** wootehfoot has joined #openstack-watcher21:18
*** jwcroppe has quit IRC21:26
*** jwcroppe has joined #openstack-watcher21:32
*** wootehfoot has quit IRC21:33
*** jwcroppe has quit IRC21:42
*** jwcroppe has joined #openstack-watcher21:45
*** jwcroppe has quit IRC22:37
*** jwcroppe has joined #openstack-watcher22:44
*** danpawlik has quit IRC23:06
*** jwcroppe has quit IRC23:06
*** hidekazu has joined #openstack-watcher23:49

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!