*** dannywil_ has quit IRC | 00:02 | |
*** ho has joined #openstack-cinder | 00:02 | |
*** markvoelker has quit IRC | 00:02 | |
*** annashen has joined #openstack-cinder | 00:03 | |
*** annegentle has joined #openstack-cinder | 00:06 | |
*** dflorea_ has quit IRC | 00:10 | |
jgriffith | thingee: you ok with merging this: https://review.openstack.org/#/c/160104/21 | 00:11 |
---|---|---|
*** dflorea has joined #openstack-cinder | 00:11 | |
* thingee checks | 00:12 | |
thingee | jgriffith: are you asking because of the late schema change? | 00:14 |
jgriffith | thingee: yeah | 00:14 |
jgriffith | thingee: I think it's important to fix... but wanted your input and final decision | 00:14 |
*** Longgeek has quit IRC | 00:15 | |
thingee | jgriffith: yeah if it's important to fix, it seems like a small scope IMO | 00:15 |
thingee | so seems fine | 00:15 |
*** dflorea has quit IRC | 00:15 | |
jgriffith | thingee: I'd agree. I guess *important* is subjective :) but you know what I mean | 00:16 |
jgriffith | thingee: speaking of which... do we have any interest in *finally* doing a migration squash? | 00:17 |
jgriffith | thingee: I've never bothered, wonder if we should consider it | 00:17 |
thingee | that'd be interesting | 00:18 |
jgriffith | thingee: hehe... good choice of words :) | 00:18 |
*** Apoorva has quit IRC | 00:18 | |
jgriffith | thingee: we should think about it... that and timing on saving some slots for backports blah blah blah :) | 00:19 |
jgriffith | thingee: but in the meantime I think I'm going to head home :) | 00:19 |
* jgriffith has done enough damage for one day | 00:19 | |
*** tsekiyam_ has joined #openstack-cinder | 00:21 | |
*** mtanino has quit IRC | 00:24 | |
*** tsekiyama has quit IRC | 00:25 | |
*** tsekiyam_ has quit IRC | 00:26 | |
*** reed has quit IRC | 00:29 | |
*** reed has joined #openstack-cinder | 00:30 | |
*** reed has quit IRC | 00:30 | |
*** reed has joined #openstack-cinder | 00:31 | |
*** leeantho has quit IRC | 00:38 | |
*** annashen has quit IRC | 00:45 | |
*** rhsu has joined #openstack-cinder | 01:00 | |
*** esker has quit IRC | 01:02 | |
*** esker has joined #openstack-cinder | 01:03 | |
*** Longgeek has joined #openstack-cinder | 01:03 | |
*** resker has joined #openstack-cinder | 01:04 | |
*** esker has quit IRC | 01:08 | |
*** marcusvrn has joined #openstack-cinder | 01:09 | |
*** annegentle has quit IRC | 01:14 | |
*** esker has joined #openstack-cinder | 01:15 | |
*** annegentle has joined #openstack-cinder | 01:17 | |
*** resker has quit IRC | 01:17 | |
*** resker has joined #openstack-cinder | 01:19 | |
*** vilobhmm1 has quit IRC | 01:19 | |
*** Apoorva has joined #openstack-cinder | 01:19 | |
*** esker has quit IRC | 01:21 | |
*** dims_ has quit IRC | 01:23 | |
*** Apoorva has quit IRC | 01:24 | |
*** Mandell has quit IRC | 01:26 | |
*** _cjones_ has quit IRC | 01:30 | |
*** Lee1092 has joined #openstack-cinder | 01:31 | |
*** _cjones_ has joined #openstack-cinder | 01:31 | |
*** rhsu has quit IRC | 01:32 | |
*** _cjones_ has quit IRC | 01:35 | |
*** Apoorva has joined #openstack-cinder | 01:38 | |
*** primechuck has joined #openstack-cinder | 01:40 | |
*** annegentle has quit IRC | 01:48 | |
*** melwitt has quit IRC | 01:48 | |
*** happyeveryday has joined #openstack-cinder | 01:50 | |
*** takedakn has joined #openstack-cinder | 01:51 | |
*** Longgeek has quit IRC | 01:52 | |
*** Longgeek has joined #openstack-cinder | 01:53 | |
*** happyeveryday has quit IRC | 01:58 | |
*** haigang has joined #openstack-cinder | 02:00 | |
*** Apoorva has quit IRC | 02:10 | |
*** haomaiwang has joined #openstack-cinder | 02:14 | |
*** anshul has joined #openstack-cinder | 02:20 | |
*** takedakn has quit IRC | 02:21 | |
*** kaisers has joined #openstack-cinder | 02:22 | |
*** kaisers1 has quit IRC | 02:23 | |
*** patrickeast has quit IRC | 02:25 | |
*** vilobhmm1 has joined #openstack-cinder | 02:28 | |
*** haigang has quit IRC | 02:29 | |
*** haigang has joined #openstack-cinder | 02:30 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Set volume_attachment to [] for the temporary volume creation https://review.openstack.org/167134 | 02:35 |
*** scottda_ has joined #openstack-cinder | 02:37 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Set volume_attachment to [] for the temporary volume creation https://review.openstack.org/167134 | 02:45 |
*** annashen has joined #openstack-cinder | 02:48 | |
*** annegentle has joined #openstack-cinder | 02:48 | |
*** annashen has quit IRC | 02:52 | |
*** annegentle has quit IRC | 02:54 | |
*** devlaps has quit IRC | 02:54 | |
*** primechuck has quit IRC | 03:15 | |
*** emagana has joined #openstack-cinder | 03:18 | |
*** liusheng has joined #openstack-cinder | 03:21 | |
*** Longgeek has quit IRC | 03:34 | |
*** rushiagr_away is now known as rushiagr | 03:46 | |
*** scottda_ has quit IRC | 03:47 | |
*** _cjones_ has joined #openstack-cinder | 03:48 | |
*** _cjones_ has quit IRC | 03:49 | |
*** annegentle has joined #openstack-cinder | 03:50 | |
*** emagana has quit IRC | 03:51 | |
*** resker has quit IRC | 03:52 | |
*** emagana has joined #openstack-cinder | 03:52 | |
*** annegentle has quit IRC | 03:55 | |
*** emagana has quit IRC | 03:56 | |
*** _cjones_ has joined #openstack-cinder | 03:58 | |
*** _cjones_ has quit IRC | 04:02 | |
*** _cjones_ has joined #openstack-cinder | 04:02 | |
*** marcusvrn has quit IRC | 04:03 | |
*** Mandell has joined #openstack-cinder | 04:05 | |
*** Longgeek has joined #openstack-cinder | 04:08 | |
*** annegentle has joined #openstack-cinder | 04:08 | |
*** annegentle has quit IRC | 04:13 | |
*** _cjones_ has quit IRC | 04:33 | |
anish | thingee: shouldn't jenkins be ignoring rechecks meant for 3rd party ci's ? | 04:33 |
*** markvoelker has joined #openstack-cinder | 04:39 | |
*** MentalRay has joined #openstack-cinder | 04:40 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Set volume_attachment to [] for the temporary volume creation https://review.openstack.org/167134 | 04:41 |
*** junhongl has quit IRC | 04:41 | |
*** sneezewort has quit IRC | 04:42 | |
*** markvoelker has quit IRC | 04:43 | |
*** annashen has joined #openstack-cinder | 04:49 | |
*** esker has joined #openstack-cinder | 04:53 | |
*** sneezewort has joined #openstack-cinder | 04:53 | |
*** junhongl has joined #openstack-cinder | 04:53 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Set volume_attachment to [] for the temporary volume creation https://review.openstack.org/167134 | 04:54 |
*** annashen has quit IRC | 04:54 | |
*** esker has quit IRC | 04:57 | |
*** reed has quit IRC | 05:03 | |
*** rushiagr is now known as rushiagr_away | 05:04 | |
*** lpetrut has joined #openstack-cinder | 05:13 | |
*** annegentle has joined #openstack-cinder | 05:19 | |
openstackgerrit | Peter Wang proposed openstack/cinder: Enhance VNX Cinder volume creation logic https://review.openstack.org/167573 | 05:19 |
*** enterprisedc has quit IRC | 05:23 | |
*** annegentle has quit IRC | 05:24 | |
*** markvoelker has joined #openstack-cinder | 05:39 | |
*** nkrinner has joined #openstack-cinder | 05:41 | |
*** anuragpalsule has joined #openstack-cinder | 05:41 | |
*** jamielennox is now known as jamielennox|away | 05:41 | |
*** nshaikh has joined #openstack-cinder | 05:42 | |
*** markvoelker has quit IRC | 05:43 | |
*** rhefner has quit IRC | 05:44 | |
*** lpetrut has quit IRC | 05:45 | |
*** lpetrut has joined #openstack-cinder | 05:46 | |
*** dims has joined #openstack-cinder | 05:46 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Set volume_attachment to [] for the temporary volume creation https://review.openstack.org/167134 | 05:50 |
*** shilpa_ has joined #openstack-cinder | 05:50 | |
*** anshul has quit IRC | 05:51 | |
*** anshul has joined #openstack-cinder | 05:52 | |
*** esker has joined #openstack-cinder | 05:53 | |
*** esker has quit IRC | 05:58 | |
*** vilobhmm1 has quit IRC | 06:06 | |
*** vilobhmm1 has joined #openstack-cinder | 06:06 | |
*** rushiagr_away is now known as rushiagr | 06:09 | |
thingee | anish: it should yes | 06:14 |
thingee | anish: I've been seeing 'recheck <ci-name>' | 06:15 |
thingee | and jenkins not responding to that | 06:15 |
thingee | anish: here's an example https://review.openstack.org/#/c/166079/ | 06:15 |
*** BharatK has joined #openstack-cinder | 06:19 | |
*** annegentle has joined #openstack-cinder | 06:20 | |
*** dims has quit IRC | 06:21 | |
*** annegentle has quit IRC | 06:25 | |
*** afazekas is now known as __afazekas | 06:28 | |
*** ishant has joined #openstack-cinder | 06:36 | |
*** MentalRay has quit IRC | 06:40 | |
*** markvoelker has joined #openstack-cinder | 06:40 | |
*** markvoelker has quit IRC | 06:44 | |
*** Longgeek has quit IRC | 06:50 | |
*** Longgeek has joined #openstack-cinder | 06:53 | |
*** esker has joined #openstack-cinder | 06:55 | |
*** pradipta has joined #openstack-cinder | 06:55 | |
*** esker has quit IRC | 06:59 | |
*** vilobhmm1 has quit IRC | 07:00 | |
openstackgerrit | Tushar Patil proposed openstack/python-cinderclient: Allow cinderclient to handle exception response https://review.openstack.org/166351 | 07:10 |
*** afazekas has joined #openstack-cinder | 07:13 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Set volume_attachment to [] for the temporary volume creation https://review.openstack.org/167134 | 07:15 |
openstackgerrit | wuyuting proposed openstack/cinder: Make backup-list support all-tenants options https://review.openstack.org/166727 | 07:16 |
*** chlong has quit IRC | 07:16 | |
*** TobiasE has joined #openstack-cinder | 07:17 | |
*** Miouge has joined #openstack-cinder | 07:26 | |
*** angela-s has quit IRC | 07:32 | |
openstackgerrit | Tushar Patil proposed openstack/python-cinderclient: Allow cinderclient to handle exception response https://review.openstack.org/166351 | 07:40 |
*** markvoelker has joined #openstack-cinder | 07:40 | |
*** lpetrut has quit IRC | 07:45 | |
*** markvoelker has quit IRC | 07:45 | |
*** takedakn has joined #openstack-cinder | 07:47 | |
*** takedakn has quit IRC | 07:52 | |
*** esker has joined #openstack-cinder | 07:55 | |
*** esker has quit IRC | 08:00 | |
*** ndipanov has joined #openstack-cinder | 08:00 | |
*** takedakn has joined #openstack-cinder | 08:00 | |
*** dulek has joined #openstack-cinder | 08:04 | |
*** Mandell_ has joined #openstack-cinder | 08:05 | |
*** pradipta has quit IRC | 08:06 | |
*** Mandell has quit IRC | 08:08 | |
*** markus_z has joined #openstack-cinder | 08:09 | |
*** lpetrut has joined #openstack-cinder | 08:13 | |
*** lpetrut has quit IRC | 08:15 | |
*** ronis has joined #openstack-cinder | 08:16 | |
*** jwang__ has joined #openstack-cinder | 08:20 | |
*** annegentle has joined #openstack-cinder | 08:22 | |
*** jwang_ has quit IRC | 08:22 | |
*** Mandell_ has quit IRC | 08:24 | |
*** btran has quit IRC | 08:24 | |
*** jwang_ has joined #openstack-cinder | 08:25 | |
*** jwang__ has quit IRC | 08:25 | |
*** lpetrut has joined #openstack-cinder | 08:27 | |
*** annegentle has quit IRC | 08:27 | |
openstackgerrit | Rick Chen proposed openstack/cinder: Added the missing attachment to detach_volume https://review.openstack.org/168266 | 08:32 |
*** pschaef has joined #openstack-cinder | 08:34 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Set volume_attachment to [] for the temporary volume creation https://review.openstack.org/167134 | 08:35 |
*** Mandell has joined #openstack-cinder | 08:40 | |
*** jistr has joined #openstack-cinder | 08:40 | |
*** markvoelker has joined #openstack-cinder | 08:41 | |
*** markvoelker has quit IRC | 08:46 | |
*** annashen has joined #openstack-cinder | 08:52 | |
*** jordanP has joined #openstack-cinder | 08:53 | |
*** TobiasE1 has joined #openstack-cinder | 08:54 | |
*** TobiasE has quit IRC | 08:56 | |
*** esker has joined #openstack-cinder | 08:56 | |
*** annashen has quit IRC | 08:57 | |
*** esker has quit IRC | 09:01 | |
*** TobiasE1 has quit IRC | 09:08 | |
*** TobiasE has joined #openstack-cinder | 09:11 | |
*** pradipta has joined #openstack-cinder | 09:13 | |
*** pradipta has quit IRC | 09:14 | |
*** pradipta has joined #openstack-cinder | 09:14 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Set volume_attachment to [] for the temporary volume creation https://review.openstack.org/167134 | 09:21 |
openstackgerrit | Vincent Hou proposed openstack/cinder: Set volume_attachment to [] for the temporary volume creation https://review.openstack.org/167134 | 09:24 |
*** TobiasE has quit IRC | 09:30 | |
*** sgotliv has quit IRC | 09:33 | |
*** markvoelker has joined #openstack-cinder | 09:42 | |
*** ho has quit IRC | 09:45 | |
*** TobiasE has joined #openstack-cinder | 09:46 | |
*** markvoelker has quit IRC | 09:46 | |
*** dims_ has joined #openstack-cinder | 09:51 | |
*** yuriy_n17 has joined #openstack-cinder | 09:51 | |
*** esker has joined #openstack-cinder | 09:57 | |
*** alecv_ has joined #openstack-cinder | 09:59 | |
*** esker has quit IRC | 10:02 | |
*** pradipta has quit IRC | 10:17 | |
*** marcusvrn has joined #openstack-cinder | 10:17 | |
*** e0ne has joined #openstack-cinder | 10:18 | |
*** IanGovett has joined #openstack-cinder | 10:23 | |
*** annegentle has joined #openstack-cinder | 10:23 | |
*** takedakn has quit IRC | 10:24 | |
*** haomaiwang has quit IRC | 10:24 | |
*** IanGovett has quit IRC | 10:25 | |
*** annegentle has quit IRC | 10:29 | |
*** Longgeek has quit IRC | 10:31 | |
*** IanGovett has joined #openstack-cinder | 10:32 | |
*** marcusvrn has quit IRC | 10:34 | |
openstackgerrit | yogeshprasad proposed openstack/cinder: Fixes snapshot creation failure in CloudByte driver. https://review.openstack.org/167978 | 10:34 |
*** IanGovett1 has joined #openstack-cinder | 10:36 | |
*** IanGovett has quit IRC | 10:39 | |
*** markvoelker has joined #openstack-cinder | 10:43 | |
*** markvoelker has quit IRC | 10:48 | |
*** haigang has quit IRC | 10:48 | |
*** timcl has joined #openstack-cinder | 10:54 | |
*** esker has joined #openstack-cinder | 10:58 | |
*** esker has quit IRC | 11:02 | |
*** Longgeek has joined #openstack-cinder | 11:08 | |
*** erlon has joined #openstack-cinder | 11:18 | |
*** BharatK has quit IRC | 11:19 | |
*** marcusvrn has joined #openstack-cinder | 11:20 | |
*** marcusvrn1 has joined #openstack-cinder | 11:23 | |
*** annegentle has joined #openstack-cinder | 11:24 | |
*** marcusvrn has quit IRC | 11:25 | |
aarefiev | dulek:hi | 11:28 |
dulek | aarefiev: Hi! | 11:28 |
aarefiev | dulek: I have info that you are expert in glance metadata catalog :) | 11:28 |
dulek | aarefiev: I was part of the team writing it. Shall we switch to #openstack-glance? | 11:29 |
*** annegentle has quit IRC | 11:29 | |
aarefiev | dulek: bug in cinder https://bugs.launchpad.net/cinder/+bug/1323660 | 11:30 |
openstack | Launchpad 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 |
aarefiev | dulek: problem is v1 glance api contain dict 'properties ' in image, and v2 doesn't | 11:31 |
*** ociuhandu has joined #openstack-cinder | 11:31 | |
dulek | aarefiev: It rings a bell, let me dig into it. | 11:31 |
dulek | aarefiev: Okay, I see, we struggled with this problem two days ago when writing some Horizon code. | 11:34 |
aarefiev | dulek: 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 them | 11:34 |
dulek | aarefiev: This isn't actually related with metadata catalog at all. | 11:35 |
dulek | aarefiev: This is how Glance V2 API is implemented. | 11:35 |
dulek | aarefiev: Actually API _and_ client. | 11:36 |
aarefiev | dulek: so the problem is how we handle it | 11:36 |
dulek | aarefiev: I think the main problem is that in v2 you're unable to decide which property is metadata and which normal property. | 11:38 |
dulek | aarefiev: Since these are treated in the same way in response. | 11:38 |
*** shilpa_ has quit IRC | 11:38 | |
dulek | aarefiev: I'm actually quite glad that this came out. | 11:39 |
dulek | aarefiev: I strongly think that this is Glance bug. | 11:40 |
dulek | aarefiev: We had numerous problems with it when implementing Horizon part of metadata catalog. | 11:40 |
aarefiev | dulek: btw what is the main idea with catalog | 11:41 |
dulek | aarefiev: And now we have clear use case where current v2 API behavior is a serious problem. | 11:41 |
*** Longgeek has quit IRC | 11:42 | |
dulek | aarefiev: It's to create a catalog of metadata you can attach to a resource that _mean_ something. | 11:42 |
aarefiev | dulek: so you think we should report a bug on glance | 11:43 |
dulek | aarefiev: Did you know for example that you can start a VM with a watchdog on it that will restart it on fail? | 11:43 |
dulek | aarefiev: This can be achieved by adding a specific extra spec to a flavor | 11:43 |
dulek | aarefiev: Metadata catalog tries to expose these options to the user. | 11:43 |
*** markvoelker has joined #openstack-cinder | 11:43 | |
dulek | aarefiev: I think you should add Glance on this bug and describe the main problem which is: | 11:44 |
dulek | aarefiev: we cannot determine which attribute is a property and which is a regular attribute | 11:44 |
openstackgerrit | Petrut Lucian proposed openstack/cinder: SMBFS: Fix missing volume provider location https://review.openstack.org/168305 | 11:45 |
dulek | aarefiev: If you're not feeling confident on this I can drive it. | 11:45 |
aarefiev | dulek: I talked yesterday with glance team, they said that I should get properties keys from image schema | 11:46 |
aarefiev | dulek: and them make decision which attribute is property and which not | 11:47 |
aarefiev | based on image schema | 11:48 |
*** e0ne is now known as e0ne_ | 11:48 | |
*** markvoelker has quit IRC | 11:48 | |
*** anuragpalsule has quit IRC | 11:49 | |
*** e0ne_ is now known as e0ne | 11:49 | |
dulek | aarefiev: Yeah, my colleague just showed me your discussion with Alex Tivelkov | 11:49 |
dulek | aarefiev: Did he know that you need to do that in Cinder? | 11:50 |
dulek | aarefiev: python-glanceclient doesn't provide you with the methods to get the schema I think, let me check. | 11:50 |
aarefiev | dulek: sure he knows | 11:50 |
aarefiev | dulek: the problem is image schema contain onle core properties | 11:52 |
dulek | aarefiev: Isn't that actually ok? If a property isn't in schema you consider it custom and pass it to image. | 11:53 |
aarefiev | dulek: 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.html | 11:56 |
aarefiev | dulek: core properties in image schema "kernel_id" "ramdisk_id" "instance_uuid" "architecture" "os_distro" "os_version" | 11:57 |
dulek | aarefiev: Ah, I see. | 11:58 |
dulek | aarefiev: And these should be passed to volume also? | 11:58 |
*** esker has joined #openstack-cinder | 11:58 | |
aarefiev | dulek: there are also a lot of json file with namespaces and properties in glance etc/metadefs | 11:59 |
dulek | aarefiev: Don't mind these files, I don't think that these are relevant. | 12:00 |
marcusvrn1 | hey 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 |
dulek | aarefiev: Concept of properties was there before metadata catalog. | 12:00 |
aarefiev | dulek: yes, my use case is try update image properties: glance image-update TestVM --property hw_disk_bus=ide --property hw_vif_model=rtl813 | 12:00 |
aarefiev | then create volume from image | 12:01 |
aarefiev | and boot vm from volume | 12:01 |
aarefiev | and it doesn't contain updated properties | 12:01 |
aarefiev | with glance api v2 | 12:02 |
*** resker has joined #openstack-cinder | 12:02 | |
dulek | aarefiev: That's right, that's because there is a bug you've showed first. | 12:02 |
aarefiev | dulek: yes | 12:02 |
*** esker has quit IRC | 12:04 | |
aarefiev | dulek: question how properties should get first to volume metadata and then to nova from volume metadata | 12:04 |
dulek | aarefiev: Just explain me one thing - should for example kernel_id be passed to volume properties as well? | 12:04 |
aarefiev | dulek: good question, I don't know what properties should be in volume image metadata | 12:06 |
dulek | aarefiev: So let me check... ;) | 12:07 |
dulek | aarefiev: For Glance v1 API kernel_id and the rest are passed to the image. | 12:08 |
*** Adriano_ has joined #openstack-cinder | 12:08 | |
dulek | aarefiev: So I see fundamental problem here - we're unable to distinguish for example kernel_id from name. | 12:09 |
dulek | aarefiev: So I think that Glance guys need to step in now. | 12:10 |
dulek | aarefiev: Aww, one mistake - "For Glance v1 API kernel_id and the rest are passed to the image." | 12:11 |
dulek | s/image/volume | 12:11 |
aarefiev | dulek: we have image attributes like name, id, size, etc and properties in one scope, also we have core properties and other properties | 12:11 |
aarefiev | dulek: what cinder should store and how we fetch it | 12:12 |
dulek | aarefiev: http://paste.openstack.org/show/197062/ - that's image schema | 12:13 |
dulek | aarefiev: http://paste.openstack.org/show/197065/ | 12:14 |
dulek | And this is what is saved to the volume when using Glance v1 API. | 12:14 |
*** bswartz has quit IRC | 12:14 | |
openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix: Boot from image with HNAS iSCSI https://review.openstack.org/167727 | 12:14 |
dulek | aarefiev: So you would need to pass to volume: | 12:15 |
dulek | aarefiev: 1. All properties that are not in schema | 12:15 |
dulek | aarefiev: 2. All properties that are in schema but are marked with is_base=false | 12:16 |
dulek | aarefiev: 3. Additional ones - size, min_disk, checksum, diskformat, min_ram, container_format, image_name | 12:18 |
*** markvoelker has joined #openstack-cinder | 12:18 | |
*** ociuhandu has quit IRC | 12:18 | |
dulek | aarefiev: Does it make sense? | 12:18 |
aarefiev | dulek: yes it make sense, but I think it isn't good approach | 12:19 |
dulek | aarefiev: I agree! :) | 12:19 |
dulek | aarefiev: That's why I mentioned that we should contact Glance guys. This certainly isn't the way to go. | 12:20 |
dulek | aarefiev: I don't see other way in current status of Glance v2 API. | 12:20 |
aarefiev | dulek: they are think that it is the right way | 12:21 |
aarefiev | dulek: maybe you help bring them over :) | 12:22 |
*** annegentle has joined #openstack-cinder | 12:22 | |
*** rushiagr is now known as rushiagr_away | 12:23 | |
dulek | aarefiev: 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 |
dulek | aarefiev: Is that okay for you? | 12:23 |
*** avishay_ has joined #openstack-cinder | 12:23 | |
*** annegentle has quit IRC | 12:23 | |
aarefiev | dulek: 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 understand | 12:25 |
dulek | aarefiev: They overcomplicated things in v2. And why do you actually want to use v2? | 12:26 |
*** rushil has joined #openstack-cinder | 12:26 | |
aarefiev | dulek: e.g. ceph use glance api v2 by default | 12:27 |
aarefiev | I think it is related to location which is only in api v2 | 12:27 |
dulek | aarefiev: 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-cinder | 12:29 | |
dulek | aarefiev: Nevermind, got it. | 12:30 |
aarefiev | dulek: ok :) | 12:30 |
*** shilpa_ has joined #openstack-cinder | 12:32 | |
*** shilpa_ has quit IRC | 12:35 | |
*** akerr has joined #openstack-cinder | 12:41 | |
*** Mandell has quit IRC | 12:46 | |
*** Mandell has joined #openstack-cinder | 12:47 | |
*** bswartz has joined #openstack-cinder | 12:49 | |
*** Yogi1 has joined #openstack-cinder | 12:53 | |
*** pschaef has quit IRC | 13:02 | |
*** e0ne is now known as e0ne_ | 13:03 | |
*** jaypipes is now known as leakypipes | 13:05 | |
*** ganso_ has joined #openstack-cinder | 13:08 | |
*** rushiagr_away is now known as rushiagr | 13:10 | |
*** e0ne_ is now known as e0ne | 13:11 | |
*** avishay_ has quit IRC | 13:12 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Set volume_attachment to [] for the temporary volume creation https://review.openstack.org/167134 | 13:16 |
*** nshaikh has quit IRC | 13:19 | |
*** dustins has joined #openstack-cinder | 13:21 | |
*** annegentle has joined #openstack-cinder | 13:24 | |
*** annegentle has quit IRC | 13:29 | |
*** ishant has quit IRC | 13:31 | |
*** Tross has quit IRC | 13:33 | |
openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Fix ImageNotAuthorized during copy volume https://review.openstack.org/168333 | 13:33 |
openstackgerrit | Sean McGinnis proposed openstack/os-brick: Logging not using oslo.i18n guidelines https://review.openstack.org/165276 | 13:34 |
openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Fix ImageNotAuthorized during copy volume https://review.openstack.org/168333 | 13:37 |
*** tbarron has joined #openstack-cinder | 13:40 | |
*** annegentle has joined #openstack-cinder | 13:43 | |
*** TobiasE has left #openstack-cinder | 13:47 | |
*** dims_ is now known as dimsum__ | 13:49 | |
*** Lee1092 has quit IRC | 13:50 | |
*** r-daneel has joined #openstack-cinder | 13:58 | |
*** dimsum__ has quit IRC | 14:01 | |
*** dimsum__ has joined #openstack-cinder | 14:02 | |
*** jcru has joined #openstack-cinder | 14:03 | |
*** nkrinner has quit IRC | 14:05 | |
dulek | aarefiev: I'd tried to explore one more idea, but failed, so I've added Glance to affected projects and written a comment. | 14:06 |
dulek | aarefiev: I'm pinging ativelkov on openstack-glance so you might want to eavesdrop there. ;) | 14:07 |
*** annegentle has quit IRC | 14:07 | |
aarefiev | dulek: thanks I will :) | 14:08 |
*** _cjones_ has joined #openstack-cinder | 14:08 | |
*** primechuck has joined #openstack-cinder | 14:11 | |
*** _cjones_ has quit IRC | 14:11 | |
*** mtanino has joined #openstack-cinder | 14:16 | |
*** anuragpalsule has joined #openstack-cinder | 14:21 | |
*** eharney has joined #openstack-cinder | 14:21 | |
*** thangp has joined #openstack-cinder | 14:23 | |
*** thangp has quit IRC | 14:24 | |
*** thangp has joined #openstack-cinder | 14:25 | |
*** thangp has quit IRC | 14:25 | |
*** thangp has joined #openstack-cinder | 14:25 | |
*** anuragpalsule has quit IRC | 14:26 | |
kaisers | DuncanT: asselin: ping | 14:28 |
kaisers | DuncanT: 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 |
openstack | Launchpad 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 |
kaisers | oh, there it is :-D | 14:32 |
*** annegentle has joined #openstack-cinder | 14:32 | |
*** reed has joined #openstack-cinder | 14:33 | |
*** e0ne is now known as e0ne_ | 14:36 | |
*** ebalduf has joined #openstack-cinder | 14:40 | |
*** thingee has quit IRC | 14:43 | |
mtanino | e0ne_: hi, | 14:43 |
*** _cjones_ has joined #openstack-cinder | 14:43 | |
mtanino | e0ne_: https://review.openstack.org/#/c/148038/ I fixed LIO test cases but ISER test cases still inherit tgt test cases. | 14:44 |
mtanino | just information. | 14:44 |
Liu | How tuoskeep just one test case when run "tox -eall -- volume" | 14:46 |
*** tsekiyama has joined #openstack-cinder | 14:46 | |
Liu | sorry | 14:46 |
Liu | how to just skeep one test case when run "tox -eall -- volume" | 14:47 |
*** dansmith is now known as superdan | 14:47 | |
*** e0ne_ is now known as e0ne | 14:48 | |
Liu | I do it like this "pretty_tox.sh volume.*(?!.*(test_volume_crud_with_volume_type_and_extra_specs))" | 14:48 |
Liu | but it does not work | 14:48 |
e0ne | mtanino: hi! thank you for the confirmation | 14:49 |
hemnafk | mornin | 14:49 |
*** hemnafk is now known as hemna | 14:49 | |
e0ne | hemnafk: evening:) | 14:49 |
mtanino | e0ne: But may be no one is not intrested in that decupled of ISER test case... | 14:50 |
hemna | :) | 14:50 |
*** rmesta has joined #openstack-cinder | 14:51 | |
e0ne | mtanino: i don't want to believe in it | 14:52 |
mtanino | oops ok. | 14:53 |
*** emagana has joined #openstack-cinder | 14:54 | |
*** lpabon has joined #openstack-cinder | 14:56 | |
mtanino | e0ne: Thank you for changing it. | 14:56 |
mtanino | I should do it :( | 14:56 |
*** _cjones_ has quit IRC | 14:58 | |
*** _cjones_ has joined #openstack-cinder | 14:59 | |
hemna | whoa | 15:02 |
hemna | just updated master here locally and ran tox -epep8 | 15:02 |
hemna | boatloads of violoations on old untouched code | 15:02 |
hemna | wth | 15:02 |
hemna | H904 Wrap long lines in parentheses instead of a backslash | 15:02 |
hemna | endless list of those | 15:02 |
*** xyang has joined #openstack-cinder | 15:02 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Hitachi: Fix access to volume instance_uuid https://review.openstack.org/167883 | 15:02 |
e0ne | hemna: did we merge a new hacking? | 15:03 |
hemna | I dunno | 15:03 |
hemna | can you update locally and run that ? | 15:04 |
hemna | see what you see | 15:04 |
*** annegentle has quit IRC | 15:04 | |
e0ne | hemna: are you talking about patch #167883 (fix for hitachi)? | 15:04 |
hemna | well I saw it running tox on my latest patchset | 15:05 |
*** annegentle has joined #openstack-cinder | 15:05 | |
hemna | but I'm getting H904's on all kinds of files that aren't part of that review | 15:05 |
hemna | e0ne, http://paste.openstack.org/show/197172/ | 15:05 |
hemna | for example | 15:06 |
hemna | that's a very short copy/paste from the output | 15:06 |
hemna | must be hundreds of those | 15:06 |
e0ne | :( | 15:06 |
e0ne | i'll try it on my env | 15:06 |
*** julim has quit IRC | 15:07 | |
*** julim has joined #openstack-cinder | 15:08 | |
eharney | hemna: H904 was removed from the current hacking release, probably need to update your env | 15:10 |
hemna | I just pulled master | 15:10 |
kaisers | jgriffith: ping Hello? :) | 15:10 |
hemna | and rebased | 15:10 |
*** devlaps has joined #openstack-cinder | 15:10 | |
hemna | maybe my tox virtualenv ? | 15:10 |
kaisers | hemna: tox -r | 15:10 |
kaisers | hemna: i'd guess that's what he means | 15:10 |
eharney | hemna: you need hacking >= 0.10 per test-requirements.txt | 15:10 |
hemna | k I'll try that | 15:11 |
hemna | thanks | 15:11 |
dulek | aarefiev: 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 |
e0ne | hemna: the same error. trying to update hacking to 'hemna' | 15:11 |
hemna | heh ok | 15:11 |
aarefiev | dulek: thx I'll check | 15:12 |
e0ne | eharney, hemna: confirmed, that issue was with old 'hacking' package | 15:13 |
*** julim has quit IRC | 15:13 | |
hemna | e0ne, ok thanks man. good to know, I'm rerunning here tox -r | 15:13 |
*** Lee1092 has joined #openstack-cinder | 15:13 | |
*** dannywilson has joined #openstack-cinder | 15:14 | |
*** julim has joined #openstack-cinder | 15:14 | |
*** dannywilson has quit IRC | 15:14 | |
*** dannywilson has joined #openstack-cinder | 15:14 | |
*** reed has quit IRC | 15:14 | |
*** MentalRay has joined #openstack-cinder | 15:17 | |
*** markus_z has quit IRC | 15:17 | |
*** fifieldt has quit IRC | 15:21 | |
*** _cjones_ has quit IRC | 15:24 | |
*** tbarron has quit IRC | 15:24 | |
*** tbarron has joined #openstack-cinder | 15:25 | |
*** lpabon has quit IRC | 15:26 | |
*** rhefner has joined #openstack-cinder | 15:30 | |
*** jdurgin1 has joined #openstack-cinder | 15:31 | |
*** primechuck has quit IRC | 15:39 | |
*** haomaiwang has joined #openstack-cinder | 15:39 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Set volume_attachment to [] for the temporary volume creation https://review.openstack.org/167134 | 15:43 |
jgriffith | kaisers: what up? | 15:44 |
*** akrog has joined #openstack-cinder | 15:48 | |
*** geguileo has quit IRC | 15:50 | |
*** _cjones_ has joined #openstack-cinder | 15:50 | |
*** jdurgin1 has quit IRC | 15:53 | |
*** Tross has joined #openstack-cinder | 15:53 | |
*** anuragpalsule has joined #openstack-cinder | 15:54 | |
*** anuragpalsule1 has joined #openstack-cinder | 15:56 | |
*** anuragpalsule has quit IRC | 15:58 | |
*** _cjones_ has quit IRC | 16:01 | |
*** _cjones_ has joined #openstack-cinder | 16:02 | |
*** annegentle has quit IRC | 16:06 | |
*** thingee has joined #openstack-cinder | 16:10 | |
*** leeantho has joined #openstack-cinder | 16:10 | |
*** btran has joined #openstack-cinder | 16:13 | |
*** Apoorva has joined #openstack-cinder | 16:14 | |
*** Miouge has quit IRC | 16:14 | |
*** dflorea has joined #openstack-cinder | 16:16 | |
*** mriedem is now known as mriedem_lunch | 16:17 | |
*** patrickeast has joined #openstack-cinder | 16:17 | |
jgriffith | thingee: you working on that ensure_export retry vishy pointed out this week still? | 16:20 |
jgriffith | thingee: 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 it | 16:20 |
*** e0ne is now known as e0ne_ | 16:20 | |
vishy | jgriffith: you mean a backport for it? | 16:21 |
jgriffith | vishy: yeah | 16:21 |
vishy | i have a fix for icehouse | 16:21 |
vishy | not sure if it applies to juno or not but i can send it to you | 16:21 |
vishy | one sec | 16:21 |
jgriffith | vishy: oh.. nice | 16:21 |
jgriffith | vishy: awesome | 16:21 |
thingee | jgriffith: yeah sorry, I do have it on my list. | 16:22 |
jgriffith | thingee: sounds like vishy 's already on it :) | 16:22 |
vishy | jgriffith: http://ix.io/h9r | 16:22 |
thingee | jgriffith, vishy: ok! | 16:22 |
jgriffith | vishy: yeah, that'll work in Juno too | 16:23 |
vishy | i upped it to 3 tries instead of two | 16:23 |
vishy | since the retry decorator used three as far as i could tell | 16:23 |
vishy | but i didn’t add the exponential backoff | 16:23 |
jgriffith | vishy: yup... I think that's ok for the backport IMHO | 16:23 |
jgriffith | vishy: thingee you guys want me to push these and mark vishy as author? | 16:23 |
vishy | if you propose it for juno | 16:24 |
vishy | can you cut out the reference to our bugtracker :) | 16:24 |
jgriffith | hehe :) | 16:24 |
jgriffith | I supose | 16:24 |
thingee | jgriffith: fine with me. let me know review link and I can help push through | 16:24 |
*** dulek has quit IRC | 16:24 | |
thingee | vishy: thanks! | 16:24 |
jgriffith | vishy: FWIW this is the same thing I was going to do anyway. Wasn't going to try and backport the decorator and lib | 16:24 |
jgriffith | in other words I don't think backoff etc matters here | 16:25 |
thingee | jgriffith: that's why I originally asked about backport the decorator. I have some retry stuff I wanted to get in for Datera. | 16:25 |
jgriffith | thingee: for a backport? | 16:25 |
thingee | jgriffith: maybe I should change Kilo to not use the decorator and then backport that for juno | 16:25 |
thingee | jgriffith: to make things simple | 16:26 |
jgriffith | thingee: I suppose if you prefer I can do that | 16:26 |
jgriffith | thingee: your call | 16:26 |
jgriffith | thingee: it's not *hard* | 16:26 |
*** e0ne_ is now known as e0ne | 16:26 | |
*** e0ne is now known as e0ne_ | 16:27 | |
*** e0ne_ is now known as e0ne | 16:27 | |
thingee | jgriffith: yeah that's originally what I was going to do. | 16:27 |
thingee | jgriffith: if you don't mind, otherwise this fix shouldn't wait on my greedy reasons. | 16:27 |
jgriffith | thingee: ok, I'll give it a whirl :) | 16:27 |
thingee | ;) | 16:28 |
openstackgerrit | Tomoki Sekiyama proposed openstack/cinder: Fix debug log parameter mismatch https://review.openstack.org/168137 | 16:37 |
*** alecv_ has quit IRC | 16:38 | |
*** ronis has quit IRC | 16:38 | |
openstackgerrit | Tomoki Sekiyama proposed openstack/cinder: Fix debug log parameter mismatch https://review.openstack.org/168137 | 16:39 |
*** afazekas has quit IRC | 16:40 | |
*** esker has joined #openstack-cinder | 16:41 | |
*** resker has quit IRC | 16:41 | |
*** annashen has joined #openstack-cinder | 16:44 | |
*** MentalRay has quit IRC | 16:46 | |
*** alexpilotti has joined #openstack-cinder | 16:50 | |
*** vilobhmm1 has joined #openstack-cinder | 16:50 | |
*** vilobhmm11 has joined #openstack-cinder | 16:52 | |
*** deepakcs has joined #openstack-cinder | 16:53 | |
alexpilotti | thingee: hello | 16:53 |
jgriffith | thingee: hmm... seems our unit tests in Juno are broken | 16:54 |
alexpilotti | thingee: we have the SMB3 linux and Windows CIs commenting | 16:54 |
*** vilobhmm1 has quit IRC | 16:55 | |
alexpilotti | thingee: was wondering if by any chance we could revert the SMB3 driver removal patch :-) | 16:55 |
*** jogo is now known as flashgordon | 16:57 | |
*** dflorea has quit IRC | 17:01 | |
*** haomaiw__ has joined #openstack-cinder | 17:02 | |
*** haomaiwang has quit IRC | 17:04 | |
*** dflorea has joined #openstack-cinder | 17:04 | |
*** annegentle has joined #openstack-cinder | 17:04 | |
*** anuragpalsule1 has quit IRC | 17:04 | |
*** marcusvrn has joined #openstack-cinder | 17:07 | |
*** _cjones_ has quit IRC | 17:08 | |
*** lpetrut has left #openstack-cinder | 17:11 | |
*** marcusvrn1 has quit IRC | 17:11 | |
*** jistr has quit IRC | 17:13 | |
*** Tross has quit IRC | 17:14 | |
jgriffith | thingee: http://goo.gl/JriLF5 | 17:15 |
ameade | thingee: I've got our FC CI putting up comments: https://review.openstack.org/#/q/status:open+project:openstack/cinder+reviewer:netapp-fc-ci,n,z | 17:17 |
*** nshaikh has joined #openstack-cinder | 17:18 | |
ameade | thingee: 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 on | 17:18 |
ameade | thingee: tried to mimic the tempest.confs of you and jgriffith | 17:18 |
*** rushil has quit IRC | 17:19 | |
*** Yogi1 has quit IRC | 17:19 | |
*** deepakcs has quit IRC | 17:21 | |
ameade | thingee: do you know a way i can do a filter for the latest patchset only? | 17:23 |
*** ndipanov has quit IRC | 17:25 | |
*** e0ne is now known as e0ne_ | 17:26 | |
*** nshaikh has quit IRC | 17:27 | |
*** lpetrut has joined #openstack-cinder | 17:28 | |
*** e0ne_ is now known as e0ne | 17:28 | |
*** mriedem_lunch is now known as mriedem | 17:30 | |
*** cebruns has quit IRC | 17:30 | |
*** dflorea has quit IRC | 17:33 | |
*** dflorea has joined #openstack-cinder | 17:34 | |
*** dflorea has quit IRC | 17:34 | |
*** dflorea has joined #openstack-cinder | 17:35 | |
*** e0ne is now known as e0ne_ | 17:39 | |
*** haomaiwa_ has joined #openstack-cinder | 17:40 | |
*** haomaiw__ has quit IRC | 17:40 | |
*** Lee1092 has quit IRC | 17:40 | |
*** Yogi1 has joined #openstack-cinder | 17:40 | |
*** Yogi11 has joined #openstack-cinder | 17:43 | |
*** _cjones_ has joined #openstack-cinder | 17:44 | |
*** Yogi1 has quit IRC | 17:45 | |
*** jordanP has quit IRC | 17:45 | |
*** e0ne_ has quit IRC | 17:50 | |
*** haomaiwa_ has quit IRC | 17:53 | |
*** ronis has joined #openstack-cinder | 17:55 | |
*** haomaiwang has joined #openstack-cinder | 17:55 | |
*** dflorea has quit IRC | 17:55 | |
*** annashen has quit IRC | 17:55 | |
*** dflorea has joined #openstack-cinder | 17:56 | |
jgriffith | asselin: ping | 17:57 |
openstackgerrit | Vilobh Meshram proposed openstack/cinder: ConsistencyGroup Object https://review.openstack.org/163641 | 17:57 |
jgriffith | asselin: ahh... never mind... found it | 18:00 |
*** timcl1 has joined #openstack-cinder | 18:00 | |
*** casusbelli has joined #openstack-cinder | 18:00 | |
*** e0ne has joined #openstack-cinder | 18:00 | |
*** dflorea has quit IRC | 18:01 | |
*** timcl has quit IRC | 18:02 | |
*** timcl has joined #openstack-cinder | 18:03 | |
*** e0ne has quit IRC | 18:03 | |
*** timcl1 has quit IRC | 18:04 | |
*** dflorea has joined #openstack-cinder | 18:04 | |
*** haomaiwang has quit IRC | 18:06 | |
*** dflorea has quit IRC | 18:12 | |
*** dflorea has joined #openstack-cinder | 18:12 | |
*** dflorea has quit IRC | 18:17 | |
*** avishay_ has joined #openstack-cinder | 18:20 | |
*** MentalRay has joined #openstack-cinder | 18:20 | |
*** yuriy_n17 has quit IRC | 18:22 | |
*** avishay_ has quit IRC | 18:22 | |
*** avishay_ has joined #openstack-cinder | 18:22 | |
*** afazekas has joined #openstack-cinder | 18:23 | |
*** esker has quit IRC | 18:26 | |
*** esker has joined #openstack-cinder | 18:27 | |
*** dflorea has joined #openstack-cinder | 18:28 | |
*** tsekiyama has quit IRC | 18:30 | |
*** lpetrut has left #openstack-cinder | 18:31 | |
*** tsekiyama has joined #openstack-cinder | 18:32 | |
rlucio | hi 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 IRC | 18:34 | |
*** annashen has joined #openstack-cinder | 18:35 | |
*** e0ne has joined #openstack-cinder | 18:35 | |
*** dflorea has joined #openstack-cinder | 18:35 | |
*** thingee has quit IRC | 18:35 | |
*** afazekas has quit IRC | 18:37 | |
*** dflorea has quit IRC | 18:39 | |
*** annegentle has quit IRC | 18:40 | |
*** annegentle has joined #openstack-cinder | 18:41 | |
*** Yogi11 has quit IRC | 18:46 | |
*** thingee has joined #openstack-cinder | 18:48 | |
*** med_ has joined #openstack-cinder | 18:49 | |
*** med_ has quit IRC | 18:49 | |
*** med_ has joined #openstack-cinder | 18:49 | |
*** nshaikh has joined #openstack-cinder | 18:49 | |
*** e0ne is now known as e0ne_ | 18:50 | |
rlucio | asselin: another gotcha on logservers, i changed servers and reconfigured everything OK but zuul is still posting links with old IP addr in reviews | 18:51 |
rlucio | asselin: files are being uploaded to the new IP though | 18:51 |
rlucio | asselin: i'm guessing some zuul config item needs to be updated, but I can't find it | 18:52 |
asselin | rlucio, double-check the /etc/zuul/zuul.conf | 18:53 |
asselin | rlucio, url_pattern= | 18:53 |
asselin | rlucio, if it's correct, probably you need to restart zuul | 18:53 |
rlucio | asselin: it shows the new url there | 18:54 |
thingee | jgriffith: back, sorry was in a meeting | 18:54 |
thingee | jgriffith: checking | 18:54 |
rlucio | asselin: i swear i restarted it already, but i'll try again | 18:54 |
thingee | alexpilotti: see http://lists.openstack.org/pipermail/openstack-dev/2015-March/059990.html | 18:55 |
*** e0ne_ has quit IRC | 18:55 | |
*** e0ne has joined #openstack-cinder | 18:58 | |
*** lpetrut has joined #openstack-cinder | 18:58 | |
thingee | ameade: thanks | 19:00 |
thingee | ameade: a way to filter the latest patch set? | 19:00 |
alexpilotti | thingee: about the tests, the SMB3 driver does not support volume snapshots yet, so we cannot run all 304 tests | 19:00 |
ameade | thingee: with a gerrit link | 19:00 |
alexpilotti | thingee: specifically snapshots created with volume in use | 19:01 |
*** e0ne is now known as e0ne_ | 19:02 | |
*** annashen has quit IRC | 19:03 | |
*** Apoorva has quit IRC | 19:05 | |
*** e0ne_ is now known as e0ne | 19:07 | |
*** alexpilotti has quit IRC | 19:07 | |
thingee | alexpilotti: those are required features for Cinder http://docs.openstack.org/developer/cinder/devref/drivers.html#minimum-features | 19:08 |
thingee | cinder volume drivers* | 19:08 |
*** emagana has quit IRC | 19:08 | |
thingee | ameade: I see, I'm not aware, especially because these are all non-voting. | 19:09 |
*** nshaikh has left #openstack-cinder | 19:09 | |
ameade | thingee: yeah, nbd...does that all look good to you? it seems to be running great | 19:09 |
ameade | thingee: i'm running with concurrency 1 because I think i found a bug in the brocade FC driver | 19:11 |
ameade | i'll dig into that later though | 19:11 |
thingee | ameade: yeah glanced at one of the logs, looks good to me. | 19:12 |
ameade | thingee: cool cool, thanks | 19:13 |
openstackgerrit | Tomoki Sekiyama proposed openstack/cinder: Log original backtrace on generic Exception in create_volume flow https://review.openstack.org/168459 | 19:13 |
thingee | ameade: 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#L535 | 19:14 |
*** melwitt has joined #openstack-cinder | 19:14 | |
thingee | ameade: 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 |
thingee | we'll need some* | 19:15 |
ameade | ah kk | 19:15 |
thingee | ameade: I'll email the list and individuals as usual whenever we get to that point. | 19:17 |
ameade | thingee: sounds good | 19:17 |
*** afazekas has joined #openstack-cinder | 19:24 | |
anish | 23:15 < thingee> anish: I've been seeing 'recheck <ci-name>' | 19:25 |
anish | thingee: ^ | 19:25 |
anish | I'm 100% certain I saw that cause a jenkins recheck with https://review.openstack.org/#/c/165143/ | 19:25 |
*** IanGovett1 has quit IRC | 19:25 | |
patrickeast | anish: jenkins will look for any string that starts with recheck or reverify | 19:27 |
*** IanGovett has joined #openstack-cinder | 19:27 | |
anish | patrickeast: right, which means it will recheck anytime someone asks for a third party ci recheck as well | 19:28 |
anish | which seems wasteful | 19:28 |
patrickeast | yep | 19:28 |
*** timcl has quit IRC | 19:28 | |
*** annashen has joined #openstack-cinder | 19:29 | |
anish | well, as long as it is a known issue | 19:29 |
patrickeast | anish: 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 formatting | 19:30 |
anish | patrickeast: aah, thanks for the info. | 19:31 |
*** afazekas has quit IRC | 19:32 | |
*** xyang has quit IRC | 19:33 | |
*** timcl has joined #openstack-cinder | 19:36 | |
*** devlaps has quit IRC | 19:36 | |
*** angela-s has joined #openstack-cinder | 19:39 | |
*** xyang has joined #openstack-cinder | 19:39 | |
*** avishay_ has quit IRC | 19:40 | |
*** dustins has quit IRC | 19:41 | |
*** emagana has joined #openstack-cinder | 19:41 | |
*** leakypipes has quit IRC | 19:42 | |
*** emagana has quit IRC | 19:46 | |
*** lpetrut has quit IRC | 19:48 | |
*** lpetrut has joined #openstack-cinder | 19:50 | |
*** rushiagr is now known as rushiagr_away | 19:52 | |
*** mwichmann has left #openstack-cinder | 19:53 | |
jgriffith | hemna: you're workign on this: http://goo.gl/2tJdlt | 19:53 |
*** rhefner has quit IRC | 19:54 | |
*** MentalRay_ has joined #openstack-cinder | 19:59 | |
*** e0ne is now known as e0ne_ | 20:00 | |
*** e0ne_ is now known as e0ne | 20:00 | |
*** emagana has joined #openstack-cinder | 20:01 | |
*** MentalRay has quit IRC | 20:02 | |
*** timcl has quit IRC | 20:03 | |
hemna | jgriffith, yes | 20:03 |
jgriffith | hemna: cool | 20:04 |
* jgriffith moves along | 20:04 | |
*** Apoorva has joined #openstack-cinder | 20:05 | |
*** dflorea has joined #openstack-cinder | 20:06 | |
*** afazekas has joined #openstack-cinder | 20:07 | |
*** akerr has quit IRC | 20:15 | |
*** ociuhandu has quit IRC | 20:16 | |
*** emagana has quit IRC | 20:19 | |
*** jwang__ has joined #openstack-cinder | 20:24 | |
*** annashen_ has joined #openstack-cinder | 20:24 | |
*** annashen has quit IRC | 20:24 | |
*** jwang_ has quit IRC | 20:28 | |
openstackgerrit | Tomoki Sekiyama proposed openstack/cinder: Fix volume creation from image with allowed_direct_url_schemes https://review.openstack.org/168477 | 20:32 |
*** ebalduf has quit IRC | 20:33 | |
*** thangp has quit IRC | 20:36 | |
*** e0ne is now known as e0ne_ | 20:40 | |
*** e0ne has joined #openstack-cinder | 20:41 | |
*** afazekas has quit IRC | 20:47 | |
*** dflorea has quit IRC | 20:47 | |
*** dflorea has joined #openstack-cinder | 20:48 | |
*** emagana has joined #openstack-cinder | 20:50 | |
*** dflorea has quit IRC | 20:52 | |
*** emagana has quit IRC | 20:54 | |
*** ebalduf has joined #openstack-cinder | 20:55 | |
*** mfisch has quit IRC | 20:56 | |
*** mfisch has joined #openstack-cinder | 20:57 | |
*** mfisch is now known as Guest90957 | 20:57 | |
*** e0ne is now known as e0ne_ | 20:57 | |
*** e0ne_ is now known as e0ne | 20:58 | |
-openstackstatus- NOTICE: Gerrit maintenance commences in 1 hour at 22:00 UTC http://lists.openstack.org/pipermail/openstack-dev/2015-March/059948.html | 21:00 | |
*** lcurtis has joined #openstack-cinder | 21:01 | |
*** tbarron has quit IRC | 21:01 | |
*** sgotliv has joined #openstack-cinder | 21:02 | |
*** annashen_ has quit IRC | 21:08 | |
*** akerr has joined #openstack-cinder | 21:09 | |
openstackgerrit | xing-yang proposed openstack/cinder: Fix the format of the system name in VMAX driver https://review.openstack.org/168487 | 21:10 |
*** annegentle has quit IRC | 21:10 | |
*** akerr_ has joined #openstack-cinder | 21:10 | |
*** marcusvrn has quit IRC | 21:12 | |
*** akerr has quit IRC | 21:13 | |
*** dflorea has joined #openstack-cinder | 21:14 | |
*** Adriano_ has quit IRC | 21:16 | |
*** Apoorva has quit IRC | 21:16 | |
*** e0ne has quit IRC | 21:18 | |
*** MentalRay_ has quit IRC | 21:20 | |
*** MentalRay_ has joined #openstack-cinder | 21:24 | |
*** Tross has joined #openstack-cinder | 21:29 | |
*** xyang has quit IRC | 21:41 | |
*** xyang has joined #openstack-cinder | 21:41 | |
*** marcusvrn has joined #openstack-cinder | 21:42 | |
thingee | anish: sounds like we should talk to infra then | 21:43 |
*** tsekiyam_ has joined #openstack-cinder | 21:43 | |
*** lpetrut has quit IRC | 21:46 | |
*** tsekiyama has quit IRC | 21:46 | |
*** tsekiyam_ has quit IRC | 21:47 | |
*** xyang has quit IRC | 21:49 | |
*** dflorea has quit IRC | 21:50 | |
*** dflorea has joined #openstack-cinder | 21:51 | |
anish | thingee: 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 format | 21:53 |
*** vilobhmm11 has quit IRC | 21:53 | |
*** dflorea has quit IRC | 21:55 | |
*** ociuhandu has joined #openstack-cinder | 21:55 | |
*** mriedem has quit IRC | 21:56 | |
*** ebalduf has quit IRC | 22:02 | |
-openstackstatus- NOTICE: Gerrit is offline for maintenance, ETA 22:30 UTC http://lists.openstack.org/pipermail/openstack-dev/2015-March/059948.html | 22: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 IRC | 22:03 | |
*** dflorea has joined #openstack-cinder | 22:04 | |
*** annegentle has joined #openstack-cinder | 22:04 | |
*** akerr_ has quit IRC | 22:06 | |
*** annashen has joined #openstack-cinder | 22:09 | |
*** dimsum__ has quit IRC | 22:09 | |
*** vilobhmm1 has joined #openstack-cinder | 22:10 | |
*** vilobhmm1 has quit IRC | 22:10 | |
*** vilobhmm1 has joined #openstack-cinder | 22:10 | |
*** vilobhmm11 has joined #openstack-cinder | 22:13 | |
*** annashen has quit IRC | 22:14 | |
*** vilobhmm1 has quit IRC | 22:15 | |
*** MentalRay_ has joined #openstack-cinder | 22:18 | |
*** vilobhmm11 has quit IRC | 22:19 | |
*** vilobhmm1 has joined #openstack-cinder | 22:20 | |
*** dimsum__ has joined #openstack-cinder | 22:20 | |
*** mdenny has quit IRC | 22:22 | |
*** markstur has joined #openstack-cinder | 22:24 | |
*** markstur is now known as markstur_away | 22:25 | |
*** markstur_away has left #openstack-cinder | 22:26 | |
*** ganso_ has quit IRC | 22:27 | |
*** ronis has quit IRC | 22:30 | |
*** erlon has quit IRC | 22: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 IRC | 22:34 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Brick: Fix race in removing iSCSI device https://review.openstack.org/168505 | 22:37 |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Brick: Fix race in removing iSCSI device https://review.openstack.org/168506 | 22:38 |
hemna | phew | 22:38 |
patrickeast | hemna: oo you got a fix for it? | 22:40 |
*** markvoelker has quit IRC | 22:40 | |
hemna | patrickeast, yah I think so | 22:41 |
*** annashen has joined #openstack-cinder | 22:41 | |
hemna | we have a Helion tester that is seeing this happen on a crappy old system | 22:41 |
patrickeast | haha | 22:41 |
hemna | basically the race happens between the time we tell the kernel to remove the volume | 22:41 |
hemna | and when we look for the volumes in /dev/disk/by-path | 22:42 |
hemna | it wasn't going away | 22:42 |
hemna | and the iSCSI session was never logged out | 22:42 |
hemna | so I added a wait verification loop | 22:42 |
patrickeast | hemna: 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 invalid | 22:42 |
hemna | python was running too quickly :) | 22:42 |
patrickeast | dont hear that every day! | 22:43 |
hemna | kernel was too busy with io to remove the volume from the scsi subsystem in time, to make it go away in /dev/disk/by-path | 22:43 |
hemna | PITA | 22:43 |
patrickeast | yea, its nice that these things are async commands… except times like this | 22:43 |
hemna | I need to do some refactoring of some of the code in brick to make this available for all transports | 22:43 |
hemna | but this fixes the iSCSI case for now. | 22:43 |
hemna | I can't wait to get brick out of Cinder, so I can do some house cleaning of the connectors | 22:44 |
patrickeast | yea its going to be nice | 22:44 |
patrickeast | esp when nova is using it too | 22:44 |
patrickeast | the differences in the attach code give me a lot of heartburn | 22:45 |
*** patrickeast has quit IRC | 22:46 | |
hemna | yah, it gives me the willies right now | 22:46 |
*** sneezewort has quit IRC | 22:47 | |
*** thingee has quit IRC | 22:49 | |
hemna | ouch | 22:52 |
hemna | Looks 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 IRC | 22:57 | |
*** annashen has quit IRC | 23:00 | |
*** annashen has joined #openstack-cinder | 23:12 | |
*** annegentle has quit IRC | 23:19 | |
*** _cjones_ has quit IRC | 23:21 | |
*** MentalRay__ has joined #openstack-cinder | 23:21 | |
*** MentalRay_ has quit IRC | 23:24 | |
*** jcru has quit IRC | 23:27 | |
*** cebruns has joined #openstack-cinder | 23:33 | |
*** bswartz has joined #openstack-cinder | 23:33 | |
*** IanGovett has quit IRC | 23:34 | |
*** rmesta has quit IRC | 23:40 | |
*** MentalRay__ has quit IRC | 23:40 | |
*** markvoelker has joined #openstack-cinder | 23:41 | |
*** markvoelker has quit IRC | 23:46 | |
*** lcurtis has quit IRC | 23:47 | |
*** IanGovett has joined #openstack-cinder | 23:49 | |
*** btran has quit IRC | 23:50 | |
*** ociuhandu has quit IRC | 23:52 | |
*** dannywilson has quit IRC | 23:53 | |
*** annashen has quit IRC | 23:54 | |
*** dimsum__ has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!