Wednesday, 2016-07-27

*** Suyi_ has quit IRC00:04
*** MikeG451_ has quit IRC00:12
*** gouthamr has joined #openstack-manila00:17
*** erlon has joined #openstack-manila00:28
*** sjjfowler has joined #openstack-manila00:45
*** mtanin___ has quit IRC00:48
*** vbellur has joined #openstack-manila00:49
*** huyang has joined #openstack-manila00:59
*** cknight1 has joined #openstack-manila01:00
*** cknight has quit IRC01:02
*** kaisers_ has joined #openstack-manila01:06
*** kaisers_ has quit IRC01:11
*** yangyapeng has joined #openstack-manila01:11
*** zengyingzhe_ has quit IRC01:29
*** openstackgerrit has quit IRC01:33
*** gouthamr_ has joined #openstack-manila01:34
*** openstackgerrit has joined #openstack-manila01:34
*** gouthamr has quit IRC01:37
*** yangyapeng has quit IRC01:44
*** yangyapeng has joined #openstack-manila01:44
*** Suyi_ has joined #openstack-manila02:12
*** yangyapeng has quit IRC02:30
*** gouthamr_ has quit IRC02:54
*** kaisers_ has joined #openstack-manila02:55
*** cknight1 has quit IRC02:59
*** kaisers_ has quit IRC02:59
*** cknight has joined #openstack-manila03:01
*** cknight has quit IRC03:14
*** yangyapeng has joined #openstack-manila03:22
*** erlon has quit IRC03:26
*** yangyape_ has joined #openstack-manila03:36
*** yangyapeng has quit IRC03:39
*** zhongjun_ has quit IRC03:49
*** zhonghua has quit IRC03:52
*** zhongjun_ has joined #openstack-manila03:53
*** zhonghua-lee has joined #openstack-manila03:53
*** lpetrut has joined #openstack-manila03:59
*** kaisers_ has joined #openstack-manila04:07
*** kaisers_ has quit IRC04:07
*** sandanar has joined #openstack-manila04:09
*** lpetrut has quit IRC04:30
*** sandanar has quit IRC04:44
*** sandanar has joined #openstack-manila05:53
*** nkrinner_afk is now known as nkrinner06:02
*** Suyi_ has quit IRC06:05
*** amitkqed has joined #openstack-manila06:35
*** amitkqed has quit IRC06:36
*** amitkqed has joined #openstack-manila06:37
openstackgerritAlexey Ovchinnikov proposed openstack/manila: [DNM] Tests for scenario tests  https://review.openstack.org/34023606:43
*** ameade has quit IRC06:49
*** rhefner has quit IRC06:49
*** rhefner has joined #openstack-manila06:50
*** amit213 has quit IRC06:50
*** adrianofr has quit IRC06:51
*** amit213 has joined #openstack-manila06:51
*** alyson_ has quit IRC06:52
*** ameade has joined #openstack-manila06:52
*** adrianofr has joined #openstack-manila06:53
*** alyson_ has joined #openstack-manila06:53
*** nkrinner has quit IRC06:54
*** sjjfowler has quit IRC06:55
*** nkrinner has joined #openstack-manila06:57
*** nkrinner has quit IRC07:05
openstackgerritSilvan Kaiser proposed openstack/manila: Minor optimization and formatting corrections in Quobyte driver  https://review.openstack.org/34529007:07
*** vponomaryov1 has joined #openstack-manila07:14
*** nkrinner has joined #openstack-manila07:17
openstackgerritValeriy Ponomaryov proposed openstack/manila: [DNM] Test scenario tests for ZFSonLinux driver  https://review.openstack.org/34745007:20
*** tovchinnikova has joined #openstack-manila07:38
openstackgerritValeriy Ponomaryov proposed openstack/manila: [ZFSonLinux] Add 'manage snapshot' feature support  https://review.openstack.org/33722907:51
openstackgerritAlexey Ovchinnikov proposed openstack/manila: [DNM] Tests for scenario tests  https://review.openstack.org/34023608:00
*** akapil has joined #openstack-manila08:02
*** akapil has quit IRC08:03
*** akapil has joined #openstack-manila08:03
*** akapil has quit IRC08:07
*** pgbridge has quit IRC08:09
*** pgbridge has joined #openstack-manila08:10
*** akapil has joined #openstack-manila08:11
*** hoonetorg has quit IRC08:13
*** hoonetorg has joined #openstack-manila08:17
openstackgerritGaurang Tapase proposed openstack/manila: Refactor GPFS driver for NFS ganesha support  https://review.openstack.org/32882908:24
*** gaurangt has joined #openstack-manila08:38
openstackgerritTatiana Ovchinnikova proposed openstack/manila-ui: Table actions improvement  https://review.openstack.org/34076808:45
*** yangyape_ has quit IRC08:45
*** yangyapeng has joined #openstack-manila08:46
openstackgerritzhongjun proposed openstack/python-manilaclient: Implement share backup  https://review.openstack.org/34467108:52
*** jcsp has joined #openstack-manila08:56
openstackgerritValeriy Ponomaryov proposed openstack/manila: [DNM] Test scenario tests for ZFSonLinux driver  https://review.openstack.org/34745008:58
*** sticker has quit IRC09:13
*** sjjfowler has joined #openstack-manila09:13
*** sticker has joined #openstack-manila09:15
*** nherciu has joined #openstack-manila09:31
*** yangyapeng has quit IRC09:34
*** yangyapeng has joined #openstack-manila09:35
*** yangyapeng has quit IRC09:40
*** yangyapeng has joined #openstack-manila09:42
*** rraja has joined #openstack-manila09:43
openstackgerritTatiana Ovchinnikova proposed openstack/manila-ui: Table actions improvement  https://review.openstack.org/34076809:47
*** chlong_POffice has quit IRC09:57
*** chlong_POffice has joined #openstack-manila10:10
*** yangyapeng has quit IRC10:12
*** furlongm has quit IRC10:15
*** furlongm has joined #openstack-manila10:16
*** tovchinnikova has quit IRC10:21
*** ociuhandu has joined #openstack-manila10:30
*** akapil has quit IRC10:48
*** akapil has joined #openstack-manila11:04
*** tpsilva has joined #openstack-manila11:05
*** akapil has quit IRC11:09
*** gouthamr has joined #openstack-manila11:14
*** akapil has joined #openstack-manila11:16
*** gouthamr_ has joined #openstack-manila11:21
*** gouthamr has quit IRC11:24
openstackgerritNidhiMittalHada proposed openstack/python-manilaclient: type-create should support specifying extra-specs  https://review.openstack.org/31501211:27
tpsilvavponomaryov: ping11:27
vponomaryov1tpsilva: pong11:29
tpsilvavponomaryov1: would it be OK to push this (https://review.openstack.org/#/c/347529/) without the tempest coverage? :)11:30
*** gouthamr_ is now known as gouthamr11:30
vponomaryov1tpsilva: it makes some problems?11:31
tpsilvavponomaryov1: oh wait, if I bump the microversion, I will automatically need the tempest tests, right?11:31
vponomaryov1tpsilva: yes, cover that diff11:31
tpsilvavponomaryov1: yes, nvm then, I'll make that11:32
vponomaryov1tpsilva: I believe in you11:32
vponomaryov1=)11:32
tpsilvavponomaryov1: thought it was a quick fix, turns out I was wrong11:32
*** ociuhandu has quit IRC11:33
*** ociuhandu has joined #openstack-manila11:35
*** erlon has joined #openstack-manila11:51
*** gaurangt has left #openstack-manila11:54
openstackgerritRodrigo Barbieri proposed openstack/manila: Fix Share Migration improper behavior for drivers  https://review.openstack.org/33226712:00
openstackgerritMarc Koderer proposed openstack/python-manilaclient: Add validation for token and bypass_url  https://review.openstack.org/34668012:07
*** timcl has joined #openstack-manila12:14
openstackgerritTiago Pasqualini da Silva proposed openstack/manila: Fix response code on snapshot manage/unmanage APIs  https://review.openstack.org/34752912:16
*** akerr has joined #openstack-manila12:21
*** chlong_POffice has quit IRC12:21
*** akerr_ has joined #openstack-manila12:23
*** gouthamr has quit IRC12:24
*** yangyapeng has joined #openstack-manila12:25
*** akerr has quit IRC12:26
*** chlong has joined #openstack-manila12:29
*** akerr_ is now known as akerr12:31
*** gouthamr has joined #openstack-manila12:36
*** tovchinnikova has joined #openstack-manila12:37
*** JoseMello has joined #openstack-manila12:38
*** rraja has quit IRC12:47
gouthamr#success Manila API ref is officially publishing from content that's in code tree - http://developer.openstack.org/api-ref/shared-file-systems12:51
openstackstatusgouthamr: Added success to Success page12:51
gouthamrdgonzalez: :D12:52
gansovponomaryov: ping12:56
openstackgerritMerged openstack/manila: Minor optimization and formatting corrections in Quobyte driver  https://review.openstack.org/34529012:56
gansovponomaryov1: ping12:57
vponomaryov1ganso: pong13:04
gansovponomaryov1: Hi Valeriy13:04
vponomaryov1ganso: Hello Rodrigo13:04
gansovponomaryov1: in https://review.openstack.org/#/c/337229/4/manila/share/drivers/zfsonlinux/driver.py, why are you not using the snapshot['id']?13:04
vponomaryov1line?13:05
gansovponomaryov1: 743, 52413:07
vponomaryov1ganso: because I store there common info for all snapshot instances13:08
vponomaryov1ganso: so, I use common snapshot ID instead of snapshot instance ID13:09
gansovponomaryov1: ok, thanks!13:09
tpsilvavponomaryov1: https://review.openstack.org/#/c/347529/3/manila_tempest_tests/tests/api/admin/test_snapshot_manage_negative.py@8013:10
tpsilvavponomaryov1: isn't the test already skipping if microversion is less than 2.12?13:10
tpsilvavponomaryov1: on line 35, in resource setup13:10
vponomaryov1comment mentions 2.1913:10
vponomaryov1it is a little bit different than 2.1213:10
tpsilvavponomaryov1: it should skip this test if microversion less than 2.19? why?13:11
*** timcl has quit IRC13:12
*** faiz89 has joined #openstack-manila13:12
vponomaryov1tpsilva: test should be skipped for any microversion you use and that is not enabled13:13
*** merooney has joined #openstack-manila13:14
*** porrua has joined #openstack-manila13:17
tpsilvavponomaryov1: sorry, I did not understand what you mean by that13:17
vponomaryov1tpsilva: you use ddt there13:18
vponomaryov1tpsilva: so it is not test, they are tests now13:19
vponomaryov1tpsilva: and I say that you should skip any of them if used microversion there is not supported13:19
*** dustins has joined #openstack-manila13:21
*** akshai has joined #openstack-manila13:21
*** gouthamr has quit IRC13:22
*** gouthamr has joined #openstack-manila13:23
tpsilvavponomaryov1: I understand now13:24
*** xyang1 has joined #openstack-manila13:24
tpsilvavponomaryov1: but all the versions being tested are supported13:24
tpsilvavponomaryov1: I should add this line in case someone adds another version in ddt?13:25
vponomaryov1tpsilva: "supported" means tempest configured to be allowed to use them13:25
*** timcl has joined #openstack-manila13:26
*** Yogi1 has joined #openstack-manila13:27
openstackgerritAlexey Ovchinnikov proposed openstack/manila: Container driver  https://review.openstack.org/30893013:28
*** david-lyle_ has joined #openstack-manila13:31
*** david-lyle_ has quit IRC13:31
*** david-lyle has quit IRC13:34
openstackgerritTiago Pasqualini da Silva proposed openstack/manila: Fix response code on snapshot manage/unmanage APIs  https://review.openstack.org/34752913:38
openstackgerritRodrigo Barbieri proposed openstack/manila: Share migration Newton improvements  https://review.openstack.org/32843113:45
openstackgerritRodrigo Barbieri proposed openstack/manila: Fix Share Migration improper behavior for drivers  https://review.openstack.org/33226713:45
-openstackstatus- NOTICE: Gerrit is being restarted now to relieve performance degradation13:52
openstackgerritValeriy Ponomaryov proposed openstack/manila: [Dummy driver] Add share migration support  https://review.openstack.org/34785313:56
vponomaryov1ganso: ^13:57
openstackgerritSilvan Kaiser proposed openstack/manila: Replaces httplib with requests lib in Quobyte RPC layer  https://review.openstack.org/34521913:57
openstackgerritRodrigo Barbieri proposed openstack/manila: Add share type change to Share Migration  https://review.openstack.org/34514213:58
openstackgerritValeriy Ponomaryov proposed openstack/manila: [Dummy driver] Add share migration support  https://review.openstack.org/34785314:00
*** tovchinnikova has quit IRC14:00
openstackgerritRodrigo Barbieri proposed openstack/manila: Base Data Copy changes  https://review.openstack.org/32917414:16
openstackgerritRodrigo Barbieri proposed openstack/manila: Driver Data Service Helper  https://review.openstack.org/29583614:16
tpsilvagouthamr: ping14:18
*** cknight has joined #openstack-manila14:18
gouthamrtpsilva: pong14:18
tpsilvagouthamr: sorry, I missed your point about the microversion on your review14:18
tpsilvagouthamr: so yes, I agree with that14:18
openstackgerritMerged openstack/manila: [ZFSonLinux] Add 'manage snapshot' feature support  https://review.openstack.org/33722914:19
*** akapil_ has joined #openstack-manila14:19
tpsilvagouthamr: you mean to change all the API 422 codes to 400 on this patch?14:19
gouthamrtpsilva: yes, please.. do you think we can do that in one patch?14:20
*** akapil has quit IRC14:22
*** akapil_ has quit IRC14:23
tpsilvagouthamr: It is, but it is a lot of code14:25
tpsilvagouthamr: some of it is in API v114:25
gouthamrtpsilva: ah. that's worrisome; can work with you on that.. how about W-1 for now and discuss this further?14:27
tpsilvagouthamr: so to answer your question, yes, I agree that this should all be fixed on a single microversion bump, but I am afraid this would take some time for me to do that now14:28
tpsilvagouthamr: I agree14:28
gouthamrtpsilva: thanks!14:28
tpsilvagouthamr: and what shoult I do with the mountable snapshots patch? leave it with the wrong code?14:29
gouthamrtpsilva: nope.. please fix that..14:29
tpsilvagouthamr: but then we fall back into the problem :)14:29
gouthamrtpsilva: we don't need new mistakes that we'll regret :)14:29
tpsilvagouthamr: I mean, I can either use the snapshot manage/unmanage code with the wrong response or duplicate code and use the correct response14:30
gouthamrtpsilva: why not use your fix-response-code logic just for your snapshot mount APIs for now?14:30
gouthamrtpsilva: send that flag to validate only the body in the /action you're defining.14:30
tpsilvagouthamr: oh, right, that works14:30
tpsilvagouthamr: nice14:31
tpsilvagouthamr: ok then, I'll leave with W-1 and as soon as we have some time we discuss the future of that14:31
tpsilvagouthamr: thank you!14:31
tpsilvagouthamr: maybe after FF14:32
gouthamrtpsilva: you're welcome.. we'll get this in at a reasonable time or boot it to next release... no biggie for now14:34
openstackgerritValeriy Ponomaryov proposed openstack/manila: [Dummy driver] Add share migration support  https://review.openstack.org/34785314:37
*** cgalan has joined #openstack-manila14:39
openstackgerritValeriy Ponomaryov proposed openstack/manila: [Dummy driver] Add share migration support  https://review.openstack.org/34785314:39
vponomaryov1ganso: have you tested your commit https://review.openstack.org/#/c/332267 with something?14:41
vponomaryov1ganso: I am asking because I am not able to make it work on my lab again14:41
vponomaryov1ganso: is it expected to work when backends with both driver modes exist in cloud?14:42
vponomaryov1ganso: in my case it fails to schedule migration host14:42
vponomaryov1ganso: when all filters pass14:42
vponomaryov1ganso: and since it fails on migration host scheduling, driver implementation should not be the cause for it14:44
gouthamrvponomaryov1: quick q.. What's your opinion of removing snapshot-support as a kwarg from type-create from python-manilaclient's shell layer? https://review.openstack.org/#/c/315012/19/manilaclient/v2/shell.py14:45
openstackgerritAndrea Frittoli proposed openstack/manila: DNM Use tempest-plugin service client registration  https://review.openstack.org/33459614:46
vponomaryov1gouthamr: it is interface and should keep backwards compatibility14:46
gouthamrvponomaryov1: why treat that extra-spec any different and maintain that `compatibility code` forever.. people who use the CLI will read the error message and react..14:47
gouthamrvponomaryov1: shell isn't the interface, is it? we should be using the client in applications..14:47
vponomaryov1gouthamr: why is that? lots of CI scripts use CLI14:47
vponomaryov1we use it too14:48
gouthamrvponomaryov1: oh... :( thought it was cleaner to shave that off. alright, works for me. thank you14:48
*** nkrinner has quit IRC14:58
*** nkrinner has joined #openstack-manila15:00
*** nkrinner has quit IRC15:08
*** mtanino has joined #openstack-manila15:14
*** rraja has joined #openstack-manila15:16
*** pcaruana has quit IRC15:17
gansovponomaryov1: I have tested with dummy driver last time I changed something major, it worked, I addressed the issue of scheduling with two driver modes15:21
gansovponomaryov1: I will retest15:21
*** nkrinner has joined #openstack-manila15:22
vponomaryov1ganso: what about AZs?15:22
vponomaryov1ganso: you set up the same or different one?15:22
gansovponomaryov1: for the bugfix patch, the same15:22
vponomaryov1ganso: I get "target" host filtered out on AZ filter15:22
gansovponomaryov1: o_O15:23
vponomaryov1when it is located in different AZ15:23
gansovponomaryov1: that is being addressed in the newton improvements patch15:23
vponomaryov1in other words, driver migration still does nto work?15:24
vponomaryov1using different AZs?15:24
*** yangyapeng has quit IRC15:26
gansovponomaryov1: it works, but separate patch15:27
gansovponomaryov1: this one https://review.openstack.org/#/c/328431/2015:27
gansovponomaryov1: PS #21 in fact15:28
vponomaryov1ganso: what it?15:32
vponomaryov1ganso: AZ filtering?15:32
*** yangyapeng has joined #openstack-manila15:33
gansovponomaryov1: it will use AZ based on destination host choice15:33
gansovponomaryov1: instead of current AZ property15:34
openstackgerritAndrea Frittoli proposed openstack/manila: DNM Use tempest-plugin service client registration  https://review.openstack.org/33459615:35
openstackgerritAndrea Frittoli proposed openstack/manila: DNM Use tempest-plugin service client registration  https://review.openstack.org/33459615:36
gansovponomaryov1: but now that you mention it, the AZ is fairly easy to move to the bugfix patch15:39
gansovponomaryov1: because it does not require API change15:39
*** faiz89 has quit IRC15:42
*** xyang_ has joined #openstack-manila15:44
*** bswartz1 has joined #openstack-manila15:52
*** bswartz1 has quit IRC15:53
*** TheDude has joined #openstack-manila15:53
*** tovchinnikova has joined #openstack-manila15:53
*** cgalan has quit IRC15:57
openstackgerritValeriy Ponomaryov proposed openstack/manila: [Dummy driver] Add share migration support  https://review.openstack.org/34785315:59
openstackgerritAlexey Ovchinnikov proposed openstack/manila: [DNM] Tests for scenario tests  https://review.openstack.org/34023616:05
vponomaryov1ganso: it should have been included in this commit -> https://review.openstack.org/#/c/33226716:08
vponomaryov1ganso: ok, I expect CI will work now16:09
vponomaryov1ganso: do you have bug ID for AZ problem?16:10
gansovponomaryov1: no, it was part of blueprint for improvements16:10
akshaihey. what version of manila client was used in liberty?16:11
vponomaryov1akshai: you have problems with latest one?16:13
rrajaakshai: https://github.com/openstack/python-manilaclient/blob/stable/liberty/manilaclient/common/constants.py#L7716:13
vponomaryov1akshai: just set env var for microversion of liberty and that's it16:13
vponomaryov1rraja: it is version of API in Liberty16:14
vponomaryov1rraja: not manilaclient's version16:14
*** nkrinner is now known as nkrinner_afk16:14
*** TheDude has quit IRC16:15
rrajavponomaryov1: oh! ok! thanks!16:15
vponomaryov1akshai: or in other words, you should be able to use latest manilaclient against liberty server, just set env var with microversion16:15
akshaivponomaryov1: thx. still don't get it though. I see that for Mitaka I need to do: sudo pip install python-manilaclient>=1.8.1 to install. what is it for liberty?16:15
vponomaryov1akshai: again, you can use latest for liberty16:16
vponomaryov1akshai: as well as for mitaka16:16
vponomaryov1akshai: your example show "lower" limit16:16
akshaigouthamr: can you confirm that same version will work for liberty?16:17
akshaivponomaryov1: thought gouthamr meant that there is a different version i need. hence checking.16:19
vponomaryov1akshai: you asked about manilaclient16:19
vponomaryov1akshai: you did not ask latest API microversion for Liberty16:20
vponomaryov1akshai: and I clarify, you can use latest "manilaclient" for liberty/mitaka/newton manila version16:20
vponomaryov1akshai: just need to specify env var explicitly\16:21
*** cgalan has joined #openstack-manila16:21
vponomaryov1so, command "sudo pip install python-manilaclient>=1.8.1" suitable for liberty too16:21
akshaigot it. what is the environment variable I need to use?16:22
vponomaryov1akshai: "export OS_SHARE_API_VERSION=2.14"16:23
vponomaryov1akshai: where "2.14" API version you need16:23
vponomaryov1akshai: in case of Liberty - maximum supported is 2.616:23
akshaithanks. no other changes needed.16:23
vponomaryov1for mitaka - 2.1516:24
akshaivponomaryov1: thanks!16:24
vponomaryov1for master/newton - client will use latest16:24
vponomaryov1akshai: for any possibility of manilaclient use "manila help" command16:27
*** vbellur has quit IRC16:41
*** Suyi has joined #openstack-manila16:45
*** akapil has joined #openstack-manila16:51
openstackgerritAlexey Ovchinnikov proposed openstack/manila: Container driver  https://review.openstack.org/30893016:55
*** akapil has quit IRC16:56
*** nherciu has quit IRC17:00
*** faiz89 has joined #openstack-manila17:06
*** sandanar has quit IRC17:16
openstackgerritMerged openstack/manila-ui: Table actions improvement  https://review.openstack.org/34076817:18
*** xyang_ has quit IRC17:31
*** xyang_ has joined #openstack-manila17:31
*** vbellur has joined #openstack-manila17:32
openstackgerritMerged openstack/manila: NetApp: Report hybrid aggregates in share stats  https://review.openstack.org/34495817:38
*** vponomaryov1 has quit IRC17:42
*** MikeG451 has joined #openstack-manila18:03
openstackgerritRodrigo Barbieri proposed openstack/manila: Fix Share Migration improper behavior for drivers  https://review.openstack.org/33226718:21
*** ociuhandu has quit IRC18:21
openstackgerritRodrigo Barbieri proposed openstack/manila: Fix Share Migration improper behavior for drivers  https://review.openstack.org/33226718:22
*** vbellur has quit IRC18:40
*** faiz89 has quit IRC18:55
*** vbellur has joined #openstack-manila18:58
*** tovchinnikova has quit IRC19:09
openstackgerritGoutham Pacha Ravi proposed openstack/manila: POC: Tooz integration  https://review.openstack.org/31833619:10
gouthamrbswartz: rebased ^19:10
openstackgerritAlexey Ovchinnikov proposed openstack/manila: [DNM] Tests for scenario tests  https://review.openstack.org/34023619:30
*** Yogi1 has quit IRC19:30
*** vbellur has quit IRC19:49
openstackgerritMerged openstack/manila: Add retry in VNX driver when DB lock error happened  https://review.openstack.org/34460019:51
*** timcl has quit IRC19:52
openstackgerritRodrigo Barbieri proposed openstack/manila: Share migration Newton improvements  https://review.openstack.org/32843119:58
openstackgerritRodrigo Barbieri proposed openstack/manila: Fix Share Migration improper behavior for drivers  https://review.openstack.org/33226719:58
openstackgerritRodrigo Barbieri proposed openstack/manila: Add share type change to Share Migration  https://review.openstack.org/34514219:58
openstackgerritRodrigo Barbieri proposed openstack/manila: Base Data Copy changes  https://review.openstack.org/32917419:59
openstackgerritRodrigo Barbieri proposed openstack/manila: Driver Data Service Helper  https://review.openstack.org/29583619:59
*** porrua has quit IRC20:00
*** vbellur has joined #openstack-manila20:04
openstackgerritRodrigo Barbieri proposed openstack/manila-ui: Add share type change to Share Migration UI  https://review.openstack.org/34566420:08
openstackgerritRodrigo Barbieri proposed openstack/manila-ui: Add Share Migration support  https://review.openstack.org/34443120:08
*** merooney has quit IRC20:13
*** akshai has quit IRC20:15
*** JoseMello has quit IRC20:30
*** faiz89 has joined #openstack-manila20:39
gansobswartz: ping20:44
*** vbellur has quit IRC20:45
bswartzganso: ping20:48
gansobswartz: Hi Ben20:48
gansobswartz: there is a review in migration's python-manilaclient patch asking for functional tests. Currently the job that runs the server is single backend. I've noticed there are also no functional tests for replication. Do you think it makes sense to have migration functional tests on python-manilaclient CI? it would run full migration using the generic20:50
gansodriver and take some additional time20:50
openstackgerritMerged openstack/manila: Add interface port configuration in EMC VNX driver  https://review.openstack.org/33939720:51
gouthamrganso: i vote on writing basic negative tests for now. i had a similar ask for the replication patch.. if we're not running them against drivers that support it, i don't see value in worrying about them right away21:06
*** dustins has quit IRC21:10
*** rraja has quit IRC21:13
bswartzganso: that feels like medium priority to me21:20
bswartzif we have coverage of the server side functionality that's what matters most21:20
bswartzclient side should be covered but maybe not right away21:21
*** faiz89 has quit IRC21:22
*** xyang_ has quit IRC21:32
*** xyang_ has joined #openstack-manila21:35
*** xyang_ has quit IRC21:39
openstackgerritClinton Knight proposed openstack/manila: Implement share revert to snapshot  https://review.openstack.org/34050221:49
*** xyang1 has quit IRC22:13
*** jcsp has quit IRC22:14
openstackgerritGoutham Pacha Ravi proposed openstack/manila: Don't merge me yet.  https://review.openstack.org/34806822:30
*** tpsilva has quit IRC22:38
*** gouthamr has quit IRC22:44
*** cknight has quit IRC23:10
*** mtanino has quit IRC23:37
*** bill_az has joined #openstack-manila23:38
*** chlong has quit IRC23:38
*** sjjfowler has quit IRC23:40
*** akerr has quit IRC23:58
*** akerr has joined #openstack-manila23:59

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