*** thorst has joined #openstack-watcher | 00:17 | |
*** thorst has quit IRC | 00:21 | |
*** thorst has joined #openstack-watcher | 00:43 | |
*** thorst has quit IRC | 00:47 | |
openstackgerrit | suzhengwei proposed openstack/watcher master: Add and identify excluded instances in compute CDM https://review.openstack.org/504303 | 02:06 |
---|---|---|
openstackgerrit | suzhengwei proposed openstack/watcher-specs master: compute CDM include all instances https://review.openstack.org/499119 | 02:12 |
*** hidekazu has joined #openstack-watcher | 02:12 | |
*** hidekazu has left #openstack-watcher | 03:00 | |
openstackgerrit | suzhengwei proposed openstack/watcher master: Add and identify excluded instances in compute CDM https://review.openstack.org/504303 | 03:17 |
*** suzhengwei has joined #openstack-watcher | 03:22 | |
*** thorst has joined #openstack-watcher | 03:34 | |
*** thorst has quit IRC | 03:40 | |
*** thorst has joined #openstack-watcher | 03:40 | |
*** thorst has quit IRC | 03:45 | |
openstackgerrit | aditi sharma proposed openstack/watcher-specs master: Multiple global efficacy indicator https://review.openstack.org/504322 | 03:59 |
*** suzhengwei has quit IRC | 04:13 | |
*** suzhengwei has joined #openstack-watcher | 04:28 | |
*** suzhengwei has quit IRC | 04:48 | |
*** suzhengwei has joined #openstack-watcher | 04:52 | |
*** suzhengwei has quit IRC | 05:39 | |
*** suzhengwei_ has joined #openstack-watcher | 05:39 | |
suzhengwei_ | adisky:ping | 06:45 |
adisky | suzhengwei_: hii | 06:45 |
suzhengwei_ | do you know aditi sharma? | 06:46 |
adisky | suzhengwei_: i am aditi sharma | 06:46 |
suzhengwei_ | nice. | 06:47 |
adisky | suzhengwei_: what happened?? | 06:48 |
suzhengwei_ | I want to discuss something with you | 06:48 |
suzhengwei_ | https://review.openstack.org/#/c/504303/ | 06:49 |
adisky | suzhengwei_: tell | 06:49 |
suzhengwei_ | the exclude instance I add 'audit' field in the instance resource. I notice that you add "optimize" filed in metadata. If are they the same means? | 06:50 |
adisky | i added | 06:51 |
adisky | where?? | 06:51 |
suzhengwei_ | If so, If i can use "optimize" filed to distinguish exclude instances in the scope? | 06:51 |
adisky | suzhengwei_: are you talking about this https://review.openstack.org/#/c/443070/ | 06:52 |
adisky | ?? | 06:52 |
suzhengwei_ | Change-Id: Iaf7b63e09534e4a67406e7f092242558b78c0bde | 06:53 |
suzhengwei_ | Partially-Implements: BP audit-tag-vm-metadata | 06:53 |
adisky | yea | 06:53 |
adisky | this is not done by me, it is done by pksingh | 06:54 |
suzhengwei_ | right. so is it ok to use "optimize" filed in the metadata to distinguish exclude instances in the scope? | 06:56 |
adisky | suzhengwei_: wait for sometime, i am studying both specs, then will get back to you | 06:57 |
suzhengwei_ | ok, thx. | 06:57 |
adisky | suzhengwei_: yes, i think you can use optimize field | 07:01 |
suzhengwei_ | I just study the commit of Change-Id: Iaf7b63e09534e4a67406e7f092242558b78c0bde. It seems not complete. | 07:05 |
adisky | ?? | 07:24 |
adisky | please give the link | 07:24 |
*** alexchadin has joined #openstack-watcher | 07:42 | |
adisky | alexchadin: hi | 07:42 |
alexchadin | adisky: hi | 07:42 |
adisky | alexchadin: i see 2-3 patches on python watcherclient | 07:43 |
adisky | on stable/pike branch unmerged | 07:44 |
adisky | are they intentionally not merged or lack of reviews?? | 07:44 |
alexchadin | adisky: yeap, two of them are from me | 07:44 |
alexchadin | lack of reviews | 07:44 |
adisky | alexchadin: ok, so can we merge them now?? | 07:45 |
alexchadin | adisky: don't touch add name for audit, it's part of blueprint | 07:45 |
alexchadin | adisky: I think so | 07:45 |
adisky | alexchadin: they are on stable/pike?? | 07:46 |
suzhengwei_ | alexchadin: Hi | 07:50 |
alexchadin | adisky: you can merge this one: https://review.openstack.org/#/c/497126/ | 07:51 |
alexchadin | adisky: and this: https://review.openstack.org/#/c/497118/ | 07:51 |
alexchadin | suzhengwei_: hi | 07:51 |
suzhengwei_ | what is the milestone for add name for audit? | 07:51 |
adisky | alexchadin: ok, i did | 07:52 |
alexchadin | suzhengwei_: queens-1 since you've already done a lot of work and we can merge it soon | 07:53 |
suzhengwei_ | thx, i will fix the commits asap. | 07:53 |
alexchadin | suzhengwei_: I mean, we have one month to fix it :) | 07:54 |
suzhengwei_ | ok. | 07:55 |
*** alexchadin has quit IRC | 08:03 | |
*** alexchadin has joined #openstack-watcher | 08:12 | |
*** efoley has joined #openstack-watcher | 08:15 | |
alexchadin | suzhengwei_: check out my review for audit naming | 08:56 |
*** brunograz1 has joined #openstack-watcher | 09:00 | |
*** brunograz has quit IRC | 09:02 | |
suzhengwei_ | alexchadin:hi, I'm confused with your meaning. | 09:27 |
alexchadin | suzhengwei_: what's wrong? | 09:28 |
suzhengwei_ | Intuitively, 'audit_uuid' represents uuid of one audit. But now I hope we can retrive an audit by uuid or name. so change the parameter to audit, instead of 'audit_uuid'. | 09:32 |
alexchadin | suzhengwei_: so, you want to change name of parameter, right? | 09:35 |
suzhengwei_ | yes | 09:36 |
alexchadin | http://eavesdrop.openstack.org/irclogs/%23openstack-watcher/%23openstack-watcher.2017-09-14.log.html#t2017-09-14T09:27:39 | 09:38 |
*** brunograz1 has quit IRC | 09:38 | |
alexchadin | suzhengwei_: if you want to use 'audit' instead of 'audit_uuid' – just replace audit_uuid with audit in my review message | 09:39 |
*** brunograz has joined #openstack-watcher | 09:42 | |
suzhengwei_ | ok | 09:43 |
openstackgerrit | suzhengwei proposed openstack/watcher-specs master: add name for audit https://review.openstack.org/474865 | 09:46 |
*** alexchadin has quit IRC | 09:53 | |
*** suzhengwei_ has quit IRC | 10:10 | |
*** thorst has joined #openstack-watcher | 10:50 | |
*** thorst has quit IRC | 11:09 | |
*** alexchadin has joined #openstack-watcher | 12:18 | |
openstackgerrit | Alexander Chadin proposed openstack/watcher master: Fix Watcher DB schema creation https://review.openstack.org/504401 | 12:32 |
*** alexchadin has quit IRC | 12:48 | |
*** alexchadin has joined #openstack-watcher | 12:58 | |
*** efoley_ has joined #openstack-watcher | 13:34 | |
*** efoley has quit IRC | 13:34 | |
*** alexchadin has quit IRC | 14:02 | |
*** efoley_ has quit IRC | 14:07 | |
*** efoley_ has joined #openstack-watcher | 14:08 | |
*** thorst has joined #openstack-watcher | 14:19 | |
*** thorst has quit IRC | 14:21 | |
*** thorst has joined #openstack-watcher | 14:43 | |
*** thorst has quit IRC | 14:55 | |
*** thorst has joined #openstack-watcher | 14:55 | |
*** thorst has quit IRC | 15:00 | |
*** efoley_ has quit IRC | 17:10 | |
*** thorst has joined #openstack-watcher | 17:48 | |
*** thorst has quit IRC | 17:52 | |
*** thorst has joined #openstack-watcher | 17:53 | |
*** thorst has quit IRC | 17:56 | |
*** thorst has joined #openstack-watcher | 18:03 | |
*** thorst has quit IRC | 18:07 | |
*** thorst has joined #openstack-watcher | 18:13 | |
*** thorst has quit IRC | 18:14 | |
*** thorst has joined #openstack-watcher | 20:21 | |
*** thorst has quit IRC | 20:26 | |
*** thorst has joined #openstack-watcher | 20:38 | |
*** thorst has quit IRC | 20:43 | |
openstackgerrit | Merged openstack/puppet-watcher master: Deprecate revocation_cache_time option https://review.openstack.org/503940 | 21:24 |
*** efoley has joined #openstack-watcher | 21:53 | |
*** efoley has quit IRC | 23:02 | |
*** thorst has joined #openstack-watcher | 23:31 | |
*** thorst has quit IRC | 23:33 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!