patrickeast | jgriffith: cool, yea we can hammer out those details later | 00:00 |
---|---|---|
patrickeast | jgriffith: trivial to move it around | 00:00 |
jgriffith | patrickeast: crazy idea.. we could also put those things in the base driver | 00:00 |
patrickeast | jgriffith: yea i was going to do that | 00:00 |
jgriffith | patrickeast: get them for free in the sub classes | 00:00 |
patrickeast | jgriffith: but this puts one more layer of abstraction from driver accessing the db :p | 00:00 |
jgriffith | and limit db stuff to base driver only so we don't kill ourselves with future changes | 00:01 |
jgriffith | patrickeast: yeah | 00:01 |
jgriffith | patrickeast: well.. my vote I guess is you should flush it out a bit more and see what others think | 00:01 |
jgriffith | patrickeast: or don't and see what others think :) | 00:01 |
patrickeast | jgriffith: sounds like a plan | 00:01 |
patrickeast | jgriffith: haha, nah i'll get it passing tests and put up a WIP review | 00:01 |
jgriffith | patrickeast: or just fix it all and merge it while we sleep :) | 00:01 |
patrickeast | jgriffith: that is always an option :D | 00:02 |
*** chlong has quit IRC | 00:05 | |
openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Add Cheesecake APIs to policy.json file https://review.openstack.org/313124 | 00:06 |
*** yangyape_ has quit IRC | 00:10 | |
*** garthb_ has quit IRC | 00:13 | |
*** prometheanfire has left #openstack-cinder | 00:13 | |
*** leeantho has quit IRC | 00:22 | |
openstackgerrit | wanghao proposed openstack/cinder: Add ability to filter by volume_glance_metadata https://review.openstack.org/147738 | 00:24 |
openstackgerrit | wanghao proposed openstack/cinder: Backup tenant attribute support https://review.openstack.org/258374 | 00:25 |
*** salv-orlando has joined #openstack-cinder | 00:26 | |
openstackgerrit | wanghao proposed openstack/cinder: Catch GlanceMetadataNotFound in creating cgsnp https://review.openstack.org/302691 | 00:29 |
*** laughterwym has joined #openstack-cinder | 00:29 | |
*** laughter_ has joined #openstack-cinder | 00:31 | |
*** salv-orlando has quit IRC | 00:32 | |
*** earlephilhower has quit IRC | 00:33 | |
*** laughterwym has quit IRC | 00:34 | |
*** laughter_ has quit IRC | 00:34 | |
*** ducttape_ has quit IRC | 00:35 | |
*** itlinux has quit IRC | 00:36 | |
*** laughterwym has joined #openstack-cinder | 00:37 | |
*** Julien-zte has joined #openstack-cinder | 00:37 | |
*** mriedem has quit IRC | 00:39 | |
*** laughterwym has quit IRC | 00:40 | |
*** laughterwym has joined #openstack-cinder | 00:40 | |
*** akshai has joined #openstack-cinder | 00:40 | |
*** akshai_ has joined #openstack-cinder | 00:42 | |
*** akshai has quit IRC | 00:45 | |
openstackgerrit | KongWei proposed openstack/cinder: Add ZTE Block Storage Driver https://review.openstack.org/312355 | 00:46 |
*** mriedem has joined #openstack-cinder | 00:46 | |
*** salv-orlando has joined #openstack-cinder | 00:51 | |
*** mtanino has quit IRC | 00:52 | |
*** salv-orlando has quit IRC | 00:56 | |
*** suyash has quit IRC | 00:57 | |
*** Julien-zte has quit IRC | 00:58 | |
*** iceyao has joined #openstack-cinder | 01:02 | |
*** iceyao_ has joined #openstack-cinder | 01:06 | |
*** ducttape_ has joined #openstack-cinder | 01:07 | |
*** ducttape_ has quit IRC | 01:09 | |
*** iceyao has quit IRC | 01:09 | |
*** yangyapeng has joined #openstack-cinder | 01:12 | |
*** enriquetaso has quit IRC | 01:14 | |
*** raunak has quit IRC | 01:18 | |
*** haplo37 has joined #openstack-cinder | 01:18 | |
*** yangyapeng has quit IRC | 01:21 | |
*** salv-orlando has joined #openstack-cinder | 01:22 | |
*** nnex has joined #openstack-cinder | 01:24 | |
*** Lee1092 has joined #openstack-cinder | 01:25 | |
openstackgerrit | Wilson Liu proposed openstack/cinder: Huawei: Raise if no FC port found https://review.openstack.org/296176 | 01:27 |
*** yangxi has joined #openstack-cinder | 01:29 | |
*** xyang1 has quit IRC | 01:29 | |
*** nnex has quit IRC | 01:30 | |
openstackgerrit | Patrick East proposed openstack/cinder: Move initiator_data access to helper methods https://review.openstack.org/313200 | 01:30 |
patrickeast | jgriffith: ^ | 01:30 |
*** salv-orlando has quit IRC | 01:30 | |
*** nnex has joined #openstack-cinder | 01:30 | |
*** chlong has joined #openstack-cinder | 01:31 | |
*** EinstCrazy has joined #openstack-cinder | 01:34 | |
openstackgerrit | Alex Meade proposed openstack/cinder-specs: Promote backend after failover (cheesecake) https://review.openstack.org/312591 | 01:37 |
openstackgerrit | Cao ShuFeng proposed openstack/cinder: remove extra quota commit of manage snapshot https://review.openstack.org/298522 | 01:49 |
*** houming has joined #openstack-cinder | 01:57 | |
*** salv-orlando has joined #openstack-cinder | 01:57 | |
*** gcb has quit IRC | 01:59 | |
*** savihou has quit IRC | 02:00 | |
*** savihou has joined #openstack-cinder | 02:01 | |
openstackgerrit | Peter Wang proposed openstack/cinder: VNX: New Cinder driver in Newton https://review.openstack.org/312427 | 02:01 |
*** ducttape_ has joined #openstack-cinder | 02:03 | |
*** EinstCra_ has joined #openstack-cinder | 02:06 | |
*** yangxi has quit IRC | 02:06 | |
*** yangxi has joined #openstack-cinder | 02:09 | |
*** EinstCrazy has quit IRC | 02:09 | |
*** salv-orlando has quit IRC | 02:09 | |
*** bardia has quit IRC | 02:10 | |
*** gcb has joined #openstack-cinder | 02:11 | |
openstackgerrit | Wilson Liu proposed openstack/cinder: Huawei: Continue to deal with other port groups https://review.openstack.org/312928 | 02:11 |
*** karthikp has joined #openstack-cinder | 02:12 | |
*** ducttape_ has quit IRC | 02:14 | |
*** apoorvad has quit IRC | 02:16 | |
*** gouthamr_ has quit IRC | 02:20 | |
*** nnex has quit IRC | 02:23 | |
openstackgerrit | Wilson Liu proposed openstack/cinder: Huawei: Raise if no FC port found https://review.openstack.org/296176 | 02:27 |
*** raunak has joined #openstack-cinder | 02:28 | |
openstackgerrit | Wilson Liu proposed openstack/cinder: Add unit tests for Huawei drivers https://review.openstack.org/283029 | 02:28 |
openstackgerrit | Wilson Liu proposed openstack/cinder: Huawei: Mask chap password in log https://review.openstack.org/271595 | 02:31 |
*** sasukeh has joined #openstack-cinder | 02:33 | |
*** yangxi has quit IRC | 02:34 | |
*** mriedem has quit IRC | 02:36 | |
*** raunak_ has joined #openstack-cinder | 02:38 | |
openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Add replication v2.1 test cases https://review.openstack.org/313209 | 02:38 |
*** raunak has quit IRC | 02:39 | |
*** raunak_ is now known as raunak | 02:39 | |
*** iceyao has joined #openstack-cinder | 02:40 | |
*** iceyao_ has quit IRC | 02:44 | |
*** daneyon has joined #openstack-cinder | 02:44 | |
*** daneyon has quit IRC | 02:44 | |
*** daneyon has joined #openstack-cinder | 02:45 | |
*** ducttape_ has joined #openstack-cinder | 02:47 | |
openstackgerrit | zhangguoqing proposed openstack/cinder: Violin: resize volume if cloned lun is larger https://review.openstack.org/295094 | 02:48 |
*** daneyon has quit IRC | 02:51 | |
*** yangxi has joined #openstack-cinder | 02:51 | |
*** Julien-zte has joined #openstack-cinder | 03:00 | |
*** sasukeh has quit IRC | 03:00 | |
*** sasukeh has joined #openstack-cinder | 03:02 | |
*** raunak has quit IRC | 03:03 | |
*** raunak has joined #openstack-cinder | 03:06 | |
openstackgerrit | Wilson Liu proposed openstack/cinder-specs: Migrate volume between backends in an async way https://review.openstack.org/312853 | 03:14 |
*** akshai_ has quit IRC | 03:15 | |
*** itlinux has joined #openstack-cinder | 03:18 | |
*** sasukeh has quit IRC | 03:19 | |
*** markvoelker has quit IRC | 03:20 | |
*** salv-orlando has joined #openstack-cinder | 03:21 | |
*** nnex has joined #openstack-cinder | 03:22 | |
*** yangyapeng has joined #openstack-cinder | 03:28 | |
*** salv-orlando has quit IRC | 03:30 | |
*** bardia has joined #openstack-cinder | 03:32 | |
*** julim has quit IRC | 03:33 | |
*** amit213 has quit IRC | 03:33 | |
*** julim has joined #openstack-cinder | 03:33 | |
*** amit213 has joined #openstack-cinder | 03:33 | |
*** bardia has quit IRC | 03:38 | |
*** ducttape_ has quit IRC | 03:40 | |
openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Apply storage policy at vmdk level https://review.openstack.org/313213 | 03:43 |
*** salv-orlando has joined #openstack-cinder | 03:44 | |
*** yangxi has quit IRC | 03:45 | |
*** salv-orlando has quit IRC | 03:46 | |
*** laughterwym has quit IRC | 03:50 | |
*** yangxi has joined #openstack-cinder | 03:50 | |
*** buhman has joined #openstack-cinder | 03:50 | |
*** laughterwym has joined #openstack-cinder | 03:50 | |
buhman | https://github.com/openstack/python-cinderclient/blob/master/cinderclient/client.py#L638 | 03:51 |
buhman | wouldn't it make more sense to do extension discovery via pkg_resources, and some openstack.cinderclient.extensions entrypoint? | 03:51 |
*** yangxi has quit IRC | 03:54 | |
*** esker has quit IRC | 03:55 | |
*** karthikp has quit IRC | 03:59 | |
*** links has joined #openstack-cinder | 04:02 | |
openstackgerrit | Cao ShuFeng proposed openstack/cinder: remove extra quota commit of manage snapshot https://review.openstack.org/298522 | 04:08 |
*** Julien-zte has quit IRC | 04:16 | |
*** markvoelker has joined #openstack-cinder | 04:21 | |
*** dave-mcc_ has quit IRC | 04:26 | |
*** markvoelker has quit IRC | 04:26 | |
*** salv-orlando has joined #openstack-cinder | 04:28 | |
*** salv-orlando has quit IRC | 04:32 | |
*** ducttape_ has joined #openstack-cinder | 04:41 | |
*** yangxi has joined #openstack-cinder | 04:43 | |
*** daneyon_ has joined #openstack-cinder | 04:46 | |
*** ducttape_ has quit IRC | 04:46 | |
*** lennyb__ has joined #openstack-cinder | 04:49 | |
*** lennyb__ has quit IRC | 04:50 | |
*** salv-orlando has joined #openstack-cinder | 05:01 | |
*** salv-orlando has quit IRC | 05:04 | |
*** yangxi has quit IRC | 05:07 | |
*** Poornima has joined #openstack-cinder | 05:07 | |
*** yangxi has joined #openstack-cinder | 05:09 | |
*** laughter_ has joined #openstack-cinder | 05:11 | |
*** raunak has quit IRC | 05:13 | |
*** laughterwym has quit IRC | 05:14 | |
*** salv-orlando has joined #openstack-cinder | 05:17 | |
*** markvoelker has joined #openstack-cinder | 05:22 | |
*** salv-orlando has quit IRC | 05:24 | |
*** markvoelker has quit IRC | 05:26 | |
*** jseiler_ has joined #openstack-cinder | 05:30 | |
*** jseiler__ has quit IRC | 05:34 | |
*** salv-orlando has joined #openstack-cinder | 05:35 | |
*** salv-orlando has quit IRC | 05:39 | |
*** vmtrooper has quit IRC | 05:42 | |
*** laughter_ has quit IRC | 05:44 | |
*** laughterwym has joined #openstack-cinder | 05:44 | |
openstackgerrit | Pranali Deore proposed openstack/cinder: Add logging when filtering returns nothing https://review.openstack.org/306285 | 05:47 |
openstackgerrit | LisaLi proposed openstack/os-brick: Copy encryptors from Nova to os-brick https://review.openstack.org/247372 | 06:01 |
*** rdx_matt has joined #openstack-cinder | 06:06 | |
*** gcb has quit IRC | 06:13 | |
*** jkraj has joined #openstack-cinder | 06:17 | |
*** vmtrooper has joined #openstack-cinder | 06:17 | |
openstackgerrit | Nitin Madhok proposed openstack/cinder-specs: Making doc fixes in backup-snapshots.rst https://review.openstack.org/313237 | 06:19 |
*** zul has quit IRC | 06:21 | |
*** yangxi has quit IRC | 06:22 | |
*** markvoelker has joined #openstack-cinder | 06:23 | |
*** yangxi has joined #openstack-cinder | 06:25 | |
*** gcb has joined #openstack-cinder | 06:26 | |
openstackgerrit | Nitin Madhok proposed openstack/cinder-specs: Making doc fixes in backup-snapshots.rst https://review.openstack.org/313237 | 06:27 |
*** markvoelker has quit IRC | 06:27 | |
openstackgerrit | Cao ShuFeng proposed openstack/cinder: remove extra quota commit of manage snapshot https://review.openstack.org/298522 | 06:31 |
*** ircuser-1 has joined #openstack-cinder | 06:34 | |
*** haplo37 has quit IRC | 06:38 | |
*** e0ne has joined #openstack-cinder | 06:39 | |
*** ducttape_ has joined #openstack-cinder | 06:42 | |
openstackgerrit | Cao ShuFeng proposed openstack/cinder: enhanche error message [1/5] https://review.openstack.org/296304 | 06:43 |
openstackgerrit | Cao ShuFeng proposed openstack/cinder: Allow API user to update multiattach attr of volume https://review.openstack.org/309674 | 06:45 |
*** salv-orlando has joined #openstack-cinder | 06:46 | |
*** ducttape_ has quit IRC | 06:48 | |
*** kfox1111 has quit IRC | 06:53 | |
*** e0ne has quit IRC | 06:53 | |
*** salv-orlando has quit IRC | 06:57 | |
*** e0ne has joined #openstack-cinder | 06:59 | |
*** e0ne has quit IRC | 07:00 | |
*** permalac has joined #openstack-cinder | 07:06 | |
*** itlinux has quit IRC | 07:14 | |
*** e0ne has joined #openstack-cinder | 07:16 | |
*** gcb has quit IRC | 07:21 | |
openstackgerrit | Peter Wang proposed openstack/cinder: VNX: New Cinder driver in Newton https://review.openstack.org/312427 | 07:23 |
*** markvoelker has joined #openstack-cinder | 07:23 | |
*** salv-orlando has joined #openstack-cinder | 07:26 | |
*** markvoelker has quit IRC | 07:28 | |
*** markus_z has joined #openstack-cinder | 07:29 | |
*** gcb has joined #openstack-cinder | 07:33 | |
*** lpetrut has joined #openstack-cinder | 07:36 | |
openstackgerrit | surya n proposed openstack/cinder: Minor typo fixes. https://review.openstack.org/313259 | 07:37 |
*** openstackstatus has quit IRC | 07:38 | |
*** openstackstatus has joined #openstack-cinder | 07:39 | |
*** ChanServ sets mode: +v openstackstatus | 07:39 | |
*** salv-orlando has quit IRC | 07:40 | |
*** lpetrut has quit IRC | 07:43 | |
*** rdx_matt has quit IRC | 07:43 | |
*** gcb has quit IRC | 07:43 | |
*** rdx_matt has joined #openstack-cinder | 07:43 | |
*** lpetrut has joined #openstack-cinder | 07:45 | |
*** rdx_matt_ has joined #openstack-cinder | 07:49 | |
*** rdx_matt has quit IRC | 07:53 | |
*** chlong has quit IRC | 07:54 | |
*** gcb has joined #openstack-cinder | 07:55 | |
*** ildikov_ has joined #openstack-cinder | 07:56 | |
*** ildikov has quit IRC | 07:58 | |
*** ildikov_ is now known as ildikov | 07:58 | |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-cinder | 08:02 | |
*** gcb has quit IRC | 08:03 | |
*** mkrcmari__ has joined #openstack-cinder | 08:05 | |
*** e0ne has quit IRC | 08:07 | |
*** mvk_ has quit IRC | 08:08 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 08:10 |
*** gcb has joined #openstack-cinder | 08:16 | |
*** laughterwym has quit IRC | 08:17 | |
*** yuriy_n17 has quit IRC | 08:19 | |
*** salv-orlando has joined #openstack-cinder | 08:19 | |
*** laughterwym has joined #openstack-cinder | 08:19 | |
*** yuriy_n17 has joined #openstack-cinder | 08:19 | |
*** e0ne has joined #openstack-cinder | 08:20 | |
*** salv-orlando has quit IRC | 08:24 | |
*** gcb has quit IRC | 08:25 | |
*** lpetrut has quit IRC | 08:26 | |
*** laughterwym has quit IRC | 08:29 | |
*** laughterwym has joined #openstack-cinder | 08:30 | |
*** jistr has joined #openstack-cinder | 08:32 | |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Handle SSL termination proxies for version list https://review.openstack.org/305152 | 08:33 |
*** laughterwym has quit IRC | 08:34 | |
*** rdx_matt_ has quit IRC | 08:35 | |
*** rdx_matt has joined #openstack-cinder | 08:35 | |
*** gcb has joined #openstack-cinder | 08:39 | |
*** rdx_matt has quit IRC | 08:40 | |
*** e0ne has quit IRC | 08:40 | |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Image virtual size doesn't fit to volume size https://review.openstack.org/311152 | 08:42 |
*** e0ne has joined #openstack-cinder | 08:42 | |
*** mkrcmari__ has quit IRC | 08:43 | |
*** ducttape_ has joined #openstack-cinder | 08:44 | |
*** ducttape_ has quit IRC | 08:49 | |
*** jordanP has joined #openstack-cinder | 08:55 | |
*** sage has quit IRC | 08:56 | |
*** tesseract has joined #openstack-cinder | 08:56 | |
*** gcb has quit IRC | 08:56 | |
*** tesseract is now known as Guest42930 | 08:56 | |
*** sage has joined #openstack-cinder | 08:57 | |
*** EinstCra_ has quit IRC | 08:59 | |
*** EinstCrazy has joined #openstack-cinder | 09:00 | |
*** lpetrut has joined #openstack-cinder | 09:14 | |
*** vmtrooper has quit IRC | 09:15 | |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Replace explicit changing of config parameters https://review.openstack.org/302249 | 09:25 |
*** mkrcmari__ has joined #openstack-cinder | 09:28 | |
*** laughterwym has joined #openstack-cinder | 09:32 | |
*** laughterwym has quit IRC | 09:37 | |
*** salv-orlando has joined #openstack-cinder | 09:41 | |
*** lpetrut has quit IRC | 09:42 | |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Move OVO methods from CinderObject to Persistent https://review.openstack.org/312930 | 09:47 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove remotable from OVOs https://review.openstack.org/312929 | 09:47 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add auto relationship mappings to VOs https://review.openstack.org/307075 | 09:47 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Link VOs with their list counterparts https://review.openstack.org/307074 | 09:47 |
*** houming has quit IRC | 09:48 | |
*** salv-orlando has quit IRC | 09:49 | |
yuriy_n17 | geguileo: Hi! | 09:51 |
geguileo | yuriy_n17: Hi | 09:51 |
yuriy_n17 | geguileo: I'd like to discuss on https://review.openstack.org/#/c/302249 if you have free time now. | 09:52 |
geguileo | yuriy_n17: Sure | 09:52 |
yuriy_n17 | geguileo: See if we use override_config method for overwriting config options there we'll get race conditions. | 09:53 |
yuriy_n17 | geguileo: Now I specially send the patch with that method to show it. | 09:54 |
*** yangxi has quit IRC | 09:54 | |
geguileo | yuriy_n17: Ok, I believe we shouldn't have races | 09:55 |
*** yangxi has joined #openstack-cinder | 09:55 | |
yuriy_n17 | geguileo: Unfortunately, we have | 09:55 |
geguileo | yuriy_n17: So I'd like to look at the errors to see if we have an underlying problem there | 09:56 |
yuriy_n17 | geguileo: Sure. https://review.openstack.org/#/c/302249 | 09:56 |
*** jkraj has quit IRC | 09:57 | |
geguileo | yuriy_n17: I am downloading it to test locally, because the gate it going to take a while | 09:57 |
geguileo | yuriy_n17: And with my memory I'll forget later to look at it :-( | 09:57 |
openstackgerrit | Merged openstack/cinder: Add devref for conditional updates https://review.openstack.org/313040 | 09:57 |
yuriy_n17 | geguileo: Ok | 09:58 |
yuriy_n17 | geguileo: You can look at http://logs.openstack.org/49/302249/5/check/gate-cinder-python27-db/f6b882d/console.html | 09:59 |
geguileo | yuriy_n17: Is that from this run? | 10:01 |
yuriy_n17 | geguileo: Yes | 10:01 |
ekarlso | asdasd | 10:03 |
*** ekarlso has quit IRC | 10:08 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 10:10 |
*** haplo37 has joined #openstack-cinder | 10:16 | |
*** ankit_ag has joined #openstack-cinder | 10:17 | |
*** yangxi has quit IRC | 10:19 | |
*** dims has quit IRC | 10:20 | |
*** chuck__ has joined #openstack-cinder | 10:25 | |
*** markvoelker has joined #openstack-cinder | 10:26 | |
*** sdague has joined #openstack-cinder | 10:28 | |
*** markvoelker has quit IRC | 10:30 | |
openstackgerrit | Ryan Liang proposed openstack/cinder: VNX: New Cinder driver in Newton https://review.openstack.org/312427 | 10:31 |
*** yangyape_ has joined #openstack-cinder | 10:33 | |
*** alaski is now known as lascii | 10:36 | |
*** yangyapeng has quit IRC | 10:36 | |
*** yangyape_ has quit IRC | 10:37 | |
geguileo | yuriy_n17: ping | 10:42 |
yuriy_n17 | geguileo: yes | 10:42 |
geguileo | yuriy_n17: Ok, the flags method works fine | 10:43 |
geguileo | yuriy_n17: The problem is that other tests are setting the CONF manually | 10:43 |
geguileo | yuriy_n17: Instead of using flags | 10:43 |
geguileo | yuriy_n17: So even if you change that test to do it right, the others are doing it wrong and break your test | 10:43 |
yuriy_n17 | geguileo: You mean in that suitcase or others? | 10:44 |
geguileo | yuriy_n17: Others | 10:44 |
geguileo | yuriy_n17: This is one example https://github.com/openstack/cinder/blob/master/cinder/tests/unit/test_hitachi_hnas_backend.py#L362 | 10:45 |
*** ducttape_ has joined #openstack-cinder | 10:45 | |
geguileo | yuriy_n17: So the interactions of that test doing it wrong (and others like that) is what's making the tests you've changed fail | 10:45 |
yuriy_n17 | geguileo: Ok, I see, thanks | 10:46 |
geguileo | yuriy_n17: As I suspected we have a bigger problem, we have to fix it everywhere :-( | 10:46 |
yuriy_n17 | geguileo: Well-well... | 10:46 |
*** ducttape_ has quit IRC | 10:49 | |
*** EinstCrazy has quit IRC | 10:50 | |
*** iceyao has quit IRC | 10:51 | |
*** lpetrut has joined #openstack-cinder | 11:01 | |
*** dims has joined #openstack-cinder | 11:02 | |
*** chuck__ has quit IRC | 11:06 | |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Prevent Multitable conditional updates https://review.openstack.org/310656 | 11:16 |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Replace manually changing CONF options https://review.openstack.org/302249 | 11:20 |
*** gouthamr has joined #openstack-cinder | 11:21 | |
*** laughterwym has joined #openstack-cinder | 11:22 | |
*** haplo37 has quit IRC | 11:25 | |
*** haplo37 has joined #openstack-cinder | 11:26 | |
*** markvoelker has joined #openstack-cinder | 11:27 | |
*** gouthamr_ has joined #openstack-cinder | 11:27 | |
*** chuck__ has joined #openstack-cinder | 11:28 | |
*** gouthamr has quit IRC | 11:31 | |
*** markvoelker has quit IRC | 11:31 | |
*** laughterwym has quit IRC | 11:36 | |
*** laughterwym has joined #openstack-cinder | 11:37 | |
*** laughterwym has quit IRC | 11:41 | |
*** yangyapeng has joined #openstack-cinder | 11:49 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Use to_utf8() instead of safe_encode() in convert_str() https://review.openstack.org/313521 | 11:52 |
*** laughterwym has joined #openstack-cinder | 11:53 | |
*** salv-orlando has joined #openstack-cinder | 11:58 | |
*** Lee1092 has quit IRC | 12:12 | |
*** markvoelker has joined #openstack-cinder | 12:14 | |
*** maestro has joined #openstack-cinder | 12:16 | |
*** ekarlso has joined #openstack-cinder | 12:19 | |
openstackgerrit | Nitin Madhok proposed openstack/cinder-specs: Making doc fixes in backup-snapshots.rst https://review.openstack.org/313237 | 12:25 |
*** iceyao has joined #openstack-cinder | 12:25 | |
*** salv-orlando has quit IRC | 12:28 | |
*** xyang1 has joined #openstack-cinder | 12:38 | |
*** iceyao has quit IRC | 12:38 | |
*** savihou has quit IRC | 12:39 | |
*** takedakn has joined #openstack-cinder | 12:40 | |
*** ankit_ag has quit IRC | 12:41 | |
openstackgerrit | Pranali Deore proposed openstack/cinder: Add logging when filtering returns nothing https://review.openstack.org/306285 | 12:44 |
*** iceyao has joined #openstack-cinder | 12:44 | |
*** ducttape_ has joined #openstack-cinder | 12:46 | |
*** tpsilva has joined #openstack-cinder | 12:47 | |
*** ducttape_ has quit IRC | 12:51 | |
*** iceyao has quit IRC | 12:51 | |
*** alejandrito has joined #openstack-cinder | 12:52 | |
*** timcl has joined #openstack-cinder | 12:53 | |
*** akshai has joined #openstack-cinder | 12:53 | |
*** rlrossit has joined #openstack-cinder | 12:54 | |
scottda | ameade: Did my comments in https://review.openstack.org/#/c/299052/ make sense, about not supporting the deprecated sort_key and sort_dir? | 12:56 |
ameade | scottda: yeah, i was being lazy and just leaving in that stuff for future proofing | 12:56 |
ameade | scottda: once sheel is done with pagination on the server side we can add in the proper client stuff | 12:57 |
sheel | scottda: I have one query here | 12:57 |
scottda | OK, cool. It looks like the server side pagination is fine for the new "sort key:dir" style. | 12:57 |
sheel | scottda: we are still using sort_dir and sort_key for volume... | 12:58 |
sheel | scottda: is it ok to update that as well? | 12:58 |
sheel | for cinderclient | 12:58 |
*** iceyao has joined #openstack-cinder | 12:58 | |
*** sdake has joined #openstack-cinder | 13:00 | |
scottda | sheel: You can see by tracing through the server code that sort_dir and sort_key are used internally, but they are set in the helper function when you use "sort key:dir"....looking for the code... | 13:00 |
sheel | scottda: yep, its ok in serverside | 13:01 |
sheel | scottda: my query was for client side | 13:01 |
sheel | scottda: we are using sort_key and sort_dir in volume get | 13:01 |
scottda | sheel: The client should only support "sort key:dir" and should not support "--sort_key <key> --sort_dir <direction>" since that is deprecated. | 13:01 |
*** EinstCrazy has joined #openstack-cinder | 13:02 | |
sheel | scottda: yep... | 13:02 |
scottda | sheel: Yes, it is still in volume get, but it is deprecated in kilo and I assume it will be removed at some point. There is no need to support for the new messages CLI if it's already deprecated. | 13:02 |
sheel | I'll see for client for volume | 13:02 |
sheel | scottda: yes, sounds good.. | 13:03 |
*** haplo37 has quit IRC | 13:03 | |
sheel | scottda: thanks .. | 13:03 |
*** iceyao has quit IRC | 13:03 | |
scottda | Thanks to you and ameade for working on this, sheel | 13:04 |
ameade | +1 | 13:04 |
sheel | scottda: :) | 13:04 |
jordanP | hi guys. Could someone have a look at https://review.openstack.org/#/c/310204/ (Assisted snapshot: use Keystone Session with Nova Client). It's needed to make Cinder compatible with Keystone v3. Thanks ! | 13:05 |
*** nnex has quit IRC | 13:11 | |
*** nnex has joined #openstack-cinder | 13:11 | |
*** savihou has joined #openstack-cinder | 13:15 | |
scottda | dulek: I have a simple question for you: What to do about extensions? | 13:15 |
scottda | dulek: Just kidding about being simple.... | 13:16 |
*** nnex has quit IRC | 13:16 | |
*** mtanino has joined #openstack-cinder | 13:17 | |
*** iceyao has joined #openstack-cinder | 13:18 | |
*** mriedem has joined #openstack-cinder | 13:20 | |
*** iceyao has quit IRC | 13:24 | |
*** porrua has joined #openstack-cinder | 13:27 | |
*** mtanino has quit IRC | 13:27 | |
*** iceyao has joined #openstack-cinder | 13:28 | |
*** salv-orlando has joined #openstack-cinder | 13:28 | |
*** jungleboyj has joined #openstack-cinder | 13:29 | |
*** salv-orlando has quit IRC | 13:31 | |
*** akerr has joined #openstack-cinder | 13:31 | |
*** jwcroppe has joined #openstack-cinder | 13:32 | |
*** cknight has joined #openstack-cinder | 13:33 | |
openstackgerrit | Sheel Rana proposed openstack/python-cinderclient: Support for cinder backup force delete https://review.openstack.org/313550 | 13:34 |
*** BigWillie has joined #openstack-cinder | 13:34 | |
*** gouthamr_ is now known as gouthamr | 13:34 | |
jgriffith | smcginnis: have you by chance noticed what this: https://review.openstack.org/#/c/300189/ does to the logs? | 13:35 |
*** Poornima has quit IRC | 13:35 | |
jgriffith | hmm... actually, may be something else | 13:36 |
*** iceyao has quit IRC | 13:36 | |
smcginnis | jgriffith: Where are the logs? | 13:36 |
*** iceyao has joined #openstack-cinder | 13:36 | |
smcginnis | Oh, mixed up. | 13:36 |
smcginnis | jgriffith: What are you seeing? | 13:36 |
jgriffith | grab a recent tempest run, look in c-vol and search on ERROR | 13:36 |
jgriffith | ooodles and oodles of these: http://paste.openstack.org/show/496331/ | 13:37 |
smcginnis | jgriffith: Not seeing anything on the tempest run of that patch. | 13:37 |
* smcginnis looking at paste | 13:37 | |
jgriffith | yeah, I don't think it's that patch that did it | 13:37 |
jgriffith | may not even be in Cinder honestly | 13:38 |
jgriffith | I'll look at it when I get in office. Something in my CI blew up last night and it looks like brick isn't working | 13:38 |
smcginnis | jgriffith: That looks like the warning that and all the other UUID patches were trying to fix. | 13:38 |
jgriffith | noticed when searching log files for "Error" all of those warnings | 13:38 |
geguileo | jgriffith: Those warnings come from a change in oslo versioned object | 13:38 |
jgriffith | geguileo: yeah, just saw that | 13:38 |
geguileo | jgriffith: There's been a work in progress by eharney and tbarron to remove them | 13:39 |
jgriffith | glad we have that in place :) Even more glad that eharney and tbarron are fixing it | 13:39 |
geguileo | jgriffith: They've removed a lot of them | 13:39 |
smcginnis | geguileo: I believe those are all merged now. | 13:39 |
jgriffith | missed some :) | 13:39 |
smcginnis | So this invalid UUID must be sneaking in somewhere else. | 13:39 |
geguileo | smcginnis: Yes, they merged, I just don't know if that removed all warnings | 13:39 |
smcginnis | geguileo: Apparently not. :) | 13:39 |
geguileo | I encountered one more yesterday in one of my patches | 13:40 |
geguileo | And I fixed it in that patch | 13:40 |
geguileo | But I don't remember on which one of my patches... | 13:40 |
smcginnis | jgriffith: What's your CI failure. Maybe we can share the misery. Mines still not 100%. | 13:40 |
smcginnis | geguileo: Awesome. | 13:40 |
geguileo | Give me a sec | 13:40 |
smcginnis | Guess it might take a little while to shake it all out. | 13:40 |
jgriffith | smcginnis: 2016-05-06 12:37:27.734 30336 ERROR oslo_messaging.rpc.dispatcher FailedISCSITargetPortalLogin: Could not login to any iSCSI portal. | 13:41 |
openstackgerrit | Sean McGinnis proposed openstack/cinder: Change provider_id to StringField https://review.openstack.org/309508 | 13:41 |
jgriffith | brick connector.py get_potential_volume_paths is broken maybe? | 13:42 |
jgriffith | full logs here: http://54.164.167.86/solidfire-ci-logs/refs-changes-38-147738-35/logs/c-vol.log.txt | 13:42 |
jgriffith | Started last night | 13:42 |
smcginnis | jgriffith: Not an issue on your array or network issue? | 13:42 |
jgriffith | hush | 13:43 |
*** jwcroppe_ has joined #openstack-cinder | 13:43 | |
smcginnis | :) | 13:43 |
*** sdague has quit IRC | 13:43 | |
smcginnis | Maybe new NetApp network security rules got put in place. No iSCSI traffic allowed in the lab. :P | 13:44 |
jgriffith | LOL... sounds about right | 13:44 |
ameade | lol | 13:46 |
geguileo | jgriffith: smcginnis Found it: https://review.openstack.org/#/c/259429/23/cinder/tests/unit/api/fakes.py | 13:46 |
ameade | >.< | 13:46 |
geguileo | That's the one that was giving me trouble with the UUIDs | 13:46 |
*** jwcroppe has quit IRC | 13:46 | |
smcginnis | ameade: I only joke because it's not too far from the truth here. :) | 13:46 |
smcginnis | geguileo: Nice! | 13:47 |
*** iceyao has quit IRC | 13:47 | |
jgriffith | geguileo: very cool | 13:47 |
ameade | smcginnis: I'm not saying it isnt a fair statement lmao | 13:47 |
smcginnis | ;) | 13:47 |
geguileo | It's not merged yet because it's in one of the API races patches | 13:47 |
*** ducttape_ has joined #openstack-cinder | 13:47 | |
jgriffith | geguileo: here's a +A for your troubles | 13:47 |
geguileo | XD | 13:47 |
smcginnis | Beat me to it. | 13:48 |
geguileo | It's the 3rd patch in the series, so it won't merge yet :-( | 13:48 |
smcginnis | Really with gerrit would make it easy to see dependency chains. | 13:48 |
jgriffith | although my vote has nothing to do with the message (not sure that will fix it anyway) | 13:48 |
jgriffith | geguileo: I'm starting to feel bad watching you have to rebase every day | 13:48 |
geguileo | rofl | 13:49 |
geguileo | I'm mostly afraid that I'll introduce a regression in one of the rebases :-( | 13:49 |
*** crose has joined #openstack-cinder | 13:52 | |
*** chris_morrell has quit IRC | 13:52 | |
*** jwcroppe has joined #openstack-cinder | 13:54 | |
*** edmondsw has joined #openstack-cinder | 13:55 | |
*** jgregor has joined #openstack-cinder | 13:56 | |
*** dustins has joined #openstack-cinder | 13:56 | |
*** dansmith is now known as snotdan | 13:56 | |
*** chuck__ is now known as zul | 13:57 | |
*** akshai has quit IRC | 13:57 | |
*** jwcroppe_ has quit IRC | 13:57 | |
*** iceyao has joined #openstack-cinder | 13:58 | |
*** jkraj has joined #openstack-cinder | 13:58 | |
*** snotdan is now known as superdan | 13:58 | |
*** akshai has joined #openstack-cinder | 13:59 | |
*** sdake_ has joined #openstack-cinder | 14:00 | |
*** ametts has joined #openstack-cinder | 14:00 | |
*** martyturner has joined #openstack-cinder | 14:00 | |
*** jdillaman has quit IRC | 14:00 | |
*** lprice has quit IRC | 14:01 | |
jgriffith | geguileo: oh geez, don't even think it! | 14:01 |
geguileo | I'll try :-) | 14:01 |
jgriffith | geguileo: there are down sides to splitting work up in to dep patches | 14:01 |
*** baumann has joined #openstack-cinder | 14:02 | |
geguileo | jgriffith: Yeah, but in general it's usually better, because reviews can be done faster and more in depth | 14:02 |
geguileo | Or at least that's whay I think | 14:02 |
*** yangyapeng has quit IRC | 14:02 | |
*** maestro has quit IRC | 14:03 | |
jgriffith | I agree with you | 14:03 |
jgriffith | for me it's more so I can kinda keep in my head what I'm working on | 14:03 |
*** sdake has quit IRC | 14:04 | |
*** alejandrito has quit IRC | 14:05 | |
*** catintheroof has joined #openstack-cinder | 14:05 | |
*** ducttape_ has quit IRC | 14:08 | |
*** mtanino has joined #openstack-cinder | 14:09 | |
*** jkraj has quit IRC | 14:11 | |
*** iceyao has quit IRC | 14:13 | |
-openstackstatus- NOTICE: Zuul is currently recovering from a large number of changes, it will take a few hours until your job is processed. Please have patience and enjoy a great weekend! | 14:15 | |
*** catintheroof has quit IRC | 14:15 | |
guitarzan | smcginnis: geguileo jgriffith we're going to get rid of requiring UUIDs in most of those fields right? | 14:15 |
geguileo | guitarzan: No | 14:15 |
guitarzan | geguileo: why? | 14:15 |
*** salv-orlando has joined #openstack-cinder | 14:15 | |
geguileo | guitarzan: What we are doing is making them have real UUIDs | 14:15 |
guitarzan | they can't be forced to uuids | 14:15 |
guitarzan | existing installs have data that aren't UUIDS | 14:16 |
geguileo | guitarzan: As the id pattern? | 14:16 |
smcginnis | guitarzan: For which fields? | 14:16 |
guitarzan | we're validating data that isn't actually of that type | 14:16 |
*** jwcroppe has quit IRC | 14:16 | |
guitarzan | smcginnis: project_id is the one that I know of for sure | 14:16 |
geguileo | guitarzan: I think there was a discussion about it somewhere | 14:16 |
smcginnis | guitarzan: There's my patch for provider_id: https://review.openstack.org/#/c/309508/ | 14:16 |
smcginnis | I haven't seen anything yet for project_id. | 14:17 |
geguileo | guitarzan: And if I remember correctly it was decided that we would use UUIDs | 14:17 |
guitarzan | geguileo: you're just saying that people can't upgrade to N | 14:17 |
*** links has quit IRC | 14:17 | |
jgriffith | guitarzan: that is a bit of a problem | 14:17 |
smcginnis | geguileo: Right, I vaguely remember that discussion and I thought it was decided project_id really should be UUID. | 14:17 |
smcginnis | But no idea why now. | 14:17 |
*** jwcroppe has joined #openstack-cinder | 14:17 | |
guitarzan | smcginnis: yes, but existing projects in our db AREN'T uuids | 14:17 |
jgriffith | guitarzan: hey EVERYONE change your project ID etc now! | 14:17 |
smcginnis | If there are deployments not using UUIDs, then we probaby should switch that to StringField as well. | 14:18 |
guitarzan | smcginnis: yeah, that's all I'm saying | 14:18 |
smcginnis | I don't see any strong reason why that would need to be handled as a UUID. | 14:18 |
geguileo | guitarzan: It looks like we are changing the project ID type, but other things like volume IDs will still be checked for valid UUIDs | 14:18 |
*** raunak has joined #openstack-cinder | 14:18 | |
geguileo | s/project/provider | 14:19 |
geguileo | guitarzan: So your project's ID is not a UUID? | 14:19 |
guitarzan | geguileo: historically, project ids were not uuids | 14:19 |
guitarzan | they could be anything | 14:20 |
*** iceyao has joined #openstack-cinder | 14:20 | |
jgriffith | guitarzan: you seem to be under the impression that any of us care to consider or learn from history :) | 14:20 |
jgriffith | guitarzan: that's not entirely accurate :) | 14:20 |
guitarzan | jgriffith: true, I assume they had to be strings :) | 14:21 |
guitarzan | probably even ascii strings | 14:21 |
jgriffith | guitarzan: indeed! | 14:21 |
geguileo | guitarzan: So you have project_ids that are not UUIDs, right? | 14:21 |
guitarzan | geguileo: yes | 14:21 |
geguileo | guitarzan: Then we need to change versioned objects | 14:22 |
geguileo | guitarzan: If there's not a bug for that yet you can create one and I'll fix it | 14:22 |
guitarzan | yes, I'm looking up nova's object field definitions | 14:22 |
geguileo | guitarzan: Yes, they have them as strings in the OVOs | 14:23 |
geguileo | guitarzan: So we need to fix them | 14:23 |
guitarzan | geguileo: I'll see if I can somehow run our data against master and see if there are other fields with invalid assumptions | 14:24 |
geguileo | guitarzan: That would be great!! | 14:25 |
*** gouthamr has quit IRC | 14:25 | |
geguileo | guitarzan: Please let me know the results | 14:25 |
*** jungleboyj has quit IRC | 14:25 | |
*** ducttape_ has joined #openstack-cinder | 14:25 | |
*** r-daneel has joined #openstack-cinder | 14:26 | |
*** iceyao has quit IRC | 14:27 | |
*** karthikp has joined #openstack-cinder | 14:27 | |
*** catintheroof has joined #openstack-cinder | 14:27 | |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Fix uncaught NotFound exceptions https://review.openstack.org/313571 | 14:28 |
*** takedakn has quit IRC | 14:29 | |
*** ducttape_ has quit IRC | 14:29 | |
*** lprice has joined #openstack-cinder | 14:29 | |
*** jwcroppe has quit IRC | 14:30 | |
*** EinstCrazy has quit IRC | 14:30 | |
*** dave-mccowan has joined #openstack-cinder | 14:35 | |
tbarron | i would think that what user_id field should be parallel to project_id | 14:36 |
tbarron | seems to me there should be a conversation with keystone folks about these two fields | 14:36 |
smcginnis | guitarzan: Is that the case for you? ^^ | 14:36 |
geguileo | tbarron: Agreed | 14:37 |
geguileo | tbarron: I'm just writing a patch to fix them | 14:37 |
*** iceyao has joined #openstack-cinder | 14:37 | |
guitarzan | smcginnis: I'm not sure about user_id | 14:37 |
geguileo | tbarron: And I've detected a problem in one of our ORM models :''-( | 14:37 |
*** jwcroppe has joined #openstack-cinder | 14:37 | |
smcginnis | geguileo: Users? I already started project_id too. :) | 14:37 |
geguileo | guitarzan: According to the ORM model yes | 14:37 |
smcginnis | Just running unit tests. | 14:37 |
geguileo | smcginnis: I just finished running them ;-P | 14:37 |
smcginnis | geguileo: OK, you win. :) | 14:38 |
geguileo | Jeje, thanks :-) | 14:38 |
tbarron | better you guys than me, I'm not touching uuids :) | 14:38 |
smcginnis | tbarron: Hah! | 14:38 |
guitarzan | smcginnis: yeah, we have uuid and non uuid user_ids | 14:38 |
smcginnis | guitarzan: OK, thanks. | 14:38 |
smcginnis | geguileo: Want to take care of both? | 14:39 |
geguileo | smcginnis: Ok | 14:39 |
geguileo | smcginnis: I was going to change both in the same patch | 14:39 |
geguileo | smcginnis: I've detected that we have a problem with the latest Message ORM model | 14:39 |
geguileo | smcginnis: Because it uses 36 as the size of the project_id | 14:40 |
*** raunak has quit IRC | 14:40 | |
smcginnis | geguileo: Too small? | 14:41 |
*** xinli has joined #openstack-cinder | 14:41 | |
geguileo | smcginnis: Well, it's inconsistent with the 255 characters defined everywhere else | 14:41 |
geguileo | smcginnis: 36 is when we are only using UUID4 | 14:41 |
smcginnis | Fine if it's just a UUID, but yeah, if we're allowing any string for project_id it should probably be large enough to accomodate whatever's needed. | 14:42 |
geguileo | And be consistent with the other models | 14:43 |
smcginnis | Yeah, consistency is good. | 14:43 |
*** e0ne has quit IRC | 14:43 | |
*** e0ne has joined #openstack-cinder | 14:44 | |
-openstackstatus- NOTICE: Zuul has been restarted. As a results, we only preserved patches in the gate queue. Be sure to recheck your patches in gerrit if needed. | 14:45 | |
*** itlinux has joined #openstack-cinder | 14:46 | |
*** Lee1092 has joined #openstack-cinder | 14:47 | |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Change OVOs project_id and user_id to StringField https://review.openstack.org/313584 | 14:47 |
*** takedakn has joined #openstack-cinder | 14:47 | |
*** jwcroppe_ has joined #openstack-cinder | 14:48 | |
*** itlinux has quit IRC | 14:51 | |
*** jwcroppe has quit IRC | 14:52 | |
*** yangxi has joined #openstack-cinder | 14:54 | |
*** chris_morrell has joined #openstack-cinder | 14:56 | |
*** yangxi has quit IRC | 14:57 | |
*** eharney has joined #openstack-cinder | 14:57 | |
*** sdake_ has quit IRC | 14:57 | |
*** akapil has joined #openstack-cinder | 14:58 | |
*** nnex has joined #openstack-cinder | 14:58 | |
*** ducttape_ has joined #openstack-cinder | 14:59 | |
*** haplo37 has joined #openstack-cinder | 14:59 | |
*** lcurtis has joined #openstack-cinder | 15:00 | |
*** akshai has quit IRC | 15:01 | |
*** smcginnis_ has joined #openstack-cinder | 15:02 | |
*** nnex has quit IRC | 15:03 | |
*** Guest42930 has quit IRC | 15:05 | |
*** david-lyle has joined #openstack-cinder | 15:06 | |
*** ociuhandu has joined #openstack-cinder | 15:07 | |
*** nnex has joined #openstack-cinder | 15:07 | |
*** arch-nemesis has joined #openstack-cinder | 15:07 | |
akerr | don't suppose anyone has seen some of the scenario tests fail because Nova couldn't find a valid host with the following error: Require both a host and instance NUMA topology to fit instance on host? | 15:10 |
*** jwcroppe has joined #openstack-cinder | 15:12 | |
*** sdake has joined #openstack-cinder | 15:16 | |
*** jwcroppe_ has quit IRC | 15:16 | |
*** jwcroppe has quit IRC | 15:17 | |
*** iceyao has quit IRC | 15:19 | |
*** laughterwym has quit IRC | 15:21 | |
*** laughterwym has joined #openstack-cinder | 15:22 | |
openstackgerrit | Xinli Guan proposed openstack/cinder: Make IBM drivers return snapshot object for cg ops https://review.openstack.org/268283 | 15:22 |
*** itlinux has joined #openstack-cinder | 15:22 | |
*** martyturner has quit IRC | 15:23 | |
*** iceyao has joined #openstack-cinder | 15:24 | |
*** jdillaman has joined #openstack-cinder | 15:24 | |
*** akshai has joined #openstack-cinder | 15:25 | |
*** laughterwym has quit IRC | 15:26 | |
*** chris_morrell has quit IRC | 15:27 | |
*** itzdilip has joined #openstack-cinder | 15:27 | |
*** iceyao has quit IRC | 15:30 | |
*** iceyao_ has joined #openstack-cinder | 15:30 | |
*** jordanP has quit IRC | 15:31 | |
*** sdake has quit IRC | 15:32 | |
openstackgerrit | xing-yang proposed openstack/cinder-specs: Generic Volume Group https://review.openstack.org/303893 | 15:33 |
*** sasukeh has joined #openstack-cinder | 15:34 | |
openstackgerrit | Merged openstack/cinder-specs: Making doc fixes in backup-snapshots.rst https://review.openstack.org/313237 | 15:34 |
*** martyturner has joined #openstack-cinder | 15:35 | |
*** catintheroof has quit IRC | 15:36 | |
*** markus_z has quit IRC | 15:38 | |
*** sasukeh has quit IRC | 15:38 | |
*** nnex has quit IRC | 15:42 | |
*** hemnafk is now known as hemna | 15:43 | |
*** leeantho has joined #openstack-cinder | 15:43 | |
*** salv-orlando has quit IRC | 15:43 | |
*** leeantho_ has joined #openstack-cinder | 15:44 | |
openstackgerrit | Eric Harney proposed openstack/cinder: NFS snapshots https://review.openstack.org/147186 | 15:46 |
*** salv-orlando has joined #openstack-cinder | 15:47 | |
*** leeantho has quit IRC | 15:47 | |
*** dave-mccowan has quit IRC | 15:50 | |
*** nnex has joined #openstack-cinder | 15:52 | |
*** raunak has joined #openstack-cinder | 15:53 | |
*** enriquetaso has joined #openstack-cinder | 15:54 | |
*** akapil has quit IRC | 15:55 | |
hemna | anyone know who the Smaug guy was in our Cinder meetings in Austin ? | 15:57 |
smcginnis | Saggi | 15:57 |
hemna | https://github.com/openstack/smaug/graphs/contributors | 15:58 |
hemna | only 6 contributors so far | 15:58 |
smcginnis | I think there are a lot of folks involved, but only a few code committers so far. | 15:58 |
*** nnex has quit IRC | 15:59 | |
*** iceyao_ has quit IRC | 16:00 | |
xyang1 | hemna: Saggi Mizrahi was at the cinder meeting | 16:02 |
*** lprice has quit IRC | 16:02 | |
hemna | ok thanks | 16:03 |
xyang1 | smcginnis: oh, did not see you already replied:) | 16:03 |
smcginnis | This might be interesting if you didn't see it: https://www.youtube.com/watch?v=_tVYuW_YMB8 | 16:03 |
smcginnis | xyang1: ;) | 16:03 |
*** sasukeh has joined #openstack-cinder | 16:05 | |
openstackgerrit | Sheel Rana proposed openstack/python-cinderclient: Support for cinder backup force delete https://review.openstack.org/313550 | 16:08 |
*** johnthetubaguy has quit IRC | 16:08 | |
*** johnthetubaguy has joined #openstack-cinder | 16:08 | |
*** apoorvad has joined #openstack-cinder | 16:14 | |
*** sdake has joined #openstack-cinder | 16:15 | |
*** yangyapeng has joined #openstack-cinder | 16:17 | |
*** yangyapeng has quit IRC | 16:18 | |
*** crose has quit IRC | 16:20 | |
*** frontrunner has joined #openstack-cinder | 16:21 | |
*** e0ne has quit IRC | 16:23 | |
*** bluex has quit IRC | 16:25 | |
*** sasukeh has quit IRC | 16:25 | |
*** salv-orlando has quit IRC | 16:25 | |
*** cebruns has quit IRC | 16:26 | |
*** catintheroof has joined #openstack-cinder | 16:27 | |
*** karthikp has quit IRC | 16:27 | |
*** karthikp has joined #openstack-cinder | 16:27 | |
*** cebruns has joined #openstack-cinder | 16:27 | |
*** mpjetta has quit IRC | 16:28 | |
*** anish has quit IRC | 16:28 | |
*** takedakn has quit IRC | 16:28 | |
*** mtreinish has quit IRC | 16:28 | |
*** anish has joined #openstack-cinder | 16:29 | |
*** jidar has quit IRC | 16:29 | |
*** baumann has left #openstack-cinder | 16:30 | |
*** bluex has joined #openstack-cinder | 16:32 | |
*** SergeyLukjanov has quit IRC | 16:32 | |
*** mpjetta has joined #openstack-cinder | 16:32 | |
*** yuriy_n17 has quit IRC | 16:33 | |
*** SergeyLukjanov has joined #openstack-cinder | 16:33 | |
*** bardia has joined #openstack-cinder | 16:33 | |
*** mtreinish has joined #openstack-cinder | 16:35 | |
*** jidar has joined #openstack-cinder | 16:39 | |
*** yangyapeng has joined #openstack-cinder | 16:42 | |
*** sdake_ has joined #openstack-cinder | 16:42 | |
openstackgerrit | Sheel Rana proposed openstack/python-cinderclient: Support for cinder backup force delete https://review.openstack.org/313550 | 16:45 |
*** lprice has joined #openstack-cinder | 16:45 | |
*** sdake has quit IRC | 16:45 | |
*** yangyapeng has quit IRC | 16:47 | |
*** takedakn has joined #openstack-cinder | 16:47 | |
*** sgotliv has joined #openstack-cinder | 16:49 | |
*** garthb has joined #openstack-cinder | 16:50 | |
*** jgregor has quit IRC | 16:58 | |
openstackgerrit | xing-yang proposed openstack/cinder-specs: Replication with More Granularity (Tiramisu) https://review.openstack.org/229722 | 17:01 |
*** jwcroppe has joined #openstack-cinder | 17:05 | |
openstackgerrit | Karthik Prabhu Vinod proposed openstack/cinder: Add ServiceTopic & ServiceDisabledReason Enum https://review.openstack.org/313179 | 17:05 |
*** jseiler__ has joined #openstack-cinder | 17:06 | |
*** jistr has quit IRC | 17:07 | |
*** jwcroppe_ has joined #openstack-cinder | 17:07 | |
*** jseiler_ has quit IRC | 17:09 | |
guitarzan | hmm, so one problem with that versioned objects fields warning is it doesn't know the most important piece of information... which field it's complaining about | 17:09 |
*** jwcroppe has quit IRC | 17:10 | |
*** akapil has joined #openstack-cinder | 17:12 | |
smcginnis | guitarzan: Hah! | 17:12 |
smcginnis | guitarzan: Guess that would make it too easy. ;) | 17:12 |
jgriffith | guitarzan: and thus my frustration with the message this morning :( | 17:13 |
guitarzan | yep | 17:13 |
guitarzan | also, the rpc version pinning seems a little odd | 17:14 |
jgriffith | guitarzan: I'm also unclear on why internal issues like that are something the operator should ever care about or see | 17:14 |
jgriffith | guitarzan: in other words, that's a dev problem I think | 17:14 |
guitarzan | it doesn't seem to auto-downgrade the rpc version, even though it knows it needs to | 17:14 |
guitarzan | "RPCVersionCapError: Requested message version, 2.0 is incompatible. It needs to be equal in major version and less than or equal in minor version as the specified version cap 1.8." | 17:15 |
guitarzan | dulek: should I expect that to work yet? | 17:15 |
guitarzan | jgriffith: it is, but the devs are the ones breaking it :D | 17:15 |
guitarzan | somebody has to be able to find the bugs | 17:16 |
guitarzan | and you won't find that with devstack | 17:16 |
jgriffith | guitarzan: good point | 17:16 |
jgriffith | guitarzan: sounds like you need Jay to write a new hacking check :) | 17:16 |
guitarzan | "code changed, failed hacking check' | 17:17 |
*** dave-mccowan has joined #openstack-cinder | 17:17 | |
*** uberjay_ has quit IRC | 17:18 | |
*** haplo37 has quit IRC | 17:18 | |
*** uberjay has joined #openstack-cinder | 17:20 | |
guitarzan | geguileo: from a simple "volume get" from old production data, project_id and user_id seem to be the only complaints | 17:20 |
jgriffith | guitarzan: seems doable | 17:20 |
jgriffith | :) | 17:20 |
tbarron | guitarzan: good | 17:21 |
*** ducttape_ has quit IRC | 17:21 | |
*** jwcroppe has joined #openstack-cinder | 17:21 | |
guitarzan | tbarron: yep, it's not too bad | 17:21 |
*** jwcroppe_ has quit IRC | 17:25 | |
*** uberjay has quit IRC | 17:27 | |
*** lprice1 has joined #openstack-cinder | 17:31 | |
*** cFouts is now known as nug | 17:31 | |
*** nug is now known as Guest47652 | 17:32 | |
*** lprice has quit IRC | 17:33 | |
*** jwcroppe has quit IRC | 17:33 | |
*** sasukeh has joined #openstack-cinder | 17:34 | |
*** ducttape_ has joined #openstack-cinder | 17:37 | |
*** plastikman has left #openstack-cinder | 17:37 | |
*** rlrossit has quit IRC | 17:37 | |
*** baumann has joined #openstack-cinder | 17:39 | |
openstackgerrit | Sheel Rana proposed openstack/cinder: WADL to RST migration https://review.openstack.org/312113 | 17:40 |
*** sasukeh has quit IRC | 17:40 | |
*** jwcroppe has joined #openstack-cinder | 17:41 | |
*** Guest47652 is now known as cFouts | 17:41 | |
*** lpetrut has quit IRC | 17:43 | |
openstackgerrit | Merged openstack/cinder: Use utils.convert_str to convert HTTP header values https://review.openstack.org/304462 | 17:48 |
*** uberjay has joined #openstack-cinder | 17:50 | |
*** prometheanfire has joined #openstack-cinder | 17:51 | |
*** jwcroppe_ has joined #openstack-cinder | 17:53 | |
*** merooney has joined #openstack-cinder | 17:55 | |
*** jwcroppe has quit IRC | 17:56 | |
*** sdake_ is now known as sdake | 17:57 | |
*** jgregor has joined #openstack-cinder | 18:03 | |
*** jwcroppe has joined #openstack-cinder | 18:03 | |
*** jwcroppe_ has quit IRC | 18:07 | |
openstackgerrit | Kaitlin Farr proposed openstack/cinder: Replace key manager with Castellan https://review.openstack.org/280492 | 18:07 |
openstackgerrit | Kaitlin Farr proposed openstack/cinder: Specify key algorithm and size for create_key https://review.openstack.org/251503 | 18:07 |
*** gouthamr has joined #openstack-cinder | 18:08 | |
*** suyash has joined #openstack-cinder | 18:08 | |
ameade | https://review.openstack.org/#/c/299052/ "merge me ^.^" | 18:09 |
*** sgotliv has quit IRC | 18:10 | |
*** nnex has joined #openstack-cinder | 18:10 | |
*** nnex has quit IRC | 18:11 | |
ameade | https://review.openstack.org/#/c/300639/ "me too >.>" | 18:11 |
*** gouthamr_ has joined #openstack-cinder | 18:13 | |
*** mriedem1 has joined #openstack-cinder | 18:15 | |
*** mriedem has quit IRC | 18:17 | |
*** gouthamr has quit IRC | 18:17 | |
*** mriedem1 is now known as mriedem | 18:20 | |
*** rlrossit has joined #openstack-cinder | 18:23 | |
*** xinli has quit IRC | 18:24 | |
*** BigWillie has quit IRC | 18:24 | |
*** salv-orlando has joined #openstack-cinder | 18:24 | |
*** gouthamr_ is now known as gouthamr | 18:26 | |
*** BigWillie has joined #openstack-cinder | 18:26 | |
*** akapil has quit IRC | 18:30 | |
*** bwallis has joined #openstack-cinder | 18:30 | |
gouthamr | hi, would like some core reviewer attention on a patch: https://review.openstack.org/#/c/246990/ - it's been there too long and would like to get it merged if possible. | 18:30 |
*** sdake_ has joined #openstack-cinder | 18:31 | |
hemna | gouthamr, I don't see any chap settings in the cinder.conf | 18:31 |
hemna | gouthamr, http://goo.gl/jUXaOF | 18:31 |
geguileo | guitarzan: Thanks! | 18:32 |
geguileo | guitarzan: The patch is already under review: https://review.openstack.org/#/c/313584/ | 18:33 |
*** sdake has quit IRC | 18:33 | |
guitarzan | geguileo: ya, it took me a bit to get set up with prod data | 18:34 |
guitarzan | geguileo: taking a look now | 18:34 |
gouthamr | hemna: we're not testing chap on the CI jobs.. we had internal tests for it. i'll find out from akerr if we can do something about that | 18:35 |
geguileo | guitarzan: I imagined it would take you some time | 18:35 |
geguileo | guitarzan: Thank you for the effort | 18:35 |
guitarzan | geguileo: now I just need to ask dulek about the rpc upgrade stuff :) | 18:35 |
geguileo | guitarzan: What RPC upgrade stuff? | 18:35 |
geguileo | guitarzan: It will not be a problem | 18:35 |
smcginnis | I think he took an extended stay in Austin. Probably on his way back now. | 18:35 |
*** sasukeh has joined #openstack-cinder | 18:36 | |
geguileo | guitarzan: Because serialization of the UUID is transformed to a string | 18:36 |
geguileo | guitarzan: And the string is serialized as a string as well | 18:36 |
guitarzan | geguileo: ya, it's not related to this uuid stuff | 18:36 |
geguileo | guitarzan: Ok, maybe I can help you | 18:36 |
openstackgerrit | Sheel Rana proposed openstack/python-cinderclient: Support for cinder backup force delete https://review.openstack.org/313550 | 18:36 |
guitarzan | my upgrade from L->master didn't quite work right | 18:36 |
guitarzan | the c-vols wouldn't start due to the rpc versioning | 18:36 |
geguileo | guitarzan: They wouldn't start? r:-?? | 18:36 |
patrickeast | hemna: gouthamr: fwiw we don't make other drivers test both chap enabled and disabled | 18:37 |
guitarzan | I thought they'd downgrade to L versioning | 18:37 |
guitarzan | L rpc version | 18:37 |
geguileo | guitarzan: Or they would start with the wrong RPC version? | 18:37 |
guitarzan | geguileo: they fail to talk to the scheduler | 18:37 |
lprice1 | DuncanT: Would you mind taking another look at https://review.openstack.org/#/c/246990/ when you get a chance? | 18:37 |
geguileo | guitarzan: Any exception? | 18:37 |
guitarzan | geguileo: Requested message version, 2.0 is incompatible. It needs to be equal in major version and less than or equal in minor version as the specified version cap 1.8. | 18:37 |
*** jwcroppe_ has joined #openstack-cinder | 18:38 | |
*** jwcroppe has quit IRC | 18:38 | |
geguileo | guitarzan: Ok, that's because they are out of sync :-( | 18:38 |
guitarzan | can you explain that to me? | 18:38 |
guitarzan | master is trying to speak 2.0 | 18:38 |
guitarzan | L is 1.8 no? | 18:38 |
geguileo | guitarzan: I don't know from the top of my head | 18:39 |
*** dramakri has joined #openstack-cinder | 18:39 | |
guitarzan | I think it means the scheduler is speaking 1.8 | 18:39 |
guitarzan | maybe? | 18:39 |
guitarzan | but the volume service won't or can't downgrade? | 18:39 |
*** e0ne has joined #openstack-cinder | 18:39 | |
guitarzan | either way, it's not a show stopper for testing this uuid stuff | 18:39 |
*** savihou has quit IRC | 18:40 | |
geguileo | guitarzan: How did you do the upgrades? | 18:40 |
*** sasukeh has quit IRC | 18:41 | |
*** catintheroof has quit IRC | 18:41 | |
guitarzan | I just upgraded the source tree, blew away the venv and reinstalled | 18:41 |
guitarzan | cinder-manage db sync | 18:41 |
guitarzan | and go | 18:41 |
geguileo | Only 1 c-sch and 1 c-vol? | 18:42 |
guitarzan | yes, although the db has tons of service entries from prod | 18:42 |
akerr | patrickeast: hemna: gouthamr: yea, its not possible to run and report on all possible permutations of our driver's options. If we turn chap on then we're not testing when chap is off, etc | 18:42 |
guitarzan | so it wanting the old rpc version (1.8) doesn't surprise me | 18:42 |
*** Lee1092 has quit IRC | 18:42 | |
guitarzan | that was to be expected I think | 18:42 |
geguileo | guitarzan: And other entries are alive (as in not deleted)? | 18:43 |
guitarzan | geguileo: so I have a bunch of "down" c-vols | 18:43 |
guitarzan | yes, lots | 18:43 |
patrickeast | akerr: yea i ran into the same thing on my ci, for all patches we just picked the most common configuration and do the others internally | 18:43 |
geguileo | guitarzan: Ok, then the problem is that you are making it pin to an incompatible version | 18:43 |
*** ociuhandu has quit IRC | 18:43 | |
geguileo | guitarzan: It detects it as a rolling upgrade | 18:43 |
geguileo | guitarzan: From L to N | 18:43 |
geguileo | guitarzan: Which doesn't work | 18:43 |
geguileo | guitarzan: Rolling upgrades are from N-1 to N only | 18:44 |
guitarzan | geguileo: ah, that's interesting | 18:44 |
geguileo | guitarzan: http://docs.openstack.org/releasenotes/cinder/mitaka.html#known-issues | 18:44 |
guitarzan | geguileo: next time I'll try stable/liberty -> stable/mitaka just for fun | 18:44 |
geguileo | guitarzan: dulek included an explanation in Mitaka Release Notes Known issues | 18:45 |
guitarzan | I'm not sure I think that "Known Issue" really explains the problem | 18:45 |
guitarzan | the problem is the n-1 to n | 18:46 |
guitarzan | assuming you're right, which you most likely are :) | 18:46 |
geguileo | guitarzan: Well, n-1 to n plus the fact that the service list is not "up to date" | 18:46 |
*** mkrcmari__ has quit IRC | 18:46 | |
openstackgerrit | Sheel Rana proposed openstack/python-cinderclient: Support for cinder backup force delete https://review.openstack.org/313550 | 18:47 |
geguileo | guitarzan: If you try to do L to M, you may have some issues (need to restart the services again) until this patch merges: https://review.openstack.org/#/c/307105/ | 18:47 |
*** nnex has joined #openstack-cinder | 18:47 | |
geguileo | guitarzan: If you find any other problems with rolling upgrades and dulek is not around you can also ask me, I've fought it a little bit as well ;-) | 18:48 |
guitarzan | geguileo: great, will do, thanks | 18:49 |
geguileo | np | 18:50 |
akerr | ameade: (and anyone else who wants to review user messages tests) https://review.openstack.org/#/c/300639 | 18:51 |
guitarzan | geguileo: no surprse, the uuid warnings went away :) | 18:51 |
geguileo | XD | 18:51 |
geguileo | That's good :-) | 18:51 |
jgriffith | anybody tried testing with chap lately? Seems like /etc/iscsi/nodes is being written incorrectly??? | 18:52 |
jgriffith | weird | 18:52 |
patrickeast | jgriffith: doesn't the gate do iscsi chap w/ lvm? | 18:53 |
patrickeast | jgriffith: or chap isn't enabled by default? | 18:53 |
jgriffith | patrickeast: yeah, and that works which is strange... but I'm wondering if chap is broken | 18:54 |
jgriffith | patrickeast: it's my backend only that's seeing a problem it seems | 18:54 |
patrickeast | mm i tested out my changes to the initiator_data stuff yesterday and it seemed to be ok | 18:54 |
jgriffith | patrickeast: I looked at the nodes file and the username and secret entries are garbage | 18:54 |
*** bwallis has quit IRC | 18:54 | |
patrickeast | jgriffith: oh weird | 18:54 |
jgriffith | yeah... really strange | 18:54 |
patrickeast | jgriffith: got logs of the iscsiadm login commands? | 18:54 |
jgriffith | patrickeast: they all just time out | 18:55 |
jgriffith | patrickeast: because the credentials are wrong | 18:55 |
patrickeast | jgriffith: oh, thats not good | 18:55 |
jgriffith | patrickeast: but yeah... my 17 failures since last night in sfci are all out there | 18:55 |
jgriffith | maybe I'll check a different cluster out and try it | 18:55 |
jgriffith | really weird | 18:55 |
*** bwallis has joined #openstack-cinder | 18:56 | |
openstackgerrit | Evgeny Antyshev proposed openstack/cinder: Move test from test_glusterfs and test_quobyte to test_remotefs https://review.openstack.org/261549 | 18:56 |
openstackgerrit | Evgeny Antyshev proposed openstack/cinder: vzstorage: add ploop volume format support https://review.openstack.org/262246 | 18:56 |
openstackgerrit | Evgeny Antyshev proposed openstack/cinder: vzstorage: fix create/delete snapshots of in-use volumes https://review.openstack.org/276465 | 18:56 |
openstackgerrit | Evgeny Antyshev proposed openstack/cinder: Use versionedobjects in remotefs.py https://review.openstack.org/261558 | 18:56 |
patrickeast | jgriffith: the sf ci passed on this one at ~1:30 AM https://review.openstack.org/#/c/313200/ | 18:57 |
jgriffith | patrickeast: yeah, that's the last pass I believe | 18:57 |
jgriffith | patrickeast: oh... hmmm | 18:57 |
jgriffith | patrickeast: http://54.164.167.86/solidfire-ci-logs/refs-changes-27-312427-5/logs/c-vol.log.txt | 18:58 |
jgriffith | I'll have to trace back and see exactly when this started, I did apparanty fill the max number of volumes for my little virt cluster last night | 18:58 |
mtanino | sheel: hi, | 18:59 |
jgriffith | but I'm trying to run this manually right now and am seeing the problem | 18:59 |
sheel | mtanino: hi | 18:59 |
*** jwcroppe has joined #openstack-cinder | 18:59 | |
mtanino | sheel: nit question. Can't we run force-delete when the volume is avairable? | 18:59 |
patrickeast | jgriffith: hmm well you can see it doing the chap username/password setting commands like usual in that log | 19:00 |
mtanino | sheel: your comment mentioned above | 19:00 |
jgriffith | patrickeast: yeah, it does them, but they're wrong :) | 19:00 |
jgriffith | go figure | 19:00 |
patrickeast | haha | 19:00 |
sheel | mtanino: sorry i could not understand your question | 19:00 |
*** lprice1 has quit IRC | 19:01 | |
sheel | mtanino: ah ok, i got it | 19:01 |
mtanino | sheel: line 1466 https://review.openstack.org/#/c/313550/5/cinderclient/v3/shell.py | 19:01 |
mtanino | sheel: allows deleting .... | 19:01 |
sheel | mtanino: actually force delete is specific to states other than available and error | 19:01 |
sheel | mtanino: this can be run for sure in available state as well | 19:02 |
sheel | mtanino: but for Available and error state we can simple run backup delete, then we dont need force delete | 19:02 |
*** jwcroppe_ has quit IRC | 19:02 | |
mtanino | sheel: ah, ok. I understand the intention. | 19:02 |
sheel | mtanino: cool.. happy to answer... | 19:03 |
mtanino | sheel: just curious. | 19:03 |
sheel | mtanino: no problem, always welcome | 19:03 |
*** lprice has joined #openstack-cinder | 19:03 | |
sheel | thanks for review | 19:03 |
*** rlrossit has quit IRC | 19:06 | |
*** asselin has joined #openstack-cinder | 19:07 | |
mtanino | sheel: oh, you edit code on the browser directly instead of using git commit... | 19:07 |
mtanino | git review | 19:07 |
sheel | mtanino: yep... i was playing with gerrit.. | 19:07 |
sheel | UI | 19:07 |
mtanino | seems difficult... | 19:08 |
sheel | :) | 19:08 |
*** dave-mccowan has quit IRC | 19:09 | |
*** xinli has joined #openstack-cinder | 19:09 | |
*** e0ne has quit IRC | 19:11 | |
*** ociuhandu has joined #openstack-cinder | 19:12 | |
*** nnex has quit IRC | 19:12 | |
openstackgerrit | Alex Meade proposed openstack/cinder: Add prefix to user message event ids https://review.openstack.org/313140 | 19:24 |
*** jwcroppe has quit IRC | 19:24 | |
*** martyturner has quit IRC | 19:24 | |
*** sdake has joined #openstack-cinder | 19:25 | |
*** dave-mccowan has joined #openstack-cinder | 19:25 | |
*** sdake has quit IRC | 19:25 | |
*** rcernin has joined #openstack-cinder | 19:26 | |
*** ducttape_ has quit IRC | 19:26 | |
*** sdake_ has quit IRC | 19:26 | |
*** raunak has quit IRC | 19:27 | |
*** jwcroppe has joined #openstack-cinder | 19:28 | |
*** mkrcmari__ has joined #openstack-cinder | 19:31 | |
*** yangyapeng has joined #openstack-cinder | 19:31 | |
*** bardia has quit IRC | 19:31 | |
*** bardia has joined #openstack-cinder | 19:32 | |
*** yangyapeng has quit IRC | 19:35 | |
*** sasukeh has joined #openstack-cinder | 19:36 | |
*** salv-orlando has quit IRC | 19:39 | |
*** timcl has quit IRC | 19:40 | |
*** apoorvad has quit IRC | 19:40 | |
*** sasukeh has quit IRC | 19:41 | |
*** salv-orlando has joined #openstack-cinder | 19:42 | |
*** akerr has quit IRC | 19:43 | |
sheel | ameade: hi | 19:43 |
sheel | ameade: I think we need not of upgrade scripts here | 19:43 |
sheel | ameade: in this patch ^^ | 19:44 |
*** ducttape_ has joined #openstack-cinder | 19:45 | |
sheel | ameade: as no cinder release is done and impact of name of eventIDs is very less which will later be deleted once expiry time is reached... we need not to add upgrade scripts :) | 19:45 |
*** bwallis has quit IRC | 19:45 | |
*** rlrossit has joined #openstack-cinder | 19:48 | |
*** xinli has quit IRC | 19:51 | |
openstackgerrit | Patrick East proposed openstack/cinder: Specify a user-agent in Pure volume drivers https://review.openstack.org/313679 | 19:56 |
*** sage has quit IRC | 19:58 | |
*** xinli has joined #openstack-cinder | 19:59 | |
*** sage has joined #openstack-cinder | 20:00 | |
openstackgerrit | Tom Barron proposed openstack/cinder: Remove deprecated cinder RequestBodySizeLimiter https://review.openstack.org/313682 | 20:01 |
*** jgregor has quit IRC | 20:01 | |
*** dramakrishna has joined #openstack-cinder | 20:06 | |
*** lprice1 has joined #openstack-cinder | 20:07 | |
*** mvk_ has joined #openstack-cinder | 20:08 | |
*** lprice has quit IRC | 20:09 | |
*** mkrcmari__ has quit IRC | 20:12 | |
*** salv-orlando has quit IRC | 20:13 | |
*** apoorvad has joined #openstack-cinder | 20:13 | |
openstackgerrit | Merged openstack/cinder-specs: Add Stochastic Weighing Scheduler https://review.openstack.org/307401 | 20:15 |
scottda | How do we fix this: | 20:15 |
scottda | https://www.irccloud.com/pastebin/BaGVcFct/ | 20:16 |
scottda | Collecting cryptography>=1.1 (from paramiko>=1.16.0->tempest-lib>=0.14.0->-r /home/jenkins/workspace/gate-python-cinderclient-pypy/test-requirements.txt (line 16)) | 20:16 |
scottda | RuntimeError: cryptography 1.0 is not compatible with PyPy < 2.6. Please upgrade PyPy to use this library. | 20:16 |
smcginnis_ | scottda, eewww | 20:17 |
sheel | yes I saw this in non voting tests today... | 20:17 |
sheel | though this is not blocker but still good to fix | 20:17 |
eharney | scottda: we still have a pypy job? | 20:17 |
*** dramakri has quit IRC | 20:18 | |
*** dramakrishna is now known as dramakri | 20:18 | |
scottda | eharney: Yes, but the job is gate-python-cinderclient-pypy, so maybe we shouldn't have it... | 20:18 |
scottda | I reckon the job is to verify packages are in pypy? | 20:18 |
smcginnis_ | I think client is the only think that potentially needs to be compatible with pypy. | 20:18 |
smcginnis_ | pypy, not pypi. | 20:18 |
openstackgerrit | Merged openstack/cinder-specs: Implement the oslo_db enginefacade proposal https://review.openstack.org/270601 | 20:19 |
smcginnis_ | Interpreter. | 20:19 |
scottda | I'm not sure, but hemna told me it's pronounced Pee-pee | 20:19 |
smcginnis_ | :D | 20:19 |
eharney | IMO if we want to test w/ pypy, we should bump up to a more current version | 20:20 |
scottda | Who controls the version? us or infra? | 20:20 |
eharney | presumably project-config | 20:20 |
smcginnis_ | It should be _at least_ 2.6 or greater I would think. | 20:22 |
eharney | well keep in mind that those aren't related to python versions | 20:22 |
smcginnis_ | True | 20:23 |
smcginnis_ | Trusty ships with 2.2.1. :/ | 20:23 |
smcginnis_ | Not sure about RH versions. Nothing on their web site I can see. | 20:24 |
eharney | i don't think we use it for anything openstack-related | 20:24 |
smcginnis_ | No, but we have broader compatibility requirements for the client. | 20:25 |
eharney | right | 20:25 |
smcginnis_ | But if a critical library doesn't work with the older versions, then I think we either need to raise the minimum... | 20:25 |
smcginnis_ | Or remove support for it. | 20:26 |
*** ducttape_ has quit IRC | 20:26 | |
eharney | i think "critical" is questionable here since it came from paramiko/tempest-lib | 20:26 |
eharney | the client doesn't actually _use_ it, right? | 20:26 |
*** prometheanfire has left #openstack-cinder | 20:27 | |
*** garthb_ has joined #openstack-cinder | 20:27 | |
eharney | just tempest.lib does | 20:27 |
smcginnis_ | Not sure. Seems like something like cryptography would be necessary internally to paramiko. | 20:28 |
eharney | right, but paramiko is not necessary for cinderclient | 20:28 |
smcginnis_ | Oh, I see what you're saying. I guess not. | 20:29 |
*** garthb has quit IRC | 20:29 | |
*** e0ne has joined #openstack-cinder | 20:29 | |
eharney | still a problem, but a tempest/CI problem is different from a client problem | 20:29 |
smcginnis_ | Nope, not just tests: https://github.com/openstack/cinder/blob/9567844f0e180a7278767fdcd4378b19913933be/cinder/ssh_utils.py#L103 | 20:30 |
eharney | yeah, we have drivers using paramiko | 20:30 |
*** BigWillie has quit IRC | 20:30 | |
eharney | but if we aren't supporting pypy there then there isn't much to worry about | 20:30 |
smcginnis_ | Derp, just realized I was looking in Cinder and not python-cinderclient. :) | 20:31 |
smcginnis_ | Guess it's getting too late on a Friday for me. | 20:31 |
*** dramakri has left #openstack-cinder | 20:31 | |
scottda | So, if we're not supporting pypy, should be remove this job? | 20:31 |
scottda | s/be/we | 20:32 |
eharney | well, i think we are (supposedly) supporting it for the client | 20:32 |
smcginnis_ | I think we do need to support pypy. But not sure as far as restricting version. | 20:32 |
*** openstackgerrit has quit IRC | 20:32 | |
*** openstackgerrit has joined #openstack-cinder | 20:33 | |
eharney | scottda: where is that job? | 20:33 |
scottda | I'm still looking for where... | 20:34 |
*** dramakri has joined #openstack-cinder | 20:34 | |
scottda | I don't see it in project-config... | 20:34 |
*** dramakri has left #openstack-cinder | 20:34 | |
eharney | scottda: it's in there as pypy-jobs | 20:35 |
smcginnis_ | https://github.com/openstack-infra/project-config/blob/c4eb24d7a1b6cec506315f90e8bc9c76e370bea6/zuul/layout.yaml#L441 | 20:35 |
smcginnis_ | I assume it will just use whatever default for the given platform it runs on. | 20:35 |
eharney | and it seems to just install pypy from trusty, which is 2.2.1 | 20:36 |
smcginnis_ | So if it's trusty thats 2.2 | 20:36 |
eharney | scottda: i meant where did you see this fail? | 20:36 |
scottda | eharney: https://review.openstack.org/#/c/313550/ | 20:36 |
scottda | http://logs.openstack.org/50/313550/5/check/gate-python-cinderclient-pypy/cf9bbc5/console.html#_2016-05-06_18_58_24_494 | 20:37 |
*** sasukeh has joined #openstack-cinder | 20:37 | |
eharney | that check has been in python-cryptography since release 1.1 | 20:38 |
*** e0ne has quit IRC | 20:39 | |
eharney | last year | 20:39 |
sheel | :( .. seems no one considered being non voting jobs | 20:39 |
*** e0ne has joined #openstack-cinder | 20:40 | |
*** e0ne has quit IRC | 20:40 | |
eharney | sheel: that doesn't mean it was failing | 20:40 |
sheel | eharney: right | 20:40 |
*** ducttape_ has joined #openstack-cinder | 20:40 | |
eharney | just saying that there may not be much to do other than figure out how to get a newer pypy | 20:40 |
smcginnis_ | Paramiko just updated to use the new version 9 days ago. | 20:40 |
smcginnis_ | https://github.com/paramiko/paramiko/commit/1e7849ba910365e6008fd276b271e90a718fd618 | 20:40 |
eharney | ahh | 20:41 |
*** dramakri has joined #openstack-cinder | 20:41 | |
*** dramakri has left #openstack-cinder | 20:41 | |
eharney | well, nobody's going to want to version cap paramiko for long since it's pretty security-sensitive... | 20:42 |
*** sasukeh has quit IRC | 20:42 | |
*** dramakri has joined #openstack-cinder | 20:42 | |
scottda | Quite a conundrum ... | 20:42 |
scottda | I guess I'll start by logging a bug. I need to be somewhere in 15 minutes, so I'm about done here... | 20:43 |
eharney | scottda: yeah, definitely do that | 20:43 |
smcginnis_ | I've asked the question over in infra. | 20:43 |
smcginnis_ | Probably not a big deal for now, but good to get a bug filed for it. | 20:43 |
smcginnis_ | Presumably manually installing a newer pypy would be fine. | 20:43 |
smcginnis_ | But we don't have any test coverage to verify it works in the mean time. | 20:44 |
eharney | can we ditch trusty and move to xenial already? | 20:44 |
smcginnis_ | hehe, might be a little premature. | 20:44 |
eharney | yeaaaah, i know | 20:44 |
smcginnis_ | I've heard they got jobs testing building the image. | 20:44 |
smcginnis_ | I manually did a tempest run and everything passed for me on it. | 20:44 |
smcginnis_ | But the right answer is to move it all to RHEL, right? ;P | 20:45 |
eharney | well duh | 20:45 |
smcginnis_ | :) | 20:45 |
eharney | but seriously, does anyone use pypy? | 20:45 |
*** cknight has quit IRC | 20:45 | |
smcginnis_ | I used it when I had to do some stuff on Windows a long time ago. Does that count? :) | 20:45 |
eharney | only if you ran openstack on it :) | 20:46 |
smcginnis_ | Nope! | 20:47 |
*** porrua has quit IRC | 20:47 | |
smcginnis_ | Good news, according to infra we will soon be running Newton jobs on 16.04. That may solve the issue. At least for our gate jobs. | 20:47 |
scottda | https://bugs.launchpad.net/python-cinderclient/+bug/1579228 | 20:48 |
openstack | Launchpad bug 1579228 in python-cinderclient "gate-python-cinderclient-pypy fails due to paramiko requirement" [Undecided,New] | 20:48 |
smcginnis_ | scottda, +1, thanks! | 20:48 |
scottda | np | 20:48 |
*** dramakri has left #openstack-cinder | 20:51 | |
*** dramakri has joined #openstack-cinder | 20:53 | |
*** raunak has joined #openstack-cinder | 20:54 | |
*** jay-mehta has quit IRC | 20:55 | |
jgriffith | eharney: :) | 20:55 |
jgriffith | eharney: I've been using xenial for a week now without any real issues | 20:56 |
jgriffith | so on another note | 20:57 |
jgriffith | as I continue to be horrified with our attach/detach workflow :( | 20:57 |
hemna | well, on the upside, you are horrified with the rest of us :) | 20:58 |
jgriffith | What do you guys think about just ditching all this nonsense, have a "volume_attach" and "volume_detach" API call and let it just do *everything* and skip this back and forth? I think hemna may have mentioned something like that at one point | 20:58 |
jgriffith | hemna: LOL... misery loves company right? :) | 20:59 |
hemna | hehe, yes it does | 20:59 |
jgriffith | hemna: it's funny.. I keep looking at git logs asking "how did we get here" | 20:59 |
smcginnis_ | That should theoretically take care of some races and make things simpler if we could. | 20:59 |
hemna | hehe | 20:59 |
hemna | ouch | 20:59 |
jgriffith | hemna: amazing how all these little changes/hacks add up over a few years, and there's a TON of contributions | 20:59 |
hemna | laughing through the tears | 20:59 |
smcginnis_ | The simplest we can make the interaction, the better off we'll all be. | 21:00 |
smcginnis_ | But no simpler. | 21:00 |
jgriffith | smcginnis_: I can certainly do it, just don't know if it will fly | 21:00 |
smcginnis_ | If it can get rid of code, I think we can convince folks. | 21:00 |
hemna | so, yah I think that's what I had hoped at one point, but man it's a big change | 21:00 |
smcginnis_ | Microversions solves everything!! | 21:00 |
smcginnis_ | :) | 21:00 |
jgriffith | smcginnis_: to start we could just make attach suck in the initialize_connection and attach methods. Make them internal | 21:01 |
smcginnis_ | +1 | 21:01 |
scottda | ++ I've been for that all along. | 21:01 |
jgriffith | well, I have my patch almost ready (still need to try and run unit tests which scares me) | 21:01 |
hemna | push it up to gerrit....let's all watch | 21:02 |
jgriffith | it seems to work.. but honestly with all the bull shit that's been added around metadata, optional args etc I'm still not over joyed | 21:02 |
hemna | :) | 21:02 |
smcginnis_ | Just don't run them with pypy. | 21:02 |
jgriffith | lol | 21:02 |
jgriffith | it's ok, I'm on xenial remember :) | 21:02 |
smcginnis_ | Oh yeah. Then carry on. | 21:02 |
smcginnis_ | :) | 21:02 |
smcginnis_ | I've had X on my laptop since release day. I like! | 21:02 |
jgriffith | smcginnis_: yeah, I've been pretty happy thus far | 21:03 |
jgriffith | no surprises/problems | 21:03 |
jgriffith | seems fairly speedy, can't complain | 21:03 |
Swanson | How long has 16.04 lts been out? | 21:04 |
smcginnis_ | My machine actually seems to run better. Trackpad support is far better, which was my biggest complaint before. | 21:04 |
smcginnis_ | Swanson, 2 weeks? | 21:04 |
Swanson | Cool. Not too late to the party. | 21:05 |
*** jungleboyj has joined #openstack-cinder | 21:07 | |
*** raunak has quit IRC | 21:09 | |
*** raunak has joined #openstack-cinder | 21:09 | |
*** dramakri has left #openstack-cinder | 21:10 | |
*** rlrossit has quit IRC | 21:10 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 21:10 |
openstackgerrit | Patrick East proposed openstack/cinder: Specify a user-agent in Pure volume drivers https://review.openstack.org/313679 | 21:11 |
*** dramakri1 has joined #openstack-cinder | 21:13 | |
*** ducttape_ has quit IRC | 21:14 | |
*** dramakri1 has quit IRC | 21:14 | |
patrickeast | has anyone pinged the openvstorage ci people? | 21:15 |
patrickeast | it has been borked for a while now | 21:16 |
*** xyang1 has quit IRC | 21:16 | |
*** dramakri has joined #openstack-cinder | 21:20 | |
smcginnis_ | I have not. | 21:20 |
*** dustins has quit IRC | 21:21 | |
smcginnis_ | I think DuncanT said he was going to start reaching out to some of the failing ones. | 21:21 |
*** dramakri has left #openstack-cinder | 21:21 | |
*** baumann has left #openstack-cinder | 21:23 | |
*** eharney has quit IRC | 21:24 | |
*** ametts has quit IRC | 21:24 | |
openstackgerrit | Tom Barron proposed openstack/cinder: Remove deprecated cinder RequestBodySizeLimiter https://review.openstack.org/313682 | 21:25 |
jgriffith | hey! That's not bad... 29 failures | 21:28 |
jgriffith | that's actually pretty good | 21:28 |
jgriffith | and those appear to all be because of a missing key in their fakes | 21:28 |
jgriffith | huh.. who'd a thunk | 21:28 |
*** vilobhmm11 has joined #openstack-cinder | 21:29 | |
*** vilobhmm11 has quit IRC | 21:30 | |
*** vilobhmm11 has joined #openstack-cinder | 21:30 | |
*** lprice1 has quit IRC | 21:31 | |
*** vilobhmm11 has quit IRC | 21:33 | |
*** merooney has quit IRC | 21:33 | |
*** sasukeh has joined #openstack-cinder | 21:38 | |
*** cknight has joined #openstack-cinder | 21:43 | |
*** jwcroppe has quit IRC | 21:43 | |
*** mriedem has quit IRC | 21:44 | |
*** dramakri has joined #openstack-cinder | 21:44 | |
*** sasukeh has quit IRC | 21:46 | |
*** bhbhavsa_ has quit IRC | 21:46 | |
mtanino | jgriffith: hi! Do you want Cheesecake fix? https://review.openstack.org/#/c/313124/2 | 21:50 |
mtanino | waiting someone eat it :) | 21:50 |
*** salv-orlando has joined #openstack-cinder | 21:51 | |
*** ozialien10 has joined #openstack-cinder | 21:51 | |
*** xinli has quit IRC | 21:52 | |
jgriffith | mtanino: nothing better on a Friday than a good dessert :) | 21:52 |
mtanino | jgriffith: +1! | 21:54 |
openstackgerrit | Merged openstack/cinder: Use to_utf8() instead of safe_encode() in convert_str() https://review.openstack.org/313521 | 21:59 |
mtanino | jgriffith: Thank you for eating quickly :) | 22:01 |
jgriffith | mtanino: haha :) | 22:01 |
jgriffith | mtanino: thanks as always for helping out! | 22:01 |
mtanino | jgriffith: sure :) | 22:02 |
patrickeast | jgriffith: mtanino: maybe dumb question, but adding the policy checks would mean thats potentially a behavior change right? Does that need a micro-version? | 22:11 |
patrickeast | scottda: ^ | 22:12 |
mtanino | patrickeast: jgriffith scottda I'd like to know about it too. | 22:12 |
jgriffith | patrickeast: oh god no, please | 22:13 |
patrickeast | haha | 22:13 |
mtanino | patrickeast: In my feeling, it doesn't change behavior of API itself, so I think we don't need microversion. | 22:13 |
jgriffith | It's not really a behavior change IMO | 22:13 |
patrickeast | good enough for me | 22:13 |
jgriffith | it's a config option/change | 22:13 |
jgriffith | change in the default config | 22:14 |
mtanino | Also cloud admin can change that config too | 22:14 |
patrickeast | yea I tend to think its fine | 22:14 |
patrickeast | figured i'd check | 22:14 |
jgriffith | patrickeast: I owe you a beer :) | 22:14 |
mtanino | !! | 22:14 |
openstack | mtanino: Error: "!" is not a valid command. | 22:14 |
mtanino | sweet. | 22:14 |
jgriffith | lol | 22:14 |
jgriffith | mtanino: found a new easter egg! | 22:14 |
jgriffith | !! | 22:14 |
openstack | jgriffith: Error: "!" is not a valid command. | 22:14 |
jgriffith | ?? | 22:14 |
jgriffith | Ahh... I see what the bot did there | 22:15 |
jgriffith | !eat cheescake | 22:15 |
openstack | jgriffith: Error: "eat" is not a valid command. | 22:15 |
mtanino | wow.. | 22:16 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/313716 | 22:16 |
patrickeast | haha | 22:16 |
patrickeast | !help | 22:16 |
openstack | patrickeast: (help [<plugin>] [<command>]) -- This command gives a useful description of what <command> does. <plugin> is only necessary if the command is in more than one plugin. | 22:16 |
mtanino | help is valid command! | 22:16 |
mtanino | help me | 22:16 |
hemna | !poop | 22:16 |
openstack | hemna: Error: "poop" is not a valid command. | 22:16 |
hemna | damn | 22:17 |
patrickeast | lol | 22:17 |
hemna | yes, I'm 6 years old. | 22:17 |
jgriffith | lol | 22:18 |
*** vilobhmm11 has joined #openstack-cinder | 22:19 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick: Updated from global requirements https://review.openstack.org/313747 | 22:21 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/313752 | 22:22 |
thingee | glad I'm not the only one | 22:22 |
*** jungleboyj has quit IRC | 22:30 | |
*** david-lyle has quit IRC | 22:35 | |
*** david-lyle has joined #openstack-cinder | 22:37 | |
*** sdake has joined #openstack-cinder | 22:45 | |
*** takedakn1 has joined #openstack-cinder | 22:46 | |
*** markvoelker has quit IRC | 22:47 | |
*** sdake_ has joined #openstack-cinder | 22:48 | |
*** takedakn has quit IRC | 22:48 | |
*** takedakn1 is now known as takedakn | 22:48 | |
*** markvoelker has joined #openstack-cinder | 22:48 | |
*** lcurtis has quit IRC | 22:50 | |
*** sdake has quit IRC | 22:50 | |
*** adrianofr has quit IRC | 22:52 | |
*** r-daneel has quit IRC | 22:52 | |
*** sdake has joined #openstack-cinder | 22:55 | |
*** sdake_ has quit IRC | 22:56 | |
*** tpsilva has quit IRC | 22:57 | |
*** karthikp has quit IRC | 23:01 | |
*** martyturner has joined #openstack-cinder | 23:03 | |
*** sdake has quit IRC | 23:05 | |
*** yangyapeng has joined #openstack-cinder | 23:07 | |
openstackgerrit | Merged openstack/cinder: Add Cheesecake APIs to policy.json file https://review.openstack.org/313124 | 23:11 |
*** yangyapeng has quit IRC | 23:12 | |
*** martyturner has quit IRC | 23:21 | |
*** sheel has quit IRC | 23:25 | |
*** rodrigods has joined #openstack-cinder | 23:29 | |
rodrigods | hi... quick question about keystone v3 support | 23:29 |
rodrigods | anyone remembers which cycle it happened? | 23:29 |
*** sdake has joined #openstack-cinder | 23:34 | |
*** edmondsw has quit IRC | 23:34 | |
*** akshai has quit IRC | 23:35 | |
*** sdake_ has joined #openstack-cinder | 23:37 | |
*** leeantho_ has quit IRC | 23:38 | |
*** sdake has quit IRC | 23:38 | |
*** garthb_ has quit IRC | 23:38 | |
*** arch-nemesis has quit IRC | 23:42 | |
*** sasukeh has joined #openstack-cinder | 23:42 | |
*** sasukeh has quit IRC | 23:47 | |
*** bardia has quit IRC | 23:50 | |
*** merooney has joined #openstack-cinder | 23:51 | |
*** salv-orlando has quit IRC | 23:56 | |
*** akshai has joined #openstack-cinder | 23:57 | |
*** akshai_ has joined #openstack-cinder | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!