| *** thorst has quit IRC | 00:58 | |
| *** tkaczynski has quit IRC | 01:01 | |
| *** thorst has joined #openstack-watcher | 01:02 | |
| *** thorst has quit IRC | 01:08 | |
| *** tkaczynski has joined #openstack-watcher | 01:11 | |
| *** thorst has joined #openstack-watcher | 01:15 | |
| *** thorst has quit IRC | 01:20 | |
| *** thorst has joined #openstack-watcher | 01:26 | |
| *** thorst has quit IRC | 01:29 | |
| *** gzhai has quit IRC | 01:53 | |
| *** jwcroppe has joined #openstack-watcher | 02:04 | |
| *** yuanying_ has quit IRC | 02:39 | |
| *** thorst has joined #openstack-watcher | 02:50 | |
| *** thorst has quit IRC | 02:54 | |
| *** thorst has joined #openstack-watcher | 02:54 | |
| *** thorst has quit IRC | 02:55 | |
| *** thorst has joined #openstack-watcher | 03:05 | |
| *** thorst has quit IRC | 03:11 | |
| *** yuanying has joined #openstack-watcher | 04:02 | |
| *** yuanying has quit IRC | 04:12 | |
| *** yuanying has joined #openstack-watcher | 04:13 | |
| *** yuanying has quit IRC | 04:36 | |
| *** yuanying has joined #openstack-watcher | 04:42 | |
| *** jwcroppe has quit IRC | 05:09 | |
| *** thorst has joined #openstack-watcher | 05:11 | |
| *** thorst has quit IRC | 05:16 | |
| *** jwcroppe has joined #openstack-watcher | 05:58 | |
| *** jwcroppe has quit IRC | 06:04 | |
| *** yuanying has quit IRC | 06:16 | |
| *** alexchadin has joined #openstack-watcher | 06:25 | |
| *** jwcroppe has joined #openstack-watcher | 06:56 | |
| *** jwcroppe has quit IRC | 07:02 | |
| *** dtardivel has joined #openstack-watcher | 07:08 | |
| dtardivel | alexchadin: hi | 07:08 |
|---|---|---|
| alexchadin | hi dtardivel | 07:08 |
| dtardivel | I validated your patchset about audit scope on my devstack. | 07:10 |
| dtardivel | alexchadin: But I think something is missing, if you have a look on this old blueprint https://blueprints.launchpad.net/watcher/+spec/persistent-audit-parameters | 07:10 |
| dtardivel | alexchadin: scope should be also a new attribute of Audit Object. Agree ? | 07:11 |
| alexchadin | Agree | 07:11 |
| alexchadin | This patch doesn't save scope in audit? | 07:11 |
| dtardivel | which patch ? yours of the persisten-audit one ? | 07:12 |
| dtardivel | s/of/or | 07:12 |
| alexchadin | mine | 07:12 |
| dtardivel | when I played with the CLI yesterday, I didn't see scope in Audit . | 07:13 |
| dtardivel | https://www.irccloud.com/pastebin/l7C2OdqQ/ | 07:14 |
| alexchadin | I don't think it is hard to implement this feature:) will try to upload new PS asap | 07:14 |
| alexchadin | I will* | 07:15 |
| dtardivel | I set +2 on your previous patchsets. Please create a new one with same topic (and link to the previous ones) | 07:15 |
| alexchadin | +1 | 07:16 |
| dtardivel | alexchadin: thx | 07:16 |
| dtardivel | alexchadin: one more question: do you plan to add a patchset for Watcher UI plugin, in order to add scope in audit template creation form? | 07:26 |
| alexchadin | dtardivel: I'm not, but I can assign this PS to my colleague. | 07:28 |
| dtardivel | alexchadin: ok, thx | 07:28 |
| *** vincentfrancoise has joined #openstack-watcher | 07:31 | |
| openstackgerrit | Merged openstack/python-watcherclient: Updated from global requirements https://review.openstack.org/378898 | 07:33 |
| openstackgerrit | David TARDIVEL proposed openstack/watcher: Add service object to the watcher_db_schema https://review.openstack.org/377634 | 07:43 |
| alexchadin | dtardivel: ping | 07:45 |
| dtardivel | alexchadin: pong | 07:45 |
| alexchadin | take a look | 07:47 |
| alexchadin | http://pastebin.com/AkzPLtv7 | 07:47 |
| alexchadin | It already has scope in response | 07:47 |
| alexchadin | This is working for audit show command | 07:48 |
| alexchadin | audit list doesn't show scope | 07:49 |
| alexchadin | hm | 07:49 |
| alexchadin | dtardivel: scope field is not main field to show it through list command | 07:57 |
| alexchadin | dtardivel: You can show scope using watcher audit list --detail | 07:57 |
| *** yuanying has joined #openstack-watcher | 08:01 | |
| dtardivel | alexchadin: Ok, good new. I will verify again from my side later, I'm working on another patchset. | 08:19 |
| *** jwcroppe has joined #openstack-watcher | 08:42 | |
| dtardivel | alexchadin: about service list, I have again same issue with your last pachset. Here is my mysql service table: | 08:46 |
| dtardivel | https://www.irccloud.com/pastebin/ifBQzaPO/ | 08:46 |
| alexchadin | dtardivel: very strange. Can you give me your devstack conf file? I'd like to launch it on my side | 08:47 |
| dtardivel | alexchadin: which one ? | 08:47 |
| alexchadin | dtardivel: local.conf I think. Is there some conf files? | 08:49 |
| dtardivel | stackrc | 08:50 |
| dtardivel | give me 15mn, I'm trying to debug this strange behavior | 08:51 |
| *** jwcroppe has quit IRC | 08:51 | |
| alexchadin | dtardivel: I'll be off for 1 hour, will read IRC history when come back. | 09:01 |
| dtardivel | alexchadin: bug is here: | 09:02 |
| dtardivel | https://www.irccloud.com/pastebin/T5LKJbgT/ | 09:03 |
| *** alexchadin has quit IRC | 09:06 | |
| *** alexchadin has joined #openstack-watcher | 09:06 | |
| *** alexchadin has quit IRC | 09:11 | |
| *** thorst has joined #openstack-watcher | 09:13 | |
| *** thorst has quit IRC | 09:19 | |
| acabot | alexchadin : ping | 09:27 |
| *** alexchadin has joined #openstack-watcher | 09:47 | |
| *** diga has joined #openstack-watcher | 09:55 | |
| diga | vincentfrancoise: acabot : Hi | 09:55 |
| acabot | diga : hi | 09:56 |
| acabot | diga : I have reviewed your spec | 09:56 |
| diga | acabot: Yeah, I saw lot of changes :) | 09:57 |
| acabot | most of them are cosmetic | 10:01 |
| acabot | but need to be fixed before merging | 10:01 |
| openstackgerrit | Alexander Chadin proposed openstack/watcher: Add service supervisor https://review.openstack.org/371632 | 10:06 |
| diga | acabot: yes | 10:13 |
| diga | alexchadin: acabot: I have replied to your comments, please go through it once | 10:14 |
| diga | acabot: so that I can update the new spec accordingly | 10:14 |
| alexchadin | diga: yeah, I see | 10:15 |
| sballe_ | Morning | 10:16 |
| alexchadin | hello sballe_ | 10:18 |
| alexchadin | diga: We need to cancel current action plan if new action plan is created, right? | 10:19 |
| openstackgerrit | Vincent Françoise proposed openstack/watcher: Added audit & strategy ObjectField for ActionPlan https://review.openstack.org/382257 | 10:20 |
| openstackgerrit | Vincent Françoise proposed openstack/watcher: Added action_plan ObjectField for Action https://review.openstack.org/382258 | 10:20 |
| diga | alexchadin: yes | 10:21 |
| diga | alexchadin: correct | 10:22 |
| diga | alexchadin: for first implementation, we should go with one action_plan at a time, later on we can maintain queue & all | 10:22 |
| alexchadin | diga: what if the current action plan is in running state? | 10:23 |
| diga | alexchadin: for now, as per the acabot suggestion we should cancel it | 10:27 |
| diga | if new action plan is proposed | 10:28 |
| *** jwcroppe has joined #openstack-watcher | 10:32 | |
| *** alexchadin has quit IRC | 10:34 | |
| *** alexchadin has joined #openstack-watcher | 10:38 | |
| alexchadin | diga: sorry for disconnecting | 10:38 |
| alexchadin | diga: How do you plan to cancel old action plan? | 10:38 |
| *** jwcroppe has quit IRC | 10:38 | |
| diga | alexchadin: we are not cancelling running action plan, we are not allowing new proposed actionplan while running current action plan | 10:39 |
| alexchadin | diga: ok, once old action plan is completed, we need to apply new one? | 10:41 |
| diga | alexchadin: yes | 10:41 |
| alexchadin | diga: so, we need queue here? | 10:46 |
| *** alexchadin has quit IRC | 10:46 | |
| diga | alexchadin: now we dont, but later on, we need it | 10:46 |
| *** alexchadin has joined #openstack-watcher | 10:46 | |
| *** alexchadin has quit IRC | 10:51 | |
| *** alexchadin has joined #openstack-watcher | 10:51 | |
| *** alexchadin has quit IRC | 10:52 | |
| *** alexchadin has joined #openstack-watcher | 10:52 | |
| alexchadin | diga: how will we know in real time if old action plan is completed? | 11:03 |
| diga | alexchadin: we only run one action plan at time | 11:03 |
| diga | old & new concept will never happen now, | 11:03 |
| diga | alexchadin: maintaining a queue, will be second phase of this implementation | 11:04 |
| alexchadin | diga: why don't include it in the first phase? | 11:06 |
| alexchadin | diga: Since it is very important feature to implement cancelling of the current working action plan and launch new one. | 11:07 |
| *** alexchadin has quit IRC | 11:29 | |
| *** alexchadin has joined #openstack-watcher | 11:29 | |
| *** thorst has joined #openstack-watcher | 11:51 | |
| diga | alexchadin: we are not cancelling the current action plan, we are not allowing to run new action plan if there is already one running | 12:04 |
| *** alexchadin has quit IRC | 12:04 | |
| diga | this is the case we are handling now | 12:05 |
| acabot | alexchadin : just to clarify, I dont think we should stop a running action plan as we are not able to roll it back today | 12:22 |
| acabot | alexchadin : I suggested in diga's spec to cancel the "new generated action-plan" in case there is a running action plan | 12:22 |
| jed56 | acabot: we can rollback an ActionPlan this is not the best solution | 12:24 |
| jed56 | but | 12:24 |
| jed56 | you can take a look to https://github.com/openstack/watcher/blob/master/watcher/applier/workflow_engine/default.py#L146 | 12:25 |
| acabot | diga : I'm fine with your comments, just waiting your PS ;-) | 12:29 |
| diga | acabot: okay | 12:29 |
| diga | acabot: I will update the spec | 12:30 |
| acabot | diga : thx | 12:32 |
| diga | acabot: wc! thnks for your guidance! | 12:33 |
| openstackgerrit | Vincent Françoise proposed openstack/watcher: Added audit & strategy ObjectField for ActionPlan https://review.openstack.org/382257 | 12:49 |
| openstackgerrit | Vincent Françoise proposed openstack/watcher: Added goal & strategy ObjectField for Audit https://review.openstack.org/382256 | 12:49 |
| openstackgerrit | Vincent Françoise proposed openstack/watcher: Added action_plan ObjectField for Action https://review.openstack.org/382258 | 12:49 |
| openstackgerrit | Vincent Françoise proposed openstack/watcher: Refactored Watcher objects to use OVO https://review.openstack.org/359065 | 12:49 |
| openstackgerrit | Vincent Françoise proposed openstack/watcher: Added 'goal' ObjectField for Strategy object https://review.openstack.org/382254 | 12:49 |
| openstackgerrit | Vincent Françoise proposed openstack/watcher: Added goal+strategy ObjectField for AuditTemplate https://review.openstack.org/382255 | 12:49 |
| diga | acabot: I am going to home now, will be driving, I may get late but will join the call around 8PM | 12:55 |
| acabot | diga : cool thx | 12:55 |
| openstackgerrit | Antoine Cabot proposed openstack/watcher-specs: Add team priorities for Ocata https://review.openstack.org/372528 | 12:57 |
| *** diga has quit IRC | 12:59 | |
| *** gzhai has joined #openstack-watcher | 13:00 | |
| openstackgerrit | Vincent Françoise proposed openstack/watcher: Moved Watcher doc plugin outside main package https://review.openstack.org/382411 | 13:27 |
| *** hvprash has quit IRC | 13:34 | |
| openstackgerrit | Vincent Françoise proposed openstack/watcher: Added missing test on GMR plugin https://review.openstack.org/382419 | 13:45 |
| *** licanwei has joined #openstack-watcher | 13:47 | |
| *** hvprash has joined #openstack-watcher | 13:56 | |
| *** alexchadin has joined #openstack-watcher | 13:57 | |
| *** alexchadin has quit IRC | 13:58 | |
| *** alexchadin has joined #openstack-watcher | 13:58 | |
| *** alexchadin has quit IRC | 13:59 | |
| *** alexchadin has joined #openstack-watcher | 14:00 | |
| alexchadin | #openstack-meeting-4 | 14:00 |
| *** alexchadin has quit IRC | 14:01 | |
| *** dtardivel has quit IRC | 14:38 | |
| *** jed56 has quit IRC | 14:38 | |
| *** gabor_antal has quit IRC | 14:38 | |
| *** gabor_antal has joined #openstack-watcher | 14:39 | |
| *** alexchad_ has joined #openstack-watcher | 14:40 | |
| *** dtardivel has joined #openstack-watcher | 14:43 | |
| *** jed56 has joined #openstack-watcher | 14:43 | |
| alexchad_ | dtardivel: new PS is available https://review.openstack.org/#/c/371632/ | 14:44 |
| *** jed561 has joined #openstack-watcher | 14:45 | |
| *** michaelgugino has joined #openstack-watcher | 14:49 | |
| dtardivel | alexchad_: yep, I've just seen it. Thx, I will review it asap | 14:49 |
| *** diga has joined #openstack-watcher | 15:00 | |
| diga | acabot: Hi | 15:00 |
| *** licanwei has quit IRC | 15:00 | |
| acabot | diga : yes | 15:00 |
| diga | got late, missed meeting | 15:00 |
| acabot | diga : no pb http://eavesdrop.openstack.org/meetings/watcher/2016/watcher.2016-10-05-14.01.log.html | 15:00 |
| diga | acabot: pushing auto triggerring patch in sometime | 15:00 |
| acabot | ok | 15:01 |
| diga | lets close it today only | 15:01 |
| *** gzhai has quit IRC | 15:07 | |
| *** hvprash has quit IRC | 15:08 | |
| openstackgerrit | David TARDIVEL proposed openstack/watcher: Add strategy template doc https://review.openstack.org/382494 | 15:12 |
| *** hvprash has joined #openstack-watcher | 15:13 | |
| *** hvprash_ has joined #openstack-watcher | 15:17 | |
| openstackgerrit | David TARDIVEL proposed openstack/python-watcherclient: Add parameters in Audit creation attributes list https://review.openstack.org/382505 | 15:19 |
| openstackgerrit | David TARDIVEL proposed openstack/watcher: Add strategy template doc https://review.openstack.org/382494 | 15:22 |
| openstackgerrit | Vincent Françoise proposed openstack/watcher: Added Tempest API tests for /scoring_engines https://review.openstack.org/371558 | 15:23 |
| *** diga_ has joined #openstack-watcher | 15:42 | |
| openstackgerrit | Merged openstack/watcher: Watcher utils cleanup https://review.openstack.org/375342 | 15:50 |
| *** hvprash has quit IRC | 15:53 | |
| *** hvprash_ has quit IRC | 16:01 | |
| *** hvprash has joined #openstack-watcher | 16:01 | |
| diga_ | acabot: Hi | 16:08 |
| diga_ | acabot: Pushed New PS as per your comments, please review https://review.openstack.org/#/c/342473/ | 16:09 |
| *** diga has quit IRC | 16:12 | |
| *** michaelgugino has quit IRC | 16:23 | |
| *** diga_ has quit IRC | 16:25 | |
| *** hvprash has quit IRC | 16:54 | |
| *** jed56 has quit IRC | 17:05 | |
| *** alexchad_ has quit IRC | 17:12 | |
| *** hvprash has joined #openstack-watcher | 17:19 | |
| *** thorst is now known as thorst_afk | 18:17 | |
| *** dtardivel has quit IRC | 18:27 | |
| *** thorst_afk is now known as thorst_ | 18:45 | |
| *** michaelgugino has joined #openstack-watcher | 20:00 | |
| *** thorst_ has quit IRC | 21:26 | |
| *** thorst_ has joined #openstack-watcher | 21:26 | |
| *** thorst_ has quit IRC | 21:31 | |
| *** gabor_antal_ has joined #openstack-watcher | 22:09 | |
| *** gabor_antal has quit IRC | 22:09 | |
| *** gabor_antal_ has quit IRC | 22:19 | |
| *** gabor_antal has joined #openstack-watcher | 22:21 | |
| *** michaelgugino has quit IRC | 22:39 | |
| *** logan- has quit IRC | 23:28 | |
| *** logan- has joined #openstack-watcher | 23:37 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!