Monday, 2016-08-01

*** yangyapeng has quit IRC00:03
*** chlong has joined #openstack-manila00:26
*** chlong has quit IRC00:30
*** chlong has joined #openstack-manila00:41
*** yangyapeng has joined #openstack-manila01:09
*** yangyapeng has quit IRC01:09
*** yangyapeng has joined #openstack-manila01:10
*** chlong has quit IRC01:10
*** cknight has joined #openstack-manila01:17
*** chlong has joined #openstack-manila01:23
*** chlong has quit IRC01:28
*** chlong has joined #openstack-manila01:30
*** liyifeng has joined #openstack-manila01:52
*** yangyape_ has joined #openstack-manila02:03
*** yangyapeng has quit IRC02:03
*** cknight has quit IRC02:56
*** amitkqed has quit IRC03:38
*** amitkqed has joined #openstack-manila03:38
*** yangyape_ has quit IRC03:54
*** yangyapeng has joined #openstack-manila03:55
*** nidhimittalhada has joined #openstack-manila03:58
*** shausy has joined #openstack-manila03:58
nidhimittalhada@vponomaryov @gouthamr @cknight @markstur https://review.openstack.org/#/c/315012/  kindly have a look for review03:58
*** chlong has quit IRC04:15
*** chlong has joined #openstack-manila04:31
*** gouthamr has quit IRC04:35
*** rraja has joined #openstack-manila04:44
*** lpetrut has joined #openstack-manila05:20
*** nidhimittalhada has quit IRC06:17
*** dsariel has joined #openstack-manila06:18
*** nkrinner_afk is now known as nkrinner06:26
*** lpetrut has quit IRC06:27
openstackgerritMarc Koderer proposed openstack/manila: Make port_binding_extension mandatory if host_id is specified  https://review.openstack.org/34132706:35
*** chlong has quit IRC06:59
*** chlong has joined #openstack-manila07:11
*** chlong has quit IRC07:34
openstackgerritOpenStack Proposal Bot proposed openstack/manila-ui: Imported Translations from Zanata  https://review.openstack.org/34940508:31
*** akapil has joined #openstack-manila09:22
*** cbader has quit IRC09:25
*** cbader has joined #openstack-manila09:26
*** akapil has quit IRC09:26
*** akapil has joined #openstack-manila09:46
*** akapil has quit IRC09:50
openstackgerritzhongjun proposed openstack/python-manilaclient: Add snapshot instances admin APIs  https://review.openstack.org/30444910:11
*** yangyapeng has quit IRC10:21
*** lpetrut has joined #openstack-manila10:34
*** zhonghua-lee has joined #openstack-manila11:02
*** ganso has joined #openstack-manila11:17
*** alyson_ has joined #openstack-manila11:19
*** akapil has joined #openstack-manila11:25
*** akapil has quit IRC11:26
*** akapil has joined #openstack-manila11:30
openstackgerritMarc Koderer proposed openstack/manila: Add neutron driver for binding  https://review.openstack.org/28349411:35
openstackgerritzhongjun proposed openstack/manila: Implement share backup  https://review.openstack.org/34398011:44
zhongjun_ganso: ping11:52
gansozhongjun_: pong11:52
zhongjun_ganso: hi11:52
gansozhongjun_: hello11:52
zhongjun_ganso: In https://review.openstack.org/#/c/343980/9/manila/share/manager.py code line 293011:52
zhongjun_ganso: Does you means: Don't use the preserve_metadata to decide witch service should do backup job?11:53
gansozhongjun_: yes11:53
gansozhongjun_: there are related comments in share/api.py as well11:54
*** tpsilva has joined #openstack-manila11:54
zhongjun_ganso: What is your opinion?11:55
gansozhongjun_: you are assuming if that if the user chooses to preserve-metadata, you will choose the driver approach and not the data-service11:56
zhongjun_ganso: How about I use share_service first, if the share_service can not support the backup, then I use the data_service to do backup job?11:56
gansozhongjun_: it is highly likely that the users will always choose to preserve-metadata11:56
gansozhongjun_: yes11:56
gansozhongjun_: the code you have today will display more failures every time preserve-metadata is true, users will prefer it, then, we may not have driver implementation, and we may have driver implementation not supporting preserve-metadata11:57
gansozhongjun_: so my suggestion is to try driver approach first always11:58
gansozhongjun_: similar to migration11:58
openstackgerritMarc Koderer proposed openstack/manila: Add binding_profile option for backends  https://review.openstack.org/28403411:58
openstackgerritMarc Koderer proposed openstack/manila: Add multi-segment support  https://review.openstack.org/27773111:58
openstackgerritMarc Koderer proposed openstack/manila: Add neutron driver for binding  https://review.openstack.org/28349411:58
*** timcl has joined #openstack-manila12:02
zhongjun_ganso: OK, I try to modify the code according to this logic12:02
*** yangyapeng has joined #openstack-manila12:02
openstackgerritMerged openstack/manila-ui: Imported Translations from Zanata  https://review.openstack.org/34940512:08
*** akapil has quit IRC12:08
*** akshai has joined #openstack-manila12:12
*** akapil has joined #openstack-manila12:13
openstackgerritzhongjun proposed openstack/manila: Implement share backup  https://review.openstack.org/34398012:14
openstackgerritzhongjun proposed openstack/manila: Implement share backup  https://review.openstack.org/34398012:16
*** gouthamr has joined #openstack-manila12:19
*** scottda_vacation is now known as scottda12:33
*** liyifeng has quit IRC12:33
*** mtanino has joined #openstack-manila13:00
openstackgerritCsaba Henk proposed openstack/manila: glusterfs-native: use Heketi layout via self-signing  https://review.openstack.org/28210113:02
openstackgerritCsaba Henk proposed openstack/manila: glusterfs: heketi: Add support for JWT Auth  https://review.openstack.org/28048613:02
openstackgerritCsaba Henk proposed openstack/manila: glusterfs: introducing Heketi layout  https://review.openstack.org/27909013:02
*** bill_az has joined #openstack-manila13:06
*** xyang1 has joined #openstack-manila13:07
openstackgerritRamana Raja proposed openstack/manila: add access_key to share_access_map  https://review.openstack.org/34330613:07
*** faiz89 has joined #openstack-manila13:11
*** akapil has quit IRC13:13
*** akapil has joined #openstack-manila13:13
*** dustins has joined #openstack-manila13:21
tpsilvagouthamr: ping13:22
gouthamrtpsilva: hey Tiago..13:22
tpsilvagouthamr: hey Goutham, good morning13:22
tpsilvagouthamr: bothering you again :P13:23
gouthamrtpsilva: Goodmorning! nope :)13:23
tpsilvagouthamr: I'm having a hard time changing that snapshot access_list from POST to GET13:24
tpsilvagouthamr: I need to add that new method to router.py right?13:24
gouthamrtpsilva: yep.. it can no longer be an action..13:24
gouthamrtpsilva: what's your current thought on the endpoint? there're a few examples in v2/router.py13:26
tpsilvagouthamr: /snapshots/{snapshot_id}/access_list?13:28
tpsilvano wait13:29
tpsilvagouthamr: yeah I figured what's wrong13:30
gouthamrtpsilva: good stuff.. however, /snapshots/{snapshot_id}/access-list instead..13:30
tpsilvagouthamr: yep13:30
*** merooney has joined #openstack-manila13:30
tpsilvagouthamr: just a sec13:30
*** porrua has joined #openstack-manila13:32
openstackgerritMarc Koderer proposed openstack/manila: Add MTU information in DB and API  https://review.openstack.org/34950613:35
tpsilvagouthamr: ok, so, the problem is, how exactly I map this to my method on the controller?13:38
tpsilvagouthamr: the examples on router.py are mapping to index, detail, etc, which are default GET methods13:38
gouthamrtpsilva: https://github.com/openstack/manila/blob/master/manila/api/v2/router.py#L161 <-- example13:39
gouthamrtpsilva: in your case action could be 'access_list', which will be the name of your method in the API module..13:41
*** dustins has quit IRC13:42
tpsilvagouthamr: ok, so action is the name of my method on the controller?13:43
*** shausy has quit IRC13:43
gouthamrtpsilva: yep13:44
tpsilvagouthamr: oh, right, cause I defined it on the decorator13:45
tpsilvagouthamr: cool13:45
gouthamrtpsilva: you won't need the wsgi.action decorator anymore...13:46
tpsilvagouthamr: that's what I thought, but I get a not found error13:46
gouthamrtpsilva: this won't be a /action API13:46
rrajaganso: hi! about your review comment on https://review.openstack.org/#/c/343306/ about having a "view" for share_accesses. were you looking for this, https://review.openstack.org/#/c/343306/7/manila/api/views/share_accesses.py?13:50
*** akerr has joined #openstack-manila13:52
*** yangyapeng has quit IRC13:53
*** yangyapeng has joined #openstack-manila13:53
*** ganso has quit IRC14:03
*** amit213 has quit IRC14:04
*** ameade has quit IRC14:04
*** DuncanT has quit IRC14:04
*** tpsilva has quit IRC14:04
*** erlon has quit IRC14:04
*** scottda has quit IRC14:04
*** rhefner has quit IRC14:04
*** draynium has quit IRC14:08
*** faiz89 has quit IRC14:11
openstackgerritGaurang Tapase proposed openstack/manila: Refactor GPFS driver for NFS ganesha support  https://review.openstack.org/32882914:12
*** faiz89 has joined #openstack-manila14:13
*** draynium has joined #openstack-manila14:15
*** tpsilva has joined #openstack-manila14:15
*** ganso has joined #openstack-manila14:16
*** dustins has joined #openstack-manila14:16
*** erlon has joined #openstack-manila14:18
*** DuncanT has joined #openstack-manila14:18
*** timcl has quit IRC14:18
*** amit213 has joined #openstack-manila14:19
*** rhefner has joined #openstack-manila14:19
*** scottda has joined #openstack-manila14:19
*** ameade has joined #openstack-manila14:19
*** akapil has quit IRC14:20
*** dustins_ has joined #openstack-manila14:22
*** dustins has quit IRC14:22
*** akapil has joined #openstack-manila14:23
*** tpsilva_ has joined #openstack-manila14:24
*** chlong has joined #openstack-manila14:26
*** eharney has joined #openstack-manila14:26
*** chlong is now known as chlong|gone14:27
*** chlong|gone has quit IRC14:32
*** chlong|gone has joined #openstack-manila14:33
*** chlong|gone is now known as chlong14:34
*** timcl has joined #openstack-manila14:35
*** nkrinner is now known as nkrinner_afk14:59
*** MikeG451 has quit IRC15:00
*** pgbridge has joined #openstack-manila15:13
*** akapil has quit IRC15:23
*** faiz89 has quit IRC15:24
*** timcl has quit IRC15:29
*** faiz89 has joined #openstack-manila15:29
*** MikeG451 has joined #openstack-manila15:42
openstackgerritValeriy Ponomaryov proposed openstack/manila: [ZFSonLinux] Fix snapshot deletion error  https://review.openstack.org/34957515:48
openstackgerritValeriy Ponomaryov proposed openstack/manila: [ZFSonLinux] Fix replicated snapshot deletion error  https://review.openstack.org/34957515:49
*** rraja has quit IRC15:52
*** faiz89 has quit IRC16:06
*** eharney has quit IRC16:07
*** timcl has joined #openstack-manila16:10
*** tpsilva_ has quit IRC16:18
openstackgerritMerged openstack/manila: Add snapshot instances admin APIs  https://review.openstack.org/23465816:39
*** vbellur has joined #openstack-manila16:52
openstackgerritValeriy Ponomaryov proposed openstack/manila: [Tempest] Fix concurrency in "test_show_share_server" test  https://review.openstack.org/34959917:01
vponomaryovganso: ^17:02
vponomaryovit is fix of mentioned new bug in https://review.openstack.org/#/c/348829/17:02
gansovponomaryov: great! thanks a lot :)17:02
gansovponomaryov: will wait for jenkins and then review17:02
vponomaryovganso: you don't beleive me? ))17:03
vponomaryovganso: hurry up, amount of reviews is limited!17:03
gansovponomaryov: lol17:03
gansovponomaryov: the earlier it is fixed, the earlier it is more stable for other patches17:03
gansovponomaryov: code looks alright17:04
*** cbader has quit IRC17:04
vponomaryovganso: jenkins is very busy today, too many visitors17:05
vponomaryovganso: you will wait very long time, don't you want to make a non-shaving bet? ))17:05
vponomaryovand come to summit with huge beard17:06
gansovponomaryov: isn't it "point-shaving" ? I just googled for "non-shaving bet" and I found things related to... shaving17:06
gansovponomaryov: oh17:07
gansovponomaryov: lol17:07
gansovponomaryov: now I get it17:07
gansovponomaryov: definitely not17:07
vponomaryovmustaches?17:07
vponomaryovat least17:08
gansovponomaryov: that would make my life difficult in other situations xD17:08
*** Suyi_ has joined #openstack-manila17:08
vponomaryovganso: then just believe me ))17:08
gansovponomaryov: I do believe you, I just don't like skipping jenkins :)17:09
*** cknight has joined #openstack-manila17:09
*** lpetrut has quit IRC17:14
*** cknight1 has joined #openstack-manila17:31
*** dsariel has quit IRC17:33
*** cknight has quit IRC17:35
*** eharney has joined #openstack-manila17:45
*** adrianofr has joined #openstack-manila17:48
*** vbellur1 has joined #openstack-manila17:59
*** vbellur has quit IRC17:59
openstackgerritMerged openstack/manila: Replaces httplib with requests lib in Quobyte RPC layer  https://review.openstack.org/34521918:08
*** lpetrut has joined #openstack-manila18:13
*** gouthamr has quit IRC18:20
*** permalac has joined #openstack-manila18:23
*** faiz89 has joined #openstack-manila18:25
*** vbellur1 has quit IRC18:30
*** gouthamr has joined #openstack-manila18:34
*** dustins_ is now known as dustins18:37
*** vbellur has joined #openstack-manila18:48
*** JoseMello has joined #openstack-manila18:49
*** lpetrut has quit IRC18:56
openstackgerritTiago Pasqualini da Silva proposed openstack/manila: Add mountable snapshots support  https://review.openstack.org/34552619:04
*** lpetrut has joined #openstack-manila19:12
*** harlowja has quit IRC19:14
*** harlowja has joined #openstack-manila19:15
openstackgerritGoutham Pacha Ravi proposed openstack/manila: NetApp cDOT vserver deletion fails if no lifs present  https://review.openstack.org/34891219:22
openstackgerritClinton Knight proposed openstack/manila: Implement share revert to snapshot  https://review.openstack.org/34050219:25
*** vbellur has quit IRC19:27
openstackgerritClinton Knight proposed openstack/python-manilaclient: Support share revert to snapshot in Manila client  https://review.openstack.org/34055619:28
*** cknight1 has quit IRC19:41
*** cknight has joined #openstack-manila19:41
*** tpsilva has quit IRC19:42
*** vbellur has joined #openstack-manila19:43
*** eharney has quit IRC19:48
*** eharney has joined #openstack-manila19:49
*** porrua has quit IRC20:07
*** merooney has quit IRC20:15
*** akerr has quit IRC20:16
*** cknight1 has joined #openstack-manila20:21
*** cknight has quit IRC20:23
*** tpsilva has joined #openstack-manila20:24
*** tpsilva has quit IRC20:32
*** timcl has quit IRC20:36
*** vbellur has quit IRC20:37
*** tpsilva has joined #openstack-manila20:44
*** akshai has quit IRC20:57
jseilerHi, I'm developing a new driver and am running the tempest tests.21:03
jseilerIn manila_tempest_tests.tests.api.test_rules.ShareIpRulesForNFSTest, the test - test_create_delete_access_rule_with_cidr is failing because of an error is reported by our file system.21:04
jseilerThe test is using an IP address with cidr notation of '1.2.3.4/32'.21:04
jseilerThe error from our file system is - "The network prefix length provided '32' is invalid or not applicable for networkId '1.2.3.4'. The prefix should be in range 1..31 for IPv4 or 1..127 for IPv6 network."21:04
jseilerSo, our file system is expecting the network prefix to be in the range of 1..3121:04
jseilerI'm not going to claim to be an IP Address guru, and I'm not any good at wading through the RFCs. But the table in this wiki article seems to indicate a prefix of /32 is valid - https://en.wikipedia.org/wiki/Classless_Inter-Domain_Routing21:04
jseilerA couple of questions (because I'm guessing my file system developers will ask)21:04
jseiler1. Is there a valid use case for an IP address with a network prevfix of 32 (i.e., meaning there will be only one host in a network).21:05
jseiler2. Do all the other file system vendors support a network prefix of 32 for IPv4 addresses?21:05
*** cknight1 has quit IRC21:06
openstackgerritTiago Pasqualini da Silva proposed openstack/manila: Add Hitachi HSP driver  https://review.openstack.org/32913421:18
openstackgerritTiago Pasqualini da Silva proposed openstack/manila: Add Hitachi HSP driver  https://review.openstack.org/32913421:18
tpsilvajseiler: you can disable this test for your CI21:22
tpsilvajseiler: if your backend does not support it21:22
jseilertpsilva: As a test, I changed the IP Address in the test to '1.2.3.4/31', then the test and the other IP Address access tests passed.21:23
*** MikeG451 has quit IRC21:23
tpsilvajseiler: you can disable this single test (test_create_delete_access_rule_with_cidr) on your vendor CI21:24
tpsilvajseiler: since your backend does not support /32 CIDRs21:24
tpsilvajseiler: I don't know how to do that though... someone else here can help you21:25
tpsilvajseiler: maybe ganso?21:25
tpsilvaganso: ^21:25
*** akapil has joined #openstack-manila21:25
jseilertpsilva: OK. Thanks. I'm new to this. I was thinking that our driver would have to pass all the tempest tests.  I did not realize it was an option to disable a single test (at least as far as having a reporting CI goes).21:27
tpsilvajseiler: yes, there is an option that you can configure to disable some tests21:27
openstackgerritAlex O'Rourke proposed openstack/manila: 3PAR: Add update_access support  https://review.openstack.org/33979521:28
jseilertpsilva: I did see the manila.conf [share] option - enable_ip_rules_for_protocols = nfs.  That looks like it would disable all IP rules tests for NFS.21:29
tpsilvajseiler: yes, that's not the option21:29
tpsilvajseiler: just a sec, I can help you in a bit21:29
jseilertpsilva: Thank you21:29
openstackgerritTiago Pasqualini da Silva proposed openstack/manila: Add mountable snapshots support  https://review.openstack.org/34552621:30
tpsilvajseiler: sorry, I can't find the option and my colleagues already left21:36
tpsilvajseiler: I can find out this option tomorrow morning21:36
tpsilvajseiler: I'll ping you then :)21:36
jseilertpsilva: OK. Thank you and thanks for checking.21:37
jseilertpsilva: I'm still developing the driver and am currently running the tempest tests to check functionality. We're not at the point of setting up the CI yet.21:37
tpsilvajseiler: alright... is this for newton?21:38
jseilertpsilva: No. We're targeting ocata.21:38
tpsilvajseiler: ok, the deadling for newton is close21:38
tpsilvas/deadling/deadline21:39
jseilertpsilva: We originally hoped for Newton, but we have other dependencies we have to get into our file system before we can support manila21:39
jseilertpsilva:  right. Newton deadline is close.  We're hoping for early ocata. (the earlier the better - right)21:39
jseilertpsilva: I realize this may be too early for ocata.21:40
tpsilvajseiler: but you're right, the earlier the better21:40
*** JoseMello has quit IRC21:40
jseilertpsilva: thanks for your help21:40
tpsilvajseiler: no problem21:40
*** akapil has quit IRC21:42
*** eharney has quit IRC21:44
*** lpetrut has quit IRC21:46
openstackgerritDustin Schoenbrun proposed openstack/manila: Check for usage of same Cephx ID as manila service  https://review.openstack.org/34971822:01
*** MikeG451 has joined #openstack-manila22:03
openstackgerritDustin Schoenbrun proposed openstack/manila: Check for usage of same Cephx ID as manila service  https://review.openstack.org/34971822:04
*** dustins has quit IRC22:05
*** tpsilva has quit IRC22:05
*** alkhodos has joined #openstack-manila22:16
*** xyang1 has quit IRC22:21
*** akapil has joined #openstack-manila22:25
openstackgerritAlexey Khodos proposed openstack/manila: Nexenta: adding share drivers for NexentaStor  https://review.openstack.org/30928622:28
*** akapil has quit IRC22:29
jseilertpsilva: I think I figured out my problem with the 1.2.3.4/32 address from the tempest test. It's really just an access for a single IP address. Our file system supports that - I was sending it down wrong.22:53
jseilertpsilva: So I probably won't need to know how to disable that specific test right now. But if you do find out, I'd still appreciate knowing how to do so.22:56
jseilertpsilva: I'm having problems running the scenario tests. Our driver does not support snapshots yet. And an error in the scheduler seems to be pointing towards a request being made for a backend that supports_snapshots=true.22:57
*** liyifeng has joined #openstack-manila22:57
*** alyson_ has quit IRC23:03
*** tpsilva has joined #openstack-manila23:07
openstackgerritTiago Pasqualini da Silva proposed openstack/manila: Add Hitachi HSP driver  https://review.openstack.org/32913423:14
*** gouthamr has quit IRC23:15
*** faiz89 has quit IRC23:17
openstackgerritRodrigo Barbieri proposed openstack/manila: Share migration Newton improvements  https://review.openstack.org/32843123:37
openstackgerritRodrigo Barbieri proposed openstack/manila: Fix Share Migration improper behavior for drivers  https://review.openstack.org/33226723:37
openstackgerritRodrigo Barbieri proposed openstack/manila: Add share type change to Share Migration  https://review.openstack.org/34514223:37
openstackgerritRodrigo Barbieri proposed openstack/manila: Base Data Copy changes  https://review.openstack.org/32917423:39
openstackgerritRodrigo Barbieri proposed openstack/manila: Driver Data Service Helper  https://review.openstack.org/29583623:39
gansojseiler: Hi. Sorry for noticing it late23:42
gansojseiler: you should not be disabling individual tempest tests23:42
gansojseiler: it is possible to disable individual tempest tests through regex, but your driver should have a CI and have them all enabled, for the best OpenStack consistency across different vendors, we advise to try handling in your driver whenever possible23:43
gansojseiler: regarding "supports_snapshot", your driver should report the appropriate capability value, which is False in your case, and when running tempest you should disable snapshot_support as well23:44
gansojseiler: see manila_tempest_tests/config.py23:44
*** alkhodos has quit IRC23:49

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