*** takedakn has joined #openstack-cinder | 00:00 | |
*** haomaiwang has quit IRC | 00:01 | |
*** zhongjun has joined #openstack-cinder | 00:01 | |
*** haomaiwang has joined #openstack-cinder | 00:01 | |
*** dimsum__ has quit IRC | 00:03 | |
*** haomaiwang has quit IRC | 00:36 | |
*** esker has quit IRC | 00:42 | |
*** zhongjun has quit IRC | 00:45 | |
*** takedakn has quit IRC | 00:46 | |
*** takedakn has joined #openstack-cinder | 00:46 | |
*** takedakn has quit IRC | 00:47 | |
*** dimsum__ has joined #openstack-cinder | 00:48 | |
*** esker has joined #openstack-cinder | 00:50 | |
*** lixiaoy1 has joined #openstack-cinder | 01:00 | |
*** lixiaoy11 has joined #openstack-cinder | 01:01 | |
*** lixiaoy1 has quit IRC | 01:04 | |
*** baojg has joined #openstack-cinder | 01:07 | |
*** rick-chen has joined #openstack-cinder | 01:10 | |
rick-chen | anteaya: Hi, do you know when Miki return? | 01:11 |
---|---|---|
rick-chen | Hi, do you know when Mike return? | 01:12 |
*** erhudy1 has quit IRC | 01:12 | |
rick-chen | thingee: HI Mike, Do you return? | 01:13 |
*** zhangjn has joined #openstack-cinder | 01:19 | |
*** Lee1092 has joined #openstack-cinder | 01:28 | |
openstackgerrit | wanghao proposed openstack/cinder: Add UniqueConstraint for quota usages table https://review.openstack.org/217465 | 01:32 |
*** esker has quit IRC | 01:32 | |
*** esker has joined #openstack-cinder | 01:33 | |
*** haomaiwang has joined #openstack-cinder | 01:35 | |
*** zhenguo has joined #openstack-cinder | 01:35 | |
*** dimsum__ has quit IRC | 01:40 | |
*** chenying has joined #openstack-cinder | 01:43 | |
*** david-ly_ has joined #openstack-cinder | 01:51 | |
*** krtaylor has quit IRC | 01:52 | |
*** david-lyle has quit IRC | 01:54 | |
*** esker has quit IRC | 01:57 | |
*** haomaiwang has quit IRC | 02:01 | |
*** haomaiwang has joined #openstack-cinder | 02:01 | |
*** krtaylor has joined #openstack-cinder | 02:05 | |
*** rick-chen has quit IRC | 02:05 | |
*** esker has joined #openstack-cinder | 02:06 | |
openstackgerrit | Thang Pham proposed openstack/cinder: Conversion to volume object https://review.openstack.org/177054 | 02:10 |
*** zxyf has joined #openstack-cinder | 02:15 | |
*** esker has quit IRC | 02:31 | |
*** davechen has joined #openstack-cinder | 02:31 | |
openstackgerrit | Peter Wang proposed openstack/os-brick: Use 'device' instead of 'volume_path' https://review.openstack.org/220902 | 02:32 |
*** zxyf has quit IRC | 02:33 | |
*** zxyf has joined #openstack-cinder | 02:33 | |
*** esker has joined #openstack-cinder | 02:38 | |
*** dimsum__ has joined #openstack-cinder | 02:40 | |
openstackgerrit | Thang Pham proposed openstack/cinder: WIP: Make RPC and objects backwards compatible https://review.openstack.org/214943 | 02:44 |
*** dimsum__ has quit IRC | 02:46 | |
lixiaoy11 | Thang? | 02:50 |
lixiaoy11 | thangp: here? | 02:51 |
*** haomaiwang has quit IRC | 03:01 | |
*** haomaiwang has joined #openstack-cinder | 03:01 | |
*** esker has quit IRC | 03:05 | |
*** esker has joined #openstack-cinder | 03:10 | |
openstackgerrit | LisaLi proposed openstack/cinder: Fix KeyError https://review.openstack.org/219566 | 03:22 |
*** esker has quit IRC | 03:26 | |
*** baojg has quit IRC | 03:31 | |
*** esker has joined #openstack-cinder | 03:35 | |
*** zxyf has quit IRC | 03:42 | |
*** zxyf has joined #openstack-cinder | 03:43 | |
*** julim has quit IRC | 03:47 | |
*** zxyf has quit IRC | 03:50 | |
*** zxyf has joined #openstack-cinder | 03:50 | |
*** zhangjn_ has joined #openstack-cinder | 03:52 | |
*** esker has quit IRC | 03:54 | |
*** zhangjn has quit IRC | 03:56 | |
*** jamielennox is now known as jamielennox|away | 03:56 | |
*** haomaiwang has quit IRC | 04:01 | |
*** esker has joined #openstack-cinder | 04:01 | |
*** haomaiwang has joined #openstack-cinder | 04:01 | |
*** baojg has joined #openstack-cinder | 04:04 | |
openstackgerrit | Cedric Zhuang proposed openstack/cinder: Add retype logic in manage_existing for VNX https://review.openstack.org/209176 | 04:22 |
*** chenying has quit IRC | 04:22 | |
*** resker has joined #openstack-cinder | 04:23 | |
*** esker has quit IRC | 04:24 | |
*** zhangjn_ has quit IRC | 04:27 | |
*** jseiler_ has joined #openstack-cinder | 04:32 | |
*** jseiler has quit IRC | 04:35 | |
*** zxyf has quit IRC | 04:37 | |
*** zxyf has joined #openstack-cinder | 04:38 | |
*** zxyf has quit IRC | 04:42 | |
*** zxyf has joined #openstack-cinder | 04:43 | |
*** resker has quit IRC | 04:44 | |
*** esker has joined #openstack-cinder | 04:48 | |
*** zxyf has quit IRC | 04:59 | |
*** haomaiwang has quit IRC | 05:01 | |
*** haomaiwang has joined #openstack-cinder | 05:01 | |
*** esker has quit IRC | 05:16 | |
*** esker has joined #openstack-cinder | 05:22 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Adds migration abortion for available volumes https://review.openstack.org/207754 | 05:25 |
*** zxyf has joined #openstack-cinder | 05:31 | |
openstackgerrit | Peter Wang proposed openstack/cinder: Update update_migrated_volume in VNX driver https://review.openstack.org/215036 | 05:32 |
*** BharatK has joined #openstack-cinder | 05:39 | |
*** kjelly has quit IRC | 05:39 | |
*** vgridnev has joined #openstack-cinder | 05:45 | |
*** esker has quit IRC | 05:45 | |
*** bkopilov has joined #openstack-cinder | 05:49 | |
*** esker has joined #openstack-cinder | 05:51 | |
*** chenying has joined #openstack-cinder | 06:00 | |
*** haomaiwang has quit IRC | 06:01 | |
*** alexpilotti_ has joined #openstack-cinder | 06:01 | |
*** haomaiwang has joined #openstack-cinder | 06:01 | |
*** lpetrut has joined #openstack-cinder | 06:05 | |
*** vlaza has joined #openstack-cinder | 06:12 | |
*** sgotliv_ has quit IRC | 06:14 | |
*** yrabl has joined #openstack-cinder | 06:14 | |
*** yrabl is now known as yrabl_wfh | 06:18 | |
*** yrabl_wfh has quit IRC | 06:18 | |
*** yrabl has joined #openstack-cinder | 06:19 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Transifex https://review.openstack.org/218741 | 06:20 |
*** lpetrut has quit IRC | 06:22 | |
*** esker has quit IRC | 06:25 | |
*** esker has joined #openstack-cinder | 06:27 | |
*** salv-orlando has joined #openstack-cinder | 06:33 | |
*** baojg has quit IRC | 06:36 | |
*** esker has quit IRC | 06:45 | |
nikeshm | hi | 06:48 |
nikeshm | i have a doubt in https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/rbd.py#L741 | 06:49 |
nikeshm | is this correct? | 06:49 |
nikeshm | volume.create_snap(snap) | 06:49 |
*** esker has joined #openstack-cinder | 06:50 | |
nikeshm | should not be it volume.volume.create_snap(snap) | 06:50 |
nikeshm | did any check rbd recently | 06:52 |
*** lixiaoy11 has quit IRC | 06:54 | |
*** lixiaoy1 has joined #openstack-cinder | 06:54 | |
lixiaoy1 | nikeshm: I had do changes in rbd. https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/rbd.py#L739 | 06:56 |
lixiaoy1 | nikeshm: here volume is RBDVolumeProxy | 06:57 |
lixiaoy1 | nikeshm: it call driver.rbd.Image.create_snap which is from librbd | 06:58 |
*** haomaiwang has quit IRC | 07:01 | |
*** haomaiwang has joined #openstack-cinder | 07:01 | |
*** anshul has joined #openstack-cinder | 07:02 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Adds migration abortion for available volumes https://review.openstack.org/207754 | 07:05 |
*** zhongjun has joined #openstack-cinder | 07:08 | |
*** yrabl has quit IRC | 07:09 | |
*** yrabl has joined #openstack-cinder | 07:09 | |
*** lixiaoy11 has joined #openstack-cinder | 07:09 | |
*** lixiaoy11 has left #openstack-cinder | 07:09 | |
*** lixiaoy11 has joined #openstack-cinder | 07:10 | |
nikeshm | lixiaoy1 : hi, with RBDVolumeProxy(self, snapshot['volume_name']) as volume, here volume is RBDVolumeProxy object and in definition of this class, self.volume = driver.rbd.Image and | 07:11 |
*** lixiaoy1 has quit IRC | 07:11 | |
nikeshm | so how can we call (RBDVolumeProxy object).create_snap | 07:12 |
nikeshm | should not be it (RBDVolumeProxy object).volume.create_snap | 07:13 |
*** fifieldt has joined #openstack-cinder | 07:13 | |
*** alexschm has joined #openstack-cinder | 07:13 | |
lixiaoy11 | nikeshm: you may look at the function __getattr__() | 07:13 |
lixiaoy11 | https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/rbd.py#L243 | 07:14 |
nikeshm | lixiaoy11 : thanks | 07:14 |
*** abhijeetm has joined #openstack-cinder | 07:14 | |
lixiaoy11 | nikeshm: welcome | 07:14 |
*** esker has quit IRC | 07:17 | |
*** zhangjn has joined #openstack-cinder | 07:19 | |
*** esker has joined #openstack-cinder | 07:23 | |
*** zhangjn_ has joined #openstack-cinder | 07:30 | |
*** deepakcs has joined #openstack-cinder | 07:33 | |
*** zhangjn has quit IRC | 07:34 | |
*** zhangjn_ has quit IRC | 07:35 | |
*** shyama has joined #openstack-cinder | 07:48 | |
*** sgotliv_ has joined #openstack-cinder | 07:49 | |
*** bluex-pl has joined #openstack-cinder | 07:52 | |
*** esker has quit IRC | 07:52 | |
*** chlong has quit IRC | 07:56 | |
*** esker has joined #openstack-cinder | 07:59 | |
*** zxyf has quit IRC | 08:00 | |
*** haomaiwang has quit IRC | 08:01 | |
*** haomaiwang has joined #openstack-cinder | 08:01 | |
*** baojg has joined #openstack-cinder | 08:01 | |
*** lpetrut has joined #openstack-cinder | 08:01 | |
*** sgotliv_ has quit IRC | 08:03 | |
wanghao | hi~, Need cores' help to review this bug patch: https://review.openstack.org/#/c/203530/. It's a performance bug. | 08:06 |
*** ronis has joined #openstack-cinder | 08:06 | |
*** ociuhandu has quit IRC | 08:12 | |
*** vgridnev has quit IRC | 08:12 | |
openstackgerrit | XinXiaohui proposed openstack/cinder: Calculate virtual free capacity and notify https://review.openstack.org/206923 | 08:15 |
*** baojg has quit IRC | 08:18 | |
*** sgotliv_ has joined #openstack-cinder | 08:23 | |
*** esker has quit IRC | 08:24 | |
*** esker has joined #openstack-cinder | 08:29 | |
*** jistr has joined #openstack-cinder | 08:30 | |
*** baojg has joined #openstack-cinder | 08:46 | |
openstackgerrit | LisaLi proposed openstack/cinder: Data transfer between encrypted volumes and images https://review.openstack.org/216567 | 08:48 |
*** aix has quit IRC | 08:53 | |
*** esker has quit IRC | 08:54 | |
*** vgridnev has joined #openstack-cinder | 08:58 | |
*** lixiaoy11 has quit IRC | 09:00 | |
*** haomaiwang has quit IRC | 09:01 | |
*** haomaiwa_ has joined #openstack-cinder | 09:01 | |
*** jordanP has joined #openstack-cinder | 09:01 | |
*** esker has joined #openstack-cinder | 09:05 | |
*** sgotliv_ has quit IRC | 09:10 | |
*** e0ne has joined #openstack-cinder | 09:16 | |
*** aix has joined #openstack-cinder | 09:18 | |
*** afazekas_ has joined #openstack-cinder | 09:22 | |
*** afazekas_ has quit IRC | 09:22 | |
*** afazekas_ has joined #openstack-cinder | 09:23 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Adds migration abortion for available volumes https://review.openstack.org/207754 | 09:23 |
*** esker has quit IRC | 09:25 | |
*** esker has joined #openstack-cinder | 09:33 | |
*** salv-orlando has quit IRC | 09:36 | |
*** chenying has quit IRC | 09:38 | |
*** NightKhaos has quit IRC | 09:39 | |
*** sgotliv_ has joined #openstack-cinder | 09:40 | |
*** e0ne has quit IRC | 09:41 | |
*** NightKhaos has joined #openstack-cinder | 09:42 | |
*** dimsum__ has joined #openstack-cinder | 09:46 | |
*** bluex-pl has quit IRC | 09:48 | |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: WIP: Small optimization in Block Device driver https://review.openstack.org/220561 | 09:48 |
*** bluex-pl has joined #openstack-cinder | 09:48 | |
*** bluex-pl has quit IRC | 09:49 | |
*** bluex-pl has joined #openstack-cinder | 09:49 | |
*** dims_ has joined #openstack-cinder | 09:50 | |
*** davechen has left #openstack-cinder | 09:54 | |
*** dimsum__ has quit IRC | 09:55 | |
*** baojg has quit IRC | 09:56 | |
*** baojg has joined #openstack-cinder | 09:56 | |
*** e0ne has joined #openstack-cinder | 09:57 | |
*** baojg has quit IRC | 10:00 | |
*** haomaiwa_ has quit IRC | 10:01 | |
*** haomaiwang has joined #openstack-cinder | 10:01 | |
*** IanGovett has joined #openstack-cinder | 10:02 | |
*** esker has quit IRC | 10:05 | |
*** ociuhandu has joined #openstack-cinder | 10:07 | |
*** bluex-pl has quit IRC | 10:10 | |
*** bluex-pl has joined #openstack-cinder | 10:10 | |
*** esker has joined #openstack-cinder | 10:11 | |
*** julim has joined #openstack-cinder | 10:29 | |
*** julim has quit IRC | 10:33 | |
*** esker has quit IRC | 10:33 | |
*** aix has quit IRC | 10:35 | |
*** aix has joined #openstack-cinder | 10:36 | |
*** esker has joined #openstack-cinder | 10:39 | |
*** zhongjun has quit IRC | 10:43 | |
openstackgerrit | Bharat Kumar Kobagana (BharatK) proposed openstack/cinder: GlusterFS: Fixing 'migrate_volume' attribute error https://review.openstack.org/221251 | 10:48 |
*** chenying has joined #openstack-cinder | 10:53 | |
*** kodokuu has joined #openstack-cinder | 10:55 | |
*** ronis has quit IRC | 10:59 | |
*** zhongjun has joined #openstack-cinder | 10:59 | |
*** salv-orlando has joined #openstack-cinder | 11:00 | |
*** haomaiwang has quit IRC | 11:01 | |
*** 92AAB1O7Y has joined #openstack-cinder | 11:01 | |
kodokuu | Hi, I update cinder-client 1.1 to 1.2 and now I can't make cinder list | 11:02 |
kodokuu | possible to have patch note ? | 11:04 |
*** sgotliv_ has quit IRC | 11:05 | |
*** esker has quit IRC | 11:10 | |
*** kjelly_ has joined #openstack-cinder | 11:12 | |
*** 92AAB1O7Y has quit IRC | 11:12 | |
kodokuu | dhellmann Hi Can you tell me why I have this issue ? http://paste.openstack.org/show/450103/ | 11:14 |
*** esker has joined #openstack-cinder | 11:18 | |
*** marcusvrn_ has joined #openstack-cinder | 11:25 | |
*** timcl has joined #openstack-cinder | 11:26 | |
e0ne | kodokuu: hi. could you please share cinder api logs with failure? | 11:30 |
*** jaypipes has joined #openstack-cinder | 11:31 | |
e0ne | kodokuu: looks like updated cinderclient doesn't pass tenant_id:( | 11:32 |
*** zhongjun has quit IRC | 11:32 | |
kodokuu | e0ne yes, I downgrade cinderclient and works :) | 11:34 |
kodokuu | 0 log in API because cinder client GET on WRONG URL | 11:36 |
*** esker has quit IRC | 11:36 | |
kodokuu | I think the pb is How cinder client make URL for contact API | 11:36 |
e0ne | kodokuu: maybe it was buh #1464160 but I don't sure yet | 11:38 |
e0ne | bug #1464160 | 11:38 |
openstack | bug 1464160 in python-cinderclient "ERROR: Bad Request (HTTP 400) with cinder 1.2.1" [Critical,Fix released] https://launchpad.net/bugs/1464160 - Assigned to John Griffith (john-griffith) | 11:38 |
kodokuu | e0ne I use public_endpoint | 11:39 |
*** dobson has quit IRC | 11:39 | |
kodokuu | I don't see the fix for this bug | 11:39 |
*** esker has joined #openstack-cinder | 11:45 | |
*** dobson has joined #openstack-cinder | 11:45 | |
*** jecarey has quit IRC | 11:56 | |
*** ankit_ag has joined #openstack-cinder | 11:56 | |
*** bkopilov has quit IRC | 12:00 | |
*** kodokuu has quit IRC | 12:01 | |
*** e0ne has quit IRC | 12:01 | |
*** alexpilotti_ has quit IRC | 12:03 | |
*** tpsilva has joined #openstack-cinder | 12:04 | |
*** lkong has joined #openstack-cinder | 12:04 | |
*** e0ne has joined #openstack-cinder | 12:04 | |
*** esker has quit IRC | 12:07 | |
*** amoturi has joined #openstack-cinder | 12:10 | |
openstackgerrit | Vipin Balachandran proposed openstack/cinder: Enable certificate verification during image copy https://review.openstack.org/221272 | 12:11 |
*** pots has joined #openstack-cinder | 12:12 | |
*** raildo-afk is now known as raildo | 12:12 | |
*** zhangjn has joined #openstack-cinder | 12:15 | |
*** zhangjn has quit IRC | 12:17 | |
*** esker has joined #openstack-cinder | 12:17 | |
*** scottda has joined #openstack-cinder | 12:17 | |
*** abhijeetm has quit IRC | 12:17 | |
*** dalgaaf has joined #openstack-cinder | 12:18 | |
*** markvoelker has joined #openstack-cinder | 12:19 | |
*** deepakcs has quit IRC | 12:20 | |
*** markvoelker has quit IRC | 12:20 | |
*** markvoelker has joined #openstack-cinder | 12:20 | |
*** ronis has joined #openstack-cinder | 12:21 | |
*** chlong has joined #openstack-cinder | 12:22 | |
*** sgotliv_ has joined #openstack-cinder | 12:23 | |
*** daemontool_ has quit IRC | 12:23 | |
*** e0ne has quit IRC | 12:26 | |
*** salv-orlando has quit IRC | 12:34 | |
*** esker has quit IRC | 12:38 | |
*** merooney has joined #openstack-cinder | 12:38 | |
*** edmondsw has joined #openstack-cinder | 12:41 | |
*** merooeny_ has joined #openstack-cinder | 12:41 | |
*** smoriya has quit IRC | 12:42 | |
*** daemontool_ has joined #openstack-cinder | 12:43 | |
openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Support initialization state in Backup Manager https://review.openstack.org/216728 | 12:44 |
*** esker has joined #openstack-cinder | 12:44 | |
*** salv-orlando has joined #openstack-cinder | 12:44 | |
*** merooney has quit IRC | 12:44 | |
*** crose has joined #openstack-cinder | 12:50 | |
*** merooeny_ has quit IRC | 12:50 | |
*** esker has quit IRC | 12:52 | |
*** esker has joined #openstack-cinder | 12:52 | |
*** ankit_ag has quit IRC | 12:54 | |
*** esker has quit IRC | 12:54 | |
*** esker has joined #openstack-cinder | 12:56 | |
*** esker has quit IRC | 12:56 | |
*** jordanP has quit IRC | 12:58 | |
*** erhudy has joined #openstack-cinder | 13:01 | |
*** zhangjn has joined #openstack-cinder | 13:01 | |
*** eduardo has joined #openstack-cinder | 13:03 | |
*** bswartz has joined #openstack-cinder | 13:03 | |
*** pots has left #openstack-cinder | 13:03 | |
*** vgridnev has quit IRC | 13:07 | |
*** vgridnev has joined #openstack-cinder | 13:07 | |
*** e0ne has joined #openstack-cinder | 13:08 | |
*** porrua has joined #openstack-cinder | 13:08 | |
*** Yogi1 has joined #openstack-cinder | 13:09 | |
*** jgregor has joined #openstack-cinder | 13:10 | |
*** haomaiwang has joined #openstack-cinder | 13:10 | |
*** Yogi11 has joined #openstack-cinder | 13:10 | |
*** changbl has quit IRC | 13:11 | |
*** jecarey has joined #openstack-cinder | 13:14 | |
*** Yogi1 has quit IRC | 13:14 | |
*** jgregor has quit IRC | 13:15 | |
*** gouthamr has joined #openstack-cinder | 13:15 | |
*** chenying has quit IRC | 13:16 | |
openstackgerrit | Wilson Liu proposed openstack/cinder: Fix way of get LUN id in Huawei driver https://review.openstack.org/209749 | 13:16 |
*** dustins has joined #openstack-cinder | 13:17 | |
*** zhangjn has quit IRC | 13:20 | |
*** crose has quit IRC | 13:20 | |
*** kjelly_ has quit IRC | 13:20 | |
*** BharatK has quit IRC | 13:22 | |
*** daemontool_ has quit IRC | 13:22 | |
*** Yogi11 has left #openstack-cinder | 13:22 | |
*** breitz has joined #openstack-cinder | 13:23 | |
*** akerr has joined #openstack-cinder | 13:24 | |
*** jgregor has joined #openstack-cinder | 13:24 | |
jgriffith | e0ne: So the problem is the version discovery stuff that was aded in 1.2 | 13:27 |
*** lprice has joined #openstack-cinder | 13:27 | |
e0ne | jgriffith: thanks for the clarification. I mentioned bug #1464160 earlier. kodokuu ^^ | 13:28 |
openstack | bug 1464160 in python-cinderclient "ERROR: Bad Request (HTTP 400) with cinder 1.2.1" [Critical,Fix released] https://launchpad.net/bugs/1464160 - Assigned to John Griffith (john-griffith) | 13:28 |
jgriffith | Ahh... yep, that looks like the right one | 13:28 |
jgriffith | :( | 13:28 |
*** ronis has quit IRC | 13:29 | |
e0ne | jgriffith: I found it in the "revert commit message" | 13:29 |
smcginnis | jgriffith: I thought we fixed those version discovery issues? | 13:30 |
smcginnis | Or rather: /we/you/ | 13:30 |
jgriffith | smcginnis: not *really*, my fix was to revert it | 13:31 |
smcginnis | :[ | 13:31 |
amoturi | jgriffith: could you take another look at this patchset. https://review.openstack.org/#/c/211299/ | 13:34 |
jgriffith | amoturi: seems "ok" | 13:36 |
jgriffith | amoturi: but if you're using pools don't you want to report them? Otherwise that will break the scheduler pool assignment for you? | 13:37 |
jgriffith | amoturi: or are you not exposing them? | 13:37 |
jgriffith | amoturi: ahh... never mind | 13:38 |
jgriffith | I think I get it | 13:38 |
amoturi | jgriffith: pools are not really relevant because all the volumes are created in a project and that report better usage data. | 13:38 |
*** martyturner has joined #openstack-cinder | 13:39 | |
jgriffith | amoturi: yeah, I kinda get that looking at how your rest call works | 13:39 |
jgriffith | emphasis on "kinda" :) | 13:39 |
amoturi | jgriffith: :) | 13:39 |
jgriffith | amoturi: I'm +2 | 13:39 |
*** lprice1 has joined #openstack-cinder | 13:39 | |
amoturi | jgriffith: i am also not sure if the 'bracket instead of \' is valid | 13:39 |
jgriffith | amoturi: well, in some cases it's not | 13:40 |
jgriffith | amoturi: and I'm not sure about your case there, I was going to ask you to respond/have a look at that | 13:40 |
jgriffith | amoturi: the \ has fallen out of favor :( | 13:40 |
jgriffith | amoturi: but there are cases like "with" where the parens don't work | 13:41 |
amoturi | jgriffith: it did pass pep8 :( | 13:41 |
jgriffith | amoturi: which is why I'm fine with it frankly :) | 13:41 |
jgriffith | amoturi: until it's automated it's a "guideline/preferrence" | 13:41 |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Implement extend_volume method to Block Device driver https://review.openstack.org/221303 | 13:42 |
*** lprice has quit IRC | 13:42 | |
amoturi | jgriffith: thanks :) . ahh i see. | 13:43 |
*** gouthamr has quit IRC | 13:44 | |
*** gouthamr has joined #openstack-cinder | 13:44 | |
jgriffith | smcginnis: is the cross dependency tag actually working? | 13:48 |
jgriffith | smcginnis: ie https://review.openstack.org/#/c/220830/ | 13:49 |
jgriffith | smcginnis: passes because it's actually loading the version of devstack? | 13:49 |
*** takedakn has joined #openstack-cinder | 13:49 | |
jgriffith | smcginnis: I didn't think that was possible (node pool and all) | 13:49 |
*** gouthamr has quit IRC | 13:55 | |
*** gouthamr has joined #openstack-cinder | 13:56 | |
*** r-daneel has joined #openstack-cinder | 13:57 | |
*** haomaiwang has quit IRC | 14:01 | |
*** haomaiwang has joined #openstack-cinder | 14:01 | |
*** jistr is now known as jistr|call | 14:01 | |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Implement extend_volume method to Block Device driver https://review.openstack.org/221303 | 14:03 |
e0ne | jgriffith: afaik, if depended(parent) patch doesn't have +A, the child one won't be merged | 14:04 |
*** eharney has joined #openstack-cinder | 14:04 | |
jgriffith | e0ne: yeah | 14:04 |
jgriffith | e0ne: I didn't know that worked with devstack :) | 14:04 |
smcginnis | jgriffith: I think the devstack patch went in last night. I need to check. | 14:05 |
jgriffith | e0ne: because the devstack nodes are actually "pre-built" | 14:05 |
e0ne | jgriffith: https://review.openstack.org/#/c/194726/ | 14:05 |
yuriy_n17 | jgriffith: Thanks a lot for review. | 14:05 |
e0ne | jgriffith: it's a patch for devstack. parent is not approved yet. | 14:05 |
smcginnis | I got around it on our CI for now by pulling in a Nova patch to explicitly add the necessary os-brick rootwrap cmd. | 14:05 |
jgriffith | yuriy_n17: thank you for the quick update | 14:05 |
e0ne | jgriffith: calls 'git clone devstack-repo' for every job | 14:06 |
e0ne | s/calls/infra calls | 14:06 |
jgriffith | e0ne: but that's different I think.. that's a devstack patch depending on another project, not the other way around | 14:06 |
jgriffith | e0ne: ahh... ok, cool | 14:06 |
jgriffith | e0ne: so I learn something new already today :) | 14:06 |
jgriffith | smcginnis: and FWIW I did look and it hadn't merged yet | 14:07 |
smcginnis | Didn't land yet: https://review.openstack.org/#/c/207677/ | 14:07 |
smcginnis | ;) | 14:07 |
jgriffith | the devstack change, thus my comment :) | 14:07 |
jgriffith | but I'm a gonna take a chance and "trust" you and e0ne :) | 14:08 |
smcginnis | jgriffith: Yeah, kind of an odd one with it being devstack. | 14:08 |
smcginnis | A bit chicken and egg. | 14:08 |
e0ne | jgriffith: http://logs.openstack.org/50/218750/11/check/gate-tempest-dsvm-full/31f5860/logs/devstack-gate-setup-workspace-new.txt.gz#_2015-09-03_06_34_46_565 - clonning devstack | 14:08 |
jgriffith | e0ne: well, that pretty much settles that :) | 14:09 |
smcginnis | Still have a bad feeling about needing to copy over os-brick's rootwrap filters though. | 14:09 |
smcginnis | I think we're going to run in to trouble with deployments missing that necessary step. | 14:10 |
*** rick-chen has joined #openstack-cinder | 14:10 | |
smcginnis | I would rather it be a little more automatic, but I guess we'll see. | 14:10 |
e0ne | jgriffith: thanks for +2 for https://review.openstack.org/#/c/221303/. but what about feature freeze? | 14:10 |
*** mriedem has joined #openstack-cinder | 14:12 | |
*** diablo_rojo has joined #openstack-cinder | 14:15 | |
yuriy_n17 | jgriffith: Sorry for disturbing. What about to review https://review.openstack.org/#/c/220561/ | 14:16 |
jgriffith | e0ne: that's on master | 14:17 |
jgriffith | e0ne: the branch is cut | 14:18 |
jgriffith | e0ne: https://www.dropbox.com/s/vj6osneq7q8b6zd/Screenshot%202015-09-08%2008.18.35.png?dl=0 | 14:18 |
jgriffith | no? | 14:18 |
e0ne | jgriffith: hm... I don't see it on github | 14:18 |
e0ne | jgriffith: may be it is not synced yet | 14:19 |
e0ne | "file is uploading...." | 14:19 |
jgriffith | e0ne: that link is a screenshot of the tag in github | 14:20 |
jgriffith | :) | 14:20 |
*** jistr|call is now known as jistr | 14:20 | |
e0ne | jgriffith: I missed something. will we have a branch from the tag? | 14:21 |
jgriffith | e0ne: that all has changed a bit and now they do tags instead of branches until stable I believe | 14:24 |
jgriffith | e0ne: in other words, any release gets a "tag" | 14:24 |
jgriffith | hmmm | 14:25 |
e0ne | jgriffith: is any wiki or openstack-dev ml thread about it? | 14:25 |
jgriffith | I'll check with ttx as things have changed since I did this | 14:25 |
jgriffith | I dunno :) | 14:25 |
*** dustins has quit IRC | 14:25 | |
rick-chen | jgriffith: HI John, Does Mike return? | 14:25 |
smcginnis | I seem to remember an ML thread about it. | 14:25 |
e0ne | jgriffith: if we'll merge something for Mitaka now, how will we create stable branch for L? | 14:25 |
*** zhangjn has joined #openstack-cinder | 14:26 | |
*** zhangjn has quit IRC | 14:26 | |
*** dims_ has quit IRC | 14:26 | |
*** dimsum__ has joined #openstack-cinder | 14:27 | |
*** bkopilov has joined #openstack-cinder | 14:27 | |
smcginnis | e0ne: Doesn't appear this has been updated: https://wiki.openstack.org/wiki/Branch_Model | 14:27 |
*** vlaza has quit IRC | 14:29 | |
jgriffith | e0ne: you are correct | 14:29 |
jgriffith | smcginnis: e0ne Mast is currently still feature freeze until an rc branch is cut | 14:29 |
jgriffith | I'll go fix up my screw ups | 14:30 |
tbarron | when are our rc targets (or is that a meeting topic) ? | 14:30 |
e0ne | jgriffith: thanks for clarification. it's good to be on the same page | 14:30 |
openstackgerrit | Derrick Wippler proposed openstack/cinder: Hacking log format arg check https://review.openstack.org/220246 | 14:30 |
openstackgerrit | Ivan Kolodyazhny proposed openstack/python-cinderclient: Remove unused code from cinderclient.utils module https://review.openstack.org/221323 | 14:32 |
e0ne | tbarron: here is a schedule https://wiki.openstack.org/wiki/Liberty_Release_Schedule | 14:32 |
*** alexschm has quit IRC | 14:33 | |
*** edtubill has joined #openstack-cinder | 14:33 | |
tbarron | e0ne: ty! so rc0, rc1, etc. may vary within the window shown, right? (trying to map https://wiki.openstack.org/wiki/Branch_Model onto the schedule you supplied). | 14:34 |
*** raildo is now known as raildo-afk | 14:34 | |
jgriffith | smcginnis: e0ne DuncanT hemna_ geguileo eharney winston-d so just to make sure we're all on the same page; master branch is still under feature freeze | 14:35 |
geguileo | jgriffith: Thanks | 14:35 |
jgriffith | You all probably already knew that and I was the only one that didn't :( | 14:35 |
jgriffith | but just to make sure :) | 14:35 |
smcginnis | jgriffith: Bug fix only, right? Real bugs, not "my driver doesn't support feature x" bugs. | 14:35 |
jgriffith | smcginnis: correct | 14:35 |
jgriffith | smcginnis: and even more so, think of it as "this is going in to Liberty" | 14:36 |
smcginnis | Yep | 14:36 |
jgriffith | so don't break stuff :) | 14:36 |
smcginnis | I do think there are some good things out there right now to get in to L. | 14:36 |
jgriffith | smcginnis: agreed | 14:36 |
smcginnis | jgriffith: But yeah, definitely should consider the risk of some of it going in at this point. | 14:36 |
*** raildo-afk is now known as raildo | 14:37 | |
*** zhangjn has joined #openstack-cinder | 14:37 | |
*** rhagarty has quit IRC | 14:38 | |
*** teju has joined #openstack-cinder | 14:38 | |
jgriffith | smcginnis: I'm lookign trhough https://review.openstack.org/#/q/status:merged+cinder,n,z to make sure I didn't merge anything i shouldn't have ;) | 14:38 |
*** rhagarty_ has quit IRC | 14:38 | |
smcginnis | jgriffith: I hadn't noticed anything questionable. :) | 14:38 |
*** zhangjn has quit IRC | 14:39 | |
smcginnis | Well, that Oracle one maybe, but that wasn't you. | 14:39 |
jgriffith | yeah... I"m fine with it | 14:39 |
*** takedakn has quit IRC | 14:39 | |
jgriffith | and I think tbarron 's stuff got merged so that's cool too | 14:39 |
tbarron | jgriffith: yes, thank you and too all that swarmed on that queue. | 14:39 |
tbarron | to all | 14:39 |
jgriffith | geguileo: ping | 14:42 |
geguileo | jgriffith: pong | 14:45 |
jgriffith | geguileo: hey... if you have some time later or maybe tomorrow morning... | 14:45 |
jgriffith | geguileo: I was wondering if you could help me understand how https://review.openstack.org/#/c/216377/10/cinder/volume/api.py improves things | 14:46 |
geguileo | jgriffith: Sure, when is a good time for you? | 14:47 |
jgriffith | geguileo: I was hoping right now, but sadly some thing has come up... and you're in an opposite TZ if I recall no? | 14:48 |
jgriffith | geguileo: maybe tomorrow morning before weekly meeting if that works ok for you? | 14:48 |
geguileo | jgriffith: Yep, here it's 16:48 | 14:48 |
jgriffith | geguileo: :) | 14:48 |
geguileo | jgriffith: Sure, just ping me and most likely I'll be available | 14:48 |
jgriffith | geguileo: I'll play around with it some more on my own tonight and see if I can educate myself a bit :) | 14:48 |
jgriffith | geguileo: sounds great, thank you! | 14:49 |
geguileo | jgriffith: np | 14:49 |
*** baojg has joined #openstack-cinder | 14:50 | |
*** david-ly_ is now known as david-lyle | 14:53 | |
*** dustins has joined #openstack-cinder | 14:54 | |
*** tsekiyama has joined #openstack-cinder | 14:55 | |
*** tpsilva has quit IRC | 14:55 | |
*** rick-chen has quit IRC | 14:57 | |
*** dustins has quit IRC | 14:57 | |
*** cbits has joined #openstack-cinder | 14:58 | |
*** anshul has quit IRC | 14:59 | |
*** martyturner has quit IRC | 14:59 | |
*** timcl1 has joined #openstack-cinder | 15:00 | |
*** zhipeng has joined #openstack-cinder | 15:00 | |
*** fifieldt has quit IRC | 15:00 | |
*** cbader has joined #openstack-cinder | 15:00 | |
*** haomaiwang has quit IRC | 15:01 | |
*** martyturner has joined #openstack-cinder | 15:01 | |
*** haomaiwang has joined #openstack-cinder | 15:01 | |
*** haomaiwang has quit IRC | 15:01 | |
*** timcl has quit IRC | 15:02 | |
*** hodos has joined #openstack-cinder | 15:02 | |
*** dustins has joined #openstack-cinder | 15:02 | |
*** cbits has left #openstack-cinder | 15:03 | |
openstackgerrit | Vipin Balachandran proposed openstack/cinder: Enable certificate verification during image copy https://review.openstack.org/221272 | 15:06 |
*** teju has quit IRC | 15:08 | |
*** baojg has quit IRC | 15:09 | |
*** akerr is now known as akerr_away | 15:12 | |
*** akerr_away is now known as akerr | 15:12 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/python-cinderclient: Remove unused code from cinderclient.utils module https://review.openstack.org/221323 | 15:14 |
*** changbl has joined #openstack-cinder | 15:14 | |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Small optimization in Block Device driver https://review.openstack.org/220561 | 15:16 |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Implement extend_volume method to Block Device driver https://review.openstack.org/221303 | 15:18 |
*** agarciam has joined #openstack-cinder | 15:18 | |
*** jungleboyj has joined #openstack-cinder | 15:20 | |
*** salv-orlando has quit IRC | 15:20 | |
*** tpsilva has joined #openstack-cinder | 15:21 | |
*** rushil has joined #openstack-cinder | 15:23 | |
*** dims_ has joined #openstack-cinder | 15:25 | |
*** afazekas_ has quit IRC | 15:28 | |
*** monsoon has joined #openstack-cinder | 15:29 | |
*** dimsum__ has quit IRC | 15:29 | |
monsoon | hi guys! | 15:30 |
*** salv-orlando has joined #openstack-cinder | 15:30 | |
monsoon | is the volume backend remotefs a real back-end? or is it just a base class? | 15:31 |
*** jdurgin1 has joined #openstack-cinder | 15:31 | |
eharney | monsoon: just a base class | 15:31 |
monsoon | thank you) | 15:32 |
*** vgridnev has quit IRC | 15:33 | |
monsoon | so that means that this documentation http://docs.openstack.org/kilo/config-reference/content/section_cinder.conf.html | 15:33 |
monsoon | is autogenerated? | 15:33 |
monsoon | since it has configuration options for remotefs | 15:33 |
eharney | monsoon: it is, but drivers do use those options | 15:34 |
monsoon | ok, now it's totally clear. thank you again) | 15:34 |
*** guest5 has joined #openstack-cinder | 15:35 | |
*** dobson has quit IRC | 15:36 | |
*** salv-orlando has quit IRC | 15:36 | |
*** dobson has joined #openstack-cinder | 15:39 | |
*** devlaps has joined #openstack-cinder | 15:43 | |
*** timcl1 has quit IRC | 15:45 | |
*** edtubill has left #openstack-cinder | 15:45 | |
*** edtubill has joined #openstack-cinder | 15:45 | |
*** hodos has quit IRC | 15:46 | |
*** timcl has joined #openstack-cinder | 15:49 | |
*** martyturner has quit IRC | 15:54 | |
*** agarciam1 has joined #openstack-cinder | 15:55 | |
*** guest5 has quit IRC | 15:56 | |
*** agarciam has quit IRC | 15:56 | |
openstackgerrit | Yusuke Hayashi proposed openstack/python-cinderclient: Fix incorrect exception message in cinderclient https://review.openstack.org/210827 | 15:56 |
*** tpsilva has quit IRC | 15:57 | |
*** rushil has quit IRC | 15:57 | |
*** hemna_ is now known as hemna | 15:57 | |
*** r-daneel has quit IRC | 15:58 | |
*** timcl1 has joined #openstack-cinder | 15:59 | |
*** tsekiyama has quit IRC | 16:00 | |
*** timcl has quit IRC | 16:01 | |
lpetrut | Hi guys. I was wondering: shouldn't there be a check for missing volume type id here? https://github.com/openstack/cinder/blob/master/cinder/objects/volume.py#L241-L243 | 16:03 |
*** martyturner has joined #openstack-cinder | 16:03 | |
*** annegentle has joined #openstack-cinder | 16:05 | |
*** Apoorva has joined #openstack-cinder | 16:06 | |
*** zhenguo has quit IRC | 16:08 | |
*** zhipeng has quit IRC | 16:08 | |
*** jistr has quit IRC | 16:11 | |
*** leeantho has joined #openstack-cinder | 16:15 | |
*** esker has joined #openstack-cinder | 16:16 | |
*** Zhongjun_ has joined #openstack-cinder | 16:19 | |
*** tpsilva has joined #openstack-cinder | 16:19 | |
*** rhagarty has joined #openstack-cinder | 16:26 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/221040 | 16:26 |
*** tpsilva has quit IRC | 16:26 | |
*** rhagarty_ has joined #openstack-cinder | 16:26 | |
*** bluex-pl has quit IRC | 16:31 | |
*** dims_ has quit IRC | 16:31 | |
*** dimsum__ has joined #openstack-cinder | 16:32 | |
*** Yogi1 has joined #openstack-cinder | 16:32 | |
*** dannywilson has joined #openstack-cinder | 16:36 | |
*** sghanekar_ has joined #openstack-cinder | 16:36 | |
*** vgridnev has joined #openstack-cinder | 16:37 | |
openstackgerrit | Anthony Lee proposed openstack/cinder: Fix QoS keys not being available to scheduler filter and weighers https://review.openstack.org/214826 | 16:38 |
*** zhangjn has joined #openstack-cinder | 16:38 | |
*** andreykurilin__ has joined #openstack-cinder | 16:39 | |
*** mdbooth has quit IRC | 16:39 | |
*** Guest60469 is now known as mgagne | 16:40 | |
*** mgagne has joined #openstack-cinder | 16:40 | |
*** jdurgin1 has quit IRC | 16:42 | |
*** leeantho has quit IRC | 16:43 | |
*** dflorea has joined #openstack-cinder | 16:44 | |
*** winston-d has quit IRC | 16:46 | |
*** winston-d has joined #openstack-cinder | 16:47 | |
*** leeantho has joined #openstack-cinder | 16:47 | |
*** ronis has joined #openstack-cinder | 16:49 | |
*** tsekiyama has joined #openstack-cinder | 16:50 | |
*** crose has joined #openstack-cinder | 16:50 | |
*** lkong has quit IRC | 16:53 | |
*** lpetrut has quit IRC | 16:56 | |
*** e0ne has quit IRC | 16:56 | |
*** earlephilhower has quit IRC | 16:57 | |
*** aix has quit IRC | 16:58 | |
*** tsekiyama has quit IRC | 17:00 | |
*** harlowja has joined #openstack-cinder | 17:03 | |
*** afazekas_ has joined #openstack-cinder | 17:05 | |
*** dannywilson has quit IRC | 17:06 | |
*** pv has joined #openstack-cinder | 17:08 | |
pv | i had a question on volume-attach, currently my attach sometimes goes through and sometimes it doesn't | 17:09 |
pv | when it doesnt go through my n-cpu log outputs Looking for Fibre Channel dev /dev/disk/by-path/pci-0000:02:00.0-fc-0x10003c912b666600-lun-0 _wait_for_device_discovery which it keeps trying until it eventually fails | 17:09 |
pv | i don't get why that path is sometimes available and sometimes it isnt found, i try to ls the path and when volume-attach fails it says the path doesn't exist | 17:10 |
pv | the target port is the only available target on the machine I'm using and everything else seems to be in place, i know for a fact it has worked before it just is really sporadic on when it does work and when it doesnt | 17:11 |
pv | any ideas? | 17:11 |
hemna | is there a FC zone for it ? | 17:11 |
hemna | if the volume doesn't show up, then there is nothing that nova/cinder can do | 17:11 |
pv | yeah | 17:11 |
pv | it is zoned | 17:12 |
hemna | need to check the fabric | 17:12 |
pv | ok thought so, its not really an openstack issue i just wanted to see if anyone's ever run into something similar | 17:12 |
pv | with their own FC driver | 17:12 |
hemna | yah | 17:12 |
hemna | that typically happens to me when the zone isn't available or is down between the 2 endpoints. | 17:12 |
pv | hmm alright, ill check the zoning again and see if its all properly zoned | 17:13 |
pv | before what use to work was if it wasnt found, i'd do a quick rescan and then the device would be found | 17:13 |
pv | but even that doesn't seem to be working today haha | 17:13 |
pv | thanks anyways! | 17:14 |
hemna | np | 17:14 |
*** tsekiyama has joined #openstack-cinder | 17:14 | |
*** markstur has quit IRC | 17:18 | |
*** ronis has quit IRC | 17:20 | |
*** zhangjn has quit IRC | 17:21 | |
*** afazekas_ has quit IRC | 17:24 | |
*** tsekiyama has quit IRC | 17:24 | |
*** Apoorva has quit IRC | 17:25 | |
*** Apoorva has joined #openstack-cinder | 17:26 | |
*** ociuhandu has quit IRC | 17:26 | |
*** earlephilhower has joined #openstack-cinder | 17:27 | |
*** dflorea has quit IRC | 17:29 | |
jungleboyj | hemna: You still around? | 17:31 |
*** markstur has joined #openstack-cinder | 17:31 | |
*** dflorea_ has joined #openstack-cinder | 17:31 | |
hemna | jungleboyj, yah | 17:32 |
jungleboyj | Oh, I think I found the answer. | 17:33 |
jungleboyj | Need to use the --allow-multiattach option to mark a volume as multi-attach capable. | 17:33 |
hemna | ah yah | 17:33 |
hemna | at create time | 17:33 |
jungleboyj | Which hasn't yet been released as we haven't done a new cinderclient drop yet. | 17:34 |
*** sghanekar_ has quit IRC | 17:37 | |
*** crose has quit IRC | 17:38 | |
*** mudassirlatif has joined #openstack-cinder | 17:42 | |
openstackgerrit | Eric Harney proposed openstack/cinder: Add migrate_volume to BaseVD https://review.openstack.org/221425 | 17:44 |
*** anshul has joined #openstack-cinder | 17:45 | |
*** e0ne has joined #openstack-cinder | 17:47 | |
openstackgerrit | Eric Harney proposed openstack/cinder: Add migrate_volume to BaseVD https://review.openstack.org/221425 | 17:51 |
*** merooney has joined #openstack-cinder | 17:53 | |
*** vgridnev has quit IRC | 18:00 | |
*** aix has joined #openstack-cinder | 18:01 | |
*** r-daneel has joined #openstack-cinder | 18:02 | |
*** rushil has joined #openstack-cinder | 18:04 | |
*** dflorea_ has quit IRC | 18:05 | |
*** vgridnev has joined #openstack-cinder | 18:06 | |
*** vgridnev has quit IRC | 18:06 | |
*** dflorea has joined #openstack-cinder | 18:11 | |
openstackgerrit | Patrick East proposed openstack/cinder: Correctly report multiattach in Pure drivers https://review.openstack.org/221439 | 18:14 |
*** dflorea has quit IRC | 18:15 | |
*** shyama has quit IRC | 18:15 | |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove API races from delete methods https://review.openstack.org/205835 | 18:17 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add atomic conditional updates to objects https://review.openstack.org/205834 | 18:17 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove API races on extend and volume_upload_image https://review.openstack.org/216378 | 18:17 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Move get_by_id to CinderObject https://review.openstack.org/218012 | 18:17 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Improve metadata update operations https://review.openstack.org/216376 | 18:17 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove API races from attach and detach methods https://review.openstack.org/216377 | 18:17 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove API races on replication methods https://review.openstack.org/221441 | 18:17 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: WIP: Remove API races from migrate_volume https://review.openstack.org/221442 | 18:17 |
*** merooeny_ has joined #openstack-cinder | 18:19 | |
*** merooney has quit IRC | 18:22 | |
*** rushil has quit IRC | 18:25 | |
openstackgerrit | Eric Harney proposed openstack/cinder: Add migrate_volume to BaseVD https://review.openstack.org/221425 | 18:26 |
openstackgerrit | Eric Harney proposed openstack/cinder: Add manage_existing and unmanage to BaseVD https://review.openstack.org/221445 | 18:26 |
*** Zhongjun_ has quit IRC | 18:27 | |
*** merooeny_ has quit IRC | 18:28 | |
*** e0ne has quit IRC | 18:29 | |
*** Apoorva_ has joined #openstack-cinder | 18:30 | |
*** merooney has joined #openstack-cinder | 18:30 | |
*** openstackgerrit has quit IRC | 18:31 | |
*** openstackgerrit has joined #openstack-cinder | 18:32 | |
*** dflorea has joined #openstack-cinder | 18:32 | |
*** Apoorva has quit IRC | 18:33 | |
*** mudassirlatif has quit IRC | 18:37 | |
*** dflorea has quit IRC | 18:37 | |
*** dflorea has joined #openstack-cinder | 18:39 | |
*** ociuhandu has joined #openstack-cinder | 18:39 | |
*** dflorea has quit IRC | 18:41 | |
*** angela-s has joined #openstack-cinder | 18:44 | |
*** vgridnev has joined #openstack-cinder | 18:48 | |
*** garthb has joined #openstack-cinder | 18:49 | |
*** dflorea has joined #openstack-cinder | 18:50 | |
*** dflorea has quit IRC | 18:50 | |
*** dflorea has joined #openstack-cinder | 18:52 | |
*** tpsilva has joined #openstack-cinder | 18:53 | |
jungleboyj | hemna: eharney jgriffith Any reviews that we are finding associated with BPs should be -2'd right now? Agreed? | 18:57 |
jgriffith | jungleboyj: correct | 18:57 |
eharney | jungleboyj: yes | 18:58 |
jgriffith | jungleboyj: we're still under feature freeze | 18:58 |
jgriffith | jungleboyj: and keep in mind what you merge now, goes in to Liberty | 18:58 |
jgriffith | jungleboyj: so be picky :) | 18:58 |
jungleboyj | Ok, great. | 18:58 |
jungleboyj | Just didn't want to be an jerk for now reason. | 18:59 |
jungleboyj | *no | 19:00 |
eharney | jungleboyj: don't raise the bar too high for the rest of us :) | 19:02 |
jungleboyj | :-) | 19:02 |
*** vgridnev has quit IRC | 19:09 | |
ameade | hemna: /usr/local/bin/nova-rootwrap: Unauthorized command: scsi_id | 19:13 |
ameade | hemna: any idea why it's using nova-rootwrap? should it be doing brick rootwrap? | 19:14 |
*** aorourke has joined #openstack-cinder | 19:16 | |
*** vgridnev has joined #openstack-cinder | 19:17 | |
eharney | ameade: iirc it uses nova-rootwrap when loaded in nova, and cinder-rootwrap when loaded in cinder? | 19:18 |
ameade | eharney: i think that's ok but nova's rootwrap filters dont have scsi_d in there | 19:20 |
ameade | i wonder how gate is working.. gonna go poke around in their configs | 19:20 |
eharney | ameade: there's a rootwrap filters config in os-brick itself that needs to get installed into /etc/nova/... | 19:21 |
ameade | oh maybe i just need ot update devstack or somthing | 19:21 |
ameade | i'm sure thats it, thanks! | 19:22 |
*** p0rtal has joined #openstack-cinder | 19:24 | |
*** martyturner has quit IRC | 19:26 | |
scottda | Anyone interested in nova force_detach of a volume there's a working patch up for review: | 19:27 |
scottda | https://review.openstack.org/#/c/184537/ | 19:27 |
*** dflorea has quit IRC | 19:28 | |
*** martyturner has joined #openstack-cinder | 19:30 | |
*** pots1 has joined #openstack-cinder | 19:30 | |
*** aix has quit IRC | 19:32 | |
*** palendae has joined #openstack-cinder | 19:32 | |
*** dflorea has joined #openstack-cinder | 19:34 | |
*** tpsilva has quit IRC | 19:34 | |
smcginnis | ameade: https://review.openstack.org/#/c/207677/7 | 19:43 |
smcginnis | ameade: It's in the works. Hopefully soon. | 19:43 |
smcginnis | ameade: That command (as far as I know) is only used for FC, so it's just affecting the FC driver test runs. | 19:44 |
smcginnis | ameade: Therefore Jenkins is fine. | 19:44 |
smcginnis | ameade: Temporarily added scsi_id to cinder, but once that devstack patch lands it should be removed. | 19:44 |
smcginnis | ameade: Nova didn't want the temporary workaround. | 19:44 |
ameade | ah gotcha, thanks | 19:45 |
smcginnis | ameade: You can add NOVA_BRANCH=refs/changes/50/220750/1 to your local.conf to pull in the nova side. | 19:45 |
*** andreykurilin__ has quit IRC | 19:46 | |
*** pots1 has quit IRC | 19:46 | |
*** dflorea has quit IRC | 19:47 | |
*** theanalyst has quit IRC | 19:48 | |
*** theanalyst has joined #openstack-cinder | 19:52 | |
hemna | ameade, the rootwrap is in brick now | 19:53 |
hemna | ameade, there is a devstack patch still outstanding that fixes it | 19:53 |
smcginnis | hemna: Any thoughts on how to make the rootwrap piece a little more automatic? | 19:56 |
smcginnis | hemna: I'm worried we are going to have issues with the brick filters not getting copied over to the right place. | 19:56 |
*** dflorea has joined #openstack-cinder | 19:56 | |
*** jecarey has quit IRC | 19:56 | |
hemna | afaik, currently the devstack patch is the only way | 19:57 |
smcginnis | hemna: It would be great if the filter file for os-brick would just get pulled in, but that probably means updates to oslo-rootwrap. | 19:57 |
hemna | nova folks objected to having the filters added in nova's existing computer.filters file | 19:57 |
hemna | which makes sense, because new filters can be added in brick and then they don't exist in nova/cinder | 19:58 |
smcginnis | hemna: Yeah, for both cinder and nova that could be a pain. | 19:58 |
eharney | hemna: i wonder if they would support some kind of #include /etc/os-brick/filters ability? does rootwrap have that? | 19:58 |
smcginnis | hemna: Hopefully my fears are unfounded and it's not an issue. | 19:58 |
*** palendae has left #openstack-cinder | 19:58 | |
hemna | nah, they aren't unfounded :P | 19:58 |
smcginnis | Probably too late in the cycle to really do much about it anyway other than document needing to copy the file. | 19:59 |
smcginnis | Bah, meeting time! bbl | 19:59 |
*** dimsum__ is now known as dims | 20:03 | |
*** dustins has quit IRC | 20:04 | |
*** dims is now known as Guest80071 | 20:04 | |
*** timcl1 has quit IRC | 20:04 | |
*** lcurtis has joined #openstack-cinder | 20:06 | |
*** Guest80071 is now known as dims__ | 20:08 | |
*** martyturner has quit IRC | 20:09 | |
*** mudassirlatif has joined #openstack-cinder | 20:09 | |
*** martyturner has joined #openstack-cinder | 20:10 | |
openstackgerrit | Patrick East proposed openstack/cinder: Correctly report multiattach in Pure drivers https://review.openstack.org/221439 | 20:10 |
*** martyturner1 has joined #openstack-cinder | 20:13 | |
*** martyturner has quit IRC | 20:15 | |
*** guest5 has joined #openstack-cinder | 20:24 | |
patrickeast | hey so for the image cache and internal tenant feature... whats the right (best?) place to add documentation for that? in the config reference somewhere like here(ish) http://docs.openstack.org/kilo/config-reference/content/ch_configuring-openstack-block-storage.html ? | 20:25 |
*** amoturi has left #openstack-cinder | 20:25 | |
patrickeast | or maybe somewhere in here http://docs.openstack.org/admin-guide-cloud/blockstorage-manage-volumes.html ? | 20:25 |
annegentle | patrickeast: both. the first table is auto-generated by scraping the cinder codebase | 20:30 |
patrickeast | annegentle: ok cool, thanks! | 20:31 |
annegentle | patrickeast: you'll find the source in the openstack/openstack-manuals repo | 20:32 |
annegentle | for the admin-guide-cloud, and then use openstack/openstack-doc-tools to find the codescraper | 20:32 |
patrickeast | annegentle: sweet, found it, thanks for the pointers | 20:33 |
*** raildo is now known as raildo-afk | 20:34 | |
*** tiny-hands has joined #openstack-cinder | 20:35 | |
*** ekarlso has quit IRC | 20:38 | |
*** vgridnev has quit IRC | 20:40 | |
mtreinish | eharney: heh, maybe I should start reading the bugs in the commit msg :) | 20:41 |
*** Yogi1 has quit IRC | 20:41 | |
eharney | mtreinish: thanks for the quick feedback though | 20:41 |
mtreinish | eharney: sure np, it actually stood out to me because I remember having a long conversation with someone about testing it | 20:42 |
mtreinish | that's how I learned about the lvm thing | 20:43 |
openstackgerrit | Eric Harney proposed openstack/cinder: Add unmanage default implementation to VolumeDriver https://review.openstack.org/221497 | 20:43 |
*** Lee1092 has quit IRC | 20:43 | |
*** raildo-afk is now known as raildo | 20:44 | |
eharney | mtreinish: i think i actually don't want it to have a skip check | 20:44 |
eharney | mtreinish: so that it can ensure something sane happens when the feature isn't supported? | 20:45 |
eharney | or maybe that needs to be a separate test | 20:45 |
mtreinish | eharney: like check the response of the not supported error from cinder? | 20:46 |
mtreinish | I think that might be a seperate test | 20:46 |
eharney | mtreinish: i think it would have to check volume state instead | 20:46 |
eharney | mtreinish: or, check that force delete works after the unmanage fails, which was what i was going for there (but I missed an else:) | 20:47 |
*** salv-orlando has joined #openstack-cinder | 20:48 | |
mtreinish | eharney: that feels like it should be a seperate test, because you're verifying a distinct path | 20:48 |
eharney | mtreinish: makes sense | 20:49 |
mtreinish | although tbh, that's not the kinda thing we normally verify in tempest | 20:49 |
eharney | hmm, i probably don't really know where the line is for sensible tests to put in there or not | 20:49 |
mtreinish | eharney: well if there were in-tree cinder functional tests I'd suggest doing it there. Because it's kinda of an edge case which is very config specific | 20:51 |
eharney | mtreinish: so maybe this should aim more at "if supported, run the basic path" | 20:52 |
mtreinish | eharney: yep, that's what I was going for. Which was what I was suggesting in my review :) | 20:52 |
eharney | :) | 20:53 |
*** vgridnev has joined #openstack-cinder | 20:53 | |
*** jwang_ has joined #openstack-cinder | 20:54 | |
*** raildo is now known as raildo-afk | 20:56 | |
*** cdelatte has quit IRC | 20:56 | |
*** patrickeast has quit IRC | 20:56 | |
*** patrickeast has joined #openstack-cinder | 20:57 | |
*** jwang has quit IRC | 20:57 | |
*** ekarlso has joined #openstack-cinder | 20:57 | |
*** ekarlso has quit IRC | 20:57 | |
*** ekarlso has joined #openstack-cinder | 20:57 | |
*** bswartz has quit IRC | 20:57 | |
openstackgerrit | Angela Smith proposed openstack/cinder: Adds VF support with HTTPS southbound connector https://review.openstack.org/184799 | 21:00 |
*** gouthamr has quit IRC | 21:00 | |
*** dflorea has quit IRC | 21:00 | |
*** aix has joined #openstack-cinder | 21:01 | |
*** dflorea has joined #openstack-cinder | 21:03 | |
*** dflorea has quit IRC | 21:04 | |
*** changbl has quit IRC | 21:07 | |
*** vgridnev has quit IRC | 21:13 | |
*** lcurtis has quit IRC | 21:13 | |
*** anshul has quit IRC | 21:14 | |
*** dflorea has joined #openstack-cinder | 21:26 | |
*** IanGovett has quit IRC | 21:27 | |
*** tiny-hands has quit IRC | 21:31 | |
*** dflorea has quit IRC | 21:31 | |
*** vgridnev has joined #openstack-cinder | 21:31 | |
*** akerr has quit IRC | 21:32 | |
openstackgerrit | John Griffith proposed openstack/cinder: Return a tuple from SolidFire update_provider_info https://review.openstack.org/221510 | 21:32 |
*** dflorea has joined #openstack-cinder | 21:34 | |
*** IlyaG has joined #openstack-cinder | 21:35 | |
*** devlaps has quit IRC | 21:38 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Move the rootwrap filters https://review.openstack.org/221513 | 21:40 |
*** mriedem is now known as mriedem_away | 21:42 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Move the rootwrap filters https://review.openstack.org/221513 | 21:43 |
*** vgridnev has quit IRC | 21:44 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Move the rootwrap filters https://review.openstack.org/221513 | 21:45 |
*** guest5 has quit IRC | 21:46 | |
*** guest5 has joined #openstack-cinder | 21:47 | |
*** dflorea has quit IRC | 21:49 | |
*** xyang1 has joined #openstack-cinder | 21:49 | |
*** jungleboyj has quit IRC | 21:50 | |
*** jamielennox|away is now known as jamielennox | 21:51 | |
*** dflorea has joined #openstack-cinder | 21:52 | |
*** eduardo has quit IRC | 21:53 | |
*** marcusvrn_ has quit IRC | 21:55 | |
*** edmondsw has quit IRC | 21:57 | |
*** jgregor has quit IRC | 22:00 | |
*** p0rtal_ has joined #openstack-cinder | 22:00 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/221040 | 22:01 |
*** p0rtal has quit IRC | 22:04 | |
*** rushil has joined #openstack-cinder | 22:06 | |
*** annegentle has quit IRC | 22:13 | |
*** edtubill has quit IRC | 22:14 | |
*** IlyaG has quit IRC | 22:20 | |
openstackgerrit | Anthony Lee proposed openstack/cinder: Fix QoS keys not being available to scheduler https://review.openstack.org/214826 | 22:25 |
*** r-daneel has quit IRC | 22:26 | |
*** IlyaG has joined #openstack-cinder | 22:30 | |
*** rushil has quit IRC | 22:30 | |
*** IlyaG_ has joined #openstack-cinder | 22:30 | |
openstackgerrit | Anthony Lee proposed openstack/cinder: LeftHand Add update_migrated_volume to driver https://review.openstack.org/221534 | 22:30 |
*** rushil has joined #openstack-cinder | 22:30 | |
*** IlyaG_ has quit IRC | 22:31 | |
*** dannywilson has joined #openstack-cinder | 22:32 | |
*** martyturner has joined #openstack-cinder | 22:32 | |
*** dannywilson has quit IRC | 22:32 | |
*** dannywilson has joined #openstack-cinder | 22:33 | |
*** martyturner1 has quit IRC | 22:33 | |
*** IlyaG has quit IRC | 22:34 | |
*** xyang1 has quit IRC | 22:36 | |
*** martyturner has quit IRC | 22:38 | |
*** dalgaaf has quit IRC | 22:42 | |
*** earlephilhower has quit IRC | 22:42 | |
*** guest5 has quit IRC | 22:43 | |
openstackgerrit | Eric Harney proposed openstack/cinder: Filter scheduler: Fix KeyError on invalid create request https://review.openstack.org/221537 | 22:44 |
*** agarciam1 has quit IRC | 22:45 | |
openstackgerrit | Anthony Lee proposed openstack/cinder: LeftHand Add update_migrated_volume to drivers https://review.openstack.org/221534 | 22:45 |
openstackgerrit | Anthony Lee proposed openstack/cinder: LeftHand Add update_migrated_volume to drivers https://review.openstack.org/221534 | 22:48 |
*** krtaylor has quit IRC | 22:49 | |
*** rushil has quit IRC | 22:54 | |
*** dflorea has quit IRC | 22:58 | |
*** angela-s has quit IRC | 23:01 | |
*** dflorea has joined #openstack-cinder | 23:03 | |
*** eharney has quit IRC | 23:06 | |
*** dflorea has quit IRC | 23:07 | |
*** hemna is now known as hemnafk | 23:10 | |
*** erlon has quit IRC | 23:18 | |
*** chlong has quit IRC | 23:27 | |
*** chlong has joined #openstack-cinder | 23:28 | |
*** cdelatte has joined #openstack-cinder | 23:34 | |
*** cdelatte has quit IRC | 23:37 | |
*** salv-orlando has quit IRC | 23:41 | |
*** lixiaoy11 has joined #openstack-cinder | 23:43 | |
*** smoriya has joined #openstack-cinder | 23:54 | |
*** jungleboyj has joined #openstack-cinder | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!