Thursday, 2016-07-07

openstackgerritEdwin Zhai proposed openstack/watcher: Documentation for strategy parameters  https://review.openstack.org/33414300:04
*** thorst has joined #openstack-watcher00:25
*** thorst has quit IRC00:32
*** thorst has joined #openstack-watcher00:49
*** thorst has quit IRC01:12
*** thorst has joined #openstack-watcher01:13
*** thorst has quit IRC01:22
*** hanrong has joined #openstack-watcher01:31
openstackgerritEdwin Zhai proposed openstack/python-watcherclient: Enable strategy parameter  https://review.openstack.org/29755701:43
*** thorst has joined #openstack-watcher02:22
*** thorst has quit IRC02:27
*** thorst has joined #openstack-watcher02:34
*** thorst has quit IRC02:34
*** jwcroppe has quit IRC03:24
*** jwcroppe has joined #openstack-watcher03:24
*** jwcroppe has quit IRC03:29
*** thorst has joined #openstack-watcher03:34
*** thorst has quit IRC03:43
*** jwcroppe has joined #openstack-watcher04:26
*** jwcroppe has quit IRC04:28
*** thorst has joined #openstack-watcher04:41
*** thorst has quit IRC04:48
*** diga has joined #openstack-watcher05:39
*** thorst has joined #openstack-watcher05:47
*** thorst has quit IRC05:54
*** danpawlik has quit IRC06:06
*** danpawlik has joined #openstack-watcher06:07
*** jwcroppe has joined #openstack-watcher06:09
*** jwcroppe has quit IRC06:24
*** jed56 has joined #openstack-watcher06:48
*** thorst has joined #openstack-watcher06:52
openstackgerritMerged openstack/watcher: Updated from global requirements  https://review.openstack.org/33625606:57
openstackgerritMerged openstack/watcher: Fix failing Tempest tests  https://review.openstack.org/33724706:58
*** thorst has quit IRC06:59
*** vincentfrancoise has joined #openstack-watcher07:18
*** dtardivel has joined #openstack-watcher07:19
*** jwcroppe has joined #openstack-watcher07:32
openstackgerritTomasz Kaczynski proposed openstack/watcher: Add scoring engines to database and API layers  https://review.openstack.org/33038207:34
*** alexchadin has joined #openstack-watcher07:37
dtardivelalexchadin: Hi Alex. Do you think Vladimir could provided a new patchset today about this changeset https://review.openstack.org/#/c/327095/07:39
dtardivel?07:39
alexchadindtardivel: hi David!07:40
alexchadindtardivel: I will ask him now07:40
*** alexchadin has quit IRC07:45
*** alexchadin has joined #openstack-watcher07:46
alexchadindtardivel: Yeap, it will be provided07:46
dtardivelalexchadin: ok, cool. I'm waiting this new patchset to validate yours as well07:47
dtardivelalexchadin: thks07:47
*** thorst has joined #openstack-watcher07:55
openstackgerritAlexander Chadin proposed openstack/watcher-dashboard: Rename type keyword across the project  https://review.openstack.org/33874708:02
*** thorst has quit IRC08:03
*** jwcroppe has quit IRC08:05
openstackgerritAlexander Chadin proposed openstack/watcher: Add Desktop Service Store to .gitignore file  https://review.openstack.org/33874808:06
alexchadindtardivel: https://review.openstack.org/#/q/topic:bug/153339208:11
alexchadindtardivel: I think it is ready to merge08:11
*** openstackgerrit has quit IRC08:18
*** openstackgerrit has joined #openstack-watcher08:18
digaalexchadin: Hi, I saw your patch, it looks good to me08:19
digadtardivel: can we merge that patch ?08:19
alexchadinhi diga08:20
digaalexchadin: +1 for your patch :)08:21
*** jwcroppe has joined #openstack-watcher08:35
*** jwcroppe has quit IRC08:37
*** jwcroppe has joined #openstack-watcher08:41
alexchadinhi vincentfrancoise08:52
vincentfrancoisehi alexchadin08:52
alexchadinI looked through your PS regarding to cluster model collector08:53
vincentfrancoiseis there any problem with it?08:54
alexchadinWill it collect all the data from the cluster according the collectors/08:54
alexchadin?08:54
vincentfrancoiseBecause I will go through the comments any time soon08:54
alexchadinI mean, if we have two audit templates with specified scope of resources08:55
vincentfrancoise1 collector for 1 model, but each collector can pull/structure the data the way they want08:55
alexchadinin the end we have to give this resources to the strategy08:56
openstackgerritMerged openstack/python-watcherclient: Fix field type to audit_type  https://review.openstack.org/33839108:56
vincentfrancoisethat was something I was thinking a bit about yesterday when you were talking about the audit scope08:56
alexchadinthere could be some ways to resolve08:57
alexchadinI see that08:57
vincentfrancoisemy idea was maybe to give the model to a some audit scope handler that would reduce the global model to the needed subset08:57
vincentfrancoisebut that was only a guess08:57
openstackgerritMerged openstack/watcher: Fix field type to audit_type  https://review.openstack.org/33639008:57
alexchadinwe can put scope json (or object, nevermind) on the model to get "AND" subset08:58
alexchadinthen we can have one model and many of scopes08:59
alexchadinhm08:59
alexchadinmy idea is like yours ;D09:00
vincentfrancoiseI didn't think about how to do it in detail but what you said could be the actual data used by the "audit scope handler" to build a submodel09:00
vincentfrancoisealexchadin: +109:00
alexchadinI will see what we can do with it :)09:00
*** thorst has joined #openstack-watcher09:01
vincentfrancoisethe submodel would have to be built here then https://review.openstack.org/#/c/336032/1/watcher/metrics_engine/cluster_model_collector/base.py@15309:01
alexchadinvincentfrancoise: I agree, I've seen your proposed changes yesterday09:03
alexchadinvincentfrancoise: looks good09:03
vincentfrancoisebtw, I'm currently going through some massive refactoring/renaming09:04
*** jwcroppe has quit IRC09:04
vincentfrancoisevm -> instance, hypervisor -> compute node09:05
vincentfrancoisethe idea is to comply to the nova terminology09:05
alexchadinvincentfrancoise: cool09:06
alexchadinvincentfrancoise: maybe we need to investigate all the project in case of renaming?09:07
alexchadinthere is a lot of places where we use different names for one thing09:07
vincentfrancoiseI don't think these terms are exposed through the API or anything09:07
vincentfrancoiseif you see some terminology that bugs, do report it so we can homogenize the whole09:08
vincentfrancoisebugs you*09:08
*** thorst has quit IRC09:09
alexchadinok09:09
vincentfrancoiseafter the Newton final release09:09
vincentfrancoisethings will be much harder to move around09:09
vincentfrancoiseso that's why09:09
alexchadinwhy they will be harder?09:10
openstackgerritMerged openstack/watcher-dashboard: Rename type keyword across the project  https://review.openstack.org/33874709:10
vincentfrancoisebecause we will have to be more cautious regarding the API with retro-compatibility and DB migrations09:10
vincentfrancoiseso that will take more time09:11
alexchadinvincentfrancoise: we can start to develop new API version only if we have a lot of things to change?09:13
vincentfrancoiseyes, it's not too difficult when we add new parameters09:14
vincentfrancoisebut if remove/rename existing ones, then that will add lots of 'if' everywhere09:15
openstackgerritDavid TARDIVEL proposed openstack/python-watcherclient: Enable strategy parameter  https://review.openstack.org/29755709:15
vincentfrancoisethat was my point09:15
dtardivelgzhai2: Hi fyi, I fixed the merge conflict on your changetset https://review.openstack.org/#/c/297557/09:16
vincentfrancoiseunless we do a v2 but with no retro-compatibility by that would still mean we have to maintain the old one for some time09:16
alexchadinvincentfrancoise: but we can still add new resources without changing old ones, right?09:24
*** acabot has joined #openstack-watcher09:24
vincentfrancoiseyes, new fields/resources will be quite easy09:25
alexchadincool09:26
vincentfrancoisethe not so easy bit will be about removing/renaming existing fields or resources09:26
vincentfrancoiseso better off having the right naming right now :)09:27
alexchadinyeap09:28
alexchadinwe have a lot of time for now09:28
alexchadinit's 4 months to release09:30
*** alexchadin has quit IRC09:39
*** alexchadin has joined #openstack-watcher09:40
*** alexchadin has quit IRC09:43
*** thorst has joined #openstack-watcher10:07
*** thorst has quit IRC10:14
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-watcher: Include openstacklib defaults manifest.  https://review.openstack.org/33884810:14
*** jwcroppe has joined #openstack-watcher10:26
*** jwcroppe has quit IRC10:27
*** jwcroppe has joined #openstack-watcher10:58
*** thorst has joined #openstack-watcher11:06
*** jwcroppe has quit IRC11:30
*** thorst_ has joined #openstack-watcher11:45
*** thorst has quit IRC11:49
*** jwcroppe has joined #openstack-watcher11:49
*** jwcroppe has quit IRC11:54
*** thorst_ is now known as thorst12:02
openstackgerritJinquan Ni proposed openstack/watcher: Make default Planner generic to  https://review.openstack.org/33892112:05
openstackgerritJinquan Ni proposed openstack/watcher: Make default Planner generic to handle new action  https://review.openstack.org/33892112:07
*** alexchadin has joined #openstack-watcher12:36
vincentfrancoisealexchadin: I forgot to tell you earlier but I talked about the audit scope spec with vmahe (the author of the spec) but I don't know yet when he'll take a look at it12:49
alexchadinvincentfrancoise: thank you, I may rewrite this spec and propose it12:50
*** diga has quit IRC12:52
alexchadinvincentfrancoise: there won't be many changes12:52
*** jwcroppe has joined #openstack-watcher12:52
alexchadinvincentfrancoise: I need to explain some questions12:53
vincentfrancoiseok12:55
*** jwcroppe has quit IRC12:57
vincentfrancoisevmahe told me that at the beginning he wanted to rely on heat, but he told us (jed56 and I) that if we can avoid that dependency we would certainly much be better off12:57
alexchadinvincentfrancoise: cool!12:58
vincentfrancoisewhat he really wanted was to have the same structure/naming convention Heat uses12:59
jed56+113:00
alexchadinvincentfrancoise: then there should be some code adoptations but I think it won't be hard to implement13:00
alexchadinhi jed5613:00
alexchadin:)13:00
jed56hi alexchadin, how are you ?13:00
alexchadinI'm great, and you?13:01
jed56good :)13:01
*** alexchad_ has joined #openstack-watcher13:05
*** alexchadin has quit IRC13:09
*** michaelgugino has joined #openstack-watcher13:27
sballe_acabot: are you there?13:30
sballe_jed56: are you there?13:34
jed56yes13:39
*** jinquan1 has joined #openstack-watcher13:45
*** esberglu has joined #openstack-watcher13:48
openstackgerritVladimir Ostroverkhov proposed openstack/watcher-dashboard: Add support continuously-optimization  https://review.openstack.org/32709513:53
openstackgerritVladimir Ostroverkhov proposed openstack/watcher-dashboard: Add support continuously-optimization  https://review.openstack.org/32709513:54
acabotsballe_ : yes14:04
sballe_acabot: just had a chat with jed. He can explain. I will have a patch for you to review tomorrow14:06
*** danpawlik has quit IRC14:08
*** hanrong1 has joined #openstack-watcher14:12
openstackgerritVladimir Ostroverkhov proposed openstack/watcher-dashboard: Add support continuously-optimization  https://review.openstack.org/32709514:16
dtardivelalexchad_: did Vladimir plan to push another changeset on CLI as well ? If not I will do it ...14:19
alexchad_dtardivel: It will be available in few moments, he is working on it right now14:20
openstackgerritVladimir Ostroverkhov proposed openstack/python-watcherclient: Add support continuously-optimization  https://review.openstack.org/33902614:24
alexchad_dtardivel: https://review.openstack.org/#/c/339026/14:25
alexchad_dtardivel: take a look14:25
*** alexchad_ is now known as alexchadin14:25
dtardivelalexchadin: oups sorry. I didn't see it. Great !!!14:26
dtardivelI will review now and test it on devstack14:26
dtardivelthx guys :)14:26
alexchadindtardivel: not a problem :)14:26
*** diga has joined #openstack-watcher14:29
alexchadindtardivel: looks like he found a bug14:32
dtardivelalexchadin: ok. Should I wait for a new patchet ? Which project is concerned ? CLI or UI ?14:33
alexchadinCLI, yeah, it will be done right now14:33
openstackgerritVincent Françoise proposed openstack/watcher: Added DE Background Scheduler w/ model sync jobs  https://review.openstack.org/33603214:35
openstackgerritVincent Françoise proposed openstack/watcher: Loadable Cluster Data Model Collectors  https://review.openstack.org/33398714:35
openstackgerritVincent Françoise proposed openstack/watcher: Updated DE architecture doc + 'period' param  https://review.openstack.org/33603314:35
openstackgerritVincent Françoise proposed openstack/watcher: Cluster data model collector plugin documentation  https://review.openstack.org/33454714:35
openstackgerritVincent Françoise proposed openstack/watcher: Merged metrics_engine package into decision_engine  https://review.openstack.org/33903114:35
openstackgerritVincent Françoise proposed openstack/watcher: Refactored the compute model and its elements  https://review.openstack.org/33903214:35
digajed56: acabot : Hi14:37
digacan you assign this BP to me - https://blueprints.launchpad.net/watcher/+spec/automatic-triggering-audit14:37
digaI am not able to assign it to me14:38
acabotdiga : hi14:39
digacan you make the assignee to the above BP14:39
acabotI can assign it to you but as you can see in the dependency tree, there is a dependency with https://blueprints.launchpad.net/watcher/+spec/continuously-optimization14:39
acabotand this one is still in code review14:40
digaokay14:40
acabotso I would suggest you review this code before starting working on the new BP14:40
digabut till the time I can study on that14:40
digayeah, sure14:40
acabothttps://review.openstack.org/#/c/321411/14:40
digaacabot: how we can decide the workflow & features14:41
acabotwhats you launchpad id ?14:41
digaI am still understanding watcher flow14:41
alexchadinhi diga14:41
digaacabot: digambarpatil1514:42
digaalexchadin: hi14:42
alexchadinI'm working on continuously-optimization14:42
alexchadinIf you have any questions feel free to ask14:42
digaI saw14:42
digasure14:42
digaalexchadin: I will go through the code & get back to you14:42
acabotdiga : I assigned the BP to you, you should start by submitting a spec, are you familiar with that ?14:43
digaacabot: I have done spec part yet, but you can guide me14:43
digaacabot: give me link how we can submit the spec so that I can follow14:44
alexchadinacabot: I can help him with it14:46
dtardivelalexchadin: Could you update your changset https://review.openstack.org/#/c/321411/ please because we merged this morning the changet about type => audit_type (https://review.openstack.org/#/c/336390/)  ?14:46
digaacabot: https://specs.openstack.org/openstack/nova-specs/readme.html14:46
dtardiveland your changeset does not work anymore14:46
digagot it14:46
digaalexchadin: thank you14:46
digabe right back14:47
acabotdiga : https://specs.openstack.org/openstack/watcher-specs/readme.html ;-)14:47
acabotdiga : submit it in newton/approved14:47
acabothttps://github.com/openstack/watcher-specs/tree/master/specs/newton/approved14:47
digaacabot: sure14:47
acabotdiga : thx14:48
digaacabot: welcome! :)14:48
openstackgerritAlexander Chadin proposed openstack/watcher: Add continuously optimization  https://review.openstack.org/32141114:49
alexchadindtardivel: done https://review.openstack.org/#/c/321411/14:50
*** jwcroppe has joined #openstack-watcher14:54
vincentfrancoisealexchadin: I just reviewed it and many issues due to the fact than 'type' was renamed 'audit_type'14:55
*** jinquan1 has quit IRC14:58
*** alexchadin has quit IRC14:58
*** alexchadin has joined #openstack-watcher14:59
*** jwcroppe has quit IRC15:00
openstackgerritVladimir Ostroverkhov proposed openstack/python-watcherclient: Add support continuously-optimization  https://review.openstack.org/33902615:01
*** jinquan1 has joined #openstack-watcher15:05
openstackgerritAlexander Chadin proposed openstack/watcher: Add continuously optimization  https://review.openstack.org/32141115:05
alexchadinoh15:05
alexchadinshould be good now15:05
openstackgerritVladimir Ostroverkhov proposed openstack/watcher-dashboard: Add support continuously-optimization  https://review.openstack.org/32709515:07
openstackgerritAlexander Chadin proposed openstack/watcher: Add continuously optimization  https://review.openstack.org/32141115:07
openstackgerritMerged openstack/watcher: Add Desktop Service Store to .gitignore file  https://review.openstack.org/33874815:09
openstackgerritMerged openstack/watcher: Documentation for strategy parameters  https://review.openstack.org/33414315:10
vincentfrancoisealexchadin: How is Peter Gurinov doing on the tempest CLI tests?15:12
vincentfrancoisehttps://blueprints.launchpad.net/python-watcherclient/+spec/tempest-cli-test15:12
alexchadinvincentfrancoise: I ask him15:13
openstackgerritMerged openstack/python-watcherclient: Enable strategy parameter  https://review.openstack.org/29755715:13
vincentfrancoisealexchadin: thanks15:15
*** alexchadin has quit IRC15:19
*** alexchadin has joined #openstack-watcher15:23
openstackgerritVladimir Ostroverkhov proposed openstack/python-watcherclient: Add support continuously-optimization  https://review.openstack.org/33902615:26
alexchadinvincentfrancoise: he needs help15:27
alexchadinvincentfrancoise: I will see where is the problem tomorrow15:27
vincentfrancoisealexchadin: ok, he can ask me as well if needed15:29
alexchadinvincentfrancoise: he knows :)15:29
alexchadinvincentfrancoise: need to integrate him to CI process15:29
vincentfrancoisealexchadin: I just -1 continuous audit15:29
alexchadinvincentfrancoise: I see15:31
vincentfrancoiseCan you look at the comment and tell me if you agree on https://review.openstack.org/#/c/321411/14/watcher/decision_engine/audit/continuous.py@7215:31
alexchadinvincentfrancoise: take a look https://review.openstack.org/#/c/321411/14/watcher/decision_engine/audit/base.py15:31
alexchadinhere is post_execute method15:32
alexchadinIt calls from continuous class15:32
vincentfrancoiseah my snippet is wrong15:32
vincentfrancoisebut read the comment15:32
vincentfrancoisebelow15:32
dtardivel:015:34
dtardivel:015:34
openstackgerritAlexander Chadin proposed openstack/watcher: Add continuously optimization  https://review.openstack.org/32141115:35
alexchadinvincentfrancoise: fixed15:35
openstackgerritRong Han proposed openstack/watcher-dashboard: [WIP]Add policies for API access control to watcher-dashboard project  https://review.openstack.org/33908215:35
alexchadinvincentfrancoise: but state_in was working for me :/15:35
vincentfrancoiseoh really?15:35
alexchadinvincentfrancoise: yeah15:36
vincentfrancoisethat's weird15:36
*** hanrong1 has left #openstack-watcher15:36
vincentfrancoiselook https://github.com/openstack/watcher/blob/master/watcher/db/sqlalchemy/api.py#L146-L17315:39
openstackgerritVladimir Ostroverkhov proposed openstack/python-watcherclient: Add support continuously-optimization  https://review.openstack.org/33902615:41
alexchadinvincentfrancoise: I see15:41
alexchadinvincentfrancoise: Will try to test it tomorrow:)15:41
openstackgerritAlexander Chadin proposed openstack/watcher: Add continuously optimization  https://review.openstack.org/32141116:03
alexchadindtardivel: are you here?16:09
dtardivelyes :)16:09
dtardivelI saw your patchset 18 :)16:09
alexchadindtardivel: have you done with reviewing?16:09
alexchadinI suggest to use continuous_audit_interval instead16:10
alexchadincontinuous_audit_check_interval makes line too long16:10
dtardivelok16:10
openstackgerritVladimir Ostroverkhov proposed openstack/python-watcherclient: Add support continuously-optimization  https://review.openstack.org/33902616:11
dtardivelI'm running again test with lastest patchsets.16:11
dtardivelI let you know tomorrow morning (dasboard and cli seems ok for me)16:12
openstackgerritAlexander Chadin proposed openstack/watcher: Add continuously optimization  https://review.openstack.org/32141116:13
openstackgerritJinquan Ni proposed openstack/watcher: Make default Planner generic to handle new action  https://review.openstack.org/33892116:17
*** diga has quit IRC16:21
*** alexchad_ has joined #openstack-watcher16:21
openstackgerritAlexander Chadin proposed openstack/watcher: Add continuously optimization  https://review.openstack.org/32141116:22
*** alexcha__ has joined #openstack-watcher16:23
*** alexchadin has quit IRC16:25
*** alexchad_ has quit IRC16:27
*** jinquan1 has quit IRC16:28
*** alexcha__ has quit IRC16:28
*** jwcroppe has joined #openstack-watcher16:58
*** wootehfoot has joined #openstack-watcher17:00
*** jwcroppe has quit IRC17:02
*** jed56 has quit IRC17:25
*** jwcroppe has joined #openstack-watcher17:58
*** jwcroppe has quit IRC18:04
*** dtardivel has quit IRC18:37
*** jwcroppe has joined #openstack-watcher19:05
*** jwcroppe has quit IRC19:05
*** jwcroppe has joined #openstack-watcher19:05
*** jwcroppe has quit IRC20:03
*** danpawlik has joined #openstack-watcher20:45
*** danpawlik has quit IRC20:54
*** danpawlik has joined #openstack-watcher21:31
*** danpawlik has quit IRC21:55
*** wootehfoot has quit IRC21:59
*** thorst has quit IRC22:03
*** esberglu has quit IRC22:14
*** thorst has joined #openstack-watcher22:50
*** thorst has quit IRC22:54
*** thorst has joined #openstack-watcher23:31
*** thorst has quit IRC23:36
*** thorst has joined #openstack-watcher23:52
*** thorst has quit IRC23:57

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