| *** pshedimb_ has quit IRC | 00:15 | |
| *** Yumeng has joined #openstack-watcher | 01:39 | |
| *** zhurong has joined #openstack-watcher | 01:58 | |
| openstackgerrit | Merged openstack/watcher-dashboard master: Remove the empty file form.py https://review.openstack.org/447280 | 02:01 |
|---|---|---|
| *** sanfern has joined #openstack-watcher | 02:52 | |
| *** zhurong has quit IRC | 03:05 | |
| *** zhurong has joined #openstack-watcher | 03:21 | |
| *** hidekazu has quit IRC | 03:38 | |
| *** zhurong has quit IRC | 04:12 | |
| *** pshedimb_ has joined #openstack-watcher | 04:20 | |
| *** pshedimb_ has quit IRC | 04:27 | |
| *** adisky_ has joined #openstack-watcher | 04:32 | |
| *** karthikpr has quit IRC | 04:32 | |
| *** zhurong has joined #openstack-watcher | 05:00 | |
| *** zhurong has quit IRC | 05:40 | |
| *** hidekazu has joined #openstack-watcher | 06:02 | |
| openstackgerrit | shihanzhang proposed openstack/watcher master: Remove log translations https://review.openstack.org/447853 | 06:10 |
| *** alexchadin has joined #openstack-watcher | 06:32 | |
| *** vincentfrancoise has joined #openstack-watcher | 08:17 | |
| openstackgerrit | Santhosh Fernandes proposed openstack/watcher master: Add period input parameter to vm workload consolidation and outlet temp control strategy. https://review.openstack.org/444429 | 08:32 |
| *** zhurong has joined #openstack-watcher | 08:34 | |
| openstackgerrit | Santhosh Fernandes proposed openstack/watcher master: Add gnocchi support in basic_consolidation strategy https://review.openstack.org/440293 | 08:36 |
| hidekazu | sanfern: hi | 08:45 |
| sanfern | hi hidekazu | 08:46 |
| hidekazu | sanfern: i am reviewing https://review.openstack.org/#/c/440293/ | 08:47 |
| hidekazu | sanfern: in my enviroment, i got warning: ceilometer.dispatcher.gnocchi [-] metric compute.node.cpu.percent is not handled by Gnocchi | 08:47 |
| hidekazu | sanfern: i can not get metric compute.node.cpu.percent. were you able to get it? | 08:48 |
| sanfern | yes knew it, there is gnocchi patch is required | 08:49 |
| alexchadin | vincentfrancoise: ping | 08:50 |
| hidekazu | sanfern: could you give me the patch? :) | 08:51 |
| *** openstackgerrit has quit IRC | 09:03 | |
| *** sanfern has quit IRC | 09:04 | |
| *** sanfern has joined #openstack-watcher | 09:04 | |
| vincentfrancoise | alexchadin: hi | 09:08 |
| vincentfrancoise | alexchadin: will be back in 20mn | 09:08 |
| alexchadin | vincentfrancoise: ok | 09:08 |
| *** openstackgerrit has joined #openstack-watcher | 09:24 | |
| openstackgerrit | Merged openstack/watcher master: Remove old oslo.messaging transport aliases https://review.openstack.org/447369 | 09:24 |
| vincentfrancoise | alexchadin: i'm back btw | 09:34 |
| *** sanfern has quit IRC | 09:35 | |
| *** sanfern has joined #openstack-watcher | 09:37 | |
| sanfern | hidekazu, sorry, i was in meeting, their is no patch submitted yet, plz see https://openstack.nimeyo.com/108133/openstack-ceilometer-gnocchi-push-compute-metrics-gnocchi | 09:42 |
| hidekazu | sanfern: yes, i saw it. | 09:43 |
| sanfern | hidekazu, we will submit patch soon in gnocchi/ceilometer | 09:45 |
| hidekazu | sanfern: thanks | 09:47 |
| sanfern | hidekazu, yw :) | 09:48 |
| alexchadin | vincentfrancoise: take a look http://paste.openstack.org/show/603583/ | 09:48 |
| vincentfrancoise | alexchadin: nice! | 09:57 |
| *** sanfern has quit IRC | 09:58 | |
| vincentfrancoise | alexchadin: apart from the use case and some sentences that are hard to read, I'm ok with the content | 09:58 |
| vincentfrancoise | alexchadin: what is missing though is how you plan to use the Service.id to route the job | 09:59 |
| alexchadin | vincentfrancoise: which ones sentences are hard to read? | 10:00 |
| alexchadin | vincentfrancoise: I'm not sure that it is part of this BP | 10:00 |
| vincentfrancoise | alexchadin: the use case sentence is too long :p | 10:00 |
| alexchadin | vincentfrancoise: got it :) | 10:01 |
| *** sanfern has joined #openstack-watcher | 10:01 | |
| vincentfrancoise | alexchadin: also I would not focus on DE job routing since this is a generic tagging system that we plan to use for the DE jobs | 10:01 |
| vincentfrancoise | alexchadin: so instead of naming it "Supporting HA for continuous audits" I would instead call it "Supporting HA for background jobs in Watcher" | 10:02 |
| vincentfrancoise | alexchadin: and one of the use cases is to have it work for continuous audits (within the same or a separate BP) | 10:03 |
| alexchadin | vincentfrancoise: I agree. BTW, process of jobs routing should be described in another blueprint | 10:04 |
| vincentfrancoise | alexchadin: so I agree maybe the use of Service.id shouldn't be in the same BP | 10:04 |
| *** zhurong has quit IRC | 10:05 | |
| vincentfrancoise | alexchadin: 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 specifically | 10:05 |
| vincentfrancoise | alexchadin: and then a 2ndblueprint with the job routing for continuous audits called "Supporting HA for continuous audits" which would then tackle the routing process | 10:06 |
| vincentfrancoise | alexchadin: what do you think? | 10:06 |
| alexchadin | vincentfrancoise: by making this BP we will be able to continue our work on cron syntax, while routing should be described later | 10:06 |
| alexchadin | vincentfrancoise: +1 | 10:06 |
| vincentfrancoise | alexchadin: new blueprints again :p | 10:07 |
| vincentfrancoise | alexchadin: I still have to create the one from yesterday BTW | 10:07 |
| alexchadin | vincentfrancoise: stop them, there are multiply all the time! | 10:08 |
| vincentfrancoise | alexchadin: yeah, they are like rabbits :D | 10:08 |
| alexchadin | vincentfrancoise: btw, dividing this BP by two will not ruin workflow process | 10:09 |
| vincentfrancoise | alexchadin: which workflow process? | 10:09 |
| alexchadin | vincentfrancoise: I mean Watcher common workflow process, functional tests | 10:10 |
| openstackgerrit | Merged openstack/watcher master: set eager=True for actionplan.list https://review.openstack.org/446874 | 10:19 |
| *** pksingh has joined #openstack-watcher | 10:20 | |
| pksingh | vincentfrancoise: 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 PS | 10:22 |
| *** zhurong has joined #openstack-watcher | 10:24 | |
| vincentfrancoise | alexchadin: ok :) | 10:24 |
| alexchadin | pksingh: I will review it before tomorrow's meeting | 10:24 |
| pksingh | alexchadin: thanks :) | 10:24 |
| vincentfrancoise | pksingh: I remember having an indirect problem with your code | 10:25 |
| pksingh | vincentfrancoise: go ahead plz | 10:25 |
| vincentfrancoise | pksingh: but I didn't dig much | 10:25 |
| vincentfrancoise | pksingh: let me explain it to you | 10:25 |
| vincentfrancoise | pksingh: basically you add metadata="{'optimize': 'True'}" in all .xml files | 10:26 |
| pksingh | vincentfrancoise: yes | 10:26 |
| vincentfrancoise | pksingh: so 1st -> it is not json formatted | 10:26 |
| pksingh | vincentfrancoise: ok | 10:27 |
| vincentfrancoise | pksingh: 2nd, how do we ascertain that 2 structures are equal given that a json object is unordered by nature? | 10:28 |
| pksingh | vincentfrancoise: am i checking equality somewhere? | 10:29 |
| vincentfrancoise | pksingh: up until now, we are comparing the string output in watcher/tests/decision_engine/model/test_model.py | 10:30 |
| vincentfrancoise | pksingh: not you hence the "indirect problem" | 10:30 |
| pksingh | vincentfrancoise: ok let me check, | 10:31 |
| pksingh | vincentfrancoise: are you talking about https://github.com/openstack/watcher/blob/master/watcher/tests/decision_engine/model/test_model.py#L46 test case | 10:33 |
| vincentfrancoise | pksingh: yeah | 10:33 |
| pksingh | vincentfrancoise: ok, and why the structure is not json formatted? | 10:34 |
| vincentfrancoise | pksingh: I am wondering whether this test works if you have 2 or more keys in the metadata dict since they can end up being shuffled | 10:34 |
| pksingh | vincentfrancoise: double code and single codes should be interchanged? | 10:35 |
| pksingh | vincentfrancoise: i will test it by adding more keys | 10:35 |
| vincentfrancoise | pksingh: yeah kind of | 10:35 |
| vincentfrancoise | pksingh: 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 files | 10:36 |
| vincentfrancoise | pksingh: then we can bootstrap from then onwards | 10:37 |
| pksingh | vincentfrancoise: do we need JSON, i am using FlexibleDictField for the metadata? | 10:37 |
| pksingh | vincentfrancoise: and it simple converts it into dict, https://github.com/openstack/watcher/blob/master/watcher/objects/fields.py#L64 | 10:37 |
| openstackgerrit | Alexander Chadin proposed openstack/watcher-specs master: Add spec for Supporting HA for background jobs in Watcher https://review.openstack.org/447996 | 10:37 |
| *** alexchadin has quit IRC | 10:39 | |
| pksingh | vincentfrancoise: ^^ | 10:41 |
| vincentfrancoise | pksingh: IMHO, everything that is serialized should follow and serialization standard | 10:41 |
| pksingh | vincentfrancoise: OK great, thanks | 10:42 |
| vincentfrancoise | pksingh: so we should produce either some JSON or XML | 10:42 |
| pksingh | vincentfrancoise: ok | 10:43 |
| vincentfrancoise | pksingh: again, it's something I didn't pay attention in the code that added support for the metadata in the CDM | 10:44 |
| vincentfrancoise | pksingh: your code juste made me realise it | 10:44 |
| pksingh | vincentfrancoise: and we dont have JSON type in field, do we need to add it? | 10:45 |
| vincentfrancoise | pksingh: I guess so, not sure what's the best way to do this | 10:46 |
| pksingh | vincentfrancoise: something like https://github.com/openstack/zun/blob/master/zun/objects/fields.py#L75? | 10:46 |
| vincentfrancoise | pksingh: yeah, looks like a sound option | 10:48 |
| pksingh | vincentfrancoise: ok, will add this too | 10:48 |
| vincentfrancoise | pksingh: thanks | 10:49 |
| pksingh | vincentfrancoise: BTW you can put your comments on PS if you dont mind | 10:49 |
| pksingh | vincentfrancoise: i will work on it | 10:49 |
| vincentfrancoise | pksingh: ok | 10:50 |
| *** pksingh has quit IRC | 10:50 | |
| *** sanfern has quit IRC | 10:51 | |
| *** alexchadin has joined #openstack-watcher | 10:54 | |
| *** alexchadin has quit IRC | 11:01 | |
| *** alexchadin has joined #openstack-watcher | 11:01 | |
| *** alexchadin has quit IRC | 11:06 | |
| *** Yumeng has quit IRC | 11:16 | |
| openstackgerrit | Merged openstack/watcher-dashboard master: Remove the extra required=True https://review.openstack.org/447281 | 11:21 |
| openstackgerrit | YumengBao proposed openstack/watcher-specs master: Build baremetal data model in watcher https://review.openstack.org/448016 | 11:32 |
| *** sanfern has joined #openstack-watcher | 11:40 | |
| *** alexchadin has joined #openstack-watcher | 12:03 | |
| openstackgerrit | YumengBao proposed openstack/watcher-specs master: Add new actions "power on" and "power off" in Watcher https://review.openstack.org/435436 | 12:29 |
| *** zhurong has quit IRC | 12:39 | |
| openstackgerrit | Alexander Chadin proposed openstack/watcher-specs master: Add spec for Supporting HA for background jobs in Watcher https://review.openstack.org/447996 | 13:07 |
| *** alexchadin has quit IRC | 13:11 | |
| openstackgerrit | Santhosh Fernandes proposed openstack/watcher master: Add period input parameter to vm workload consolidation and outlet temp control strategy. https://review.openstack.org/444429 | 13:27 |
| *** alexchadin has joined #openstack-watcher | 14:31 | |
| openstackgerrit | lidaohan proposed openstack/python-watcherclient master: Remove log translations https://review.openstack.org/448155 | 15:24 |
| *** karthikpr has joined #openstack-watcher | 15:28 | |
| *** karthikpr has quit IRC | 15:37 | |
| openstackgerrit | Alexander Chadin proposed openstack/watcher-specs master: Add spec for Supporting HA for background jobs in Watcher https://review.openstack.org/447996 | 15:41 |
| openstackgerrit | lidaohan proposed openstack/python-watcherclient master: Remove log translations https://review.openstack.org/448155 | 16:20 |
| *** karthikpr has joined #openstack-watcher | 16:30 | |
| *** alexchadin has quit IRC | 16:33 | |
| *** Daviey_ is now known as Daviey | 16:49 | |
| *** wootehfoot has joined #openstack-watcher | 17:26 | |
| *** ss4 has joined #openstack-watcher | 17:33 | |
| *** wootehfoot has quit IRC | 17:33 | |
| *** vincentfrancoise has quit IRC | 17:43 | |
| *** ss4 has quit IRC | 17:59 | |
| *** karthikpr has quit IRC | 18:16 | |
| *** karthikpr has joined #openstack-watcher | 18:31 | |
| *** wootehfoot has joined #openstack-watcher | 19:29 | |
| *** wootehfoot has quit IRC | 20:36 | |
| *** pshedimb has joined #openstack-watcher | 20:49 | |
| *** wootehfoot has joined #openstack-watcher | 21:05 | |
| *** wootehfoot has quit IRC | 21:19 | |
| *** wootehfoot has joined #openstack-watcher | 21:23 | |
| *** karthikpr has quit IRC | 21:25 | |
| *** adisky_ has quit IRC | 21:39 | |
| *** karthikpr has joined #openstack-watcher | 22:03 | |
| *** harlowja has quit IRC | 23:06 | |
| *** hidekazu has left #openstack-watcher | 23:12 | |
| *** wootehfoot has quit IRC | 23:15 | |
| *** d34dh0r53 has quit IRC | 23:50 | |
| *** _d34dh0r53_ has joined #openstack-watcher | 23:55 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!