openstackgerrit | OpenStack Proposal Bot proposed openstack/watcher: Updated from global requirements https://review.openstack.org/358293 | 01:40 |
---|---|---|
*** thorst_ has joined #openstack-watcher | 01:52 | |
*** aswadr_ has joined #openstack-watcher | 03:17 | |
*** michaelgugino has quit IRC | 03:36 | |
*** thorst_ has quit IRC | 03:38 | |
*** thorst_ has joined #openstack-watcher | 03:39 | |
*** thorst_ has quit IRC | 03:47 | |
*** bauzas has left #openstack-watcher | 03:50 | |
*** thorst_ has joined #openstack-watcher | 04:45 | |
*** thorst_ has quit IRC | 04:52 | |
*** thorst_ has joined #openstack-watcher | 05:50 | |
*** thorst_ has quit IRC | 05:58 | |
openstackgerrit | Andreas Jaeger proposed openstack/watcher: Remove pot files https://review.openstack.org/359625 | 06:41 |
*** jinquan1 has joined #openstack-watcher | 06:42 | |
*** hanrong1 has joined #openstack-watcher | 06:43 | |
*** hanrong has quit IRC | 06:43 | |
*** jinquan has quit IRC | 06:45 | |
*** alexchadin has joined #openstack-watcher | 06:52 | |
*** thorst_ has joined #openstack-watcher | 06:56 | |
*** thorst_ has quit IRC | 07:03 | |
*** hanrong has joined #openstack-watcher | 07:03 | |
*** jinquan has joined #openstack-watcher | 07:03 | |
*** hanrong1 has quit IRC | 07:04 | |
*** jinquan1 has quit IRC | 07:05 | |
*** hanrong has quit IRC | 07:08 | |
*** jinquan1 has joined #openstack-watcher | 07:08 | |
*** hanrong has joined #openstack-watcher | 07:08 | |
*** jinquan has quit IRC | 07:08 | |
*** vincentfrancoise has joined #openstack-watcher | 07:37 | |
*** diga has joined #openstack-watcher | 07:52 | |
*** thorst_ has joined #openstack-watcher | 08:00 | |
*** alexchadin has quit IRC | 08:09 | |
*** thorst_ has quit IRC | 08:09 | |
*** alexchadin has joined #openstack-watcher | 08:10 | |
*** alexchadin has quit IRC | 08:11 | |
*** alexchadin has joined #openstack-watcher | 08:20 | |
*** gabor_antal has joined #openstack-watcher | 08:23 | |
*** jed56 has joined #openstack-watcher | 08:33 | |
openstackgerrit | Jinquan Ni proposed openstack/watcher-specs: Resize instance base on workload https://review.openstack.org/359690 | 08:40 |
*** Ananth_ has joined #openstack-watcher | 09:00 | |
tkaczynski | hi | 09:00 |
vincentfrancoise | tkaczynski: morning | 09:01 |
Ananth_ | hello | 09:01 |
tkaczynski | I hope that the scoring module will be merged soon :) so I was thinking that it would be good to update some documentation: add a page about new plugin, extend glossary etc. | 09:02 |
vincentfrancoise | oh yeah sure that would be great :) | 09:03 |
tkaczynski | I did review some stuff under the doc folder, but I guess there is some tool which generates the final HTML from these sources. where can I learn about that? | 09:03 |
tkaczynski | I have no idea about this process, so any help would be great :) | 09:03 |
vincentfrancoise | yes, the tox -e docs command generates the lot in doc/build/ | 09:03 |
vincentfrancoise | it will generate everything that is in the doc/source/ folder | 09:04 |
tkaczynski | sounds great :) what about sections like that: .. list-plugins:: watcher_strategies | 09:05 |
tkaczynski | how this is populated? based on stevedore plugins? | 09:06 |
*** thorst_ has joined #openstack-watcher | 09:06 | |
tkaczynski | I would like to have something like that for scoring_engine plugins | 09:07 |
vincentfrancoise | these are directive that list everything under an entrypoint namespace | 09:12 |
vincentfrancoise | so you can use this with something like "list-plugin:: watcher_scoring_engine" | 09:12 |
vincentfrancoise | so yeah based on the stevedore plugins | 09:13 |
tkaczynski | ok, I will try. thanks for help (as always :) ) | 09:13 |
*** thorst_ has quit IRC | 09:13 | |
vincentfrancoise | yeah if you don't understand something, please do ask :) | 09:14 |
tkaczynski | I should be good now, but won't hesitate if I need anything ;) | 09:14 |
*** Ananth_ has left #openstack-watcher | 09:15 | |
openstackgerrit | Merged openstack/watcher: Remove pot files https://review.openstack.org/359625 | 09:23 |
*** thorst_ has joined #openstack-watcher | 10:11 | |
openstackgerrit | Vincent Françoise proposed openstack/watcher: Notification and CDM partial update https://review.openstack.org/357802 | 10:16 |
openstackgerrit | Vincent Françoise proposed openstack/watcher: Doc on how to add notification endpoints https://review.openstack.org/358007 | 10:16 |
*** thorst_ has quit IRC | 10:18 | |
*** alexchadin has quit IRC | 10:37 | |
*** diga has quit IRC | 11:02 | |
*** thorst_ has joined #openstack-watcher | 11:16 | |
*** thorst_ has quit IRC | 11:23 | |
*** thorst_ has joined #openstack-watcher | 11:37 | |
*** alexchadin has joined #openstack-watcher | 11:38 | |
sballe_ | Morning | 11:39 |
*** alexchadin has quit IRC | 11:39 | |
vincentfrancoise | sballe_: hi | 11:40 |
*** dtardivel has joined #openstack-watcher | 11:47 | |
openstackgerrit | Jinquan Ni proposed openstack/watcher: Check unspecified parameters create audit https://review.openstack.org/359818 | 11:49 |
openstackgerrit | Jinquan Ni proposed openstack/watcher: Check unspecified parameters when create audit https://review.openstack.org/359818 | 11:49 |
jinquan1 | vincentfrancoise: hi? | 11:51 |
vincentfrancoise | jinquan1: hello | 11:51 |
jinquan1 | openstack/watcher-specs: Resize instance base on workload https://review.openstack.org/359690 | 11:51 |
jinquan1 | Check unspecified parameters when create audit https://review.openstack.org/359818 | 11:51 |
jinquan1 | vincentfrancoise: these two patchs need review , should i add into meeting agenda? | 11:52 |
vincentfrancoise | I'm looking at the jsonschema one right now | 11:52 |
vincentfrancoise | but yes add them to the agenda ;) | 11:53 |
jinquan1 | vincentfrancoise: ok, i add by myself, am i has permission? | 11:54 |
openstackgerrit | Merged openstack/watcher: Updated from global requirements https://review.openstack.org/358293 | 11:55 |
jinquan1 | vincentfrancoise:thank you! this one , i already test, it's work! :) | 11:55 |
vincentfrancoise | the problem with what you did is that it will only work in the post() method | 11:56 |
vincentfrancoise | I would prefer seeing something that is part of the JsonSchemaValidator | 11:56 |
vincentfrancoise | like for the default value that was done here: https://github.com/openstack/watcher/blob/master/watcher/common/utils.py#L136-L154 | 11:57 |
jinquan1 | vincentfrancoise: you mean move it into utils? | 11:59 |
jinquan1 | common.utils | 11:59 |
*** alexchadin has joined #openstack-watcher | 12:00 | |
vincentfrancoise | yes | 12:00 |
vincentfrancoise | IMHO all the validations should be inside the validator so that the validate() is responsible for checking everything | 12:01 |
jinquan1 | ok, i will done it , so i will just add another patch to agenda | 12:01 |
jinquan1 | vincentfrancoise: by the way , i can't join the meeting tonight , let you known, sorry for this | 12:02 |
vincentfrancoise | I pushed my comments as well | 12:02 |
vincentfrancoise | jinquan1: not problem | 12:03 |
jinquan1 | vincentfrancoise: great! thank you! | 12:03 |
jinquan1 | i will push a new patch as soon as possible | 12:03 |
vincentfrancoise | cool :) | 12:03 |
acabot | jinquan : I will update the agenda | 12:06 |
acabot | jinquan1 : thx | 12:06 |
jinquan1 | acabot: no thx :), | 12:06 |
acabot | jinquan1 : https://review.openstack.org/#/c/359690/ could you please retarget in Ocata ? | 12:14 |
*** diga has joined #openstack-watcher | 12:20 | |
*** alexchadin has quit IRC | 12:23 | |
*** alexchadin has joined #openstack-watcher | 12:24 | |
alexchadin | hi vincentfrancoise | 12:32 |
openstackgerrit | Vincent Françoise proposed openstack/watcher: Notification and CDM partial update https://review.openstack.org/357802 | 12:37 |
openstackgerrit | Vincent Françoise proposed openstack/watcher: Doc on how to add notification endpoints https://review.openstack.org/358007 | 12:37 |
openstackgerrit | Vincent Françoise proposed openstack/watcher: Fixed flaky tempest test https://review.openstack.org/359851 | 12:47 |
*** diga_ has joined #openstack-watcher | 12:48 | |
*** diga has quit IRC | 12:49 | |
*** diga_ has quit IRC | 13:04 | |
openstackgerrit | Vincent Françoise proposed openstack/watcher: Notification and CDM partial update https://review.openstack.org/357802 | 13:07 |
openstackgerrit | Vincent Françoise proposed openstack/watcher: Doc on how to add notification endpoints https://review.openstack.org/358007 | 13:07 |
openstackgerrit | Digambar proposed openstack/watcher-specs: automatic triggering audit https://review.openstack.org/342473 | 13:17 |
*** diga has joined #openstack-watcher | 13:19 | |
openstackgerrit | Digambar proposed openstack/watcher-specs: automatic triggering audit https://review.openstack.org/342473 | 13:25 |
*** openstackgerrit has quit IRC | 13:26 | |
*** openstackgerrit has joined #openstack-watcher | 13:26 | |
diga | Hi | 13:37 |
diga | Getting error while creating goal & strategy - http://paste.openstack.org/show/563002/ | 13:38 |
vincentfrancoise | diga: hi | 13:41 |
vincentfrancoise | you didn't specify the goal in your command (only the strategy) | 13:42 |
diga | vincentfrancoise: got the above error in command line | 13:42 |
vincentfrancoise | watcher audittemplate create my_first_audit_template bcb395c6-f44f-4ffc-a8ef-44ca782d5de0 --strategy d484b5fc-923b-4666-94b5-6ae8e8881764 | 13:42 |
diga | let me check | 13:43 |
diga | okay | 13:44 |
diga | thanks vincentfrancoise | 13:44 |
diga | but while creating goal, what am i missing | 13:44 |
diga | sorry - watcher strategy list --goal-uuid bcb395c6-f44f-4ffc-a8ef-44ca782d5de0 | 13:44 |
vincentfrancoise | it's not --goal-uuid anymore, use --goal instead | 13:45 |
diga | okay | 13:45 |
vincentfrancoise | because you can also specify the goal name instead of its uuid if you want | 13:45 |
diga | okay | 13:46 |
diga | we need to update readme | 13:46 |
diga | will update it | 13:46 |
tkaczynski | vincentfrancoise: suddenly the name field becomes a unique identifier then, this relates to the discussion we had the other day about id, uuid and name fields | 13:47 |
diga | acabot: submitted spec - https://review.openstack.org/#/c/342473/ | 13:47 |
vincentfrancoise | tkaczynski: it's a psudo unique field | 13:47 |
vincentfrancoise | it is unique business logic wise but not DB wise due to the soft delete feature | 13:48 |
sballe_ | diga: I will review | 13:48 |
diga | thanks sballe_ | 13:48 |
tkaczynski | exactly, I know, there is no way to enforce uniqueness | 13:48 |
acabot | diga : thx | 13:49 |
diga | we have meeting in 10 mins, right ? | 13:49 |
vincentfrancoise | tkaczynski: exactly | 13:49 |
tkaczynski | this is why I had to rework scoring engine | 13:49 |
diga | acabot: wc! | 13:49 |
tkaczynski | I've hit the wall with that, it is so much part of the platform | 13:49 |
acabot | diga : could you please resubmit for ocata ? | 13:49 |
diga | acabot: okay | 13:50 |
vincentfrancoise | yes, this is really annoying but it's now a little too widespread to change it without spending too much time | 13:50 |
acabot | diga : yes, weekly meeting in 10 minutes on #openstack-meeting-4 | 13:52 |
diga | yep | 13:53 |
*** tri2sing has joined #openstack-watcher | 13:53 | |
acabot | diga : are you still working on https://bugs.launchpad.net/watcher/+bug/1527163 ? | 13:53 |
openstack | Launchpad bug 1527163 in watcher "Add unit test for watcher/doc.py" [Low,Triaged] - Assigned to Digambar (digambarpatil15) | 13:53 |
diga | acabot: No, It was assigned to me | 13:54 |
diga | acabot: I need to discuss before start working on it | 13:54 |
acabot | diga : ok I will add it in the agenda | 13:55 |
diga | ok | 13:55 |
openstackgerrit | Digambar proposed openstack/watcher-specs: automatic triggering audit https://review.openstack.org/342473 | 13:58 |
diga | acabot: submitted to ocata | 13:59 |
acabot | thx | 13:59 |
*** tkaczynski_ has joined #openstack-watcher | 13:59 | |
openstackgerrit | Vincent Françoise proposed openstack/watcher: Notification and CDM partial update https://review.openstack.org/357802 | 14:04 |
openstackgerrit | Vincent Françoise proposed openstack/watcher: Doc on how to add notification endpoints https://review.openstack.org/358007 | 14:04 |
*** tkaczynski_ has quit IRC | 14:05 | |
*** alexchadin has quit IRC | 14:18 | |
*** alexchadin has joined #openstack-watcher | 14:21 | |
*** michaelgugino has joined #openstack-watcher | 14:35 | |
acabot | gzhai : are you there ? | 15:00 |
*** tri2sing has quit IRC | 15:05 | |
*** tri2sing has joined #openstack-watcher | 15:24 | |
openstackgerrit | Alexander Chadin proposed openstack/watcher: [WIP] Add Audit Scope Handler https://review.openstack.org/359963 | 15:29 |
*** diga has quit IRC | 15:36 | |
*** danpawlik has quit IRC | 16:15 | |
*** michaelgugino has quit IRC | 16:37 | |
openstackgerrit | Vincent Françoise proposed openstack/watcher: Refactored Watcher objects to use OVO https://review.openstack.org/359065 | 16:44 |
openstackgerrit | Vincent Françoise proposed openstack/watcher: [WIP] Added Versioned notification support for audits https://review.openstack.org/360008 | 16:44 |
*** vincentfrancoise has quit IRC | 16:50 | |
*** alexchadin has quit IRC | 17:27 | |
*** michaelgugino has joined #openstack-watcher | 17:41 | |
*** aswadr_ has quit IRC | 17:51 | |
*** tri2sing has quit IRC | 17:53 | |
*** tri2sing has joined #openstack-watcher | 17:53 | |
*** d34dh0r53 is now known as b3rnard0-b0n-h4r | 18:25 | |
*** b3rnard0-b0n-h4r is now known as d34dh0r53 | 19:14 | |
*** dtardivel has quit IRC | 19:27 | |
*** wootehfoot has joined #openstack-watcher | 20:16 | |
openstackgerrit | Daniel Pawlik proposed openstack/puppet-watcher: Change authtoken to class https://review.openstack.org/356792 | 20:55 |
*** thorst_ has quit IRC | 21:21 | |
*** michaelgugino has quit IRC | 21:38 | |
*** thorst_ has joined #openstack-watcher | 21:51 | |
*** thorst_ has quit IRC | 21:55 | |
*** wootehfoot has quit IRC | 22:44 | |
*** thorst has joined #openstack-watcher | 22:48 | |
*** zigo has quit IRC | 22:56 | |
*** zigo has joined #openstack-watcher | 22:57 | |
*** tri2sing has quit IRC | 23:32 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!