Thursday, 2016-02-11

*** haomaiwa_ has quit IRC00:01
*** haomaiwang has joined #openstack-manila00:01
*** gouthamr has joined #openstack-manila00:03
*** cknight has quit IRC00:04
*** gouthamr_ has joined #openstack-manila00:05
*** gouthamr has quit IRC00:08
*** cknight has joined #openstack-manila00:20
*** xyang1_ has joined #openstack-manila00:28
*** xyang1 has quit IRC00:28
*** xyang1_ is now known as xyang100:28
*** ganso has quit IRC00:29
*** sgotliv has quit IRC00:37
*** cknight has quit IRC00:38
*** cknight has joined #openstack-manila00:39
*** a_ta has joined #openstack-manila00:39
*** martyturner has joined #openstack-manila00:43
*** a_ta has quit IRC00:44
*** gouthamr_ has quit IRC00:46
*** merooney has joined #openstack-manila00:47
*** haomaiwang has quit IRC01:01
*** haomaiwa_ has joined #openstack-manila01:02
*** merooney has quit IRC01:06
*** esker has joined #openstack-manila01:08
*** merooney has joined #openstack-manila01:09
*** chlong has joined #openstack-manila01:18
*** martyturner has quit IRC01:23
*** esker has quit IRC01:27
*** cknight has quit IRC01:35
*** jasonsb has joined #openstack-manila01:35
*** gouthamr has joined #openstack-manila01:36
*** cknight has joined #openstack-manila01:48
*** merooney has quit IRC01:59
*** akshai has quit IRC01:59
*** haomaiwa_ has quit IRC02:01
*** a_ta has joined #openstack-manila02:02
*** haomaiwang has joined #openstack-manila02:03
*** mtanino has quit IRC02:04
*** a_ta has quit IRC02:06
*** cknight1 has joined #openstack-manila02:08
*** jasonsb has quit IRC02:10
*** a_ta has joined #openstack-manila02:12
*** jasonsb has joined #openstack-manila02:13
*** jasonsb has quit IRC02:14
*** cknight has quit IRC02:15
*** openstackgerrit has quit IRC02:15
*** chlong has quit IRC02:15
*** a_ta has quit IRC02:16
*** openstackgerrit has joined #openstack-manila02:24
*** cknight1 has quit IRC02:24
*** chlong has joined #openstack-manila02:29
*** xyang1 has quit IRC02:29
*** tpsilva has quit IRC02:36
openstackgerritGoutham Pacha Ravi proposed openstack/manila: Scheduler enhancements for Share Replication  https://review.openstack.org/27754503:00
openstackgerritGoutham Pacha Ravi proposed openstack/manila: Share Replication API and Scheduler Support  https://review.openstack.org/23857203:00
*** haomaiwang has quit IRC03:01
*** haomaiwang has joined #openstack-manila03:01
openstackgerritGoutham Pacha Ravi proposed openstack/manila: Scheduler enhancements for Share Replication  https://review.openstack.org/27754503:02
openstackgerritCsaba Henk proposed openstack/manila: gluster*: clean up volume option querying  https://review.openstack.org/27714103:07
*** bill_az_ has quit IRC03:15
*** esker has joined #openstack-manila03:32
openstackgerritGoutham Pacha Ravi proposed openstack/manila: Add tempest tests for Share Replication  https://review.openstack.org/24484303:49
*** jasonsb has joined #openstack-manila03:52
*** esker has quit IRC03:53
*** gouthamr has quit IRC03:58
*** esker has joined #openstack-manila03:58
*** haomaiwang has quit IRC04:01
*** haomaiwang has joined #openstack-manila04:01
*** esker has quit IRC04:03
*** cknight has joined #openstack-manila04:58
*** haomaiwang has quit IRC05:01
*** haomaiwa_ has joined #openstack-manila05:01
*** cknight1 has joined #openstack-manila05:05
*** a_ta has joined #openstack-manila05:06
*** cknight has quit IRC05:08
openstackgerritvenkatamahesh proposed openstack/python-manilaclient: Abbreviate e.g to "for example"  https://review.openstack.org/27886205:10
*** a_ta has quit IRC05:10
openstackgerritvenkatamahesh proposed openstack/python-manilaclient: Abbreviate e.g to "for example"  https://review.openstack.org/27886205:11
*** lpetrut has joined #openstack-manila05:39
*** haomaiwa_ has quit IRC06:01
*** cknight1 has quit IRC06:01
*** 18VAACNL5 has joined #openstack-manila06:01
*** dcain1 has quit IRC06:02
*** lpetrut has quit IRC06:03
*** lpetrut has joined #openstack-manila06:19
*** abhi has quit IRC06:21
*** abhi has joined #openstack-manila06:23
*** rraja has quit IRC06:32
*** rraja has joined #openstack-manila06:37
*** lpetrut has quit IRC06:53
*** 18VAACNL5 has quit IRC07:01
*** haomaiwang has joined #openstack-manila07:01
*** a_ta has joined #openstack-manila07:12
*** baojg has joined #openstack-manila07:13
*** a_ta has quit IRC07:17
*** baojg has quit IRC07:35
*** baojg has joined #openstack-manila07:35
*** baojg has quit IRC07:42
*** sgotliv has joined #openstack-manila07:49
*** baojg has joined #openstack-manila07:55
*** nkrinner has joined #openstack-manila07:59
*** haomaiwang has quit IRC08:01
*** haomaiwang has joined #openstack-manila08:01
*** nkrinner has quit IRC08:26
*** nkrinner has joined #openstack-manila08:32
*** openstackgerrit has quit IRC08:47
*** openstackgerrit_ has joined #openstack-manila08:47
*** openstackgerrit_ is now known as openstackgerrit08:48
*** nkrinner has quit IRC08:52
*** sgotliv has quit IRC08:52
*** nkrinner has joined #openstack-manila08:57
*** haomaiwang has quit IRC09:01
*** haomaiwa_ has joined #openstack-manila09:01
openstackgerritAlexey Ovchinnikov proposed openstack/manila: LXC/LXD driver  https://review.openstack.org/24575109:03
*** ociuhandu has quit IRC09:44
openstackgerritRodrigo Barbieri proposed openstack/manila: Implement admin network in generic driver  https://review.openstack.org/27869909:51
*** ganso has joined #openstack-manila09:51
*** kaisers has joined #openstack-manila09:54
*** baojg_ has joined #openstack-manila09:58
*** baojg has quit IRC09:58
*** haomaiwa_ has quit IRC10:01
*** haomaiwang has joined #openstack-manila10:01
rrajajcsp: ping10:15
*** rraja is now known as rraja|afk10:24
*** jcsp has quit IRC10:30
*** jcsp has joined #openstack-manila10:34
*** tpsilva has joined #openstack-manila10:36
*** lpetrut has joined #openstack-manila10:50
openstackgerritJohn Spray proposed openstack/manila: Add Ceph Native driver  https://review.openstack.org/27021110:58
*** haomaiwang has quit IRC11:01
*** haomaiwa_ has joined #openstack-manila11:01
*** a_ta has joined #openstack-manila11:11
*** baojg_ has quit IRC11:12
*** baojg has joined #openstack-manila11:16
*** a_ta has quit IRC11:17
openstackgerritAlexey Ovchinnikov proposed openstack/manila: LXC/LXD driver  https://review.openstack.org/24575111:18
*** dencaval has joined #openstack-manila11:50
*** haomaiwa_ has quit IRC12:01
*** haomaiwang has joined #openstack-manila12:01
*** ig0r_ has joined #openstack-manila12:12
openstackgerritRodrigo Barbieri proposed openstack/manila: Implement admin network in generic driver  https://review.openstack.org/27869912:15
*** rraja|afk is now known as rraja12:15
openstackgerritRodrigo Barbieri proposed openstack/manila: Move Share Migration code to Data Service  https://review.openstack.org/25051512:19
openstackgerritRodrigo Barbieri proposed openstack/manila: Add new Share migration API functions  https://review.openstack.org/25964212:21
openstackgerritValeriy Ponomaryov proposed openstack/manila: Add ZFSonLinux share driver  https://review.openstack.org/27719212:28
gansobswartz: ping12:28
*** aovchinnikov has joined #openstack-manila12:40
*** gouthamr has joined #openstack-manila12:52
*** sgotliv has joined #openstack-manila12:56
*** haomaiwang has quit IRC13:01
*** haomaiwang has joined #openstack-manila13:01
*** a_ta has joined #openstack-manila13:14
*** gouthamr has quit IRC13:17
*** eharney has joined #openstack-manila13:18
openstackgerritRodrigo Barbieri proposed openstack/manila: Implement update_access() method in generic driver  https://review.openstack.org/25628113:18
*** martyturner has joined #openstack-manila13:18
*** a_ta has quit IRC13:19
*** esker has joined #openstack-manila13:20
openstackgerritValeriy Ponomaryov proposed openstack/manila: Add ZFSonLinux share driver  https://review.openstack.org/27719213:21
*** xyang1 has joined #openstack-manila13:24
*** akerr has joined #openstack-manila13:31
*** akerr_ has joined #openstack-manila13:33
*** csaba has quit IRC13:34
*** akerr_ has quit IRC13:35
*** akerr_ has joined #openstack-manila13:35
*** akerr has quit IRC13:36
*** akerr has joined #openstack-manila13:38
*** lpetrut has quit IRC13:39
*** akerr has quit IRC13:39
*** akerr_ has quit IRC13:40
*** gouthamr has joined #openstack-manila13:40
openstackgerritRamana Raja proposed openstack/manila: Fix cephx rule tempest tests  https://review.openstack.org/27551813:40
*** akerr has joined #openstack-manila13:40
*** akerr_ has joined #openstack-manila13:41
*** akerr has quit IRC13:42
*** csaba has joined #openstack-manila13:42
*** akerr has joined #openstack-manila13:43
*** dencaval has quit IRC13:44
*** akerr_ has quit IRC13:46
*** dencaval has joined #openstack-manila13:56
*** haomaiwang has quit IRC14:01
*** haomaiwang has joined #openstack-manila14:01
*** esker has quit IRC14:08
openstackgerritAlexey Ovchinnikov proposed openstack/manila: LXC/LXD driver  https://review.openstack.org/24575114:09
openstackgerritCsaba Henk proposed openstack/manila: glusterfs: introducing Heketi layout  https://review.openstack.org/27909014:10
*** a_ta has joined #openstack-manila14:13
openstackgerritJohn Spray proposed openstack/manila: Add Ceph Native driver  https://review.openstack.org/27021114:14
*** dustins has joined #openstack-manila14:14
openstackgerritCsaba Henk proposed openstack/manila: glusterfs: introducing Heketi layout  https://review.openstack.org/27909014:15
*** krotscheck_dcm is now known as krotscheck14:15
*** a_ta has quit IRC14:18
*** esker has joined #openstack-manila14:18
*** merooney has joined #openstack-manila14:21
*** esker has quit IRC14:23
*** martyt has joined #openstack-manila14:25
bswartzganso: pong (sorry for high latency)14:31
*** JoseMello has joined #openstack-manila14:34
*** martyt has quit IRC14:36
bswartzvponomaryov: I'm confused about your references to the FUSE version of ZFS14:36
bswartzwhen I run the devstack plugin from your patch on trusty, I end up with the native version of ZFS14:37
bswartzhow is it possible that the gate would get something different? It must use the same devstack plugin...14:37
*** baojg has quit IRC14:37
*** akshai has joined #openstack-manila14:38
*** ociuhandu has joined #openstack-manila14:39
vponomaryovbswartz: it already has ZFS14:40
vponomaryovbswartz: and doe snot install it in Devstack14:40
*** lpetrut has joined #openstack-manila14:41
bswartzthe dsvm nodes have zfs-fuse pre-installed?14:41
vponomaryovbswartz: yes14:42
vponomaryovbswartz: you can see it in any other CI run in "dpkg" list14:42
bswartzthat seems bad...14:43
vponomaryovbswartz: why?14:43
bswartzbecause then the gate tests will always be FUSE-based which won't match what real users will do14:44
bswartzdo you have any idea why they have those packages?14:44
vponomaryovbswartz: I did not investigate why, but we can make install kernel version14:44
vponomaryovbswartz: forcibly14:45
bswartzwould it be very bad to "apt-get remove zfs-fuse" in our devstack plugin?14:45
* mkoderer would never use any fuse driver in production :)14:45
vponomaryovbswartz: not sure we need to remove it, we just need to install kernel version14:46
*** akerr has quit IRC14:46
bswartzI bet the 2 would conflict14:46
vponomaryovbswartz: one of two my labs with kernel version has both14:47
vponomaryovbswartz: "ubuntu-zfs" is set of packages14:47
vponomaryovbswartz: and fuse one of them14:47
bswartzvponomaryov: when you have both installed, how do you decide which one gets used?14:47
vponomaryovbswartz: kernel used if it is available14:48
vponomaryovbswartz: no one asks me14:48
bswartzhmm14:48
bswartzI like the idea of force-installing the native package then14:48
vponomaryovbswartz: ok, will update14:48
bswartzI still don't understand why the user-space tools for export management are so funky14:48
mkodererbswartz: is zfs  officialy supported by U14:49
bswartzthe solaris-like syntax for sharenfs is pretty insane14:49
mkoderersry supported by Ubuntu LTS14:49
mkodererbswartz: seems a special PPA needs to be activated?14:49
vponomaryovmkoderer: yes14:49
vponomaryovmkoderer: yes for PPA14:49
bswartzmkoderer: PPA is needed for trusty -- for wily and later it's in standard distro14:50
vponomaryovbswartz: have you seen my answer about naming of driver?14:50
bswartzvponomaryov: I just read it14:51
bswartzI think in the short term we will not test on Solaris or BSD, but in the future if someone wanted to support those platforms, they would be much better off extending this driver instead of writing a new one14:52
bswartzso we should name the driver with possible future extensions in mind14:52
bswartzI agree we need to clearly distinguish from zfssa14:52
bswartzmaybe "plainzfs"? or "unixzfs"?14:53
vponomaryovbswartz: None, "unix" is not supported OS, plain is unclear14:54
bswartzI just know once it merges it will get very hard to change the name14:54
bswartzand I can see someone adding BSD support with fairly small effort in future release14:54
bswartzor Solaris14:55
*** erlon has joined #openstack-manila14:55
vponomaryovbswartz: yes, need just to think how to call Solaris, BSD and Linux stuff under one word14:55
*** cknight has joined #openstack-manila14:55
vponomaryovbswartz: how to name their similarity14:55
bswartztypically people use the term "UNIX" even though it's technically incorrect, or sometimes "UNIX-like"14:56
mkodererLiUnixZFS14:56
bswartzmkoderer: lol14:56
*** markstur_ has joined #openstack-manila14:57
vponomaryovZfsONix, ZFSonNix14:57
tbarronunix-like14:58
tbarronnixish14:58
tbarron:)14:58
mkoderertbarron: we definitly need to vote in the manila meeting :)14:59
tbarronmkoderer: short agenda, huh?14:59
tbarronnaming is too important to be done democratically15:00
gansobswartz: Could you please approve? https://blueprints.launchpad.net/manila/+spec/data-service-migration15:00
*** haomaiwang has quit IRC15:01
*** haomaiwang has joined #openstack-manila15:01
*** nug has left #openstack-manila15:01
*** alyson_ has joined #openstack-manila15:02
*** eharney has quit IRC15:02
openstackgerritxing-yang proposed openstack/manila: Manage and unmanage snapshot  https://review.openstack.org/24954215:03
*** a_ta has joined #openstack-manila15:07
*** jwang_ has joined #openstack-manila15:08
*** jwang has quit IRC15:12
*** martyt has joined #openstack-manila15:20
*** ig0r_ has quit IRC15:21
*** eharney has joined #openstack-manila15:21
*** martyt has quit IRC15:24
*** bsuchok1 has joined #openstack-manila15:29
*** dcain has joined #openstack-manila15:34
bswartzcsaba: #openstack-meeting-alt15:36
*** bsuchok_ has left #openstack-manila15:37
*** akerr has joined #openstack-manila15:38
*** markstur_ has quit IRC15:40
*** mtanino has joined #openstack-manila15:40
*** bsuchok1 has left #openstack-manila15:40
*** alkhodos has quit IRC15:44
openstackgerritxing-yang proposed openstack/manila: oManage and unmanage snapshot  https://review.openstack.org/24954215:50
*** sgotliv has quit IRC15:51
*** martyt has joined #openstack-manila15:53
*** martyt has quit IRC15:55
openstackgerritxing-yang proposed openstack/manila: Manage and unmanage snapshot  https://review.openstack.org/24954215:57
*** martyt has joined #openstack-manila15:59
*** fthiagogv has joined #openstack-manila15:59
*** haomaiwang has quit IRC16:01
*** haomaiwa_ has joined #openstack-manila16:01
*** xyang_ has joined #openstack-manila16:01
xyang_gouthamr: ping16:02
gouthamrxyang_: pong16:02
gouthamrxyang_: hi!16:02
xyang_gouthamr: hi, regarding your comment, I added explanation, did you see it?16:02
gouthamrxyang_: didn't see that.. i wasn't sure if driver_private_storage is meant only to be used with snapshots..16:03
gouthamrxyang_: also, is that meant to be an "admin" level API?16:04
xyang_gouthamr: I discussed this with bswartz, cknight, vponomaryov, initially they asked me to submit a separate patch for driver private data, but then bswartz he is ok if they put them in one patch16:04
xyang_gouthamr: yes, admin api, not just for snapshots16:05
xyang_gouthamr: I need to do some cleanup in the patch to remove some code I added in snapshot views16:05
gouthamrxyang_: okay, if there's general agreement, its good :)16:05
xyang_gouthamr: thanks16:05
gouthamrxyang_: yep. I can help by reviewing it..16:05
xyang_great16:06
xyang_bswartz, cknight, vponomaryov: can you also take a look of the driver private data API I added in the patch?  https://review.openstack.org/#/c/249542/16:06
xyang_WIP, but I'd like to get some early feedback16:07
*** aovchinnikov has quit IRC16:07
*** timcl has joined #openstack-manila16:07
cknightxyang1: yes16:11
cknightvponomaryov: ping16:12
vponomaryovcknight: pong16:12
cknightvponomaryov: Did you define any standard keys for export location metadata?  Such as a key to identify preferred vs. non-preferred paths?16:12
vponomaryovcknight: no16:13
vponomaryovcknight: only wto - "user" and "admin"16:13
cknightvponomaryov: Ben recalls suggesting we do that, but I guess we didn't, then.  I'm going to use 'preferred': True/False, and we can discuss a standard for that later.16:14
vponomaryovcknight: it si up to the EL metadata16:14
vponomaryovcknight: I didn't implement specific keys16:14
cknightvponomaryov: I also noticed that the EL metadata isn't returned from the REST interface.  Any reason why?16:14
xyang_cknight, vponomaryov: uri is driver_private_data/<host>/<entity_id>16:14
vponomaryovcknight: they can be easily added16:14
vponomaryovcknight: it is intentional16:15
vponomaryovcknight: each "agreed" key should be added explicitly16:15
vponomaryovcknight: to API view16:15
cknightvponomaryov: but what's the harm in also returning anything in the metadata dictionary?16:15
vponomaryovcknight: https://github.com/openstack/manila/blob/b1b723ad0b8a61eef296977c03ef2bb244e90e27/manila/api/views/export_locations.py#L3116:16
vponomaryovcknight: API stability contract?16:16
cknightvponomaryov: Yeah, I saw that.  OK, something to discuss later, then.  I have the basic EL metadata working in my drivers.  So far, so good.16:17
vponomaryovcknight: so, it is question of APi versioning, that's all, DB structure will not be changed since we have ELM16:18
vponomaryovxyang_: you say it is WIP?16:19
vponomaryovxyang_: what exactly are you planning to add?16:19
vponomaryovxyang_: or what is unfinished yet on your opinion?16:19
xyang1vponomaryov: can you take a look of driver_private_data files16:20
vponomaryov xyang1: it will take a while16:20
xyang_vponomaryov: manila/api/v2/driver_private_data.py, manila/api/views/driver_private_data.py, manila/api/v2/router.py16:20
xyang_vponomaryov: sure.  I just want to know if the API looks ok16:21
*** bsuchok1 has joined #openstack-manila16:21
xyang_vponomaryov: so right now it requires host and entity_id16:22
vponomaryovxyang_: where is list of all entities?16:23
vponomaryovxyang_: planned, no?16:23
xyang_vponomaryov: I didn't add that.  that is a question too16:23
xyang_vponomaryov: we have db.driver_private_data_get which requires host and entity_id16:23
xyang_vponomaryov: do we want a more general one that retrieves all?16:24
xyang_vponomaryov: and host and entity_id will be filter?16:24
xyang_that is the part I need feedback16:24
vponomaryovxyang_: if we do not have use case for it, then, maybe no16:24
xyang_vponomaryov: or we can have this now and expand later if there is use case16:25
vponomaryovxyang_: right now there is only one use case - make "manage snapshot" feature work in a good way16:25
xyang_vponomaryov: ok16:26
*** vkmc|midcycle is now known as vkmc16:27
*** absubram has joined #openstack-manila16:28
*** martyt has quit IRC16:30
tpsilvavponomaryov: ping16:33
vponomaryovtpsilva: pong16:33
rraja /nick rraja|afk16:38
*** ganso has quit IRC16:38
*** tpsilva has quit IRC16:38
*** rraja is now known as rraja|afk16:38
*** JoseMello has quit IRC16:39
*** vbellur1 has quit IRC16:39
*** vbellur has joined #openstack-manila16:40
*** ganso has joined #openstack-manila16:40
*** esker has joined #openstack-manila16:42
*** JoseMello has joined #openstack-manila16:43
*** bsuchok1 has left #openstack-manila16:46
*** bsuchok has joined #openstack-manila16:46
*** rraja|afk has quit IRC16:49
*** timcl has quit IRC16:51
*** tpsilva has joined #openstack-manila16:53
gansotpsilva: ping16:54
tpsilvaganso: pong16:54
tpsilvavponomaryov: my irc is acting funny, so I don't know if you got my messages, so here it goes16:55
tpsilvavponomaryov: When you have time, could you take a look at https://review.openstack.org/#/c/279164/? It's that patch that adds the separate process for the data service. It's not quite ready for reviews yet, but I would like to know if that approach would be ok16:55
tpsilvavponomaryov: it's really small, so it shouldn't take much time :)16:55
*** sgotliv has joined #openstack-manila16:58
*** esker has quit IRC16:58
openstackgerritTiago Pasqualini da Silva proposed openstack/manila: Fixed Hitachi HNAS slow test  https://review.openstack.org/27918717:00
*** haomaiwa_ has quit IRC17:01
*** haomaiwang has joined #openstack-manila17:01
*** esker has joined #openstack-manila17:02
*** sgotliv has quit IRC17:06
*** esker has quit IRC17:07
openstackgerritTiago Pasqualini da Silva proposed openstack/manila: Add separate process for Data Service copy  https://review.openstack.org/27916417:07
*** ociuhandu has quit IRC17:12
*** porrua has joined #openstack-manila17:12
*** esker has joined #openstack-manila17:14
*** esker has quit IRC17:14
*** esker has joined #openstack-manila17:14
*** nkrinner has quit IRC17:19
*** a_ta has quit IRC17:21
*** a_ta has joined #openstack-manila17:22
*** a_ta has quit IRC17:26
*** esker has quit IRC17:31
*** ociuhandu has joined #openstack-manila17:33
*** esker has joined #openstack-manila17:33
*** a_ta has joined #openstack-manila17:34
bswartzvponomaryov: see my latest note?17:34
vponomaryovbswartz: yes, agree, and already working on it17:35
bswartzok17:35
*** esker has quit IRC17:39
*** esker has joined #openstack-manila17:39
*** esker has quit IRC17:45
openstackgerritTiago Pasqualini da Silva proposed openstack/manila: Fixed Hitachi HNAS slow test  https://review.openstack.org/27918717:49
*** esker has joined #openstack-manila17:53
gansobswartz: ping17:53
*** timcl has joined #openstack-manila18:00
*** haomaiwang has quit IRC18:01
*** haomaiwang has joined #openstack-manila18:01
*** timcl1 has joined #openstack-manila18:04
*** timcl has quit IRC18:05
openstackgerritValeriy Ponomaryov proposed openstack/manila: Add ZFSonLinux share driver  https://review.openstack.org/27719218:08
tpsilvazhongjun: ping18:08
vponomaryovbswartz: fixed your concerns and uploaded new PS18:09
*** esker has quit IRC18:10
*** esker has joined #openstack-manila18:21
*** nkrinner has joined #openstack-manila18:31
*** timcl1 has quit IRC18:33
*** mtanino__ has quit IRC18:35
*** mtanino__ has joined #openstack-manila18:35
*** alyson_ has quit IRC18:36
*** xyang_ has quit IRC18:37
*** alyson_ has joined #openstack-manila18:37
*** xyang_ has joined #openstack-manila18:37
*** timcl has joined #openstack-manila18:40
gouthamrvponomaryov: any further review/concerns about: https://review.openstack.org/#/c/238572/ ?18:40
vponomaryovgouthamr: quotas?18:41
gouthamrvponomaryov: working on a separate patch for that..18:41
vponomaryovgouthamr: so, you will be able to finish it before FPF?18:42
gouthamrbswartz thought we might need some more discussion around quotas18:42
gouthamrvponomaryov: yes sir18:42
*** aovchinnikov has joined #openstack-manila18:42
* gouthamr checks calendar - FPF is next week.. 18:42
gouthamr:)18:42
vponomaryovgouthamr: if we implement quotas after replication, then we will have some problems with API versioning18:43
vponomaryovgouthamr: it will be possible to use replication microversion to bypass quotass18:43
*** a_ta has quit IRC18:43
gouthamrvponomaryov: not really. This won't be an APIImpact;18:44
vponomaryovgouthamr: so, if it is implemented as separate patch, then it should be same microversion anyway18:44
*** a_ta has joined #openstack-manila18:44
gouthamrvponomaryov: yes, it will be the same microversion for the quotas..18:44
vponomaryovgouthamr: it will be security hole18:44
gouthamrvponomaryov: perhaps a different microversion for the admin APIs.. perhaps..18:44
vponomaryovgouthamr: then ok18:44
gouthamrvponomaryov: thanks.. appreciate your review.18:45
gouthamrcknight, xyang1, bswartz, ganso, toabctl, markstur: could you please look at https://review.openstack.org/#/c/238572/18:47
*** a_ta has quit IRC18:48
vponomaryovgouthamr: have you tested it with some driver that supports replication? NetApp?18:48
vponomaryovgouthamr: latest version18:49
*** esker has quit IRC18:51
*** timcl has quit IRC18:51
*** ig0r_ has joined #openstack-manila18:58
gouthamrvponomaryov: yes18:59
gouthamrthe functional tests here: https://review.openstack.org/#/c/244843 have been run on it18:59
vponomaryovgouthamr: managed share does not get "replication_type" value18:59
vponomaryovgouthamr: only created one gets19:00
gouthamrvponomaryov: you mean the API?19:00
*** haomaiwang has quit IRC19:01
vponomaryovgouthamr: if I manage share with share-type and host that support replication, managed share has "None" value for replication_type19:01
vponomaryovgouthamr: and it is bug19:01
*** haomaiwang has joined #openstack-manila19:01
*** lpetrut has quit IRC19:01
gouthamrvponomaryov: you mean, if the share-type has replication_type extra-spec on it?19:02
vponomaryovgouthamr: also, after its fix, need add tempest "mange" tests and verify in them that "manged" share has same value for "replication_type" as source one19:03
*** ig0r_ has quit IRC19:03
vponomaryovgouthamr: yes19:03
vponomaryovs/mange/manage/19:03
gouthamrvponomaryov: so extra_specs aren't getting copied in case of manage currently..19:06
gouthamrvponomaryov: isn't that true about snapshot_support too?19:06
vponomaryovgouthamr: "snapshot_support" looks ok19:07
vponomaryovgouthamr: from firs sight19:07
vponomaryovs/firs/first/19:07
gouthamrvponomaryov: i was trying to run through the manage code; the extra_specs seem to not be copied to the share model like we're doing in case of create19:08
vponomaryovgouthamr: you confirm presense of the bug?19:09
gouthamrvponomaryov: i'll test it and file the bug if it exists19:09
vponomaryovgouthamr: don't want to upload new patch-set? ))19:09
gouthamrvponomaryov: downstream CI is broken for the moment.. regression testing any changes I make would have latency :S19:10
vponomaryovgouthamr: downstream CI ?19:11
vponomaryovgouthamr: broken?19:11
gouthamrvponomaryov: @NetApp, we use a fancy system ;)19:11
*** akshai has quit IRC19:13
*** akshai has joined #openstack-manila19:13
bswartzvponomaryov: we run all kinds of netapp-specific tests that can't go in upstream tempest because they do white box testing or test features that only exist in netapp driver19:15
vponomaryovbswartz: so, you are ok merging replication commit with already known bugs?19:16
bswartzother than lack of quotas, what known bugs are you thinking of?19:17
vponomaryovbswartz: the one above -> managed shares do not get "replication_type" value19:17
bswartzah19:17
bswartzyes I consider those 2 things missing features rather than bugs19:18
bswartzit's experimental -- it's more important to have it usable so we can get feedback than to have everything perfect19:19
bswartzthe most important thing is that it doesn't break any existing feature19:19
bswartzusers who choose not to use replication should see NO negative impact from gouthamr's patch19:20
bswartzif they do, then that's a bug that needs to be fixed19:20
*** aovchinnikov has quit IRC19:23
vponomaryovbswartz, gouthamr: +2'ed replication19:24
gouthamrthanks vponomaryov.. investigating the manage feature; will file a bug if i find one against existing code.. or submit a blueprint to make changes to mine.19:24
*** lpetrut has joined #openstack-manila19:30
*** a_ta has joined #openstack-manila19:39
dustinsgouthamr: Been looking at the patch myself, only thing that was glaring was the lack of DocImpact tag in the commit message19:40
dustinsStill going through, but I wanted to get that in before it got lost in the mix19:41
gouthamrdustins: we're writing the adminref, devref, userguide, adminguide and api_site docs currently..19:43
bswartztbarron: ping19:44
gouthamrdustins: do you still think a DocImpact is necessary?19:44
dustinsgouthamr: I think so, just so it doesn't fall through the cracks19:44
dustinsNice to have tracking on such things as well19:45
gouthamrdustins: sure thing. will add it19:45
openstackgerritGoutham Pacha Ravi proposed openstack/manila: Share Replication API and Scheduler Support  https://review.openstack.org/23857219:45
dustinsgouthamr: Thanks!19:49
*** ociuhandu has quit IRC19:59
*** haomaiwang has quit IRC20:01
*** haomaiwa_ has joined #openstack-manila20:01
*** timcl has joined #openstack-manila20:08
*** esker has joined #openstack-manila20:09
*** jasonsb has quit IRC20:09
*** dencaval has quit IRC20:13
*** timcl1 has joined #openstack-manila20:17
*** timcl has quit IRC20:17
*** a_ta has quit IRC20:33
*** timcl1 has quit IRC20:38
*** xyang_ has quit IRC20:45
*** xyang_ has joined #openstack-manila20:48
*** madhu_ak_ has joined #openstack-manila20:49
*** ociuhandu has joined #openstack-manila20:57
*** haomaiwa_ has quit IRC21:01
*** haomaiwang has joined #openstack-manila21:01
*** merooney has quit IRC21:06
*** esker has quit IRC21:23
*** esker has joined #openstack-manila21:26
*** esker has quit IRC21:31
*** akerr has quit IRC21:41
*** cknight has quit IRC21:43
*** esker has joined #openstack-manila21:46
*** a_ta_ has joined #openstack-manila21:51
*** eharney has quit IRC21:52
*** ociuhandu has quit IRC21:52
*** ociuhandu has joined #openstack-manila21:53
*** timcl has joined #openstack-manila21:55
*** pgbridge has quit IRC21:58
*** nkrinner has quit IRC22:00
*** haomaiwang has quit IRC22:01
*** haomaiwang has joined #openstack-manila22:01
openstackgerritxing-yang proposed openstack/manila: Manage and unmanage snapshot  https://review.openstack.org/24954222:01
*** esker has quit IRC22:02
*** chlong has quit IRC22:03
*** esker has joined #openstack-manila22:03
*** pgbridge has joined #openstack-manila22:04
*** timcl has left #openstack-manila22:09
*** tpsilva has quit IRC22:16
*** a_ta_ has quit IRC22:17
*** esker has quit IRC22:17
*** cknight has joined #openstack-manila22:18
*** a_ta has joined #openstack-manila22:18
*** a_ta has quit IRC22:18
*** a_ta has joined #openstack-manila22:18
*** ganso has quit IRC22:19
*** porrua has quit IRC22:24
*** gouthamr has quit IRC22:28
*** sgotliv has joined #openstack-manila22:28
*** fthiagogv has quit IRC22:31
*** dcain has quit IRC22:42
*** dustins has quit IRC22:46
*** cknight1 has joined #openstack-manila22:46
*** JoseMello has quit IRC22:48
*** cknight has quit IRC22:49
openstackgerritting wang proposed openstack/manila: replace string format arguments with function parameters  https://review.openstack.org/27842522:52
*** xyang_ has quit IRC22:56
*** xyang1 has quit IRC22:58
*** haomaiwang has quit IRC23:01
*** haomaiwang has joined #openstack-manila23:01
*** cknight1 has quit IRC23:01
*** martyturner has quit IRC23:07
*** lpetrut has quit IRC23:13
*** madhu_ak_ has left #openstack-manila23:18
*** a_ta has quit IRC23:24
*** a_ta has joined #openstack-manila23:24
*** a_ta has quit IRC23:28
*** chlong has joined #openstack-manila23:30
*** marcusvrn_ has quit IRC23:47
*** absubram has quit IRC23:57

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