Monday, 2017-04-10

*** gouthamr has quit IRC00:00
*** markstur has quit IRC00:35
*** a-pugachev has quit IRC00:36
*** markstur has joined #openstack-manila00:36
*** markstur has quit IRC00:40
*** kaisers has quit IRC01:04
*** kaisers has joined #openstack-manila01:44
openstackgerritTommyLike proposed openstack/manila master: Add perodic task to clean up expired reservation  https://review.openstack.org/44015501:58
*** tuanluong has joined #openstack-manila01:59
*** tuanluong_ has joined #openstack-manila02:02
*** tuanluong has quit IRC02:05
*** gouthamr has joined #openstack-manila02:05
*** markstur has joined #openstack-manila02:26
*** markstur has quit IRC02:44
*** kaisers has quit IRC02:46
*** tuanluong has joined #openstack-manila02:47
*** tuanluong_ has quit IRC02:49
*** markstur has joined #openstack-manila03:14
*** gouthamr has quit IRC03:16
*** markstur has quit IRC03:19
*** kaisers has joined #openstack-manila03:42
*** catintheroof has joined #openstack-manila03:43
*** markstur has joined #openstack-manila03:50
*** catintheroof has quit IRC03:51
*** markstur has quit IRC03:53
*** kaisers has quit IRC04:23
*** kaisers has joined #openstack-manila04:24
*** markstur has joined #openstack-manila04:24
*** kaisers has quit IRC04:25
*** kaisers has joined #openstack-manila04:25
*** kaisers has quit IRC04:31
*** kaisers has joined #openstack-manila04:55
*** ociuhandu has quit IRC05:00
*** ociuhandu has joined #openstack-manila05:01
*** markstur has quit IRC05:38
*** ociuhandu has quit IRC05:55
*** gcb has joined #openstack-manila05:57
*** tuanluong has quit IRC06:00
*** pcaruana has joined #openstack-manila06:01
*** ociuhandu has joined #openstack-manila06:05
*** sticker has joined #openstack-manila06:11
*** tuanluong has joined #openstack-manila06:14
*** dsariel has joined #openstack-manila06:23
*** ociuhandu has quit IRC06:31
*** ociuhandu has joined #openstack-manila06:44
*** jprovazn has joined #openstack-manila06:46
*** gaurangt has joined #openstack-manila06:59
*** dmellado has joined #openstack-manila07:08
*** markstur has joined #openstack-manila07:14
*** markstur has quit IRC07:19
*** gcb has quit IRC07:46
*** lpetrut has joined #openstack-manila07:48
*** a-pugachev has joined #openstack-manila07:58
*** gcb has joined #openstack-manila08:12
*** ociuhandu has quit IRC08:51
*** ociuhandu has joined #openstack-manila09:00
*** tuanluong has quit IRC09:10
*** ociuhandu has quit IRC09:14
*** ociuhandu has joined #openstack-manila09:18
*** rpittau has joined #openstack-manila09:31
openstackgerritValeriy Ponomaryov proposed openstack/manila-specs master: Add spec for share_groups quota resource  https://review.openstack.org/45227009:43
openstackgerritValeriy Ponomaryov proposed openstack/manila-specs master: Add spec for share_groups quota resource  https://review.openstack.org/45227009:45
openstackgerritValeriy Ponomaryov proposed openstack/manila master: Add possibility to run 'manila-api' with wsgi web servers  https://review.openstack.org/44819009:49
*** gcb has quit IRC10:16
zhongjun_vponomaryov: ping10:21
*** ociuhandu has quit IRC10:25
vponomaryovzhongjun_: pong10:31
zhongjun_vponomaryov: in https://review.openstack.org/#/c/448969/7/specs/pike/support-filter-share-by-export-location.rst10:33
zhongjun_vponomaryov: Only shares API? Why not "share-instances" API too?10:33
zhongjun_vponomaryov: We can get share instances from share id.10:34
zhongjun_vponomaryov: s/from/by10:36
zhongjun_vponomaryov: Could we focous on share filter?10:36
vponomaryovzhongjun_: still don't  see argument for not implementing filter in share-instances LIST API10:38
vponomaryovzhongjun_: now you say that you will use 2 API calls instead of 110:38
vponomaryovzhongjun_: 1 - get share ID and then filter share instances using share ID10:39
vponomaryovzhongjun_: instad of filtering share instances directly10:39
vponomaryovs/instad/instead/10:39
zhongjun_vponomaryov: Use this logic, it sounds like we need to add more filters in share-instances LIST API :D10:41
vponomaryovcould be10:42
vponomaryovbut others are out of scope for your spec10:42
*** markstur has joined #openstack-manila10:50
*** jprovazn has quit IRC10:51
zhongjun_vponomaryov: OK, It will good for admin.10:53
*** markstur has quit IRC10:55
*** ociuhandu has joined #openstack-manila10:55
*** ganso has joined #openstack-manila10:59
zhongjun_vponomaryov: We could get share instance ID from share export location path, and use "share-instance-show <instance>" command to find out associated share.11:10
vponomaryovzhongjun_: mot always you can do it11:11
vponomaryovs/mot/not/11:11
zhongjun_vponomaryov: just 1 API, but not directly to get instance ID11:11
zhongjun_vponomaryov: in spec, gason said: I don't see share-instances as a much needed requirement since shares API is the one user-facing whi share-instances is admin-only11:18
zhongjun_vponomaryov: How's your opinion about this?11:18
vponomaryovgason ))11:19
zhongjun_vponomaryov: When we run share-instances-list command, we can not see the export location in the share instances list https://github.com/openstack/python-manilaclient/blob/master/manilaclient/v2/shell.py#L157211:21
vponomaryovzhongjun_: policies can be changed anyhow. admin/user - should not make diff11:21
vponomaryovzhongjun_: "export-locaion" is share instance's part, not share's11:22
vponomaryovzhongjun_:and still don't see argument for not implementing it11:22
zhongjun_vponomaryov: yes, just not show up in share-instances-list command11:22
vponomaryovzhongjun_: what exactly do you lose if you implement that filtering there too?11:22
zhongjun_vponomaryov: OK, I think your idea is good for admin11:23
openstackgerritzhongjun proposed openstack/manila-specs master: Support retrieve shares filtered by export-location  https://review.openstack.org/44896911:25
zhongjun_vponomaryov: If I am a admin, I think I will like you very much.11:26
zhongjun_vponomaryov: Done, Please take a look again : )11:26
openstackgerritzhongjun proposed openstack/manila-specs master: Support retrieve shares filtered by export-location  https://review.openstack.org/44896911:31
*** jprovazn has joined #openstack-manila11:32
openstackgerritzhongjun proposed openstack/manila-specs master: Support retrieve shares filtered by export-location  https://review.openstack.org/44896911:32
*** belmoreira has joined #openstack-manila11:36
openstackgerritzhongshengping proposed openstack/puppet-manila master: oslo db: check puppet resource instead of actual config in spec  https://review.openstack.org/45527411:39
vponomaryovzhongjun_: do not use "master" as reference does not mean "delete it"11:40
tommylikehuzhongjun, you could aslo like vponomaryov11:40
tommylikehueven if you are not a administrator11:40
vponomaryovtommylikehu: ^_^11:40
gansotommylikehu: lol11:40
vponomaryovzhongjun_: do not use "master" as reference means use specific tag or commit reference instead11:41
vponomaryovzhongjun_: in other words - "immutable" reference11:42
zhongjun_vponomaryov: Ok, I got it.11:42
zhongjun_vponomaryov: Done, Dear vponomaryov. haha11:44
*** mtanino has joined #openstack-manila11:51
openstackgerritzhongjun proposed openstack/manila-specs master: Support retrieve shares filtered by export-location  https://review.openstack.org/44896911:51
openstackgerritzhongjun proposed openstack/manila-specs master: Add spec for infinite share  https://review.openstack.org/45209711:57
openstackgerritMerged openstack/manila master: Updated from global requirements  https://review.openstack.org/45388212:09
*** gouthamr has joined #openstack-manila12:28
openstackgerritzhongjun proposed openstack/manila-specs master: Allow to filter resources by inexact value matching in 'list' APIs  https://review.openstack.org/44777512:30
vponomaryovzhongjun_: any other "stable" branch is mutable too12:30
zhongjun_vponomaryov:I think I could just remove it12:31
vponomaryovzhongjun_: you could just use immutable one based on commit id12:31
zhongjun_vponomaryov: The links are not very important12:32
openstackgerritzhongjun proposed openstack/manila-specs master: Support retrieve shares filtered by export-location  https://review.openstack.org/44896912:33
openstackgerritzhongjun proposed openstack/manila-specs master: Support retrieve shares filtered by export-location  https://review.openstack.org/44896912:34
*** catintheroof has joined #openstack-manila12:36
*** dsariel has quit IRC12:41
openstackgerritzhongjun proposed openstack/manila-specs master: Enhance update shares  https://review.openstack.org/45355312:46
*** gcb has joined #openstack-manila12:50
*** dustins has joined #openstack-manila12:51
vponomaryovzhongjun_: set +2, keeping hope in heart to see appropriate links when they appear in future12:53
zhongjun_vponomaryov: Thanks :) I will12:54
vponomaryovexample: https://github.com/openstack/manila/blob/31a19b1f/manila/api/v2/share_instances.py#L9012:54
vponomaryovit is immutable link12:54
*** dsariel has joined #openstack-manila12:54
*** cknight has joined #openstack-manila12:55
zhongjun_vponomaryov: lol12:56
zhongjun_tbarron: hi,  " A sequence diagram would be quite helpful here I think."  What kind of sequence diagram? Is there an example?12:57
zhongjun_tbarron: link: https://review.openstack.org/#/c/446494/20/specs/pike/ensure-share.rst12:58
*** xyang1 has joined #openstack-manila12:58
vponomaryovzhongjun_: about https://review.openstack.org/#/c/453553/2..3/specs/pike/enhance-update-shares.rst13:01
vponomaryovzhongjun_: you cannot add new sections13:01
vponomaryovzhongjun_: we have template13:02
vponomaryovzhongjun_: it should be common for all13:02
zhongjun_vponomaryov: oh13:02
vponomaryovzhongjun_: second, other existing comments are not fully addressed13:02
vponomaryovzhongjun_: amount of commit updates should be minimized as much as possible13:02
vponomaryovzhongjun_: else you make reviewer life nightmare to track down all discussions about all concerns13:03
zhongjun_vponomaryov: Which part is suitable for workflow example13:04
vponomaryovzhongjun_: how do you think?13:04
zhongjun_vponomaryov: I think "Proposed change" is suitable :)13:06
vponomaryovzhongjun_: why?13:06
zhongjun_ vponomaryov: Because it is a part of proposed change13:06
zhongjun_vponomaryov: we still change the workflow13:07
vponomaryovzhongjun_: look, your spec https://review.openstack.org/#/c/453553/ means following: run "ensure_share" using periodic task and stop running it on manila-share service start-up13:08
zhongjun_vponomaryov: The spec don't means stop running it on manila-share service start-up13:09
vponomaryovzhongjun_: is it intended?13:10
vponomaryovs/intended/intentional/13:10
zhongjun_vponomaryov: maybe I did not express the intention well13:11
zhongjun_vponomaryov:  We can support run "ensure_share" using periodic task   and  running it on manila-share service start-up13:12
vponomaryovzhongjun_: anyway, need to describe how and when it will be called now compared to old approach and why13:12
zhongjun_vponomaryov: Cool13:13
zhongjun_vponomaryov: lol13:15
vponomaryovzhongjun_: ?13:15
zhongjun_vponomaryov: I agree with you13:15
vponomaryovzhongjun_: the main problem is that you do not imagine whole picture of your personal proposal, I do such statement reading spec and looking at changes that "address" comments. If you do not imagine, who else should do it?13:17
vponomaryovzhongjun_: If you really want to keep "workflow" then it is a point for "use cases" section13:19
vponomaryovzhongjun_: the idea of it13:19
*** Administrator_ has quit IRC13:23
*** Administrator_ has joined #openstack-manila13:24
zhongjun_vponomaryov: It was a complete spec in the past,  I separeted it(enhance-update-shares) from ensure_share spec. It is too many mixed together, I will focus more on that.13:25
*** eharney has joined #openstack-manila13:26
*** mtanino has quit IRC13:28
*** chlong has joined #openstack-manila13:31
*** openstackgerrit has quit IRC13:33
*** porrua has joined #openstack-manila13:35
*** cknight has quit IRC13:47
*** cknight has joined #openstack-manila13:48
*** hongbin has joined #openstack-manila14:01
*** openstackgerrit has joined #openstack-manila14:04
openstackgerritJan Provaznik proposed openstack/manila-specs master: Add spec for user messages  https://review.openstack.org/43427714:04
*** a-pugachev has quit IRC14:11
*** a-pugachev has joined #openstack-manila14:12
*** mtanino has joined #openstack-manila14:16
*** esker has joined #openstack-manila14:25
*** markstur has joined #openstack-manila14:27
*** markstur has quit IRC14:31
*** belmoreira has quit IRC14:46
*** scottda has joined #openstack-manila14:57
*** carthaca_ has joined #openstack-manila15:05
*** cknight has quit IRC15:34
*** dsariel has quit IRC15:38
*** esker has quit IRC15:40
*** makowals_ has joined #openstack-manila15:41
*** chenying_ has quit IRC15:43
*** makowals has quit IRC15:43
*** chenying_ has joined #openstack-manila15:44
*** lpetrut has quit IRC15:47
*** esker has joined #openstack-manila15:52
openstackgerritzhongjun proposed openstack/manila-specs master: Allow to filter resources by inexact value matching in 'list' APIs  https://review.openstack.org/44777515:52
openstackgerritMerged openstack/puppet-manila master: oslo db: check puppet resource instead of actual config in spec  https://review.openstack.org/45527415:58
openstackgerritzhongjun proposed openstack/manila-specs master: Allow to filter resources by inexact value matching in 'list' APIs  https://review.openstack.org/44777516:10
*** Zhongjun__ has joined #openstack-manila16:11
*** esker has quit IRC16:33
openstackgerritzhongjun proposed openstack/manila-specs master: Add spec for infinite share  https://review.openstack.org/45209716:37
*** a-pugachev has quit IRC16:44
openstackgerritValeriy Ponomaryov proposed openstack/manila master: [WIP] Add quotas per share type support  https://review.openstack.org/45215816:51
*** markstur has joined #openstack-manila16:53
openstackgerritzhongjun proposed openstack/manila-specs master: Add spec for infinite share  https://review.openstack.org/45209716:57
*** markstur has quit IRC16:58
*** chenying_ has quit IRC16:58
*** chenying_ has joined #openstack-manila16:58
openstackgerritValeriy Ponomaryov proposed openstack/manila master: [WIP] Add quotas per share type support  https://review.openstack.org/45215817:08
*** kaisers has quit IRC17:09
*** kaisers has joined #openstack-manila17:09
*** kaisers has quit IRC17:11
*** kaisers has joined #openstack-manila17:12
*** kaisers has quit IRC17:12
*** kaisers has joined #openstack-manila17:14
*** ociuhandu has quit IRC17:39
openstackgerritzhongjun proposed openstack/manila-specs master: Add spec for change ensure share to make startup faster  https://review.openstack.org/44649417:51
*** kaisers has quit IRC17:51
*** kaisers has joined #openstack-manila17:52
*** kaisers has quit IRC17:56
*** cknight has joined #openstack-manila18:00
openstackgerritzhongjun proposed openstack/manila-specs master: Add spec for infinite share  https://review.openstack.org/45209718:01
*** ociuhandu has joined #openstack-manila18:04
*** cknight has quit IRC18:21
openstackgerritJan Provaznik proposed openstack/manila-specs master: Add spec for user messages  https://review.openstack.org/43427718:22
*** a-pugachev has joined #openstack-manila18:40
*** jprovazn has quit IRC18:50
*** cknight has joined #openstack-manila18:51
*** chatter29 has joined #openstack-manila19:07
chatter29hey guys19:07
chatter29allah is doing19:07
chatter29sun is not doing allah is doing19:07
chatter29to accept Islam say that i bear witness that there is no deity worthy of worship except Allah and Muhammad peace be upon him is his slave and messenger19:08
*** cknight has quit IRC19:08
*** cknight has joined #openstack-manila19:12
*** chatter29 has quit IRC19:18
*** cknight has quit IRC19:18
*** zhonghua has quit IRC19:37
*** zhonghua has joined #openstack-manila19:38
*** gouthamr has quit IRC19:54
*** porrua has quit IRC19:57
*** Zhongjun__ has quit IRC20:11
*** markstur has joined #openstack-manila20:29
*** markstur has quit IRC20:34
*** dustins has quit IRC20:55
*** pcaruana has quit IRC21:15
*** catintheroof has quit IRC21:25
*** eharney has quit IRC21:34
*** a-pugachev has quit IRC21:58
*** xyang1 has quit IRC22:08
*** hoonetorg has quit IRC22:16
*** zhonghua has quit IRC22:22
*** zhonghua has joined #openstack-manila22:23
*** ganso has quit IRC22:26
*** catintheroof has joined #openstack-manila22:27
*** gouthamr has joined #openstack-manila22:31
*** hoonetorg has joined #openstack-manila22:39
*** scottda has quit IRC23:15
*** zhonghua has quit IRC23:23
*** zhonghua has joined #openstack-manila23:24

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