| *** jerrygb has quit IRC | 00:00 | |
| *** jerrygb has joined #openstack-cinder | 00:02 | |
| *** sgotliv_ has quit IRC | 00:02 | |
| *** salv-orl_ has quit IRC | 00:03 | |
| *** pv has quit IRC | 00:03 | |
| hemna | man it takes a long time to create and attach 300 volumes....for 1 test. | 00:05 |
|---|---|---|
| *** jamielennox|away is now known as jamielennox | 00:05 | |
| *** takedakn has quit IRC | 00:07 | |
| *** takedakn has joined #openstack-cinder | 00:07 | |
| *** s-moriya has joined #openstack-cinder | 00:08 | |
| *** salv-orlando has joined #openstack-cinder | 00:09 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/250967 | 00:10 |
| openstackgerrit | Dan Nguyen proposed openstack/cinder: Fixes cinder quota mgmt for keystone v3 https://review.openstack.org/253759 | 00:16 |
| *** merooney has joined #openstack-cinder | 00:17 | |
| *** jerrygb has quit IRC | 00:18 | |
| *** jerrygb has joined #openstack-cinder | 00:18 | |
| *** ianbrown has quit IRC | 00:20 | |
| *** ianbrown has joined #openstack-cinder | 00:20 | |
| *** lprice has joined #openstack-cinder | 00:22 | |
| *** jerrygb has quit IRC | 00:23 | |
| *** ianbrown has quit IRC | 00:25 | |
| *** ianbrown_ has joined #openstack-cinder | 00:25 | |
| *** lprice has quit IRC | 00:26 | |
| *** martyturner has quit IRC | 00:27 | |
| *** lprice has joined #openstack-cinder | 00:27 | |
| *** patrickeast has quit IRC | 00:32 | |
| *** mtanino has quit IRC | 00:33 | |
| *** asselin__ has quit IRC | 00:39 | |
| *** asselin__ has joined #openstack-cinder | 00:39 | |
| *** mc_nair_ has joined #openstack-cinder | 00:40 | |
| *** mc_nair has quit IRC | 00:40 | |
| hemna | so I just confirmed https://bugs.launchpad.net/cinder/+bug/1493350 | 00:42 |
| openstack | Launchpad bug 1493350 in Cinder "Attach Volume fail : Cinder - ISCSI device symlinks under /dev/disk/by-path in hex." [Undecided,Confirmed] | 00:42 |
| hemna | against my 3PAR iSCSI | 00:42 |
| hemna | seems that after 255 volumes, the LUN id is a hex number | 00:42 |
| hemna | and then we can't discover it. | 00:42 |
| *** takedakn has quit IRC | 00:43 | |
| jgriffith | hemna: know issue I though | 00:43 |
| jgriffith | thought | 00:43 |
| hemna | I'd never seen it before | 00:43 |
| jgriffith | hemna: oh.. "confirmed" HEHE | 00:43 |
| hemna | yah confirmed it. | 00:43 |
| jgriffith | hemna: it's an open-iscsi thing | 00:44 |
| hemna | so who creates the /dev/disk/by-path entries ? | 00:44 |
| hemna | udev ? | 00:44 |
| hemna | or scsi subsystem | 00:44 |
| hemna | I might try this with my FC driver | 00:44 |
| hemna | as we have increasing lun ids for it as well | 00:44 |
| jgriffith | hemna: wierd, I thought a fix landed for that | 00:44 |
| hemna | http://paste.openstack.org/show/481102/ | 00:45 |
| jgriffith | hemna: https://bugs.launchpad.net/os-brick/+bug/1488233 | 00:45 |
| openstack | Launchpad bug 1488233 in os-brick "FC with LUN ID >255 not recognized" [Undecided,Fix released] - Assigned to Stefan Amann (stefan-amann) | 00:45 |
| *** alejandrito has quit IRC | 00:46 | |
| jgriffith | hemna: you approved it :) | 00:46 |
| hemna | that's against the FC S390X connector | 00:47 |
| jgriffith | hemna: so maybe I'm wrong but when I went through that with Stefan my impression was it didn't matter | 00:47 |
| hemna | that's only used on the S390X architecture | 00:47 |
| jgriffith | hemna: ie it didn't matter which backend | 00:47 |
| jgriffith | hemna: ahhh | 00:47 |
| jgriffith | he put it in the class only :( | 00:47 |
| hemna | I'm guessing then it's a scsi subsystem thing | 00:47 |
| hemna | in which case every connector would need it. | 00:48 |
| jgriffith | hemna: just need the same change globally | 00:48 |
| hemna | yah | 00:48 |
| hemna | I'll put something together tomorrow and hack at it | 00:48 |
| jgriffith | hemna: yes, open-iscsi is what sets that up | 00:48 |
| hemna | but open-iscsi isn't involved w/ FC | 00:48 |
| jgriffith | hemna: nope it's not | 00:48 |
| hemna | ok I'll do some FC tests tomorrow | 00:49 |
| hemna | and verify it happens there as well | 00:49 |
| hemna | 256 volumes on a single compute host....*sigh* | 00:49 |
| hemna | that's a hell of a lot of volumes. | 00:49 |
| jgriffith | yup | 00:49 |
| hemna | also, fwiw, it looks like I can't attach more than 24 per vm | 00:49 |
| hemna | nova vomits after 24 | 00:49 |
| hemna | so I had to create 12 vms | 00:49 |
| hemna | to get to that # on the host. | 00:49 |
| jgriffith | hemna: did it puke because of the bdm size? | 00:50 |
| hemna | crap, lost the log | 00:50 |
| hemna | screen scrollback is too short... | 00:50 |
| jgriffith | :) | 00:50 |
| *** alonma has joined #openstack-cinder | 00:51 | |
| hemna | http://paste.openstack.org/show/481104/ | 00:52 |
| *** bardia has quit IRC | 00:52 | |
| hemna | that's the one I get when it's reached 24 volumes to a single vm. the 25 dumps that | 00:52 |
| *** bardia has joined #openstack-cinder | 00:53 | |
| hemna | oh | 00:53 |
| hemna | hah | 00:53 |
| hemna | yah it starts at vdb and goes to vdz and runs out | 00:53 |
| jgriffith | hemna: :) | 00:54 |
| jgriffith | hemna: FWIW I've never been completely clear on what some backends are using in disk/by-path that results in this problem | 00:55 |
| hemna | so I just checked my array console | 00:55 |
| hemna | and it's exporting as LUN id 257 | 00:55 |
| *** alonma has quit IRC | 00:55 | |
| jgriffith | hemna: Well... I mean, I understand the why | 00:56 |
| *** EinstCrazy has joined #openstack-cinder | 00:56 | |
| jgriffith | hemna: the part I don't necessarily get on iSCSI is why use the higher luns at all :) | 00:56 |
| jgriffith | hemna: you have a device iqn so you don't really "need" to increment lun count :) | 00:56 |
| jgriffith | hemna: anyway... it doesn't help with the bug, just a product design thing I've thought about | 00:57 |
| hemna | for ours, the iqn is always the exact same | 00:57 |
| hemna | our iqn is tied to a port on the array | 00:57 |
| *** alonma has joined #openstack-cinder | 00:57 | |
| hemna | the LUN id + the iqn is the unique identifier | 00:57 |
| jgriffith | hemna: yeah, a number of devices are like that | 00:57 |
| hemna | our other array, left hand is a unique iqn per volume, with LUN 0 every time. | 00:58 |
| jgriffith | hemna: I'm just saying it's a design thing I've thought/wondered about | 00:58 |
| jgriffith | hemna: yeah.. same here and with lvm | 00:58 |
| hemna | dunno really what the advantage of one way or the other is | 00:58 |
| jgriffith | hemna: this is one of the reasons. So it used to be you had a limit of 256 luns | 00:58 |
| jgriffith | hemna: so we got away from it by not using them that way | 00:59 |
| jgriffith | hemna: ie limit in scsi | 00:59 |
| hemna | ah heh. LUN=0 every time. never reaching the 256 limit! | 00:59 |
| hemna | heh | 00:59 |
| hemna | I think the default max luns in ubuntu is 512 nowadays | 00:59 |
| jgriffith | hemna: yea... and notice the reference LVM driver does the same thing | 00:59 |
| jgriffith | hemna: except it always uses "lun-1" | 01:00 |
| hemna | ah ok cool. | 01:00 |
| jgriffith | but it's the same logic | 01:00 |
| jgriffith | hemna: so FWIW an easier way for you to test this out to start is outside of OpenStack | 01:00 |
| *** changbl has quit IRC | 01:00 | |
| jgriffith | hemna: or just simply attach the one volume you have that's lun > 255 or whatever | 01:01 |
| jgriffith | you shouldn't have to attach all of them | 01:01 |
| jgriffith | just the ones with the higher lun number | 01:01 |
| jgriffith | that should save you some trouble | 01:01 |
| hemna | yah I wasn't sure if it was an artifact of the number coming from the array or the scsi system itself. | 01:01 |
| hemna | yah that's a good idea though thanks | 01:01 |
| *** alonma has quit IRC | 01:01 | |
| hemna | I can force the LUN id #'s to whatever I want on the array | 01:01 |
| hemna | do 1 attach with an id of 257 | 01:02 |
| hemna | should have the same result I'm guessing | 01:02 |
| jgriffith | hemna: yeah, that should do it | 01:02 |
| jgriffith | hemna: BTW it's an old/common problem | 01:03 |
| jgriffith | google has quite a few examples for Microsoft and VMware on the subject | 01:03 |
| jgriffith | probably something similar for open-iscsi and linux out there somewhere | 01:03 |
| jgriffith | hemna: when I talked with stefan about it I just converted it to hex and handled it that way | 01:03 |
| hemna | ah | 01:04 |
| *** alonma has joined #openstack-cinder | 01:04 | |
| jgriffith | anyway... gotta run for now, let me know if you get jammed up, happy to help later | 01:04 |
| hemna | ok man thanks! | 01:04 |
| hemna | I'd like to pick your brain about the cinder/brick/local_dev/lvm.py stuff tomorrow. | 01:04 |
| hemna | have a good evenin | 01:04 |
| *** hemna is now known as hemnafk | 01:04 | |
| *** vinayp has quit IRC | 01:07 | |
| *** alonma has quit IRC | 01:08 | |
| *** alonma has joined #openstack-cinder | 01:10 | |
| *** dilip has joined #openstack-cinder | 01:11 | |
| *** changbl has joined #openstack-cinder | 01:13 | |
| *** changbl has quit IRC | 01:14 | |
| *** alonma has quit IRC | 01:14 | |
| *** ianbrown_ has quit IRC | 01:20 | |
| *** coolsvap has quit IRC | 01:23 | |
| openstackgerrit | Ryan McNair proposed openstack/cinder: WIP - Move QoS_Specs to be a VersionedObject https://review.openstack.org/251989 | 01:23 |
| *** zhenguo has joined #openstack-cinder | 01:23 | |
| *** dilip has quit IRC | 01:24 | |
| *** salv-orlando has quit IRC | 01:25 | |
| *** coolsvap has joined #openstack-cinder | 01:25 | |
| *** yangyapeng has joined #openstack-cinder | 01:26 | |
| *** links has joined #openstack-cinder | 01:27 | |
| *** dilip has joined #openstack-cinder | 01:28 | |
| *** Lee1092 has joined #openstack-cinder | 01:28 | |
| *** garthb has quit IRC | 01:31 | |
| *** yangyapeng has quit IRC | 01:33 | |
| openstackgerrit | Ryan McNair proposed openstack/cinder: Fix non-migration swap with error https://review.openstack.org/252089 | 01:33 |
| *** changbl has joined #openstack-cinder | 01:34 | |
| *** smoriya is now known as smoriya_afk | 01:38 | |
| *** patrickeast has joined #openstack-cinder | 01:39 | |
| *** smoriya has joined #openstack-cinder | 01:43 | |
| *** r-daneel has quit IRC | 01:43 | |
| *** s-moriya has quit IRC | 01:47 | |
| *** changbl has quit IRC | 01:47 | |
| *** ianbrown_ has joined #openstack-cinder | 01:48 | |
| *** akshai has quit IRC | 01:51 | |
| *** apoorvad has quit IRC | 01:51 | |
| *** 7F1AA32U1 has joined #openstack-cinder | 01:58 | |
| *** changbl has joined #openstack-cinder | 02:00 | |
| *** 7F1AA32U1 has quit IRC | 02:01 | |
| *** haomaiwang has joined #openstack-cinder | 02:01 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/253753 | 02:05 |
| *** alonma has joined #openstack-cinder | 02:11 | |
| *** jerrygb has joined #openstack-cinder | 02:14 | |
| *** alonma has quit IRC | 02:15 | |
| *** jerrygb has quit IRC | 02:16 | |
| *** leeantho has quit IRC | 02:17 | |
| *** alonma has joined #openstack-cinder | 02:18 | |
| *** yangyape_ has joined #openstack-cinder | 02:22 | |
| *** alonma has quit IRC | 02:23 | |
| *** alonma has joined #openstack-cinder | 02:26 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/253753 | 02:28 |
| *** alonma has quit IRC | 02:30 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick: Updated from global requirements https://review.openstack.org/254499 | 02:32 |
| *** houming has joined #openstack-cinder | 02:42 | |
| *** shihanzhang has joined #openstack-cinder | 02:46 | |
| *** p0rtal has quit IRC | 02:46 | |
| *** bardia has quit IRC | 02:48 | |
| *** merooney has quit IRC | 02:48 | |
| *** shz has quit IRC | 02:49 | |
| *** dilip has quit IRC | 02:50 | |
| *** jgjhgsonchn88523 has joined #openstack-cinder | 02:56 | |
| *** ianbrown_ has quit IRC | 02:57 | |
| *** dilip has joined #openstack-cinder | 02:58 | |
| *** haomaiwang has quit IRC | 03:01 | |
| *** haomaiwang has joined #openstack-cinder | 03:01 | |
| *** akshai has joined #openstack-cinder | 03:04 | |
| *** alonma has joined #openstack-cinder | 03:04 | |
| *** edmondsw has quit IRC | 03:07 | |
| *** alonma has quit IRC | 03:09 | |
| *** alonma has joined #openstack-cinder | 03:10 | |
| *** alonma has quit IRC | 03:14 | |
| *** haomaiwang has quit IRC | 03:15 | |
| *** haomaiwang has joined #openstack-cinder | 03:15 | |
| *** haomaiwang has quit IRC | 03:16 | |
| *** haomaiwang has joined #openstack-cinder | 03:16 | |
| *** haomaiwang has quit IRC | 03:17 | |
| *** alonma has joined #openstack-cinder | 03:17 | |
| *** haomaiwang has joined #openstack-cinder | 03:18 | |
| *** haomaiwang has quit IRC | 03:19 | |
| *** 17WAA5E9R has joined #openstack-cinder | 03:19 | |
| *** 17WAA5E9R has quit IRC | 03:20 | |
| *** haomaiwang has joined #openstack-cinder | 03:20 | |
| *** haomaiwang has quit IRC | 03:21 | |
| *** alonma has quit IRC | 03:22 | |
| *** haomaiwang has joined #openstack-cinder | 03:24 | |
| *** haomaiwang has quit IRC | 03:25 | |
| *** julim has quit IRC | 03:25 | |
| *** haomaiwang has joined #openstack-cinder | 03:25 | |
| *** haomaiwang has quit IRC | 03:26 | |
| *** haomaiwang has joined #openstack-cinder | 03:26 | |
| *** haomaiwang has quit IRC | 03:27 | |
| *** haomaiwang has joined #openstack-cinder | 03:27 | |
| *** haomaiwang has quit IRC | 03:28 | |
| *** haomaiwang has joined #openstack-cinder | 03:28 | |
| *** haomaiwang has quit IRC | 03:29 | |
| *** salv-orlando has joined #openstack-cinder | 03:29 | |
| *** haomaiwang has joined #openstack-cinder | 03:29 | |
| *** haomaiwang has quit IRC | 03:30 | |
| *** 7F1AA34GJ has joined #openstack-cinder | 03:30 | |
| *** 7F1AA34GJ has quit IRC | 03:31 | |
| *** haomaiwang has joined #openstack-cinder | 03:31 | |
| *** haomaiwang has quit IRC | 03:32 | |
| *** haomaiwa_ has joined #openstack-cinder | 03:33 | |
| *** haomaiwa_ has quit IRC | 03:34 | |
| *** salv-orlando has quit IRC | 03:34 | |
| *** haomaiwang has joined #openstack-cinder | 03:34 | |
| *** haomaiwang has quit IRC | 03:35 | |
| *** jerrygb has joined #openstack-cinder | 03:35 | |
| *** haomaiwa_ has joined #openstack-cinder | 03:38 | |
| *** p0rtal has joined #openstack-cinder | 03:41 | |
| *** jgjhgsonchn88523 has quit IRC | 03:49 | |
| *** ianbrown has joined #openstack-cinder | 03:55 | |
| *** jgjhgsonchn88523 has joined #openstack-cinder | 03:57 | |
| *** haomaiwa_ has quit IRC | 04:01 | |
| *** haomaiwang has joined #openstack-cinder | 04:01 | |
| *** itzdilip has joined #openstack-cinder | 04:02 | |
| *** jgjhgsonchn88523 has quit IRC | 04:04 | |
| *** mc_nair_ has quit IRC | 04:09 | |
| *** vinayp has joined #openstack-cinder | 04:11 | |
| *** baumann has quit IRC | 04:16 | |
| *** alonma has joined #openstack-cinder | 04:20 | |
| *** alonma has quit IRC | 04:25 | |
| *** gouthamr has joined #openstack-cinder | 04:29 | |
| *** ebalduf has quit IRC | 04:30 | |
| openstackgerrit | javeme proposed openstack/cinder: encode the url parameters for XIO driver https://review.openstack.org/254157 | 04:35 |
| *** ianbrown has quit IRC | 04:35 | |
| *** vinayp has quit IRC | 04:35 | |
| *** vinayp has joined #openstack-cinder | 04:36 | |
| *** boris-42_ has quit IRC | 04:43 | |
| *** bardia has joined #openstack-cinder | 04:45 | |
| *** dave-mccowan has quit IRC | 04:55 | |
| *** dave-mccowan has joined #openstack-cinder | 04:57 | |
| *** haomaiwang has quit IRC | 05:01 | |
| *** haomaiwang has joined #openstack-cinder | 05:01 | |
| *** ianbrown has joined #openstack-cinder | 05:02 | |
| openstackgerrit | Takashi NATSUME proposed openstack/cinder: Enable volume owners to execute migrate_volume_completion https://review.openstack.org/253363 | 05:12 |
| *** anshul has quit IRC | 05:14 | |
| *** dave-mccowan has quit IRC | 05:14 | |
| *** jordant has joined #openstack-cinder | 05:16 | |
| *** dave-mccowan has joined #openstack-cinder | 05:17 | |
| *** jordant has left #openstack-cinder | 05:19 | |
| *** akshai has quit IRC | 05:22 | |
| *** vgridnev has joined #openstack-cinder | 05:24 | |
| *** ankit_ag has quit IRC | 05:28 | |
| *** alonma has joined #openstack-cinder | 05:29 | |
| *** alonma has quit IRC | 05:33 | |
| *** dilip has quit IRC | 05:41 | |
| *** dilip has joined #openstack-cinder | 05:41 | |
| *** dave-mccowan has quit IRC | 05:45 | |
| *** jerrygb has quit IRC | 05:50 | |
| *** gouthamr_ has joined #openstack-cinder | 05:54 | |
| *** ianbrown has quit IRC | 05:57 | |
| *** jaypipes has joined #openstack-cinder | 05:57 | |
| *** ianbrown has joined #openstack-cinder | 05:57 | |
| *** ianbrown_ has joined #openstack-cinder | 05:57 | |
| *** ianbrown_ has quit IRC | 05:57 | |
| *** dims has quit IRC | 05:57 | |
| *** gouthamr has quit IRC | 05:58 | |
| *** ianbrown_ has joined #openstack-cinder | 06:00 | |
| *** ianbrown has quit IRC | 06:00 | |
| *** haomaiwang has quit IRC | 06:01 | |
| *** haomaiwang has joined #openstack-cinder | 06:01 | |
| *** pradipta has joined #openstack-cinder | 06:02 | |
| vinayp | Can someone have a look at this bug ?? - https://bugs.launchpad.net/cinder/+bug/1493350 | 06:02 |
| openstack | Launchpad bug 1493350 in Cinder "Attach Volume fail : Cinder - ISCSI device symlinks under /dev/disk/by-path in hex." [Undecided,Confirmed] | 06:02 |
| vinayp | It is stopping our user acceptance test . Any help will be greatly appreciated. | 06:03 |
| *** harlowja_at_home has joined #openstack-cinder | 06:03 | |
| *** wolsen has quit IRC | 06:04 | |
| *** wolsen has joined #openstack-cinder | 06:05 | |
| *** donghao has joined #openstack-cinder | 06:10 | |
| *** ildikov has quit IRC | 06:16 | |
| *** ianbrown_ has quit IRC | 06:17 | |
| *** dims has joined #openstack-cinder | 06:20 | |
| *** sgotliv_ has joined #openstack-cinder | 06:22 | |
| *** ianbrown has joined #openstack-cinder | 06:23 | |
| *** lprice1 has joined #openstack-cinder | 06:23 | |
| *** dilip has quit IRC | 06:24 | |
| *** dilip has joined #openstack-cinder | 06:25 | |
| *** lprice has quit IRC | 06:26 | |
| *** martyturner has joined #openstack-cinder | 06:28 | |
| *** yangyape_ is now known as Yangyapeng | 06:28 | |
| *** isaacb has joined #openstack-cinder | 06:31 | |
| *** alonma has joined #openstack-cinder | 06:31 | |
| *** haomaiwang has quit IRC | 06:32 | |
| *** martyturner has quit IRC | 06:33 | |
| *** 18WABB396 has joined #openstack-cinder | 06:33 | |
| *** alonma has quit IRC | 06:36 | |
| *** donghao has quit IRC | 06:37 | |
| *** e0ne has joined #openstack-cinder | 06:38 | |
| *** ianbrown has quit IRC | 06:40 | |
| *** ianbrown has joined #openstack-cinder | 06:40 | |
| *** dilip has quit IRC | 06:40 | |
| *** dilip has joined #openstack-cinder | 06:41 | |
| *** ianbrown has quit IRC | 06:41 | |
| *** ianbrown_ has joined #openstack-cinder | 06:41 | |
| *** ianbrown__ has joined #openstack-cinder | 06:43 | |
| *** ianbrown_ has quit IRC | 06:43 | |
| *** e0ne has quit IRC | 06:45 | |
| *** ChubYann has quit IRC | 06:47 | |
| *** salv-orlando has joined #openstack-cinder | 06:49 | |
| *** smoriya has quit IRC | 06:49 | |
| *** alonma has joined #openstack-cinder | 06:49 | |
| *** e0ne has joined #openstack-cinder | 06:49 | |
| *** e0ne has quit IRC | 06:52 | |
| *** salv-orlando has quit IRC | 06:54 | |
| *** alonma has quit IRC | 06:54 | |
| *** alonma has joined #openstack-cinder | 06:56 | |
| *** alonma has quit IRC | 07:00 | |
| *** 18WABB396 has quit IRC | 07:01 | |
| *** p0rtal has quit IRC | 07:01 | |
| *** haomaiwang has joined #openstack-cinder | 07:01 | |
| *** alonma has joined #openstack-cinder | 07:02 | |
| *** e0ne has joined #openstack-cinder | 07:02 | |
| *** alonma has quit IRC | 07:06 | |
| *** houming has quit IRC | 07:08 | |
| *** haomaiwang has quit IRC | 07:09 | |
| *** haomaiwang has joined #openstack-cinder | 07:09 | |
| *** dims has quit IRC | 07:11 | |
| *** houming has joined #openstack-cinder | 07:11 | |
| *** isaacb has quit IRC | 07:11 | |
| *** houming has quit IRC | 07:16 | |
| *** jordant has joined #openstack-cinder | 07:17 | |
| *** nikeshm has quit IRC | 07:18 | |
| *** e0ne has quit IRC | 07:20 | |
| *** e0ne has joined #openstack-cinder | 07:21 | |
| *** bardia has quit IRC | 07:22 | |
| *** gouthamr_ has quit IRC | 07:22 | |
| *** nkrinner has joined #openstack-cinder | 07:22 | |
| *** gouthamr has joined #openstack-cinder | 07:24 | |
| *** chenying__ has joined #openstack-cinder | 07:24 | |
| *** anshul has joined #openstack-cinder | 07:25 | |
| *** gouthamr_ has joined #openstack-cinder | 07:25 | |
| *** dilip has quit IRC | 07:26 | |
| *** chenying_ has quit IRC | 07:27 | |
| *** haomaiwang has quit IRC | 07:28 | |
| *** gouthamr has quit IRC | 07:28 | |
| *** haomaiwa_ has joined #openstack-cinder | 07:28 | |
| *** harlowja_at_home has quit IRC | 07:30 | |
| *** dims has joined #openstack-cinder | 07:31 | |
| *** lprice1 has quit IRC | 07:36 | |
| *** dilip has joined #openstack-cinder | 07:42 | |
| *** houming has joined #openstack-cinder | 07:42 | |
| *** jaypipes has quit IRC | 07:44 | |
| *** lpetrut has joined #openstack-cinder | 07:45 | |
| *** alonma has joined #openstack-cinder | 07:51 | |
| *** dims has quit IRC | 07:52 | |
| *** [1]Thelo has joined #openstack-cinder | 07:53 | |
| *** dilip has quit IRC | 07:54 | |
| *** dilip has joined #openstack-cinder | 07:54 | |
| *** [2]Thelo has quit IRC | 07:55 | |
| *** e0ne has quit IRC | 07:56 | |
| *** alonma has quit IRC | 07:56 | |
| *** alonma has joined #openstack-cinder | 07:57 | |
| *** dilip has quit IRC | 07:59 | |
| *** alonma has quit IRC | 08:02 | |
| *** ildikov has joined #openstack-cinder | 08:03 | |
| *** alonma has joined #openstack-cinder | 08:04 | |
| *** jerrygb has joined #openstack-cinder | 08:06 | |
| openstackgerrit | Merged openstack/cinder: Fix StrOpts with integer defaults https://review.openstack.org/253827 | 08:06 |
| *** alonma has quit IRC | 08:08 | |
| *** alonma has joined #openstack-cinder | 08:10 | |
| *** jerrygb has quit IRC | 08:12 | |
| *** alonma has quit IRC | 08:14 | |
| *** alonma has joined #openstack-cinder | 08:16 | |
| *** dilip has joined #openstack-cinder | 08:17 | |
| *** alonma has quit IRC | 08:18 | |
| *** alonma has joined #openstack-cinder | 08:18 | |
| *** dilip has quit IRC | 08:22 | |
| *** dilip has joined #openstack-cinder | 08:22 | |
| *** klkumar has joined #openstack-cinder | 08:23 | |
| *** liverpooler has joined #openstack-cinder | 08:26 | |
| *** liverpooler has quit IRC | 08:26 | |
| *** gcb has joined #openstack-cinder | 08:27 | |
| *** liverpooler has joined #openstack-cinder | 08:30 | |
| *** markus_z has joined #openstack-cinder | 08:37 | |
| *** lprice has joined #openstack-cinder | 08:37 | |
| *** klkumar has quit IRC | 08:38 | |
| *** vgridnev has quit IRC | 08:43 | |
| *** klkumar has joined #openstack-cinder | 08:52 | |
| *** dilip has quit IRC | 08:55 | |
| *** links has quit IRC | 08:55 | |
| *** e0ne has joined #openstack-cinder | 09:03 | |
| *** links has joined #openstack-cinder | 09:09 | |
| *** ndipanov has quit IRC | 09:12 | |
| *** ndipanov has joined #openstack-cinder | 09:13 | |
| *** shakamunyi has joined #openstack-cinder | 09:14 | |
| *** ildikov has quit IRC | 09:16 | |
| *** ramineni1 has joined #openstack-cinder | 09:16 | |
| *** ramineni1 has left #openstack-cinder | 09:17 | |
| *** ildikov has joined #openstack-cinder | 09:18 | |
| *** sheel has joined #openstack-cinder | 09:20 | |
| sheel | dulek: Hi | 09:21 |
| *** shakamunyi has quit IRC | 09:23 | |
| dulek | sheel: hi | 09:25 |
| dulek | sheel: So… Tackling bug bug 1518363, right? | 09:25 |
| openstack | bug 1518363 in Cinder "Cannot do type-access-remove for a second time" [Undecided,In progress] https://launchpad.net/bugs/1518363 - Assigned to Sheel Rana (ranasheel2000) | 09:25 |
| sheel | dulek: regarding https://bugs.launchpad.net/cinder/+bug/1518363 | 09:25 |
| *** pradipta has quit IRC | 09:26 | |
| dulek | sheel: Please give me a minute to think this through… | 09:26 |
| sheel | dulek: oh sure. | 09:26 |
| dulek | sheel: So the problem is that if we'll align with proposed Online DB schema upgrades path, then we need to save in both places in Mitaka. This will keep the bug present through M release. | 09:28 |
| dulek | This is baaad… | 09:28 |
| sheel | dulek: Ok, I got your point. My proposal is to update the constraint "volume_type_id", "project_id", "deleted", to "volume_type_id", "project_id" and using True false in deleted column instead of volume_type_id | 09:28 |
| sheel | dulek : schema.UniqueConstraint("volume_type_id", "project_id", "deleted") to schema.UniqueConstraint("volume_type_id", "project_id") | 09:29 |
| dulek | sheel: This won't do the job I think. Right now the constraint is to keep a tuple ("volume_type_id", "project_id", "deleted") unique. | 09:30 |
| *** isaacb has joined #openstack-cinder | 09:30 | |
| dulek | sheel: When we'll remove deleted from there we still wont be able to add second mapping between vt_id and p_id, as we will keep ("volume_type_id", "project_id") unique. | 09:31 |
| * dulek is looking at models.py… | 09:31 | |
| dulek | sheel: ID there should be unique, but adding it to the constraint is like dropping it. | 09:32 |
| sheel | dulek:right | 09:34 |
| dulek | sheel: Nova dropped a constraint once and they follow similar approach. Let me take a look on their code… | 09:35 |
| sheel | dulek:ok | 09:35 |
| *** vgridnev has joined #openstack-cinder | 09:35 | |
| dulek | sheel: Does anything bad happen when we simply drop that constraint? | 09:37 |
| sheel | dulek: I need to see to it | 09:39 |
| dulek | sheel: I'm trying to find where the associate vt to project call is done, but with all the contrib stuff API code is so messy… | 09:40 |
| dulek | Ah, here it is - api.contrib.volume_type_access | 09:41 |
| *** Lickitysplitted_ has quit IRC | 09:41 | |
| sheel | dulek:yes | 09:42 |
| dulek | sheel: So detecting if volume_type_access exists is based on this constraint. To remove it from the DB we would need to mock it in the code. | 09:43 |
| dulek | sheel: It's not that bad - just select and check in a transaction… I don't believe this code is called very often so additional DB query shouldn't be harmful… | 09:44 |
| *** ildikov has quit IRC | 09:45 | |
| sheel | dulek:hmmm, you are right. I see to it | 09:45 |
| *** jordanP has joined #openstack-cinder | 09:46 | |
| dulek | It's db.sqlalchemy.api#2780-2795 | 09:46 |
| sheel | dulek: ok, great | 09:46 |
| *** jordant has quit IRC | 09:46 | |
| *** Lickitysplitted has joined #openstack-cinder | 09:47 | |
| dulek | And below is remove code. It filters by volume_type_id and project_id, so even if we happen to add more than one access per vol_type project_id pair this should mark both as removed. | 09:47 |
| dulek | sheel: Ha, there's even one more bug in delete I think. It will update deleted_at also for already deleted items… :D | 09:48 |
| * dulek hates to read Cinder's code carefully - you find obvious bugs too often… | 09:48 | |
| sheel | dulek:a lot of bugs are there in cinder | 09:49 |
| sheel | I will raise some later | 09:49 |
| sheel | dulek : so, is it ok if i close this bug by reverting old solution and open new bug for solving this issue... | 09:49 |
| sheel | dulek : I will check as per our discussion and see if no other problem occurs due to discussed solution | 09:50 |
| dulek | So this thing with deleted_at is unrelated to bug 1523598 | 09:51 |
| openstack | bug 1523598 in Cinder "Cannot stop cinder-backup with systemd or SIGTERM" [Undecided,New] https://launchpad.net/bugs/1523598 - Assigned to Ravi Shekhar Jethani (ravishekar-jethani) | 09:51 |
| dulek | Ah, sorry, wrong id. | 09:51 |
| dulek | bug 1518363 I've meant | 09:52 |
| openstack | bug 1518363 in Cinder "Cannot do type-access-remove for a second time" [Undecided,In progress] https://launchpad.net/bugs/1518363 - Assigned to Sheel Rana (ranasheel2000) | 09:52 |
| dulek | I wouldn't revert my solution as it will undercover bug 1496747 | 09:52 |
| openstack | bug 1496747 in Cinder "Removing project access to volume type will not work after 128+ access rule" [Medium,Fix released] https://launchpad.net/bugs/1496747 - Assigned to Michal Dulko (michal-dulko-f) | 09:52 |
| dulek | s/undercover/uncover | 09:52 |
| dulek | I would advice: | 09:53 |
| dulek | 1. Remove the constraint and add similar check in DB API code in Cinder. | 09:53 |
| dulek | This should be backward-compatible change. | 09:53 |
| sheel | dulek:Ok, i got your point | 09:54 |
| dulek | 2. Then we should probably convert all our deleted columns from Boolean to String and start saving record ids there. To do so we should follow the guidelines for upgrading the schema - so create a new column like deleted_id and start moving data there… | 09:55 |
| dulek | I'm not even sure if 2 is worth all the mess it would create in the code. Maybe we can live with deleted being a Boolean? | 09:56 |
| sheel | dulek:for now, i think we should stick to "deleted" as boolean | 09:56 |
| sheel | to limit the complications | 09:57 |
| dulek | sheel: Anyway - dropping a constraint and adding a check in the code should make it for bug 1518363 | 09:57 |
| openstack | bug 1518363 in Cinder "Cannot do type-access-remove for a second time" [Undecided,In progress] https://launchpad.net/bugs/1518363 - Assigned to Sheel Rana (ranasheel2000) | 09:57 |
| dulek | sheel: Please add me as reviewer once you'll post the patch. :) | 09:57 |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Add synchronization in Block Device driver https://review.openstack.org/252979 | 09:57 |
| sheel | dulek:sure, I will add you | 09:58 |
| *** gouthamr_ has quit IRC | 09:58 | |
| sheel | I may ping you sometime again..hope its ok with you!! | 09:58 |
| *** ildikov has joined #openstack-cinder | 09:59 | |
| *** houming has quit IRC | 09:59 | |
| sheel | dulek:thank you for your suggestions | 10:00 |
| dulek | sheel: Thanks for working on this. I'm happy to help. :) | 10:01 |
| *** aix has joined #openstack-cinder | 10:01 | |
| sheel | :) | 10:01 |
| *** zhenguo has quit IRC | 10:03 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Add synchronization in Block Device driver https://review.openstack.org/252979 | 10:04 |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Implement snapshots-related features for Block Device Driver https://review.openstack.org/253111 | 10:08 |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Add synchronization in Block Device driver https://review.openstack.org/252979 | 10:14 |
| *** jistr has joined #openstack-cinder | 10:14 | |
| *** sheel has quit IRC | 10:15 | |
| dulek | e0ne: Maaan, I cannot keep up with how quickly Yuriy is addressing the comments. :D | 10:16 |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Add synchronization in Block Device driver https://review.openstack.org/252979 | 10:16 |
| *** yuriy_n17 has joined #openstack-cinder | 10:17 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Implement snapshots-related features for Block Device Driver https://review.openstack.org/253111 | 10:19 |
| *** aix has quit IRC | 10:21 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Implement snapshots-related features for Block Device Driver https://review.openstack.org/253111 | 10:23 |
| *** salv-orlando has joined #openstack-cinder | 10:25 | |
| *** gcb has quit IRC | 10:29 | |
| e0ne | dulek, yuriy_n17: it's coding in a real-time! | 10:30 |
| *** salv-orlando has quit IRC | 10:30 | |
| *** gcb has joined #openstack-cinder | 10:31 | |
| *** aix has joined #openstack-cinder | 10:33 | |
| *** links has quit IRC | 10:37 | |
| yuriy_n17 | dulek: Thank you ever so much for your comments. | 10:46 |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Implement snapshots-related features for Block Device Driver https://review.openstack.org/253111 | 10:46 |
| *** Lickitysplitted has quit IRC | 10:53 | |
| *** shihanzhang has quit IRC | 10:55 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/253753 | 10:55 |
| *** shz has joined #openstack-cinder | 10:56 | |
| *** links has joined #openstack-cinder | 10:58 | |
| *** Yangyapeng has quit IRC | 10:58 | |
| *** EinstCrazy has quit IRC | 10:59 | |
| *** openstackgerrit has quit IRC | 11:02 | |
| *** Lickitysplitted has joined #openstack-cinder | 11:03 | |
| *** openstackgerrit has joined #openstack-cinder | 11:03 | |
| *** salv-orlando has joined #openstack-cinder | 11:06 | |
| *** john5223 is now known as zz_john5223 | 11:06 | |
| *** haomaiwa_ has quit IRC | 11:06 | |
| *** salv-orlando has quit IRC | 11:14 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Storwize: Implement v2 replication (mirror) https://review.openstack.org/249042 | 11:14 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Storwize: Implement v2 replication (split IO) https://review.openstack.org/237387 | 11:14 |
| *** EinstCrazy has joined #openstack-cinder | 11:24 | |
| *** ociuhandu has quit IRC | 11:31 | |
| openstackgerrit | Silvan Kaiser proposed openstack/cinder: Add Quobyte policy tag support to Quobyte Cinder driver https://review.openstack.org/254708 | 11:44 |
| *** netapp has quit IRC | 11:46 | |
| *** netapp has joined #openstack-cinder | 11:47 | |
| *** vinayp has quit IRC | 11:52 | |
| *** yhayashi has quit IRC | 11:55 | |
| *** houming has joined #openstack-cinder | 11:59 | |
| *** e0ne has quit IRC | 12:03 | |
| openstackgerrit | wanghao proposed openstack/cinder: Get the replica volume ref for DR https://review.openstack.org/252250 | 12:04 |
| openstackgerrit | wanghao proposed openstack/cinder: Get the replica volume ref for DR https://review.openstack.org/252250 | 12:05 |
| *** esp has quit IRC | 12:05 | |
| *** esp has joined #openstack-cinder | 12:06 | |
| *** jerrygb has joined #openstack-cinder | 12:09 | |
| *** e0ne has joined #openstack-cinder | 12:11 | |
| *** jerrygb has quit IRC | 12:13 | |
| e0ne | geguileo: hi! do you need any help with https://review.openstack.org/#/c/212501/ (Fix volume backup import test)? | 12:22 |
| e0ne | geguileo: it would be great to get it labded in Mitaka | 12:22 |
| geguileo | e0ne: Hi | 12:24 |
| geguileo | e0ne: The problem with that test was that it needs to run differently on pre L systems than in L and onwards | 12:25 |
| *** gouthamr has joined #openstack-cinder | 12:25 | |
| e0ne | geguileo: we can make this feature configurable | 12:25 |
| *** gouthamr is now known as gouthamr_away | 12:25 | |
| geguileo | e0ne: And the only way to do that AFAIK is to add a new configuration variable | 12:25 |
| *** gouthamr_away is now known as gouthamr | 12:25 | |
| geguileo | e0ne: But I was looking into a way for the test to actually know the version instead of adding yet another configuration option | 12:26 |
| e0ne | geguileo: you're right. I don't know other methods to to it | 12:26 |
| geguileo | e0ne: But it looked weird to me to have 1 configuration variable to enable backup tests and another to specify the release of the backup service (so we can run one test or another) :-( | 12:27 |
| geguileo | But I guess that's what I'll have to do :-( | 12:27 |
| geguileo | Today is a national holiday here in Spain, but I'll back to that patch when I come back | 12:28 |
| *** EinstCra_ has joined #openstack-cinder | 12:35 | |
| *** links has quit IRC | 12:36 | |
| *** EinstCrazy has quit IRC | 12:37 | |
| e0ne | geguileo: ok, thanks. ping me if you'll need any help | 12:38 |
| geguileo | e0ne: Ok, thanks! | 12:38 |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Tooz locks https://review.openstack.org/183537 | 12:56 |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Support locking Versioned Objects https://review.openstack.org/246352 | 12:56 |
| *** yangyapeng has joined #openstack-cinder | 12:59 | |
| *** gouthamr has quit IRC | 13:02 | |
| *** baumann has joined #openstack-cinder | 13:02 | |
| *** gouthamr has joined #openstack-cinder | 13:03 | |
| *** jgregor has joined #openstack-cinder | 13:04 | |
| *** gouthamr has quit IRC | 13:12 | |
| *** gouthamr has joined #openstack-cinder | 13:12 | |
| *** martyturner has joined #openstack-cinder | 13:14 | |
| *** dims has joined #openstack-cinder | 13:16 | |
| *** gouthamr has quit IRC | 13:19 | |
| *** gouthamr_ has joined #openstack-cinder | 13:19 | |
| *** ianbrown__ has quit IRC | 13:22 | |
| *** ianbrown_ has joined #openstack-cinder | 13:22 | |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Tooz locks https://review.openstack.org/183537 | 13:23 |
| *** gouthamr_ has quit IRC | 13:25 | |
| *** gouthamr has joined #openstack-cinder | 13:27 | |
| *** edmondsw has joined #openstack-cinder | 13:31 | |
| *** crose has joined #openstack-cinder | 13:34 | |
| *** diablo_rojo has joined #openstack-cinder | 13:34 | |
| *** jaypipes has joined #openstack-cinder | 13:35 | |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Support locking Versioned Objects https://review.openstack.org/246352 | 13:36 |
| *** timcl has joined #openstack-cinder | 13:40 | |
| *** jerrygb has joined #openstack-cinder | 13:44 | |
| *** ociuhandu has joined #openstack-cinder | 13:46 | |
| *** julim has joined #openstack-cinder | 13:55 | |
| *** baumann1 has joined #openstack-cinder | 13:56 | |
| *** dave-mccowan has joined #openstack-cinder | 13:56 | |
| *** baumann has quit IRC | 13:57 | |
| *** diogogmt has quit IRC | 13:58 | |
| *** diogogmt has joined #openstack-cinder | 13:58 | |
| *** zhipeng has joined #openstack-cinder | 14:00 | |
| *** akshai has joined #openstack-cinder | 14:04 | |
| *** changbl has quit IRC | 14:10 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Add synchronization in Block Device driver https://review.openstack.org/252979 | 14:14 |
| *** merooney has joined #openstack-cinder | 14:14 | |
| *** crose has quit IRC | 14:15 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Implement snapshots-related features for Block Device Driver https://review.openstack.org/253111 | 14:16 |
| *** jungleboyj has quit IRC | 14:18 | |
| *** breitz has quit IRC | 14:18 | |
| *** breitz has joined #openstack-cinder | 14:19 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Add synchronization in Block Device driver https://review.openstack.org/252979 | 14:21 |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Implement snapshots-related features for Block Device Driver https://review.openstack.org/253111 | 14:23 |
| *** diogogmt has quit IRC | 14:25 | |
| *** akerr has joined #openstack-cinder | 14:26 | |
| *** mriedem has joined #openstack-cinder | 14:28 | |
| *** zhangjn has joined #openstack-cinder | 14:32 | |
| *** zhangjn has quit IRC | 14:32 | |
| *** zhangjn has joined #openstack-cinder | 14:32 | |
| openstackgerrit | Dave McCowan proposed openstack/cinder: Check context before returning cached value https://review.openstack.org/254357 | 14:40 |
| *** xyang has joined #openstack-cinder | 14:41 | |
| openstackgerrit | Sean McGinnis proposed openstack/python-cinderclient: Add reno for release notes management https://review.openstack.org/253206 | 14:45 |
| openstackgerrit | Sean McGinnis proposed openstack/os-brick: Add reno for release notes management https://review.openstack.org/253207 | 14:47 |
| *** edmondsw has quit IRC | 14:48 | |
| *** xyang has quit IRC | 14:50 | |
| *** dims_ has joined #openstack-cinder | 14:52 | |
| *** dims has quit IRC | 14:55 | |
| *** eharney has joined #openstack-cinder | 14:56 | |
| *** xyang has joined #openstack-cinder | 15:00 | |
| *** zhangjn has quit IRC | 15:01 | |
| *** dustins|gone is now known as dustins | 15:01 | |
| *** yangyapeng has quit IRC | 15:01 | |
| *** xyang has quit IRC | 15:07 | |
| *** zhipeng has quit IRC | 15:09 | |
| *** jerrygb has quit IRC | 15:09 | |
| *** jerrygb has joined #openstack-cinder | 15:10 | |
| *** bluex has quit IRC | 15:10 | |
| *** edmondsw has joined #openstack-cinder | 15:13 | |
| *** lprice1 has joined #openstack-cinder | 15:14 | |
| *** ebalduf has joined #openstack-cinder | 15:14 | |
| *** jerrygb has quit IRC | 15:15 | |
| *** lprice has quit IRC | 15:17 | |
| *** timcl has quit IRC | 15:24 | |
| *** jerrygb has joined #openstack-cinder | 15:24 | |
| *** merooney has quit IRC | 15:26 | |
| *** merooney has joined #openstack-cinder | 15:27 | |
| *** xyang has joined #openstack-cinder | 15:31 | |
| *** eduardo___ has joined #openstack-cinder | 15:32 | |
| *** mtanino has joined #openstack-cinder | 15:33 | |
| *** xyang1 has joined #openstack-cinder | 15:33 | |
| *** bluex has joined #openstack-cinder | 15:35 | |
| *** diogogmt has joined #openstack-cinder | 15:36 | |
| *** erhudy has joined #openstack-cinder | 15:36 | |
| *** thangp has joined #openstack-cinder | 15:37 | |
| *** esker has joined #openstack-cinder | 15:40 | |
| *** ntpttr has joined #openstack-cinder | 15:43 | |
| *** jistr is now known as jistr|mtg | 15:43 | |
| *** timcl has joined #openstack-cinder | 15:44 | |
| *** mc_nair has joined #openstack-cinder | 15:52 | |
| *** nkrinner has quit IRC | 15:53 | |
| *** manjeets has joined #openstack-cinder | 15:55 | |
| *** qeelee has joined #openstack-cinder | 15:57 | |
| *** qeelee has left #openstack-cinder | 15:57 | |
| *** qeelee has joined #openstack-cinder | 15:58 | |
| *** garthb has joined #openstack-cinder | 15:58 | |
| *** qeelee has left #openstack-cinder | 15:58 | |
| *** vgridnev has quit IRC | 16:00 | |
| *** martyturner has quit IRC | 16:01 | |
| *** martyturner has joined #openstack-cinder | 16:02 | |
| *** salv-orlando has joined #openstack-cinder | 16:03 | |
| *** salv-orlando has quit IRC | 16:03 | |
| *** lpetrut has quit IRC | 16:03 | |
| *** salv-orlando has joined #openstack-cinder | 16:03 | |
| *** hemnafk is now known as hemna | 16:06 | |
| *** aix has quit IRC | 16:08 | |
| scottda | Does anyone recall the deprecation schedule for API v1? | 16:13 |
| *** zhipeng has joined #openstack-cinder | 16:13 | |
| scottda | e0ne: Are we deprecating API V1, do you know? | 16:14 |
| smcginnis | scottda: I believe it's been deprecated for a few releases now. | 16:14 |
| smcginnis | scottda: But it was decided at the summit that even though it's deprecated, it can't go away. | 16:14 |
| scottda | OK. I guess that's the clarification I was looking for... | 16:15 |
| e0ne | scottda: it's already depricated | 16:15 |
| scottda | e0ne Thanks for the review of cinderclient support for API microversions. I'm trying to decide what to do about v1 API... | 16:15 |
| *** changbl has joined #openstack-cinder | 16:15 | |
| scottda | In the existing code I do a switch to v2 if the user attempts to use v1. But I think I cannot do that. | 16:16 |
| e0ne | scottda: you're welcome! thanks for working on it | 16:16 |
| scottda | So, I think I'll just leave a warning about deprecation but let them continue with v1 | 16:16 |
| scottda | We'll see what people think about that. | 16:16 |
| *** ianbrown_ has quit IRC | 16:17 | |
| e0ne | scottda: agree. warning sounds good in such case. we do the same on the server-side | 16:18 |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Implement snapshots-related features for Block Device Driver https://review.openstack.org/253111 | 16:22 |
| *** mriedem is now known as rodermang | 16:23 | |
| *** rodermang is now known as mriedem | 16:23 | |
| *** zhipeng has quit IRC | 16:24 | |
| *** changbl has quit IRC | 16:25 | |
| *** changbl has joined #openstack-cinder | 16:26 | |
| *** salv-orl_ has joined #openstack-cinder | 16:26 | |
| *** ianbrown_ has joined #openstack-cinder | 16:29 | |
| *** salv-orlando has quit IRC | 16:29 | |
| *** changbl has quit IRC | 16:30 | |
| *** jdurgin1 has joined #openstack-cinder | 16:31 | |
| *** timcl has quit IRC | 16:31 | |
| *** p0rtal has joined #openstack-cinder | 16:34 | |
| *** crose has joined #openstack-cinder | 16:35 | |
| hemna | jgriffith, so it looks like udev is who is creating the /dev/disk/by-path/* entries | 16:36 |
| mtanino | hemna: I think so too. | 16:37 |
| hemna | http://paste.openstack.org/show/481198/ | 16:37 |
| e0ne | hemna: +1 | 16:37 |
| *** changbl has joined #openstack-cinder | 16:43 | |
| *** boris-42_ has joined #openstack-cinder | 16:44 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Add synchronization in Block Device driver https://review.openstack.org/252979 | 16:45 |
| *** openstackgerrit has quit IRC | 16:47 | |
| *** openstackgerrit has joined #openstack-cinder | 16:48 | |
| jgriffith | hemna: well... | 16:48 |
| jgriffith | hemna: the thing isn't really that it's a problem with the device being created | 16:49 |
| smcginnis | ls | 16:49 |
| smcginnis | Doh | 16:49 |
| jgriffith | hemna: at least in the cases I've looked at in the past. You can create and attach it just fine, the problem is that it turns into a hex representation for the lun number | 16:49 |
| hemna | yah | 16:49 |
| jgriffith | hemna: and brick doesn't know what to do with it | 16:49 |
| hemna | we have to fix it in os-brick | 16:49 |
| jgriffith | hemna: correct | 16:49 |
| hemna | I'm just trying to dig deeper and understand the entire process as best as possible | 16:49 |
| jgriffith | hemna: ahh... ok, got ya | 16:50 |
| jgriffith | hemna: so FYI, that also comes up from the scsi kernel modules IIRC | 16:50 |
| jgriffith | hemna: either way, the good thing is the fix should be relatively simple | 16:50 |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Add synchronization in Block Device driver https://review.openstack.org/252979 | 16:50 |
| jgriffith | hemna: I'm amazed that you guys haven't had customers report this already | 16:51 |
| jgriffith | hemna: unless you don't often have more than 255 volumes on an array which seems surprising | 16:51 |
| *** garthb_ has joined #openstack-cinder | 16:51 | |
| *** garthb has quit IRC | 16:51 | |
| hemna | http://paste.openstack.org/show/481202/ | 16:51 |
| hemna | well, not openstack users yet at least | 16:51 |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Implement snapshots-related features for Block Device Driver https://review.openstack.org/253111 | 16:52 |
| openstackgerrit | Manjeet Singh Bhatia proposed openstack/python-cinderclient: Pass proxy environment variable to tox https://review.openstack.org/252198 | 16:53 |
| *** vgridnev has joined #openstack-cinder | 16:53 | |
| *** ianbrown__ has joined #openstack-cinder | 16:55 | |
| *** alejandrito has joined #openstack-cinder | 16:55 | |
| *** apoorvad has joined #openstack-cinder | 16:56 | |
| *** apoorvad has quit IRC | 16:56 | |
| *** ianbrown_ has quit IRC | 16:57 | |
| *** alonma has quit IRC | 17:00 | |
| *** alonma has joined #openstack-cinder | 17:00 | |
| *** ildikov has quit IRC | 17:02 | |
| *** alonma has quit IRC | 17:05 | |
| *** apoorvad has joined #openstack-cinder | 17:06 | |
| *** alonma has joined #openstack-cinder | 17:06 | |
| *** anshul has quit IRC | 17:07 | |
| *** houming has quit IRC | 17:07 | |
| *** alonma has quit IRC | 17:11 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/python-cinderclient: Set default service type to 'volumev2' https://review.openstack.org/254876 | 17:12 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/python-cinderclient: WIP. Discover extentions in in 'contib' subdirectories https://review.openstack.org/254877 | 17:12 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/python-cinderclient: WIP. Add Python and CLI interfaces for os-brick as an extention https://review.openstack.org/254878 | 17:12 |
| e0ne | jgriffith, hemna: could you please take a look on ^^ | 17:12 |
| e0ne | jgriffith, hemna: I've added cinderclient extension to use os-brick stuff | 17:13 |
| hemna | e0ne, sure | 17:13 |
| e0ne | thanks | 17:14 |
| *** alonma has joined #openstack-cinder | 17:14 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Implement snapshots-related features for Block Device Driver https://review.openstack.org/253111 | 17:15 |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Implement snapshots-related features for Block Device Driver https://review.openstack.org/253111 | 17:15 |
| *** salv-orl_ has quit IRC | 17:16 | |
| *** leeantho has joined #openstack-cinder | 17:18 | |
| *** alonma has quit IRC | 17:19 | |
| *** alonma has joined #openstack-cinder | 17:21 | |
| *** jistr|mtg is now known as jistr | 17:21 | |
| *** akerr has quit IRC | 17:24 | |
| *** alonma has quit IRC | 17:25 | |
| *** alonma has joined #openstack-cinder | 17:27 | |
| *** zhonghua-lee has quit IRC | 17:28 | |
| *** zhonghua-lee has joined #openstack-cinder | 17:29 | |
| *** merooney has quit IRC | 17:30 | |
| *** akerr has joined #openstack-cinder | 17:30 | |
| *** alonma has quit IRC | 17:31 | |
| *** martyturner has quit IRC | 17:34 | |
| *** alonma has joined #openstack-cinder | 17:34 | |
| scottda | e0ne: I'm trying to use ddt as you have pointed out in cinderclient microversion review, but I get an error: | 17:35 |
| scottda | http://paste.openstack.org/show/481210/ | 17:35 |
| scottda | When I add this patch: | 17:35 |
| scottda | http://paste.openstack.org/show/481208/ | 17:35 |
| scottda | e0ne: Any ideas why that doesn't work? | 17:35 |
| *** martyturner has joined #openstack-cinder | 17:36 | |
| *** esker has quit IRC | 17:37 | |
| e0ne | scottda: I'm not sure that you need @ddt.unpack | 17:37 |
| *** garthb_ has quit IRC | 17:38 | |
| e0ne | scottda: AFAIR, ddt.unpack is needed when you need to pass several arguments | 17:38 |
| scottda | ok, thanks. That might be it... | 17:39 |
| *** alonma has quit IRC | 17:39 | |
| *** klkumar has quit IRC | 17:39 | |
| e0ne | scottda: or it could be used for named arguments too | 17:40 |
| scottda | e0ne: Thanks. That seemed to be the problem. | 17:40 |
| e0ne | scottda: great! I'm glad that I can be helpful | 17:40 |
| *** angela-s has joined #openstack-cinder | 17:41 | |
| openstackgerrit | Scott DAngelo proposed openstack/python-cinderclient: api-microversion support for python-cinderclient https://review.openstack.org/248163 | 17:43 |
| *** ctina_ has joined #openstack-cinder | 17:46 | |
| *** alonma has joined #openstack-cinder | 17:50 | |
| *** e0ne has quit IRC | 17:50 | |
| *** ianbrown__ has quit IRC | 17:52 | |
| scottda | hemna: Did you see that this is re-opened: https://bugs.launchpad.net/cinder/+bug/1471278 | 17:53 |
| openstack | Launchpad bug 1471278 in Cinder "target removal on detaching volume with multi-attach flag" [Undecided,New] | 17:53 |
| hemna | no | 17:53 |
| scottda | After our discussion on the Nova multi-attach spec, it seems that we must fix this in Cinder. | 17:54 |
| hemna | ah ok | 17:54 |
| hemna | yah, I think I need to just disable multi-attach for lvm for the time being | 17:54 |
| scottda | Is it only an LVM thing? | 17:54 |
| *** [2]Thelo has joined #openstack-cinder | 17:54 | |
| hemna | that bug is I believe | 17:54 |
| scottda | OK, so you are on it? | 17:54 |
| hemna | yah I can disable it | 17:55 |
| scottda | cool. Thanks. | 17:55 |
| *** alonma has quit IRC | 17:55 | |
| *** ianbrown__ has joined #openstack-cinder | 17:56 | |
| *** [1]Thelo has quit IRC | 17:56 | |
| *** markus_z has quit IRC | 17:57 | |
| *** p0rtal has quit IRC | 17:59 | |
| *** alonma has joined #openstack-cinder | 17:59 | |
| hemna | scottda, I don't see a way to find the host attaches for lvm | 18:00 |
| *** shz has quit IRC | 18:00 | |
| hemna | with my arrays it's easy. the array keeps track of the attaches for a volume on a host, so I can turn that into a noop. | 18:00 |
| *** shz has joined #openstack-cinder | 18:01 | |
| scottda | by 'my arrays' you mean os-brick? | 18:01 |
| hemna | no | 18:01 |
| hemna | 3PAR, LeftHand | 18:01 |
| scottda | oh, right. | 18:01 |
| scottda | But sooner or later, all backends will need to track this, so that they can do the final tgt removal when the last attach from a particular host is done. | 18:02 |
| scottda | or cinder manager will need to. | 18:03 |
| scottda | Which might be a better place. | 18:03 |
| *** alonma has quit IRC | 18:03 | |
| *** akerr has quit IRC | 18:03 | |
| *** akerr has joined #openstack-cinder | 18:04 | |
| hemna | yup | 18:04 |
| *** jistr has quit IRC | 18:05 | |
| *** eduardo___ has quit IRC | 18:08 | |
| *** merooney has joined #openstack-cinder | 18:16 | |
| *** salv-orlando has joined #openstack-cinder | 18:17 | |
| *** alejandrito has quit IRC | 18:19 | |
| *** alejandrito has joined #openstack-cinder | 18:19 | |
| *** salv-orlando has quit IRC | 18:21 | |
| *** thangp has quit IRC | 18:25 | |
| *** shakamunyi has joined #openstack-cinder | 18:26 | |
| openstackgerrit | Mikhail Khodos proposed openstack/cinder: --amend https://review.openstack.org/254898 | 18:26 |
| *** jaypipes has quit IRC | 18:27 | |
| *** dims_ has quit IRC | 18:30 | |
| *** ircuser-1 has quit IRC | 18:30 | |
| *** xyang1 has quit IRC | 18:31 | |
| *** xyang1 has joined #openstack-cinder | 18:31 | |
| *** bardia has joined #openstack-cinder | 18:33 | |
| *** timcl has joined #openstack-cinder | 18:34 | |
| *** isaacb has quit IRC | 18:38 | |
| *** xyang1 has quit IRC | 18:38 | |
| *** jerrygb has quit IRC | 18:39 | |
| *** e0ne has joined #openstack-cinder | 18:39 | |
| *** ctina_ has quit IRC | 18:46 | |
| *** jerrygb has joined #openstack-cinder | 18:46 | |
| *** crose has quit IRC | 18:46 | |
| *** p0rtal has joined #openstack-cinder | 18:49 | |
| *** david-ly_ has joined #openstack-cinder | 18:50 | |
| *** david-ly_ is now known as david-lyle_ | 18:52 | |
| *** david-lyle has quit IRC | 18:54 | |
| *** ildikov has joined #openstack-cinder | 18:55 | |
| *** sgotliv_ has quit IRC | 18:55 | |
| Swanson | Evidence suggests the cat vomited in my laundry hamper. | 18:58 |
| zao | At least it's pre-laundry. | 19:02 |
| *** openstackgerrit has quit IRC | 19:02 | |
| *** openstackgerrit has joined #openstack-cinder | 19:03 | |
| *** salv-orlando has joined #openstack-cinder | 19:04 | |
| *** jungleboyj has joined #openstack-cinder | 19:04 | |
| *** martyturner has quit IRC | 19:05 | |
| *** martyturner has joined #openstack-cinder | 19:06 | |
| *** alonma has joined #openstack-cinder | 19:08 | |
| *** lpetrut has joined #openstack-cinder | 19:10 | |
| jgriffith | hemna: scottda you most certainly do have that info | 19:10 |
| jgriffith | hemna: scottda it's in the target | 19:10 |
| *** salv-orlando has quit IRC | 19:11 | |
| hemna | jgriffith, the target has how many times it's used on a host ? | 19:11 |
| *** ChubYann has joined #openstack-cinder | 19:12 | |
| jgriffith | hemna: it has how many I_T nexus connections it has | 19:12 |
| *** alonma has quit IRC | 19:12 | |
| hemna | --op show --tid ? | 19:13 |
| jgriffith | hemna: if you reuse a target and connection then that's a problem in multi-attach. If you actually create a new IT Nexus it will be shown in the target with a connection number | 19:13 |
| jgriffith | hemna: just a sec.... | 19:13 |
| jgriffith | hemna: https://gist.github.com/43dedd7ae3608c8527a2 | 19:13 |
| *** david-lyle_ is now known as david-lyle | 19:14 | |
| jgriffith | This is why I preach about using the REFERENCE as the reference for what's implemented :) | 19:14 |
| hemna | yah understood. every backend is very different though, so it's hard to translate lvm'isms to vendor arrays at times. | 19:14 |
| jgriffith | hemna: true statement | 19:14 |
| hemna | so I'd like to see if I can fix this | 19:15 |
| hemna | to keep multiattach working for lvm | 19:15 |
| hemna | but I have a learning curve to climb a bit for lvm it seems | 19:15 |
| jgriffith | hemna: nobody behaves the same, but IMO we should try and fit to LVM as opposed to anything else.... anyway, that wasn't really a relevant statement anyway :) | 19:15 |
| jgriffith | hemna: so the good thing is we have the tools to get that info | 19:15 |
| jgriffith | hemna: in other words, "MY" statement about the reference wasn't relevant | 19:16 |
| hemna | ok so that info is showing 2 different volumes attached to the same initiator yes ? | 19:16 |
| jgriffith | hemna: no | 19:16 |
| hemna | heh | 19:16 |
| jgriffith | hemna: one is the controller | 19:17 |
| hemna | damnit beavis | 19:17 |
| hemna | oh | 19:17 |
| jgriffith | with tgt-adm and LVM's impl you have a controller at lun-0 and a volume at lun-1 | 19:17 |
| hemna | LUN 0 is type 'controller' | 19:17 |
| jgriffith | ALL volumes are at lun-1 for us | 19:17 |
| hemna | so if we have LUN-1 attached to a host (n-cpu VM) and we get another request to attach to the same host | 19:18 |
| jgriffith | hemna: in the case of a multi-attach scenario you *should* just end up with anotehr IT Nexus entry | 19:18 |
| hemna | re: multi-attach. | 19:18 |
| hemna | ok so we'd see I_T nexus: 2 ? | 19:18 |
| hemna | maybe | 19:18 |
| jgriffith | hemna: yes | 19:18 |
| hemna | guess I'll have to pull all the patches and test it out | 19:18 |
| jgriffith | hemna: if you don't have time with the other stuff you've got going let me know, I can work it out this afternoon | 19:19 |
| scottda | What if the backend re-uses the connection? Is there a way to detect that using the tgt info? | 19:19 |
| hemna | I'd like to be able to do it, to help get some more lvm experience | 19:19 |
| scottda | Or do we need the drivers who do that to fix that up? | 19:19 |
| hemna | scottda, so our LeftHand driver takes care of it in terminate_connection | 19:20 |
| hemna | scottda, https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/hpe/hpe_lefthand_iscsi.py#L574-L578 | 19:21 |
| hemna | if we detect there are other volumes still attached to the host, we leave it be. | 19:21 |
| hemna | I think that might work for LH | 19:21 |
| hemna | I'll have to test it though | 19:21 |
| hemna | 3PAR is a new export per attach call, so it doesnt' care. | 19:21 |
| scottda | yeah, OK. It seems we'd have to look at the drivers and the ones that don't create a new export will need some logic similar to what you have in LH. | 19:22 |
| scottda | Not a big deal, just trying to figure out where the work is. | 19:22 |
| *** esker has joined #openstack-cinder | 19:24 | |
| *** esker has quit IRC | 19:28 | |
| openstackgerrit | Ryan McNair proposed openstack/cinder: Move QualityOfServiceSpecs to use VersionedObject https://review.openstack.org/251989 | 19:28 |
| openstackgerrit | Mikhail Khodos proposed openstack/cinder: Re-add Nexenta driver https://review.openstack.org/188877 | 19:29 |
| *** sgotliv_ has joined #openstack-cinder | 19:34 | |
| *** martyturner has quit IRC | 19:36 | |
| *** martyturner has joined #openstack-cinder | 19:36 | |
| *** harlowja has quit IRC | 19:38 | |
| *** harlowja has joined #openstack-cinder | 19:38 | |
| *** erhudy has quit IRC | 19:40 | |
| *** xyang has quit IRC | 19:47 | |
| *** mc_nair_ has joined #openstack-cinder | 19:51 | |
| *** mc_nair has quit IRC | 19:52 | |
| *** xyang has joined #openstack-cinder | 19:55 | |
| dulek | Hey, anone knows why we're silencing quota exceptions on delete calls? | 19:55 |
| *** vgridnev has quit IRC | 19:56 | |
| *** martyturner has quit IRC | 19:58 | |
| *** vgridnev has joined #openstack-cinder | 20:02 | |
| *** garthb has joined #openstack-cinder | 20:04 | |
| *** jungleboyj has quit IRC | 20:08 | |
| *** alonma has joined #openstack-cinder | 20:09 | |
| *** ociuhandu has quit IRC | 20:09 | |
| *** diablo_rojo has quit IRC | 20:10 | |
| *** diablo_rojo has joined #openstack-cinder | 20:13 | |
| openstackgerrit | Merged openstack/python-cinderclient: Add the version attribute to the Client class https://review.openstack.org/237034 | 20:13 |
| *** alonma has quit IRC | 20:14 | |
| *** esker has joined #openstack-cinder | 20:17 | |
| *** alonma has joined #openstack-cinder | 20:17 | |
| *** gouthamr has quit IRC | 20:17 | |
| *** crose has joined #openstack-cinder | 20:19 | |
| *** esker has quit IRC | 20:21 | |
| *** alonma has quit IRC | 20:22 | |
| *** crose has quit IRC | 20:23 | |
| *** ociuhandu has joined #openstack-cinder | 20:23 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Use branch-specific tox dir https://review.openstack.org/254945 | 20:24 |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Add validation for volume_type at copy_volume_to_image() https://review.openstack.org/254414 | 20:25 |
| *** rhagarty has quit IRC | 20:30 | |
| *** rhagarty has joined #openstack-cinder | 20:30 | |
| smcginnis | dhellmann: ping | 20:31 |
| *** Lee1092 has quit IRC | 20:31 | |
| *** apoorvad has quit IRC | 20:35 | |
| openstackgerrit | Mike Rooney proposed openstack/cinder: Add serial number to eseries ASUP payload https://review.openstack.org/254949 | 20:35 |
| openstackgerrit | Merged openstack/os-brick: Minor documentation fixes for the method parameters https://review.openstack.org/252866 | 20:37 |
| *** esker has joined #openstack-cinder | 20:37 | |
| *** asselin__ has quit IRC | 20:42 | |
| *** ircuser-1 has joined #openstack-cinder | 20:43 | |
| *** xyang1 has joined #openstack-cinder | 20:45 | |
| mtanino | dulek: Hi, | 20:47 |
| *** martyturner has joined #openstack-cinder | 20:47 | |
| openstackgerrit | Scott DAngelo proposed openstack/python-cinderclient: api-microversion support for python-cinderclient https://review.openstack.org/248163 | 20:48 |
| *** esker has quit IRC | 20:49 | |
| Swanson | Replication again. If I have a backend with replication_device set does the volume type calling me have to have replication_enabled? | 20:59 |
| Swanson | "volume type calling me"? You get what I mean. | 21:00 |
| smcginnis | Swanson: I would htink if the backend is configured with it, it would just mean its capable of having replication. | 21:01 |
| smcginnis | And it would be up to the volume type selected as to whether replication is actually set up or not. | 21:01 |
| Swanson | I'd think that, too. | 21:02 |
| Swanson | Oh! I see what happened. | 21:02 |
| *** merooney has quit IRC | 21:02 | |
| dhellmann | smcginnis : sup? | 21:06 |
| smcginnis | dhellmann: Hey! | 21:07 |
| smcginnis | dhellmann: Hopefully a quick question for you. | 21:07 |
| * smcginnis pulls up link again... | 21:07 | |
| smcginnis | dhellmann: So question abou this: https://review.openstack.org/#/c/226024/ | 21:07 |
| dhellmann | k | 21:08 |
| smcginnis | We've discussed stable branches for clients in the past and decided they are pretty much just kind of a snpashot/checkpoint. | 21:08 |
| smcginnis | dhellmann: And that we won't ever be dpoing a new release from them. | 21:08 |
| smcginnis | Only going forward on master. | 21:08 |
| dhellmann | mostly. though we've had some security issues backported, and I wanted you to be ready in case that happened | 21:08 |
| smcginnis | dhellmann: So do you think there is a need to get this in? | 21:08 |
| smcginnis | dhellmann: Ahm that makes sense. So we probably should get that in then. | 21:09 |
| dhellmann | as it is right now, it's really easy to mistakenly submit a patch against that branch, but have it merge to master, so yeah | 21:09 |
| smcginnis | dhellmann: OK, good enough for me. Thanks! | 21:09 |
| dhellmann | if you never use it, there's no harm in having the branch configured properly | 21:09 |
| smcginnis | dhellmann: True | 21:09 |
| dhellmann | smcginnis : that's why submitting that patch is part of the process of creating the branch (it's automated and everything) | 21:10 |
| smcginnis | dhellmann: Oh, ok. Didn't realize that. | 21:10 |
| *** liverpooler has quit IRC | 21:11 | |
| dhellmann | smcginnis : yep, most of the release work is scripted, even if it's not running on the CI system | 21:11 |
| smcginnis | dhellmann: Makes a lot of sense. I'm sure there's a lot of common things that need to be done across several projects at release time. | 21:12 |
| dhellmann | exactly | 21:12 |
| *** lcurtis has joined #openstack-cinder | 21:15 | |
| *** alonma has joined #openstack-cinder | 21:19 | |
| *** ianbrown__ has quit IRC | 21:19 | |
| *** ianbrown has joined #openstack-cinder | 21:23 | |
| *** cdelatte has quit IRC | 21:23 | |
| *** alonma has quit IRC | 21:24 | |
| *** alonma has joined #openstack-cinder | 21:26 | |
| *** ianbrown has quit IRC | 21:27 | |
| *** ianbrown has joined #openstack-cinder | 21:28 | |
| *** earlephilhower has quit IRC | 21:30 | |
| *** alonma has quit IRC | 21:30 | |
| *** dims has joined #openstack-cinder | 21:31 | |
| *** ianbrown_ has joined #openstack-cinder | 21:33 | |
| *** ianbrown has quit IRC | 21:34 | |
| *** apoorvad has joined #openstack-cinder | 21:34 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick: Updated from global requirements https://review.openstack.org/254499 | 21:35 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/250967 | 21:36 |
| openstackgerrit | Nate Potter proposed openstack/cinder: Move snapshot and volume quota checks to API https://review.openstack.org/249388 | 21:37 |
| *** p0rtal has quit IRC | 21:38 | |
| *** p0rtal has joined #openstack-cinder | 21:38 | |
| *** ianbrown has joined #openstack-cinder | 21:41 | |
| *** baumann1 has quit IRC | 21:42 | |
| openstackgerrit | Nate Potter proposed openstack/cinder: Move snapshot and volume quota checks to API https://review.openstack.org/249388 | 21:43 |
| *** cdelatte has joined #openstack-cinder | 21:49 | |
| *** lpetrut has quit IRC | 21:50 | |
| *** salv-orlando has joined #openstack-cinder | 21:50 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Add validation for volume_type at copy_volume_to_image() https://review.openstack.org/254414 | 21:51 |
| openstackgerrit | Ryan McNair proposed openstack/cinder: Cleanup lock files for volume/snap after deletion https://review.openstack.org/239678 | 21:51 |
| *** salv-orlando has quit IRC | 21:55 | |
| openstackgerrit | Jacob Gregor proposed openstack/cinder: Storwize: Split up __init__ into separate files https://review.openstack.org/240329 | 21:56 |
| *** dims has quit IRC | 21:59 | |
| *** sgotliv_ has quit IRC | 22:00 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Add validation for volume_type at copy_volume_to_image() https://review.openstack.org/254414 | 22:02 |
| openstackgerrit | Erik Johannes proposed openstack/cinder: Implement CHAP Authentication for E-Series Driver https://review.openstack.org/246990 | 22:02 |
| *** akerr has quit IRC | 22:03 | |
| *** salv-orlando has joined #openstack-cinder | 22:05 | |
| openstackgerrit | Merged openstack/python-cinderclient: Remove ureachable code in fakes.py https://review.openstack.org/250193 | 22:07 |
| *** chris_morrell has joined #openstack-cinder | 22:10 | |
| *** david-ly_ has joined #openstack-cinder | 22:10 | |
| *** timcl has quit IRC | 22:12 | |
| *** alonma has joined #openstack-cinder | 22:12 | |
| *** jgregor has quit IRC | 22:12 | |
| *** david-lyle has quit IRC | 22:13 | |
| *** david-ly_ is now known as david-lyle | 22:13 | |
| openstackgerrit | Nate Potter proposed openstack/cinder: Move retype quota checks to API https://review.openstack.org/254980 | 22:13 |
| *** ntpttr has quit IRC | 22:14 | |
| *** timcl has joined #openstack-cinder | 22:15 | |
| *** alonma has quit IRC | 22:16 | |
| *** timcl has quit IRC | 22:17 | |
| *** alonma has joined #openstack-cinder | 22:18 | |
| *** belmoreira has joined #openstack-cinder | 22:19 | |
| *** julim has quit IRC | 22:21 | |
| *** ndipanov has quit IRC | 22:21 | |
| *** alonma has quit IRC | 22:22 | |
| *** e0ne has quit IRC | 22:26 | |
| *** alonma has joined #openstack-cinder | 22:26 | |
| *** dustins is now known as dustins|gone | 22:27 | |
| *** manjeets has quit IRC | 22:29 | |
| *** changbl has quit IRC | 22:31 | |
| *** alonma has quit IRC | 22:31 | |
| esp | hey folks could I get some eyes on this please https://review.openstack.org/#/c/253759/ | 22:31 |
| esp | ^ fixes a bug with cinder quota mgmt | 22:31 |
| *** jordanP has quit IRC | 22:35 | |
| *** diablo_rojo has quit IRC | 22:38 | |
| *** jungleboyj has joined #openstack-cinder | 22:48 | |
| *** adrianofr has quit IRC | 22:50 | |
| *** salv-orlando has quit IRC | 22:53 | |
| *** salv-orlando has joined #openstack-cinder | 22:54 | |
| *** ianbrown_ has quit IRC | 22:55 | |
| *** esp has quit IRC | 22:56 | |
| *** eharney has quit IRC | 22:56 | |
| *** ianbrown_ has joined #openstack-cinder | 22:57 | |
| *** chenying_ has joined #openstack-cinder | 22:58 | |
| *** chenying__ has quit IRC | 23:01 | |
| *** apoorvad has quit IRC | 23:02 | |
| *** ianbrown has quit IRC | 23:02 | |
| *** xyang1 has quit IRC | 23:08 | |
| *** mriedem is now known as mriedem_away | 23:09 | |
| *** chlong has quit IRC | 23:10 | |
| *** mkoderer has quit IRC | 23:15 | |
| *** mkoderer has joined #openstack-cinder | 23:17 | |
| *** mdenny has quit IRC | 23:18 | |
| *** jungleboyj has quit IRC | 23:19 | |
| *** ntpttr has joined #openstack-cinder | 23:21 | |
| *** edmondsw has quit IRC | 23:21 | |
| *** ntpttr has left #openstack-cinder | 23:22 | |
| *** alonma has joined #openstack-cinder | 23:27 | |
| *** alonma has quit IRC | 23:31 | |
| *** martyturner has quit IRC | 23:36 | |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: LeftHand: Implement v2 replication (managed) https://review.openstack.org/255002 | 23:37 |
| hemna | mtanino, ping | 23:44 |
| mtanino | hemna: Hi, I just replied on the review. | 23:45 |
| openstackgerrit | Ryan McNair proposed openstack/cinder: Move QualityOfServiceSpecs to use VersionedObject https://review.openstack.org/251989 | 23:46 |
| hemna | ok it's not obvious that's what you should do though | 23:46 |
| hemna | mtanino, ok I see it in the output | 23:48 |
| mtanino | hemna: yes, until cheking cinder show, user doesn't know valid fields. | 23:48 |
| mtanino | hemna: so, multiattach is in the show result. | 23:48 |
| hemna | it's not ideal, but ok | 23:49 |
| hemna | doh | 23:49 |
| mtanino | oops, | 23:50 |
| hemna | http://paste.openstack.org/show/481259/ | 23:50 |
| *** yhayashi has joined #openstack-cinder | 23:53 | |
| mtanino | hmm, should add a check for removing space? | 23:54 |
| *** jerrygb has quit IRC | 23:55 | |
| *** jerrygb has joined #openstack-cinder | 23:56 | |
| hemna | possibly | 23:56 |
| hemna | I nuked the space and it ran fine | 23:56 |
| hemna | I'm so pre-programmed to space after, that it was hard to type it w/o the space | 23:56 |
| *** takedakn has joined #openstack-cinder | 23:56 | |
| mtanino | lol | 23:57 |
| mtanino | I'll post followup patch | 23:57 |
| mtanino | just only removing space. | 23:57 |
| hemna | ok coolio thanks! | 23:57 |
| *** smoriya has joined #openstack-cinder | 23:57 | |
| *** manjeets has joined #openstack-cinder | 23:57 | |
| *** manjeets has left #openstack-cinder | 23:58 | |
| mtanino | hemna: thank you for the review. I'd appreciate | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!