*** jwcroppe has joined #openstack-watcher | 00:05 | |
*** jwcroppe has quit IRC | 00:33 | |
*** jwcroppe has joined #openstack-watcher | 00:33 | |
*** thorst has joined #openstack-watcher | 00:55 | |
*** thorst has quit IRC | 01:04 | |
*** thorst has joined #openstack-watcher | 02:03 | |
*** thorst has quit IRC | 02:10 | |
*** yuanying has quit IRC | 02:52 | |
*** yuanying has joined #openstack-watcher | 02:53 | |
*** thorst has joined #openstack-watcher | 03:06 | |
*** thorst has quit IRC | 03:14 | |
*** yuanying has quit IRC | 03:44 | |
*** yuanying has joined #openstack-watcher | 03:51 | |
*** thorst has joined #openstack-watcher | 04:12 | |
*** thorst has quit IRC | 04:19 | |
*** yuanying has quit IRC | 04:59 | |
*** yuanying_ has joined #openstack-watcher | 04:59 | |
*** thorst has joined #openstack-watcher | 05:18 | |
*** thorst has quit IRC | 05:24 | |
*** dtardivel has joined #openstack-watcher | 06:17 | |
openstackgerrit | Tuan Luong-Anh proposed openstack/watcher-specs: Fix typo: remove redundant 'the' https://review.openstack.org/380934 | 06:21 |
---|---|---|
*** thorst has joined #openstack-watcher | 06:22 | |
*** thorst has quit IRC | 06:29 | |
openstackgerrit | Merged openstack/watcher: Fix capital letter in doc https://review.openstack.org/376277 | 06:31 |
*** yuanying_ has quit IRC | 06:33 | |
openstackgerrit | Merged openstack/watcher: Docstrings should not start with a space https://review.openstack.org/377257 | 06:33 |
*** alexchadin has joined #openstack-watcher | 07:06 | |
*** thorst has joined #openstack-watcher | 07:27 | |
*** vincentfrancoise has joined #openstack-watcher | 07:29 | |
*** thorst has quit IRC | 07:34 | |
*** danpawlik has joined #openstack-watcher | 07:38 | |
*** zigo has quit IRC | 07:39 | |
*** zigo has joined #openstack-watcher | 07:42 | |
*** zigo is now known as Guest24756 | 07:42 | |
alexchadin | hi dtardivel | 07:47 |
alexchadin | dtardivel: it is very strange error you have mentioned on supervisor thread | 07:48 |
alexchadin | dtardivel: I have installed watcher on dedicated controller node from the beginning and it works great, it doesn't create multiply services. | 07:49 |
alexchadin | dtardivel: I have unique constraint in addition | 07:49 |
alexchadin | it is set on deleted, host and name of service | 07:50 |
alexchadin | I have just installed* ^^& | 07:51 |
sballe_ | Morning | 07:59 |
sballe_ | dtardivel: are you online | 07:59 |
dtardivel | hi | 07:59 |
*** thorst has joined #openstack-watcher | 08:32 | |
*** thorst has quit IRC | 08:40 | |
*** Guest24756 is now known as zigo | 08:54 | |
*** yuanying has joined #openstack-watcher | 08:59 | |
*** thorst has joined #openstack-watcher | 09:37 | |
*** thorst has quit IRC | 09:44 | |
openstackgerrit | Alexander Chadin proposed openstack/watcher: Add Audit Scope Handler https://review.openstack.org/359963 | 09:58 |
*** alexchadin has quit IRC | 09:59 | |
openstackgerrit | Nishant Kumar proposed openstack/watcher: Stop adding ServiceAvailable group option https://review.openstack.org/381024 | 10:10 |
*** diga_ has joined #openstack-watcher | 10:41 | |
*** thorst has joined #openstack-watcher | 10:42 | |
diga_ | acabot: Hi | 10:42 |
diga_ | acabot: need one help from you! | 10:44 |
*** alexchadin has joined #openstack-watcher | 10:47 | |
alexchadin | hi vincentfrancoise and dtardivel | 10:47 |
alexchadin | new PS of audit scope commit is ready to review | 10:48 |
*** openstackgerrit has quit IRC | 10:48 | |
*** openstackgerrit has joined #openstack-watcher | 10:48 | |
alexchadin | dtardivel: I've taken into account your comments, thanks for your review. | 10:48 |
*** thorst has quit IRC | 10:49 | |
*** alexchadin has quit IRC | 11:13 | |
*** alexchadin has joined #openstack-watcher | 11:46 | |
dtardivel | alexchadin: hi | 11:47 |
*** thorst has joined #openstack-watcher | 11:47 | |
*** alexchadin has quit IRC | 11:50 | |
dtardivel | alexchadin: about issue on 'watcher service list', I tested it into my devstack. I think duplicated entries were created when I started several times the watcher. I would try to reproduce this behavior this afternoon to be more precise | 11:50 |
*** gabor_antal has joined #openstack-watcher | 11:56 | |
dtardivel | alexchadin: here more traces: | 12:01 |
dtardivel | https://www.irccloud.com/pastebin/9VgItcP9/ | 12:02 |
*** Kevin_Zheng has joined #openstack-watcher | 12:20 | |
*** gzhai has joined #openstack-watcher | 13:08 | |
gzhai | dtardivel:hi | 13:08 |
openstackgerrit | Alexander Chadin proposed openstack/watcher: Add service supervisor https://review.openstack.org/371632 | 13:09 |
*** alexchadin has joined #openstack-watcher | 13:11 | |
alexchadin | dtardivel: ping | 13:13 |
acabot | diga_ : hi | 13:17 |
diga_ | acabot: need one help from you! | 13:17 |
diga_ | acabot: I need your help on Applier part of implementation | 13:18 |
diga_ | for automatic triggering spec | 13:18 |
acabot | diga_ : did you update your spec ? | 13:19 |
acabot | diga_ : how can I help you | 13:19 |
acabot | ? | 13:19 |
diga_ | acabot: only main changes are remaining are related to Applier, rest are ready, will push in sometime | 13:19 |
diga_ | acabot: currently applier takes many action plan & run simultaneosly correct ? | 13:21 |
diga_ | as part of auto_triggerring, once we submit action_plan, only one action plan for that audit should get triggered | 13:21 |
acabot | diga_ : no | 13:22 |
acabot | diga_ : the applier handle only one action plan at a time | 13:22 |
diga_ | acabot: okay | 13:22 |
diga_ | acabot: then in this case, what changes are required at applier side, I dont understand what vincentfrancoise is talking about | 13:23 |
diga_ | acabot: https://review.openstack.org/#/c/342473/11/specs/ocata/approved/automatic-triggering-audit.rst | 13:24 |
acabot | diga_ : when you auto-trigger an audit, you have to check that there is no action plan currently running | 13:24 |
vincentfrancoise | diga_: hi | 13:24 |
diga_ | acabot: okay, if its running then what will be the next steps (wait) | 13:24 |
acabot | yes you will have to wait until the action plan is complete | 13:25 |
diga_ | vincentfrancoise: its about your comment on "How do plan on making sure there only one action plan that is executed at a time?" | 13:25 |
diga_ | okay | 13:25 |
acabot | if you don't your audit will be unreliable | 13:26 |
vincentfrancoise | diga_: take a look at this config parameter https://github.com/openstack/watcher/blob/master/watcher/applier/manager.py#L29-L31 | 13:26 |
diga_ | acabot: then acabot in this case, we need to check continuously to previous action plan to complete & make sure nobody's action plan should start before this | 13:26 |
vincentfrancoise | by default, acabot is right when saying that we only run one action plan at a time | 13:27 |
diga_ | acabot: in this case, we need to maintain queue | 13:27 |
diga_ | vincentfrancoise: got that answer, | 13:27 |
vincentfrancoise | and this is was not a big deal for us since action plans are triggered by hand | 13:27 |
diga_ | vincentfrancoise: okay | 13:28 |
vincentfrancoise | diga_: but with your blueprint the likelihood of having 2 workers processing 2 different action plans has risen | 13:28 |
diga_ | vincentfrancoise: how ? I think we only get only one action plan at a time, once completes then another one will start | 13:30 |
vincentfrancoise | diga_: since we do not have a way of guaranteeing the veracity of an action plan given that another one is running in parrallel, we need to, at least, limit their execution | 13:30 |
diga_ | vincentfrancoise: okay | 13:30 |
vincentfrancoise | diga_: if you have to peridioc audits planned at the same time, the likelihood of this situation would be close to a 100% | 13:31 |
*** alexchad_ has joined #openstack-watcher | 13:31 | |
vincentfrancoise | 2 periodic* | 13:31 |
diga_ | okay | 13:31 |
diga_ | vincentfrancoise: "limiting the execution" you mean how many times it should execute in a day/hour ?? | 13:33 |
diga_ | vincentfrancoise: I talk about "same action plan " | 13:34 |
*** alexchadin has quit IRC | 13:34 | |
vincentfrancoise | as in writing something that would, for now, make sure that 1 and only 1 action plan gets executed at a time across all workers | 13:36 |
diga_ | okay | 13:36 |
vincentfrancoise | diga_: good | 13:36 |
vincentfrancoise | :) | 13:36 |
diga_ | vincentfrancoise: thank you! | 13:37 |
diga_ | vincentfrancoise: going to home now, will ping you if something to discuss | 13:38 |
*** alexchad_ has quit IRC | 13:41 | |
*** diga_ has quit IRC | 13:43 | |
*** michaelgugino has joined #openstack-watcher | 14:14 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/watcher: Updated from global requirements https://review.openstack.org/377587 | 14:17 |
*** jed56 has joined #openstack-watcher | 14:21 | |
*** Kevin_Zheng has quit IRC | 14:26 | |
*** michaelgugino has quit IRC | 14:28 | |
*** michaelgugino has joined #openstack-watcher | 14:30 | |
*** gzhai has quit IRC | 14:31 | |
*** hvprash_ has joined #openstack-watcher | 14:52 | |
*** hvprash_ has quit IRC | 14:57 | |
*** hvprash_ has joined #openstack-watcher | 14:59 | |
*** danpawlik has quit IRC | 15:21 | |
*** hvprash_ has quit IRC | 15:34 | |
*** hvprash_ has joined #openstack-watcher | 15:35 | |
*** hvprash_ has quit IRC | 15:39 | |
*** hvprash_ has joined #openstack-watcher | 15:41 | |
*** edleafe- has joined #openstack-watcher | 15:53 | |
*** edleafe has quit IRC | 15:56 | |
openstackgerrit | haris proposed openstack/watcher: HasLength() rewritten to assertEqual() https://review.openstack.org/381223 | 16:22 |
*** hvprash_ has quit IRC | 16:36 | |
*** hvprash_ has joined #openstack-watcher | 16:37 | |
*** hvprash__ has joined #openstack-watcher | 16:45 | |
*** hvprash_ has quit IRC | 16:45 | |
*** gfhellma has joined #openstack-watcher | 16:45 | |
*** hvprash__ has quit IRC | 16:50 | |
*** hvprash_ has joined #openstack-watcher | 16:52 | |
*** edleafe- is now known as edleafe | 16:56 | |
*** hvprash_ has quit IRC | 17:00 | |
*** hvprash_ has joined #openstack-watcher | 17:04 | |
*** hvprash_ has quit IRC | 17:50 | |
*** hvprash_ has joined #openstack-watcher | 17:52 | |
*** hvprash_ has quit IRC | 17:55 | |
*** hvprash_ has joined #openstack-watcher | 17:56 | |
openstackgerrit | haris proposed openstack/watcher: HasLength() rewritten to assertEqual() https://review.openstack.org/381223 | 17:59 |
*** harlowja has quit IRC | 18:38 | |
*** harlowja has joined #openstack-watcher | 18:39 | |
*** dtardivel has quit IRC | 18:57 | |
*** jed56 has quit IRC | 19:15 | |
*** jwcroppe has quit IRC | 20:58 | |
*** thorst has quit IRC | 20:58 | |
*** jwcroppe has joined #openstack-watcher | 20:59 | |
*** jwcroppe has quit IRC | 20:59 | |
*** jwcroppe has joined #openstack-watcher | 20:59 | |
*** openstackgerrit has quit IRC | 21:19 | |
*** openstackgerrit has joined #openstack-watcher | 21:19 | |
*** thorst has joined #openstack-watcher | 21:21 | |
*** thorst has quit IRC | 21:26 | |
*** hvprash_ has quit IRC | 21:39 | |
*** jwcroppe has quit IRC | 22:10 | |
*** michaelgugino has quit IRC | 22:14 | |
*** jwcroppe has joined #openstack-watcher | 22:14 | |
*** jwcroppe has quit IRC | 22:19 | |
*** gfhellma has quit IRC | 23:10 | |
*** thorst has joined #openstack-watcher | 23:20 | |
*** thorst has quit IRC | 23:28 | |
*** thorst has joined #openstack-watcher | 23:28 | |
*** thorst has quit IRC | 23:31 | |
*** thorst has joined #openstack-watcher | 23:32 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!