Friday, 2015-03-27

*** dannywil_ has quit IRC00:02
*** ho has joined #openstack-cinder00:02
*** markvoelker has quit IRC00:02
*** annashen has joined #openstack-cinder00:03
*** annegentle has joined #openstack-cinder00:06
*** dflorea_ has quit IRC00:10
jgriffiththingee: you ok with merging this: https://review.openstack.org/#/c/160104/2100:11
*** dflorea has joined #openstack-cinder00:11
* thingee checks00:12
thingeejgriffith: are you asking because of the late schema change?00:14
jgriffiththingee: yeah00:14
jgriffiththingee: I think it's important to fix... but wanted your input and final decision00:14
*** Longgeek has quit IRC00:15
thingeejgriffith: yeah if it's important to fix, it seems like a small scope IMO00:15
thingeeso seems fine00:15
*** dflorea has quit IRC00:15
jgriffiththingee: I'd agree.  I guess *important* is subjective :)  but you know what I mean00:16
jgriffiththingee: speaking of which... do we have any interest in *finally* doing a migration squash?00:17
jgriffiththingee: I've never bothered, wonder if we should consider it00:17
thingeethat'd be interesting00:18
jgriffiththingee: hehe... good choice of words :)00:18
*** Apoorva has quit IRC00:18
jgriffiththingee: we should think about it... that and timing on saving some slots for backports blah blah blah :)00:19
jgriffiththingee: but in the meantime I think I'm going to head home :)00:19
* jgriffith has done enough damage for one day00:19
*** tsekiyam_ has joined #openstack-cinder00:21
*** mtanino has quit IRC00:24
*** tsekiyama has quit IRC00:25
*** tsekiyam_ has quit IRC00:26
*** reed has quit IRC00:29
*** reed has joined #openstack-cinder00:30
*** reed has quit IRC00:30
*** reed has joined #openstack-cinder00:31
*** leeantho has quit IRC00:38
*** annashen has quit IRC00:45
*** rhsu has joined #openstack-cinder01:00
*** esker has quit IRC01:02
*** esker has joined #openstack-cinder01:03
*** Longgeek has joined #openstack-cinder01:03
*** resker has joined #openstack-cinder01:04
*** esker has quit IRC01:08
*** marcusvrn has joined #openstack-cinder01:09
*** annegentle has quit IRC01:14
*** esker has joined #openstack-cinder01:15
*** annegentle has joined #openstack-cinder01:17
*** resker has quit IRC01:17
*** resker has joined #openstack-cinder01:19
*** vilobhmm1 has quit IRC01:19
*** Apoorva has joined #openstack-cinder01:19
*** esker has quit IRC01:21
*** dims_ has quit IRC01:23
*** Apoorva has quit IRC01:24
*** Mandell has quit IRC01:26
*** _cjones_ has quit IRC01:30
*** Lee1092 has joined #openstack-cinder01:31
*** _cjones_ has joined #openstack-cinder01:31
*** rhsu has quit IRC01:32
*** _cjones_ has quit IRC01:35
*** Apoorva has joined #openstack-cinder01:38
*** primechuck has joined #openstack-cinder01:40
*** annegentle has quit IRC01:48
*** melwitt has quit IRC01:48
*** happyeveryday has joined #openstack-cinder01:50
*** takedakn has joined #openstack-cinder01:51
*** Longgeek has quit IRC01:52
*** Longgeek has joined #openstack-cinder01:53
*** happyeveryday has quit IRC01:58
*** haigang has joined #openstack-cinder02:00
*** Apoorva has quit IRC02:10
*** haomaiwang has joined #openstack-cinder02:14
*** anshul has joined #openstack-cinder02:20
*** takedakn has quit IRC02:21
*** kaisers has joined #openstack-cinder02:22
*** kaisers1 has quit IRC02:23
*** patrickeast has quit IRC02:25
*** vilobhmm1 has joined #openstack-cinder02:28
*** haigang has quit IRC02:29
*** haigang has joined #openstack-cinder02:30
openstackgerritVincent Hou proposed openstack/cinder: Set volume_attachment to [] for the temporary volume creation  https://review.openstack.org/16713402:35
*** scottda_ has joined #openstack-cinder02:37
openstackgerritVincent Hou proposed openstack/cinder: Set volume_attachment to [] for the temporary volume creation  https://review.openstack.org/16713402:45
*** annashen has joined #openstack-cinder02:48
*** annegentle has joined #openstack-cinder02:48
*** annashen has quit IRC02:52
*** annegentle has quit IRC02:54
*** devlaps has quit IRC02:54
*** primechuck has quit IRC03:15
*** emagana has joined #openstack-cinder03:18
*** liusheng has joined #openstack-cinder03:21
*** Longgeek has quit IRC03:34
*** rushiagr_away is now known as rushiagr03:46
*** scottda_ has quit IRC03:47
*** _cjones_ has joined #openstack-cinder03:48
*** _cjones_ has quit IRC03:49
*** annegentle has joined #openstack-cinder03:50
*** emagana has quit IRC03:51
*** resker has quit IRC03:52
*** emagana has joined #openstack-cinder03:52
*** annegentle has quit IRC03:55
*** emagana has quit IRC03:56
*** _cjones_ has joined #openstack-cinder03:58
*** _cjones_ has quit IRC04:02
*** _cjones_ has joined #openstack-cinder04:02
*** marcusvrn has quit IRC04:03
*** Mandell has joined #openstack-cinder04:05
*** Longgeek has joined #openstack-cinder04:08
*** annegentle has joined #openstack-cinder04:08
*** annegentle has quit IRC04:13
*** _cjones_ has quit IRC04:33
anishthingee: shouldn't jenkins be ignoring rechecks meant for 3rd party ci's ?04:33
*** markvoelker has joined #openstack-cinder04:39
*** MentalRay has joined #openstack-cinder04:40
openstackgerritVincent Hou proposed openstack/cinder: Set volume_attachment to [] for the temporary volume creation  https://review.openstack.org/16713404:41
*** junhongl has quit IRC04:41
*** sneezewort has quit IRC04:42
*** markvoelker has quit IRC04:43
*** annashen has joined #openstack-cinder04:49
*** esker has joined #openstack-cinder04:53
*** sneezewort has joined #openstack-cinder04:53
*** junhongl has joined #openstack-cinder04:53
openstackgerritVincent Hou proposed openstack/cinder: Set volume_attachment to [] for the temporary volume creation  https://review.openstack.org/16713404:54
*** annashen has quit IRC04:54
*** esker has quit IRC04:57
*** reed has quit IRC05:03
*** rushiagr is now known as rushiagr_away05:04
*** lpetrut has joined #openstack-cinder05:13
*** annegentle has joined #openstack-cinder05:19
openstackgerritPeter Wang proposed openstack/cinder: Enhance VNX Cinder volume creation logic  https://review.openstack.org/16757305:19
*** enterprisedc has quit IRC05:23
*** annegentle has quit IRC05:24
*** markvoelker has joined #openstack-cinder05:39
*** nkrinner has joined #openstack-cinder05:41
*** anuragpalsule has joined #openstack-cinder05:41
*** jamielennox is now known as jamielennox|away05:41
*** nshaikh has joined #openstack-cinder05:42
*** markvoelker has quit IRC05:43
*** rhefner has quit IRC05:44
*** lpetrut has quit IRC05:45
*** lpetrut has joined #openstack-cinder05:46
*** dims has joined #openstack-cinder05:46
openstackgerritVincent Hou proposed openstack/cinder: Set volume_attachment to [] for the temporary volume creation  https://review.openstack.org/16713405:50
*** shilpa_ has joined #openstack-cinder05:50
*** anshul has quit IRC05:51
*** anshul has joined #openstack-cinder05:52
*** esker has joined #openstack-cinder05:53
*** esker has quit IRC05:58
*** vilobhmm1 has quit IRC06:06
*** vilobhmm1 has joined #openstack-cinder06:06
*** rushiagr_away is now known as rushiagr06:09
thingeeanish: it should yes06:14
thingeeanish: I've been seeing 'recheck <ci-name>'06:15
thingeeand jenkins not responding to that06:15
thingeeanish: here's an example https://review.openstack.org/#/c/166079/06:15
*** BharatK has joined #openstack-cinder06:19
*** annegentle has joined #openstack-cinder06:20
*** dims has quit IRC06:21
*** annegentle has quit IRC06:25
*** afazekas is now known as __afazekas06:28
*** ishant has joined #openstack-cinder06:36
*** MentalRay has quit IRC06:40
*** markvoelker has joined #openstack-cinder06:40
*** markvoelker has quit IRC06:44
*** Longgeek has quit IRC06:50
*** Longgeek has joined #openstack-cinder06:53
*** esker has joined #openstack-cinder06:55
*** pradipta has joined #openstack-cinder06:55
*** esker has quit IRC06:59
*** vilobhmm1 has quit IRC07:00
openstackgerritTushar Patil proposed openstack/python-cinderclient: Allow cinderclient to handle exception response  https://review.openstack.org/16635107:10
*** afazekas has joined #openstack-cinder07:13
openstackgerritVincent Hou proposed openstack/cinder: Set volume_attachment to [] for the temporary volume creation  https://review.openstack.org/16713407:15
openstackgerritwuyuting proposed openstack/cinder: Make backup-list support all-tenants options  https://review.openstack.org/16672707:16
*** chlong has quit IRC07:16
*** TobiasE has joined #openstack-cinder07:17
*** Miouge has joined #openstack-cinder07:26
*** angela-s has quit IRC07:32
openstackgerritTushar Patil proposed openstack/python-cinderclient: Allow cinderclient to handle exception response  https://review.openstack.org/16635107:40
*** markvoelker has joined #openstack-cinder07:40
*** lpetrut has quit IRC07:45
*** markvoelker has quit IRC07:45
*** takedakn has joined #openstack-cinder07:47
*** takedakn has quit IRC07:52
*** esker has joined #openstack-cinder07:55
*** esker has quit IRC08:00
*** ndipanov has joined #openstack-cinder08:00
*** takedakn has joined #openstack-cinder08:00
*** dulek has joined #openstack-cinder08:04
*** Mandell_ has joined #openstack-cinder08:05
*** pradipta has quit IRC08:06
*** Mandell has quit IRC08:08
*** markus_z has joined #openstack-cinder08:09
*** lpetrut has joined #openstack-cinder08:13
*** lpetrut has quit IRC08:15
*** ronis has joined #openstack-cinder08:16
*** jwang__ has joined #openstack-cinder08:20
*** annegentle has joined #openstack-cinder08:22
*** jwang_ has quit IRC08:22
*** Mandell_ has quit IRC08:24
*** btran has quit IRC08:24
*** jwang_ has joined #openstack-cinder08:25
*** jwang__ has quit IRC08:25
*** lpetrut has joined #openstack-cinder08:27
*** annegentle has quit IRC08:27
openstackgerritRick Chen proposed openstack/cinder: Added the missing attachment to detach_volume  https://review.openstack.org/16826608:32
*** pschaef has joined #openstack-cinder08:34
openstackgerritVincent Hou proposed openstack/cinder: Set volume_attachment to [] for the temporary volume creation  https://review.openstack.org/16713408:35
*** Mandell has joined #openstack-cinder08:40
*** jistr has joined #openstack-cinder08:40
*** markvoelker has joined #openstack-cinder08:41
*** markvoelker has quit IRC08:46
*** annashen has joined #openstack-cinder08:52
*** jordanP has joined #openstack-cinder08:53
*** TobiasE1 has joined #openstack-cinder08:54
*** TobiasE has quit IRC08:56
*** esker has joined #openstack-cinder08:56
*** annashen has quit IRC08:57
*** esker has quit IRC09:01
*** TobiasE1 has quit IRC09:08
*** TobiasE has joined #openstack-cinder09:11
*** pradipta has joined #openstack-cinder09:13
*** pradipta has quit IRC09:14
*** pradipta has joined #openstack-cinder09:14
openstackgerritVincent Hou proposed openstack/cinder: Set volume_attachment to [] for the temporary volume creation  https://review.openstack.org/16713409:21
openstackgerritVincent Hou proposed openstack/cinder: Set volume_attachment to [] for the temporary volume creation  https://review.openstack.org/16713409:24
*** TobiasE has quit IRC09:30
*** sgotliv has quit IRC09:33
*** markvoelker has joined #openstack-cinder09:42
*** ho has quit IRC09:45
*** TobiasE has joined #openstack-cinder09:46
*** markvoelker has quit IRC09:46
*** dims_ has joined #openstack-cinder09:51
*** yuriy_n17 has joined #openstack-cinder09:51
*** esker has joined #openstack-cinder09:57
*** alecv_ has joined #openstack-cinder09:59
*** esker has quit IRC10:02
*** pradipta has quit IRC10:17
*** marcusvrn has joined #openstack-cinder10:17
*** e0ne has joined #openstack-cinder10:18
*** IanGovett has joined #openstack-cinder10:23
*** annegentle has joined #openstack-cinder10:23
*** takedakn has quit IRC10:24
*** haomaiwang has quit IRC10:24
*** IanGovett has quit IRC10:25
*** annegentle has quit IRC10:29
*** Longgeek has quit IRC10:31
*** IanGovett has joined #openstack-cinder10:32
*** marcusvrn has quit IRC10:34
openstackgerrityogeshprasad proposed openstack/cinder: Fixes snapshot creation failure in CloudByte driver.  https://review.openstack.org/16797810:34
*** IanGovett1 has joined #openstack-cinder10:36
*** IanGovett has quit IRC10:39
*** markvoelker has joined #openstack-cinder10:43
*** markvoelker has quit IRC10:48
*** haigang has quit IRC10:48
*** timcl has joined #openstack-cinder10:54
*** esker has joined #openstack-cinder10:58
*** esker has quit IRC11:02
*** Longgeek has joined #openstack-cinder11:08
*** erlon has joined #openstack-cinder11:18
*** BharatK has quit IRC11:19
*** marcusvrn has joined #openstack-cinder11:20
*** marcusvrn1 has joined #openstack-cinder11:23
*** annegentle has joined #openstack-cinder11:24
*** marcusvrn has quit IRC11:25
aarefievdulek:hi11:28
dulekaarefiev: Hi!11:28
aarefievdulek: I have info that you are expert in glance metadata catalog :)11:28
dulekaarefiev: I was part of the team writing it. Shall we switch to #openstack-glance?11:29
*** annegentle has quit IRC11:29
aarefievdulek: bug in cinder https://bugs.launchpad.net/cinder/+bug/132366011:30
openstackLaunchpad bug 1323660 in Cinder "Glance image properties not copied to cinder volume with glance V2 API" [Undecided,Confirmed] - Assigned to Pranali Deore (pranali-deore)11:30
aarefievdulek: problem is v1 glance api contain dict 'properties ' in image, and v2 doesn't11:31
*** ociuhandu has joined #openstack-cinder11:31
dulekaarefiev: It rings a bell, let me dig into it.11:31
dulekaarefiev: Okay, I see, we struggled with this problem two days ago when writing some Horizon code.11:34
aarefievdulek: the properties get with other attributes in same scope, and I try to understand the new concept with glance metadata, and how cinder should work with them11:34
dulekaarefiev: This isn't actually related with metadata catalog at all.11:35
dulekaarefiev: This is how Glance V2 API is implemented.11:35
dulekaarefiev: Actually API _and_ client.11:36
aarefievdulek: so the problem is how we handle it11:36
dulekaarefiev: I think the main problem is that in v2 you're unable to decide which property is metadata and which normal property.11:38
dulekaarefiev: Since these are treated in the same way in response.11:38
*** shilpa_ has quit IRC11:38
dulekaarefiev: I'm actually quite glad that this came out.11:39
dulekaarefiev: I strongly think that this is Glance bug.11:40
dulekaarefiev: We had numerous problems with it when implementing Horizon part of metadata catalog.11:40
aarefievdulek: btw what is the main idea with catalog11:41
dulekaarefiev: And now we have clear use case where current v2 API behavior is a serious problem.11:41
*** Longgeek has quit IRC11:42
dulekaarefiev: It's to create a catalog of metadata you can attach to a resource that _mean_ something.11:42
aarefievdulek: so you think we should report a bug on glance11:43
dulekaarefiev: Did you know for example that you can start a VM with a watchdog on it that will restart it on fail?11:43
dulekaarefiev: This can be achieved by adding a specific extra spec to a flavor11:43
dulekaarefiev: Metadata catalog tries to expose these options to the user.11:43
*** markvoelker has joined #openstack-cinder11:43
dulekaarefiev: I think you should add Glance on this bug and describe the main problem which is:11:44
dulekaarefiev: we cannot determine which attribute is a property and which is a regular attribute11:44
openstackgerritPetrut Lucian proposed openstack/cinder: SMBFS: Fix missing volume provider location  https://review.openstack.org/16830511:45
dulekaarefiev: If you're not feeling confident on this I can drive it.11:45
aarefievdulek: I talked yesterday with glance team, they said that I should get properties keys from image schema11:46
aarefievdulek: and them make decision which attribute is property and which not11:47
aarefievbased on image schema11:48
*** e0ne is now known as e0ne_11:48
*** markvoelker has quit IRC11:48
*** anuragpalsule has quit IRC11:49
*** e0ne_ is now known as e0ne11:49
dulekaarefiev: Yeah, my colleague just showed me your discussion with Alex Tivelkov11:49
dulekaarefiev: Did he know that you need to do that in Cinder?11:50
dulekaarefiev: python-glanceclient doesn't provide you with the methods to get the schema I think, let me check.11:50
aarefievdulek: sure he knows11:50
aarefievdulek: the problem is image schema contain onle core properties11:52
dulekaarefiev: Isn't that actually ok? If a property isn't in schema you consider it custom and pass it to image.11:53
aarefievdulek: the problem is can't understand the concept of properties in glance, the doc with properties http://docs.openstack.org/cli-reference/content/chapter_cli-glance-property.html11:56
aarefievdulek: core properties in image schema "kernel_id" "ramdisk_id" "instance_uuid" "architecture" "os_distro"  "os_version"11:57
dulekaarefiev: Ah, I see.11:58
dulekaarefiev: And these should be passed to volume also?11:58
*** esker has joined #openstack-cinder11:58
aarefievdulek: there are also a lot of json file with namespaces and properties in glance etc/metadefs11:59
dulekaarefiev: Don't mind these files, I don't think that these are relevant.12:00
marcusvrn1hey guys....I'm trying to exclude a test that is failing in my CI (the bugfix has already posted on Gerrit)...but I can't do it... Regex... :(  can someone help with that regex (I'm using devstack-gate)?12:00
dulekaarefiev: Concept of properties was there before metadata catalog.12:00
aarefievdulek: yes, my use case is try update image properties: glance image-update TestVM --property hw_disk_bus=ide --property hw_vif_model=rtl81312:00
aarefievthen create volume from image12:01
aarefievand boot vm from volume12:01
aarefievand it doesn't contain updated properties12:01
aarefievwith glance api v212:02
*** resker has joined #openstack-cinder12:02
dulekaarefiev: That's right, that's because there is a bug you've showed first.12:02
aarefievdulek: yes12:02
*** esker has quit IRC12:04
aarefievdulek: question how properties should get first to volume metadata and then to nova from volume metadata12:04
dulekaarefiev: Just explain me one thing - should for example kernel_id be passed to volume properties as well?12:04
aarefievdulek: good question, I don't know what properties should be in volume image metadata12:06
dulekaarefiev: So let me check... ;)12:07
dulekaarefiev: For Glance v1 API kernel_id and the rest are passed to the image.12:08
*** Adriano_ has joined #openstack-cinder12:08
dulekaarefiev: So I see fundamental problem here - we're unable to distinguish for example kernel_id from name.12:09
dulekaarefiev: So I think that Glance guys need to step in now.12:10
dulekaarefiev: Aww, one mistake - "For Glance v1 API kernel_id and the rest are passed to the image."12:11
duleks/image/volume12:11
aarefievdulek: we have image attributes like name, id, size, etc and properties in one scope, also we have core properties and other properties12:11
aarefiev dulek: what cinder should store and how we fetch it12:12
dulekaarefiev: http://paste.openstack.org/show/197062/ - that's image schema12:13
dulekaarefiev: http://paste.openstack.org/show/197065/12:14
dulekAnd this is what is saved to the volume when using Glance v1 API.12:14
*** bswartz has quit IRC12:14
openstackgerritErlon R. Cruz proposed openstack/cinder: Fix: Boot from image with HNAS iSCSI  https://review.openstack.org/16772712:14
dulekaarefiev: So you would need to pass to volume:12:15
dulekaarefiev: 1. All properties that are not in schema12:15
dulekaarefiev: 2. All properties that are in schema but are marked with is_base=false12:16
dulekaarefiev: 3. Additional ones - size, min_disk, checksum, diskformat, min_ram, container_format, image_name12:18
*** markvoelker has joined #openstack-cinder12:18
*** ociuhandu has quit IRC12:18
dulekaarefiev: Does it make sense?12:18
aarefievdulek: yes it make sense, but I think it isn't good approach12:19
dulekaarefiev: I agree! :)12:19
dulekaarefiev: That's why I mentioned that we should contact Glance guys. This certainly isn't the way to go.12:20
dulekaarefiev: I don't see other way in current status of Glance v2 API.12:20
aarefievdulek: they are think that it is the right way12:21
aarefievdulek: maybe you help bring them over :)12:22
*** annegentle has joined #openstack-cinder12:22
*** rushiagr is now known as rushiagr_away12:23
dulekaarefiev: So we have deadlock here. :D I'll add the bug you mentioned to Glance, write the rationale and ping some cores I know.12:23
dulekaarefiev: Is that okay for you?12:23
*** avishay_ has joined #openstack-cinder12:23
*** annegentle has quit IRC12:23
aarefievdulek: yes thanks for your help, I think the glance api is very difficult, we just need get image property, why it is so hard, I don't understand12:25
dulekaarefiev: They overcomplicated things in v2. And why do you actually want to use v2?12:26
*** rushil has joined #openstack-cinder12:26
aarefievdulek: e.g. ceph use glance api v2 by default12:27
aarefievI think it is related to location which is only in api v212:27
dulekaarefiev: Ah, okay. So this gets important. Can you point me to the place in Cinder code where image metadata are fetched? Should I look in the drivers code?12:28
*** ociuhandu has joined #openstack-cinder12:29
dulekaarefiev: Nevermind, got it.12:30
aarefievdulek: ok :)12:30
*** shilpa_ has joined #openstack-cinder12:32
*** shilpa_ has quit IRC12:35
*** akerr has joined #openstack-cinder12:41
*** Mandell has quit IRC12:46
*** Mandell has joined #openstack-cinder12:47
*** bswartz has joined #openstack-cinder12:49
*** Yogi1 has joined #openstack-cinder12:53
*** pschaef has quit IRC13:02
*** e0ne is now known as e0ne_13:03
*** jaypipes is now known as leakypipes13:05
*** ganso_ has joined #openstack-cinder13:08
*** rushiagr_away is now known as rushiagr13:10
*** e0ne_ is now known as e0ne13:11
*** avishay_ has quit IRC13:12
openstackgerritVincent Hou proposed openstack/cinder: Set volume_attachment to [] for the temporary volume creation  https://review.openstack.org/16713413:16
*** nshaikh has quit IRC13:19
*** dustins has joined #openstack-cinder13:21
*** annegentle has joined #openstack-cinder13:24
*** annegentle has quit IRC13:29
*** ishant has quit IRC13:31
*** Tross has quit IRC13:33
openstackgerritVipin Balachandran proposed openstack/cinder: VMware: Fix ImageNotAuthorized during copy volume  https://review.openstack.org/16833313:33
openstackgerritSean McGinnis proposed openstack/os-brick: Logging not using oslo.i18n guidelines  https://review.openstack.org/16527613:34
openstackgerritVipin Balachandran proposed openstack/cinder: VMware: Fix ImageNotAuthorized during copy volume  https://review.openstack.org/16833313:37
*** tbarron has joined #openstack-cinder13:40
*** annegentle has joined #openstack-cinder13:43
*** TobiasE has left #openstack-cinder13:47
*** dims_ is now known as dimsum__13:49
*** Lee1092 has quit IRC13:50
*** r-daneel has joined #openstack-cinder13:58
*** dimsum__ has quit IRC14:01
*** dimsum__ has joined #openstack-cinder14:02
*** jcru has joined #openstack-cinder14:03
*** nkrinner has quit IRC14:05
dulekaarefiev: I'd tried to explore one more idea, but failed, so I've added Glance to affected projects and written a comment.14:06
dulekaarefiev: I'm pinging ativelkov on openstack-glance so you might want to eavesdrop there. ;)14:07
*** annegentle has quit IRC14:07
aarefievdulek: thanks I will :)14:08
*** _cjones_ has joined #openstack-cinder14:08
*** primechuck has joined #openstack-cinder14:11
*** _cjones_ has quit IRC14:11
*** mtanino has joined #openstack-cinder14:16
*** anuragpalsule has joined #openstack-cinder14:21
*** eharney has joined #openstack-cinder14:21
*** thangp has joined #openstack-cinder14:23
*** thangp has quit IRC14:24
*** thangp has joined #openstack-cinder14:25
*** thangp has quit IRC14:25
*** thangp has joined #openstack-cinder14:25
*** anuragpalsule has quit IRC14:26
kaisersDuncanT: asselin: ping14:28
kaisersDuncanT: asselin: Q regarding thirdparty CI. Our ci is currently hitting this bug:  https://bugs.launchpad.net/tempest/+bug/1437328 . I deactivated it in the morning to sort things out. Now i'm unsure, should i turn it back on (which fill cause false negatives) or keep it offline (meaning nobody sees the CI reporting and possibly getting paddled for that later on) ? :)14:31
openstackLaunchpad bug 1437328 in tempest "No networks found in fixed_network.py (list index out of range)" [High,Triaged] - Assigned to Matthew Treinish (treinish)14:31
kaisersoh, there it is :-D14:32
*** annegentle has joined #openstack-cinder14:32
*** reed has joined #openstack-cinder14:33
*** e0ne is now known as e0ne_14:36
*** ebalduf has joined #openstack-cinder14:40
*** thingee has quit IRC14:43
mtaninoe0ne_: hi,14:43
*** _cjones_ has joined #openstack-cinder14:43
mtaninoe0ne_: https://review.openstack.org/#/c/148038/ I fixed LIO test cases but ISER test cases still inherit tgt test cases.14:44
mtaninojust information.14:44
LiuHow tuoskeep just one test case when run "tox -eall -- volume"14:46
*** tsekiyama has joined #openstack-cinder14:46
Liusorry14:46
Liuhow to just skeep one test case when run "tox -eall -- volume"14:47
*** dansmith is now known as superdan14:47
*** e0ne_ is now known as e0ne14:48
LiuI do it like this "pretty_tox.sh volume.*(?!.*(test_volume_crud_with_volume_type_and_extra_specs))"14:48
Liubut it does not work14:48
e0nemtanino: hi! thank you for the confirmation14:49
hemnafkmornin14:49
*** hemnafk is now known as hemna14:49
e0nehemnafk: evening:)14:49
mtaninoe0ne: But may be no one is not intrested in that decupled of ISER test case...14:50
hemna:)14:50
*** rmesta has joined #openstack-cinder14:51
e0nemtanino: i don't want to believe in it14:52
mtaninooops ok.14:53
*** emagana has joined #openstack-cinder14:54
*** lpabon has joined #openstack-cinder14:56
mtaninoe0ne: Thank you for changing it.14:56
mtaninoI should do it :(14:56
*** _cjones_ has quit IRC14:58
*** _cjones_ has joined #openstack-cinder14:59
hemnawhoa15:02
hemnajust updated master here locally and ran tox -epep815:02
hemnaboatloads of violoations on old untouched code15:02
hemnawth15:02
hemnaH904  Wrap long lines in parentheses instead of a backslash15:02
hemnaendless list of those15:02
*** xyang has joined #openstack-cinder15:02
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Hitachi: Fix access to volume instance_uuid  https://review.openstack.org/16788315:02
e0nehemna: did we merge a new hacking?15:03
hemnaI dunno15:03
hemnacan you update locally and run that ?15:04
hemnasee what you see15:04
*** annegentle has quit IRC15:04
e0nehemna: are you talking about patch #167883 (fix for hitachi)?15:04
hemnawell I saw it running tox on my latest patchset15:05
*** annegentle has joined #openstack-cinder15:05
hemnabut I'm getting H904's on all kinds of files that aren't part of that review15:05
hemnae0ne, http://paste.openstack.org/show/197172/15:05
hemnafor example15:06
hemnathat's a very short copy/paste from the output15:06
hemnamust be hundreds of those15:06
e0ne:(15:06
e0nei'll try it on my env15:06
*** julim has quit IRC15:07
*** julim has joined #openstack-cinder15:08
eharneyhemna: H904 was removed from the current hacking release, probably need to update your env15:10
hemnaI just pulled master15:10
kaisersjgriffith: ping Hello? :)15:10
hemnaand rebased15:10
*** devlaps has joined #openstack-cinder15:10
hemnamaybe my tox virtualenv ?15:10
kaisershemna: tox -r15:10
kaisershemna: i'd guess that's what he means15:10
eharneyhemna: you need hacking >= 0.10 per test-requirements.txt15:10
hemnak I'll try that15:11
hemnathanks15:11
dulekaarefiev: https://review.openstack.org/#/c/168364/ - downloading the schema and checking each attribute using is_base_property should do the trick for Glance v2 support.15:11
e0nehemna: the same error. trying to update hacking to 'hemna'15:11
hemnaheh ok15:11
aarefievdulek: thx I'll check15:12
e0neeharney, hemna: confirmed, that issue was with old 'hacking' package15:13
*** julim has quit IRC15:13
hemnae0ne, ok thanks man.   good to know, I'm rerunning here tox -r15:13
*** Lee1092 has joined #openstack-cinder15:13
*** dannywilson has joined #openstack-cinder15:14
*** julim has joined #openstack-cinder15:14
*** dannywilson has quit IRC15:14
*** dannywilson has joined #openstack-cinder15:14
*** reed has quit IRC15:14
*** MentalRay has joined #openstack-cinder15:17
*** markus_z has quit IRC15:17
*** fifieldt has quit IRC15:21
*** _cjones_ has quit IRC15:24
*** tbarron has quit IRC15:24
*** tbarron has joined #openstack-cinder15:25
*** lpabon has quit IRC15:26
*** rhefner has joined #openstack-cinder15:30
*** jdurgin1 has joined #openstack-cinder15:31
*** primechuck has quit IRC15:39
*** haomaiwang has joined #openstack-cinder15:39
openstackgerritVincent Hou proposed openstack/cinder: Set volume_attachment to [] for the temporary volume creation  https://review.openstack.org/16713415:43
jgriffithkaisers: what up?15:44
*** akrog has joined #openstack-cinder15:48
*** geguileo has quit IRC15:50
*** _cjones_ has joined #openstack-cinder15:50
*** jdurgin1 has quit IRC15:53
*** Tross has joined #openstack-cinder15:53
*** anuragpalsule has joined #openstack-cinder15:54
*** anuragpalsule1 has joined #openstack-cinder15:56
*** anuragpalsule has quit IRC15:58
*** _cjones_ has quit IRC16:01
*** _cjones_ has joined #openstack-cinder16:02
*** annegentle has quit IRC16:06
*** thingee has joined #openstack-cinder16:10
*** leeantho has joined #openstack-cinder16:10
*** btran has joined #openstack-cinder16:13
*** Apoorva has joined #openstack-cinder16:14
*** Miouge has quit IRC16:14
*** dflorea has joined #openstack-cinder16:16
*** mriedem is now known as mriedem_lunch16:17
*** patrickeast has joined #openstack-cinder16:17
jgriffiththingee: you working on that ensure_export retry vishy pointed out this week still?16:20
jgriffiththingee: you said you'd work on it, just thought I'd check if you got tied up and want me to push somethign to fix it16:20
*** e0ne is now known as e0ne_16:20
vishyjgriffith: you mean a backport for it?16:21
jgriffithvishy: yeah16:21
vishyi have a fix for icehouse16:21
vishynot sure if it applies to juno or not but i can send it to you16:21
vishyone sec16:21
jgriffithvishy: oh.. nice16:21
jgriffithvishy: awesome16:21
thingeejgriffith: yeah sorry, I do have it on my list.16:22
jgriffiththingee: sounds like vishy 's already on it :)16:22
vishyjgriffith: http://ix.io/h9r16:22
thingeejgriffith, vishy: ok!16:22
jgriffithvishy: yeah, that'll work in Juno too16:23
vishyi upped it to 3 tries instead of two16:23
vishysince the retry decorator used three as far as i could tell16:23
vishybut i didn’t add the exponential backoff16:23
jgriffithvishy: yup... I think that's ok for the backport IMHO16:23
jgriffithvishy: thingee you guys want me to push these and mark vishy as author?16:23
vishyif you propose it for juno16:24
vishycan you cut out the reference to our bugtracker :)16:24
jgriffithhehe :)16:24
jgriffithI supose16:24
thingeejgriffith: fine with me. let me know review link and I can help push through16:24
*** dulek has quit IRC16:24
thingeevishy: thanks!16:24
jgriffithvishy: FWIW this is the same thing I was going to do anyway.  Wasn't going to try and backport the decorator and lib16:24
jgriffithin other words I don't think backoff etc matters here16:25
thingeejgriffith: that's why I originally asked about backport the decorator. I have some retry stuff I wanted to get in for Datera.16:25
jgriffiththingee: for a backport?16:25
thingeejgriffith: maybe I should change Kilo to not use the decorator and then backport that for juno16:25
thingeejgriffith: to make things simple16:26
jgriffiththingee: I suppose if you prefer I can do that16:26
jgriffiththingee: your call16:26
jgriffiththingee: it's not *hard*16:26
*** e0ne_ is now known as e0ne16:26
*** e0ne is now known as e0ne_16:27
*** e0ne_ is now known as e0ne16:27
thingeejgriffith: yeah that's originally what I was going to do.16:27
thingeejgriffith: if you don't mind, otherwise this fix shouldn't wait on my greedy reasons.16:27
jgriffiththingee: ok, I'll give it a whirl :)16:27
thingee;)16:28
openstackgerritTomoki Sekiyama proposed openstack/cinder: Fix debug log parameter mismatch  https://review.openstack.org/16813716:37
*** alecv_ has quit IRC16:38
*** ronis has quit IRC16:38
openstackgerritTomoki Sekiyama proposed openstack/cinder: Fix debug log parameter mismatch  https://review.openstack.org/16813716:39
*** afazekas has quit IRC16:40
*** esker has joined #openstack-cinder16:41
*** resker has quit IRC16:41
*** annashen has joined #openstack-cinder16:44
*** MentalRay has quit IRC16:46
*** alexpilotti has joined #openstack-cinder16:50
*** vilobhmm1 has joined #openstack-cinder16:50
*** vilobhmm11 has joined #openstack-cinder16:52
*** deepakcs has joined #openstack-cinder16:53
alexpilottithingee: hello16:53
jgriffiththingee: hmm... seems our unit tests in Juno are broken16:54
alexpilottithingee: we have the SMB3 linux and Windows CIs commenting16:54
*** vilobhmm1 has quit IRC16:55
alexpilottithingee: was wondering if by any chance we could revert the SMB3 driver removal patch :-)16:55
*** jogo is now known as flashgordon16:57
*** dflorea has quit IRC17:01
*** haomaiw__ has joined #openstack-cinder17:02
*** haomaiwang has quit IRC17:04
*** dflorea has joined #openstack-cinder17:04
*** annegentle has joined #openstack-cinder17:04
*** anuragpalsule1 has quit IRC17:04
*** marcusvrn has joined #openstack-cinder17:07
*** _cjones_ has quit IRC17:08
*** lpetrut has left #openstack-cinder17:11
*** marcusvrn1 has quit IRC17:11
*** jistr has quit IRC17:13
*** Tross has quit IRC17:14
jgriffiththingee: http://goo.gl/JriLF517:15
ameadethingee: I've got our FC CI putting up comments: https://review.openstack.org/#/q/status:open+project:openstack/cinder+reviewer:netapp-fc-ci,n,z17:17
*** nshaikh has joined #openstack-cinder17:18
ameadethingee: let me know if there is anything to tweak, it's running 303 tests because the boto test class is being skipped because ec2 is not turned on17:18
ameadethingee: tried to mimic the tempest.confs of you and jgriffith17:18
*** rushil has quit IRC17:19
*** Yogi1 has quit IRC17:19
*** deepakcs has quit IRC17:21
ameadethingee: do you know a way i can do a filter for the latest patchset only?17:23
*** ndipanov has quit IRC17:25
*** e0ne is now known as e0ne_17:26
*** nshaikh has quit IRC17:27
*** lpetrut has joined #openstack-cinder17:28
*** e0ne_ is now known as e0ne17:28
*** mriedem_lunch is now known as mriedem17:30
*** cebruns has quit IRC17:30
*** dflorea has quit IRC17:33
*** dflorea has joined #openstack-cinder17:34
*** dflorea has quit IRC17:34
*** dflorea has joined #openstack-cinder17:35
*** e0ne is now known as e0ne_17:39
*** haomaiwa_ has joined #openstack-cinder17:40
*** haomaiw__ has quit IRC17:40
*** Lee1092 has quit IRC17:40
*** Yogi1 has joined #openstack-cinder17:40
*** Yogi11 has joined #openstack-cinder17:43
*** _cjones_ has joined #openstack-cinder17:44
*** Yogi1 has quit IRC17:45
*** jordanP has quit IRC17:45
*** e0ne_ has quit IRC17:50
*** haomaiwa_ has quit IRC17:53
*** ronis has joined #openstack-cinder17:55
*** haomaiwang has joined #openstack-cinder17:55
*** dflorea has quit IRC17:55
*** annashen has quit IRC17:55
*** dflorea has joined #openstack-cinder17:56
jgriffithasselin: ping17:57
openstackgerritVilobh Meshram proposed openstack/cinder: ConsistencyGroup Object  https://review.openstack.org/16364117:57
jgriffithasselin: ahh... never mind... found it18:00
*** timcl1 has joined #openstack-cinder18:00
*** casusbelli has joined #openstack-cinder18:00
*** e0ne has joined #openstack-cinder18:00
*** dflorea has quit IRC18:01
*** timcl has quit IRC18:02
*** timcl has joined #openstack-cinder18:03
*** e0ne has quit IRC18:03
*** timcl1 has quit IRC18:04
*** dflorea has joined #openstack-cinder18:04
*** haomaiwang has quit IRC18:06
*** dflorea has quit IRC18:12
*** dflorea has joined #openstack-cinder18:12
*** dflorea has quit IRC18:17
*** avishay_ has joined #openstack-cinder18:20
*** MentalRay has joined #openstack-cinder18:20
*** yuriy_n17 has quit IRC18:22
*** avishay_ has quit IRC18:22
*** avishay_ has joined #openstack-cinder18:22
*** afazekas has joined #openstack-cinder18:23
*** esker has quit IRC18:26
*** esker has joined #openstack-cinder18:27
*** dflorea has joined #openstack-cinder18:28
*** tsekiyama has quit IRC18:30
*** lpetrut has left #openstack-cinder18:31
*** tsekiyama has joined #openstack-cinder18:32
rluciohi guys would anyone mind reviewing my driver bug fixes -- they are needed for ci stability https://review.openstack.org/#/c/165763/ https://review.openstack.org/#/c/165655/18:32
rlucio(for my CI that is)18:32
*** dflorea has quit IRC18:34
*** annashen has joined #openstack-cinder18:35
*** e0ne has joined #openstack-cinder18:35
*** dflorea has joined #openstack-cinder18:35
*** thingee has quit IRC18:35
*** afazekas has quit IRC18:37
*** dflorea has quit IRC18:39
*** annegentle has quit IRC18:40
*** annegentle has joined #openstack-cinder18:41
*** Yogi11 has quit IRC18:46
*** thingee has joined #openstack-cinder18:48
*** med_ has joined #openstack-cinder18:49
*** med_ has quit IRC18:49
*** med_ has joined #openstack-cinder18:49
*** nshaikh has joined #openstack-cinder18:49
*** e0ne is now known as e0ne_18:50
rlucioasselin: another gotcha on logservers, i changed servers and reconfigured everything OK but zuul is still posting links with old IP addr in reviews18:51
rlucioasselin: files are being uploaded to the new IP though18:51
rlucioasselin: i'm guessing some zuul config item needs to be updated, but I can't find it18:52
asselinrlucio, double-check the /etc/zuul/zuul.conf18:53
asselinrlucio, url_pattern=18:53
asselinrlucio, if it's correct, probably you need to restart zuul18:53
rlucioasselin: it shows the new url there18:54
thingeejgriffith: back, sorry was in a meeting18:54
thingeejgriffith: checking18:54
rlucioasselin: i swear i restarted it already, but i'll try again18:54
thingeealexpilotti: see http://lists.openstack.org/pipermail/openstack-dev/2015-March/059990.html18:55
*** e0ne_ has quit IRC18:55
*** e0ne has joined #openstack-cinder18:58
*** lpetrut has joined #openstack-cinder18:58
thingeeameade: thanks19:00
thingeeameade: a way to filter the latest patch set?19:00
alexpilottithingee: about the tests, the SMB3 driver does not support volume snapshots yet, so we cannot run all 304 tests19:00
ameadethingee: with a gerrit link19:00
alexpilottithingee: specifically snapshots created with volume in use19:01
*** e0ne is now known as e0ne_19:02
*** annashen has quit IRC19:03
*** Apoorva has quit IRC19:05
*** e0ne_ is now known as e0ne19:07
*** alexpilotti has quit IRC19:07
thingeealexpilotti: those are required features for Cinder http://docs.openstack.org/developer/cinder/devref/drivers.html#minimum-features19:08
thingeecinder volume drivers*19:08
*** emagana has quit IRC19:08
thingeeameade: I see, I'm not aware, especially because these are all non-voting.19:09
*** nshaikh has left #openstack-cinder19:09
ameadethingee: yeah, nbd...does that all look good to you? it seems to be running great19:09
ameadethingee: i'm running with concurrency 1 because I think i found a bug in the brocade FC driver19:11
ameadei'll dig into that later though19:11
thingeeameade: yeah glanced at one of the logs, looks good to me.19:12
ameadethingee: cool cool, thanks19:13
openstackgerritTomoki Sekiyama proposed openstack/cinder: Log original backtrace on generic Exception in create_volume flow  https://review.openstack.org/16845919:13
thingeeameade: I think asselin has been talking to folks about including the test_results.html.gz file, which in devstack-gate I think is generated like so https://github.com/openstack-infra/devstack-gate/blob/master/functions.sh#L53519:14
*** melwitt has joined #openstack-cinder19:14
thingeeameade: it is mentioned in http://ci.openstack.org/third_party.html#requirements ... but I'm not going to ding people on that in Kilo...I think we'll some instructions added to the wiki first.19:15
thingeewe'll need some*19:15
ameadeah kk19:15
thingeeameade: I'll email the list and individuals as usual whenever we get to that point.19:17
ameadethingee: sounds good19:17
*** afazekas has joined #openstack-cinder19:24
anish23:15 < thingee> anish: I've been seeing 'recheck <ci-name>'19:25
anishthingee: ^19:25
anishI'm 100% certain I saw that cause a jenkins recheck with https://review.openstack.org/#/c/165143/19:25
*** IanGovett1 has quit IRC19:25
patrickeastanish: jenkins will look for any string that starts with recheck or reverify19:27
*** IanGovett has joined #openstack-cinder19:27
anishpatrickeast: right, which means it will recheck anytime someone asks for a third party ci recheck as well19:28
anishwhich seems wasteful19:28
patrickeastyep19:28
*** timcl has quit IRC19:28
*** annashen has joined #openstack-cinder19:29
anishwell, as long as it is a known issue19:29
patrickeastanish: i think it was a decision by the infra team to make it easier to recheck patches, iirc it used to be much more strict about formatting19:30
anishpatrickeast: aah, thanks for the info.19:31
*** afazekas has quit IRC19:32
*** xyang has quit IRC19:33
*** timcl has joined #openstack-cinder19:36
*** devlaps has quit IRC19:36
*** angela-s has joined #openstack-cinder19:39
*** xyang has joined #openstack-cinder19:39
*** avishay_ has quit IRC19:40
*** dustins has quit IRC19:41
*** emagana has joined #openstack-cinder19:41
*** leakypipes has quit IRC19:42
*** emagana has quit IRC19:46
*** lpetrut has quit IRC19:48
*** lpetrut has joined #openstack-cinder19:50
*** rushiagr is now known as rushiagr_away19:52
*** mwichmann has left #openstack-cinder19:53
jgriffithhemna: you're workign on this: http://goo.gl/2tJdlt19:53
*** rhefner has quit IRC19:54
*** MentalRay_ has joined #openstack-cinder19:59
*** e0ne is now known as e0ne_20:00
*** e0ne_ is now known as e0ne20:00
*** emagana has joined #openstack-cinder20:01
*** MentalRay has quit IRC20:02
*** timcl has quit IRC20:03
hemnajgriffith, yes20:03
jgriffithhemna: cool20:04
* jgriffith moves along20:04
*** Apoorva has joined #openstack-cinder20:05
*** dflorea has joined #openstack-cinder20:06
*** afazekas has joined #openstack-cinder20:07
*** akerr has quit IRC20:15
*** ociuhandu has quit IRC20:16
*** emagana has quit IRC20:19
*** jwang__ has joined #openstack-cinder20:24
*** annashen_ has joined #openstack-cinder20:24
*** annashen has quit IRC20:24
*** jwang_ has quit IRC20:28
openstackgerritTomoki Sekiyama proposed openstack/cinder: Fix volume creation from image with allowed_direct_url_schemes  https://review.openstack.org/16847720:32
*** ebalduf has quit IRC20:33
*** thangp has quit IRC20:36
*** e0ne is now known as e0ne_20:40
*** e0ne has joined #openstack-cinder20:41
*** afazekas has quit IRC20:47
*** dflorea has quit IRC20:47
*** dflorea has joined #openstack-cinder20:48
*** emagana has joined #openstack-cinder20:50
*** dflorea has quit IRC20:52
*** emagana has quit IRC20:54
*** ebalduf has joined #openstack-cinder20:55
*** mfisch has quit IRC20:56
*** mfisch has joined #openstack-cinder20:57
*** mfisch is now known as Guest9095720:57
*** e0ne is now known as e0ne_20:57
*** e0ne_ is now known as e0ne20:58
-openstackstatus- NOTICE: Gerrit maintenance commences in 1 hour at 22:00 UTC http://lists.openstack.org/pipermail/openstack-dev/2015-March/059948.html21:00
*** lcurtis has joined #openstack-cinder21:01
*** tbarron has quit IRC21:01
*** sgotliv has joined #openstack-cinder21:02
*** annashen_ has quit IRC21:08
*** akerr has joined #openstack-cinder21:09
openstackgerritxing-yang proposed openstack/cinder: Fix the format of the system name in VMAX driver  https://review.openstack.org/16848721:10
*** annegentle has quit IRC21:10
*** akerr_ has joined #openstack-cinder21:10
*** marcusvrn has quit IRC21:12
*** akerr has quit IRC21:13
*** dflorea has joined #openstack-cinder21:14
*** Adriano_ has quit IRC21:16
*** Apoorva has quit IRC21:16
*** e0ne has quit IRC21:18
*** MentalRay_ has quit IRC21:20
*** MentalRay_ has joined #openstack-cinder21:24
*** Tross has joined #openstack-cinder21:29
*** xyang has quit IRC21:41
*** xyang has joined #openstack-cinder21:41
*** marcusvrn has joined #openstack-cinder21:42
thingeeanish: sounds like we should talk to infra then21:43
*** tsekiyam_ has joined #openstack-cinder21:43
*** lpetrut has quit IRC21:46
*** tsekiyama has quit IRC21:46
*** tsekiyam_ has quit IRC21:47
*** xyang has quit IRC21:49
*** dflorea has quit IRC21:50
*** dflorea has joined #openstack-cinder21:51
anishthingee: yep. I was also trying to check if recheck-foo works similar to recheck foo, but I can't remember which CIs were using that format21:53
*** vilobhmm11 has quit IRC21:53
*** dflorea has quit IRC21:55
*** ociuhandu has joined #openstack-cinder21:55
*** mriedem has quit IRC21:56
*** ebalduf has quit IRC22:02
-openstackstatus- NOTICE: Gerrit is offline for maintenance, ETA 22:30 UTC http://lists.openstack.org/pipermail/openstack-dev/2015-March/059948.html22:03
*** ChanServ changes topic to "Gerrit is offline for maintenance, ETA 22:30 UTC http://lists.openstack.org/pipermail/openstack-dev/2015-March/059948.html"22:03
*** MentalRay_ has quit IRC22:03
*** dflorea has joined #openstack-cinder22:04
*** annegentle has joined #openstack-cinder22:04
*** akerr_ has quit IRC22:06
*** annashen has joined #openstack-cinder22:09
*** dimsum__ has quit IRC22:09
*** vilobhmm1 has joined #openstack-cinder22:10
*** vilobhmm1 has quit IRC22:10
*** vilobhmm1 has joined #openstack-cinder22:10
*** vilobhmm11 has joined #openstack-cinder22:13
*** annashen has quit IRC22:14
*** vilobhmm1 has quit IRC22:15
*** MentalRay_ has joined #openstack-cinder22:18
*** vilobhmm11 has quit IRC22:19
*** vilobhmm1 has joined #openstack-cinder22:20
*** dimsum__ has joined #openstack-cinder22:20
*** mdenny has quit IRC22:22
*** markstur has joined #openstack-cinder22:24
*** markstur is now known as markstur_away22:25
*** markstur_away has left #openstack-cinder22:26
*** ganso_ has quit IRC22:27
*** ronis has quit IRC22:30
*** erlon has quit IRC22:31
*** ChanServ changes topic to "The OpenStack Block Storage Project Cinder | The New Kids On the Block | https://wiki.openstack.org/wiki/Cinder"22:32
*** bswartz has quit IRC22:34
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Brick: Fix race in removing iSCSI device  https://review.openstack.org/16850522:37
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: Brick: Fix race in removing iSCSI device  https://review.openstack.org/16850622:38
hemnaphew22:38
patrickeasthemna: oo you got a fix for it?22:40
*** markvoelker has quit IRC22:40
hemnapatrickeast, yah I think so22:41
*** annashen has joined #openstack-cinder22:41
hemnawe have a Helion tester that is seeing this happen on a crappy old system22:41
patrickeasthaha22:41
hemnabasically the race happens between the time we tell the kernel to remove the volume22:41
hemnaand when we look for the volumes in /dev/disk/by-path22:42
hemnait wasn't going away22:42
hemnaand the iSCSI session was never logged out22:42
hemnaso I added a wait verification loop22:42
patrickeasthemna: yea i had a bug fix for this in juno, but i had always wondered if there was still an issue before the requests even got to udev to make the symlinks invalid22:42
hemnapython was running too quickly :)22:42
patrickeastdont hear that every day!22:43
hemnakernel was too busy with io to remove the volume from the scsi subsystem in time, to make it go away in /dev/disk/by-path22:43
hemnaPITA22:43
patrickeastyea, its nice that these things are async commands… except times like this22:43
hemnaI need to do some refactoring of some of the code in brick to make this available for all transports22:43
hemnabut this fixes the iSCSI case for now.22:43
hemnaI can't wait to get brick out of Cinder, so I can do some house cleaning of the connectors22:44
patrickeastyea its going to be nice22:44
patrickeastesp when nova is using it too22:44
patrickeastthe differences in the attach code give me a lot of heartburn22:45
*** patrickeast has quit IRC22:46
hemnayah, it gives me the willies right now22:46
*** sneezewort has quit IRC22:47
*** thingee has quit IRC22:49
hemnaouch22:52
hemnaLooks like Huawei's CI is broken.22:52
hemna"15:38:34 Caused by: java.io.IOException: No space left on device"22:52
*** anshul has quit IRC22:57
*** annashen has quit IRC23:00
*** annashen has joined #openstack-cinder23:12
*** annegentle has quit IRC23:19
*** _cjones_ has quit IRC23:21
*** MentalRay__ has joined #openstack-cinder23:21
*** MentalRay_ has quit IRC23:24
*** jcru has quit IRC23:27
*** cebruns has joined #openstack-cinder23:33
*** bswartz has joined #openstack-cinder23:33
*** IanGovett has quit IRC23:34
*** rmesta has quit IRC23:40
*** MentalRay__ has quit IRC23:40
*** markvoelker has joined #openstack-cinder23:41
*** markvoelker has quit IRC23:46
*** lcurtis has quit IRC23:47
*** IanGovett has joined #openstack-cinder23:49
*** btran has quit IRC23:50
*** ociuhandu has quit IRC23:52
*** dannywilson has quit IRC23:53
*** annashen has quit IRC23:54
*** dimsum__ has quit IRC23:59

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