Wednesday, 2017-07-26

openstackgerritMerged openstack/watcher-dashboard master: Imported Translations from Zanata  https://review.openstack.org/48248700:23
*** thorst has joined #openstack-watcher00:29
*** jwcroppe has joined #openstack-watcher00:29
*** thorst has quit IRC00:33
*** jwcroppe has quit IRC00:33
*** jwcroppe has joined #openstack-watcher00:33
*** deep-book-gk_ has joined #openstack-watcher00:34
*** deep-book-gk_ has left #openstack-watcher00:37
*** thorst has joined #openstack-watcher00:59
*** thorst has quit IRC01:07
openstackgerritMerged openstack/watcher master: Remove all sphinx warnings  https://review.openstack.org/48661901:07
*** harlowja has quit IRC01:29
*** nicolasbock has joined #openstack-watcher04:04
openstackgerritaditi sharma proposed openstack/watcher master: Validation of states for actionplan delete  https://review.openstack.org/48600704:33
*** hidekazu has joined #openstack-watcher04:38
*** harlowja has joined #openstack-watcher04:40
*** thorst has joined #openstack-watcher05:04
*** thorst has quit IRC05:10
openstackgerritHidekazu Nakamura proposed openstack/watcher master: [Doc] Add Configure Cinder Notifications  https://review.openstack.org/48728005:38
*** harlowja has quit IRC05:39
adisky__hidekazu: hii05:44
hidekazuadisky__: hi05:44
adisky__hidekazu:  as you can see https://github.com/openstack/watcher/blob/master/watcher/api/controllers/v1/action_plan.py#L45205:45
adisky__https://github.com/openstack/watcher/blob/master/watcher/api/controllers/v1/audit.py#L60405:45
adisky__there is no vakdiation for state transitions in both audits and action plan05:45
adisky__*validation05:45
hidekazuadisky__: you mean validation of delete method05:46
adisky__yes05:46
hidekazuadisky__* at reviewing , i said only validation patch method..05:47
hidekazuadisky__* at the same time, i found https://github.com/openstack/watcher/blob/master/watcher/api/controllers/v1/action_plan.py#L50505:48
hidekazuadisky__* but i do not know we can check state transition using jsonschema api validation smartly05:49
hidekazuadisky__* old days i heard ironic created strong state checker using python decorator.05:50
adisky__hidekazu:  ok i will check with ironic.05:51
adisky__hidekazu:  i found no blueprint for 'watcher-api-validation' as mentioned in TODO05:52
hidekazuadisky__* if we can use jsonschema api validation, that is the best approach. ithink.05:52
hidekazuadisky__: please take https://bugs.launchpad.net/watcher/+bug/1659556  into consideration.05:53
openstackLaunchpad bug 1659556 in watcher "Action plan continues to be recommended after updating continuous audit to oneshot" [Medium,Triaged] - Assigned to aditi sharma (adi-sky17)05:53
adisky__hidekazu:  Then i need to study, and file blueperint for that05:53
adisky__hidekazu: yes i will take it into account05:55
hidekazuadisky__: yeah, before ptg, i thought jsonschema-validation spec is for api validation. but now we do not have.05:57
adisky__ok will register05:57
adisky__hidekazu: I saw many projects implemening that05:57
hidekazuadisky__: yes yes05:57
hidekazuadisky__: even state transition not possible by api validation, api validation using jsonschema should be implemented.05:58
adisky__hidekazu: ok, thanks for your inputs05:59
hidekazuadisky__* i hope you will find good solution :)06:00
adisky__hidekazu:  :)06:00
*** Yumeng has joined #openstack-watcher06:22
openstackgerritMerged openstack/watcher-dashboard master: Updated from global requirements  https://review.openstack.org/47056706:26
*** thorst has joined #openstack-watcher06:29
*** thorst has quit IRC06:33
openstackgerritYumengBao proposed openstack/watcher master: Saving Energy Strategy  https://review.openstack.org/47676707:06
openstackgerritWei Zheng proposed openstack/watcher master: Update the documention for doc migration  https://review.openstack.org/48612307:29
openstackgerritHidekazu Nakamura proposed openstack/watcher master: [Doc] Add Configure Cinder Notifications  https://review.openstack.org/48728007:31
*** alexchadin has joined #openstack-watcher07:43
Yumenghi all´╝îI have a question: why watcher audit create cannot specify scope of an audit but only watcher audittemplate create can?07:43
Yumengwhat's the purpose of this?07:44
openstackgerritAlexander Chadin proposed openstack/watcher master: Add release notes for Pike  https://review.openstack.org/48614607:45
alexchadinhi Yumeng07:45
Yumengalexchadin:hi07:45
alexchadinYumeng: Audit Template is base thing for Audit, and it should include things are going to achieve (goal), the way we use to achieve goal (strategy) and pool of resources where we want to achieve goal07:47
alexchadinYumeng: everything else is temporary and can be handle by Audit07:48
hidekazuYumeng: alexchadin: honestly i had a same question. i usually execute 'watcher audit create -g xxx -s yyy'07:50
alexchadinhidekazu: will this command create audit template?07:51
Yumengalexchadin,hidekazu: yeah. that's my point. watcher audit create can create audit without audittemplate07:52
hidekazualexchadin: no, i do not use audittemplate usually.07:52
hidekazuYumeng: i do not know we can update scope dynamically.07:53
Yumengme either. seldom use audittemplate if using cli to create an audit07:53
YumengIMHO, audittemplate is a thing that when we want to save and reuse it. if we don't want to reuse it, just create an audit. that's enough07:54
hidekazuYumeng: i think too.07:55
alexchadinhidekazu, Yumeng https://github.com/openstack/watcher/blob/master/watcher/api/controllers/v1/audit.py#L70 try to remove readonly flag, lines 99-113 should manage scope too07:57
Yumengso "watcher audit create" should also be able to specify the scope just like "watcher create audittemplate" can do.07:57
alexchadinhidekazu, Yumeng let's see what it brings to us07:57
hidekazualexchadin: thanks, i understand audit scope can not be updated dynamically.08:00
openstackgerritAlexander Chadin proposed openstack/python-watcherclient master: Fix Audit Update functional test  https://review.openstack.org/48548808:09
*** brunograz has quit IRC08:20
*** efoley has joined #openstack-watcher08:23
*** efoley_ has joined #openstack-watcher08:24
*** efoley has quit IRC08:24
hidekazualexchadin: i have a question: 1. can we w+1 now even if we don't merge any specs until stable/pike branch is created? 2. feature freeze is thursday not 31st?08:24
alexchadinhidekazu: 1. we can merge only pike-3 specs for now08:26
alexchadinhidekazu: 2. Feature freeze is on Thursday, right08:26
hidekazualexchadin: 1. https://review.openstack.org/#/c/474865/ was w+1 today even though that is not for pike-3.08:29
hidekazualexchadin: 2. Thursday is tomorrow! ok..08:30
*** thorst has joined #openstack-watcher08:30
alexchadinhidekazu: well, that's not good08:30
alexchadinhidekazu: We may leave it as is, on an exceptional basis.08:31
hidekazualexchadin: ok, anyway that is not implemented.08:32
*** thorst has quit IRC08:34
adisky__alexchadin: hi08:35
alexchadinhi adisky__08:35
*** efoley_ is now known as efoley08:35
*** brunograz has joined #openstack-watcher08:36
adisky__alexchadin:  do you plan to review or merge https://review.openstack.org/#/c/482902/??08:36
adisky__for pike cycle or it will got queens??08:38
adisky__* go to08:38
*** hidekazu has left #openstack-watcher08:38
*** hidekazu has joined #openstack-watcher08:39
alexchadinadisky__: It will go for Queens08:40
adisky__alexchadin:  so i should not summit implementation for now??08:41
alexchadinadisky__: I will review your patch, but I'm leaving my workplace soon (hidekazu will hold meeting today). You absolutely may submit your implementation, we won't merge it until Pike release08:43
adisky__alexchadin:  ok thanks08:43
*** hidekazu has quit IRC08:48
openstackgerritAlexander Chadin proposed openstack/python-watcherclient master: Update .gitignore because of doc migration  https://review.openstack.org/48732508:59
openstackgerritYumengBao proposed openstack/watcher master: Saving Energy Strategy  https://review.openstack.org/47676709:30
*** alexchadin has quit IRC09:46
*** dpawlik has joined #openstack-watcher09:52
*** danpawlik has quit IRC09:54
*** thorst has joined #openstack-watcher09:58
*** thorst has quit IRC10:06
*** thorst has joined #openstack-watcher10:07
*** thorst has quit IRC10:11
*** hidekazu has joined #openstack-watcher11:25
*** hidekazu has quit IRC12:03
*** hidekazu has joined #openstack-watcher12:07
*** thorst has joined #openstack-watcher12:16
openstackgerritMerged openstack/python-watcherclient master: Update .gitignore because of doc migration  https://review.openstack.org/48732512:26
*** efoley has quit IRC12:47
*** efoley has joined #openstack-watcher12:50
openstackgerritHidekazu Nakamura proposed openstack/watcher master: [Doc] Add cinder to architecture diagram  https://review.openstack.org/48740212:52
hidekazumeeting will be held one hour later.13:01
hidekazutoday alexchadin can not hold, so i will hold the meeting.13:02
*** jwcroppe has quit IRC13:08
*** lakerzhou1 has joined #openstack-watcher13:25
*** hidekazu has quit IRC13:46
*** jwcroppe has joined #openstack-watcher13:52
*** hidekazu has joined #openstack-watcher14:00
hidekazuwe are meeting in #openstack-meeting-414:32
*** hidekazu has left #openstack-watcher14:58
*** harlowja has joined #openstack-watcher16:15
*** jwcroppe has quit IRC16:20
*** harlowja has quit IRC17:46
*** Yumeng has quit IRC18:33
*** wootehfoot has joined #openstack-watcher18:55
*** nicolasbock has quit IRC19:06
*** harlowja has joined #openstack-watcher19:21
*** harlowja has quit IRC19:25
*** thorst has quit IRC20:50
*** efoley has quit IRC21:24
*** thorst has joined #openstack-watcher21:32
*** lakerzhou1 has quit IRC21:36
*** thorst has quit IRC21:37
*** harlowja has joined #openstack-watcher22:00
*** wootehfoot has quit IRC22:05
*** thorst has joined #openstack-watcher22:25
*** thorst has quit IRC22:26
openstackgerritHidekazu Nakamura proposed openstack/watcher-specs master: Spec for multi-datasource support  https://review.openstack.org/46710223:18
*** hidekazu has joined #openstack-watcher23:18
*** thorst has joined #openstack-watcher23:27
*** thorst has quit IRC23:32
*** thorst has joined #openstack-watcher23:50
*** thorst has quit IRC23:50

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