*** Yogi1 has joined #openstack-cinder | 00:00 | |
*** lixiaoy1 has joined #openstack-cinder | 00:02 | |
*** david-lyle has quit IRC | 00:03 | |
-openstackstatus- NOTICE: Zuul was restarted due to an error; events (such as approvals or new patchsets) since 23:01 UTC have been lost and affected changes will need to be rechecked | 00:04 | |
*** smoriya has joined #openstack-cinder | 00:08 | |
*** vincent_hou has quit IRC | 00:09 | |
*** sghanekar_ has quit IRC | 00:12 | |
*** diogogmt has quit IRC | 00:12 | |
*** IanGovett has quit IRC | 00:23 | |
openstackgerrit | wanghao proposed openstack/cinder: Improve performance listing detail for volumes https://review.openstack.org/203530 | 00:24 |
---|---|---|
openstackgerrit | Joseph Vokt proposed openstack/cinder: 3PAR: Add performance metrics for volume placement https://review.openstack.org/200737 | 00:25 |
*** vokt has quit IRC | 00:30 | |
*** laughterwym has joined #openstack-cinder | 00:38 | |
openstackgerrit | wanghao proposed openstack/cinder: Implement function to manage/unmanage snapshots https://review.openstack.org/144590 | 00:43 |
*** rick__ has joined #openstack-cinder | 00:46 | |
openstackgerrit | wanghao proposed openstack/cinder: Add updated_at into response of listing detail https://review.openstack.org/202029 | 00:46 |
rick__ | thingee: HI, Are you here? | 00:47 |
*** tsekiyam_ has joined #openstack-cinder | 00:50 | |
*** xyang1 has quit IRC | 00:51 | |
*** mtanino has quit IRC | 00:53 | |
*** tsekiyama has quit IRC | 00:53 | |
*** david-lyle has joined #openstack-cinder | 00:54 | |
*** tsekiyam_ has quit IRC | 00:55 | |
*** skylerberg has quit IRC | 00:59 | |
*** _cjones_ has quit IRC | 01:00 | |
*** vlaza has quit IRC | 01:01 | |
*** haomaiwang has quit IRC | 01:09 | |
*** rick__ has quit IRC | 01:12 | |
*** akerr has joined #openstack-cinder | 01:13 | |
*** smoriya has quit IRC | 01:13 | |
*** IlyaG has quit IRC | 01:13 | |
*** annashen has joined #openstack-cinder | 01:15 | |
*** zhenguo has joined #openstack-cinder | 01:16 | |
*** annashen has quit IRC | 01:19 | |
*** smoriya has joined #openstack-cinder | 01:19 | |
*** skylerberg has joined #openstack-cinder | 01:21 | |
lixiaoy1 | :vincent_hou here? | 01:24 |
openstackgerrit | Erickson Filipe Guedes dos Santos proposed openstack/cinder: Nested Quota Driver: Get Project Hierarchy https://review.openstack.org/206171 | 01:25 |
openstackgerrit | Erickson Filipe Guedes dos Santos proposed openstack/cinder: Cinder Nested Quota Driver https://review.openstack.org/205369 | 01:25 |
*** davechen has joined #openstack-cinder | 01:25 | |
*** diogogmt has joined #openstack-cinder | 01:28 | |
*** apoorvad has quit IRC | 01:28 | |
*** Lee1092 has joined #openstack-cinder | 01:31 | |
*** apoorvad has joined #openstack-cinder | 01:33 | |
*** Yogi1 has quit IRC | 01:35 | |
openstackgerrit | Patrick East proposed openstack/cinder: Modify naming helpers in Pure Volume Drivers https://review.openstack.org/211801 | 01:36 |
openstackgerrit | Patrick East proposed openstack/cinder: Switch Pure volume drivers to use Snapshot Objects https://review.openstack.org/211802 | 01:36 |
openstackgerrit | Patrick East proposed openstack/cinder: Implement Clone CG in Pure Volume Drivers https://review.openstack.org/211803 | 01:36 |
openstackgerrit | Patrick East proposed openstack/cinder: Fix Pure create volume from cgsnapshot https://review.openstack.org/211804 | 01:36 |
*** breitz1 has joined #openstack-cinder | 01:38 | |
*** breitz has quit IRC | 01:38 | |
*** davechen1 has joined #openstack-cinder | 01:38 | |
*** tobe_ has joined #openstack-cinder | 01:39 | |
*** sghanekar_ has joined #openstack-cinder | 01:39 | |
*** davechen has quit IRC | 01:40 | |
*** davechen has joined #openstack-cinder | 01:48 | |
*** mriedem is now known as mriedem_movie | 01:49 | |
*** davechen1 has quit IRC | 01:50 | |
*** Yogi1 has joined #openstack-cinder | 01:55 | |
*** alejandrito has quit IRC | 01:55 | |
*** annashen has joined #openstack-cinder | 01:56 | |
*** Yogi1 has quit IRC | 01:59 | |
*** rushil has joined #openstack-cinder | 02:03 | |
*** erlon has quit IRC | 02:03 | |
*** patrickeast has quit IRC | 02:04 | |
*** laughterwym_ has joined #openstack-cinder | 02:05 | |
*** garthb has joined #openstack-cinder | 02:05 | |
*** vilobhmm has quit IRC | 02:05 | |
*** laughterwym__ has joined #openstack-cinder | 02:07 | |
*** laughterwym has quit IRC | 02:08 | |
*** laughterwym_ has quit IRC | 02:10 | |
*** akerr is now known as akerr_away | 02:11 | |
*** akerr_away has quit IRC | 02:13 | |
openstackgerrit | wangxiyuan proposed openstack/cinder: Improve the efficiency of volume index https://review.openstack.org/211079 | 02:18 |
*** rushil has quit IRC | 02:19 | |
*** haomaiwa_ has joined #openstack-cinder | 02:21 | |
*** sghanekar_ has quit IRC | 02:21 | |
*** arif-ali has quit IRC | 02:21 | |
*** jwcroppe has joined #openstack-cinder | 02:23 | |
*** Yogi1 has joined #openstack-cinder | 02:25 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Adds the migration progress support for migration https://review.openstack.org/204953 | 02:26 |
openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 02:26 |
*** arif-ali has joined #openstack-cinder | 02:26 | |
*** laughterwym__ has quit IRC | 02:27 | |
*** laughterwym has joined #openstack-cinder | 02:27 | |
*** mylu has joined #openstack-cinder | 02:27 | |
*** p0rtal_ has joined #openstack-cinder | 02:28 | |
*** mdbooth has quit IRC | 02:29 | |
*** p0rtal has quit IRC | 02:31 | |
*** david-lyle has quit IRC | 02:33 | |
*** mylu has quit IRC | 02:34 | |
*** mdbooth has joined #openstack-cinder | 02:34 | |
*** jdandrea has quit IRC | 02:43 | |
*** apoorvad has quit IRC | 02:47 | |
*** hakimo has joined #openstack-cinder | 02:52 | |
*** BharatK has joined #openstack-cinder | 02:53 | |
*** hakimo_ has quit IRC | 02:54 | |
*** patrickeast has joined #openstack-cinder | 02:55 | |
*** vincent_hou has joined #openstack-cinder | 02:59 | |
*** vilobhmm has joined #openstack-cinder | 03:02 | |
*** vilobhmm1 has joined #openstack-cinder | 03:04 | |
*** vilobhmm has quit IRC | 03:07 | |
*** mylu has joined #openstack-cinder | 03:07 | |
*** mriedem_movie has quit IRC | 03:14 | |
openstackgerrit | Patrick East proposed openstack/cinder: Implement Clone CG in Pure Volume Drivers https://review.openstack.org/211803 | 03:15 |
openstackgerrit | Patrick East proposed openstack/cinder: Fix Pure create volume from cgsnapshot https://review.openstack.org/211804 | 03:15 |
*** patrickeast has quit IRC | 03:15 | |
*** jwcroppe has quit IRC | 03:15 | |
*** tobe_ has quit IRC | 03:15 | |
*** tobe_ has joined #openstack-cinder | 03:16 | |
*** mylu has quit IRC | 03:20 | |
*** mylu has joined #openstack-cinder | 03:20 | |
openstackgerrit | wanghao proposed openstack/cinder: Add pagination to backups https://review.openstack.org/204493 | 03:23 |
*** humble__ has joined #openstack-cinder | 03:24 | |
*** p0rtal_ has quit IRC | 03:25 | |
*** p0rtal has joined #openstack-cinder | 03:25 | |
*** danieru has quit IRC | 03:25 | |
*** danieru has joined #openstack-cinder | 03:25 | |
*** diogogmt has quit IRC | 03:29 | |
*** david-lyle has joined #openstack-cinder | 03:30 | |
*** lcurtis has quit IRC | 03:38 | |
*** BharatK has quit IRC | 03:39 | |
*** garthb has quit IRC | 03:39 | |
*** mylu has quit IRC | 03:43 | |
*** harlowja_at_home has joined #openstack-cinder | 03:45 | |
*** hemna__ has joined #openstack-cinder | 03:50 | |
*** vincent_hou has quit IRC | 03:51 | |
*** BharatK has joined #openstack-cinder | 03:52 | |
*** annashen has quit IRC | 03:54 | |
*** BharatK has quit IRC | 04:00 | |
*** annashen_ has joined #openstack-cinder | 04:02 | |
*** p0rtal has quit IRC | 04:02 | |
*** vilobhmm has joined #openstack-cinder | 04:03 | |
*** vilobhmm1 has quit IRC | 04:03 | |
*** p0rtal has joined #openstack-cinder | 04:03 | |
*** p0rtal has quit IRC | 04:10 | |
*** p0rtal has joined #openstack-cinder | 04:11 | |
*** p0rtal has quit IRC | 04:15 | |
*** BharatK has joined #openstack-cinder | 04:15 | |
*** BharatK has quit IRC | 04:22 | |
*** annashen_ has quit IRC | 04:23 | |
*** morgan_404 is now known as morgan_410 | 04:26 | |
*** morgan_410 is now known as morgan_404 | 04:26 | |
*** BharatK has joined #openstack-cinder | 04:33 | |
*** edtubill has joined #openstack-cinder | 04:34 | |
*** links has joined #openstack-cinder | 04:42 | |
*** chhavi has joined #openstack-cinder | 04:45 | |
*** tobe_ has quit IRC | 04:53 | |
*** tobe_ has joined #openstack-cinder | 05:04 | |
*** ircuser-1 has joined #openstack-cinder | 05:07 | |
*** chhavi_ has joined #openstack-cinder | 05:07 | |
*** chhavi has quit IRC | 05:07 | |
*** chhavi_ has quit IRC | 05:08 | |
*** chhavi_ has joined #openstack-cinder | 05:08 | |
*** hemna__ has quit IRC | 05:08 | |
*** deepakcs has joined #openstack-cinder | 05:12 | |
*** Yogi1 has quit IRC | 05:16 | |
*** BharatK has quit IRC | 05:17 | |
*** BharatK has joined #openstack-cinder | 05:18 | |
*** harlowja_at_home has quit IRC | 05:24 | |
*** chhavi_ has quit IRC | 05:29 | |
*** skylerberg has quit IRC | 05:32 | |
*** vilobhmm has quit IRC | 05:32 | |
*** ankit_ag has joined #openstack-cinder | 05:35 | |
*** theanalyst has quit IRC | 05:37 | |
*** theanalyst has joined #openstack-cinder | 05:38 | |
*** chhavi_ has joined #openstack-cinder | 05:46 | |
*** edtubill has quit IRC | 05:48 | |
*** humble__ has quit IRC | 05:48 | |
*** haomaiwa_ has quit IRC | 05:53 | |
*** haomaiwang has joined #openstack-cinder | 05:54 | |
davechen | thingee: ping? | 05:58 |
*** haomaiwang has quit IRC | 06:04 | |
*** zhenguo has quit IRC | 06:13 | |
*** haomaiwang has joined #openstack-cinder | 06:15 | |
*** lixiaoy1 has quit IRC | 06:15 | |
*** lixiaoy11 has joined #openstack-cinder | 06:15 | |
*** lixiaoy1 has joined #openstack-cinder | 06:19 | |
*** lixiaoy11 has quit IRC | 06:19 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Use cinder internal tenant to create the target volume https://review.openstack.org/210237 | 06:21 |
*** e0ne has joined #openstack-cinder | 06:22 | |
*** svilgelm has joined #openstack-cinder | 06:25 | |
openstackgerrit | wanghao proposed openstack/cinder: Improve performance listing detail for volumes https://review.openstack.org/203530 | 06:27 |
*** laughterwym has quit IRC | 06:28 | |
*** svilgelm has quit IRC | 06:31 | |
openstackgerrit | wanghao proposed openstack/cinder: Add updated_at into response of listing detail https://review.openstack.org/202029 | 06:33 |
*** vlaza has joined #openstack-cinder | 06:35 | |
*** e0ne has quit IRC | 06:39 | |
*** e0ne has joined #openstack-cinder | 06:40 | |
*** e0ne has quit IRC | 06:45 | |
*** mdnadeem has joined #openstack-cinder | 06:45 | |
*** e0ne has joined #openstack-cinder | 06:49 | |
*** e0ne has quit IRC | 06:54 | |
*** abehl has joined #openstack-cinder | 07:02 | |
*** xek has quit IRC | 07:04 | |
*** xek has joined #openstack-cinder | 07:05 | |
*** ronis has joined #openstack-cinder | 07:07 | |
*** ronis_ has joined #openstack-cinder | 07:09 | |
*** chhavi_ has quit IRC | 07:13 | |
*** ronis has quit IRC | 07:13 | |
*** ig0r_ has joined #openstack-cinder | 07:15 | |
*** ishant has joined #openstack-cinder | 07:16 | |
openstackgerrit | Dave Chen proposed openstack/python-cinderclient: Useless methods cleanup https://review.openstack.org/210287 | 07:18 |
openstackgerrit | wangxiyuan proposed openstack/cinder: Improve the efficiency of volume index https://review.openstack.org/211079 | 07:22 |
*** markus_z has joined #openstack-cinder | 07:26 | |
*** lixiaoy1 has quit IRC | 07:28 | |
*** chhavi_ has joined #openstack-cinder | 07:28 | |
*** lixiaoy1 has joined #openstack-cinder | 07:28 | |
*** hw_chenying has joined #openstack-cinder | 07:31 | |
*** afazekas has joined #openstack-cinder | 07:31 | |
*** lixiaoy11 has joined #openstack-cinder | 07:35 | |
*** lixiaoy1 has quit IRC | 07:36 | |
*** zhenguo has joined #openstack-cinder | 07:36 | |
*** ig0r__ has joined #openstack-cinder | 07:39 | |
*** ig0r_ has quit IRC | 07:41 | |
*** tobe_ has quit IRC | 07:44 | |
*** chhavi_ has quit IRC | 07:47 | |
*** chhavi_ has joined #openstack-cinder | 07:49 | |
*** boris-42 has quit IRC | 07:50 | |
*** tobe_ has joined #openstack-cinder | 07:50 | |
*** sgotliv has joined #openstack-cinder | 07:50 | |
*** chhavi_ has quit IRC | 07:57 | |
*** tobe_ has quit IRC | 08:00 | |
*** tobe_ has joined #openstack-cinder | 08:05 | |
*** jordanP has joined #openstack-cinder | 08:13 | |
openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Add volume ID in vCenter's volume config https://review.openstack.org/194534 | 08:13 |
*** ig0r__ has quit IRC | 08:13 | |
openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Set virtual disk UUID to volume ID https://review.openstack.org/205494 | 08:15 |
*** ig0r_ has joined #openstack-cinder | 08:17 | |
*** ig0r_ has quit IRC | 08:19 | |
*** DTadrzak has quit IRC | 08:19 | |
*** jistr has joined #openstack-cinder | 08:21 | |
*** Maike has joined #openstack-cinder | 08:25 | |
lixiaoy11 | :winston-d here? | 08:54 |
*** lixiaoy11 has left #openstack-cinder | 09:03 | |
openstackgerrit | Wilson Liu proposed openstack/cinder: Fix way of get LUN id in Huawei driver https://review.openstack.org/209749 | 09:08 |
openstackgerrit | Michal Dulko proposed openstack/cinder: Service object https://review.openstack.org/160417 | 09:12 |
*** sayali has quit IRC | 09:21 | |
openstackgerrit | Jordan Pittier proposed openstack/cinder: Bump min tox version to 1.7 https://review.openstack.org/211614 | 09:26 |
*** sayali has joined #openstack-cinder | 09:33 | |
*** ishant_ has joined #openstack-cinder | 09:34 | |
*** ishant has quit IRC | 09:36 | |
*** davechen has left #openstack-cinder | 09:46 | |
*** jwcroppe has joined #openstack-cinder | 09:49 | |
*** IanGovett has joined #openstack-cinder | 09:54 | |
*** mdbooth has quit IRC | 09:55 | |
BharatK | eharney, Please review the patch https://review.openstack.org/#/c/182254/ when you have sometime. | 09:58 |
*** mdbooth has joined #openstack-cinder | 10:01 | |
*** ig0r_ has joined #openstack-cinder | 10:10 | |
*** gaurangt has joined #openstack-cinder | 10:11 | |
jordanP | smcginnis, ping ? (I don't remember your TZ...) | 10:16 |
*** jkraj has joined #openstack-cinder | 10:19 | |
*** hw_chenying has quit IRC | 10:35 | |
*** jwcroppe has quit IRC | 10:35 | |
*** jwcroppe has joined #openstack-cinder | 10:36 | |
*** abehl has quit IRC | 10:36 | |
*** jwcroppe has quit IRC | 10:40 | |
*** ig0r_ has quit IRC | 10:46 | |
*** kmartin has quit IRC | 10:48 | |
*** abehl has joined #openstack-cinder | 10:49 | |
*** MIDENN_ has quit IRC | 10:52 | |
BharatK | thingee, Need your blessings for the patch https://review.openstack.org/#/c/182254/ | 10:52 |
*** ishant_ has quit IRC | 10:54 | |
*** alexpilotti has joined #openstack-cinder | 10:55 | |
*** ishant has joined #openstack-cinder | 10:56 | |
*** ishant has quit IRC | 10:56 | |
*** IanGovett1 has joined #openstack-cinder | 10:58 | |
*** IanGovett has quit IRC | 11:00 | |
*** tobe_ has quit IRC | 11:03 | |
*** zhenguo has quit IRC | 11:03 | |
*** tobe_ has joined #openstack-cinder | 11:04 | |
*** markus_z has quit IRC | 11:05 | |
*** tobe_ has quit IRC | 11:09 | |
openstackgerrit | Vipin Balachandran proposed openstack/cinder: Remove unused fake objects in vmdk test module https://review.openstack.org/211967 | 11:14 |
*** haomaiwang has quit IRC | 11:19 | |
*** markus_z has joined #openstack-cinder | 11:21 | |
*** marcusvrn has joined #openstack-cinder | 11:28 | |
*** IanGovett1 has quit IRC | 11:33 | |
*** tobe_ has joined #openstack-cinder | 11:33 | |
*** IanGovett has joined #openstack-cinder | 11:33 | |
*** julim has joined #openstack-cinder | 11:38 | |
*** tobe_ has quit IRC | 11:39 | |
*** timcl has joined #openstack-cinder | 11:43 | |
*** DTadrzak has joined #openstack-cinder | 11:46 | |
*** gaurangt has left #openstack-cinder | 11:59 | |
*** tellesnobrega_ has joined #openstack-cinder | 12:08 | |
*** diogogmt has joined #openstack-cinder | 12:11 | |
*** smoriya has quit IRC | 12:12 | |
*** markus_z has quit IRC | 12:13 | |
*** markus_z has joined #openstack-cinder | 12:14 | |
*** abehl has quit IRC | 12:15 | |
*** akerr has joined #openstack-cinder | 12:16 | |
*** tellesnobrega_ has quit IRC | 12:16 | |
*** anshul has joined #openstack-cinder | 12:17 | |
*** deepakcs has quit IRC | 12:21 | |
*** diogogmt has quit IRC | 12:24 | |
*** raildo-afk is now known as raildo | 12:26 | |
*** olmy0414 has joined #openstack-cinder | 12:27 | |
*** IanGovett1 has joined #openstack-cinder | 12:29 | |
*** IanGovett has quit IRC | 12:30 | |
*** erlon has joined #openstack-cinder | 12:31 | |
*** bapalm has joined #openstack-cinder | 12:31 | |
*** shausy has joined #openstack-cinder | 12:32 | |
*** bnemec has joined #openstack-cinder | 12:35 | |
olmy0414 | Hello everyone, I am trying to figure out, cinder LVM driver (with thin provisioning) supports copy-on-write when make clone from volume or snapshot? When T try to do "cinder create --snapshot-id" or "cinder create --source-volid" - full size volume is created | 12:35 |
olmy0414 | version - openstack-cinder-2015.1.0-3.el7 | 12:36 |
*** zhipeng has joined #openstack-cinder | 12:40 | |
*** porrua has joined #openstack-cinder | 12:40 | |
*** gouthamr has joined #openstack-cinder | 12:40 | |
*** edmondsw has joined #openstack-cinder | 12:41 | |
openstackgerrit | Jordan Pittier proposed openstack/cinder: Rework Scality SOFS driver to use RemoteFS class https://review.openstack.org/200610 | 12:42 |
*** akshai has joined #openstack-cinder | 12:43 | |
*** alexpilotti has quit IRC | 12:44 | |
*** alexpilotti has joined #openstack-cinder | 12:45 | |
*** ganso_ has joined #openstack-cinder | 12:47 | |
*** mdnadeem has quit IRC | 12:47 | |
*** IanGovett has joined #openstack-cinder | 12:55 | |
*** ankit_ag has quit IRC | 12:56 | |
openstackgerrit | Anton Arefiev proposed openstack/cinder: WIP:Add rootwrap daemon mode support https://review.openstack.org/149677 | 12:56 |
*** IanGovett1 has quit IRC | 12:56 | |
*** merooney has joined #openstack-cinder | 12:57 | |
*** porrua has left #openstack-cinder | 12:58 | |
*** gman-tx has joined #openstack-cinder | 12:59 | |
*** akshai has quit IRC | 12:59 | |
*** sgotliv has quit IRC | 13:00 | |
*** sgotliv has joined #openstack-cinder | 13:00 | |
*** rick__ has joined #openstack-cinder | 13:02 | |
rick__ | thingee: Hi Sir, are you here? | 13:02 |
*** jgregor has joined #openstack-cinder | 13:06 | |
*** svilgelm has joined #openstack-cinder | 13:06 | |
*** jecarey has joined #openstack-cinder | 13:07 | |
*** svilgelm has quit IRC | 13:08 | |
*** jwcroppe has joined #openstack-cinder | 13:10 | |
*** skraynev has quit IRC | 13:15 | |
*** BharatK has quit IRC | 13:19 | |
*** dustins has joined #openstack-cinder | 13:19 | |
*** xyang1 has joined #openstack-cinder | 13:21 | |
*** jwcroppe has quit IRC | 13:23 | |
*** skraynev has joined #openstack-cinder | 13:23 | |
*** jwcroppe has joined #openstack-cinder | 13:23 | |
*** edmondsw has quit IRC | 13:24 | |
*** lixiaoy1 has joined #openstack-cinder | 13:24 | |
smcginnis | jordanP: I'm here now. Central time (UTC+5) | 13:26 |
smcginnis | jordanP: What's up? | 13:26 |
smcginnis | jordanP: Guessing this? https://review.openstack.org/#/c/200610/ | 13:27 |
jordanP | smcginnis, hi :) I didn"t quite understand one of your comment you left me | 13:27 |
smcginnis | jordanP: Sure, which one? | 13:27 |
*** jwcroppe has quit IRC | 13:27 | |
jordanP | smcginnis, the six.str_type | 13:27 |
jordanP | (thanks btw) | 13:27 |
smcginnis | np | 13:28 |
jordanP | I have submitted another patch but I am not sure it addresses your point... | 13:28 |
smcginnis | jordanP: I see your response. | 13:28 |
smcginnis | jordanP: So forcing it is discouraged when passing it in to the logger. | 13:28 |
smcginnis | jordanP: Mainly because internally the logger will do it again, so it's redundnant. | 13:28 |
smcginnis | jordanP: And if the log level is set higher it might not ever need to do it. | 13:29 |
smcginnis | jordanP: But in the case I pointed out you are performing string formatting directly. | 13:29 |
*** timcl has quit IRC | 13:29 | |
smcginnis | So it won't be deferred and it won't be handled by the logger. | 13:29 |
jordanP | yeah I think the logger does something because I can't make to raise a Unicode error or something | 13:29 |
smcginnis | In that case it could potentially cause an exception. | 13:29 |
*** mriedem has joined #openstack-cinder | 13:29 | |
smcginnis | jordanP: I had a paste somewhere that was able to reproduce the Unicode error when _not_ using the logger and just doing string formatting. | 13:30 |
smcginnis | Not sure where that is now. | 13:30 |
*** Yogi1 has joined #openstack-cinder | 13:30 | |
smcginnis | jordanP: That said, this was in the existing code and hasn't been an issue so far, so no worries if you don't address it. | 13:30 |
* smcginnis hasn't seen the latest update | 13:30 | |
smcginnis | jordanP: Just thought I should point it out. | 13:31 |
jordanP | yeah but even in the exception ' raise exception.VolumeBackendAPIException(data=msg % e)' I am doing string formating but even if "e" contains Unicode char, "it works" (I have a unit test that covers this) | 13:31 |
smcginnis | jordanP: Hmm. I did have code that would reproduce the error at one point. If I get time I will track that down or redo it. | 13:32 |
*** BharatK has joined #openstack-cinder | 13:32 | |
jordanP | smcginnis, ok. that would be interesting | 13:33 |
*** rick__ has quit IRC | 13:34 | |
lixiaoy1 | :smcginnis may I talk with you about creating encrypted volume from image? | 13:35 |
smcginnis | lixiaoy1: I haven't done much with encrypted volumes. | 13:35 |
smcginnis | lixiaoy1: Maybe someone else might be a better resource here. | 13:36 |
smcginnis | lixiaoy1: What's your question? Maybe I can answer or someone else will know. | 13:36 |
*** breitz1 has quit IRC | 13:36 | |
*** crose has joined #openstack-cinder | 13:37 | |
lixiaoy1 | :smcginnis anyone is OK. :) I found that when creating an encrypted from image, it just writes unencrypted data into the volume which as a result Nova can't boot from it | 13:37 |
smcginnis | lixiaoy1: Sounds vaguely familiar. I think someone else was running in to this recently. | 13:38 |
lixiaoy1 | I think we can read the unencrypted data from image, encrypts and writes to a encrypted volume. This is what I want to do | 13:38 |
smcginnis | lixiaoy1: I don't know the specifics here. Anyone else around know anything? | 13:38 |
lixiaoy1 | but I didn't find any bp in cinder | 13:38 |
lixiaoy1 | and bugs | 13:38 |
*** timcl has joined #openstack-cinder | 13:41 | |
*** akshai has joined #openstack-cinder | 13:42 | |
*** jaypipes has joined #openstack-cinder | 13:42 | |
smcginnis | lixiaoy1: What are you using for your backend storage? | 13:42 |
lixiaoy1 | lvm | 13:43 |
lixiaoy1 | this case is just for creating encrypted volume with image. | 13:44 |
lixiaoy1 | as in cinder code, when we copying data from image to volumes, we don't consider encryption here | 13:44 |
lixiaoy1 | there is a bug here: https://bugs.launchpad.net/nova/+bug/1465656 | 13:46 |
openstack | Launchpad bug 1465656 in OpenStack Compute (nova) "Booting encrypted volume with whole image fails" [Undecided,In progress] - Assigned to Dane Fichter (dane-fichter) | 13:46 |
openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix HNAS iSCSI 32 targets limitation error https://review.openstack.org/206651 | 13:47 |
*** tsekiyama has joined #openstack-cinder | 13:50 | |
*** links has quit IRC | 13:51 | |
*** annasort has quit IRC | 13:51 | |
*** annasort has joined #openstack-cinder | 13:52 | |
*** edmondsw has joined #openstack-cinder | 13:52 | |
*** breitz has joined #openstack-cinder | 13:53 | |
*** diogogmt has joined #openstack-cinder | 13:55 | |
*** diogogmt has quit IRC | 13:55 | |
*** bswartz has quit IRC | 13:56 | |
*** annegentle has joined #openstack-cinder | 13:56 | |
*** lixiaoy1 has quit IRC | 13:57 | |
*** jistr is now known as jistr|mtg | 13:57 | |
*** mriedem has quit IRC | 14:01 | |
*** tsekiyama has quit IRC | 14:03 | |
*** thangp has joined #openstack-cinder | 14:04 | |
*** annasort has quit IRC | 14:04 | |
*** mriedem has joined #openstack-cinder | 14:06 | |
*** rushil has joined #openstack-cinder | 14:12 | |
*** zhipeng has quit IRC | 14:15 | |
*** Rock has joined #openstack-cinder | 14:17 | |
*** shausy has quit IRC | 14:17 | |
*** shausy has joined #openstack-cinder | 14:17 | |
*** xyang has joined #openstack-cinder | 14:19 | |
Rock | thingee: are you here? | 14:19 |
*** Rick__ has joined #openstack-cinder | 14:23 | |
*** Rock has quit IRC | 14:23 | |
*** mdenny has joined #openstack-cinder | 14:28 | |
*** mtanino has joined #openstack-cinder | 14:30 | |
*** vlaza has quit IRC | 14:32 | |
*** Rick__ has quit IRC | 14:32 | |
*** jistr|mtg is now known as jistr | 14:38 | |
*** jungleboyj has joined #openstack-cinder | 14:40 | |
*** annegentle has quit IRC | 14:41 | |
*** BharatK has quit IRC | 14:43 | |
*** mriedem1 has joined #openstack-cinder | 14:43 | |
*** mriedem has quit IRC | 14:44 | |
*** afazekas has quit IRC | 14:44 | |
*** dannywilson has joined #openstack-cinder | 14:51 | |
*** tsekiyama has joined #openstack-cinder | 14:51 | |
*** wilson has joined #openstack-cinder | 14:52 | |
*** merooney has quit IRC | 14:52 | |
openstackgerrit | Eric Harney proposed openstack/cinder: LVM Thin Provisioning auto-detect https://review.openstack.org/104653 | 14:52 |
*** jdandrea has joined #openstack-cinder | 14:54 | |
*** vilobhmm has joined #openstack-cinder | 14:54 | |
*** diogogmt has joined #openstack-cinder | 14:57 | |
*** bswartz has joined #openstack-cinder | 14:58 | |
*** merooney has joined #openstack-cinder | 14:59 | |
*** patrickeast has joined #openstack-cinder | 15:00 | |
*** jdandrea has left #openstack-cinder | 15:00 | |
*** jgregor has quit IRC | 15:02 | |
*** alexpilotti has quit IRC | 15:03 | |
*** diogogmt has quit IRC | 15:03 | |
*** diogogmt has joined #openstack-cinder | 15:04 | |
*** DuncanT has quit IRC | 15:05 | |
*** vaidy has quit IRC | 15:05 | |
*** ganso_ has quit IRC | 15:06 | |
*** thingee has quit IRC | 15:06 | |
*** serverascode has quit IRC | 15:06 | |
*** ronis_ has quit IRC | 15:06 | |
*** ganso_ has joined #openstack-cinder | 15:07 | |
*** DuncanT has joined #openstack-cinder | 15:08 | |
*** thingee has joined #openstack-cinder | 15:09 | |
*** serverascode has joined #openstack-cinder | 15:09 | |
*** vaidy has joined #openstack-cinder | 15:11 | |
*** kmartin has joined #openstack-cinder | 15:15 | |
*** IanGovett has quit IRC | 15:16 | |
*** annasort has joined #openstack-cinder | 15:16 | |
*** lcurtis has joined #openstack-cinder | 15:20 | |
*** haomaiwang has joined #openstack-cinder | 15:23 | |
*** timcl has quit IRC | 15:23 | |
*** rushil has quit IRC | 15:23 | |
*** vilobhmm has quit IRC | 15:23 | |
*** IanGovett has joined #openstack-cinder | 15:26 | |
*** shausy has quit IRC | 15:27 | |
*** merooney has quit IRC | 15:28 | |
*** annegentle has joined #openstack-cinder | 15:28 | |
*** edtubill has joined #openstack-cinder | 15:32 | |
*** zhenguo has joined #openstack-cinder | 15:36 | |
*** salv-orl_ has joined #openstack-cinder | 15:37 | |
*** timcl has joined #openstack-cinder | 15:39 | |
*** IanGovett1 has joined #openstack-cinder | 15:40 | |
*** salv-orlando has quit IRC | 15:40 | |
*** IanGovett has quit IRC | 15:41 | |
*** zhenguo has quit IRC | 15:42 | |
*** IlyaG has joined #openstack-cinder | 15:42 | |
*** edtubill has quit IRC | 15:44 | |
*** adurbin_ has joined #openstack-cinder | 15:45 | |
*** davechen has joined #openstack-cinder | 15:47 | |
*** apoorvad has joined #openstack-cinder | 15:47 | |
*** akshai has quit IRC | 15:47 | |
*** IlyaG has quit IRC | 15:49 | |
*** bapalm has quit IRC | 15:49 | |
*** bapalm has joined #openstack-cinder | 15:49 | |
*** sseverson has joined #openstack-cinder | 15:51 | |
*** edtubill has joined #openstack-cinder | 15:54 | |
*** morganfainberg_ has quit IRC | 15:55 | |
*** jseiler has joined #openstack-cinder | 15:55 | |
*** BharatK has joined #openstack-cinder | 15:56 | |
jordanP | thingee, ping. I've just replied to https://review.openstack.org/#/c/211614/ :) I am fine with abandonning the patch but I just want a small chat about it | 15:57 |
*** p0rtal has joined #openstack-cinder | 15:58 | |
eharney | jordanP: is there a proposed fix other than just reverting the changes that introduced this? | 15:59 |
flip214 | meeeeeeeeeeeting | 15:59 |
*** kjnelson is now known as fireball | 15:59 | |
jordanP | eharney, bump tox to 1.7 :) | 16:00 |
*** fireball is now known as Guest72293 | 16:00 | |
*** skylerberg has joined #openstack-cinder | 16:00 | |
jordanP | (fwiw i like the idea of optional requirements) | 16:00 |
*** vilobhmm has joined #openstack-cinder | 16:01 | |
jgriffith | They're not requirements if they're optional :) That's a contradiction | 16:01 |
thingee | mriedem1: hey your topic is on in the meeting | 16:01 |
*** akshai has joined #openstack-cinder | 16:02 | |
jordanP | jgriffith, I am a man of contradiction :) | 16:02 |
jgriffith | jordanP: LOL | 16:02 |
mriedem1 | oh crap | 16:02 |
*** anshul has quit IRC | 16:03 | |
*** merooney has joined #openstack-cinder | 16:04 | |
*** Guest72293 is now known as diablo_rojo | 16:04 | |
*** akshai has quit IRC | 16:09 | |
*** deepakcs has joined #openstack-cinder | 16:10 | |
*** mdbooth has quit IRC | 16:10 | |
*** jkraj has quit IRC | 16:11 | |
*** mdbooth has joined #openstack-cinder | 16:12 | |
jgriffith | thingee: have you seen Wang's numbers in this bug: particularly around metadata | 16:12 |
jgriffith | xyang: tbarron BTW I think this is somewhat relational to our talk yesterday. I think we've gone over the edge on metadata :( | 16:13 |
*** jistr has quit IRC | 16:13 | |
tbarron | jgriffith: will look | 16:13 |
*** ebalduf has joined #openstack-cinder | 16:14 | |
*** anshul has joined #openstack-cinder | 16:15 | |
openstackgerrit | Eric Harney proposed openstack/cinder: Add retype() to BaseVD https://review.openstack.org/206559 | 16:16 |
*** haomaiwang has quit IRC | 16:16 | |
guitarzan | jgriffith: metadata soft deletes are the only soft deletes that concern me as well | 16:17 |
xyang1 | jgriffith: We'll check the code on that | 16:20 |
*** anshul has quit IRC | 16:21 | |
*** rick__ has joined #openstack-cinder | 16:22 | |
rick__ | thingee: HI | 16:22 |
*** _cjones_ has joined #openstack-cinder | 16:26 | |
*** jordanP has quit IRC | 16:29 | |
*** leeantho has joined #openstack-cinder | 16:32 | |
*** btran has joined #openstack-cinder | 16:32 | |
*** delattec has joined #openstack-cinder | 16:33 | |
*** cdelatte has joined #openstack-cinder | 16:33 | |
*** ronis_ has joined #openstack-cinder | 16:34 | |
*** Maike has quit IRC | 16:35 | |
*** anshul has joined #openstack-cinder | 16:36 | |
*** salv-orl_ has quit IRC | 16:40 | |
*** salv-orlando has joined #openstack-cinder | 16:41 | |
*** rick__ has quit IRC | 16:41 | |
*** akshai has joined #openstack-cinder | 16:46 | |
*** akshai has quit IRC | 16:47 | |
*** davechen has left #openstack-cinder | 16:49 | |
*** p0rtal has quit IRC | 16:50 | |
*** bapalm has quit IRC | 16:52 | |
*** salv-orlando has quit IRC | 16:54 | |
*** salv-orlando has joined #openstack-cinder | 16:54 | |
winston-d | jgriffith: but the metadata issues, only affects admin when he/she does list --all-tenant=1 and Cinder has lots and lots of volumes and glance metadata. | 16:55 |
ericksonsantos | thingee, jgriffith : Hi :) it would be nice to have your reviews in : https://review.openstack.org/#/c/206171 and https://review.openstack.org/#/c/205369/ | 16:56 |
jgriffith | winston-d: I wasn't necessarily pointing out the calls themselves, but rather the amount of crap we're stuffing in there | 16:56 |
jgriffith | winston-d: seems like we may be abusing things a bit? | 16:56 |
thingee | hey ericksonsantos, please stop harassing me about reviews | 16:56 |
thingee | thank you | 16:56 |
*** rushil has joined #openstack-cinder | 16:56 | |
thingee | vilobhmm: ^ | 16:57 |
ericksonsantos | thingee, srry | 16:57 |
thingee | you're not the only people in the world with reviews | 16:57 |
winston-d | i mean it is a definitely a bug, but it only hurts in certain situation. | 16:57 |
jgriffith | thingee: can you help me with what we consider a bug vs a feature vs xxxx these days? | 16:57 |
jgriffith | thingee: https://bugs.launchpad.net/cinder/+bug/1472803 | 16:57 |
openstack | Launchpad bug 1472803 in Cinder "Move to thin LVM as the default configuration" [Undecided,In progress] - Assigned to Eric Harney (eharney) | 16:57 |
*** Yogi1 has quit IRC | 16:57 | |
jgriffith | thingee: I don't quite follow why that's a bug and not a bp or just a proposal | 16:58 |
vilobhmm | thingee : it was a polite request since we had open-discussion…if it was not the right time to ask for it…sorry bt it :( | 16:58 |
winston-d | bots in our channel is less informative. :) | 16:58 |
thangp | xyang: i posted the workaround in the patch - https://review.openstack.org/#/c/195027/25/cinder/consistencygroup/api.py | 16:58 |
jgriffith | thingee: sort of like the persona back port that just went in | 16:58 |
xyang | thangp: ok, thanks | 16:58 |
thangp | xyang: i "think" it works | 16:58 |
thangp | :) | 16:58 |
jgriffith | or maybe I should ask eharney ? | 16:59 |
xyang | thangp: I'll find out:) | 16:59 |
vilobhmm | thingee : the only intention was not to miss out on the Liberty deadline so ericksonsantos thought to bring it up in the open-discussion | 16:59 |
thangp | xyang: thanks! | 16:59 |
thingee | vilobhmm: you don't need to ping me three times in the same five minutes. | 16:59 |
thingee | vilobhmm: quit it | 16:59 |
eharney | jgriffith: we can make it one if we feel the need to, but i think it was touched on in the thin provisioning spec | 17:00 |
thingee | vilobhmm: you're getting to the point of harassing. I've already expressed I'm testing image caching and migration at the moment | 17:00 |
thingee | vilobhmm: if you have all this time , why not go and test some of these other blueprint proposals | 17:00 |
eharney | jgriffith: i was writing a spec for this at one point, but i think it disappeared somewhere | 17:00 |
thingee | vilobhmm: I'm stretched thin as it is. | 17:00 |
vilobhmm | thingee : i just thought my msg was missed in the objects discussion so i dropped a msg again…nothing more than that…I understand..would be more than happy to help you with reviews wherever needed | 17:01 |
thingee | vilobhmm: I will repeat you're harassing me. | 17:02 |
thingee | vilobhmm: stop it | 17:02 |
*** rushil has quit IRC | 17:02 | |
jgriffith | eharney: yeah, I don't even think a spec is necessary | 17:03 |
jgriffith | eharney: I just want to better understand what we consider a bug these days | 17:03 |
thingee | jgriffith: I don't know why that would be a bug | 17:03 |
*** esker has joined #openstack-cinder | 17:03 | |
jgriffith | don't get me wrong, I don't care.... just want to know what I'm missing :) | 17:03 |
thingee | I don't need a spec though | 17:03 |
thingee | "devstack by default doesn't thin...lets make it do by default" | 17:04 |
thingee | that's not a bug | 17:05 |
thingee | that's not a feature. | 17:05 |
thingee | jgriffith, eharney ^ | 17:05 |
eharney | so the preference is a blueprint to keep track of it instead of a bug? | 17:05 |
smcginnis | Kind of why I wanted that topic at the midcycle. Folks seem really confused about bugs, blueprints, and specs. | 17:05 |
*** sbezverk has quit IRC | 17:06 | |
thingee | smcginnis: we sort of did. no one ask about bugs though | 17:06 |
*** sbezverk has joined #openstack-cinder | 17:06 | |
thingee | smcginnis: why didn't you bring it up in that time? | 17:06 |
smcginnis | thingee: Thought I did. :) | 17:06 |
thingee | smcginnis: then why don't you have an answer? | 17:06 |
smcginnis | I'm like Ben with process. No bugs. :D | 17:07 |
thingee | bringing focus back... | 17:07 |
thingee | eharney: I'd say make a bug when it's a middle point. because 1) I'm not accepting anymore bps, so SOL for liberty otherwise. 2) Mark as enhancement in the bug and be done | 17:08 |
*** xek has quit IRC | 17:08 | |
*** DTadrzak has quit IRC | 17:10 | |
*** deepakcs has quit IRC | 17:10 | |
*** sghanekar_ has joined #openstack-cinder | 17:10 | |
*** sgotliv has quit IRC | 17:12 | |
*** xek has joined #openstack-cinder | 17:13 | |
*** _cjones_ has quit IRC | 17:16 | |
*** drjones has joined #openstack-cinder | 17:16 | |
*** patrickeast has quit IRC | 17:16 | |
thingee | jgriffith, eharney: also really, how often do we have these middle points of confusion? examples please | 17:16 |
eharney | thingee: not often afaik | 17:17 |
*** belmoreira has joined #openstack-cinder | 17:19 | |
*** belmoreira has quit IRC | 17:21 | |
*** annasort has quit IRC | 17:24 | |
*** p0rtal has joined #openstack-cinder | 17:27 | |
*** sbezverk has quit IRC | 17:28 | |
*** sbezverk has joined #openstack-cinder | 17:29 | |
*** jwcroppe has joined #openstack-cinder | 17:30 | |
openstackgerrit | Eric Harney proposed openstack/cinder: Remove RetypeVD class https://review.openstack.org/212118 | 17:30 |
*** patrickeast has joined #openstack-cinder | 17:31 | |
*** dflorea has joined #openstack-cinder | 17:31 | |
*** p0rtal has quit IRC | 17:32 | |
*** winston-d is now known as winston-d_zZZ | 17:33 | |
*** Yogi1 has joined #openstack-cinder | 17:36 | |
*** sbezverk has quit IRC | 17:39 | |
jgriffith | tbarron: did you even read my comment on this: https://review.openstack.org/#/c/206559/ | 17:39 |
jgriffith | eharney: why is that two patches? | 17:40 |
jgriffith | Maybe I should just go away | 17:40 |
*** sbezverk has joined #openstack-cinder | 17:40 | |
tbarron | jgriffith: I think you are making a good point but there doesn't seem to me to be any danger in doing it as two patches | 17:41 |
tbarron | jgriffith: there might be more "discussion" about removing RetypeVD, and we should get NFS driver working again. | 17:41 |
jgriffith | tbarron: Well... except that patch I referenced and gave a -1 that you then promptly gave a +1 I pointed out that all it did was short circuit the abc class that is in place | 17:41 |
tbarron | jgriffith: does that make sense? | 17:41 |
eharney | just submitted it as two patches so that we can separate a) fixing a bug vs b) people hashing out the abc details further | 17:41 |
*** p0rtal has joined #openstack-cinder | 17:41 | |
jgriffith | tbarron: making it absolutely useless | 17:41 |
eharney | yeha, what tbarron said ^ | 17:41 |
eharney | yeah* | 17:42 |
tbarron | jgriffith: yeah, but does that do any real harm? | 17:42 |
tbarron | jgriffith: only if we keep it | 17:42 |
jgriffith | tbarron: eharney so again.... the multi-class abc stuff is flawed, and just ignoring it isn't "fixing" your nfs problem | 17:42 |
jgriffith | tbarron: eharney you guys need to decide you're in or out on the multi-class abc thing | 17:42 |
jgriffith | tbarron: eharney you can't have it both ways | 17:42 |
*** iurygregory has joined #openstack-cinder | 17:43 | |
jgriffith | which is what I tried to propose last week | 17:43 |
eharney | this isn't trying to have it both ways, and i still don't really see it as flawed | 17:43 |
eharney | the problem is that retype() isn't abstract | 17:43 |
eharney | which is much different from something like consistency groups | 17:43 |
jgriffith | eharney: you just completely circumvented it here: https://review.openstack.org/#/c/206559/ | 17:43 |
jgriffith | eharney: by implementing it in the base class with a true imply the abc decorator will always work, it now does nothing | 17:43 |
eharney | retype isn't an abstract method... so i'm not sure why it should be in an abc class | 17:43 |
jgriffith | eharney: which was my point last week that all these classes for each feature are rather silly | 17:44 |
tbarron | I thought last week we agreed that there were things that needed to be fixed and cleaned up but that, all things considered, we were moving ahead with what agreed to in Kilo. | 17:44 |
eharney | i still think the general theory makes sense for optional features | 17:44 |
*** annegentle has quit IRC | 17:44 | |
jgriffith | and that's fine | 17:45 |
jgriffith | but it makes no sense to submit a patch that intentionally circumvents the check that is in place, unless you remove that check in the same patch IMO | 17:45 |
jgriffith | all you did was created a dead piece of code | 17:45 |
eharney | it's not circumventing a check... it's putting a method into a class where it needs to exist | 17:45 |
eharney | which code is dead? the NFS driver uses the method in the base class, other drivers still pull from RetypeVD until we decide if we are keeping that or ont | 17:46 |
eharney | not* | 17:46 |
jgriffith | eharney: but the whole point of the abstract class "retype" is to enforce imply in the drivers that inherit form it. If you impl it in the base class that EVERY driver inherits then it does absolutely nothing | 17:46 |
jgriffith | and it is in fact circumventing the check... which is why you did it to make NFS work | 17:46 |
jgriffith | eharney: so that's my point!!! | 17:47 |
eharney | the NFS driver didn't have a check, because it doesn't inherit from RetypeVD... it's not circumventing anything... | 17:47 |
jgriffith | the NFS driver uses the method in the base class, other drivers still pull from RetypeVD | 17:47 |
jgriffith | ^^ your quote | 17:47 |
jgriffith | What's the point if we just say... "we have this.. but you can just work around it and not inherit it and do your own thing" | 17:47 |
*** earlephilhower has quit IRC | 17:47 | |
jgriffith | I don't understand, all this does is just continue to make things more confusing and difficult to follow | 17:48 |
eharney | the goal is to have all drivers inherit from the appropriate abc classes, but we haven't gotten that far with implementation yet | 17:48 |
eharney | nobody is going to port all the drivers when everyone is still arguing over whether they like using abc in general | 17:48 |
jgriffith | driver A may or may not be using the abc meatballs... but it still implements the feature defined in the metaclass | 17:48 |
jgriffith | while driver C does it a different way | 17:48 |
jgriffith | and so on and so on | 17:48 |
eharney | the metaclass was wrong, though | 17:49 |
jgriffith | Nobody is arguing, you all rejected my change proposal promptly | 17:49 |
eharney | Retype doesn't make sense as a metaclass | 17:49 |
eharney | though i left plenty of room on the review for someone to come tell me otherwise | 17:49 |
jgriffith | I agree.. none of the features as metaclasses make sense IMO :) | 17:49 |
jgriffith | eharney: I'm saying your first patch need to remove it | 17:49 |
jgriffith | that's all | 17:49 |
jgriffith | not hack around it | 17:49 |
tbarron | all three of us are for removing it, I just didn't have a problem with it being two reviews to get it done | 17:51 |
Swanson | jungleboyj: looks like ci is working again for stable/kilo. If you get a chance to poke at https://review.openstack.org/#/c/210564/ that would be great. | 17:56 |
*** markus_z has quit IRC | 17:58 | |
*** salv-orl_ has joined #openstack-cinder | 17:59 | |
*** earlephilhower has joined #openstack-cinder | 18:00 | |
*** salv-orlando has quit IRC | 18:01 | |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Adapt SnapshotController to view builder https://review.openstack.org/211592 | 18:02 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add pagination to snapshots https://review.openstack.org/195071 | 18:02 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: On Volume list only retrieve needed data from DB https://review.openstack.org/212126 | 18:02 |
*** dannywilson has quit IRC | 18:04 | |
*** DTadrzak has joined #openstack-cinder | 18:05 | |
*** dflorea has quit IRC | 18:06 | |
*** akshai has joined #openstack-cinder | 18:06 | |
*** dflorea has joined #openstack-cinder | 18:07 | |
openstackgerrit | Anthony Lee proposed openstack/cinder: Add multipath support to 3PAR iSCSI driver https://review.openstack.org/180779 | 18:07 |
*** anshul has quit IRC | 18:08 | |
*** vokt has joined #openstack-cinder | 18:13 | |
*** bapalm_ has joined #openstack-cinder | 18:14 | |
*** Maike has joined #openstack-cinder | 18:20 | |
*** xyang has quit IRC | 18:22 | |
*** bapalm has joined #openstack-cinder | 18:23 | |
*** bapalm has quit IRC | 18:24 | |
*** tim_o has joined #openstack-cinder | 18:24 | |
*** annasort has joined #openstack-cinder | 18:34 | |
*** akshai has quit IRC | 18:35 | |
*** skylerberg has quit IRC | 18:36 | |
*** salv-orl_ has quit IRC | 18:46 | |
openstackgerrit | Kurt Martin proposed openstack/cinder: Over subscription for HP 3PAR drivers https://review.openstack.org/212144 | 18:46 |
*** xyang has joined #openstack-cinder | 18:47 | |
*** salv-orlando has joined #openstack-cinder | 18:49 | |
*** boris-42 has joined #openstack-cinder | 18:54 | |
openstackgerrit | Joseph Vokt proposed openstack/cinder: 3PAR: Adding performance metrics to volume status https://review.openstack.org/200737 | 18:54 |
*** IanGovett has joined #openstack-cinder | 18:56 | |
*** anshul has joined #openstack-cinder | 18:56 | |
*** IanGovett1 has quit IRC | 18:56 | |
*** dannywilson has joined #openstack-cinder | 18:59 | |
*** esker has quit IRC | 18:59 | |
*** dflorea has quit IRC | 19:03 | |
*** dflorea has joined #openstack-cinder | 19:04 | |
*** jungleboyj has quit IRC | 19:04 | |
*** dflorea has quit IRC | 19:08 | |
*** dflorea has joined #openstack-cinder | 19:08 | |
*** dflorea has quit IRC | 19:10 | |
*** shyama has quit IRC | 19:13 | |
*** wilson has quit IRC | 19:16 | |
*** jungleboyj has joined #openstack-cinder | 19:17 | |
*** apoorvad has quit IRC | 19:19 | |
openstackgerrit | Diem Tran proposed openstack/cinder: Local cache feature of Oracle ZFSSA drivers https://review.openstack.org/212155 | 19:21 |
*** mriedem1 is now known as mriedem | 19:24 | |
Swanson | jungleboyj: Thanks for the review! | 19:27 |
Swanson | Now to get someone else to look at it... | 19:28 |
openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: Adds manage/unmanage methods for HNAS drivers. https://review.openstack.org/197999 | 19:35 |
*** annegentle has joined #openstack-cinder | 19:40 | |
*** adurbin_ has quit IRC | 19:40 | |
*** amoturi has quit IRC | 19:41 | |
*** amoturi has joined #openstack-cinder | 19:42 | |
Swanson | xyang: Can you take a peek at this one? https://review.openstack.org/#/c/210564/ (This is the stable/kilo port and you reviewed the master branch version.) | 19:43 |
xyang | Swanson: sure | 19:44 |
Swanson | xyang: Thanks! | 19:44 |
xyang | Swanson: np | 19:45 |
openstackgerrit | Diem Tran proposed openstack/cinder: Local cache feature of Oracle ZFSSA drivers https://review.openstack.org/212155 | 19:46 |
*** Maike has quit IRC | 19:46 | |
*** esker has joined #openstack-cinder | 19:46 | |
openstackgerrit | xing-yang proposed openstack/os-brick: Add device_path to ScaleIO connector https://review.openstack.org/212165 | 19:46 |
*** esker has quit IRC | 19:46 | |
Adriano__ | smcginnis: thingee : I had to rebase the patch you approved yesterday (https://review.openstack.org/197999) | 19:47 |
Adriano__ | smcginnis: thingee : No additional changes :) | 19:47 |
*** esker has joined #openstack-cinder | 19:47 | |
*** sgotliv has joined #openstack-cinder | 19:48 | |
*** DTadrzak_ has joined #openstack-cinder | 19:49 | |
*** esker has quit IRC | 19:50 | |
*** timcl has quit IRC | 19:50 | |
DTadrzak_ | thangp: ping | 19:51 |
thangp | DTadrzak: hey | 19:51 |
*** anshul has quit IRC | 19:52 | |
DTadrzak_ | Hello Thang I noticed that we have one incorrect property in base.py. According to versionedobjects spec base.py should contain a OBJ_PROJECT_NAMESPACE. I think that we should use OBJ_PROJECT_NAMESPACE ='cinder' instead of OBJ_SERIAL_NAMESPACE. But if we change this We migth have a problem with compability mode in snapshots. What shall we do? | 19:52 |
thangp | OBJ_PROJECT_NAMESPACE is correct | 19:53 |
thangp | https://github.com/openstack/cinder/blob/master/cinder/objects/base.py#L44 | 19:53 |
thangp | OBJ_PROJECT_NAMESPACE = 'cinder' | 19:53 |
scottda | tbarron: Does the PosixBackupDriver for incremental backups work for more than NFS? | 19:53 |
thangp | DTadrzak: not sure what you are talking about | 19:53 |
*** BharatK has quit IRC | 19:56 | |
tbarron | scottda: yes, it just assumes a local file system path, can be mounted from remote (nfs, smb, glusterfs, ..) or just local with space. | 19:58 |
tbarron | scottda: handy for when you have devstack w/o swift enabled and you need to run backup tests | 19:58 |
DTadrzak_ | Well I think that i'm so tired and i need to rest. I don't know why did I saw OBJ_SERIAL_NAMESPACE instead of OBJ_PROJECT_NAMESPACE . Nvm. ;) | 19:58 |
scottda | tbarron: thanks | 19:59 |
thangp | :) | 19:59 |
openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Adds framework for get_capabilities() feature https://review.openstack.org/201243 | 19:59 |
scottda | jaypipes: see tbarron reply ^^^ | 19:59 |
tbarron | beetlejuice | 19:59 |
*** dflorea has joined #openstack-cinder | 19:59 | |
tbarron | scottda: jaypipes: np | 20:00 |
jaypipes | cheers | 20:00 |
*** lcurtis has quit IRC | 20:01 | |
*** lcurtis has joined #openstack-cinder | 20:02 | |
*** sgotliv has quit IRC | 20:02 | |
*** dustins has quit IRC | 20:03 | |
DTadrzak_ | thangp: I have one more question shouldn't we add a objects.register_all() to methods main to some files in cinder/cmd? I think that we might have a problem here when we try to operate on objects which aren't imported? | 20:03 |
openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Efficient volume copy for generic volume migration https://review.openstack.org/183701 | 20:04 |
DTadrzak_ | thangp: today I have deployed a setup on devstack and i tried to call cinder-all and I have a lot of error which contain information that obejcts.Backup is unknown. | 20:05 |
thangp | DTadrzak: some of the cinder/cmd dont require it | 20:06 |
thangp | DTadrzak: like rtstool | 20:06 |
DTadrzak_ | that's right but what do you think about cmd/all.py? | 20:07 |
*** apoorvad has joined #openstack-cinder | 20:07 | |
thangp | DTadrzak: probably, but i havent run into the error you see | 20:07 |
*** skylerberg has joined #openstack-cinder | 20:10 | |
DTadrzak_ | i'm going to investigate it deeper tomorrow If Problem still exist I will send you some more information about it | 20:10 |
*** apoorvad has quit IRC | 20:10 | |
thangp | sure | 20:10 |
*** apoorvad has joined #openstack-cinder | 20:11 | |
*** akshai has joined #openstack-cinder | 20:12 | |
*** hemna__ has joined #openstack-cinder | 20:12 | |
*** akshai has quit IRC | 20:12 | |
*** amoturi has quit IRC | 20:13 | |
*** merooney has quit IRC | 20:14 | |
*** ronis_ has quit IRC | 20:17 | |
*** tsbot_ has joined #openstack-cinder | 20:18 | |
*** dustins has joined #openstack-cinder | 20:21 | |
*** xyang has quit IRC | 20:29 | |
*** akerr has quit IRC | 20:29 | |
*** garthb has joined #openstack-cinder | 20:29 | |
openstackgerrit | Jay Bryant proposed openstack/cinder: Add oslo.vmware into test-requirements https://review.openstack.org/212182 | 20:35 |
openstackgerrit | Thang Pham proposed openstack/cinder: Register RPC and object versions https://review.openstack.org/209701 | 20:36 |
openstackgerrit | Thang Pham proposed openstack/cinder: Create service_versions table https://review.openstack.org/205622 | 20:36 |
*** aorourke has quit IRC | 20:39 | |
*** bswartz has quit IRC | 20:40 | |
*** dflorea has quit IRC | 20:41 | |
openstackgerrit | Joel Coffman proposed openstack/cinder-specs: Add volume encryption metadata spec https://review.openstack.org/176050 | 20:45 |
*** akshai has joined #openstack-cinder | 20:47 | |
*** mriedem is now known as mriedem_away | 20:47 | |
*** DTadrzak_ has quit IRC | 20:48 | |
*** mriedem_away has quit IRC | 20:48 | |
*** xyang has joined #openstack-cinder | 20:51 | |
openstackgerrit | Thang Pham proposed openstack/cinder: Conversion to volume object https://review.openstack.org/177054 | 20:52 |
*** edtubill has quit IRC | 20:52 | |
*** garthb has quit IRC | 20:55 | |
*** ebalduf_ has joined #openstack-cinder | 20:57 | |
*** dflorea has joined #openstack-cinder | 20:58 | |
*** krtaylor has quit IRC | 20:58 | |
*** ebalduf has quit IRC | 21:00 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: WIP create_export only call when not attached https://review.openstack.org/210140 | 21:00 |
openstackgerrit | Jose Porrua proposed openstack/os-brick: Adding CHAP discovery logic to os-brick https://review.openstack.org/212191 | 21:02 |
*** Lee1092 has quit IRC | 21:03 | |
*** thangp has quit IRC | 21:04 | |
openstackgerrit | Thang Pham proposed openstack/cinder: Continue conversion to volume object https://review.openstack.org/201404 | 21:07 |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Don't return Exception when volume is detached https://review.openstack.org/212193 | 21:08 |
*** IanGovett has quit IRC | 21:09 | |
hemna_ | xyang1, ^^ | 21:09 |
xyang1 | hemna_: Thanks, will take a look | 21:10 |
*** krtaylor has joined #openstack-cinder | 21:11 | |
*** Yogi1 has left #openstack-cinder | 21:12 | |
*** bswartz has joined #openstack-cinder | 21:18 | |
jungleboyj | winston-d_zZZ: Which patch from Vincent did you have concerns upon that weren't addressed. I don't see any comments in the migration ones I was looking at. | 21:20 |
*** crose has quit IRC | 21:21 | |
*** zul has quit IRC | 21:21 | |
*** mdnadeem has joined #openstack-cinder | 21:30 | |
*** dflorea has quit IRC | 21:33 | |
*** dflorea has joined #openstack-cinder | 21:34 | |
*** zul has joined #openstack-cinder | 21:35 | |
*** mdnadeem has quit IRC | 21:38 | |
*** akshai has quit IRC | 21:39 | |
*** dflorea has quit IRC | 21:39 | |
*** xyang has quit IRC | 21:41 | |
*** qwebirc61555 has joined #openstack-cinder | 21:47 | |
*** qwebirc61555 has quit IRC | 21:47 | |
*** strony-zhang has joined #openstack-cinder | 21:47 | |
*** dustins has quit IRC | 21:48 | |
strony-zhang | Generally, how long does restoring 1 GB volume backup take? The backup driver is swift. | 21:49 |
strony-zhang | It is a full restore. | 21:49 |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Don't return Exception when volume is detached https://review.openstack.org/212193 | 21:50 |
*** merooney has joined #openstack-cinder | 21:52 | |
*** annegentle has quit IRC | 21:54 | |
*** olmy0414 has quit IRC | 21:57 | |
*** jungleboyj has quit IRC | 21:59 | |
*** dflorea has joined #openstack-cinder | 21:59 | |
strony-zhang | Again. Generally, how long does restoring an backup of 1 GB volume take? The backup driver is swift. | 22:00 |
jgriffith | strony-zhang: I'd answer if I could.... I know the cinder-backup operations are pretty slow; but for 1G it shouldn't be bad | 22:02 |
jgriffith | strony-zhang: depends on your bandwidth / read throughput from SWIFT and of course the dd to the volume | 22:02 |
jgriffith | strony-zhang: things like cinder backend make a difference, network topology etc | 22:03 |
jgriffith | hemna: dang you!!! :) I have a version of that detach patch half done... then went to meetings :( | 22:04 |
strony-zhang | Thanks John. In the same area, like Bay area, in my tests, restoring 1 GB volume takes 15 minutes. Is that normal? | 22:05 |
*** apoorvad has quit IRC | 22:05 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Don't return Exception when volume is detached https://review.openstack.org/212193 | 22:06 |
*** edmondsw has quit IRC | 22:06 | |
jgriffith | strony-zhang: OUCH | 22:08 |
jgriffith | strony-zhang: that's pretty horrid IMO | 22:08 |
jgriffith | strony-zhang: but I wouldn't be surprised; and no I don't have any real experience with it. I used device optimized backups | 22:08 |
thingee | harlowja: are you doing a 0.6.2 release? | 22:09 |
jgriffith | strony-zhang: I have heard that a 200G volume can take over 20 hours | 22:09 |
thingee | harlowja: https://bugs.launchpad.net/taskflow/+bug/1484267 | 22:09 |
openstack | Launchpad bug 1484267 in taskflow "taskflow 0.6.1 has uncapped networkx which breaks cinder py26 unit tests in juno" [Undecided,Confirmed] | 22:09 |
jgriffith | so that doesn't seem that out of the realm of possibility | 22:09 |
hemna_ | jgriffith, c-data service :P | 22:09 |
jgriffith | hemna_: :) | 22:10 |
hemna_ | jgriffith, https://review.openstack.org/#/c/212193/ I don't see any inline comments ? | 22:10 |
vilobhmm | thingee : harlowja is away on vacation..someone or sputnik13 on #opestack-state-management should be able to answer | 22:10 |
jgriffith | hemna: I made some comments on your detach patch. Good idea moving it into the DB layer, but I think you missed a few other calls that use that exception | 22:10 |
strony-zhang | jgriffith: Backup seems to be better than restoring. Thanks. | 22:11 |
jgriffith | strony-zhang: cool | 22:12 |
jgriffith | strony-zhang: sorry I don't have more concrete info for you | 22:12 |
strony-zhang | jgriffith: for the backup, I almost got the same performance, 6minutes for 1 GB. For the restoring, it will take more time. Thanks, John. :) | 22:14 |
jgriffith | strony-zhang: welcome... boy that's speedy :) | 22:15 |
*** apoorvad has joined #openstack-cinder | 22:15 | |
*** jecarey has quit IRC | 22:18 | |
*** diogogmt has quit IRC | 22:22 | |
*** hemna__ has quit IRC | 22:23 | |
patrickeast | wanghao: hey, i’m looking at itesting manage/unmanage snapshot https://review.openstack.org/#/c/144590/ and was just wondering if there is a cinderclient change that goes along with it or if i’m going to have to hand roll the rest calls to test it out? | 22:25 |
patrickeast | testing* | 22:26 |
*** dflorea has quit IRC | 22:26 | |
*** jungleboyj has joined #openstack-cinder | 22:30 | |
*** krtaylor has quit IRC | 22:30 | |
*** marcusvrn has quit IRC | 22:33 | |
*** bnemec has quit IRC | 22:41 | |
*** erlon has quit IRC | 22:43 | |
*** annegentle has joined #openstack-cinder | 22:44 | |
*** devlaps has joined #openstack-cinder | 22:44 | |
*** annegentle has quit IRC | 22:45 | |
*** annegentle has joined #openstack-cinder | 22:46 | |
*** annegentle has quit IRC | 22:52 | |
*** merooney has quit IRC | 22:53 | |
*** dflorea has joined #openstack-cinder | 22:56 | |
openstackgerrit | Thang Pham proposed openstack/cinder: Continue conversion to volume object https://review.openstack.org/201404 | 23:05 |
*** porrua has joined #openstack-cinder | 23:08 | |
*** bill_az has joined #openstack-cinder | 23:10 | |
*** mylu has joined #openstack-cinder | 23:11 | |
*** markvoelker has quit IRC | 23:14 | |
*** porrua has quit IRC | 23:15 | |
*** zul has quit IRC | 23:17 | |
*** gouthamr has quit IRC | 23:19 | |
openstackgerrit | Patrick East proposed openstack/cinder: Implement Clone CG in Pure Volume Drivers https://review.openstack.org/211803 | 23:25 |
openstackgerrit | Patrick East proposed openstack/cinder: Switch Pure volume drivers to use Snapshot Objects https://review.openstack.org/211802 | 23:25 |
openstackgerrit | Patrick East proposed openstack/cinder: Modify naming helpers in Pure Volume Drivers https://review.openstack.org/211801 | 23:25 |
openstackgerrit | Patrick East proposed openstack/cinder: Fix Pure create volume from cgsnapshot https://review.openstack.org/211804 | 23:25 |
*** chlong has quit IRC | 23:25 | |
*** xyang has joined #openstack-cinder | 23:26 | |
*** xyang has quit IRC | 23:27 | |
*** salv-orl_ has joined #openstack-cinder | 23:35 | |
*** salv-orlando has quit IRC | 23:37 | |
*** gouthamr has joined #openstack-cinder | 23:39 | |
*** theanalyst has quit IRC | 23:42 | |
*** theanalyst has joined #openstack-cinder | 23:42 | |
tsekiyama | patrickeast: Have you ever seen an error like this with the generic image cache patch? > "InternalError: (1292, u"Incorrect datetime value: '2015-08-11 22:13:42+00:00' for column 'image_updated_at' at row 1") | 23:44 |
patrickeast | tsekiyama: :O nope | 23:45 |
patrickeast | thats a new one | 23:45 |
tsekiyama | patrickeast: It looks like mariadb dislike '+00:00' part | 23:45 |
patrickeast | tsekiyama: do you have a stack trace? | 23:45 |
*** lcurtis has quit IRC | 23:45 | |
tsekiyama | patrickeast: just a moment | 23:46 |
patrickeast | tsekiyama: afaik it should be stripping out the timezone info before we try and insert anything, must be some case where it doesn’t | 23:46 |
tsekiyama | patrickeast: http://paste.openstack.org/show/412805/ | 23:47 |
patrickeast | tsekiyama: thanks! | 23:48 |
patrickeast | tsekiyama: just to check, you are using the latest patchset of it? | 23:50 |
patrickeast | tsekiyama: i ask because ‘2015-08-12 19:32:00.865 TRACE oslo_messaging.rpc.dispatcher File "/opt/stack/cinder/cinder/image/cache.py", line 116, in create_cache_entry’ | 23:50 |
patrickeast | line 116 in the code i’m looking at isn’t doing a db operation, its down on 118 | 23:51 |
*** akerr has joined #openstack-cinder | 23:51 | |
tsekiyama | patrickeast: ah that might be because I've cherry_pick'ed your patch | 23:52 |
patrickeast | tsekiyama: gotcha | 23:52 |
tsekiyama | patrickeast: what I did is "git fetch https://review.openstack.org/openstack/cinder refs/changes/95/195795/34 && git cherry-pick FETCH_HEAD" | 23:53 |
tsekiyama | patrickeast: so it is the latest patch | 23:53 |
patrickeast | tsekiyama: on top of your glance change right? | 23:54 |
tsekiyama | yes | 23:54 |
*** dflorea has quit IRC | 23:54 | |
patrickeast | ok cool, just wanna make sure i can repo it | 23:54 |
patrickeast | tsekiyama: ooh i see, i think it needs something like ‘image_updated_at.replace(tzinfo=None)’ before we try and pass it in | 23:55 |
patrickeast | or maybe in the db api implementation | 23:55 |
openstackgerrit | Petrut Lucian proposed openstack/cinder: RemoteFS: fix Python 3.4 compatibility https://review.openstack.org/210123 | 23:55 |
patrickeast | tsekiyama: i convert it to UTC but left in the time zone info | 23:56 |
tsekiyama | patrickeast: that makes sense | 23:56 |
tsekiyama | patrickeast: OK great, image_updated_at.replace(tzinfo=None) works! | 23:58 |
patrickeast | tsekiyama: sweet, im updating the patch with fixes for your other comments, i’ll add that in too | 23:58 |
tsekiyama | patrickeast: thanks | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!