Tuesday, 2016-02-09

*** haomaiwang has quit IRC00:01
*** haomaiwang has joined #openstack-cinder00:01
*** IlyaG has quit IRC00:04
*** IlyaG has joined #openstack-cinder00:05
*** mylu has quit IRC00:09
openstackgerritScott DAngelo proposed openstack/cinder: Volume create fails with FakeISCSIDriver  https://review.openstack.org/27761900:11
*** IlyaG has quit IRC00:12
jgriffithpatrickeast: ping00:13
*** IlyaG has joined #openstack-cinder00:14
*** IlyaG has quit IRC00:14
patrickeastjgriffith: pong00:15
jgriffithpatrickeast: hey.. I just wanted to confirm input from others on cast vs call for the failover call00:15
jgriffithpatrickeast: I can argue either way in my mind00:16
jgriffithpatrickeast: it's a critical enough op that blocking may not be a bad idea IMHO... but on the other hand it could be a long process?00:16
*** angela-s has quit IRC00:16
patrickeastjgriffith: hmm it seems like blocking would make sense00:16
patrickeastjgriffith: but we don't really have like a return value or something00:17
jgriffithpatrickeast: well00:17
jgriffithpatrickeast: I just put a return of the "active_backend_id" in the api layer00:17
patrickeastjgriffith: oh00:17
patrickeastjgriffith: well then00:17
jgriffithpatrickeast: now whether I want to return that in the json call or not I dunno... vs just updating the db at that point00:17
*** mylu has joined #openstack-cinder00:18
patrickeastjgriffith: sry got side tracked looking at the code00:20
patrickeastjgriffith: so uh, i like the idea of returning the id from the api... makes automating this much nicer00:20
jgriffithno worries, I've been side-tracked for a week now :)00:20
patrickeastjgriffith: and i know for like the pure backend the failover is quick, just a handful of rest api calls00:21
jgriffithpatrickeast: yeah, the initial reason for me was that I didn't have a place to pop it in the service data :)00:21
patrickeastjgriffith: but i don't know enough about other backends to say for sure if they could handle doing it in a reasonable amount of time00:21
jgriffithpatrickeast: and I don't like messing with the view-builder :)00:21
patrickeastjgriffith: haha yea00:22
*** cknight has quit IRC00:23
jgriffithpatrickeast: I guess I'll move forward with it and see if somebody screams :)00:23
patrickeastjgriffith: sounds good to me00:23
patrickeastjgriffith: can always fix it with microversions on the api later :D00:24
jgriffithpatrickeast: LOL00:24
jgriffithpatrickeast: well, I was thinking worse case, we can break the call into two pieces at the manager->driver layer.  But sure... let's just say "micro-versions", that's the one size fits all answer these days :)00:24
patrickeastjgriffith: haha, exactly, a one-size-fits-all answer00:25
patrickeastjgriffith: but i guess my point was its easy enough to just do the split later, and if anyone complains that its a behavior change.. boom increment the api number and call it a day00:25
jgriffithpatrickeast: +100:26
*** dramakri has quit IRC00:27
jgriffithok, have a note in the code that hopefully vendors will read and provide input if needed00:29
jgriffiththanks patrickeast00:29
patrickeastnp00:29
*** bill_az has quit IRC00:35
*** tpsilva has quit IRC00:36
*** arch-nemesis has quit IRC00:39
*** jwang has quit IRC00:39
*** jwang_ has joined #openstack-cinder00:39
*** jwang_ has quit IRC00:43
*** jwang has joined #openstack-cinder00:49
*** itlinux has joined #openstack-cinder00:52
*** jwang_ has joined #openstack-cinder00:53
*** jwang has quit IRC00:53
*** haomaiwang has quit IRC01:01
*** haomaiwang has joined #openstack-cinder01:01
*** garthb has quit IRC01:03
openstackgerritVasanthi Thirumalai proposed openstack/cinder: Add an implementation for validate_connector api in VMEM cinder driver  https://review.openstack.org/27764401:15
*** funeutron has joined #openstack-cinder01:18
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] Trying py27/34 with oslo-master  https://review.openstack.org/27765301:27
*** mtanino has quit IRC01:34
*** cknight has joined #openstack-cinder01:40
*** cknight1 has joined #openstack-cinder01:42
*** cknight has quit IRC01:45
fernnest__smcginnis, is it reasonable to break a large commit into smaller filesets that don't get called (until a later commit)?01:54
*** apoorvad has quit IRC01:54
*** itzdilip has quit IRC01:55
*** itzdilip has joined #openstack-cinder01:56
*** ChubYann has quit IRC01:56
openstackgerritMerged openstack/cinder: IBM XIV/DS8K: Implements Replication V2  https://review.openstack.org/26949001:57
*** haomaiwang has quit IRC02:01
*** haomaiwang has joined #openstack-cinder02:01
openstackgerritVincent Hou proposed openstack/cinder: Storwize: Implement v2 replication  https://review.openstack.org/23738702:03
*** leeantho has quit IRC02:12
*** csky has quit IRC02:20
*** csky has joined #openstack-cinder02:21
*** mudassirlatif has quit IRC02:21
*** thurloat is now known as thurloat_isgone02:21
*** xyang1 has quit IRC02:26
*** erlon has quit IRC02:26
*** mylu has quit IRC02:32
*** mylu has joined #openstack-cinder02:35
*** edmondsw has quit IRC02:41
*** dan_nguyen has quit IRC02:55
*** haomaiwang has quit IRC03:01
*** haomaiwang has joined #openstack-cinder03:01
*** EinstCrazy has joined #openstack-cinder03:02
*** EinstCrazy has quit IRC03:07
*** bardia has quit IRC03:07
openstackgerritAngus Lees proposed openstack/os-brick: Trivial rootwrap -> privsep replacement  https://review.openstack.org/27722403:10
*** mylu has quit IRC03:14
*** dims has joined #openstack-cinder03:27
*** dims_ has quit IRC03:28
openstackgerritMerged openstack/cinder: Port objects unit tests to Python 3  https://review.openstack.org/27257603:29
*** dims has quit IRC03:30
*** mylu has joined #openstack-cinder03:33
*** links has joined #openstack-cinder03:41
*** dims has joined #openstack-cinder03:43
*** mylu has quit IRC03:45
*** mylu has joined #openstack-cinder03:46
*** dims has quit IRC03:47
*** mylu has quit IRC03:49
*** mylu has joined #openstack-cinder03:49
*** Poornima has joined #openstack-cinder03:50
openstackgerritMerged openstack/cinder: ScaleIO QoS Support  https://review.openstack.org/26071903:56
*** haomaiwang has quit IRC04:01
*** haomaiwa_ has joined #openstack-cinder04:01
*** apoorvad has joined #openstack-cinder04:13
apoorvadxyang1 EMC XIO CI is voting multiple times on https://review.openstack.org/#/c/261126/, is there any known issue?04:14
*** apoorvad has quit IRC04:19
*** mylu has quit IRC04:19
*** mylu has joined #openstack-cinder04:20
openstackgerritAngus Lees proposed openstack/os-brick: Trivial rootwrap -> privsep replacement  https://review.openstack.org/27722404:24
*** mylu has quit IRC04:38
*** shyama has joined #openstack-cinder04:40
*** salv-orl_ has joined #openstack-cinder04:41
*** mylu has joined #openstack-cinder04:42
*** salv-orlando has quit IRC04:43
*** shausy has joined #openstack-cinder04:52
*** mylu has quit IRC04:55
*** dan_nguyen has joined #openstack-cinder04:56
*** dulek has joined #openstack-cinder05:00
*** haomaiwa_ has quit IRC05:01
*** haomaiwang has joined #openstack-cinder05:01
*** cknight1 has quit IRC05:02
*** funeutron has quit IRC05:03
*** mylu has joined #openstack-cinder05:05
*** jgriffith is now known as jgriffith_away05:07
*** cfriesen has quit IRC05:11
*** itzdilip has quit IRC05:11
*** mylu has quit IRC05:13
*** nikeshm has quit IRC05:14
*** chhavi has joined #openstack-cinder05:17
*** mylu has joined #openstack-cinder05:20
*** shausy has quit IRC05:37
*** shausy has joined #openstack-cinder05:38
*** dan_nguyen has quit IRC05:40
*** mylu has quit IRC05:45
*** mylu has joined #openstack-cinder05:46
*** mylu has quit IRC05:49
*** mylu has joined #openstack-cinder05:53
*** mylu has quit IRC05:55
*** Poornima has quit IRC05:56
*** haomaiwang has quit IRC06:01
*** Poornima has joined #openstack-cinder06:01
*** haomaiwang has joined #openstack-cinder06:01
*** shausy has quit IRC06:07
*** shausy has joined #openstack-cinder06:07
*** Poornima has quit IRC06:08
*** savihou has joined #openstack-cinder06:11
*** Poornima has joined #openstack-cinder06:13
*** Poornima has quit IRC06:13
*** Poornima has joined #openstack-cinder06:14
*** sheel has joined #openstack-cinder06:16
*** avishay has joined #openstack-cinder06:31
*** avishay has quit IRC06:51
*** avishay has joined #openstack-cinder06:53
*** leeantho has joined #openstack-cinder06:55
*** haomaiwang has quit IRC07:01
*** haomaiwang has joined #openstack-cinder07:01
*** avishay has quit IRC07:02
*** nkrinner has joined #openstack-cinder07:03
*** lpetrut has joined #openstack-cinder07:03
*** anshul has joined #openstack-cinder07:06
*** isaacb has joined #openstack-cinder07:21
*** savihou has quit IRC07:28
*** sgotliv has joined #openstack-cinder07:30
*** lpetrut has quit IRC07:31
*** avishay has joined #openstack-cinder07:34
*** alyson_ has quit IRC07:37
*** alyson_ has joined #openstack-cinder07:37
*** belmoreira has joined #openstack-cinder07:41
*** haomaiwang has quit IRC08:01
*** haomaiwa_ has joined #openstack-cinder08:01
*** liverpooler has joined #openstack-cinder08:03
*** mvk has joined #openstack-cinder08:04
*** savihou has joined #openstack-cinder08:05
*** leeantho has quit IRC08:11
*** alonma has joined #openstack-cinder08:16
*** sinese_ has joined #openstack-cinder08:24
*** deepakcs has joined #openstack-cinder08:27
*** sgotliv has quit IRC08:34
*** e0ne has joined #openstack-cinder08:49
*** mvk has quit IRC08:49
*** ndipanov has joined #openstack-cinder08:55
*** bkumar has joined #openstack-cinder08:59
*** haomaiwa_ has quit IRC09:01
*** geguileo has joined #openstack-cinder09:01
*** haomaiwang has joined #openstack-cinder09:01
*** arecknag has joined #openstack-cinder09:03
*** shausy has quit IRC09:08
*** arecknag has quit IRC09:08
*** shausy has joined #openstack-cinder09:09
*** arecknag has joined #openstack-cinder09:09
*** sheel has quit IRC09:17
*** mvk has joined #openstack-cinder09:17
*** e0ne has quit IRC09:19
*** alonma has quit IRC09:25
*** alonma has joined #openstack-cinder09:25
*** arecknag has quit IRC09:26
*** arecknag has joined #openstack-cinder09:28
*** mvk has quit IRC09:29
*** alonma has quit IRC09:29
*** mvk has joined #openstack-cinder09:30
*** alonma has joined #openstack-cinder09:31
*** jwcroppe has joined #openstack-cinder09:33
*** shausy has quit IRC09:33
*** alonma has quit IRC09:35
*** jwcroppe has quit IRC09:36
*** jwcroppe has joined #openstack-cinder09:37
*** alonma has joined #openstack-cinder09:37
*** ociuhandu has quit IRC09:38
nttHi, I'd like to use nfs as a cinder backend but I have a question: Compute nodes will have a local mountpoint or traffic passes through the cinder-volume node?09:40
*** jwcroppe has quit IRC09:41
*** lpetrut has joined #openstack-cinder09:51
*** yhayashi has quit IRC09:59
*** haomaiwang has quit IRC10:01
*** haomaiwa_ has joined #openstack-cinder10:01
*** arecknag has quit IRC10:05
*** arecknag has joined #openstack-cinder10:06
*** e0ne has joined #openstack-cinder10:21
dulekntt: Trafic will not pass through c-vols10:22
nttdulek: thank you. So each compute node will have N mountpoints, where N = number of volume used by that compute node. Is this correct?10:24
*** Roamer`_ is now known as Roamer`10:28
dulekntt: I believe so, but it may be more complicated when taking multiattach into account.10:28
*** esp has joined #openstack-cinder10:30
*** esp has quit IRC10:37
*** salv-orlando has joined #openstack-cinder10:40
*** salv-orl_ has quit IRC10:43
*** dims has joined #openstack-cinder10:45
*** shyama has quit IRC10:47
*** alonma has quit IRC10:47
*** dims has quit IRC10:49
*** dims has joined #openstack-cinder10:50
*** deepakcs has quit IRC10:52
openstackgerritVictor Stinner proposed openstack/cinder: Port backup drivers to Python 3  https://review.openstack.org/27777810:53
*** alonma has joined #openstack-cinder10:54
*** tpsilva has joined #openstack-cinder10:55
*** erlon has joined #openstack-cinder10:56
*** alonma has quit IRC10:58
openstackgerritMichal Dulko proposed openstack/cinder: OVO request_spec in migrate/manage/retype calls  https://review.openstack.org/25245810:59
openstackgerritMichal Dulko proposed openstack/cinder: Switch request_spec in create volume calls to ovo  https://review.openstack.org/24660010:59
openstackgerritMichal Dulko proposed openstack/cinder: Switch request_spec_list in create CG to OVO  https://review.openstack.org/25303310:59
openstackgerritMichal Dulko proposed openstack/cinder: Convert filter_properties to OVO (create, retype)  https://review.openstack.org/25730510:59
*** alonma has joined #openstack-cinder11:00
*** haomaiwa_ has quit IRC11:01
*** haomaiwang has joined #openstack-cinder11:01
*** sgotliv has joined #openstack-cinder11:04
*** alonma has quit IRC11:05
*** alonma has joined #openstack-cinder11:06
*** mvk has quit IRC11:07
*** mvk has joined #openstack-cinder11:08
*** chhavi has quit IRC11:08
*** chhavi has joined #openstack-cinder11:08
*** alonma has quit IRC11:11
*** alonma has joined #openstack-cinder11:12
openstackgerritIvan Kolodyazhny proposed openstack/os-brick: Implement disconnect_volume for RemoteFsConnector  https://review.openstack.org/27778811:14
openstackgerritIvan Kolodyazhny proposed openstack/os-brick: Implement disconnect_volume for RemoteFsConnector (NFS only)  https://review.openstack.org/27778811:14
*** alonma has quit IRC11:17
*** alonma has joined #openstack-cinder11:17
*** bkumar has quit IRC11:32
*** dims_ has joined #openstack-cinder11:40
*** dims has quit IRC11:40
*** boris-42 has quit IRC11:43
*** arecknag has quit IRC11:44
*** arecknag has joined #openstack-cinder11:45
*** xek_ is now known as xek11:51
*** lprice has joined #openstack-cinder11:53
*** shyama has joined #openstack-cinder11:53
*** lprice1 has quit IRC11:55
openstackgerritwanghao proposed openstack/cinder: Clean up image tmp file if c-vol gets restarted  https://review.openstack.org/22508111:55
*** haomaiwang has quit IRC12:01
*** haomaiwang has joined #openstack-cinder12:01
*** dims_ has quit IRC12:11
*** Poornima has quit IRC12:15
*** sgotliv has quit IRC12:16
*** dims_ has joined #openstack-cinder12:19
*** thurloat_isgone is now known as thurloat12:22
*** avishay has quit IRC12:22
*** zhangjn has quit IRC12:24
*** zhangjn has joined #openstack-cinder12:27
*** mtanino has joined #openstack-cinder12:28
openstackgerritSzymon Borkowski proposed openstack/cinder: Fixed logging for oslo versioned objects  https://review.openstack.org/26628912:31
openstackgerritAdriano Freires Rosso proposed openstack/cinder: HNAS driver: Fix SSH and cluster_admin_ip0 bug  https://review.openstack.org/27749512:37
*** openstackgerrit_ has joined #openstack-cinder12:39
*** marcusvrn_ has joined #openstack-cinder12:44
*** alonma has quit IRC12:53
*** alonma has joined #openstack-cinder12:54
*** alonma has quit IRC12:58
*** alonma has joined #openstack-cinder13:00
*** porrua has joined #openstack-cinder13:00
*** haomaiwang has quit IRC13:01
*** haomaiwang has joined #openstack-cinder13:01
*** alonma has quit IRC13:03
*** alonma has joined #openstack-cinder13:04
*** ildikov has quit IRC13:06
*** avishay has joined #openstack-cinder13:13
*** pots has joined #openstack-cinder13:14
smcginnisfernnest__: Saw your question. Yes, that would be reasonable.13:15
*** links has quit IRC13:17
*** smoriya_ has quit IRC13:18
e0nesmcginnis: hi. could you please review https://review.openstack.org/#/c/267801/?13:19
smcginnise0ne: Sure, I'll take a look.13:20
e0nesmcginnis: thanks13:20
smcginnisAny time13:21
*** deepakcs has joined #openstack-cinder13:24
*** bill_az has joined #openstack-cinder13:25
swamireddyHi13:31
swamireddycinder quota-show and quota-usage  -  CLIs are not showing the backup related details?13:31
swamireddyis it expected?13:31
swamireddyI used cinder quota-show <tenant-id>13:32
*** avishay has quit IRC13:34
*** diablo_rojo has joined #openstack-cinder13:38
*** mriedem has joined #openstack-cinder13:41
mtaninoswamireddy: I think these commands show backup_gigabytes and backups like in this report. https://bugzilla.redhat.com/show_bug.cgi?id=128968113:41
openstackbugzilla.redhat.com bug 1289681 in openstack-cinder "VolumeLimitExceeded error even though quota is updated to larger value" [High,Closed: worksforme] - Assigned to eharney13:41
*** jordanP has joined #openstack-cinder13:41
openstackgerritVictor Stinner proposed openstack/cinder: Port API v1 to Python 3  https://review.openstack.org/27783513:42
*** alyson_ has quit IRC13:42
*** eharney has joined #openstack-cinder13:45
*** haomaiwang has quit IRC13:46
*** haomaiwa_ has joined #openstack-cinder13:47
swamireddymtanino: hm....I use cinder 1.1.1 - its not showing the backup details...(backup limits and usage)13:47
*** crose has joined #openstack-cinder13:48
mtaninoswamireddy: I tried Liberty base version and it13:48
*** akerr has joined #openstack-cinder13:49
mtaninosupports backup related details.13:49
*** alonma has quit IRC13:52
*** anshul has quit IRC13:53
*** avishay has joined #openstack-cinder13:54
*** cbader has joined #openstack-cinder13:55
*** edmondsw has joined #openstack-cinder13:56
*** openstackgerrit_ has quit IRC13:56
*** alonma has joined #openstack-cinder13:57
openstackgerritVictor Stinner proposed openstack/cinder: Port API v1 and v2 to Python 3  https://review.openstack.org/27783514:00
openstackgerritVictor Stinner proposed openstack/cinder: hacking: Fix false positive in C302 check  https://review.openstack.org/27784114:00
*** vgridnev has joined #openstack-cinder14:00
*** alonma has quit IRC14:00
*** haomaiwa_ has quit IRC14:01
*** alonma has joined #openstack-cinder14:01
*** haomaiwa_ has joined #openstack-cinder14:01
*** akerr has quit IRC14:01
*** xyang1 has joined #openstack-cinder14:02
*** ildikov has joined #openstack-cinder14:02
*** dims_ has quit IRC14:03
*** alonma has quit IRC14:05
*** akerr has joined #openstack-cinder14:05
*** cknight has joined #openstack-cinder14:05
*** jungleboyj has joined #openstack-cinder14:05
*** deepakcs has quit IRC14:07
*** mtanino has quit IRC14:07
*** haomaiwa_ has quit IRC14:08
*** haypo has joined #openstack-cinder14:09
*** alonma has joined #openstack-cinder14:09
*** dims has joined #openstack-cinder14:10
haypoeharney: hey! it's really cool to get a +2 in less than 6 months :-) i appreciate your reactivity ;)14:11
eharney:)14:11
smcginnishaypo: I thought we've been pretty good about getting those through faster?14:13
hayposmcginnis: it's a pleasure to contribute to cinder, you are usually quite fast compared to other openstack services14:14
smcginnishaypo: Haha, good to hear. :)14:14
jungleboyjThat is a good way to start Tuesday.  :-)14:14
smcginnisWell, for us. Not for you with the other services.14:14
smcginnis:)14:14
*** mylu has joined #openstack-cinder14:14
*** knikolla has joined #openstack-cinder14:15
jungleboyjOther services?  ;-)14:15
smcginnisjungleboyj: ;)14:15
haypoeharney, smcginnis : see my grumpy email sent yesterday about a different service, almost no review after 4 months: http://lists.openstack.org/pipermail/openstack-dev/2016-February/086060.html14:16
smcginnishaypo: Ooo, sadness.14:16
*** links has joined #openstack-cinder14:16
smcginnishaypo: I think you are justified though.14:16
smcginnishaypo: You can't keep rebasing if no one's going to ever look at it.14:17
smcginnisIt's unfortunate though, for sure.14:17
*** rlrossit has joined #openstack-cinder14:17
*** alonma has quit IRC14:17
hayposmcginnis: i'm annoyed because we agreed on a plan, but then, nothing14:17
smcginnishaypo: Yeah, just reading through that all the way. Sounded like it was on track. :/14:18
jungleboyjsmcginnis: Have you seen these start popping up?  https://bugs.launchpad.net/cinder/+bug/154340714:18
openstackLaunchpad bug 1543407 in Cinder " ScaleIO QoS Support" [Undecided,New]14:18
smcginnishaypo: Well, I can say for me at least, I really appreciate the work you've done in Cinder for py3.14:18
haypowell, i also understand that reviewing my boring patches is less exciting than working on new stuff14:18
smcginnishaypo: It's a huge undertaking but very necessary.14:19
haypobut i'm no more interested to ping dev forever14:19
hayposmcginnis: we are closer everyday to get all unit tests running on py3, for cinder14:19
*** alonma has joined #openstack-cinder14:19
smcginnishaypo: +1 :)14:19
smcginnisjungleboyj: No, what's the deal there?14:19
*** eharney has quit IRC14:19
smcginnisjungleboyj: Oh, those. Yeah.14:20
jungleboyjsmcginnis: I don't know and the message that comes with it doesn't totally make sense to me.14:20
smcginnisjungleboyj: I have that on the agenda for tomorrow.14:20
jungleboyjsmcginnis: Ok, cool.14:20
smcginnisjungleboyj: Basically there's been a change in how DocImpact works.14:20
*** jgriffith_away is now known as jgriffith14:21
smcginnisIt's now on us (who actually know what changed) to provide the documentation update details.14:21
*** esp has joined #openstack-cinder14:21
smcginnisjungleboyj: Before DocImpact would open bugs against the doc team, then they would have to go figure out what they were supposed to do.14:21
jungleboyjAhhh, so do we need to write the documentation, or just put the information in the bug?14:22
smcginnisjungleboyj: From what I understand, we just need to put the information in the bug, then pass it on to them.14:22
smcginnisjungleboyj: Although I'm sure that team would be open to getting patches directly to update the docs. ;)14:23
openstackgerritMerged openstack/cinder: Support cinder_img_volume_type in image metadata  https://review.openstack.org/25864914:24
hayposmcginnis: what's the plan for XML? deprecated ok, but do you plan to drop code to handle XML?14:24
jungleboyjsmcginnis: Ok.  Thanks for the info.14:24
smcginnishaypo: Yes. I wouldn't bother with updating that.14:24
e0nehaypo: we're going to drop XML API in Newton14:25
*** alonma has quit IRC14:25
hayposmcginnis: when i port code, i don't really care if it's deprecated or not. i just port it :) it was a general question14:25
haypoe0ne: ok, so next cycle14:25
smcginnis:)14:25
*** jgregor has joined #openstack-cinder14:25
*** alonma has joined #openstack-cinder14:26
*** esp has quit IRC14:29
*** akshai has joined #openstack-cinder14:30
*** alonma has quit IRC14:30
flip214I'd like to ask for another small review: https://review.openstack.org/#/c/277314/214:33
*** mylu has quit IRC14:34
*** dave-mccowan has joined #openstack-cinder14:34
*** jgriffith is now known as jgriffith_away14:35
*** mylu has joined #openstack-cinder14:35
*** akerr has quit IRC14:36
*** gouthamr has joined #openstack-cinder14:37
*** akerr has joined #openstack-cinder14:37
*** jordanP has quit IRC14:39
*** tongli has joined #openstack-cinder14:40
*** jgriffith_away is now known as jgriffith14:43
*** ociuhandu has joined #openstack-cinder14:43
*** alonma has joined #openstack-cinder14:43
*** ociuhandu has quit IRC14:46
*** dustins has joined #openstack-cinder14:47
*** alonma has quit IRC14:48
*** julim_ has quit IRC14:49
openstackgerritVictor Stinner proposed openstack/cinder: Port volume_type_encryption to Python 3  https://review.openstack.org/27786114:49
*** cfriesen has joined #openstack-cinder14:49
*** alonma has joined #openstack-cinder14:49
*** amit213 has quit IRC14:51
*** tpatzig has quit IRC14:53
*** mtanino has joined #openstack-cinder14:54
*** alonma has quit IRC14:54
*** laughterwym has joined #openstack-cinder14:54
*** links has quit IRC14:54
*** mkoderer has quit IRC14:55
*** alonma has joined #openstack-cinder14:55
*** mkoderer has joined #openstack-cinder14:56
*** tpatzig has joined #openstack-cinder14:56
*** mylu has quit IRC14:56
*** mylu has joined #openstack-cinder14:57
*** ociuhandu has joined #openstack-cinder14:58
diablo_rojosmcginnis: hemna e0ne patrickeast DuncanT jungleboyj  jgriffith xyang1 geguileo In the cross project meeting today we will be discussing these two reviews, if there are any opinions you would like me to bring forward please let me know as soon as you can. https://review.openstack.org/#/c/242852/ https://review.openstack.org/#/c/245629/5 Thanks!14:59
jgriffithdiablo_rojo: they both look good to me14:59
*** alonma has quit IRC14:59
jgriffithdiablo_rojo: I'll do a proper review and add comments before you're meeting15:00
diablo_rojojgriffith:  Cool. Thanks John :)15:00
jgriffithdiablo_rojo: thanks for pointing them out15:00
diablo_rojojgriffith:  No problem.15:00
*** laughterwym has quit IRC15:00
smcginnisdiablo_rojo: Yeah, looks fine to me. Thanks!15:01
diablo_rojosmcginnis: Excellent :)15:01
*** laughterwym has joined #openstack-cinder15:04
*** crose has quit IRC15:05
*** lprice1 has joined #openstack-cinder15:05
e0nediablo_rojo: looks like a right direction. reading15:07
e0nejgriffith: btw, please take a look on  https://review.openstack.org/#/c/267801/ - functional tests for cinder15:08
*** sheel has joined #openstack-cinder15:08
*** lprice has quit IRC15:09
*** alonma has joined #openstack-cinder15:09
*** baumann has joined #openstack-cinder15:09
diablo_rojoe0ne: Thanks for checking it out :)15:10
*** edtubill has joined #openstack-cinder15:11
jgriffithdiablo_rojo: so I did have some objections on this one: https://review.openstack.org/#/c/245629/5  Noted in comments15:11
jgriffithdiablo_rojo: let me know if there's any questions that come up I can help with.15:11
jgriffithdiablo_rojo: I'm certainly in the minority on that one :)15:12
jgriffithbut that's where I like to be usually15:12
diablo_rojojgriffith: I definitely will and I will call you into the meeting if they have any questions about what you've said .15:12
e0nediablo_rojo: should it https://review.openstack.org/#/c/242852/ be re-writed to be a cross-project?15:12
e0nefor now, it's about keystone15:12
diablo_rojojgriffith: It is more fun to play devils advocate :)15:12
jgriffithdiablo_rojo: :)15:12
diablo_rojojgriffith: Not even trying to be clever there with my nick :)15:13
jgriffithdiablo_rojo: uh-huh :)15:13
jgriffithdiablo_rojo: I thought the same thing when you typed that15:13
smcginnisHah15:14
*** alonma has quit IRC15:14
diablo_rojoe0ne: It was originally for keystone, but I think they are just looking at it like a cross project spec15:14
diablo_rojojgriffith: You can be my advocate any time you want though :)15:14
*** cdelatte has joined #openstack-cinder15:14
jgriffithhaha... "devils advocate"  I like it15:14
jgriffithdiablo_rojo: I see what you did there15:15
*** alonma has joined #openstack-cinder15:15
diablo_rojojgriffith: :D15:15
diablo_rojoe0ne: It looks like the spec is in the correct repo, so I guess we are just assuming it would be good for all projects and Keystone is just one application for the spec.15:17
e0nediablo_rojo: as I understood, the same should be implemented in each project, right?15:18
diablo_rojoe0ne: That is correct.15:19
*** ildikov has quit IRC15:19
e0nediablo_rojo: good. I'll -1 on it. it woould be good to have more general spec to not confuse anybody15:19
*** ildikov has joined #openstack-cinder15:19
*** alonma has quit IRC15:19
smcginnisDo users have problems with spec files now?15:20
smcginnisI never heard that feedback?15:20
jgriffithsmcginnis: users and spec files?15:20
diablo_rojoe0ne: I was confused at first too till I reread the commit message.15:20
e0nesmcginnis: what problems you you mean?15:20
jgriffithsmcginnis: or do you mean policy files?15:20
smcginnisI'm not against formalizing some current conventions, but just surprised that it seems to be such a big issue for others.15:21
smcginnisjgriffith: policy files.15:21
jgriffithsmcginnis: ahhh15:21
jgriffithsmcginnis: yeah, so Brick exposed a big problem15:21
smcginnisOops, I see my typo.15:21
e0nediablo_rojo: yeah, commit message is good15:21
jgriffithsmcginnis: the problem is when you have things like libs that are written such that they need policy files... making them all "line up"15:21
jgriffithsmcginnis: some like decoupling.. some don' t :)15:22
smcginnisjgriffith: Yeah, true. Same issue with rootwrap.15:22
jgriffithsmcginnis: exactly15:22
*** knomura has joined #openstack-cinder15:22
jgriffithsmcginnis: my fear is what's proposed there has the potential to be WAAAAY more complex than I would prefer15:22
smcginnisjgriffith: Kind of what I'm thinking.15:23
smcginnisThere's definitely some caveats as it is today.15:23
jgriffithsmcginnis: I'm also somewhat unclear on how that file *looks* when it's done... like "10K lines of json" Puke15:23
smcginnisBut when it comes down to it, it's fairly simple really.15:23
smcginnisHah, yeah, that will be a good end user experience.15:23
jgriffith:)15:23
jgriffithgetting breakfast, back in a bit15:24
*** alonma has joined #openstack-cinder15:25
*** dustins has quit IRC15:26
smcginnisGaahh! Love how gerrit randomly jumps around as you scroll. :|15:26
diablo_rojosmcginnis: Is it time for gerrit complaints already today?15:27
diablo_rojoWhere is hemna? I bet he has a thing or two to say here :)15:27
smcginnisdiablo_rojo: Kind of early I guess. hemna's not even here to join in.15:27
*** timcl has joined #openstack-cinder15:27
diablo_rojosmcginnis: Hold that thought for an hour or two, make a list even, then we can go for the gold.15:28
smcginnis;)15:28
diablo_rojosmcginnis: We really should have spent all friday of the meetup filing gerrit bugs and fixing them15:28
*** dustins has joined #openstack-cinder15:28
smcginnisHah, that would have kept us busy.15:29
diablo_rojosmcginnis: No doubt.15:29
*** alonma has quit IRC15:30
*** dustins has quit IRC15:33
*** dustins has joined #openstack-cinder15:36
*** laughterwym has quit IRC15:36
*** mragupat has joined #openstack-cinder15:38
*** krtaylor has quit IRC15:42
jungleboyjdiablo_rojo: Ok, I think e0ne is right that the first one needs to be generalized.15:44
jungleboyjdiablo_rojo: Looks like jgriffith has made some good comments on the second one that should be addressed.15:44
jungleboyjdiablo_rojo: Thanks for bringing those up.  Good to know that those items are being discussed.15:45
*** julim has joined #openstack-cinder15:45
diablo_rojojungleboyj: No problem. I will bring all of that up in the meeting this afternoon.15:45
openstackgerritShilpa Jagannath proposed openstack/python-cinderclient: Allow  "cinder backup-delete" to delete multiple backups in one request  https://review.openstack.org/27789515:46
*** alonma has joined #openstack-cinder15:47
jungleboyjdiablo_rojo: Good deal.  Thanks!15:47
*** marcusvrn_ has quit IRC15:47
sheeljungleboyj: smcginnis: I too want to mention 2 points where we should think about15:49
sheel1. limiting size of patch to around 1000 LOC : may be all guys find it difficult to review large patchsets..so it would be good if we can force this policy..15:50
sheel2. review requests : Is it possible to find some other way of requesting for reviews?15:51
smcginnissheel: That's a hard policy to enforce. Sometimes it is hard to break something down into smaller chunks.15:51
smcginnissheel: It's definitely the recommended way, and folks will have a lot better chance of getting timely reviews if their patches aren't so large.15:52
*** alonma has quit IRC15:52
smcginnissheel: But I don't think we can have a hard fast rule about it.15:52
sheelsmcginnis: yes T&C applies...  some exception would be there...15:52
smcginnissheel: As for review requests, I think that's something folks have struggled with a long time.15:52
*** liverpooler has quit IRC15:53
smcginnisNot sure if anyone has come up with a good solution, other than to wait for folks to get the time to review15:53
*** alonma has joined #openstack-cinder15:53
akerrbribes help15:53
smcginnisI'm not against getting requests for reviews (reasonably) but I know others definitely do not like it.15:53
smcginnisHah!15:53
sheelsmcginnis: yes, time factor matters most15:53
smcginnissheel: Yeah, it is unfortunate when I see fairly trivial patches out there for weeks.15:54
*** isaacb has quit IRC15:54
smcginnisSo I can state here what I try to tell everyone when asked - if anyone is looking at getting involved but not sure where to start - code reviews are always welcome.15:55
sheelsmcginnis: agreed with every word you said15:55
sheelsmcginnis: actually why I raised this point because sometimes its demotivating for new guys to get their things go ahead...15:55
smcginnissheel: Yeah, I know. :{15:56
sheelsmcginnis: so, I was just looking emphatically and was trying if anything is possible to mobilize same...15:56
smcginnissheel: If you come up with a solution, let me know. ;)15:56
sheelsmcginnis: will see if anything is possible for sure and will discuss with you guys15:57
*** alonma has quit IRC15:57
sheelsmcginnis: thanks for discussion ... :)15:58
smcginnissheel: Thanks for spending time on it. Always worth considering.15:58
sheel:)15:58
*** jdurgin1 has joined #openstack-cinder15:58
hemnamornin16:01
*** esker has joined #openstack-cinder16:01
smcginnishemna: howdy16:02
diablo_rojohemna: Morning :)16:02
*** eharney has joined #openstack-cinder16:02
*** garthb has joined #openstack-cinder16:02
*** arch-nemesis has joined #openstack-cinder16:02
*** sinese_ has quit IRC16:04
*** lpetrut has quit IRC16:04
Swansonhemna: happy National Home Warranty Day Eve.16:04
*** alonma has joined #openstack-cinder16:04
smcginnisWuh16:04
hemna?!16:04
smcginnisSwanson: You suck for making me waste time on googling this: http://www.2-10.com/blog/announcing-the-first-annual-national-home-warranty-day/16:05
SwansonTomorrow is National Home Warranty Day so today is National Home Warranty Day Eve.  It's a greeting.16:05
smcginnis:)16:05
smcginnishemna: I'll walk over and smack him for you.16:06
*** JoseMello has joined #openstack-cinder16:06
SwansonToday is something like national bagel day.  Of which I'm seeing none around.16:06
SwansonSo obviously we don't celebrate that here.16:06
smcginnisSwanson: You should fix that.16:07
Swansonsmcginnis, I'll just submit a national no bagels day.  Doesn't seem too hard to get one in the old registry.16:07
*** mvk has quit IRC16:07
openstackgerritMerged openstack/cinder: Volume create fails with FakeISCSIDriver  https://review.openstack.org/27761916:07
hemna:)16:09
*** boris-42 has joined #openstack-cinder16:11
jgriffithSwanson: problem solved on my end16:14
*** krtaylor has joined #openstack-cinder16:14
jgriffithSwanson: ham, egg and cheese on an everything!!  Mmmmmmmm16:14
*** laughterwym has joined #openstack-cinder16:14
*** jungleboyj has quit IRC16:14
smcginnis:)16:16
Swansonjgriffith, I hope you enjoyed that.  Off to the vending machine, I guess.16:18
*** crose has joined #openstack-cinder16:18
jgriffithSwanson: yuck :(16:19
openstackgerritChaozhe Chen(ccz) proposed openstack/python-cinderclient: Fix some flake8 violations  https://review.openstack.org/27792516:19
*** isaacb has joined #openstack-cinder16:20
*** belmoreira has quit IRC16:21
*** alonma has quit IRC16:23
*** ntpttr has joined #openstack-cinder16:25
*** knikolla has quit IRC16:27
*** knikolla has joined #openstack-cinder16:27
*** vgridnev has quit IRC16:27
*** jungleboyj has joined #openstack-cinder16:29
*** alonma has joined #openstack-cinder16:30
*** avihou has joined #openstack-cinder16:32
*** savihou has quit IRC16:34
*** laughterwym has quit IRC16:36
*** knomura has quit IRC16:36
*** laughterwym has joined #openstack-cinder16:37
*** savihou has joined #openstack-cinder16:38
*** avihou has quit IRC16:40
*** salv-orl_ has joined #openstack-cinder16:41
*** laughterwym has quit IRC16:43
*** krtaylor has quit IRC16:44
*** salv-orlando has quit IRC16:44
scottda If my 'git review' of a patch has picked up a bunch of new commits from master, I can just squash them and submit, right? It's just rebasing against master?16:45
smcginnisscottda: Doesn't it just merge your code with master?16:45
smcginnisscottda: I thought squash was only needed if you locally had several commits that you wanted to combine into one.16:45
scottda16:46
scottdaYes, that's when I normally need to squash. But somehow I've picked up a few of the latest patches from master...16:46
smcginnisHmm, something doesn't seem right there.16:46
scottdaI agree.16:46
scottdaThat's why I'm not pushing the button...16:47
scottdahttps://www.irccloud.com/pastebin/3aKpwyRt/16:48
*** esker has quit IRC16:49
scottdasmcginnis: But those patches are on master, so it seems that if I squash them and submit, it will have no net effect16:50
*** knikolla has quit IRC16:50
smcginnisscottda: If you do git log does it look right?16:51
smcginnisscottda: I thought the squash would try to put all of the changes into one commit.16:51
smcginnisscottda: Did you have a rebase that didn't complete?16:51
scottdaI had a rebase, but I don't think it failed to complete16:52
scottdaYou are right, squashing is wrong, that will lose the history16:52
smcginnisAll: BTW, eventlet release got yanked, so CI's might start failing now trying to stack and not being able to find the package.16:52
smcginnisscottda: Maybe try git rebase --continue?16:52
smcginnisscottda: Just to see what happens.16:52
*** jungleboyj has quit IRC16:52
scottdaNo rebase in progress?16:53
scottdaThat's the message ^^^16:53
smcginnispatrickeast is a git pro. Maybe he knows?16:53
smcginnisscottda: Otherwise I've redownloaded my patches in the past with git review -d [number], then tried rebasing again.16:54
scottdaMaybe the answer is "yes", submit the above commits. Which should preserve them as separate16:54
smcginnisscottda: Maybe make sure your local master is up to date before trying to rebase on it?16:54
smcginnisI think git review does an implicit rebase on to remote master, so maybe that's why?16:55
* smcginnis is grasping at straws.16:55
*** knomura has joined #openstack-cinder16:55
scottdaThanks for the grasping :)16:55
*** krtaylor has joined #openstack-cinder16:55
smcginnisscottda: Or yeah, you could just answer Y and if it messes anything up just clean up after. :]16:56
scottdaWhat could go wrong?16:56
*** apoorvad has joined #openstack-cinder16:56
smcginnisHehe, "what does this button do?" - BOOM16:56
scottda"failed to push"16:57
scottdaOK, this seems familiar. I'll dig a bit more. Thanks smcginnis16:57
*** alonma has quit IRC16:57
smcginnisscottda: Leave your local branch, git checkout master; git pull; git review -d [patchid]; git rebase master16:58
*** alonma has joined #openstack-cinder16:58
*** jgriffith is now known as jgriffith_away16:58
openstackgerritEric Harney proposed openstack/cinder: Log stack trace for middleware faults  https://review.openstack.org/27796316:58
openstackgerritScott DAngelo proposed openstack/cinder: cinder-api-microversions code  https://review.openstack.org/22491016:59
sheeldulek: hi16:59
hemnaCI = busted16:59
scottdaBOOM16:59
hemnasomeone pushed a new version of eventlet out and remove the older version from pypi17:00
hemnaglobal requirements lists the older version as the upper limit.17:00
hemnaoops.17:00
smcginnishemna: Yep, actually deleted the old version of eventlet off of pypi.17:00
*** mragupat has quit IRC17:00
sheeldulek: could we discuss your comment on https://review.openstack.org/#/c/274589/?17:00
smcginnishemna: Bad form.17:00
*** timcl has quit IRC17:00
hemnasmcginnis, save space = break the world.17:00
hemnawee!17:00
smcginnishemna: Watch it all burn.17:00
*** mragupat has joined #openstack-cinder17:01
hemnakick back, drink your coffee and watch it all....17:01
smcginnisWell, if anyone was waiting for a window to do some CI maintenance, nows your time. :)17:01
scottdasmcginnis: Actually, I think that lost all my changes from today17:01
hemna:)17:01
smcginnisscottda: You didn't delete your original branch, did you?17:01
scottdaI was on master17:02
*** alonma has quit IRC17:02
smcginnisscottda: As hemna would say: pewp17:02
scottdaha17:02
scottdaNo big. I can fix17:03
smcginnisscottda: Hopefully it wasn't too much.17:03
* patrickeast catching up17:03
*** raildo-afk is now known as raildo17:03
*** alonma has joined #openstack-cinder17:04
scottdano, not to bad17:04
patrickeastWe're they committed at some point?17:04
scottdapatrickeast: locally17:04
scottdaBut not on a branch. on master17:04
smcginnisPro tip: work on branches. :P17:04
patrickeastThat's fine, look at git reflog17:04
patrickeastAnd check-out the sha id of when they were on master17:05
scottdaOH yeah! reflog magic.17:05
*** savihou has quit IRC17:06
smcginnisscottda: See, I told you patrickeast was a git pro. ;)17:06
*** knomura has quit IRC17:07
*** isaacb has quit IRC17:07
*** alonma has quit IRC17:08
*** dan_nguyen has joined #openstack-cinder17:08
openstackgerritRaildo Mascena proposed openstack/cinder: Include allocated quota value in the quota reserve  https://review.openstack.org/24022817:08
scottdapatrickeast: That did the trick. Thanks!17:09
*** chris_morrell has joined #openstack-cinder17:09
patrickeastnp17:09
*** mylu has quit IRC17:10
scottdaI shall send you a fruit log this Christmas17:10
*** alonma has joined #openstack-cinder17:11
patrickeasthaha17:12
*** arecknag has quit IRC17:14
*** jungleboyj has joined #openstack-cinder17:15
*** leeantho has joined #openstack-cinder17:15
*** dramakri has joined #openstack-cinder17:15
*** alonma has quit IRC17:16
*** martyturner has joined #openstack-cinder17:18
*** mudassirlatif has joined #openstack-cinder17:19
*** alonma has joined #openstack-cinder17:20
openstackgerritVasanthi Thirumalai proposed openstack/cinder: Add an implementation for validate_connector api in VMEM cinder driver  https://review.openstack.org/27764417:20
-openstackstatus- NOTICE: Gerrit is restarting now, to alleviate current performance impact and WebUI errors.17:23
diablo_rojo^^awesome17:24
*** knomura has joined #openstack-cinder17:24
*** alonma has quit IRC17:24
*** alonma has joined #openstack-cinder17:26
*** amoturi has joined #openstack-cinder17:26
*** mylu has joined #openstack-cinder17:27
*** knomura has quit IRC17:28
*** bardia has joined #openstack-cinder17:29
*** alonma has quit IRC17:30
*** jdurgin1 has quit IRC17:30
openstackgerritRyan McNair proposed openstack/cinder: Split out NestedQuotas into a separate driver  https://review.openstack.org/27482517:31
*** e0ne has quit IRC17:32
*** mragupat has quit IRC17:32
*** mragupat has joined #openstack-cinder17:33
*** knikolla has joined #openstack-cinder17:33
amoturiHi Sean, could u take a look at https://review.openstack.org/#/c/273741/  The CI results are now posted.17:33
openstackgerritVictor Stinner proposed openstack/cinder: Port utils.safe_minidom_parse_string() to Python 3  https://review.openstack.org/27798317:33
openstackgerritScott DAngelo proposed openstack/cinder: cinder-api-microversions code  https://review.openstack.org/22491017:34
*** mylu has quit IRC17:34
*** alonma has joined #openstack-cinder17:34
openstackgerritVictor Stinner proposed openstack/cinder: Port utils.safe_minidom_parse_string() to Python 3  https://review.openstack.org/27798317:34
*** jdurgin1 has joined #openstack-cinder17:36
*** jdurgin1 has left #openstack-cinder17:36
*** timcl has joined #openstack-cinder17:36
*** mylu has joined #openstack-cinder17:36
*** mragupat has quit IRC17:36
*** mragupat has joined #openstack-cinder17:37
*** Vasanthi has joined #openstack-cinder17:38
*** alonma has quit IRC17:38
*** laughterwym has joined #openstack-cinder17:39
*** dustins_ has joined #openstack-cinder17:40
*** dustins has quit IRC17:41
jungleboyjsmcginnis: When you have a chance can you put this in?  https://review.openstack.org/#/c/275927/  :-)  Thanks.17:43
openstackgerritVasanthi Thirumalai proposed openstack/cinder: Add an implementation for validate_connector  https://review.openstack.org/27764417:43
*** laughterwym has quit IRC17:44
*** sinese_ has joined #openstack-cinder17:46
openstackgerritVictor Stinner proposed openstack/cinder: Port API contribs to Python 3  https://review.openstack.org/27786117:50
openstackgerritVictor Stinner proposed openstack/cinder: Port utils.safe_minidom_parse_string() to Python 3  https://review.openstack.org/27798317:50
openstackgerritVictor Stinner proposed openstack/cinder: py3: Fix usage of JSON in API contrib  https://review.openstack.org/27798917:50
*** ntt has quit IRC17:50
*** ildikov has quit IRC17:53
openstackgerritVictor Stinner proposed openstack/cinder: Port API v1 and v2 to Python 3  https://review.openstack.org/27783517:53
*** vgridnev has joined #openstack-cinder17:53
*** timcl1 has joined #openstack-cinder17:53
*** alkhodos has joined #openstack-cinder17:53
*** knikolla has quit IRC17:54
*** lpetrut has joined #openstack-cinder17:55
*** jordanP has joined #openstack-cinder17:55
*** timcl has quit IRC17:56
*** bardia has quit IRC17:56
*** mylu has quit IRC17:56
*** mylu has joined #openstack-cinder17:57
*** knikolla has joined #openstack-cinder17:58
*** mylu has quit IRC17:58
*** jwang has joined #openstack-cinder18:00
*** jwang_ has quit IRC18:00
*** akshai has quit IRC18:00
*** mylu has joined #openstack-cinder18:00
kmartin__jungleboyj, do you want to add a release note for this? https://review.openstack.org/#/c/275927/ :)18:00
*** akshai has joined #openstack-cinder18:02
*** jordanP has quit IRC18:03
*** mylu has quit IRC18:03
*** ntpttr has quit IRC18:06
*** dramakri has quit IRC18:06
openstackgerritKendall Nelson proposed openstack/os-brick: Lun id's > 255 should be converted to hex  https://review.openstack.org/27760618:07
*** mylu has joined #openstack-cinder18:08
*** alonma has joined #openstack-cinder18:10
*** dustins_ has quit IRC18:11
*** alonma has quit IRC18:15
*** dustins has joined #openstack-cinder18:16
*** e0ne has joined #openstack-cinder18:17
*** alonma has joined #openstack-cinder18:18
*** martyturner has quit IRC18:22
*** alonma has quit IRC18:22
*** martyturner has joined #openstack-cinder18:22
*** timcl1 has quit IRC18:23
*** delattec has joined #openstack-cinder18:23
*** cdelatte has quit IRC18:26
*** alonma has joined #openstack-cinder18:28
smcginnisjungleboyj: At least you name yourself in those statements. :)18:28
*** dustins has quit IRC18:28
*** dustins has joined #openstack-cinder18:28
*** dustins has quit IRC18:29
jungleboyjkmartin__: You are poking a bear today my friend.18:29
kmartin__jungleboyj, :)18:30
jungleboyjkmartin__: :-)18:30
*** esker has joined #openstack-cinder18:30
jungleboyjsmcginnis: Yes, easy to figure out who did it that way.18:30
smcginnisjungleboyj: Hey, if you're consistent on that, we really could add a hacking check.18:31
jungleboyj:-)  I will write that up.18:32
*** mragupat_ has joined #openstack-cinder18:32
*** alonma has quit IRC18:32
*** jgriffith_away is now known as jgriffith18:34
*** mragupat has quit IRC18:36
*** ildikov has joined #openstack-cinder18:36
*** akshai has quit IRC18:36
e0nehemna: hi. are you around? I've got a question about brickclient18:37
*** akshai has joined #openstack-cinder18:38
*** avishay has quit IRC18:39
*** ociuhandu has quit IRC18:39
jungleboyjpatrickeast: kmartin__ What a bunch of wise guys!18:41
hemnae0ne, hey18:42
e0nehemna: according to my patch with attach/dettach18:43
e0ne1) https://review.openstack.org/277788 - my patch for NFS connector18:44
e0neand 2) are we going to add rootwrap for brickclient for Mitaka release?18:44
hemnae0ne, https://review.openstack.org/#/c/27722418:44
e0new/o rootwrap I need to do few hacks and it will be very ugly UX18:44
hemnaI'm working on helping that18:44
*** ChubYann has joined #openstack-cinder18:44
hemnabut it's currently not working :(18:44
e0ne:(18:45
*** mylu has quit IRC18:45
e0neI can take a look on it tomorrow18:45
e0nebut in case, if we'll not merge it in Mitaka18:45
hemnahttps://review.openstack.org/#/c/277957/18:45
hemnaI had to post that patch against privsep fwiw18:45
e0newhat are we going to  to do with attach/detach stuff in brickclient?18:45
hemnato fix one of it's problems18:46
hemnawhat do you mean?18:46
hemnathe cinder client extension is what you mean I presume ?18:46
*** rlrossit has quit IRC18:46
e0neyes18:46
e0nepython-brick-cinderclient-ext18:46
VasanthiAny idea why all CIs are failing with this error?  2016-02-09 16:25:43.641 |   Could not find a version that satisfies the requirement eventlet===0.18.1 (from -c /opt/stack/new/requirements/upper-constraints.txt (line 122)) (from versions: 0.2, 0.5.3, 0.6.1, 0.7, 0.8, 0.8.16, 0.9.17, 0.10.0, 0.11.0, 0.12.1, 0.13.0, 0.14.0, 0.15.2, 0.16.1, 0.17.4, 0.18.2) 2016-02-09 16:25:43.662 | No matching distribution found for eventl18:46
*** chhavi has quit IRC18:47
*** bardia has joined #openstack-cinder18:47
hemnaVasanthi, there is a fix working it's way through the gate18:47
e0neVasanthi: there was an issue with eventlet==0.18. maybe  0.18.1 it not available on all pypi mirrors18:47
hemnasomeone released a new version of eventlet 0.18.2, and then deleted 0.18.118:47
hemnawhich broke the world.18:47
e0ne:(18:47
*** delatte has joined #openstack-cinder18:48
diablo_rojopatrickeast: How did testing go of the multipath brick patch go yesterday? :)18:49
*** mriedem has quit IRC18:49
*** wN has quit IRC18:50
kmartin__Vasanthi, https://review.openstack.org/#/c/277912/18:51
*** delattec has quit IRC18:51
*** mriedem has joined #openstack-cinder18:51
e0nehemna: https://review.openstack.org/#/c/263744/9/brick_cinderclient_ext/brick_utils.py - line #3418:51
hemnae0ne, ew that is ugly18:52
hemnaso....yah18:52
hemna:P18:52
e0nerootwrap:(18:52
*** timcl has joined #openstack-cinder18:52
hemnaI'm not sure how else to do that18:52
hemnaw/o privsep fix18:52
e0nethe same for me18:53
hemnahttp://paste.openstack.org/show/486460/18:53
hemnaso that's the latest w/ the privsep patch18:53
e0neI'll try to take a look on privsep patch tomorrow.18:53
hemnaI guess I can try and debug it and see if I can offer up a fix18:53
hemnaI'd like to get this working ASAP18:53
e0neprobably I will help somehow18:53
hemnanot sure what timezone gus is in18:53
*** rlrossit has joined #openstack-cinder18:54
e0nehemna: the error means that you try access to the file w/o sudo - but I'm sure you know it18:54
*** wN has joined #openstack-cinder18:55
*** wN has joined #openstack-cinder18:55
*** martyturner has quit IRC18:56
*** krtaylor has quit IRC18:57
*** cknight has quit IRC19:00
hemnawell if you cat the file w/o sudo it says permission denied19:00
hemnanot file not found19:00
hemnacat /etc/iscsi/initiatorname.iscsi19:00
hemnacat: /etc/iscsi/initiatorname.iscsi: Permission denied19:00
*** akshai has quit IRC19:01
*** akshai has joined #openstack-cinder19:02
*** martyturner has joined #openstack-cinder19:02
*** raildo is now known as raildo-afk19:02
e0nehemna: please, share brick_initiator.py19:02
e0nehemna: in shell19:03
*** dramakri has joined #openstack-cinder19:03
e0nehemna: in python I'd got "file not found" error19:03
*** dramakri has quit IRC19:03
*** cknight has joined #openstack-cinder19:03
hemnahttps://github.com/WaltHP/diediedie19:03
hemnahttps://github.com/WaltHP/diediedie/blob/master/diediedie/brick_initiator.py19:03
*** ociuhandu has joined #openstack-cinder19:03
smcginnis:)19:03
hemna:)19:04
hemnakinda looks like sudo is dropped19:05
hemnahrmm19:05
patrickeastdiablo_rojo: hey, sorry haven't had a chance to test it out, got side tracked with other stuff19:06
diablo_rojopatrickeast: No worries. Think you have time today or tomorrow? The deadline is Friday for brick patches.19:07
*** jungleboyj has quit IRC19:07
patrickeastdiablo_rojo: yea, should be able to19:08
diablo_rojopatrickeast: Cool, so long as we are all happy with it by Friday we should be good. I appreciate the help :)19:08
*** krtaylor has joined #openstack-cinder19:09
*** mylu has joined #openstack-cinder19:12
alkhodosHi everyone! Can some of the cores take a look at this patch please? https://review.openstack.org/#/c/271537/ Thank you.19:14
ildikovhemna: hi19:14
e0nehemna: too late for me to be useful on it. will try tomorrow morning19:15
ildikovhemna: I just wanted to ask that when do you think you can write as that mail about your approach for disconnect_volume for multiattach?19:15
*** timcl1 has joined #openstack-cinder19:15
hemnaildikov, hey19:15
hemnaildikov, crap, I'll do that today.19:15
ildikovhemna: np, today is fine too :)19:16
*** esker has quit IRC19:16
ildikovhemna: I just would like to have the thread started19:16
ildikovhemna: thanks!19:16
hemnaI'll start it now.19:16
hemnasorry about that19:16
*** mylu has quit IRC19:17
*** esker has joined #openstack-cinder19:17
*** dims_ has joined #openstack-cinder19:17
*** dims has quit IRC19:17
*** timcl has quit IRC19:18
ildikovhemna: no probs, I don't think one day matters here19:18
ildikovhemna: just don't tell anyone I said that ;)19:18
*** esker has quit IRC19:19
*** mylu has joined #openstack-cinder19:20
hemnaI'll get it out here before lunch.19:20
hemnabad hemna19:20
*** mylu has quit IRC19:24
*** dims_ has quit IRC19:27
*** alonma has joined #openstack-cinder19:28
*** JoseMello has quit IRC19:28
*** SDub has joined #openstack-cinder19:29
openstackgerritKendall Nelson proposed openstack/os-brick: Lun id's > 255 should be converted to hex  https://review.openstack.org/27760619:29
SDubStruggling with this bug right now using CirrOS 0.3.4 and Juno: https://bugs.launchpad.net/cirros/+bug/131219919:29
openstackLaunchpad bug 1312199 in OpenStack Compute (nova) icehouse "cirros 0.3.1 fails to boot" [High,Fix released] - Assigned to Attila Fazekas (afazekas)19:29
SDubon all accounts this issue shouldn't be happening but for some reason it is.19:29
*** dims has joined #openstack-cinder19:30
openstackgerritMerged openstack/cinder: Added Keystone and RequestID headers to CORS middleware  https://review.openstack.org/26540919:31
*** alonma has quit IRC19:32
openstackgerritMerged openstack/cinder: Adds support for configuring zoning in a virtual fabric  https://review.openstack.org/27540719:33
*** ChubYann has quit IRC19:33
apoorvadHello cores! Please take a loot at the reviews https://review.openstack.org/#/c/268370/ and https://review.openstack.org/#/c/261126/19:34
*** SDub has left #openstack-cinder19:34
*** esker has joined #openstack-cinder19:34
*** esker has quit IRC19:34
openstackgerritMerged openstack/cinder: Port backup drivers to Python 3  https://review.openstack.org/27777819:34
*** esker has joined #openstack-cinder19:34
*** alonma has joined #openstack-cinder19:34
openstackgerritMerged openstack/cinder: DRBD: Fix arguments for resize_volume DBus API call.  https://review.openstack.org/27731419:35
*** sheel has quit IRC19:37
ildikovhemna: :)19:37
*** alonma has quit IRC19:39
*** can8dnSix has joined #openstack-cinder19:41
*** dims has quit IRC19:41
*** dims has joined #openstack-cinder19:42
*** alonma has joined #openstack-cinder19:42
*** jungleboyj has joined #openstack-cinder19:43
*** ChubYann has joined #openstack-cinder19:44
*** mriedem has quit IRC19:45
*** dims_ has joined #openstack-cinder19:46
*** alonma has quit IRC19:46
*** mylu has joined #openstack-cinder19:46
*** mriedem has joined #openstack-cinder19:47
*** dims has quit IRC19:49
*** e0ne has quit IRC19:49
*** fernnest__ has quit IRC19:52
hemnaildikov, ok sent19:52
hemnahopefully that makes sense.19:52
ildikovhemna: cool, thanks much!19:52
*** fernnest has quit IRC19:52
*** fernnest has joined #openstack-cinder19:52
ildikovhemna: I will check, I cannot promise I can put my brain together today to write a reply that makes sense, so I might do it tomorrow morning instead :)19:53
*** mylu has quit IRC19:53
hemnaok no problemo.19:53
hemnaI just described 1 single issue, which is determining how to know when to call brick's disconnect_volume or not.19:53
hemnaI figured it was best to not confuse everyone with the other somewhat related issues of storing the connector, etc.19:54
*** esker has quit IRC19:56
*** dims_ has quit IRC19:57
*** can8dnSix has quit IRC19:59
*** dustins has joined #openstack-cinder20:00
*** ndipanov has quit IRC20:02
*** akshai has quit IRC20:02
*** timcl1 has quit IRC20:03
*** timcl has joined #openstack-cinder20:04
*** jgriffith has quit IRC20:05
openstackgerritMatan Sabag proposed openstack/cinder: Manage/unmanage volume in ScaleIO driver  https://review.openstack.org/26834420:05
*** akshai has joined #openstack-cinder20:05
*** jgriffith has joined #openstack-cinder20:06
*** jgriffith is now known as Guest5542120:06
*** rcernin has joined #openstack-cinder20:08
smcginnishemna: Good write up.20:13
*** jungleboyj has quit IRC20:13
*** alonma has joined #openstack-cinder20:18
*** krtaylor has quit IRC20:19
*** martyturner has quit IRC20:19
*** martyturner has joined #openstack-cinder20:21
*** alonma has quit IRC20:23
openstackgerritMerged openstack/cinder: Move wsgi to oslo_service.wsgi  https://review.openstack.org/25854120:24
*** Guest55421 is now known as jgriffith20:24
openstackgerritMerged openstack/cinder: hacking: Fix false positive in C302 check  https://review.openstack.org/27784120:25
*** amoturi has left #openstack-cinder20:25
jgriffithdid anybody see the requirements update yet?20:28
jgriffithie did it merge and I missed it :)20:29
smcginnisjgriffith: Nope20:29
jgriffithhmm20:29
jgriffithannoying20:29
smcginnisjgriffith: Yep20:29
jgriffithof course, I'm assuming it's in process20:29
smcginnisjgriffith: Top of the gate queue right now.20:29
jgriffithsmcginnis: :)  thanks20:29
smcginnisjgriffith: Looks like it's done and just wrapping up. Says 0 minutes left.20:29
*** mylu has joined #openstack-cinder20:29
smcginnisjgriffith: Taking way too long! :)20:29
*** raildo-afk is now known as raildo20:29
*** ndipanov has joined #openstack-cinder20:31
*** krtaylor has joined #openstack-cinder20:31
*** shyama has quit IRC20:34
*** esp has joined #openstack-cinder20:35
*** salv-orl_ has quit IRC20:36
*** esp has quit IRC20:39
*** mylu has quit IRC20:40
*** mylu has joined #openstack-cinder20:40
openstackgerritMitsuhiro Tanino proposed openstack/cinder: [LVM] Restore target config during driver initialization  https://review.openstack.org/27142420:42
openstackgerritMitsuhiro Tanino proposed openstack/cinder: [LVM] Restore target config during ensure_export  https://review.openstack.org/27142420:42
mc_nairsmcginnis, jgriffith, DuncanT - the nested quota updates are ready for review at https://review.openstack.org/#/c/274825/.  Feedback certainly welcome on the code or overall approach20:46
*** esp has joined #openstack-cinder20:47
smcginnismc_nair: Thanks, I'll try to take a look soon.20:48
*** mylu has quit IRC20:48
mc_nairshweet20:48
mc_nairunfortunately it's a decent amount of code :/ but some of that is test coverage + releasenotes20:49
*** knikolla has quit IRC20:49
*** knikolla has joined #openstack-cinder20:50
*** esp has quit IRC20:51
smcginnisYay, my CI can finally stack again!20:52
*** salv-orlando has joined #openstack-cinder20:53
*** esker has joined #openstack-cinder20:53
*** krtaylor has quit IRC20:53
*** akerr has quit IRC20:54
hemnasmcginnis, w00t!20:54
*** chris_morrell has quit IRC20:58
jgriffithsmcginnis: sweeeet!20:59
jgriffithsmcginnis: now to send nasty emails to the eventlet team :)20:59
*** esker has quit IRC20:59
smcginnisjgriffith: I'm staying out of that one!20:59
Swansonjgriffith, repl v2.1 patchset 4 due up today?21:00
jgriffithsmcginnis: well that's no fun :)21:00
jgriffithSwanson: yes... sorry I'm behind21:00
jgriffithSwanson: I'm sick today and keep nodding off :(21:00
smcginnisUgg21:01
Swansonjgriffith, Not a problem.  I'm off to a dr's appt myself in a few.  Was just wondering if I should start stacking with what is up there now or wait until later.21:01
Swansonjgriffith, get better, btw.21:01
jgriffithSwanson: thanks21:01
jgriffithwell... there *will* be an update sometime today :)21:02
jgriffithdepending on your TZ :)21:02
*** dims has joined #openstack-cinder21:03
*** krotscheck is now known as krotscheck_dcm21:03
*** e0ne has joined #openstack-cinder21:03
*** knikolla has quit IRC21:03
jgriffithsigh...  2 email inboxes, 2 calendars, 2 vpns; not fun21:04
jgriffitheven with forwarding... still painful21:04
*** timcl has quit IRC21:06
smcginnisjgriffith: Oh, just wait for the aquisition to get into full swing.21:06
*** krtaylor has joined #openstack-cinder21:07
jgriffithsmcginnis: sigh21:11
jgriffithsmcginnis: maybe I can convince them to just move everything to Google for Biz :)21:11
smcginnisjgriffith: Hah, sure. Let me know how that goes. :P21:12
jgriffithsmcginnis: yeah... kinda what I'm thinking.  But hey!  It would be EPIC!!!  I could put it on my resume :)21:12
smcginnisTrue!21:12
ildikovhemna: read the mail finally, I think it is a good start, thanks!21:15
ildikovhemna: I will reply to it tomorrow21:15
scottdayes hemna , nice work21:16
ildikovscottda: +1 :)21:16
*** lpetrut has quit IRC21:18
*** alonma has joined #openstack-cinder21:19
*** martyturner has quit IRC21:20
*** alonma has quit IRC21:23
*** martyturner has joined #openstack-cinder21:23
*** erlon has quit IRC21:26
*** alonma has joined #openstack-cinder21:27
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Move integrated tests to 'functional' directory  https://review.openstack.org/26780121:29
smcginnismc_nair: Thanks for jumping in there.21:31
*** jgriffith is now known as jgriffith_away21:31
mc_nairsmcginnis: sure thing.  Some common implementation sounds like the way to go in the future, hoping that if we fix up our nested quota support that could be leveraged as a starting  base21:31
mc_nairthanks diablo_rojo for tipping me off about the convo21:32
*** alonma has quit IRC21:32
mc_nairbut regardless, I think we need to blaze forward on the current fixup plan for our existing nested quotas, since we've already released and are bit behind the 8ball because of that21:32
smcginnismc_nair: I agree.21:33
diablo_rojomc_nair: No problem. I had said basically the same as what you did that they should look at implementing quotas and then extending it, but I figured if they started asking more questions it would be better to have you there21:33
smcginnismc_nair: Then hopefully whatever that ends up looking like fits in with whatever cross project lands on.21:33
*** alonma has joined #openstack-cinder21:34
*** baumann has quit IRC21:34
mc_nairsmcginnis: yea - think if it's ends up coded nicely we should be able to pull out most of the logic and generalize it21:35
*** baumann has joined #openstack-cinder21:35
*** tpsilva has quit IRC21:36
*** alonma has quit IRC21:38
openstackgerritMerged openstack/python-cinderclient: Remove debug statement  https://review.openstack.org/27592721:38
*** vgridnev has quit IRC21:40
*** alonma has joined #openstack-cinder21:40
ildikovhemna: scottda: still here?21:40
scottdahi21:40
ildikovhey :)21:40
ildikovI just had some thoughts about hemna's approach about which I wanted to ask questions before writing a silly mail :)21:41
scottdaok21:42
*** laughterwym has joined #openstack-cinder21:42
ildikovbasically if I remember correctly our issue with storing the target was that when initialize_connection is called we don't have the instance or smth like this21:42
ildikovdo I remember correctly?21:42
scottdaYes, you are correct21:42
*** cknight has quit IRC21:43
ildikovok, so what I was wondering about is that what if we add the possibility to the Cinder API to provide this information after attach?21:43
ildikovI mean Nova gets the info, it should have everything and it can call out to Cinder to feed back the info if the attach is successful21:44
scottdaSure, that's possible21:44
ildikovwe could store the info as an additional item for the attachment in the attachments list in the volume info21:44
*** alonma has quit IRC21:44
ildikovand then both components could easily check who's attached to which target21:45
ildikovif the info is missing because of race condition or any other issue we can still fall back to those options we discussed already21:45
scottdaYes, that all makes sense to me.21:45
ildikovit could work with live migration too I think21:46
scottdahemna: might have feedback, but it sounds like a possibility worth thinking about.21:46
ildikovnot sure about race conditions in that case though :(21:46
*** laughterwym has quit IRC21:46
ildikovok, cool tnx21:46
ildikovI wanted only a base sanity check, I will add this to the mail thread21:47
scottdaYou do indeed sound sane :)21:47
ildikovcool, I will refer to you if someone proves the opposite ;)21:47
*** raildo is now known as raildo-afk21:49
*** nkrinner has quit IRC21:50
*** martyturner has quit IRC21:54
openstackgerritMerged openstack/cinder: Zfssaiscsi driver should return target_lun as int  https://review.openstack.org/27374121:54
*** martyturner has joined #openstack-cinder21:56
*** alonma has joined #openstack-cinder21:57
*** esker has joined #openstack-cinder22:00
*** alonma has quit IRC22:01
*** jwcroppe has joined #openstack-cinder22:02
hemnaback22:02
hemnasup22:02
hemnathe issue with storing the connector is live migration22:03
hemnawe need a new Cinder api to have cinder update the attachment22:03
hemnaduring live migration.22:03
hemnaand to do that, we need microversions to land22:04
hemna....22:04
hemna...22:04
scottdanew patch set for microversions is coming soon....just one small fix....22:04
*** esker has quit IRC22:05
ildikovwhat I propose is definitely a new API22:05
*** alonma has joined #openstack-cinder22:05
hemnaildikov, I had discussed the same thing in an earlier ML thread22:05
ildikovhemna: I added this idea now to this thread as well22:05
ildikovhemna: what was the outcome of the previous?22:06
*** gouthamr has quit IRC22:06
hemnathe outcome was, we couldn't do it in M, because we needed microversions.22:06
hemnaIt's on my N TODO list22:06
ildikovit works in N as well if we have microversion22:06
ildikovyour initial proposal is needed as a fallback option anyway IMHO22:07
*** mylu has joined #openstack-cinder22:08
*** mylu has quit IRC22:09
*** alonma has quit IRC22:09
ildikovhemna: I guess based on the outcome of the ML thread we can add this item to the TODO list for Cinder for N, right?22:10
hemnaildikov, that's the hope yah22:10
ildikovhemna: scottda: how far is microversion from flying?22:11
*** cknight has joined #openstack-cinder22:11
*** martyturner has quit IRC22:11
scottdaildikov: Not sure. I had failures in CI that I couldn't repro on my desktop, but I've fixed some things since then. I've a new patch just waiting on a unit test fix. Might be in OK shape for review as early as today.22:12
*** sinese_ has quit IRC22:12
ildikovscottda: ok, that sounds promising22:12
scottdaYes, but note that I did not make any promises :)22:13
ildikovscottda: I just wasn't sure as hemna mentioned his TODO list22:13
scottdahaha, no , it's on my TODO list.22:13
ildikovbut then I think I misunderstood which item is on that list :)22:13
scottdaapi microversions is on my list22:14
ildikovok, I see now :)22:14
openstackgerritIvan Kolodyazhny proposed openstack/cinder: WIP!!!!! Fix Rally job  https://review.openstack.org/27811922:14
ildikovscottda: thanks for the clarifications22:14
scottdaNew API to update Cinder connector info (for live migration) is on hemna22:14
hemnascottda, +122:14
hemnawe need to be able to hand the connector at attach time, and the new update_attach api22:15
ildikovis that new API equals to what I wrote above?22:15
ildikovI imagined the whole process as update by having the connector_info empty when the attachment created and then update as many times as needed22:16
*** porrua has quit IRC22:17
e0neit's needed not only for live migration22:17
e0nesee also thread "Cleanly detaching volumes from failed nodes"22:17
ildikovbut this part looks like a detail22:17
scottdaildikov: had the idea of updating the connector with the instance id using the new API....22:18
scottdaThis info is not known at attach time...22:18
scottdaBut could be updated later by Nova22:18
apoorvadsmcginnis patrickeast hemna : my CI is failing on test_volume_create_get_update_delete_as_clone does anyone of u know what changed? http://openstack-ci.tintri.com/tintri/refs-changes-35-277835-3/console.log.out22:18
ildikove0ne: you mean the connector_info would be needed in the failed nodes case, right?22:19
*** mylu has joined #openstack-cinder22:19
e0neildikov: yes22:19
hemnae0ne, ildikov it's the same data needed to detach22:19
*** martyturner has joined #openstack-cinder22:19
ildikovhemna: yeah exactly, so it's failed nodes and multiattach, where we definitely need this data22:20
hemnawhen either 1) nova evacuate is called or 2) when nova has no record of the instance anymore and cinder still thinks it's attached.22:20
smcginnisapoorvad: Volume driver reported an error: Failed to create snapshot for volume /tintri/openstack/volume-4e0d19d0-715e-4943-bdcf-9fec1561a9b422:20
smcginnisapoorvad: Looks like it's something on your array side.22:20
ildikovhemna: yeah, agreed22:20
openstackgerritIvan Kolodyazhny proposed openstack/cinder: WIP!!!!! Fix Rally job  https://review.openstack.org/27811922:20
apoorvadsmcginnis: thanks for the help. I will look into it22:21
hemna-1  excessive !'s   4 !'s is ok.  5....excessive22:21
smcginnisHah22:21
smcginnisapoorvad: Yeah, looking through your screen-c-vol log it looks like errors in there, but I don't see anything logging exactly why it failed.22:22
ildikovhemna: ok, cool, then I think we are on the same page regarding the slightly longer term approach here, let's see the outcome of the ML thread22:24
ildikovI will ping the Nova folks later if no response from that side soon22:24
ildikovaaand going to have some sleep now :)22:24
smcginnisapoorvad: Ah, I at least see why there's nothing useful in the log.22:24
hemnaildikov, ok cool.  thanks for the updates in the ML22:24
smcginnisapoorvad: You check the requests response.status_code but none of the other data returned.22:25
e0ne#NOTE (e0ne): do not use 5 '!' in one place22:25
ildikovhemna: np, it would be good to have all the options captured there22:25
hemnaildikov, +122:25
*** diablo_rojo has quit IRC22:25
smcginnisapoorvad: Might want to do what I did here: https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/dell/dell_storagecenter_api.py#L24822:26
apoorvadsmcginnis: checking on the array what's going on22:26
ildikovhemna: have a nice rest of the day :)22:26
hemnaildikov, thanks :)  get some Zzzss22:26
ildikovhemna: I stop re-inventing ideas now :)22:26
ildikovlaters22:26
apoorvadsmcginnis; that would be helpful. thanks22:27
*** crose has quit IRC22:30
*** julim has quit IRC22:32
*** esker has joined #openstack-cinder22:34
*** wN has quit IRC22:34
*** jgregor has quit IRC22:35
*** baumann has quit IRC22:35
*** dustins has quit IRC22:35
*** dschoenb__ has joined #openstack-cinder22:35
*** dschoenb__ is now known as dustins22:35
notmorganhmm22:39
*** dims_ has joined #openstack-cinder22:39
*** salv-orl_ has joined #openstack-cinder22:40
*** dims has quit IRC22:40
*** wN has joined #openstack-cinder22:41
*** wN has joined #openstack-cinder22:41
*** jwcroppe has quit IRC22:42
*** rlrossit has quit IRC22:42
*** salv-orlando has quit IRC22:43
*** esker has quit IRC22:46
*** martyturner has quit IRC22:47
*** dramakri has joined #openstack-cinder22:50
*** esker has joined #openstack-cinder22:52
*** daneyon has joined #openstack-cinder22:55
*** mylu has quit IRC22:56
*** tongli has quit IRC22:56
*** rcernin has quit IRC22:58
openstackgerritVincent Hou proposed openstack/cinder: Migration: take the local_path for the source volume  https://review.openstack.org/25022022:58
*** daneyon_ has quit IRC22:58
*** mylu has joined #openstack-cinder22:59
*** dustins has quit IRC23:00
*** edtubill has quit IRC23:01
*** edtubill has joined #openstack-cinder23:02
*** eharney has quit IRC23:03
*** alonma has joined #openstack-cinder23:05
*** edtubill has quit IRC23:06
*** xyang1 has quit IRC23:09
*** alonma has quit IRC23:10
*** alonma has joined #openstack-cinder23:11
*** alonma has quit IRC23:16
*** alonma has joined #openstack-cinder23:17
*** daneyon_ has joined #openstack-cinder23:18
*** daneyon has quit IRC23:21
*** alonma has quit IRC23:22
*** mriedem has quit IRC23:22
openstackgerritIvan Kolodyazhny proposed openstack/cinder: WIP! Fix Rally job  https://review.openstack.org/27811923:24
*** jungleboyj has joined #openstack-cinder23:26
hemnaonly 1!23:26
hemna+A23:26
hemna:P23:26
e0newait, you forgot -1 on commit message:)23:27
hemnacrap23:27
*** alonma has joined #openstack-cinder23:29
e0nehemna: https://review.openstack.org/277788 - here is one more good candidate to +2 or -1 ;)23:30
hemna*click*23:30
e0neall gate-tempest-dsvm-full are broken:(23:30
hemnaCI has been borked all day today23:31
hemnasux23:31
e0neit's good time to sleep23:32
e0neI've raised a question about gates in #openstack-qa23:32
e0ne[01:32:16]  <jlvillal>Trying to figure out if it is my patch or something else.23:33
hemnabed time at 3:30pm.  sounds good...23:33
hemna:P23:33
*** alonma has quit IRC23:34
hemnae0ne, so the unmount23:34
e0ne1.34am23:34
hemnaI think folks have rejected that in the past for some reason for NFS23:34
hemnaeharney might have some insight23:35
e0nehemna: I have to ask him23:35
hemnaI don't really see an issue with it.23:35
e0nehemna: because for now, nova does it23:35
e0nehemna: and I'm not sure how volume migration will work23:35
e0nehemna: volume could be attached after migration23:36
e0neit will be easier to get it landed in os-brick before nova will use this connector23:37
*** hongbin has joined #openstack-cinder23:38
*** prithiv has joined #openstack-cinder23:39
*** jwcroppe has joined #openstack-cinder23:40
hemnayah I'm sure it's left behind at this point23:41
hemnawell23:41
hemnaI dunno, because the nfs driver doesn't use the brick connector23:41
gushemna: I'm UTC+11 (Sydney) fwiw.23:43
hemnagus, hey! :)23:43
hemnagus, ok cool thanks, I wasn't sure.23:43
gus:)23:43
hemnagus, did you happen to see my last comment on the brick patch?23:44
hongbinHi Cinder team, prithiv attempts to attach a volume to VM (via Magnum) and encounter an error from Cinder. I guess he will show the details soon23:44
hemnaafter I applied my privsep-helper patch23:44
*** jwcroppe has quit IRC23:44
hemnagus, http://paste.openstack.org/show/486460/23:44
hemnaI'm not sure what's up with that23:44
prithivhi23:45
gusI have the one-character fix for the latest bug - just trying to get a unittest together (because I constantly underestimate how many bugs can be hiding in "obviously correct" python code)23:45
hemnaI tried running my script as sudo and it still happens23:45
prithivhttp://paste.openstack.org/show/486501/23:45
hemnagus, :)23:45
*** jwcroppe has joined #openstack-cinder23:45
hemnaprithiv, I'm not sure what is causing that.  I've never seen that one23:46
*** smoriya_ has joined #openstack-cinder23:46
prithivi am seeing this error when my magnum client tries to attach volume to vm23:47
prithivi see this error in c-api log23:47
*** wanghao has quit IRC23:47
*** wanghao has joined #openstack-cinder23:48
*** yhayashi has joined #openstack-cinder23:48
prithivanyone have any idea about this ?23:49
hemnaprithiv, I've not tried that :(23:49
hemnacan you do an attach to a nova instance with the nova client ?23:49
*** ndipanov has quit IRC23:49
*** jwcroppe has quit IRC23:49
*** alonma has joined #openstack-cinder23:51
*** alonma has quit IRC23:56
*** mragupat_ has quit IRC23:57
*** alonma has joined #openstack-cinder23:57
openstackgerritDina Belova proposed openstack/cinder: Use OSprofiler options consolidated in lib itself  https://review.openstack.org/27815523:58

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