Wednesday, 2016-02-10

*** alonma has quit IRC00:02
*** mylu has quit IRC00:02
*** hongbin has quit IRC00:04
*** alonma has joined #openstack-cinder00:06
*** mylu has joined #openstack-cinder00:07
*** mylu has quit IRC00:10
*** alonma has quit IRC00:11
*** mylu has joined #openstack-cinder00:12
openstackgerritAngus Lees proposed openstack/os-brick: Trivial rootwrap -> privsep replacement  https://review.openstack.org/27722400:12
*** csky has quit IRC00:13
openstackgerritIvan Kolodyazhny proposed openstack/cinder: WIP! Fix Rally job  https://review.openstack.org/27811900:14
*** alonma has joined #openstack-cinder00:15
*** e0ne has quit IRC00:18
*** alonma has quit IRC00:20
*** mylu has quit IRC00:20
notmorganso, good news everyone! I have all but 4 tests working with cinderclinet using keystoneauth! :)00:21
notmorganonce i have this passed, who should i poke at to review the changes? I'd like to get this landed quickly so we're on KSA on the major client libs [hopefully occ too]00:22
hemnagus, http://paste.openstack.org/show/486502/00:25
hemnaafter updating the latest patchset00:25
gus"Error opening class fc_host" ?00:27
gusThe logs suggest it's running "systool -c fc_host -v" with uid/gid=0 and getting back that error00:28
gusDoes that work when you run that directly (perhaps with sudo)?00:28
*** esker has quit IRC00:28
gus.. and then later it's running "multipathd show status" and getting No such file or directory - so I assume you don't have multipathd installed?00:32
gusThat last seems to be the stacktrace that finally aborts your diediedie test00:32
*** dramakri has quit IRC00:35
gusoh, huh.00:35
gusI wonder if PATH is never updated to include /sbin?00:36
gus...  oh no, I think it should be, because the privsep-helper is still run through sudo and so should have the same environment as with rootwrap.00:36
*** geguileo has quit IRC00:38
gusaha. ..00:40
gusI note that simply exec()ing "bogus" gives a different error return to exec()ing "sudo bogus" (or indeed "sh -c bogus")00:41
gusI wonder if processutils.execute actually knows how to catch errors with shell=False.  Looking..00:41
*** akshai has quit IRC00:43
*** prithiv has quit IRC00:45
gushuh, indeed putils.execute('foo') throws OSError(no such file or directory), and not ProcessExecutionError as expected.00:46
*** geguileo has joined #openstack-cinder00:46
*** arch-nemesis has quit IRC00:50
gus.. and indeed putils.execute is documented as raising either ProcessExecutionError or OSError.00:51
gusI claim this is an os-brick bug.00:51
gusI believe lots of places where we currently catch only ProcessExecutionError should also catch OSError.00:52
gusNow: should I convert OSError into ProcesExecutionError in priv_rootwrap.execute?00:52
gus(as a workaround)00:52
*** mylu has joined #openstack-cinder00:58
*** jungleboyj has quit IRC01:00
*** markvoelker has joined #openstack-cinder01:01
*** mylu has quit IRC01:02
*** mylu has joined #openstack-cinder01:04
*** garthb has quit IRC01:14
*** alonma has joined #openstack-cinder01:16
*** alonma has quit IRC01:20
*** alonma has joined #openstack-cinder01:22
*** fernnest is now known as fernnest_offWed01:24
*** esker has joined #openstack-cinder01:26
*** alonma has quit IRC01:27
openstackgerritVincent Hou proposed openstack/cinder: Migration: take the local_path for the source volume  https://review.openstack.org/25022001:27
*** fernnest_offWed is now known as fernnest01:27
openstackgerritScott DAngelo proposed openstack/cinder: cinder-api-microversions code  https://review.openstack.org/22491001:28
*** mtanino has quit IRC01:29
*** mylu has quit IRC01:31
*** alonma has joined #openstack-cinder01:31
*** leeantho has quit IRC01:34
*** mylu has joined #openstack-cinder01:35
*** alonma has quit IRC01:35
*** shakamunyi has quit IRC01:40
*** esker has quit IRC01:40
*** esker has joined #openstack-cinder01:41
*** laughterwym has joined #openstack-cinder01:44
*** mylu has quit IRC01:44
*** esker has quit IRC01:45
*** laughterwym has quit IRC01:48
*** bswartz has quit IRC01:49
*** bswartz has joined #openstack-cinder01:50
*** mylu has joined #openstack-cinder01:55
*** sid14726 has joined #openstack-cinder01:55
*** funeutron has joined #openstack-cinder01:56
*** mylu has quit IRC01:58
*** gouthamr has joined #openstack-cinder02:13
*** itzdilip has joined #openstack-cinder02:20
*** funeutron has quit IRC02:23
*** mylu has joined #openstack-cinder02:30
*** alonma has joined #openstack-cinder02:31
*** alonma has quit IRC02:35
*** dims_ has quit IRC02:45
*** Poornima has joined #openstack-cinder02:48
*** gouthamr_ has joined #openstack-cinder02:51
*** gouthamr has quit IRC02:54
*** dan_nguyen has quit IRC02:56
*** links has joined #openstack-cinder03:00
*** mudassirlatif has quit IRC03:06
*** mudassirlatif has joined #openstack-cinder03:07
*** cknight has quit IRC03:07
*** csky has joined #openstack-cinder03:10
*** logan-- has joined #openstack-cinder03:10
*** mudassirlatif has quit IRC03:11
*** alonma has joined #openstack-cinder03:14
*** logan-- has quit IRC03:14
*** mylu has quit IRC03:15
*** mylu has joined #openstack-cinder03:16
*** alonma has quit IRC03:18
*** alonma has joined #openstack-cinder03:20
*** logan- has quit IRC03:22
*** logan- has joined #openstack-cinder03:23
*** bardia has quit IRC03:23
*** alonma has quit IRC03:25
*** mylu has quit IRC03:26
*** dims has joined #openstack-cinder03:27
*** mylu has joined #openstack-cinder03:29
smcginnisnotmorgan: Awesome on the keystoneauth!03:29
smcginnisnotmorgan: That was on my "this would be good if I can get to it someday" queue.03:29
smcginnisnotmorgan: Let me know when you have something and I'll check it out. I would like to get that switched too.03:29
notmorganStruggling with a few tests. Should have it wound up soon.03:29
smcginnisnotmorgan: Cool. Hopefully the struggling stays to a minimum. ;)03:30
notmorganIt's just unwinding duplicated code03:31
*** funeutron has joined #openstack-cinder03:31
notmorganWell copied code.03:32
notmorganOnly 5 clients to fix this week. Cinder is the first.03:32
smcginnisnotmorgan: Hah, should hopefully be pretty trivial by the fifth.03:33
notmorganEach are wildly different :(03:33
notmorganWorst will be keystone client actually. :P03:34
*** xyang1 has joined #openstack-cinder03:34
*** sheel has joined #openstack-cinder03:36
*** wanghao_ has joined #openstack-cinder03:36
*** dims has quit IRC03:37
*** mylu has quit IRC03:38
openstackgerritAngus Lees proposed openstack/os-brick: Trivial rootwrap -> privsep replacement  https://review.openstack.org/27722403:39
*** mylu has joined #openstack-cinder03:39
*** apoorvad has quit IRC03:40
*** jwcroppe has joined #openstack-cinder03:41
openstackgerritAngus Lees proposed openstack/os-brick: Trivial rootwrap -> privsep replacement  https://review.openstack.org/27722403:42
openstackgerritAngus Lees proposed openstack/os-brick: Trivial rootwrap -> privsep replacement  https://review.openstack.org/27722403:44
PoornimaHi wanghao, swamireddy regarding https://review.openstack.org/#/c/235752/03:48
Poornimajust removing blank space leading to gate failure03:48
*** bill_az has quit IRC03:49
*** edmondsw has quit IRC03:49
*** xyang1 has quit IRC03:50
openstackgerritAngus Lees proposed openstack/os-brick: Trivial rootwrap -> privsep replacement  https://review.openstack.org/27722403:51
smcginnisPoornima: There are issues in gate right now.03:51
smcginnisPoornima: Not sure if things are cleared up yet or not, but I've kicked off a recheck on there.03:51
Poornimasmcginnis, this is strange03:52
*** gouthamr_ has quit IRC03:52
openstackgerritMerged openstack/cinder-specs: Remove Py33 support  https://review.openstack.org/26611104:05
*** shyama has joined #openstack-cinder04:09
*** mylu has quit IRC04:15
*** markvoelker has quit IRC04:15
*** markvoelker has joined #openstack-cinder04:16
*** alonma has joined #openstack-cinder04:20
*** mylu has joined #openstack-cinder04:25
*** alonma has quit IRC04:25
*** mylu has quit IRC04:26
openstackgerritSean McGinnis proposed openstack/cinder: Remove Cisco FC Zone Manager Driver  https://review.openstack.org/27820104:31
smcginnishemna: ^^04:32
*** Poornima has quit IRC04:32
*** funeutron has quit IRC04:40
*** salv-orlando has joined #openstack-cinder04:41
*** sgotliv has joined #openstack-cinder04:42
*** salv-orl_ has quit IRC04:43
*** mylu has joined #openstack-cinder04:44
*** wanghao has quit IRC04:46
*** wanghao_ is now known as wanghao04:46
*** laughterwym has joined #openstack-cinder04:55
*** laughterwym has quit IRC04:59
*** Poornima has joined #openstack-cinder05:00
*** mragupat has joined #openstack-cinder05:00
*** alonma has joined #openstack-cinder05:04
*** mragupat has quit IRC05:06
*** mragupat has joined #openstack-cinder05:06
*** chhavi has joined #openstack-cinder05:07
*** alonma has quit IRC05:08
*** alonma has joined #openstack-cinder05:10
*** alonma has quit IRC05:14
*** alonma has joined #openstack-cinder05:16
*** alonma has quit IRC05:21
*** alonma has joined #openstack-cinder05:22
*** Poornima has quit IRC05:24
*** anshul has joined #openstack-cinder05:24
*** alonma has quit IRC05:27
*** Poornima has joined #openstack-cinder05:29
*** alonma has joined #openstack-cinder05:32
*** anshul has quit IRC05:32
*** dave-mccowan has quit IRC05:33
*** Poornima has quit IRC05:34
*** Poornima has joined #openstack-cinder05:36
*** alonma has quit IRC05:37
*** alkhodos has quit IRC05:39
*** leeantho has joined #openstack-cinder05:39
*** huangtianhua has quit IRC05:39
*** huanan_L has quit IRC05:40
*** huangtianhua has joined #openstack-cinder05:40
*** huanan_L has joined #openstack-cinder05:40
*** alonma has joined #openstack-cinder05:40
*** mylu has quit IRC05:41
*** Poornima has quit IRC05:42
*** ak has joined #openstack-cinder05:43
*** ianbrown_ has quit IRC05:44
*** alonma has quit IRC05:45
*** ianbrown_ has joined #openstack-cinder05:45
*** mylu has joined #openstack-cinder05:46
*** alonma has joined #openstack-cinder05:46
*** anshul has joined #openstack-cinder05:48
akhi guys enable_service c-bak should enable the cinder backup service05:49
akright can some one confirm ?05:49
*** alonma has quit IRC05:51
*** alonma has joined #openstack-cinder05:53
akhi guys05:56
akcan some please help me out ?05:56
*** alonma has quit IRC05:57
*** alonma has joined #openstack-cinder05:59
*** sgotliv has quit IRC06:00
*** alonma has quit IRC06:03
*** mylu has quit IRC06:04
*** mylu has joined #openstack-cinder06:05
*** Poornima has joined #openstack-cinder06:07
*** esker has joined #openstack-cinder06:08
*** anshul has quit IRC06:09
*** ak has quit IRC06:12
*** mylu has quit IRC06:12
*** mylu has joined #openstack-cinder06:13
*** anshul has joined #openstack-cinder06:21
*** esker has quit IRC06:21
*** jwcroppe has quit IRC06:22
*** ChubYann has quit IRC06:22
*** nkrinner has joined #openstack-cinder06:25
*** anshul has quit IRC06:26
sheelak: yes, what;s the problem?06:26
sheelak:tell me06:27
*** lpetrut has joined #openstack-cinder06:30
*** esker has joined #openstack-cinder06:32
openstackgerritVincent Hou proposed openstack/cinder: Migration: take the local_path for the source volume  https://review.openstack.org/25022006:34
*** alonma has joined #openstack-cinder06:41
thingeesmcginnis: for the record, this reference implementation thing is starting to get annoying https://twitter.com/Thingee/status/69729148484485120106:41
thingeeoh trust me, this was in my head the moment I started06:44
thingeewhoops wrong window06:44
*** alonma has quit IRC06:45
sheel:)06:46
*** alonma has joined #openstack-cinder06:47
*** anshul has joined #openstack-cinder06:47
*** csky has quit IRC06:48
*** alonma has quit IRC06:51
*** ianbrown_ has quit IRC06:52
*** ianbrown_ has joined #openstack-cinder06:53
*** alonma has joined #openstack-cinder06:53
*** bvivek has joined #openstack-cinder06:56
*** bvivek_ has joined #openstack-cinder06:56
*** alonma has quit IRC06:58
*** hparekh has joined #openstack-cinder07:00
*** alonma has joined #openstack-cinder07:00
*** chhavi has quit IRC07:04
*** alonma has quit IRC07:05
*** cfriesen has quit IRC07:06
*** alonma has joined #openstack-cinder07:07
*** alonma has quit IRC07:11
*** alonma has joined #openstack-cinder07:13
*** mylu has quit IRC07:13
*** belmoreira has joined #openstack-cinder07:13
*** arecknag has joined #openstack-cinder07:14
*** rcernin has joined #openstack-cinder07:14
*** alonma has quit IRC07:17
*** chhavi has joined #openstack-cinder07:18
*** alonma has joined #openstack-cinder07:19
*** alonma has quit IRC07:23
*** lpetrut has quit IRC07:23
*** alonma has joined #openstack-cinder07:25
*** mragupat has quit IRC07:27
*** lpetrut has joined #openstack-cinder07:27
*** alonma has quit IRC07:30
*** mudassirlatif has joined #openstack-cinder07:30
*** anshul has quit IRC07:34
*** alonma has joined #openstack-cinder07:38
*** alonma has quit IRC07:43
*** alonma has joined #openstack-cinder07:44
*** openstackgerrit has quit IRC07:47
*** openstackgerrit has joined #openstack-cinder07:48
*** alonma has quit IRC07:49
*** alonma has joined #openstack-cinder07:51
*** rcernin has quit IRC07:52
*** isaacb has joined #openstack-cinder07:55
*** lpetrut has quit IRC07:55
*** andymaier has joined #openstack-cinder07:55
*** alonma has quit IRC07:55
*** esker has quit IRC07:56
*** alonma has joined #openstack-cinder07:57
*** e0ne has joined #openstack-cinder07:58
*** Poornima is now known as Poornima|lunch07:58
*** Poornima|lunch is now known as Poornima07:58
*** huanan_L has quit IRC07:58
*** esker has joined #openstack-cinder07:59
*** huanan_L has joined #openstack-cinder07:59
*** alonma has quit IRC08:01
*** alonma has joined #openstack-cinder08:03
*** leeantho has quit IRC08:05
*** alonma has quit IRC08:07
*** alonma has joined #openstack-cinder08:15
*** rcernin has joined #openstack-cinder08:15
*** sinese_ has joined #openstack-cinder08:17
*** liverpooler has joined #openstack-cinder08:19
*** alonma has quit IRC08:20
*** shyama has quit IRC08:21
*** alonma has joined #openstack-cinder08:22
*** jistr|doc has joined #openstack-cinder08:22
*** alonma has quit IRC08:26
*** alonma has joined #openstack-cinder08:28
*** ndipanov has joined #openstack-cinder08:32
*** alonma has quit IRC08:32
*** ndipanov has quit IRC08:33
*** alonma has joined #openstack-cinder08:34
*** alonma has quit IRC08:35
*** alonma has joined #openstack-cinder08:35
*** smoriya_ has quit IRC08:37
*** e0ne has quit IRC08:39
*** salv-orlando has quit IRC08:44
*** shyama has joined #openstack-cinder08:52
*** laughterwym has joined #openstack-cinder08:56
*** laughterwym has quit IRC09:00
openstackgerritMatan Sabag proposed openstack/cinder: Support for consistency groups in ScaleIO driver  https://review.openstack.org/26962409:12
openstackgerritMatan Sabag proposed openstack/cinder: Manage/unmanage volume in ScaleIO driver  https://review.openstack.org/26834409:12
*** ndipanov has joined #openstack-cinder09:13
*** jwcroppe has joined #openstack-cinder09:22
*** isaacb has quit IRC09:23
*** alonma has quit IRC09:23
*** jwcroppe has quit IRC09:26
*** lpetrut has joined #openstack-cinder09:27
*** alonma has joined #openstack-cinder09:28
*** jistr|doc has quit IRC09:46
*** mvk has joined #openstack-cinder09:52
*** mvk has quit IRC09:56
*** sheel has quit IRC10:07
*** e0ne has joined #openstack-cinder10:07
*** mvk has joined #openstack-cinder10:16
*** sinese_ has quit IRC10:16
*** yhayashi has quit IRC10:26
*** alonma has quit IRC10:37
*** alonma has joined #openstack-cinder10:37
*** salv-orlando has joined #openstack-cinder10:41
*** alonma has quit IRC10:42
*** alonma has joined #openstack-cinder10:43
*** dims has joined #openstack-cinder10:48
*** alonma has quit IRC10:48
*** alonma has joined #openstack-cinder10:49
*** tpsilva has joined #openstack-cinder10:53
*** alonma has quit IRC10:54
*** alonma has joined #openstack-cinder10:56
*** laughterwym has joined #openstack-cinder10:58
*** alonma has quit IRC11:00
*** alonma has joined #openstack-cinder11:02
*** NightKhaos has quit IRC11:04
*** lprice has joined #openstack-cinder11:05
*** alonma has quit IRC11:06
*** lprice1 has quit IRC11:07
*** alonma has joined #openstack-cinder11:08
*** laughterwym has quit IRC11:09
*** NightKhaos has joined #openstack-cinder11:11
*** alonma has quit IRC11:12
*** alonma has joined #openstack-cinder11:16
*** NightKhaos has quit IRC11:16
*** NightKhaos has joined #openstack-cinder11:19
*** alonma has quit IRC11:19
*** alonma has joined #openstack-cinder11:20
openstackgerritSheel Rana proposed openstack/cinder-specs: Summary Messages for different cinder operations  https://review.openstack.org/27393811:22
*** changbl has quit IRC11:25
openstackgerritPetrut Lucian proposed openstack/os-brick: [WIP] Add Windows connectors  https://review.openstack.org/27252211:31
*** ociuhandu has quit IRC11:35
*** ildikov has quit IRC11:38
*** erlon has joined #openstack-cinder11:43
*** kmartin__ has quit IRC11:48
*** sinese_ has joined #openstack-cinder11:49
swamireddyPoornima: Please fix the Jenkins failiures for  https://review.openstack.org/#/c/235752/11:52
*** xek has quit IRC11:53
*** sid14726 has quit IRC11:54
Poornimaswamireddy, going through http://logs.openstack.org/52/235752/8/check/gate-tempest-dsvm-large-ops/dc1f5ad/console.html11:54
*** permalac has joined #openstack-cinder11:55
*** links has quit IRC11:56
Poornimaswamireddy, I see number of failures in console but check through local setup patch works fine11:57
*** szblcs has joined #openstack-cinder11:58
szblcshello11:58
szblcsCan I use FreeNAS as a storage provider for OpenStack? I've got FreeNAS configured with iSCSI and NFS.11:58
*** mvk has quit IRC12:00
swamireddyPoornima: do "recheck"12:02
*** mvk has joined #openstack-cinder12:02
*** ociuhandu has joined #openstack-cinder12:05
*** Poornima has quit IRC12:10
*** Poornima has joined #openstack-cinder12:12
*** ildikov has joined #openstack-cinder12:22
openstackgerritRonen Mesonzhnik proposed openstack/cinder: Support backup import on another Storage database  https://review.openstack.org/15396912:24
*** sinese_ has quit IRC12:25
*** sinese_ has joined #openstack-cinder12:27
*** edmondsw has joined #openstack-cinder12:29
*** funenutron has joined #openstack-cinder12:29
*** links has joined #openstack-cinder12:36
*** ildikov has quit IRC12:38
*** krotscheck_dcm is now known as krotscheck12:38
*** bvivek_ has quit IRC12:40
*** bvivek has quit IRC12:40
*** ildikov has joined #openstack-cinder12:44
*** vgridnev has joined #openstack-cinder12:44
*** funenutron has quit IRC12:45
*** alonma has quit IRC12:53
*** sid14726 has joined #openstack-cinder12:57
*** Poornima has quit IRC12:59
*** alonma has joined #openstack-cinder12:59
*** alonma has quit IRC13:03
*** alonma has joined #openstack-cinder13:03
*** bvivek has joined #openstack-cinder13:06
*** bvivek_ has joined #openstack-cinder13:06
*** e0ne has quit IRC13:09
*** vgridnev has quit IRC13:09
*** e0ne has joined #openstack-cinder13:09
openstackgerritDinesh Bhor proposed openstack/cinder: Fix 500 error if 'offset' is out of range  https://review.openstack.org/27835413:10
*** haomaiwa_ has joined #openstack-cinder13:10
*** edmondsw has quit IRC13:11
*** gouthamr has joined #openstack-cinder13:11
*** gouthamr_ has joined #openstack-cinder13:13
DuncanTszblcs: I'm not aware of a FreeNAS plugin, so the only way you could do it without writing code would be to create a large lun on it, then run cinder/LVM on top of the lun13:15
DuncanTszblcs: This works fine if your c-vol node has good networking13:16
*** gouthamr has quit IRC13:16
*** martyturner has joined #openstack-cinder13:19
*** vgridnev has joined #openstack-cinder13:19
dulekgeguileo: I've just walked through c-api's volume flow with sborkowski to identify possible race conditions.13:19
szblcsDuncanT: thanks. I'm a bit over my head. So you mean I create the iSCSI target in FreeNAS, then mount on the OpenStack node?13:19
geguileodulek: And how did it go?13:19
dulekgeguileo: And this is craaaazy… I mean - how paranoid we want to be?13:20
DuncanTszblcs: Yes13:20
geguileodulek: XD XD XD13:20
dulekgeguileo: For example - what if source volume's size changes?13:20
szblcsDuncanT: thank you, I'm going to try this.13:20
dulekgeguileo: Or source vol gets retyped?13:20
geguileodulek: That's a valid question13:21
dulekgeguileo: Or snapshot's AZ changes?13:21
*** vgridnev has quit IRC13:21
dulekgeguileo: Or extra specs on volume type changes?13:21
DuncanTdulek: does the cinder API allow those operations? If so, they are valid race conditions13:21
geguileodulek: Mmmmm, we really should be protecting those cases in the other operations13:21
dulekDuncanT: Yeah, we don't lock source snap/vol from anything more than deletion.13:21
DuncanTdulek: Editing extra specs does not change existing volumes, so that one at least is safe13:21
geguileodulek: Like not allowing an extend if it is being used as a source to create another volume13:22
dulekDuncanT: Ah, good to know.13:22
geguileodulek: I think the way to go for now is to ignore the extra paranoia and add those cases as bugs13:22
geguileodulek: Like "extend action should not be allowed when the volume is used as a source volume"13:23
geguileodulek: Same for retype operation13:23
*** e0ne has quit IRC13:24
dulekOkay, so how about - we'll start with something simple (I think ensuring that source resource wasn't deleted while processing), and start to test "paranoid" cases to confirm if there's a bug to report?13:24
*** e0ne has joined #openstack-cinder13:25
dulekgeguileo: I've identified places in your commits where some cases like that aren't taken into account. I can point these while reviewing, but I think these should be addressed in future commits, otherwise we end up with really complicated reviews.13:26
geguileodulek: I agree13:27
geguileodulek: I haven't taken those into account, because I was just removing races using the same logic we currently have13:27
*** vgridnev has joined #openstack-cinder13:27
geguileodulek: I think we should open new bugs for those cases and then fix the swap-and-compare code to fix them13:27
openstackgerritPetrut Lucian proposed openstack/os-brick: [WIP] Add Windows connectors  https://review.openstack.org/27252213:31
dulekgeguileo: Okay, this sometimes seems to be pretty hard, but fixing them one-by-one may work. Or at least we'll know about crazy edge-cases. Thanks for opinion!13:31
geguileodulek: Thanks for looking into it13:32
*** gouthamr_ has quit IRC13:32
*** csky has joined #openstack-cinder13:38
*** xek has joined #openstack-cinder13:38
*** permalac has quit IRC13:39
*** akerr has joined #openstack-cinder13:39
*** shyama_ has joined #openstack-cinder13:41
*** timcl has joined #openstack-cinder13:41
*** shyama has quit IRC13:42
*** shyama_ is now known as shyama13:42
*** ildikov has quit IRC13:43
*** gouthamr has joined #openstack-cinder13:47
*** dave-mccowan has joined #openstack-cinder13:51
*** eharney has joined #openstack-cinder13:52
*** alonma has quit IRC13:54
*** laughterwym has joined #openstack-cinder13:54
*** alonma has joined #openstack-cinder13:57
*** akshai has joined #openstack-cinder13:59
*** haomaiwa_ has quit IRC14:01
*** haomaiwa_ has joined #openstack-cinder14:01
*** sheel has joined #openstack-cinder14:04
e0neeharney: hi Eric. could you please review my patch to NFS connector https://review.openstack.org/27778814:05
*** rlrossit has joined #openstack-cinder14:06
*** isaacb has joined #openstack-cinder14:10
*** mtanino has joined #openstack-cinder14:12
*** julim has joined #openstack-cinder14:15
*** porrua has joined #openstack-cinder14:16
smcginnisthingee: Yeah, saw that. :[14:16
*** bvivek_ has quit IRC14:17
*** bvivek has quit IRC14:17
*** eharney has quit IRC14:21
*** cfriesen has joined #openstack-cinder14:21
e0neeharney: thanks for the review14:21
krotscheckI've got a middleware configuration update that should be pretty quick, any cores around to look at https://review.openstack.org/#/c/275751/  ?14:21
*** mriedem has joined #openstack-cinder14:22
*** jordanP has joined #openstack-cinder14:22
*** permalac has joined #openstack-cinder14:22
smcginniskrotscheck: Thanks for following up on that one.14:25
*** eharney has joined #openstack-cinder14:25
krotschecksmcginnis: Of course, thanks for the +2 :)14:25
*** krtaylor has quit IRC14:25
*** sid14726 has quit IRC14:25
*** links has quit IRC14:27
*** changbl has joined #openstack-cinder14:29
*** jgregor has joined #openstack-cinder14:30
*** cknight has joined #openstack-cinder14:30
*** crose has joined #openstack-cinder14:31
*** ildikov has joined #openstack-cinder14:32
*** bill_az has joined #openstack-cinder14:33
*** akshai has quit IRC14:34
*** szblcs has quit IRC14:34
*** itzdilip has quit IRC14:34
*** Vasanthi has quit IRC14:34
*** pradipm has joined #openstack-cinder14:35
*** xyang1 has joined #openstack-cinder14:37
*** akshai has joined #openstack-cinder14:39
*** knikolla has joined #openstack-cinder14:43
jordanPsmcginnis, DuncanT hi. I am trying to add a new tempest scenario to validate volume extend. This is not tested in Tempest. The review is here: https://review.openstack.org/#/c/249100/ We have a disagreement with sdague. Maybe you can weight in ?14:44
jordanPi think this test is super useful (just look a the 2 bug reports linked in the commit message)14:44
DuncanTSure, I'll take a look14:45
jordanPI figured that if some cinder cores say 'yeah, we need this', maybe Sean will change his mind14:45
*** merooney has joined #openstack-cinder14:45
jordanPthanks !14:45
smcginnisjordanP: We've been having a lot of "disagreements" when it comes to tempest lately. :/14:46
eharneyjordanP: i'll take a look there too.  I updated my manage/unmanage tests last week based on your comments, thanks for the reviews there14:47
jordanPsmcginnis, how so ? I live in both world (tempest and cinder, mostly Tempest though) maybe I can help ?14:47
smcginnisjordanP: Not sure if you can. Seems a lot of folks want to dictate what cinder can and can't implement for features.14:48
jordanPhum that sucks14:48
jordanPas far as I know the general idea is that "what can't be tested in the Gate can't land into projects" which I can understand. Now for a couple of weeks it seems the QA/infra community is worried about excessive test duration14:50
*** alonma has quit IRC14:50
DuncanTjordanP: We're having to re-evaluate the gate since the amount of resources dedicated to it started to look bounded. It's definitely a discussion worth having, but to block patches to tempest on it now is not good. I've commented along those lines on the patch14:51
smcginnisjordanP: Yeah, I get the desire to test in gate. I just don't think that's practical to have a healthy, useful platform if it's strictly bound by that.14:52
jordanPDuncanT, thanks a lot14:52
smcginnisjordanP: My problem has been resistance to actually get tests to cover some of that functionality.14:52
*** esker has quit IRC14:52
*** timcl has quit IRC14:53
scottdae0ne: DuncanT dulek smcginnis The patch set for api-microversions is passing Jenkins and could use some review attention:14:53
scottdahttps://review.openstack.org/#/c/22491014:53
DuncanTsmcginnis: I'm going to suggest that we split out some of the tempest tests so they only get run on cinder/nova commits, and that neutron look at doing the same...14:53
smcginnisscottda: Great news! Thanks!14:53
DuncanTscottda: Thanks!14:53
smcginnisDuncanT: I'd be behind that.14:54
scottdaI'll put something on the meeting agenda for this as well. I can also schedule the demo I talked about during mid-cycle.14:54
*** alonma_ has joined #openstack-cinder14:54
smcginnisscottda: Excellent, I think that could help.14:54
scottdaAnd on the subject of Tempest tests...I've the same concerns/issues about how to get Tempest tests for microversions to land14:54
DuncanTscottda: Got a link to the review, please?14:54
scottdaDuncanT: scroll up ^^^14:55
scottdahttps://review.openstack.org/#/c/22491014:55
DuncanTscottda: For the tempest bits14:55
*** xyang has joined #openstack-cinder14:55
scottdaoh...no review yet14:55
scottdafor tempest14:55
DuncanTscottda: Ok, thanks14:55
scottdaI'm going to start with 1 or more functional tests14:55
*** baumann has joined #openstack-cinder14:55
scottdaI think that may be the best place to start. Plus cinder team has more control. I believe the Tempest bits will stretch out past the Mitaka deadlines.14:56
eharneyscottda: what about in-tree tempest tests?14:57
scottdaeharney: I'm all for that. smcginnis What is the status of your patch to move (some) Tempest tests into the Cinder tree?14:57
eharneyscottda: seems like a good fit for what you're doing if we can get it into a usable state soon (not sure how far out that is)14:58
smcginnisscottda: It works, but I'm hung up on why they aren't picked up by CIs using the *volume* filtering.14:58
jordanPin-tree tempest tests are nice but it's also fragile. You will have to import half of the tempest tree at runtime so each change in tempest can break your gate14:58
smcginnisscottda: If I don't specify a regex it does end up getting run.14:58
jordanPsmcginnis, you need to run the tox target like this : tox -all-plugin -- volume14:58
smcginnisjordanP: Ah! Thanks!14:58
*** diablo_rojo has joined #openstack-cinder14:59
*** akshai has quit IRC14:59
scottdaIn the long run, I'd like to have it all: functional tests, some in-tree Tempest, some regular Tempest (nova<->cinder). I'm mostly concerned about the schedule and deadlines.14:59
smcginnisjordanP: See, that's another one. When I went looking for help to figure that out, instead of getting useful information like changing the command line and warnings on fragility, I just started to get a lecture on not allowing features in. :/14:59
* smcginnis is sorry to be venting15:00
xyangsmcginnis: we need to ping to a specific commit.  see what manila does: https://github.com/openstack/manila/blob/master/contrib/ci/common.sh#L315:00
jordanPmanila is doing this in-tree tempest tests but now they are stuck at some specific revision of tempest (https://github.com/openstack/manila/blob/5244bf372e7e40d5ff51c488e41a7e92b39a5a59/contrib/ci/pre_test_hook.sh#L76)15:00
smcginnisxyang: What does that do?15:00
*** haomaiwa_ has quit IRC15:01
jordanPsmcginnis, see my github link15:01
xyangsmcginnis: so manila test with that tempest commit15:01
xyangsmcginnis: it won't be broken by any checkin in master of tempest15:01
smcginnisjordanP: Ah, gotcha.15:01
*** haomaiwa_ has joined #openstack-cinder15:01
jordanPsmcginnis, sorry about your bad experience with QA folks, we try to be nice though :)15:01
*** akshai has joined #openstack-cinder15:01
dulekhttps://review.openstack.org/261110 - finally merged!15:01
smcginnisjordanP: Haha, they are.15:01
smcginnisjordanP: Just got frstrated on this issue.15:02
eharneywould it be reasonable at all to just start by using the stable n-1 tempest branch for in-tree?15:02
*** csky has quit IRC15:02
dulekWe're one Gerrit restart from getting "Toggle CI" on Gerrits compare revisions pages. :D15:02
smcginnisjordanP: For the record, QA folks are unsung heros. ;)15:02
smcginnisdulek: Nice!15:03
smcginniseharney: I do think that's reasonable.15:03
eharneyjordanP: smcginnis: ^15:03
*** timcl has joined #openstack-cinder15:04
openstackgerritClinton Knight proposed openstack/cinder: Filter & goodness functions in NetApp drivers  https://review.openstack.org/27842015:06
xyangsmcginnis: there are definitely challenges to have in-tree tempest tests but we can't test some functionalities without them unfortunately as those tests are not accepted by tempest15:07
smcginnisxyang: That's how I feel.15:07
*** lprice has quit IRC15:07
*** esp has joined #openstack-cinder15:09
jordanPeharney, there's no "stable branch" for tempest15:09
*** vgridnev has quit IRC15:09
eharneyjordanP: oh, well, ok :)15:09
*** mtanino has quit IRC15:09
jordanPtempest has been "branch less"  for almost 2 years now15:09
jordanPit's "always stable" in a way :p15:09
*** vgridnev has joined #openstack-cinder15:11
*** dustins has joined #openstack-cinder15:12
*** permalac has quit IRC15:13
*** esp has quit IRC15:13
openstackgerritVincent Hou proposed openstack/cinder: Migration: take the local_path for the source volume  https://review.openstack.org/25022015:15
*** daneyon has joined #openstack-cinder15:16
openstackgerritGorka Eguileor proposed openstack/cinder-specs: Job Distribution to support HA A/A  https://review.openstack.org/23259515:17
*** daneyon__ has joined #openstack-cinder15:17
*** vgridnev has quit IRC15:17
openstackgerritClinton Knight proposed openstack/cinder: Add variable QoS to NetApp cDOT drivers  https://review.openstack.org/27843015:19
*** daneyon_ has quit IRC15:20
*** daneyon has quit IRC15:21
openstackgerritVincent Hou proposed openstack/cinder: Do not merge: check purpose  https://review.openstack.org/27843615:22
*** vincent_hou has joined #openstack-cinder15:24
*** daneyon has joined #openstack-cinder15:24
*** haypo has left #openstack-cinder15:25
*** thangp has joined #openstack-cinder15:25
*** daneyon__ has quit IRC15:27
xyangDuncanT: hi, are you okay with this one?  https://review.openstack.org/#/c/257170/15:27
DuncanTxyang: Yup, approved15:28
xyangDuncanT: you gave +1:)15:28
xyangDuncanT: thanks!15:28
*** mvk has quit IRC15:29
DuncanTxyang: It'll be important when we start doing the error listing stuff15:29
anteayaokay so extra-atcs15:29
anteayahttp://git.openstack.org/cgit/openstack/governance/tree/reference/projects.yaml#n92215:29
xyangDuncanT: sure15:29
*** mvk has joined #openstack-cinder15:29
*** mragupat has joined #openstack-cinder15:30
anteayaso there used to be a file called extra-atcs and it appears it has disappeared in favour of putting the information in projects.yaml in the governance repo15:30
*** mvk has quit IRC15:30
*** yuriy_n17 has joined #openstack-cinder15:30
xyangDuncanT: you probably noticed that there are a bunch of other patches for that same subject.  Appreciate you can review some more when you have time:)15:30
DuncanTxyang: Sure15:31
DuncanTanteaya: Thanks15:31
anteayaso smcginnis offers a patch to governance which adds whoever he wants to the projects.yaml file in that format15:31
*** mvk has joined #openstack-cinder15:31
anteayawelcome do ask me or fungi if there is anything that isn't clear15:31
anteayaand offer the patch without delay as summit passes are starting to be issued15:31
anteayaand the tc meeting is skipped next week15:31
*** daneyon_ has joined #openstack-cinder15:32
*** rcernin has quit IRC15:33
*** vgridnev has joined #openstack-cinder15:33
mc_nairDuncanT: were you able to dig up any of the background info for passing user encryption key in headers on any request that requires encryption?15:34
*** e0ne has quit IRC15:34
*** daneyon has quit IRC15:35
xyangsmcginnis: FYI - This one that I talked to DuncanT about (https://review.openstack.org/#/c/257170/) is for an approved cross-project spec https://review.openstack.org/#/c/156508.  There are still quite a few patches outstanding15:35
*** e0ne has joined #openstack-cinder15:35
*** wanghao has quit IRC15:37
smcginnisDuncanT, anteaya: Got it, thanks!15:38
smcginnisxyang: OK, I'll take a look.15:38
*** krtaylor has joined #openstack-cinder15:38
xyangsmcginnis: thanks15:38
*** arecknag has quit IRC15:39
vincent_houeharney: what is your comment for https://review.openstack.org/#/c/250220/? I have not found any yet.15:39
eharneyvincent_hou: patch set 1415:39
anteayasmcginnis: awesome, thank you15:40
anteayasmcginnis: if you can offer today then ttx might be able to get it rolling before he heads off for a week of holidays15:40
smcginnisanteaya: Thanks for pointing that out. I had no idea about it.15:40
*** jungleboyj has joined #openstack-cinder15:40
anteayasmcginnis: glad DuncanT asked about it15:40
smcginnisanteaya: Doing it right now.15:40
vincent_houeharney: you mean 'introduce a driver flag like _sparse_copy_volume_data'?15:40
anteayayeah so tough to keep folks informed about all the things15:40
anteayathanks for asking15:40
*** lprice has joined #openstack-cinder15:41
vincent_houeharney: I added it in the new patch set.15:41
DuncanTxyang: I've found 4 cinder-client patches for request id, I've got them all queued for review today15:42
eharneyvincent_hou: ok let me take a look15:42
xyangDuncanT: great.  thanks!15:42
vincent_houeharney: thx.15:43
openstackgerritVincent Hou proposed openstack/cinder: Do not merge: check purpose  https://review.openstack.org/27843615:44
*** vgridnev has quit IRC15:45
*** vgridnev has joined #openstack-cinder15:45
*** shyama has quit IRC15:45
*** liverpooler has quit IRC15:46
*** nkrinner has quit IRC15:49
*** jseiler has joined #openstack-cinder15:53
*** mtanino has joined #openstack-cinder15:56
*** haomaiwa_ has quit IRC16:01
*** sid14726 has joined #openstack-cinder16:01
*** rcernin has joined #openstack-cinder16:01
*** Guest67795 is now known as cfouts16:01
*** haomaiwa_ has joined #openstack-cinder16:01
*** salv-orlando has quit IRC16:01
*** esp has joined #openstack-cinder16:04
*** kmartin has joined #openstack-cinder16:05
smcginnisjbernard: Can you join the meeting?16:09
jbernardsmcginnis: yep16:10
jbernardsmcginnis: i thought noon16:10
smcginnisjbernard: Thanks!16:10
*** esp has quit IRC16:11
*** rajinir has joined #openstack-cinder16:11
*** esker has joined #openstack-cinder16:12
openstackgerritMerged openstack/python-cinderclient: Add Wrapper classes for list, dict, tuple  https://review.openstack.org/25717016:16
*** amoturi has joined #openstack-cinder16:16
*** esker has quit IRC16:16
*** garthb has joined #openstack-cinder16:17
*** esp has joined #openstack-cinder16:17
*** esker has joined #openstack-cinder16:17
*** cbader has quit IRC16:22
*** rcernin has quit IRC16:24
*** esp has quit IRC16:24
*** pradipm has quit IRC16:28
*** raildo-afk is now known as raildo16:29
*** jdurgin1 has joined #openstack-cinder16:31
*** kfarr has joined #openstack-cinder16:33
*** boris-42 has quit IRC16:33
*** laughterwym has quit IRC16:34
*** vgridnev has quit IRC16:36
*** knomura has joined #openstack-cinder16:36
*** jgriffith_away is now known as jgriffith16:37
*** belmoreira has quit IRC16:41
*** mudassirlatif has quit IRC16:41
*** pradipm has joined #openstack-cinder16:42
*** pradipm has quit IRC16:44
*** alonma_ has quit IRC16:44
*** alonma has joined #openstack-cinder16:45
*** alonma has quit IRC16:49
*** salv-orlando has joined #openstack-cinder16:50
*** alonma has joined #openstack-cinder16:51
*** apoorvad has joined #openstack-cinder16:51
*** knomura has quit IRC16:51
*** asselin has quit IRC16:52
*** marcusvrn_ has joined #openstack-cinder16:53
*** alonma has quit IRC16:55
openstackgerritMichael Price proposed openstack/cinder: NetApp: E-Series remove snapshot limitations  https://review.openstack.org/27851516:55
mriedemdiablo_rojo: here is a great example of unnecessary DocImpact https://review.openstack.org/#/c/245827/16:55
mriedemb/c there are no docs for package requirements16:56
mriedemespecially unused ones16:56
mriedemderp16:56
*** asselin has joined #openstack-cinder16:57
diablo_rojomriedem: Ha ha thanks16:57
*** alonma has joined #openstack-cinder16:57
*** leeantho has joined #openstack-cinder16:59
*** jungleboyj has quit IRC16:59
*** asselin__ has joined #openstack-cinder16:59
jgriffithmriedem: Nice one!!!17:00
*** haomaiwa_ has quit IRC17:01
*** asselin has quit IRC17:01
smcginnis:)17:01
*** haomaiwang has joined #openstack-cinder17:01
DuncanTjgriffith: I'm worried that taking /too/ simple a solution will result in a design that is hard to move forward and require us to do another redesign with nasty migration/upgrade cases again17:01
jgriffithDuncanT: haha, that's funny I thought the opposite17:02
geguileoDuncanT: Would you be ok if first it gets implemented with polling17:02
*** alonma has quit IRC17:02
jgriffithDuncanT: but you and geguileo have way more insight here than I do so that's cool17:02
jgriffithDuncanT: geguileo just throwing it out there17:02
geguileojgriffith: Oh, and that's why I brought it up17:02
jgriffithwe tend to build some overly complex things that end up being less than maintainable17:02
geguileojgriffith: Looking for an simpler alternative  :-)17:02
smcginnisjgriffith: +117:02
jgriffithgeguileo: :)17:02
smcginnisbbl17:03
geguileojgriffith: That's what I thought when I was writting the specs17:03
jgriffithgeguileo: I wish I had something more concrete to offer up right now17:03
geguileoWe should try to do something less complicated17:03
DuncanTgeguileo: If you can explain how it would evolve in future (roughly) then maybe... I'm not a big fan of the new job distribution stuff already, I liked the cluster attribute stuff much more17:03
geguileoDuncanT: Well, it's basically the same17:03
jgriffithgeguileo: effecting the scheduler seems like the right thing... which then comes back to "everything" should be going through the scheduler17:03
geguileoDuncanT: But I just skip the whole renaming host to cluster17:03
geguileoDuncanT: The rest is the same (same problems)17:04
DuncanTgeguileo: That makes the host disable stuff harder, since identifying a host is harder now17:04
geguileojgriffith: The problem with the scheduler approach is that it sends it to the same queue for all the nodes17:04
DuncanTgeguileo: host now means different things in different places17:04
geguileoDuncanT: I believe it means the same everywhere17:04
DuncanTgeguileo: For when you're not talking to c-vol, it means a host.... when you're talking to c-vol it means several hosts17:05
*** alonma has joined #openstack-cinder17:05
geguileoDuncanT: But isn't that the same for the scheduler?17:06
DuncanTgeguileo: No... schedulers can run on different hosts...17:06
geguileoDuncanT: Your are right, we just don't address them directly17:07
DuncanTgeguileo: Yup. Maybe we should do the same for c-vol - add 'cluster', which is a service name like scheduler is?17:07
DuncanTgeguileo: Old code will not be H/A aware, but that is ok17:07
DuncanTgeguileo: So both the hsot and the cluster queues will be there, and we can make stuff H/A (cluster) aware a bit at a time?17:08
*** e0ne has quit IRC17:08
geguileoDuncanT: It will take 3 releases to make the change17:08
geguileoRename host to cluster17:08
DuncanTgeguileo: No, don't rename, have both17:08
*** sid14726 has quit IRC17:09
geguileoDuncanT: But at some point host field will be not used17:09
geguileoAnd we will want to drop it17:09
DuncanTgeguileo: host will be used for things like disables17:09
DuncanTgeguileo: and differentiating where things came from (stats updates etc) for debugging)17:09
*** alonma has quit IRC17:10
DuncanTgeguileo: Maybe we drop it in the volumes table eventually, but initially leave it there17:10
geguileoThat's the same as we would have with this proposal host and node_name instead of cluster and node_name17:10
geguileoDuncanT: If you really hate the concept of using host instead of cluster17:10
geguileoI will take the renaming approach17:11
DuncanTgeguileo: give me a few minutes to type this up17:11
geguileoBut I think it will be a lot worse for us in the code and with backports17:11
*** alonma has joined #openstack-cinder17:11
* mc_nair puts on his best salesman face...17:12
mc_nairdo you know what's been missing from *your* life?  Nested quota reviews.... but for a limited time.... https://review.openstack.org/#/c/274825/17:12
*** xyang has quit IRC17:13
*** xyang has joined #openstack-cinder17:13
hemnamc_nair, :)17:13
hemnaos-brick + privsep update17:13
hemnagus and I have been working on getting his patch against os-brick working17:14
hemnatoday is the first time that I've seen it work.17:14
hemnahttp://paste.openstack.org/show/486583/17:14
hemnaI'm not sure it's going to make the deadline for friday17:14
hemnaI've been using my brick_initiator.py script that lives in my diediedie repo on github.  http://github.com/WaltHP/diediedie17:15
hemnaI'm going to start testing it against nova/cinder today17:15
*** vgridnev has joined #openstack-cinder17:15
*** alonma has quit IRC17:16
hemnathe os-brick privsep patch : https://review.openstack.org/#/c/277224/17:17
mc_nairhemna: is there anything I could help w/ review for the time being on that? or is it mostly just testing / stabilizing at this point?17:17
hemnamc_nair, I haven't gone through the CI failures at this point because it just hasn't worked locally until today.17:18
mc_nairgotcha17:18
hemnaI need to go through those as well as doing some tests17:18
*** alonma has joined #openstack-cinder17:18
hemnaI'm going to try testing iSCSI first, then FC17:18
*** arch-nemesis has joined #openstack-cinder17:22
hemnacinder list17:22
hemnaERROR: Versioning for this project requires either an sdist tarball, or access to an upstream git repository. Are you sure that git is installed?17:22
hemnahuh17:22
hemnanever seen that one before17:22
*** alonma has quit IRC17:22
diablo_rojohemna: I've seen that one before, but I don't remember what had caused it..17:23
diablo_rojohemna: Part of me wants to say it was some connection issue and when I reran it was fine. Idk. I'd have to go look.17:23
hemnaI get it every time17:23
hemnait's odd.17:23
*** vincent_hou has quit IRC17:24
DuncanTgeguileo: https://gist.github.com/Funcan/321e955ee4990f3c3ed217:26
diablo_rojohemna: Well do you have git installed? ;)17:27
DuncanThemna: I've seen that before on a devstack... I made a new one rather than trying to fix it17:27
hemnahehe yah, that's probably what I'll do.17:27
hemnafire up a new vagrant and be done with it.17:28
geguileoDuncanT: I don't think that would work...17:29
*** leeantho has quit IRC17:29
geguileoDuncanT: What would you set in the host field of the Volume table?17:29
geguileoDuncanT: As it should be the cluster17:30
DuncanTgeguileo: As it said, pick any host in the cluster17:30
hemnagus, ping17:30
DuncanTgeguileo: For calls we've moved over to go via the scheduler, it will be ignored17:30
*** mudassirlatif has joined #openstack-cinder17:30
geguileoDuncanT: Then when we have to decide on which cluster it is, how do we do it?17:30
DuncanTgeguileo: The scheduler will know via the stats broadcast from the c-vol node which cluster it is in, if any17:31
*** mylu has joined #openstack-cinder17:31
geguileoCreate will ignore it, but all other operations will pay attention to it17:31
DuncanTgeguileo: It will be *a* valid host to talk to that volume, every time, just not necessarily the only one17:31
geguileoBut it could be down17:31
DuncanTgeguileo: Yes. We fix the other operations to go via the scheduler one by one. Until they're all done (and there aren't that many) then you don't get HA17:32
DuncanTgeguileo: It means we can move (and test) them one by one17:33
DuncanTgeguileo: It's more patches, but each one will be simple17:33
geguileoSo the scheduler would decide which node gets to do each operation from the up and not disabled ones?17:33
DuncanTYes, as jgriffith suggested17:34
*** krtaylor has quit IRC17:34
geguileoBut then we could be losing requests17:34
geguileoSince we don't know immediately when a node is down17:34
geguileoWe could have sent that node a lot of requests until the heartbeat expires17:34
geguileoI discarded the option of the scheduler choosing the specific node17:35
geguileoWhen I started looking into this because that would create a lot of issues17:35
DuncanTgeguileo: The scheduler can send each rpc to the cluster and let rabbit pick a node... then you have to do host disabling in the manager, but that might be ok17:36
*** porrua has quit IRC17:37
DuncanTthe disable can stop the node listening to the cluster queue, but not the node queue (so it can get enabled again)17:37
DuncanTDisable would behave the same as today for none H/A calls, but that's ok too17:37
geguileoThat's the alternative I was proposing to the DB polling17:37
geguileoBut without changing everything to go through the scheduler17:38
DuncanTYup, I get that now. I think it is part of the solution for sure. Moving everything to go via the scheduler makes upgrade easier I think17:38
geguileoBecause my experience with creating a lot of patches for something is *not* good17:38
geguileoDuncanT: I don't want to mix things17:38
DuncanTWell, being too clever and doing things implicitly has hurt us before too17:39
*** boris-42 has joined #openstack-cinder17:39
geguileoWhat do you mean by doing things implicitly?17:39
DuncanTChanging the meaning of the host field to not mean a host (yes, I'm aware people hack it today, but that comes with its own set of weird behaviours)17:40
DuncanTActually, if you add the cluster field, does it need things to go via the scheduler? If there's a cluster set, send the RPC there, job done17:41
geguileoDuncanT: Ok, so for you using host is a no go17:41
geguileoDuncanT: I can do everything exactly the same but using cluster field instead17:41
geguileoDuncanT: I don't need to pass everything through the scheduler17:41
DuncanTYou'd need some way to set the cluster attribute for all existing volumes, but you could add an API call and let the scheduler fill them in in batches17:41
geguileoDuncanT: That's not a problem, because that's part of the reason why we use 3 releases for the change17:42
geguileoSince we support rolling upgrades17:42
geguileoFirst we write to both fields the same value and only use the host field17:43
geguileoThen we write to both values and use the cluster field17:43
DuncanTgeguileo: I don't think you need 3 releases17:43
geguileoFinally we only read and write from the cluster field17:43
geguileoAnd you are finally able to drop the host field17:43
geguileoThe last 2 happen on the same rolling upgrade17:44
DuncanTWhy write both and only use the host?17:44
geguileoBecause you have to be compatible while doing rolling upgrades17:44
geguileoAnd you want to start populating the cluster field17:45
DuncanTYou can write it and use it if it is there, in the same release I think - it will never get set until there's a c-vol ready to receive it, and a scheduler filling it in17:45
*** krtaylor has joined #openstack-cinder17:45
geguileoYou cannot use it in the first release17:45
geguileoBecause some old node may have only written to host17:46
DuncanTWhy not?17:46
geguileoWhile you were upgrading17:46
geguileoSo host and cluster would be different17:46
DuncanTIf cluster is None, use host17:46
geguileoIt's not that simple17:46
DuncanTWhy not?17:47
*** eharney has quit IRC17:47
geguileoBecause we may create it with a new version and migrate it with an old version17:48
geguileoSo we would end up with the cluster and the host being different17:48
geguileoAnd the right one is the host17:48
geguileoThat's part of the pain of implementing this together with rolling upgrades17:48
geguileo*Everything* is more complicated17:49
*** dustins has quit IRC17:49
DuncanTAh, got you. Give me a few minutes to think, please :-)17:49
geguileoDuncanT: Thanks :-)17:49
DuncanTPretty sure we can do it in one release via version pin walking, but I need to check17:50
DuncanTIt's a bit of a hack, but I'll try to gist it17:50
*** timcl has quit IRC17:52
geguileoDuncanT: I really hope you can figure out how to do it in just 1 release  :-)17:52
*** rcernin has joined #openstack-cinder17:53
*** martyturner has quit IRC17:53
diablo_rojohttps://review.openstack.org/#/c/245629/ Anyone else care to have an opinion on a common policy scenario across all projects? DuncanT? eharney? Already got input from jgriffith and jungleboyj- thank you again :)17:55
*** sid14726 has joined #openstack-cinder17:55
*** knomura has joined #openstack-cinder17:56
*** knomura has quit IRC17:56
*** mvk has quit IRC17:59
*** martyturner has joined #openstack-cinder17:59
*** haomaiwang has quit IRC18:01
*** haomaiwa_ has joined #openstack-cinder18:01
*** dustins has joined #openstack-cinder18:01
*** igornsa_ has joined #openstack-cinder18:03
*** esker has quit IRC18:04
*** krtaylor has quit IRC18:08
*** ociuhandu_ has joined #openstack-cinder18:08
*** itlinux has quit IRC18:11
*** ociuhandu has quit IRC18:11
*** ociuhandu_ is now known as ociuhandu18:11
*** porrua has joined #openstack-cinder18:13
*** mylu has quit IRC18:16
*** mylu has joined #openstack-cinder18:17
openstackgerritSean McGinnis proposed openstack/cinder: Remove Cisco FC Zone Manager Driver  https://review.openstack.org/27820118:18
*** alonma has joined #openstack-cinder18:18
*** jdurgin1 has quit IRC18:19
*** alonma_ has joined #openstack-cinder18:19
*** krtaylor has joined #openstack-cinder18:20
*** e0ne has joined #openstack-cinder18:20
openstackgerritMichael Rowden proposed openstack/cinder: Correcting thin provisioning behavior  https://review.openstack.org/27540818:22
*** cbader has joined #openstack-cinder18:23
*** alonma has quit IRC18:23
*** alonma_ has quit IRC18:24
DuncanTdiablo_rojo: Commented18:25
*** alonma has joined #openstack-cinder18:26
diablo_rojoDuncanT: Thank you :)18:26
DuncanTgeguileo: Im pretty sure I know how to do it, as long as nobody thinks it is too much of a hack. Thanks for going through the problems with me slowly until I understood them :-)18:26
geguileoDuncanT: So you found a way to do it in only 1 release?18:27
geguileoThat's awesome!!18:27
*** mylu has quit IRC18:28
*** mriedem has quit IRC18:28
*** jordanP has quit IRC18:29
geguileoDuncanT: Can you tell me the general idea?18:29
geguileoOr are you going to write something?18:29
*** ChubYann has joined #openstack-cinder18:29
*** alonma has quit IRC18:30
*** IlyaG has joined #openstack-cinder18:32
*** eharney has joined #openstack-cinder18:33
*** leeantho has joined #openstack-cinder18:34
*** mriedem has joined #openstack-cinder18:36
DuncanTI'll write something18:38
DuncanTJust having dinner18:39
geguileoDuncanT: Ok, we can talk tomorrow, thanks18:39
*** JoseMello has joined #openstack-cinder18:41
*** martyturner has quit IRC18:41
*** martyturner has joined #openstack-cinder18:42
*** cbader_ has joined #openstack-cinder18:46
*** jgregor has quit IRC18:46
*** liverpooler has joined #openstack-cinder18:47
*** sid14726 has quit IRC18:49
*** vincent_hou has joined #openstack-cinder18:49
*** cbader has quit IRC18:50
*** eharney has quit IRC18:50
vincent_houpatrickeast, smcginnis: Hi. I was wondering if you can help to look at this patch https://review.openstack.org/#/c/237387/.18:51
*** bardia has joined #openstack-cinder18:52
*** leeantho has quit IRC18:52
*** lpetrut has quit IRC18:53
*** xyang has quit IRC18:53
*** xyang has joined #openstack-cinder18:54
*** mylu has joined #openstack-cinder18:54
*** dramakri has joined #openstack-cinder18:57
ameadeDuncanT, eharney: https://review.openstack.org/#/c/274958/ another look?18:59
DuncanTameade: Added to my pile18:59
ameadety sir18:59
*** thurloat is now known as thurloat_isgone19:00
*** haomaiwa_ has quit IRC19:01
*** haomaiwang has joined #openstack-cinder19:01
*** eharney has joined #openstack-cinder19:03
*** igornsa_ has quit IRC19:04
*** xyang has quit IRC19:08
*** xyang1 has quit IRC19:09
*** mylu_ has joined #openstack-cinder19:11
*** mylu has quit IRC19:15
DuncanTgeguileo: https://gist.github.com/Funcan/7d21172017cb905fb71d19:18
*** mylu_ has quit IRC19:19
openstackgerritJohn Griffith proposed openstack/cinder: Replication v2.1  https://review.openstack.org/27579719:21
*** mylu has joined #openstack-cinder19:21
*** mylu has quit IRC19:23
*** mylu has joined #openstack-cinder19:24
scottdasmcginnis: You back from meetings?19:25
*** alonma has joined #openstack-cinder19:26
*** mylu has quit IRC19:27
*** mylu has joined #openstack-cinder19:30
*** alonma has quit IRC19:30
*** mylu has quit IRC19:31
*** angela-s has joined #openstack-cinder19:32
*** alonma has joined #openstack-cinder19:32
*** fthiagogv has quit IRC19:32
*** jgregor has joined #openstack-cinder19:32
*** jgregor has quit IRC19:33
*** mylu has joined #openstack-cinder19:35
*** jgregor has joined #openstack-cinder19:35
*** jungleboyj has joined #openstack-cinder19:36
*** alonma has quit IRC19:37
*** mylu has quit IRC19:37
*** sid14726 has joined #openstack-cinder19:38
openstackgerritMerged openstack/cinder: HNAS driver: retry on Connection reset fix  https://review.openstack.org/27635719:41
*** mylu has joined #openstack-cinder19:42
*** mylu has quit IRC19:43
*** rajinir has quit IRC19:45
openstackgerritMitsuhiro Tanino proposed openstack/cinder: [LVM] Restore target config during ensure_export  https://review.openstack.org/27142419:45
*** krotscheck is now known as krotscheck_dcm19:47
*** ociuhandu has quit IRC19:53
*** merooney has quit IRC19:55
*** jgriffith is now known as jgriffith_away19:57
openstackgerritMerged openstack/cinder: Added osprofiler headers to cors middleware  https://review.openstack.org/27575119:57
*** jgregor has quit IRC19:57
smcginnisscottda: Back now.19:58
*** haomaiwang has quit IRC20:01
*** haomaiwang has joined #openstack-cinder20:01
DuncanTjgriffith: You missed a bunch of comments from Xing on patch set 3 I think?20:01
*** mylu has joined #openstack-cinder20:01
DuncanTjgriffith_away: For replication20:01
scottdasmcginnis: I was wondering about the scheduling of a demo for microversions...20:03
scottdaYou and hemna cannot come Fri morning. Should we push to Monday-ish?20:04
smcginnisscottda: I'd like to be there, but I also don't want to be a roadblock.20:04
smcginnisscottda: If it works best Friday then do it and I can catch up on the recording.20:04
smcginnisscottda: Otherwise, Monday would work for me other than 1-2 CST.20:04
hemnascottda, Monday would be better for me20:04
hemnaI can't do Friday morning20:04
smcginnisSorry, 1-2:3020:05
scottdaOK. I'll just plan for Monday, morning like 16:00 UTC/9:00 MST20:05
scottdaand I'll send something out on the ML20:05
smcginnisscottda: Works for me.20:05
scottdathanks20:05
smcginnisscottda: Thanks for arranging it.20:05
* scottda is off to lunch20:05
hemnaok thanks man20:06
*** jgregor has joined #openstack-cinder20:06
*** xyang has joined #openstack-cinder20:08
*** alonma has joined #openstack-cinder20:09
*** xyang1 has joined #openstack-cinder20:12
openstackgerritKendall Nelson proposed openstack/os-brick: Lun id's > 255 should be converted to hex  https://review.openstack.org/27760620:13
*** andymaier has quit IRC20:13
*** lpetrut has joined #openstack-cinder20:13
openstackgerritEric Harney proposed openstack/cinder: Tests: Set volume listen port to test_service_listen_port  https://review.openstack.org/27859420:13
openstackgerritEric Harney proposed openstack/cinder: Profiler: don't call trace_cls if profiler is not enabled  https://review.openstack.org/27859520:13
*** alonma has quit IRC20:13
openstackgerritEric Harney proposed openstack/cinder: Profiler: make it possible to run without loading osprofiler  https://review.openstack.org/27859620:13
*** alonma has joined #openstack-cinder20:15
openstackgerritJohn Griffith proposed openstack/cinder: Replication v2.1  https://review.openstack.org/27579720:16
*** alonma has quit IRC20:20
*** jgriffith_away is now known as jgriffith20:22
*** dave-mcc_ has joined #openstack-cinder20:22
*** dave-mccowan has quit IRC20:22
openstackgerritSheel Rana proposed openstack/cinder: Bootable filter for listening volumes from CLI  https://review.openstack.org/27860520:22
*** alonma has joined #openstack-cinder20:22
*** dave-mccowan has joined #openstack-cinder20:23
*** ociuhandu has joined #openstack-cinder20:23
hemnanew gerrit ui, I hate you.  that is all.20:26
e0ne:)20:26
smcginnisThe rants are getting much shorter. :)20:26
sheel:)20:26
*** dave-mcc_ has quit IRC20:26
hemnamy tapeworm tells me what to do.  he's telling me to keep it brief.20:27
e0nehemna: any updates about privsep? I didn't have time to take a look on it today:(20:27
hemnae0ne, so the latest patch works from the brick_initiator.py standpoint.20:27
hemnabut....20:27
hemnait makes nova lock up.20:27
hemnaship it!20:27
*** alonma has quit IRC20:27
hemnae0ne, https://review.openstack.org/#/c/277224/20:28
hemnaI just posted a review on it.20:28
smcginnisSounds like a nova problem.20:28
smcginnisjk20:28
hemnahehe20:28
hemnae0ne, http://paste.openstack.org/show/486583/20:28
openstackgerritVincent Hou proposed openstack/cinder: Migration: take the local_path for the source volume  https://review.openstack.org/25022020:28
hemnae0ne, that's it 'working'20:28
hemnaat least from my command line script viewpoint.20:29
e0ne:)20:29
hemnaI haven't had time to look into the n-cpu lock up yet.20:29
diablo_rojopatrickeast: Hello today :)20:29
openstackgerritSheel Rana proposed openstack/cinder: Api-microversions patch for bug #1535749  https://review.openstack.org/27860520:29
openstackbug 1535749 in python-cinderclient "cinder client does not have handling for "--bootable" parameter for cinder list command" [Medium,Fix released] https://launchpad.net/bugs/1535749 - Assigned to Sheel Rana (ranasheel2000)20:29
vincent_houeharney: Eric, could you take a look at https://review.openstack.org/#/c/250220/ to see if it is better?20:30
*** hemna is now known as hemnafk20:30
e0nehemna: does it need any config changes for nova?20:30
*** e0ne has quit IRC20:32
*** dims has quit IRC20:34
sheeldiablo_rojo:20:34
sheeldiablo_rojo: hi20:34
*** vincent_hou has quit IRC20:34
*** dims has joined #openstack-cinder20:35
*** sid14726 has quit IRC20:37
openstackgerritHelen Walsh proposed openstack/cinder: EMC VMAX - Changing PercentSynced to CopyState in isSynched  https://review.openstack.org/24699220:38
openstackgerritMatan Sabag proposed openstack/cinder: Manage/unmanage volume in ScaleIO driver  https://review.openstack.org/26834420:38
*** chhavi has quit IRC20:40
*** IlyaG has quit IRC20:41
openstackgerritSheel Rana proposed openstack/cinder: Api-microversions patch for bug #1535749  https://review.openstack.org/27860520:41
openstackbug 1535749 in python-cinderclient "cinder client does not have handling for "--bootable" parameter for cinder list command" [Medium,Fix released] https://launchpad.net/bugs/1535749 - Assigned to Sheel Rana (ranasheel2000)20:41
openstackgerritMatan Sabag proposed openstack/cinder: Manage/unmanage volume in ScaleIO driver  https://review.openstack.org/26834420:42
*** chris_morrell has joined #openstack-cinder20:43
diablo_rojosheel: Hello20:43
sheeldiablo_rojo: could not find exactly where to raise bp in cross projects20:44
sheelI was looking in at https://launchpad.net/openstack/20:44
sheeldiablo_rojo: but it again is specific to project20:44
sheel:(20:44
diablo_rojosheel: You want to create a spec, not a bp20:44
*** alexpilotti_ has joined #openstack-cinder20:44
diablo_rojosheel: There is a cinder-specs repo and an openstack-specs repo20:45
sheelyup, will create spec20:45
*** jgriffith is now known as jgriffith_away20:45
smcginnissheel: https://github.com/openstack/openstack-specs20:46
diablo_rojosmcginnis: Ha ha was just about to link that. Thanks :)20:46
sheelsmcginnis: diablo_rojo thanks20:46
*** knikolla has quit IRC20:50
*** xyang has quit IRC20:50
*** kfarr has quit IRC20:51
sheelsmcginnis: As we are going to deprecate cli part...I think I should leave implementing https://blueprints.launchpad.net/python-cinderclient/+spec/retries-for-overlimit-response20:51
sheelsmcginnis: may be by  marking it obsolete!!20:52
*** e0ne has joined #openstack-cinder20:53
smcginnissheel: We're not going to deprecate our CLI yet.20:54
smcginnissheel: There's a lot of work to do with openstack client first.20:55
smcginnissheel: Eventually we may want to, but it will be a while before we get to that point.20:55
smcginnissheel: So I think your change should still go through.20:55
smcginnissheel: Thanks for checking!20:55
sheelsmcginnis: ok20:55
sheelso, please approve as well..20:55
smcginnissheel: Hah, I'll take a look in a bit.20:55
sheelsmcginnis: I will take it ahead in next week for sure..20:55
sheelsmcginnis: thanks...20:56
openstackgerritEdward Hope-Morley proposed openstack/cinder: Conditionally restore display_name  https://review.openstack.org/27862920:57
openstackgerritSheel Rana proposed openstack/cinder: Added Bootable filter for cinder list command  https://review.openstack.org/27860520:58
*** apoorvad has quit IRC20:59
*** merooney has joined #openstack-cinder20:59
*** haomaiwang has quit IRC21:01
*** haomaiwang has joined #openstack-cinder21:01
scottdasmcginnis: hemnafk Forgot, monday is a holiday. I'll schedule demo for Tues21:03
*** hemnafk is now known as hemna21:03
hemnaooh yah I forgot that too21:03
smcginnisscottda: What are you, a government worker? :)21:03
*** haomaiw__ has joined #openstack-cinder21:04
scottda:)21:04
*** sinese_ has quit IRC21:04
smcginnisscottda: Does HPE have off Monday?21:04
hemnayah I think so21:05
scottdanot sure about that. But kids have it off, so trip to Denver, etc.21:05
smcginnisAh, nice.21:05
scottdaWe're hiring.21:05
* smcginnis brushes up resume21:06
smcginnisHah!21:06
hemna:)21:06
*** haomaiwang has quit IRC21:06
scottdahemna: How's the champagne and caviare in Roseville today? I think yesterday's was much better.21:07
smcginnisLol21:07
hemnait was a bit different, they only had barrel aged stout and other beers today.   I think we ran out of the caviar from our free lunches every day this week.21:08
*** e0ne has quit IRC21:09
smcginnisI get free coffee. :]21:09
diablo_rojoscottda: Free cadillacs? ;)21:10
*** porrua has quit IRC21:10
scottdaI ain't going there...21:10
* smcginnis shakes head21:10
diablo_rojoscottda: But I thought you liked cadaillacs and ham sandwhiches?21:11
diablo_rojoscottda: Did you get a text last weekend from a number you didn't recognize?21:12
diablo_rojojungleboyj: ^^21:13
scottdayup, found out it was jay.21:13
diablo_rojoscottda: Ha ha good.21:13
*** merooney has quit IRC21:14
jungleboyjscottda: How was that ham sandwich the other day?21:16
jungleboyjsmcginnis: So when is the demo now?21:17
scottdaI'm going to schedule the demo for Tues 16:00 UTC/9:00 MST . Does that work for you jungleboyj ?21:18
smcginnisscottda: Works for me.21:18
jungleboyjscottda: Yep.  Cool.21:18
jungleboyjPutting it on the calendar.21:18
*** alexpilotti_ has quit IRC21:19
scottdahemna: Is there any magic to streaming a hangout to youtube cinder channel?21:21
hemnascottda, there are a few steps21:21
hemnascottda, https://support.google.com/plus/answer/2553119?hl=en21:22
*** akerr has quit IRC21:23
smcginnishemna: Do you need to do the streaming, or is scottda able to stream to the cinder channel?21:24
*** apoorvad has joined #openstack-cinder21:24
hemnawhoever owns the youtube channel can stream to it21:25
hemnawhen I'm on hangouts, I just switch my account over to the openstack-cinder user and then start a hangout on air21:26
smcginnisscottda: So I probably need to add your account to that group.21:26
jgriffith_awayeharney: did you have new/outstanding patches on that DB unicode issue, or were you just pointing out that you started the work on volumes and the *rest* needs done?21:26
*** alonma has joined #openstack-cinder21:26
scottdasmcginnis: is it as simple as adding Scott.dangelo@gmail ?21:27
smcginnisscottda: I think so. I'll let you know if I need anything more than that.21:27
smcginnisscottda: Did you get an email?21:29
*** dustins_ has joined #openstack-cinder21:29
eharneyjgriffith_away: the latter21:29
*** jgriffith_away is now known as jgriffith21:29
jgriffithstoopid auto-away21:29
*** leeantho has joined #openstack-cinder21:29
jgriffithauto-im-back needs some help21:30
eharney:)21:30
scottdasmcginnis: No email yet21:30
jgriffitheharney: ok, thanks... just reading what I missed in the scroll back from this AM's meeting21:30
*** alonma has quit IRC21:30
smcginnisscottda: OK, I think it should work though. You're added.21:31
smcginnisjgriffith: A lot to read there.21:31
jgriffithsmcginnis: yar, busy meeting21:31
jgriffithsmcginnis: I caught the second half at least21:31
smcginnisNo doubt21:31
smcginnisjgriffith: Feeling any better today?21:31
*** dustins has quit IRC21:32
*** dustins_ is now known as dustins21:33
*** leeantho has quit IRC21:33
*** leeantho has joined #openstack-cinder21:34
*** diablo_rojo has quit IRC21:34
*** shout28 has joined #openstack-cinder21:34
openstackgerritRyan McNair proposed openstack/cinder: Adds support for min/max volume size on vol_type  https://review.openstack.org/27864821:35
*** al_indigo has joined #openstack-cinder21:39
*** salv-orl_ has joined #openstack-cinder21:39
*** dave-mcc_ has joined #openstack-cinder21:39
*** alonma has joined #openstack-cinder21:40
*** rwall_ has joined #openstack-cinder21:40
*** tpatzig_ has joined #openstack-cinder21:40
*** cbader_2 has joined #openstack-cinder21:40
*** erlon_ has joined #openstack-cinder21:41
*** jordanP has joined #openstack-cinder21:41
*** sheel_ has joined #openstack-cinder21:41
*** mragupat_ has joined #openstack-cinder21:41
*** dramakri1 has joined #openstack-cinder21:42
*** dramakri1 has quit IRC21:42
*** dramakri1 has joined #openstack-cinder21:43
*** smcginni1 has joined #openstack-cinder21:44
*** alonma has quit IRC21:44
openstackgerritSarvesh Ranjan proposed openstack/cinder: Confusing typos fixed.  https://review.openstack.org/27864921:44
*** wolsen has joined #openstack-cinder21:44
*** bill_az_ has joined #openstack-cinder21:44
*** mragupat has quit IRC21:45
*** cknight has quit IRC21:45
*** andymaier has joined #openstack-cinder21:45
*** HenryG_ has joined #openstack-cinder21:47
*** raildo is now known as raildo-afk21:47
*** cbader_2 has quit IRC21:48
*** cbader has joined #openstack-cinder21:48
*** leeantho has quit IRC21:48
*** dave-mccowan has quit IRC21:48
*** dramakri has quit IRC21:48
*** cbader_ has quit IRC21:48
*** JoseMello has quit IRC21:48
*** salv-orlando has quit IRC21:48
*** bill_az has quit IRC21:48
*** sheel has quit IRC21:48
*** erlon has quit IRC21:48
*** jgriffith has quit IRC21:48
*** tpatzig has quit IRC21:48
*** wolsen_ has quit IRC21:48
*** smcginnis has quit IRC21:48
*** rwall has quit IRC21:48
*** zhenguo_ has quit IRC21:48
*** al_indig_ has quit IRC21:48
*** cfouts has quit IRC21:48
*** HenryG has quit IRC21:48
*** dhellmann has quit IRC21:48
*** xyang has joined #openstack-cinder21:48
*** rwall_ is now known as rwall21:48
*** HenryG_ is now known as HenryG21:48
*** sheel_ is now known as sheel21:48
*** gnarld_ has joined #openstack-cinder21:48
*** alonma has joined #openstack-cinder21:49
*** dhellmann has joined #openstack-cinder21:49
*** sgotliv has joined #openstack-cinder21:49
*** erlon_ is now known as erlon21:49
*** jgriffith has joined #openstack-cinder21:51
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/27722821:52
*** HenryG has quit IRC21:52
*** smcginni1 is now known as smcginnis21:52
*** thangp has left #openstack-cinder21:53
*** alonma has quit IRC21:54
openstackgerritAlex O'Rourke proposed openstack/cinder: Pass a host's volumes to the driver upon fail-over  https://review.openstack.org/27868721:54
*** hemna has quit IRC21:55
*** leeantho has joined #openstack-cinder21:55
jgriffithsmcginnis: little bit, thanks!21:55
*** HenryG has joined #openstack-cinder21:56
openstackgerritOpenStack Proposal Bot proposed openstack/os-brick: Updated from global requirements  https://review.openstack.org/27870221:57
*** lpetrut has quit IRC21:58
*** vgridnev has quit IRC21:58
openstackgerritOpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements  https://review.openstack.org/26616321:58
*** JoseMello has joined #openstack-cinder21:58
*** alonma has joined #openstack-cinder21:58
*** amoturi has left #openstack-cinder21:59
*** zhenguo_ has joined #openstack-cinder22:00
*** haomaiw__ has quit IRC22:01
*** haomaiwang has joined #openstack-cinder22:01
*** alonma has quit IRC22:03
openstackgerritAlex O'Rourke proposed openstack/cinder: Pass a host's volumes to the driver upon fail-over  https://review.openstack.org/27868722:03
*** jgregor has quit IRC22:03
*** baumann has left #openstack-cinder22:04
*** mylu has quit IRC22:06
*** cknight has joined #openstack-cinder22:06
*** rcernin has quit IRC22:07
*** alonma has joined #openstack-cinder22:08
*** dustins has quit IRC22:10
*** gnarld_ is now known as cfouts22:11
openstackgerritMerged openstack/cinder: Wrap the method to calculate virtual free capacity  https://review.openstack.org/25762022:12
*** alonma has quit IRC22:13
thingeexyang: http://lists.openstack.org/pipermail/openstack/2016-February/015151.html22:14
thingeexyang: I'm confused22:14
xyang1thingee: let me take a look22:14
xyang1thingee: what is your question22:14
openstackgerritSean McGinnis proposed openstack/cinder: WIP: Add in-tree tempest tests  https://review.openstack.org/27447122:15
thingeexyang1: I couldn't really parse the question in this22:16
thingee:)22:16
xyang1thingee: the email is not clear22:16
*** lprice1 has joined #openstack-cinder22:16
xyang1thingee: let me take a good look and try to respond22:16
*** julim has quit IRC22:16
thingeexyang1: also saw the thread about xtremeio versus xio.py :P22:17
*** boris-42_ has joined #openstack-cinder22:17
*** alonma has joined #openstack-cinder22:17
*** julim has joined #openstack-cinder22:17
*** dramakri has joined #openstack-cinder22:17
*** julim has quit IRC22:17
xyang1thingee: :)22:17
xyang1thingee: I think I responded, is there a new one22:18
xyang1thingee: we were calling xio at first but then was told to spell it out22:18
*** xyang has quit IRC22:19
thingeexyang1: no new message to that one, just found it funny because I've had the same confusion sometimes.22:19
xyang1thingee: I still get them mixed sometimes22:20
xyang1thingee: have to look at it twice22:20
*** dustins has joined #openstack-cinder22:20
xyang1thingee: our CI is EMC xio ci.  I named it before I learned we cannot use xio any more22:21
*** yuriy_n17_ has joined #openstack-cinder22:21
thingeeha22:21
*** alonma has quit IRC22:22
*** chlong_ has joined #openstack-cinder22:22
*** dramakri1 has quit IRC22:22
*** boris-42 has quit IRC22:22
*** lprice has quit IRC22:22
*** dhellmann has quit IRC22:22
*** yuriy_n17 has quit IRC22:22
*** ndipanov has quit IRC22:22
*** chlong has quit IRC22:22
*** haomaiwang has quit IRC22:22
*** leeantho has quit IRC22:22
*** dhellmann has joined #openstack-cinder22:23
*** haomaiwang has joined #openstack-cinder22:23
*** leeantho has joined #openstack-cinder22:23
*** ndipanov has joined #openstack-cinder22:23
*** boris-42_ is now known as boris-4222:24
*** dramakri has quit IRC22:28
*** IlyaG has joined #openstack-cinder22:28
smcginnisThey used to be Xiotech, but now everyone wants to have a three letter name. :)22:28
*** cknight1 has joined #openstack-cinder22:30
xyang1smcginnis: is it?22:30
smcginnisEMC, XIO, HPE, IBM... :D22:30
xyang1smcginnis: I tried to ping rhedlind since yesterday but no response22:30
xyang1smcginnis: dell is 4 letter:)22:31
smcginnisHah, that will be the announcemnt after the acquisition, rename to DEL. :)22:31
xyang1smcginnis: we are in trouble:)22:32
smcginnishehe22:32
*** cknight has quit IRC22:32
*** jordanP has quit IRC22:33
openstackgerritHelen Walsh proposed openstack/cinder: EMC VMAX - Operations and timeout issues  https://review.openstack.org/27276022:34
*** jwang has quit IRC22:34
*** eharney has quit IRC22:35
*** jwang has joined #openstack-cinder22:35
*** salv-orlando has joined #openstack-cinder22:42
*** mylu has joined #openstack-cinder22:42
xyang1thingee: the last email was not clear at all.  I sent a question22:43
thingeexyang1: ha ok22:43
xyang1thingee: looks like there are a bunch more emails on this thread, I will go thru those22:43
*** salv-orl_ has quit IRC22:44
*** lprice1 has quit IRC22:45
*** jgriffith is now known as jgriffith_away22:46
*** mylu has quit IRC22:48
*** IlyaG has quit IRC22:50
*** mriedem has quit IRC22:52
*** crose has quit IRC22:53
*** IlyaG has joined #openstack-cinder22:54
*** merooney has joined #openstack-cinder22:56
*** mylu has joined #openstack-cinder22:57
*** xyang has joined #openstack-cinder22:57
*** dramakri has joined #openstack-cinder22:58
*** JoseMello has quit IRC22:58
*** dramakri has quit IRC22:59
*** haomaiwang has quit IRC23:01
*** IlyaG has quit IRC23:01
*** haomaiwa_ has joined #openstack-cinder23:01
*** jgriffith_away is now known as jgriffith23:02
*** arch-nemesis has quit IRC23:05
*** sheel has quit IRC23:07
*** rlrossit has quit IRC23:07
*** IlyaG has joined #openstack-cinder23:08
*** jungleboyj has quit IRC23:09
*** sid14726 has joined #openstack-cinder23:12
openstackgerritScott DAngelo proposed openstack/cinder: cinder-api-microversions code  https://review.openstack.org/22491023:12
*** xyang has quit IRC23:13
openstackgerritMerged openstack/cinder: Remove old client version checks from 3PAR driver  https://review.openstack.org/26834623:16
*** mylu has quit IRC23:16
*** erlon has quit IRC23:16
*** alonma has joined #openstack-cinder23:18
*** dustins has quit IRC23:18
*** gouthamr has quit IRC23:18
*** mylu has joined #openstack-cinder23:19
*** mylu has quit IRC23:21
smcginnisangela-s: ping23:21
*** mylu has joined #openstack-cinder23:21
*** alonma has quit IRC23:22
*** mylu has quit IRC23:23
*** mylu has joined #openstack-cinder23:24
*** alonma has joined #openstack-cinder23:27
*** hemna has joined #openstack-cinder23:29
*** alonma has quit IRC23:31
*** cknight1 has quit IRC23:32
*** edmondsw has joined #openstack-cinder23:34
*** mragupat_ has quit IRC23:35
*** andymaier has quit IRC23:38
angela-ssmcginnis: pong23:39
aorourkejgriffith, do you mind if I make a revision to your patch to include passing in the volumes? That way I can abandon the separate patch I have and not worry about constantly rebasing it?23:43
*** merooney has quit IRC23:44
Swansonaorourke, which patch is getting volumes sent in?23:45
hemnaSwanson, https://review.openstack.org/#/c/27868723:46
aorourkeSwanson, https://review.openstack.org/#/c/27868723:46
*** chlong_ has quit IRC23:46
*** mylu has quit IRC23:47
Swansonaorourke, hemna, oh! So they are going to be sent down!23:48
hemnaSwanson, :)  yah looking that way23:49
SwansonI'd just come to grips with the fact that they weren't.23:49
hemnaone of our arrays can't do failover at all w/o it.23:49
jgriffithaorourke: sure, or I can do it and add you as co-contributor23:49
jgriffithaorourke: I'll be making another pass at it here shortly23:49
jgriffithaorourke: but either way is fine by me23:50
Swansonhemna, it helps but it isn't required.  If I have the info I can break all the replications immediately.  Without I have to wait for the next cinder call involving each impacted volume.  That would likely be the next attach.23:51
aorourkejgriffith, I don't mind making the revision, as long as it doesn't complicate things23:53
jgriffithaorourke: nahh... what could go wrong ;)23:53
Swansonhow many madison and sacagawea $1 coins can I use to pay a pizza delivery guy before they punch me in the face?23:57
*** martyturner has quit IRC23:58
*** cknight has joined #openstack-cinder23:59

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