Thursday, 2016-06-23

*** thorst has joined #openstack-watcher00:48
*** thorst has quit IRC00:56
*** esberglu has joined #openstack-watcher01:36
*** thorst has joined #openstack-watcher01:53
*** thorst has quit IRC01:59
*** esberglu has quit IRC02:00
*** thorst has joined #openstack-watcher02:05
*** thorst has quit IRC02:11
openstackgerritEdwin Zhai proposed openstack/python-watcherclient: Revert "Add support continuously-optimization"  https://review.openstack.org/33309002:41
hanrong1 https://blueprints.launchpad.net/watcher/+spec/watcher-policies Does this bp need to submit a spec file before writing code?02:51
*** thorst has joined #openstack-watcher03:16
*** thorst has quit IRC03:21
*** thorst has joined #openstack-watcher04:19
*** thorst has quit IRC04:25
*** ecelik has joined #openstack-watcher05:22
*** thorst has joined #openstack-watcher05:23
*** thorst has quit IRC05:31
*** Kevin_Zheng has joined #openstack-watcher06:08
*** thorst has joined #openstack-watcher06:28
*** thorst has quit IRC06:36
*** vtech has joined #openstack-watcher06:58
jed56hanrong1: no you don't need a blueprint07:08
jed56Good UTC Moorning07:08
*** brunograz has joined #openstack-watcher07:25
*** vincentfrancoise has joined #openstack-watcher07:28
*** thorst has joined #openstack-watcher07:34
openstackgerritMerged openstack/watcher: Centralize plugin loaders in decision engine  https://review.openstack.org/33262407:38
*** thorst has quit IRC07:41
openstackgerritTin Lam proposed openstack/watcher: Centralize plugin loaders in watcher/applier  https://review.openstack.org/33262707:51
*** alexchadin has joined #openstack-watcher07:54
alexchadinvincentfrancoise: hi!07:54
alexchadinhow are you? :)07:54
*** alexchadin has quit IRC08:05
*** alexchadin has joined #openstack-watcher08:05
hanrong1 jed56:hi,  don't need a blueprint?08:12
*** alexstav has joined #openstack-watcher08:18
alexstavHi, guys! Review my bug fix, please :) https://review.openstack.org/#/c/332241/08:18
*** alexchadin has quit IRC08:22
vincentfrancoisehi08:22
*** alexchadin has joined #openstack-watcher08:22
*** dtardivel has joined #openstack-watcher08:23
dtardivelhanrong1: Hi08:23
*** acabot has quit IRC08:26
*** acabot has joined #openstack-watcher08:29
hanrong1dtardivel:hi08:38
*** thorst has joined #openstack-watcher08:39
hanrong1 https://blueprints.launchpad.net/watcher/+spec/watcher-policies Does this bp need to submit a spec file before writing code?08:39
dtardivelIMHO, I don't think so.08:39
dtardivelSpec is not mandatory, when the BP is simple and cleat.08:39
dtardivelclear08:39
hanrong1dtardivel: ok, I will submit a patch code.08:40
dtardivelDo you want we discuss about this feature to implement ?08:40
dtardivelok, I will review it .08:41
alexchadintpeoples: hi08:43
alexchadintpeoples: are you working on https://bugs.launchpad.net/watcher/+bug/1533392 ?08:43
openstackLaunchpad bug 1533392 in watcher "Fields should not be named the built-in type keyword" [Low,Triaged] - Assigned to Taylor Peoples (tpeoples)08:43
*** thorst has quit IRC08:46
hanrong1 dtardivel: All the actions should be performed by only admin in watcher project. It's similar to ironic project.08:48
hanrong1 dtardivel: So policy.json file doesn't need to modified.08:49
jed56alexchadin: hello08:50
alexchadinjed56: good morning08:51
jed56alexchadin: how are you ?08:51
alexchadinjed56: I'm fine, are you?08:51
jed56very well thanks you08:51
jed56Regarding the continuous optimization. There is too many decisions in the patchset Add Oslo Versioned Object + Indirection API08:52
jed56so, we can't merge it quickly08:52
jed56so, for the version of continuous audit08:52
jed56we will not use events08:52
jed56You just to add a periodic task in charge of updating the audit status08:53
jed56*need08:53
alexchadinhm08:53
jed56what do you think ?08:54
alexchadinyou propose to remove any using of APScheduler?08:54
jed56No, you have to kepp APScheduler08:54
jed56keep08:54
alexchadinlet me see the code08:54
jed56only the even driven08:54
jed56I think that you code is already doing that08:55
jed56alexchadin: I will review it today08:56
alexchadinwwwwait :D08:56
jed56vincentfrancoise: could you also review it today ?08:56
alexchadinare you speaking about updating audit state?08:57
jed56alexchadin: yes08:57
vincentfrancoisejed56 alexchadin: yes I'll do it08:57
jed56vincentfrancoise: thanks08:57
vincentfrancoiseSo as a quick recap08:58
vincentfrancoisealexchadin doesn't do the notification part of his BP08:58
vincentfrancoisewe create a BP + spec for oslo versionedobjects and notifications08:59
vincentfrancoisewhen it gets implemented08:59
vincentfrancoisewe can add back the notification system to make it even driven08:59
vincentfrancoiseis that right?08:59
alexchadinLooks good08:59
vincentfrancoisejed56: ^08:59
alexchadinvincentfrancoise: may I ask you some questions regarding test_objects?09:00
jed56vincentfrancoise: yes exactly09:00
vincentfrancoisealexchadin: what about it?09:00
alexchadinjed56: could you advise some little and easy BP for my trainee?09:01
alexchadinvincentfrancoise: for example test_obj_to_primitive_list09:01
jed56alexchadin: let me take a look09:01
alexchadinI see that we don't have obj_to_primitive_list method anymore09:01
alexchadinDon't we use this method anymore?09:02
vincentfrancoisealexchadin: how about https://blueprints.launchpad.net/python-watcherclient/+spec/tempest-cli-test ?09:03
jed56vincentfrancoise: +109:03
vincentfrancoiseit will take some time to do it but I don't think it will be too complicated09:03
alexchadinvincentfrancoise: I agree09:04
alexchadinvincentfrancoise: Tests are good starting point09:04
jed56the functional tests are always a good start09:04
alexchadinjed56: +109:04
vincentfrancoisealexchadin: about obj_to_primitive_list, we are doing list comprehensions wherever needed instead09:05
alexchadinAs I understand, most work with objects is handled by OVO now09:06
dtardivelhanrong1: I prefer the magnum approach. Did you see it ?  https://github.com/openstack/magnum/blob/master/etc/magnum/policy.json09:06
vincentfrancoisealexchadin: I amended the CLI tempest test BP to add the doc link and a github repo which has done CLI tempest tests already09:08
dtardivelhanrong1: With this code, an admin can defined another admin roles: one the creators of audit templates, another one for the guy who will launch an audit ...09:09
vincentfrancoisealexchadin: Yes, OVO simplifies quite a lot the logic we previously had in Watcher09:09
alexchadinvincentfrancoise: thank you so much, I sent to him this BP09:10
dtardivelhanrong1: and the policy check is very easy to implement: https://github.com/openstack/magnum/blob/cd1297148ba6906be44c46e3cf22377c704a6046/magnum/api/controllers/v1/baymodel.py#L26509:10
alexchadinvincentfrancoise: so, I think we have to remove all the test methods that we don't have now09:11
alexchadinall the tests for methods*09:12
hanrong1dtardivel: Thank you very much.09:12
dtardivelhanrong1: you're welcome09:13
*** thorst has joined #openstack-watcher09:44
*** thorst has quit IRC09:51
*** alexstav has quit IRC10:06
openstackgerritVincent Fran├žoise proposed openstack/watcher-specs: Cluster model objects wrapper  https://review.openstack.org/28701910:11
*** vtech has quit IRC10:11
*** vtech has joined #openstack-watcher10:12
*** vincentfrancoise has quit IRC10:15
openstackgerritEdwin Zhai proposed openstack/watcher: Enable strategy parameters  https://review.openstack.org/29755610:31
*** thorst has joined #openstack-watcher10:49
*** thorst has quit IRC10:56
*** alexchadin has quit IRC11:14
*** vtech has quit IRC11:22
*** thorst has joined #openstack-watcher11:41
openstackgerritJean-Emile DARTOIS proposed openstack/watcher-specs: Update the continuously optimization spec.  https://review.openstack.org/33002112:11
*** vtech has joined #openstack-watcher12:11
openstackgerritJean-Emile DARTOIS proposed openstack/watcher-specs: Update the continuously optimization spec.  https://review.openstack.org/33002112:16
*** vincentfrancoise has joined #openstack-watcher12:21
*** alexstav has joined #openstack-watcher12:48
openstackgerritMerged openstack/watcher: Centralize plugin loaders in watcher/applier  https://review.openstack.org/33262713:00
*** alexchadin has joined #openstack-watcher13:20
alexstavvincentfrancoise: thanks for review! What does it need for merging? :)13:21
vincentfrancoisealexstav: another core reviewers needs to validate it as well13:28
openstackgerritVincent Fran├žoise proposed openstack/python-watcherclient: Restore "Add support continuously-optimization"  https://review.openstack.org/33335613:35
*** esberglu has joined #openstack-watcher13:37
*** hvprash has joined #openstack-watcher13:51
*** esberglu has quit IRC13:55
*** hvprash_ has joined #openstack-watcher13:56
*** hvprash has quit IRC13:59
*** esberglu has joined #openstack-watcher14:02
*** esberglu has quit IRC14:02
openstackgerritAlexander Chadin proposed openstack/watcher: Add rally-jobs folder to get rally support  https://review.openstack.org/33337114:14
*** hvprash has joined #openstack-watcher14:19
*** hvprash__ has joined #openstack-watcher14:20
*** hvprash_ has quit IRC14:22
*** vtech has quit IRC14:23
*** hvprash has quit IRC14:24
openstackgerritAlexander Chadin proposed openstack/watcher: Add rally-jobs folder to get rally support  https://review.openstack.org/33337114:39
*** hvprash__ has quit IRC14:51
*** hvprash has joined #openstack-watcher14:54
*** esberglu has joined #openstack-watcher15:08
*** ostroverkhov has joined #openstack-watcher15:19
vincentfrancoisealexchadin: are you there?15:21
alexchadinvincentfrancoise: hi again15:21
vincentfrancoiseregarding the rally changeset you made15:21
alexchadinvincentfrancoise: yes?15:22
vincentfrancoiseare you planning on creating a new rally job for the gate?15:22
alexchadinyes15:22
vincentfrancoiseok cool15:23
vincentfrancoisethen can you do it like for the tempest job: both non-voting and experimental for now ?15:24
alexchadinYes, of course15:24
vincentfrancoisecan you put me as reviewer on it once you push it to project-config?15:24
alexchadinvincentfrancoise: will do15:25
alexchadinDo you have experience with gates?15:25
vincentfrancoisea bit yes15:25
alexchadinIs there any way to test gates before submitting?15:25
vincentfrancoisenot really ^^15:26
alexchadinwoah15:26
vincentfrancoiseyou have to "mentally" test it15:26
vincentfrancoiseunless you have a full replica of zuul and stuff15:27
alexchadinI also thought so15:27
alexchadinCould you give me link15:28
alexchadinwith your gate commit?15:28
*** alexstav has quit IRC15:29
*** alexstav has joined #openstack-watcher15:30
vincentfrancoisehttps://review.openstack.org/#/q/project:openstack-infra/project-config+owner:v-francoise15:30
*** alexstav_ has joined #openstack-watcher15:32
*** alexstav has quit IRC15:33
alexchadinvincentfrancoise: thanks :)15:36
openstackgerritMerged openstack/watcher: Add importing modules instead of classes  https://review.openstack.org/33224115:39
ostroverkhovvincentfrancoise: hi, this commit https://review.openstack.org/#/c/331552/ is already merged, but it depend on https://review.openstack.org/#/c/331551/ witch is not merged yet. I think there could be problems :(15:39
jed56alexchadin: we discussed with vincentfrancoise about the implement of continuous is going to propose some few modifications15:48
alexchadinjed56: I'm listening15:48
jed56vincentfrancoise is currently making the comments on the patchset15:50
jed56:)15:50
alexchadinjed56: cool ;)15:50
jed56alexstav_ and ostroverkhov you are now member of watcher-drivers15:50
alexchadincongrats!15:50
alexstav_jed56: thanks!15:50
jed56welcome is great to have new contributors !15:51
ostroverkhovs/witch/which15:51
ostroverkhovjed56: thanks!15:51
alexchadinvincentfrancoise: are you here?15:55
vincentfrancoisealexchadin: yeah15:55
vincentfrancoisealexchadin: I am writing comments on your continuously-optimization patchset15:57
alexchadinwhether should I add gate-rally-dsvm-watcher to check or experimental branch?15:57
vincentfrancoiseafter discussing it with jed5615:57
vincentfrancoisebut it takes a bit of time15:57
alexchadinIt is non-voting15:57
vincentfrancoiseexperimental first15:58
alexchadinWe don't have any experimental gates for now, right?15:58
vincentfrancoisethen after a couple of weeks, if we see there's no issue, we can move on to make it 'check'15:58
alexchadinvincentfrancoise: cool, got it15:59
vincentfrancoisewe had experimental jobs before16:00
vincentfrancoisethe gate-dsvm-multinode-watcher one16:00
*** alexstav_ has quit IRC16:22
*** alexstav has joined #openstack-watcher16:23
vincentfrancoiseostroverkhov: hi16:27
vincentfrancoiseare you still there ?16:28
ostroverkhovyes16:28
*** esberglu has quit IRC16:28
*** esberglu has joined #openstack-watcher16:32
*** vincentfrancoise has quit IRC16:33
*** ostroverkhov has quit IRC16:38
*** ostroverkhov has joined #openstack-watcher16:38
*** ostroverkhov has quit IRC16:39
*** alexstav has quit IRC16:47
*** alexchadin has quit IRC16:48
*** esberglu_ has joined #openstack-watcher17:04
*** esberglu has quit IRC17:07
*** esberglu_ has quit IRC17:10
*** hvprash has quit IRC17:10
*** esberglu has joined #openstack-watcher17:12
*** ostroverkhov has joined #openstack-watcher17:25
*** ostroverkhov has quit IRC17:27
*** hvprash has joined #openstack-watcher17:28
*** hvprash has quit IRC17:33
*** wootehfoot has joined #openstack-watcher17:35
*** hvprash has joined #openstack-watcher17:54
*** thorst is now known as thorst_away18:55
*** dtardivel has quit IRC19:57
*** zigo has quit IRC20:27
*** junjie has quit IRC20:27
*** odyssey4me has quit IRC20:27
*** odyssey4me has joined #openstack-watcher20:28
*** zigo has joined #openstack-watcher20:29
*** junjie has joined #openstack-watcher20:31
*** thorst_away has quit IRC21:10
*** thorst_away has joined #openstack-watcher21:13
*** thorst_a_ has joined #openstack-watcher21:15
*** thorst_away has quit IRC21:17
*** thorst_a_ has quit IRC21:19
*** esberglu has quit IRC21:27
openstackgerritMerged openstack/puppet-watcher: Implement api.pp  https://review.openstack.org/32983021:28
*** thorst_away has joined #openstack-watcher21:38
*** thorst_away has quit IRC21:43
*** thorst_away has joined #openstack-watcher21:43
*** thorst_away has quit IRC21:47
*** esberglu has joined #openstack-watcher21:48
*** hvprash_ has joined #openstack-watcher22:25
*** hvprash has quit IRC22:28
*** hvprash_ has quit IRC22:31
*** hvprash has joined #openstack-watcher22:43
*** hvprash has quit IRC22:44
*** hvprash has joined #openstack-watcher22:48
*** hvprash has quit IRC22:52
*** wootehfoot has quit IRC23:12
*** vtech has joined #openstack-watcher23:22

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!