Tuesday, 2017-03-21

*** pshedimb_ has quit IRC00:15
*** Yumeng has joined #openstack-watcher01:39
*** zhurong has joined #openstack-watcher01:58
openstackgerritMerged openstack/watcher-dashboard master: Remove the empty file form.py  https://review.openstack.org/44728002:01
*** sanfern has joined #openstack-watcher02:52
*** zhurong has quit IRC03:05
*** zhurong has joined #openstack-watcher03:21
*** hidekazu has quit IRC03:38
*** zhurong has quit IRC04:12
*** pshedimb_ has joined #openstack-watcher04:20
*** pshedimb_ has quit IRC04:27
*** adisky_ has joined #openstack-watcher04:32
*** karthikpr has quit IRC04:32
*** zhurong has joined #openstack-watcher05:00
*** zhurong has quit IRC05:40
*** hidekazu has joined #openstack-watcher06:02
openstackgerritshihanzhang proposed openstack/watcher master: Remove log translations  https://review.openstack.org/44785306:10
*** alexchadin has joined #openstack-watcher06:32
*** vincentfrancoise has joined #openstack-watcher08:17
openstackgerritSanthosh Fernandes proposed openstack/watcher master: Add period input parameter to vm workload consolidation and outlet temp control strategy.  https://review.openstack.org/44442908:32
*** zhurong has joined #openstack-watcher08:34
openstackgerritSanthosh Fernandes proposed openstack/watcher master: Add gnocchi support in basic_consolidation strategy  https://review.openstack.org/44029308:36
hidekazusanfern: hi08:45
sanfernhi hidekazu08:46
hidekazusanfern: i am reviewing https://review.openstack.org/#/c/440293/08:47
hidekazusanfern: in my enviroment, i got warning: ceilometer.dispatcher.gnocchi [-] metric compute.node.cpu.percent is not handled by Gnocchi08:47
hidekazusanfern: i can not get metric compute.node.cpu.percent. were you able to get it?08:48
sanfernyes knew it, there is gnocchi patch is required08:49
alexchadinvincentfrancoise: ping08:50
hidekazusanfern: could you give me the patch? :)08:51
*** openstackgerrit has quit IRC09:03
*** sanfern has quit IRC09:04
*** sanfern has joined #openstack-watcher09:04
vincentfrancoisealexchadin: hi09:08
vincentfrancoisealexchadin: will be back in 20mn09:08
alexchadinvincentfrancoise: ok09:08
*** openstackgerrit has joined #openstack-watcher09:24
openstackgerritMerged openstack/watcher master: Remove old oslo.messaging transport aliases  https://review.openstack.org/44736909:24
vincentfrancoisealexchadin: i'm back btw09:34
*** sanfern has quit IRC09:35
*** sanfern has joined #openstack-watcher09:37
sanfernhidekazu, sorry, i was in meeting, their is no patch submitted yet, plz see https://openstack.nimeyo.com/108133/openstack-ceilometer-gnocchi-push-compute-metrics-gnocchi09:42
hidekazusanfern: yes, i saw it.09:43
sanfernhidekazu, we will submit patch soon in gnocchi/ceilometer09:45
hidekazusanfern: thanks09:47
sanfernhidekazu, yw :)09:48
alexchadinvincentfrancoise: take a look http://paste.openstack.org/show/603583/09:48
vincentfrancoisealexchadin: nice!09:57
*** sanfern has quit IRC09:58
vincentfrancoisealexchadin: apart from the use case and some sentences that are hard to read, I'm ok with the content09:58
vincentfrancoisealexchadin: what is missing though is how you plan to use the Service.id to route the job09:59
alexchadinvincentfrancoise: which ones sentences are hard to read?10:00
alexchadinvincentfrancoise: I'm not sure that it is part of this BP10:00
vincentfrancoisealexchadin: the use case sentence is too long :p10:00
alexchadinvincentfrancoise: got it :)10:01
*** sanfern has joined #openstack-watcher10:01
vincentfrancoisealexchadin: also I would not focus on DE job routing since this is a generic tagging system that we plan to use for the DE jobs10:01
vincentfrancoisealexchadin: so instead of naming it "Supporting HA for continuous audits" I would instead call it "Supporting HA for background jobs in Watcher"10:02
vincentfrancoisealexchadin: and one of the use cases is to have it work for continuous audits (within the same or a separate BP)10:03
alexchadinvincentfrancoise: I agree. BTW, process of jobs routing should be described in another blueprint10:04
vincentfrancoisealexchadin: so I agree maybe the use of Service.id shouldn't be in the same BP10:04
*** zhurong has quit IRC10:05
vincentfrancoisealexchadin: so IMHO, the 1st blueprint is "Supporting HA for background jobs in Watcher" and does what you described here but without referring to continuous audits specifically10:05
vincentfrancoisealexchadin: and then a 2ndblueprint with the job routing for continuous audits called "Supporting HA for continuous audits" which would then tackle the routing process10:06
vincentfrancoisealexchadin: what do you think?10:06
alexchadinvincentfrancoise: by making this BP we will be able to continue our work on cron syntax, while routing should be described later10:06
alexchadinvincentfrancoise: +110:06
vincentfrancoisealexchadin: new blueprints again :p10:07
vincentfrancoisealexchadin: I still have to create the one from yesterday BTW10:07
alexchadinvincentfrancoise: stop them, there are multiply all the time!10:08
vincentfrancoisealexchadin: yeah, they are like rabbits :D10:08
alexchadinvincentfrancoise: btw, dividing this BP by two will not ruin workflow process10:09
vincentfrancoisealexchadin: which workflow process?10:09
alexchadinvincentfrancoise: I mean Watcher common workflow process, functional tests10:10
openstackgerritMerged openstack/watcher master: set eager=True for actionplan.list  https://review.openstack.org/44687410:19
*** pksingh has joined #openstack-watcher10:20
pksinghvincentfrancoise: alexchadin can you please look into https://review.openstack.org/#/c/443070/, and give your valuable suggestions, I will add UTs and FTs too in next PS10:22
*** zhurong has joined #openstack-watcher10:24
vincentfrancoisealexchadin: ok :)10:24
alexchadinpksingh: I will review it before tomorrow's meeting10:24
pksinghalexchadin: thanks :)10:24
vincentfrancoisepksingh: I remember having an indirect problem with your code10:25
pksinghvincentfrancoise: go ahead plz10:25
vincentfrancoisepksingh: but I didn't dig much10:25
vincentfrancoisepksingh: let me explain it to you10:25
vincentfrancoisepksingh: basically you add metadata="{'optimize': 'True'}" in all .xml files10:26
pksinghvincentfrancoise: yes10:26
vincentfrancoisepksingh: so 1st -> it is not json formatted10:26
pksinghvincentfrancoise: ok10:27
vincentfrancoisepksingh: 2nd, how do we ascertain that 2 structures are equal given that a json object is unordered by nature?10:28
pksinghvincentfrancoise: am i checking equality somewhere?10:29
vincentfrancoisepksingh: up until now, we are comparing the string output in watcher/tests/decision_engine/model/test_model.py10:30
vincentfrancoisepksingh: not you hence the "indirect problem"10:30
pksinghvincentfrancoise: ok let me check,10:31
pksinghvincentfrancoise: are you talking about https://github.com/openstack/watcher/blob/master/watcher/tests/decision_engine/model/test_model.py#L46 test case10:33
vincentfrancoisepksingh: yeah10:33
pksinghvincentfrancoise: ok, and why the structure is not json formatted?10:34
vincentfrancoisepksingh: I am wondering whether this test works if you have 2 or more keys in the metadata dict since they can end up being shuffled10:34
pksinghvincentfrancoise: double code and single codes should be interchanged?10:35
pksinghvincentfrancoise: i will test it by adding more keys10:35
vincentfrancoisepksingh: yeah kind of10:35
vincentfrancoisepksingh: basically, we need to make sure that the model we build via pure python code is equivalent to the one we generate from the XML files10:36
vincentfrancoisepksingh: then we can bootstrap from then onwards10:37
pksinghvincentfrancoise: do we need JSON, i am using FlexibleDictField for the metadata?10:37
pksinghvincentfrancoise: and it simple converts it into dict, https://github.com/openstack/watcher/blob/master/watcher/objects/fields.py#L6410:37
openstackgerritAlexander Chadin proposed openstack/watcher-specs master: Add spec for Supporting HA for background jobs in Watcher  https://review.openstack.org/44799610:37
*** alexchadin has quit IRC10:39
pksinghvincentfrancoise: ^^10:41
vincentfrancoisepksingh: IMHO, everything that is serialized should follow and serialization standard10:41
pksinghvincentfrancoise: OK great, thanks10:42
vincentfrancoisepksingh: so we should produce either some JSON or XML10:42
pksinghvincentfrancoise: ok10:43
vincentfrancoisepksingh: again, it's something I didn't pay attention in the code that added support for the metadata in the CDM10:44
vincentfrancoisepksingh: your code juste made me realise it10:44
pksinghvincentfrancoise: and we dont have JSON type in field, do we need to add it?10:45
vincentfrancoisepksingh: I guess so, not sure what's the best way to do this10:46
pksinghvincentfrancoise: something like https://github.com/openstack/zun/blob/master/zun/objects/fields.py#L75?10:46
vincentfrancoisepksingh: yeah, looks like a sound option10:48
pksinghvincentfrancoise: ok, will add this too10:48
vincentfrancoisepksingh: thanks10:49
pksinghvincentfrancoise: BTW you can put your comments on PS if you dont mind10:49
pksinghvincentfrancoise: i will work on it10:49
vincentfrancoisepksingh: ok10:50
*** pksingh has quit IRC10:50
*** sanfern has quit IRC10:51
*** alexchadin has joined #openstack-watcher10:54
*** alexchadin has quit IRC11:01
*** alexchadin has joined #openstack-watcher11:01
*** alexchadin has quit IRC11:06
*** Yumeng has quit IRC11:16
openstackgerritMerged openstack/watcher-dashboard master: Remove the extra required=True  https://review.openstack.org/44728111:21
openstackgerritYumengBao proposed openstack/watcher-specs master: Build baremetal data model in watcher  https://review.openstack.org/44801611:32
*** sanfern has joined #openstack-watcher11:40
*** alexchadin has joined #openstack-watcher12:03
openstackgerritYumengBao proposed openstack/watcher-specs master: Add new actions "power on" and "power off" in Watcher  https://review.openstack.org/43543612:29
*** zhurong has quit IRC12:39
openstackgerritAlexander Chadin proposed openstack/watcher-specs master: Add spec for Supporting HA for background jobs in Watcher  https://review.openstack.org/44799613:07
*** alexchadin has quit IRC13:11
openstackgerritSanthosh Fernandes proposed openstack/watcher master: Add period input parameter to vm workload consolidation and outlet temp control strategy.  https://review.openstack.org/44442913:27
*** alexchadin has joined #openstack-watcher14:31
openstackgerritlidaohan proposed openstack/python-watcherclient master: Remove log translations  https://review.openstack.org/44815515:24
*** karthikpr has joined #openstack-watcher15:28
*** karthikpr has quit IRC15:37
openstackgerritAlexander Chadin proposed openstack/watcher-specs master: Add spec for Supporting HA for background jobs in Watcher  https://review.openstack.org/44799615:41
openstackgerritlidaohan proposed openstack/python-watcherclient master: Remove log translations  https://review.openstack.org/44815516:20
*** karthikpr has joined #openstack-watcher16:30
*** alexchadin has quit IRC16:33
*** Daviey_ is now known as Daviey16:49
*** wootehfoot has joined #openstack-watcher17:26
*** ss4 has joined #openstack-watcher17:33
*** wootehfoot has quit IRC17:33
*** vincentfrancoise has quit IRC17:43
*** ss4 has quit IRC17:59
*** karthikpr has quit IRC18:16
*** karthikpr has joined #openstack-watcher18:31
*** wootehfoot has joined #openstack-watcher19:29
*** wootehfoot has quit IRC20:36
*** pshedimb has joined #openstack-watcher20:49
*** wootehfoot has joined #openstack-watcher21:05
*** wootehfoot has quit IRC21:19
*** wootehfoot has joined #openstack-watcher21:23
*** karthikpr has quit IRC21:25
*** adisky_ has quit IRC21:39
*** karthikpr has joined #openstack-watcher22:03
*** harlowja has quit IRC23:06
*** hidekazu has left #openstack-watcher23:12
*** wootehfoot has quit IRC23:15
*** d34dh0r53 has quit IRC23:50
*** _d34dh0r53_ has joined #openstack-watcher23:55

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