*** 16WAALNTP has quit IRC | 00:01 | |
*** haomaiwang has joined #openstack-cinder | 00:01 | |
*** alkhodos has quit IRC | 00:06 | |
*** alkhodos has joined #openstack-cinder | 00:07 | |
openstackgerrit | Nate Potter proposed openstack/python-cinderclient: Implement cinder free command https://review.openstack.org/301445 | 00:14 |
---|---|---|
*** diogogmt has quit IRC | 00:14 | |
*** garthb has quit IRC | 00:15 | |
*** yangyapeng has quit IRC | 00:19 | |
*** Lickitysplitted has quit IRC | 00:20 | |
*** laughterwym has joined #openstack-cinder | 00:22 | |
*** bjornar has quit IRC | 00:22 | |
*** arch-nemesis has quit IRC | 00:25 | |
*** mtanino has quit IRC | 00:25 | |
*** alkhodos has quit IRC | 00:30 | |
*** akerr has quit IRC | 00:42 | |
*** esker has quit IRC | 00:43 | |
*** dflorea has quit IRC | 00:46 | |
*** esker has joined #openstack-cinder | 00:47 | |
*** dave-mcc_ has joined #openstack-cinder | 00:50 | |
*** dave-mccowan has quit IRC | 00:51 | |
*** knikolla has quit IRC | 00:56 | |
*** iceyao has joined #openstack-cinder | 00:56 | |
*** haomaiwang has quit IRC | 01:01 | |
*** haomaiwang has joined #openstack-cinder | 01:01 | |
*** mylu has quit IRC | 01:03 | |
*** jwcroppe has joined #openstack-cinder | 01:06 | |
*** erlon has quit IRC | 01:07 | |
*** diogogmt has joined #openstack-cinder | 01:07 | |
*** zzzeek_ has quit IRC | 01:10 | |
*** jwcroppe has quit IRC | 01:12 | |
*** zzzeek has joined #openstack-cinder | 01:13 | |
*** yangyapeng has joined #openstack-cinder | 01:13 | |
*** baojg has joined #openstack-cinder | 01:19 | |
*** wxy has joined #openstack-cinder | 01:20 | |
*** laughterwym has quit IRC | 01:20 | |
*** mriedem has quit IRC | 01:20 | |
*** laughterwym has joined #openstack-cinder | 01:20 | |
*** akshai has joined #openstack-cinder | 01:21 | |
*** akshai_ has joined #openstack-cinder | 01:22 | |
*** haomaiwang has quit IRC | 01:23 | |
*** esker has quit IRC | 01:24 | |
*** akshai has quit IRC | 01:25 | |
*** laughterwym has quit IRC | 01:25 | |
*** cheneydc has joined #openstack-cinder | 01:27 | |
*** Lee1092 has joined #openstack-cinder | 01:29 | |
*** salv-orlando has joined #openstack-cinder | 01:32 | |
*** EinstCrazy has joined #openstack-cinder | 01:34 | |
*** apoorvad has quit IRC | 01:35 | |
*** mylu has joined #openstack-cinder | 01:38 | |
*** shausy has joined #openstack-cinder | 01:39 | |
*** baojg has quit IRC | 01:40 | |
*** jamielennox is now known as jamielennox|away | 01:43 | |
*** enriquetaso has quit IRC | 01:44 | |
*** laughterwym has joined #openstack-cinder | 01:46 | |
*** salv-orlando has quit IRC | 01:47 | |
*** iceyao has quit IRC | 01:50 | |
*** iceyao has joined #openstack-cinder | 01:51 | |
*** jamielennox|away is now known as jamielennox | 01:53 | |
*** laughterwym has quit IRC | 01:54 | |
*** laughterwym has joined #openstack-cinder | 01:57 | |
*** houming has joined #openstack-cinder | 01:59 | |
*** laughterwym has quit IRC | 01:59 | |
*** laughterwym has joined #openstack-cinder | 01:59 | |
*** karthikp has joined #openstack-cinder | 02:00 | |
*** EinstCra_ has joined #openstack-cinder | 02:01 | |
*** EinstCrazy has quit IRC | 02:04 | |
*** lixiaoy1 has quit IRC | 02:08 | |
*** haomaiwang has joined #openstack-cinder | 02:13 | |
*** shausy has quit IRC | 02:13 | |
openstackgerrit | Nate Potter proposed openstack/cinder: Implement backend usage reporting https://review.openstack.org/301444 | 02:15 |
*** mylu has quit IRC | 02:16 | |
*** mylu has joined #openstack-cinder | 02:18 | |
*** zzzeek has quit IRC | 02:21 | |
*** zzzeek has joined #openstack-cinder | 02:23 | |
*** diablo_rojo has quit IRC | 02:29 | |
*** jwcroppe has joined #openstack-cinder | 02:35 | |
*** Julien-zte has joined #openstack-cinder | 02:36 | |
*** Suyash has joined #openstack-cinder | 02:42 | |
*** tobe has joined #openstack-cinder | 02:42 | |
*** tobe has quit IRC | 02:42 | |
*** mylu has quit IRC | 02:44 | |
*** lixiaoy1 has joined #openstack-cinder | 02:45 | |
*** raunak_ has quit IRC | 02:49 | |
*** mylu has joined #openstack-cinder | 02:49 | |
*** Suyash has quit IRC | 02:56 | |
*** iceyao_ has joined #openstack-cinder | 02:57 | |
*** laughter_ has joined #openstack-cinder | 03:00 | |
*** haomaiwang has quit IRC | 03:01 | |
*** gouthamr has quit IRC | 03:01 | |
*** haomaiwang has joined #openstack-cinder | 03:01 | |
*** iceyao has quit IRC | 03:01 | |
*** salv-orlando has joined #openstack-cinder | 03:03 | |
*** laughterwym has quit IRC | 03:04 | |
*** mylu has quit IRC | 03:10 | |
*** jwcroppe_ has joined #openstack-cinder | 03:11 | |
*** laughter_ has quit IRC | 03:12 | |
*** laughterwym has joined #openstack-cinder | 03:12 | |
*** mylu has joined #openstack-cinder | 03:13 | |
*** salv-orlando has quit IRC | 03:13 | |
*** mylu has quit IRC | 03:14 | |
*** mylu has joined #openstack-cinder | 03:15 | |
*** jwcroppe has quit IRC | 03:15 | |
*** laughterwym has quit IRC | 03:17 | |
*** dflorea has joined #openstack-cinder | 03:19 | |
*** dflorea has quit IRC | 03:20 | |
*** karthikp has quit IRC | 03:21 | |
*** laughterwym has joined #openstack-cinder | 03:22 | |
openstackgerrit | Cao ShuFeng proposed openstack/cinder: rmemove extra quota commit of manage snapshot https://review.openstack.org/298522 | 03:22 |
*** markvoelker has quit IRC | 03:26 | |
*** laughter_ has joined #openstack-cinder | 03:32 | |
*** dflorea has joined #openstack-cinder | 03:34 | |
*** esker has joined #openstack-cinder | 03:34 | |
*** laughterwym has quit IRC | 03:35 | |
*** Julien-zte has quit IRC | 03:38 | |
*** Julien-zte has joined #openstack-cinder | 03:41 | |
*** mylu has quit IRC | 03:41 | |
*** mylu has joined #openstack-cinder | 03:41 | |
*** mylu has quit IRC | 03:42 | |
*** karthikp has joined #openstack-cinder | 03:42 | |
*** mylu has joined #openstack-cinder | 03:42 | |
*** dflorea has quit IRC | 03:43 | |
*** dflorea has joined #openstack-cinder | 03:43 | |
*** links has joined #openstack-cinder | 03:52 | |
*** esker has quit IRC | 03:52 | |
*** resker has joined #openstack-cinder | 03:52 | |
*** dflorea has quit IRC | 03:52 | |
*** esker has joined #openstack-cinder | 03:57 | |
*** resker has quit IRC | 03:59 | |
*** haomaiwang has quit IRC | 04:01 | |
*** 6A4AAM2CW has joined #openstack-cinder | 04:01 | |
*** Poornima has joined #openstack-cinder | 04:02 | |
openstackgerrit | Dao Cong Tien proposed openstack/python-cinderclient: Fix typos in docstrings and comments https://review.openstack.org/302002 | 04:02 |
*** esker has quit IRC | 04:05 | |
*** esker has joined #openstack-cinder | 04:07 | |
*** sgotliv has joined #openstack-cinder | 04:08 | |
*** jwcroppe_ has quit IRC | 04:13 | |
*** jwcroppe has joined #openstack-cinder | 04:14 | |
*** laughter_ has quit IRC | 04:17 | |
*** sgotliv has quit IRC | 04:17 | |
*** jwcroppe_ has joined #openstack-cinder | 04:17 | |
*** jwcroppe has quit IRC | 04:19 | |
*** laughterwym has joined #openstack-cinder | 04:19 | |
*** jamielennox is now known as jamielennox|away | 04:21 | |
*** sgotliv has joined #openstack-cinder | 04:23 | |
*** laughterwym has quit IRC | 04:24 | |
*** markvoelker has joined #openstack-cinder | 04:26 | |
*** jamielennox|away is now known as jamielennox | 04:29 | |
*** Julien-zte has quit IRC | 04:31 | |
*** salv-orlando has joined #openstack-cinder | 04:31 | |
*** markvoelker has quit IRC | 04:32 | |
*** diogogmt has quit IRC | 04:33 | |
*** sgotliv has quit IRC | 04:34 | |
*** avishay has joined #openstack-cinder | 04:36 | |
*** shausy has joined #openstack-cinder | 04:39 | |
*** salv-orlando has quit IRC | 04:39 | |
*** edtubill has joined #openstack-cinder | 04:41 | |
*** ChubYann has quit IRC | 04:46 | |
*** mylu has quit IRC | 04:51 | |
*** laughterwym has joined #openstack-cinder | 04:56 | |
openstackgerrit | Sheel Rana proposed openstack/cinder: Microversion of Bootable filter in cinder list https://review.openstack.org/278605 | 05:00 |
*** 6A4AAM2CW has quit IRC | 05:01 | |
*** jamielennox is now known as jamielennox|away | 05:01 | |
*** haomaiwa_ has joined #openstack-cinder | 05:01 | |
*** laughter_ has joined #openstack-cinder | 05:06 | |
*** mylu has joined #openstack-cinder | 05:08 | |
*** laughterwym has quit IRC | 05:09 | |
*** jamielennox|away is now known as jamielennox | 05:11 | |
*** sgotliv has joined #openstack-cinder | 05:11 | |
*** mylu has quit IRC | 05:12 | |
*** jamielennox is now known as jamielennox|away | 05:22 | |
*** jwcroppe_ has quit IRC | 05:25 | |
openstackgerrit | Cao ShuFeng proposed openstack/cinder: remove extra quota commit of manage snapshot https://review.openstack.org/298522 | 05:28 |
openstackgerrit | Cao ShuFeng proposed openstack/cinder: remove extra quota commit of manage snapshot https://review.openstack.org/298522 | 05:28 |
*** akshai_ has quit IRC | 05:28 | |
*** krtaylor has quit IRC | 05:29 | |
*** jamielennox|away is now known as jamielennox | 05:30 | |
*** sekrit has quit IRC | 05:30 | |
*** krtaylor has joined #openstack-cinder | 05:31 | |
*** laughter_ has quit IRC | 05:34 | |
*** iceyao has joined #openstack-cinder | 05:34 | |
*** shausy has quit IRC | 05:35 | |
*** timcl has quit IRC | 05:35 | |
*** nkrinner has joined #openstack-cinder | 05:35 | |
*** shausy has joined #openstack-cinder | 05:36 | |
*** karthikp has quit IRC | 05:36 | |
*** divab has quit IRC | 05:36 | |
*** laughterwym has joined #openstack-cinder | 05:37 | |
*** iceyao_ has quit IRC | 05:38 | |
*** liverpooler has quit IRC | 05:44 | |
*** salv-orlando has joined #openstack-cinder | 05:46 | |
*** salv-orlando has quit IRC | 05:55 | |
*** iceyao has quit IRC | 05:55 | |
*** iceyao has joined #openstack-cinder | 05:56 | |
*** dave-mcc_ has quit IRC | 05:56 | |
*** qeelee has joined #openstack-cinder | 05:57 | |
*** rcernin has joined #openstack-cinder | 05:59 | |
*** haomaiwa_ has quit IRC | 06:01 | |
*** haomaiwang has joined #openstack-cinder | 06:01 | |
*** sgotliv has quit IRC | 06:01 | |
*** aswadr_ has joined #openstack-cinder | 06:14 | |
*** mjura has joined #openstack-cinder | 06:23 | |
*** sekrit has joined #openstack-cinder | 06:23 | |
*** lpetrut has joined #openstack-cinder | 06:24 | |
*** edtubill has quit IRC | 06:28 | |
*** jwcroppe has joined #openstack-cinder | 06:28 | |
*** markvoelker has joined #openstack-cinder | 06:29 | |
*** belmoreira has joined #openstack-cinder | 06:30 | |
*** dave-mcc_ has joined #openstack-cinder | 06:33 | |
*** markvoelker has quit IRC | 06:35 | |
*** jwcroppe has quit IRC | 06:35 | |
*** jamielennox is now known as jamielennox|away | 06:38 | |
*** dave-mccowan has joined #openstack-cinder | 06:49 | |
*** dave-mcc_ has quit IRC | 06:54 | |
*** haomaiwang has quit IRC | 07:01 | |
*** haomaiwang has joined #openstack-cinder | 07:01 | |
*** openstackgerrit has quit IRC | 07:02 | |
*** openstackgerrit has joined #openstack-cinder | 07:03 | |
*** sheel has joined #openstack-cinder | 07:05 | |
*** liverpooler has joined #openstack-cinder | 07:05 | |
*** rcernin has quit IRC | 07:08 | |
*** rcernin has joined #openstack-cinder | 07:09 | |
*** houming has quit IRC | 07:11 | |
*** houming has joined #openstack-cinder | 07:12 | |
*** salv-orlando has joined #openstack-cinder | 07:12 | |
openstackgerrit | Vipin Balachandran proposed openstack/cinder: WIP VMware: Scale driver to large vCenter clusters https://review.openstack.org/296934 | 07:25 |
*** lpetrut has quit IRC | 07:27 | |
*** bjornar has joined #openstack-cinder | 07:31 | |
*** gcb has quit IRC | 07:41 | |
*** laughter_ has joined #openstack-cinder | 07:49 | |
*** laughterwym has quit IRC | 07:53 | |
*** gcb has joined #openstack-cinder | 07:53 | |
*** goretoxo has joined #openstack-cinder | 07:55 | |
*** jordanP has joined #openstack-cinder | 07:58 | |
*** ndipanov has joined #openstack-cinder | 07:59 | |
*** haomaiwang has quit IRC | 08:01 | |
*** haomaiwang has joined #openstack-cinder | 08:01 | |
*** sgotliv has joined #openstack-cinder | 08:09 | |
*** atuvenie has joined #openstack-cinder | 08:10 | |
*** jistr has joined #openstack-cinder | 08:22 | |
openstackgerrit | XinXiaohui proposed openstack/cinder: Update _usage_from_backup() method with versionedobjects https://review.openstack.org/262640 | 08:23 |
*** laughter_ has quit IRC | 08:24 | |
*** laughterwym has joined #openstack-cinder | 08:24 | |
*** laughter_ has joined #openstack-cinder | 08:26 | |
*** laughter_ has quit IRC | 08:26 | |
*** laughter_ has joined #openstack-cinder | 08:26 | |
*** ndipanov has quit IRC | 08:26 | |
*** laughterwym has quit IRC | 08:29 | |
*** ndipanov has joined #openstack-cinder | 08:29 | |
*** markvoelker has joined #openstack-cinder | 08:30 | |
*** bharathk has joined #openstack-cinder | 08:32 | |
*** jwcroppe has joined #openstack-cinder | 08:33 | |
*** ayush7rajoria has joined #openstack-cinder | 08:35 | |
*** markvoelker has quit IRC | 08:35 | |
*** e0ne has joined #openstack-cinder | 08:35 | |
*** jwcroppe has quit IRC | 08:38 | |
*** dave-mccowan has quit IRC | 08:43 | |
*** salv-orlando has quit IRC | 08:45 | |
*** lixiaoy1 has quit IRC | 08:54 | |
*** e0ne has quit IRC | 09:00 | |
*** haomaiwang has quit IRC | 09:01 | |
*** lpetrut has joined #openstack-cinder | 09:01 | |
*** haomaiwang has joined #openstack-cinder | 09:01 | |
openstackgerrit | Cao ShuFeng proposed openstack/cinder: ehanche error message https://review.openstack.org/296304 | 09:10 |
*** nikeshm has joined #openstack-cinder | 09:12 | |
openstackgerrit | Cao ShuFeng proposed openstack/cinder: ehanche error message [1/5] https://review.openstack.org/296304 | 09:13 |
nikeshm | hi | 09:13 |
*** salv-orlando has joined #openstack-cinder | 09:15 | |
nikeshm | my client array does not support clone volume feature inbuilt and they are looking for cinder driver so if we write a driver, "Clone Volume" feature will dependent on base driver so is it acceptable? | 09:15 |
nikeshm | http://docs.openstack.org/developer/cinder/devref/drivers.html | 09:16 |
*** mjura has quit IRC | 09:17 | |
*** salv-orlando has quit IRC | 09:19 | |
*** haomaiwang has quit IRC | 09:24 | |
*** mjura has joined #openstack-cinder | 09:30 | |
*** laughterwym has joined #openstack-cinder | 09:31 | |
*** arecknag has joined #openstack-cinder | 09:33 | |
*** laughter_ has quit IRC | 09:34 | |
*** mjura_ has joined #openstack-cinder | 09:41 | |
*** mjura has quit IRC | 09:43 | |
*** yangyapeng has quit IRC | 09:43 | |
*** yangyapeng has joined #openstack-cinder | 09:44 | |
*** mjura_ has quit IRC | 09:45 | |
*** mjura has joined #openstack-cinder | 09:46 | |
*** bjornar has quit IRC | 09:46 | |
*** yangyapeng has quit IRC | 09:49 | |
*** EinstCra_ is now known as EinstCrazy | 10:00 | |
*** yhayashi has quit IRC | 10:01 | |
*** haomaiwa_ has joined #openstack-cinder | 10:01 | |
*** cheneydc has quit IRC | 10:01 | |
*** salv-orlando has joined #openstack-cinder | 10:02 | |
*** salv-orlando has quit IRC | 10:03 | |
*** salv-orlando has joined #openstack-cinder | 10:03 | |
*** e0ne has joined #openstack-cinder | 10:06 | |
*** bjornar has joined #openstack-cinder | 10:07 | |
*** ekarlso- has quit IRC | 10:09 | |
*** sdague has joined #openstack-cinder | 10:10 | |
*** EinstCrazy has quit IRC | 10:12 | |
*** ChanServ changes topic to "npm lint jobs are failing due to a problem with npm registry. The problem is under investigation, and we will update once the issue is solved." | 10:16 | |
*** salv-orlando has quit IRC | 10:18 | |
*** lprice has joined #openstack-cinder | 10:18 | |
*** lprice1 has quit IRC | 10:18 | |
*** ekarlso- has joined #openstack-cinder | 10:21 | |
*** ekarlso- has quit IRC | 10:22 | |
*** ekarlso has joined #openstack-cinder | 10:22 | |
*** mvk_ has quit IRC | 10:23 | |
*** salv-orlando has joined #openstack-cinder | 10:26 | |
*** salv-orlando has quit IRC | 10:30 | |
*** bjornar has quit IRC | 10:31 | |
*** markvoelker has joined #openstack-cinder | 10:31 | |
*** markvoelker has quit IRC | 10:36 | |
*** jwcroppe has joined #openstack-cinder | 10:36 | |
*** gcb has quit IRC | 10:37 | |
*** gcb has joined #openstack-cinder | 10:38 | |
*** yangyapeng has joined #openstack-cinder | 10:40 | |
*** jwcroppe has quit IRC | 10:41 | |
*** tpsilva has joined #openstack-cinder | 10:43 | |
*** gcb has quit IRC | 10:43 | |
*** gcb has joined #openstack-cinder | 10:44 | |
*** salv-orlando has joined #openstack-cinder | 10:47 | |
*** salv-orlando has quit IRC | 10:48 | |
*** salv-orlando has joined #openstack-cinder | 10:57 | |
*** salv-orlando has quit IRC | 10:57 | |
*** salv-orlando has joined #openstack-cinder | 10:57 | |
*** salv-orlando has quit IRC | 10:57 | |
*** salv-orlando has joined #openstack-cinder | 10:58 | |
*** salv-orlando has quit IRC | 10:58 | |
*** haomaiwa_ has quit IRC | 11:01 | |
*** 18VAAKXIZ has joined #openstack-cinder | 11:01 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Make Nova API version configurable Edit https://review.openstack.org/302177 | 11:12 |
*** houming has quit IRC | 11:15 | |
openstackgerrit | Danny Al-Gaaf proposed openstack/cinder: Fix doc and source documentation errors and warning https://review.openstack.org/293050 | 11:18 |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Make Nova API version configurable https://review.openstack.org/302177 | 11:20 |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: cinder-all is deprecated https://review.openstack.org/302189 | 11:29 |
*** ccesario has quit IRC | 11:30 | |
*** timcl has joined #openstack-cinder | 11:31 | |
*** mvk_ has joined #openstack-cinder | 11:38 | |
*** laughterwym has quit IRC | 11:46 | |
*** laughterwym has joined #openstack-cinder | 11:46 | |
*** laughter_ has joined #openstack-cinder | 11:49 | |
*** laughterwym has quit IRC | 11:51 | |
*** laughter_ has quit IRC | 11:54 | |
*** takedakn has joined #openstack-cinder | 11:59 | |
*** 18VAAKXIZ has quit IRC | 12:01 | |
*** haomaiwang has joined #openstack-cinder | 12:01 | |
*** al_indig_ has joined #openstack-cinder | 12:03 | |
*** timcl has quit IRC | 12:04 | |
*** al_indigo has quit IRC | 12:06 | |
openstackgerrit | venkatamahesh proposed openstack/cinder: Update the Administrator guide links https://review.openstack.org/302202 | 12:07 |
*** salv-orlando has joined #openstack-cinder | 12:10 | |
*** e0ne has quit IRC | 12:13 | |
*** raildo-afk is now known as raildo | 12:14 | |
*** markvoelker has joined #openstack-cinder | 12:15 | |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Deprecate nas_ip and change this to nas_host https://review.openstack.org/302207 | 12:16 |
*** e0ne has joined #openstack-cinder | 12:17 | |
*** iceyao has quit IRC | 12:18 | |
*** crose has joined #openstack-cinder | 12:20 | |
*** salv-orlando has quit IRC | 12:26 | |
*** gouthamr has joined #openstack-cinder | 12:27 | |
*** laughterwym has joined #openstack-cinder | 12:28 | |
*** andymaier has joined #openstack-cinder | 12:30 | |
*** laughterwym has quit IRC | 12:33 | |
*** julim has joined #openstack-cinder | 12:39 | |
openstackgerrit | Stefan Amann proposed openstack/cinder-specs: Add support for FICON attached ECKD volumes for Linux on System z https://review.openstack.org/302221 | 12:39 |
*** ociuhandu has quit IRC | 12:39 | |
*** jwcroppe has joined #openstack-cinder | 12:39 | |
*** jwcroppe has quit IRC | 12:44 | |
*** julim has quit IRC | 12:45 | |
*** akerr has joined #openstack-cinder | 12:48 | |
*** timcl has joined #openstack-cinder | 12:51 | |
smcginnis | Wow, FICON support in OpenStack. Crazy. | 12:52 |
DuncanT | What's FICON? | 12:52 |
smcginnis | If you don't know, you most likely don't care and not worth your time to find out. ;) | 12:53 |
smcginnis | I think mostly used with mainframes. | 12:53 |
smcginnis | A different flavor of fibre channel. | 12:53 |
*** jdillaman has joined #openstack-cinder | 12:53 | |
DuncanT | Blimey. Does somebody actually want to use that with Openstack? Does it need a CI? | 12:54 |
*** merooney has joined #openstack-cinder | 12:54 | |
smcginnis | Yes to the second question. | 12:54 |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Deprecate nas_ip and change this to nas_host https://review.openstack.org/302207 | 12:54 |
smcginnis | Not so sure on the first one, but if they are proposing it... maybe? | 12:54 |
*** bharathk has quit IRC | 12:54 | |
*** links has quit IRC | 12:56 | |
*** ccesario has joined #openstack-cinder | 12:58 | |
*** haomaiwang has quit IRC | 13:01 | |
*** akshai has joined #openstack-cinder | 13:01 | |
*** haomaiwang has joined #openstack-cinder | 13:01 | |
*** ociuhandu has joined #openstack-cinder | 13:02 | |
*** crose has quit IRC | 13:02 | |
*** smoriya_ has quit IRC | 13:03 | |
*** merooney has quit IRC | 13:03 | |
*** porrua has joined #openstack-cinder | 13:05 | |
*** rlrossit has joined #openstack-cinder | 13:07 | |
*** salv-orlando has joined #openstack-cinder | 13:09 | |
*** edmondsw has joined #openstack-cinder | 13:09 | |
*** rajoriaa_ has joined #openstack-cinder | 13:11 | |
*** shausy has quit IRC | 13:11 | |
*** rajoriaa__ has joined #openstack-cinder | 13:12 | |
*** ayush7rajoria has quit IRC | 13:12 | |
*** rajoriaa_ has quit IRC | 13:15 | |
*** Julien-zte has joined #openstack-cinder | 13:16 | |
*** mriedem has joined #openstack-cinder | 13:16 | |
*** martyt has joined #openstack-cinder | 13:17 | |
*** martyt has quit IRC | 13:17 | |
*** salv-orlando has quit IRC | 13:17 | |
openstackgerrit | Stefan Amann proposed openstack/cinder-specs: Add support for FICON attached ECKD volumes for Linux on System z https://review.openstack.org/302221 | 13:20 |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Changing of bad approach in using CONF. https://review.openstack.org/302249 | 13:21 |
*** ccesario has quit IRC | 13:22 | |
*** dustins has joined #openstack-cinder | 13:24 | |
*** baumann has joined #openstack-cinder | 13:27 | |
*** takedakn has quit IRC | 13:29 | |
*** merooney has joined #openstack-cinder | 13:29 | |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: cinder-all is deprecated https://review.openstack.org/302189 | 13:31 |
*** timcl has quit IRC | 13:32 | |
yuriy_n17 | DuncanT: Hi! Sorry for disturbing. Please review https://review.openstack.org/#/c/281511/ whenever you have free time. | 13:32 |
*** rajoriaa__ has quit IRC | 13:34 | |
*** ametts has joined #openstack-cinder | 13:35 | |
*** Poornima has quit IRC | 13:36 | |
*** mtanino has joined #openstack-cinder | 13:37 | |
*** timcl has joined #openstack-cinder | 13:38 | |
*** qeelee has quit IRC | 13:38 | |
*** merooney has quit IRC | 13:40 | |
*** diablo_rojo has joined #openstack-cinder | 13:41 | |
*** eharney has quit IRC | 13:41 | |
*** garthb has joined #openstack-cinder | 13:42 | |
*** esker has quit IRC | 13:42 | |
*** diogogmt has joined #openstack-cinder | 13:42 | |
*** iceyao has joined #openstack-cinder | 13:43 | |
*** atuvenie has quit IRC | 13:44 | |
*** avishay has quit IRC | 13:45 | |
*** ccesario has joined #openstack-cinder | 13:46 | |
*** merooney has joined #openstack-cinder | 13:47 | |
*** jgregor has joined #openstack-cinder | 13:49 | |
*** cbader has quit IRC | 13:50 | |
openstackgerrit | Abhishek Kekane proposed openstack/cinder: volume upload-to-image fails for iso disk-format https://review.openstack.org/287606 | 13:52 |
*** edtubill has joined #openstack-cinder | 13:53 | |
*** eharney has joined #openstack-cinder | 13:55 | |
*** diogogmt has quit IRC | 13:55 | |
*** diogogmt has joined #openstack-cinder | 13:56 | |
*** knikolla has joined #openstack-cinder | 13:57 | |
*** sage has quit IRC | 13:57 | |
*** sage has joined #openstack-cinder | 13:58 | |
*** EinstCrazy has joined #openstack-cinder | 13:59 | |
*** EinstCrazy has quit IRC | 13:59 | |
*** EinstCrazy has joined #openstack-cinder | 13:59 | |
*** mriedem is now known as mriedem_meeting | 14:00 | |
*** ccesario has quit IRC | 14:00 | |
openstackgerrit | Stefan Amann proposed openstack/cinder-specs: Add support for FICON attached ECKD volumes for Linux on System z https://review.openstack.org/302221 | 14:00 |
*** jwcroppe has joined #openstack-cinder | 14:00 | |
*** haomaiwang has quit IRC | 14:01 | |
*** haomaiwang has joined #openstack-cinder | 14:01 | |
*** avishay has joined #openstack-cinder | 14:04 | |
*** haomaiwang has quit IRC | 14:07 | |
*** mtanino has quit IRC | 14:08 | |
*** EinstCrazy has quit IRC | 14:09 | |
*** takedakn has joined #openstack-cinder | 14:10 | |
*** liverpooler has quit IRC | 14:10 | |
*** EinstCrazy has joined #openstack-cinder | 14:10 | |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: cinder-all is deprecated https://review.openstack.org/302189 | 14:11 |
*** eharney has quit IRC | 14:11 | |
*** takedakn has quit IRC | 14:11 | |
*** ccesario has joined #openstack-cinder | 14:13 | |
*** Julien-zte has quit IRC | 14:13 | |
*** avishay has quit IRC | 14:14 | |
*** cbader has joined #openstack-cinder | 14:20 | |
*** diogogmt has quit IRC | 14:23 | |
*** cknight has joined #openstack-cinder | 14:24 | |
*** sheel has quit IRC | 14:27 | |
*** diogogmt has joined #openstack-cinder | 14:28 | |
*** laughterwym has joined #openstack-cinder | 14:29 | |
*** dustins has quit IRC | 14:30 | |
*** eharney has joined #openstack-cinder | 14:30 | |
*** mriedem_meeting is now known as mriedem | 14:31 | |
*** arch-nemesis has joined #openstack-cinder | 14:32 | |
*** arch-nemesis has quit IRC | 14:32 | |
*** arch-nemesis has joined #openstack-cinder | 14:33 | |
*** nkrinner has quit IRC | 14:33 | |
*** dustins has joined #openstack-cinder | 14:34 | |
*** zul has quit IRC | 14:35 | |
*** ccesario has quit IRC | 14:37 | |
openstackgerrit | Stefan Amann proposed openstack/cinder-specs: Add support for FICON attached ECKD volumes for Linux on System z https://review.openstack.org/302221 | 14:38 |
*** eharney has quit IRC | 14:39 | |
*** laughterwym has quit IRC | 14:40 | |
*** eharney has joined #openstack-cinder | 14:40 | |
*** lprice1 has joined #openstack-cinder | 14:41 | |
*** laughterwym has joined #openstack-cinder | 14:41 | |
*** karthikp has joined #openstack-cinder | 14:41 | |
*** lprice has quit IRC | 14:42 | |
*** laughterwym has quit IRC | 14:48 | |
*** ericksonsantos has quit IRC | 14:48 | |
*** laughterwym has joined #openstack-cinder | 14:49 | |
*** raildo is now known as raildo-afk | 14:49 | |
*** raildo-afk is now known as raildo | 14:51 | |
*** salv-orlando has joined #openstack-cinder | 14:51 | |
*** mtanino has joined #openstack-cinder | 14:54 | |
scottda | e0ne: geguileo: Would you have time to look at cinderclient /v3 endpoint support before today's Cinder meeting? | 14:57 |
scottda | https://review.openstack.org/#/c/300028/5 | 14:57 |
scottda | I've put cinderclient reviews on the agenda. | 14:58 |
*** laughter_ has joined #openstack-cinder | 14:58 | |
e0ne | scottda: I would like to test it too | 14:59 |
*** eharney has quit IRC | 14:59 | |
scottda | e0ne: That would be great. I've also pushed up dependent patch for microversion support: | 14:59 |
scottda | https://review.openstack.org/#/c/301941/ | 15:00 |
*** zul has joined #openstack-cinder | 15:01 | |
*** laughterwym has quit IRC | 15:01 | |
*** jwcroppe has quit IRC | 15:01 | |
*** laughterwym has joined #openstack-cinder | 15:02 | |
*** laughter_ has quit IRC | 15:03 | |
openstackgerrit | Karthik Prabhu Vinod proposed openstack/cinder: Cleanup for Volume Object https://review.openstack.org/300708 | 15:06 |
*** julim has joined #openstack-cinder | 15:06 | |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Make cinder-all binary deprecated https://review.openstack.org/302189 | 15:10 |
*** mjura has quit IRC | 15:12 | |
*** avishay has joined #openstack-cinder | 15:13 | |
*** dustins_ has joined #openstack-cinder | 15:13 | |
*** eharney has joined #openstack-cinder | 15:14 | |
*** garthb has quit IRC | 15:14 | |
DuncanT | yuriy_n17: Regarding deprecating cinder-all - why? It's always been true that you can spread the services round, but one command to start them all if that's what you want to do is surely useful? | 15:14 |
*** dave-mccowan has joined #openstack-cinder | 15:15 | |
*** lcurtis has joined #openstack-cinder | 15:15 | |
*** dustins has quit IRC | 15:16 | |
*** belmoreira has quit IRC | 15:17 | |
e0ne | scottda: do we need to create new endpoint in keystone? | 15:19 |
yuriy_n17 | DuncanT: This command is hardly used and it will be removed in the next release. | 15:20 |
DuncanT | yuriy_n17: Not without more justification than that, it won't. Is it broken? Harmful? Or do you just not like it? | 15:21 |
*** merooney has quit IRC | 15:22 | |
*** salv-orlando has quit IRC | 15:22 | |
DuncanT | yuriy_n17: Removing it is a breakage of anybody who is using it - if there's a good reason to, then fine, but currently no reason has been given it the blueprint of the patch | 15:23 |
yuriy_n17 | DuncanT: We don't know whether it is used by anyone. | 15:23 |
*** bjornar has joined #openstack-cinder | 15:23 | |
*** mylu has joined #openstack-cinder | 15:24 | |
yuriy_n17 | DuncanT: This is not tested either. | 15:24 |
DuncanT | yuriy_n17: Ok, not tested is a justification. Can you add that to the commit message, please? | 15:25 |
*** avishay has quit IRC | 15:25 | |
yuriy_n17 | DuncanT: Sure | 15:25 |
DuncanT | yuriy_n17: Thanks. At least if somebody objects we've got some reason why we removed it logged | 15:26 |
*** sheel has joined #openstack-cinder | 15:26 | |
*** zul_ has joined #openstack-cinder | 15:27 | |
*** zul has quit IRC | 15:27 | |
*** esker has joined #openstack-cinder | 15:29 | |
*** dave-mcc_ has joined #openstack-cinder | 15:30 | |
*** merooney has joined #openstack-cinder | 15:30 | |
*** EinstCrazy has quit IRC | 15:31 | |
*** rajinir has joined #openstack-cinder | 15:32 | |
xinli | baumann: if I want to connet to a cinder database to remove a volume, I find two scenarios: mySQL -D cinder or mySQL -r root cinder what's the diff? | 15:32 |
*** dave-mccowan has quit IRC | 15:32 | |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Make cinder-all binary deprecated https://review.openstack.org/302189 | 15:33 |
*** avishay has joined #openstack-cinder | 15:34 | |
*** haplo37 has joined #openstack-cinder | 15:34 | |
*** ametts has quit IRC | 15:36 | |
*** crose has joined #openstack-cinder | 15:37 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Make Nova API version configurable https://review.openstack.org/302177 | 15:37 |
baumann | xinli: So -D will specify 'cinder' as the database name. -u will specify the username (root in this case), and then the database of 'cinder' is specified after. | 15:39 |
baumann | xinli: As far as I understand it anyways :) | 15:39 |
*** julim has quit IRC | 15:43 | |
*** jdurgin1 has joined #openstack-cinder | 15:44 | |
openstackgerrit | Eric Harney proposed openstack/cinder: Make cinder-all binary deprecated https://review.openstack.org/302189 | 15:45 |
*** p01nt3r75 has joined #openstack-cinder | 15:46 | |
scottda | e0ne: Yes, you need to create a new endpoint in Keystone.... | 15:47 |
scottda | ameade: Has a patch up to add /v3 to devstack...looking.... | 15:47 |
e0ne | scottda: do you have patch to devstack for it? | 15:47 |
e0ne | :) | 15:47 |
scottda | https://review.openstack.org/#/c/300585 | 15:48 |
smcginnis | scottda: Wasn't there a request from keystone not to add version specific endpoints anymore? | 15:48 |
ameade | what's the alternative? | 15:48 |
e0ne | scottda: cool! thank you | 15:48 |
scottda | smcginnis: I'm not sure whom it was actually from, but there was pushback on adding a new endpoint. | 15:48 |
ameade | besides not having version in the url | 15:48 |
smcginnis | I think the idea was one endpoint for the service, then... maybe discover version from there? | 15:49 |
smcginnis | Honestly not sure. | 15:49 |
ameade | I feel like i've stood next to a conversation about this lol | 15:49 |
scottda | Well, I believe one goal was not to have a proliferation of endpoints in the service catalogue... | 15:49 |
*** avishay has quit IRC | 15:49 | |
scottda | But, we should never have to add another, since we can now version in the HTTP header. | 15:50 |
*** andymaier has quit IRC | 15:50 | |
scottda | Speaking of which, Cindeclient had version discovery, but we backed it out during Kilo due to a problem with proxies... | 15:50 |
scottda | I'm not sure we ever got that back into the client/ | 15:50 |
scottda | ? | 15:50 |
smcginnis | scottda: No, it didn't. | 15:51 |
*** iceyao has quit IRC | 15:51 | |
smcginnis | Would be nice. Not sure what all the issues were there. I think something to do with proxies and NAT'ing of the IP address or something. | 15:51 |
scottda | smcginnis: Do you know if we've a bug filed? I'd like to track it, and perhaps work on it. | 15:51 |
e0ne | scottda: IMO, it' would be a good feature if we'll fix issues with proxies | 15:51 |
smcginnis | scottda: Good point about now with microversions. Shouldn't ever have to add another endpoint. | 15:52 |
smcginnis | scottda: Not that I've seen. Maybe thingee knows? | 15:52 |
*** adrianofr_ has joined #openstack-cinder | 15:52 | |
* thingee reads scrollback | 15:52 | |
scottda | or DuncanT ...I know we discovered the problem in our public cloud. Actually mordred discovered it. | 15:52 |
*** salv-orlando has joined #openstack-cinder | 15:53 | |
*** merooney has quit IRC | 15:53 | |
scottda | thingee: Is there a bug filed to re-enable automatic version discovery in the cinderclient? | 15:53 |
scottda | I'm sensing we could have a Summit topic on this.... | 15:53 |
*** ccesario has joined #openstack-cinder | 15:53 | |
thingee | scottda: not sure. | 15:53 |
DuncanT | Oh, yeah, that was the issue where it didn't have some header you could set to re-write the URLs to point at proxies | 15:53 |
thingee | do we really need a summit topic? it was merged in the past | 15:53 |
scottda | thingee: maybe not. I wasn't sure if there were other issues. | 15:54 |
scottda | around endpoints. | 15:54 |
thingee | in the past the issue was because of ha proxy cases and ssl termination | 15:54 |
DuncanT | I don't think we need a summit session; the issue is well understood (other than the fact it needs to be configurable and default to off, for security reasons) and the code is in several otehr projects | 15:54 |
thingee | we eliminated this problem in Kilo i think with a config option | 15:54 |
thingee | however, expecting deployments to be using kilo was not ideal | 15:55 |
thingee | so we reverted | 15:55 |
smcginnis | Maybe well understood, but it might be nice to capture that understanding in a bug report so others can understand the full issue. | 15:55 |
openstackgerrit | Christopher J Schaefer proposed openstack/python-cinderclient: Adding tox support for bandit https://review.openstack.org/296082 | 15:55 |
*** jseiler has joined #openstack-cinder | 15:55 | |
scottda | thingee: That's right. So re-introducing in the client would have to either be configurable or discoverable based on server version (We can do this now with /v3) | 15:55 |
smcginnis | thingee: Oh, so post kilo now we should be able to reimplement that change? | 15:55 |
thingee | smcginnis: I think i did capture it many times, and even explained in the past midcycles, but people had a hard time to follow along | 15:55 |
thingee | DuncanT's only comment was "well glad you're looking at it" | 15:56 |
DuncanT | Ok. I'll see if I can find the bug report or review that talked about it in depth | 15:56 |
hogepodge | smcginnis: Do you have any thoughts on this review? Particularly eglute's comments and inclusion of extend as an advisory capability. https://review.openstack.org/#/c/290271/4 | 15:56 |
smcginnis | thingee: Hah, probably true. Guess I should look through those logs. | 15:56 |
thingee | smcginnis: if we were to readd it, the assumption is people are mostly on kilo. | 15:56 |
*** salv-orlando has quit IRC | 15:56 | |
scottda | DuncanT: OK. Keep me in the loop. I've been in that code a lot lately and can have a look if there are any fixes needed. | 15:56 |
openstackgerrit | Ryan Rossiter proposed openstack/cinder: Add test for checking object compatibilities https://review.openstack.org/259123 | 15:56 |
thingee | otherwise the client won't work for those still on juno | 15:56 |
*** garthb has joined #openstack-cinder | 15:56 | |
smcginnis | hogepodge: I'll take a look later. Got it open in a tab now. | 15:56 |
thingee | and we couldn't backport just that fix to juno for some reason | 15:56 |
hogepodge | anyone else on the cinder team, feel free to weigh in. We're scoring capabilities for the next defcore round. | 15:57 |
hogepodge | smcginnis: thanks | 15:57 |
DuncanT | X-Forwarded-Host is the standard header. Looks like we might have hard-coded it, which is actually a security risk according to what I was reading (and it made sense why at the time) | 15:57 |
*** xyang1 has joined #openstack-cinder | 15:57 | |
smcginnis | Junp will be OEL now, so maybe we're good. | 15:57 |
DuncanT | vi common. | 15:57 |
DuncanT | Bah | 15:57 |
*** arecknag has quit IRC | 15:57 | |
smcginnis | :) | 15:57 |
*** salv-orlando has joined #openstack-cinder | 15:58 | |
*** avishay has joined #openstack-cinder | 15:58 | |
DuncanT | We couldn't rely on a backport since the client couldn't know if the fix had been deloyed or not | 15:59 |
*** akerr is now known as akerr_away | 16:00 | |
DuncanT | So it looks like we don't have a version discovery issue as long as the proxy or ssl termination sets X-Forwarded-Host | 16:00 |
DuncanT | But we might have a minor security issue, I'll try to dig out references | 16:00 |
*** karthikp has quit IRC | 16:03 | |
*** apoorvad has joined #openstack-cinder | 16:06 | |
*** erlon has joined #openstack-cinder | 16:07 | |
*** laughterwym has quit IRC | 16:07 | |
*** laughterwym has joined #openstack-cinder | 16:08 | |
*** jistr has quit IRC | 16:10 | |
*** laughterwym has quit IRC | 16:12 | |
openstackgerrit | Nate Potter proposed openstack/cinder-specs: Backend Usage Reporting https://review.openstack.org/301946 | 16:14 |
*** mylu has quit IRC | 16:14 | |
*** leeantho has joined #openstack-cinder | 16:15 | |
*** timcl has quit IRC | 16:16 | |
*** leeantho_ has joined #openstack-cinder | 16:16 | |
*** dflorea has joined #openstack-cinder | 16:17 | |
*** jwcroppe has joined #openstack-cinder | 16:18 | |
*** jwcroppe has quit IRC | 16:18 | |
*** jwcroppe has joined #openstack-cinder | 16:18 | |
*** thrawn01_ is now known as thrawn01 | 16:18 | |
*** dflorea has quit IRC | 16:18 | |
*** avishay has quit IRC | 16:19 | |
*** leeantho has quit IRC | 16:19 | |
*** merooney has joined #openstack-cinder | 16:20 | |
*** mylu has joined #openstack-cinder | 16:20 | |
*** dflorea has joined #openstack-cinder | 16:24 | |
*** sgotliv has quit IRC | 16:26 | |
*** goretoxo has quit IRC | 16:27 | |
*** jistr has joined #openstack-cinder | 16:28 | |
*** jwcroppe has quit IRC | 16:30 | |
*** merooney has quit IRC | 16:31 | |
*** jistr is now known as jistr|off | 16:34 | |
*** zul_ has quit IRC | 16:35 | |
*** jordanP has quit IRC | 16:36 | |
*** akerr_away is now known as akerr | 16:37 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Make Nova API version configurable https://review.openstack.org/302177 | 16:37 |
*** timcl has joined #openstack-cinder | 16:42 | |
*** merooney has joined #openstack-cinder | 16:44 | |
ntpttr | e0ne: ping | 16:45 |
e0ne | ntpttr: hi | 16:45 |
ntpttr | e0ne: hey, so in you comment on my usage API you mentioned that the XML API is going to be removed https://review.openstack.org/#/c/301444/5/cinder/api/contrib/usage.py | 16:46 |
*** ChubYann has joined #openstack-cinder | 16:46 | |
e0ne | ntpttr: I did | 16:46 |
ntpttr | I tried taking it out but when I do running the command gives me a 404, when with it it works | 16:46 |
ntpttr | is there something I should be replacing it with? | 16:46 |
e0ne | ntpttr: JSON REST API | 16:46 |
e0ne | ntpttr: here is a patch with removal https://review.openstack.org/231663 | 16:47 |
ntpttr | e0ne: okay, thanks | 16:47 |
*** zul has joined #openstack-cinder | 16:51 | |
*** ndipanov has quit IRC | 16:52 | |
*** mylu has quit IRC | 16:54 | |
*** dflorea has quit IRC | 16:54 | |
*** zhangjn has quit IRC | 16:57 | |
*** zhangjn has joined #openstack-cinder | 16:58 | |
* sheel looking for review on https://review.openstack.org/#/c/278605/32 and https://review.openstack.org/#/c/273938/ | 16:59 | |
*** bjornar has quit IRC | 16:59 | |
*** bjornar has joined #openstack-cinder | 17:00 | |
*** dflorea has joined #openstack-cinder | 17:00 | |
openstackgerrit | Eric Harney proposed openstack/cinder: Make cinder-all binary deprecated https://review.openstack.org/302189 | 17:00 |
*** esker has quit IRC | 17:05 | |
*** esker has joined #openstack-cinder | 17:06 | |
*** ChubYann has quit IRC | 17:08 | |
*** salv-orlando has quit IRC | 17:12 | |
*** e0ne has quit IRC | 17:17 | |
*** p01nt3r75 has quit IRC | 17:17 | |
*** raunak_ has joined #openstack-cinder | 17:18 | |
*** akerr has quit IRC | 17:18 | |
openstackgerrit | Nate Potter proposed openstack/cinder: Implement backend usage reporting https://review.openstack.org/301444 | 17:19 |
*** merooney has quit IRC | 17:20 | |
*** akerr has joined #openstack-cinder | 17:20 | |
*** dflorea has quit IRC | 17:20 | |
*** ChubYann has joined #openstack-cinder | 17:20 | |
sjackson | Hello all. I have Juno using ceph as a cinder backend. The ceph cluster has some issues and I need to rebuild it. My first thought is to add secondary cinder storage (another ceph cluster or normal cinder) and migrate the data. Was wondering if anyone knew if it was possible to add a 2nd cinder/ceph cluster, and if os, it it possible to migrate 100% of my data off the old cluster? | 17:21 |
DuncanT | sjackson: I don't know the status of volume migration in Juno, but it is certainly possible in Kilo onwards. I think it is ok - should be easy enough to spin up a ceph based devstack and test it | 17:24 |
*** dflorea has joined #openstack-cinder | 17:25 | |
*** martyturner has joined #openstack-cinder | 17:25 | |
*** jwcroppe has joined #openstack-cinder | 17:25 | |
openstackgerrit | Nate Potter proposed openstack/cinder-specs: Backend Usage Reporting https://review.openstack.org/301946 | 17:26 |
*** mylu has joined #openstack-cinder | 17:32 | |
*** esker has quit IRC | 17:39 | |
*** crose has quit IRC | 17:40 | |
*** david-lyle has quit IRC | 17:44 | |
*** lpetrut has quit IRC | 17:49 | |
openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: WIP: Deprecating XML config file https://review.openstack.org/294076 | 17:50 |
openstackgerrit | Nate Potter proposed openstack/python-cinderclient: Implement cinder free command https://review.openstack.org/301445 | 17:51 |
openstackgerrit | Nate Potter proposed openstack/python-cinderclient: Implement cinder usage command https://review.openstack.org/301445 | 17:51 |
*** vgridnev has joined #openstack-cinder | 17:53 | |
*** dflorea has quit IRC | 17:55 | |
*** rcernin has quit IRC | 17:56 | |
*** mriedem has quit IRC | 17:56 | |
*** xyang1 has quit IRC | 17:58 | |
*** dflorea has joined #openstack-cinder | 17:58 | |
*** david-lyle has joined #openstack-cinder | 18:02 | |
*** e0ne has joined #openstack-cinder | 18:04 | |
*** ChubYann has quit IRC | 18:05 | |
* hemna is trying different 'uml' editors | 18:07 | |
hemna | all so unsatisfying | 18:07 |
smcginnis | hemna: Yep. | 18:07 |
smcginnis | hemna: Did you try plantuml? | 18:07 |
smcginnis | I had done the same thing and ended up there. | 18:07 |
hemna | not yet | 18:07 |
hemna | I really liked gliffy | 18:08 |
smcginnis | What's the issue there? Not free? | 18:08 |
hemna | not free :( | 18:08 |
hemna | 14 day trial | 18:08 |
smcginnis | Bleh | 18:08 |
hemna | looks like most of the desktop editors like that also don't export to something readable | 18:08 |
smcginnis | What's this GUI stuff? You're supposed to be a hard core command line guy. :) | 18:09 |
hemna | argouml...junk | 18:09 |
hemna | umlet....junk | 18:09 |
hemna | evoluspencil....junk | 18:09 |
*** karthikp has joined #openstack-cinder | 18:09 | |
smcginnis | evoluspencil? I would kill that one just for the name. | 18:09 |
hemna | hehe yah | 18:10 |
smcginnis | Argo looked promising to me at first, but IIRC it's way out of date. | 18:10 |
hemna | I really like the gliffy editor and it outputs nice images | 18:10 |
hemna | https://www.gliffy.com/go/publish/image/10360157/L.png | 18:10 |
hemna | I just tried Argo | 18:10 |
hemna | the screens look good | 18:10 |
hemna | just tried to use the interface and it was awful | 18:10 |
smcginnis | StarUML was OK-ish. | 18:10 |
hemna | downloading yEd at the moment... | 18:11 |
*** mriedem has joined #openstack-cinder | 18:11 | |
scottda | hemna: I liked the look of plantuml | 18:11 |
hemna | ok ok | 18:11 |
hemna | I'll try pulling that down | 18:11 |
hemna | I didn't care for the look of the screens on it at all. | 18:11 |
smcginnis | Hehe, we're breaking you down. | 18:11 |
hemna | https://www.gliffy.com/go/publish/image/10360157/L.png | 18:12 |
hemna | that's what I want! | 18:12 |
hemna | but free | 18:12 |
hemna | that is way readable imho. | 18:12 |
smcginnis | Should be able to get that. | 18:12 |
*** p01nt3r75 has joined #openstack-cinder | 18:12 | |
* hemna is platuml'ing | 18:12 | |
scottda | hemna: I want you to like plantuml so I can start using it. | 18:13 |
smcginnis | Although that is kind of a mix of a UML diagram, a sequence diagram, and a workflow diagram. | 18:13 |
hemna | hehe | 18:13 |
smcginnis | scottda: Hah! | 18:13 |
*** e0ne has quit IRC | 18:14 | |
smcginnis | Ooh, kist noticed staruml is updated and now supports markdown. | 18:14 |
smcginnis | http://staruml.io/ | 18:14 |
hemna | crap | 18:14 |
hemna | ok, well I created a simple ass.txt | 18:15 |
hemna | and platuml didn't see it | 18:15 |
smcginnis | I think it would be ideal if we can store in-repo as reviewable text. | 18:15 |
hemna | :( so I can't even run it against my sample txt file | 18:15 |
smcginnis | hemna: What do you mean by it didn't see it? | 18:15 |
hemna | java -jar platuml.jar | 18:15 |
hemna | dialog comes up to select a file to run | 18:15 |
hemna | doesn't see my file | 18:15 |
smcginnis | There are some online editors. Maybe try that? http://www.planttext.com/ | 18:17 |
*** ChubYann has joined #openstack-cinder | 18:17 | |
hemna | ahh ok | 18:17 |
hemna | yah the desktop one can't even see my files on disk | 18:17 |
smcginnis | Or this would make it easy to share: https://sites.google.com/site/plantumlgizmo/home | 18:17 |
hemna | strange | 18:17 |
hemna | frigging java | 18:17 |
smcginnis | +1 | 18:17 |
erlon | hemna: if you only are using it for drawing, you can try 'draw.io' | 18:18 |
hemna | yah tried that... sadness | 18:18 |
smcginnis | hemna: Can you paste your ass.txt content? | 18:19 |
scottda | That does not parse well | 18:20 |
hemna | http://goo.gl/pNf18A | 18:20 |
smcginnis | scottda: hehe | 18:21 |
smcginnis | hemna: I think you are missing some content. @startuml is needed I think. | 18:21 |
*** karthikp has quit IRC | 18:21 | |
smcginnis | @startuml/@enduml | 18:22 |
*** timcl1 has joined #openstack-cinder | 18:23 | |
hemna | ok let me spend a bit of time on this language | 18:23 |
hemna | I need to create something that looks like https://www.gliffy.com/go/publish/image/10360157/L.png | 18:23 |
hemna | if it can do it.... | 18:23 |
DuncanT | hemna: Can you pastebin the text version of that in gliffy? | 18:24 |
*** dustins_ is now known as dustins | 18:24 | |
*** e0ne has joined #openstack-cinder | 18:24 | |
hemna | DuncanT, there is a way to collaborate on the gliffy site | 18:24 |
hemna | DuncanT, I can invite you to edit the gliffy version | 18:24 |
*** ociuhandu has quit IRC | 18:24 | |
smcginnis | hemna: ass.txt worked for me locally with the addition of the @startuml and @enduml tags, FWIW. | 18:24 |
DuncanT | hemna: please do - duncan.thomas@gmail.com | 18:24 |
hemna | DuncanT, sent | 18:25 |
hemna | not sure what's wrong the platuml.jar refuses to see any files | 18:25 |
*** timcl has quit IRC | 18:25 | |
hemna | whoa | 18:26 |
hemna | there we go | 18:26 |
hemna | smcginnis, that was the magic sauce. I had to have both of those tags in there. | 18:26 |
hemna | lame | 18:26 |
hemna | ok I have something to hack on now....bbiab. | 18:26 |
*** sgotliv has joined #openstack-cinder | 18:26 | |
*** dflorea has quit IRC | 18:27 | |
hemna | man this language is complicated | 18:28 |
hemna | to do something simple | 18:28 |
hemna | :( | 18:28 |
* hemna is a complainer | 18:28 | |
*** martyturner has quit IRC | 18:29 | |
*** dflorea has joined #openstack-cinder | 18:30 | |
*** jdurgin1 has quit IRC | 18:32 | |
*** dflorea has quit IRC | 18:35 | |
*** dflorea has joined #openstack-cinder | 18:35 | |
*** mvk_ has quit IRC | 18:35 | |
*** martyturner has joined #openstack-cinder | 18:38 | |
*** jwcroppe has quit IRC | 18:39 | |
*** jgregor has quit IRC | 18:42 | |
*** timcl1 has quit IRC | 18:46 | |
*** e0ne has quit IRC | 18:46 | |
*** lpetrut has joined #openstack-cinder | 18:47 | |
*** timcl has joined #openstack-cinder | 18:47 | |
*** rlrossit has quit IRC | 18:48 | |
openstackgerrit | Karthik Prabhu Vinod proposed openstack/cinder: Cleanup for Volume Object https://review.openstack.org/300708 | 18:50 |
*** rlrossit has joined #openstack-cinder | 18:53 | |
*** dflorea has quit IRC | 18:54 | |
*** ociuhandu has joined #openstack-cinder | 18:56 | |
*** mylu has quit IRC | 18:59 | |
*** dflorea_ has joined #openstack-cinder | 18:59 | |
*** vgridnev has quit IRC | 19:00 | |
*** mylu has joined #openstack-cinder | 19:01 | |
*** vgridnev has joined #openstack-cinder | 19:01 | |
*** vgridnev has quit IRC | 19:02 | |
*** martyturner has quit IRC | 19:05 | |
*** timcl has quit IRC | 19:05 | |
*** xyang1 has joined #openstack-cinder | 19:06 | |
*** martyturner has joined #openstack-cinder | 19:06 | |
*** timcl has joined #openstack-cinder | 19:07 | |
*** Yogi1 has joined #openstack-cinder | 19:10 | |
*** Yogi1 has quit IRC | 19:10 | |
*** alkhodos has joined #openstack-cinder | 19:17 | |
*** sheel has quit IRC | 19:17 | |
*** e0ne has joined #openstack-cinder | 19:20 | |
*** dflorea_ has quit IRC | 19:24 | |
*** dflorea has joined #openstack-cinder | 19:25 | |
*** diablo_rojo has quit IRC | 19:25 | |
*** dflorea_ has joined #openstack-cinder | 19:29 | |
*** dflorea has quit IRC | 19:29 | |
*** esker has joined #openstack-cinder | 19:30 | |
*** ametts has joined #openstack-cinder | 19:31 | |
*** mvk_ has joined #openstack-cinder | 19:31 | |
*** arch-nemesis has quit IRC | 19:35 | |
*** esker has quit IRC | 19:35 | |
*** esker has joined #openstack-cinder | 19:36 | |
*** mylu has quit IRC | 19:36 | |
*** salv-orlando has joined #openstack-cinder | 19:37 | |
*** dflorea_ has quit IRC | 19:40 | |
*** diablo_rojo has joined #openstack-cinder | 19:42 | |
*** salv-orlando has quit IRC | 19:44 | |
*** mylu has joined #openstack-cinder | 19:45 | |
*** aswadr_ has quit IRC | 19:50 | |
*** e0ne has quit IRC | 19:50 | |
*** salv-orlando has joined #openstack-cinder | 19:54 | |
*** salv-orlando has quit IRC | 19:54 | |
*** salv-orlando has joined #openstack-cinder | 19:55 | |
*** eharney has quit IRC | 19:55 | |
*** julim has joined #openstack-cinder | 19:57 | |
*** timcl has quit IRC | 19:59 | |
*** mylu has quit IRC | 19:59 | |
*** adrianofr_ has quit IRC | 20:00 | |
*** mkrcmari__ has joined #openstack-cinder | 20:03 | |
*** mgagne_ is now known as mgagne | 20:04 | |
*** cbader has quit IRC | 20:05 | |
*** kmartin has quit IRC | 20:06 | |
*** leeantho_ has quit IRC | 20:06 | |
*** garthb has quit IRC | 20:06 | |
*** asselin_ has quit IRC | 20:06 | |
*** mdenny has quit IRC | 20:06 | |
*** fernnest has quit IRC | 20:06 | |
*** mvk_ has quit IRC | 20:06 | |
*** ebalduf_ has joined #openstack-cinder | 20:06 | |
*** markstur has joined #openstack-cinder | 20:07 | |
*** fernnest has joined #openstack-cinder | 20:07 | |
*** cbader has joined #openstack-cinder | 20:07 | |
*** kmartin has joined #openstack-cinder | 20:07 | |
*** leeantho has joined #openstack-cinder | 20:07 | |
*** garthb has joined #openstack-cinder | 20:07 | |
*** mdenny has joined #openstack-cinder | 20:07 | |
*** asselin has joined #openstack-cinder | 20:07 | |
*** jgregor has joined #openstack-cinder | 20:08 | |
*** ebalduf_ has quit IRC | 20:08 | |
*** eharney has joined #openstack-cinder | 20:09 | |
*** rcernin has joined #openstack-cinder | 20:09 | |
*** akerr has quit IRC | 20:10 | |
hemna | ok I give up on plantuml | 20:18 |
hemna | the syntax is rediculously complex for trying to build the same diagram | 20:18 |
Swanson | xyang1, Tiramisu question. Is there a reason we don't just use volume types for this? Enable/disable/failover volume_type? | 20:19 |
hemna | doing simple things like making the diagram be in 2 columns is hard | 20:19 |
* hemna quits | 20:19 | |
*** e0ne has joined #openstack-cinder | 20:22 | |
*** mylu has joined #openstack-cinder | 20:27 | |
*** e0ne has quit IRC | 20:28 | |
hemna | yEd almost was good enough | 20:28 |
hemna | couldn't do arrows between boxes though. | 20:28 |
hemna | *sigh* | 20:28 |
*** dflorea has joined #openstack-cinder | 20:28 | |
*** cknight has quit IRC | 20:30 | |
*** karthikp has joined #openstack-cinder | 20:30 | |
karthikp | tbarron, I am getting jenkins -1 when I updated line 662 https://review.openstack.org/#/c/300708/3/cinder/volume/api.py@662 | 20:32 |
*** dflorea has quit IRC | 20:33 | |
tbarron | karthikp: I saw that. Just tried to repro locally with your patch set and I don't hit that. | 20:33 |
karthikp | Me too ..for me tox -e py27 passes | 20:34 |
tbarron | karthikp: puzzling failure. | 20:34 |
tbarron | karthikp: try reverting that line and let's see what happens. | 20:34 |
karthikp | tbarron, shall i revert line 662 and submit another patchset? | 20:34 |
tbarron | karthikp: I would, we might learn something at least. | 20:34 |
karthikp | tbarron, or should I try with running tempest locally? | 20:35 |
tbarron | karthikp: hold on, i thought it was a unit test failure. Looking ... | 20:35 |
*** dustins has quit IRC | 20:38 | |
*** Lee1092 has quit IRC | 20:42 | |
*** gouthamr has quit IRC | 20:45 | |
smcginnis | Seems like there have been a lot of failures lately. Might be some gremlins in the gate. | 20:46 |
openstackgerrit | Saverio Proto proposed openstack/cinder: RBD: delete snapshots if missing in the backend https://review.openstack.org/300433 | 20:46 |
hemna | smcginnis, dia seems to be able to do it. | 20:47 |
hemna | and it's free | 20:47 |
smcginnis | hemna: Sounds familiar. | 20:47 |
jgriffith | hemna: http://plantuml.com/sequence.html#Skinparam | 20:48 |
jgriffith | hemna: why not just make a proper sequence diagram? | 20:49 |
jgriffith | Nova classes on the left, cinder classes on the left | 20:49 |
jgriffith | or vice versa | 20:49 |
*** mvk has joined #openstack-cinder | 20:50 | |
jgriffith | hemna: but having it start with "user/nova volume-attach xxxx, yyyy /dev/vdb" and sequence through? | 20:50 |
jgriffith | I thought that's what you were trying to do? | 20:50 |
hemna | I find them hard to follow | 20:50 |
hemna | but it's probably 'better' | 20:50 |
hemna | dunno | 20:50 |
jgriffith | hemna: you find sequence diagrams hard to follow? | 20:51 |
*** salv-orl_ has joined #openstack-cinder | 20:51 | |
tbarron | karthikp: i'm interested in the unit test failures, which seem bogus on the face of it. so if you wouldn't mind submitting again with that one line backed out we can see if the issue tracs with that change. | 20:51 |
*** mkrcmari__ has quit IRC | 20:52 | |
hemna | that example, yes. maybe I'm dumb. | 20:52 |
hemna | not sure how to combine 2 sequences into the same diagram | 20:52 |
jgriffith | hemna: well you don't | 20:52 |
hemna | as we have flows for nova calls, and cinder calls intermixed. | 20:52 |
jgriffith | hemna: and we don't | 20:52 |
hemna | https://www.gliffy.com/go/publish/image/10360157/L.png | 20:52 |
jgriffith | hemna: it's all sequential either way.. hence "sequence" diagram | 20:52 |
hemna | that just seems easy to follow to me. | 20:52 |
karthikp | tbarron: doing that right now | 20:53 |
jgriffith | hemna: yeah, honestly I don't know what you're trying to show me there other than a pretty picture | 20:53 |
hemna | ok, then I'll just stop and someone else can do this | 20:53 |
hemna | F it | 20:53 |
jgriffith | hemna: now now | 20:53 |
jgriffith | hemna: no need for that attitude | 20:53 |
jgriffith | I wasn't even criticizing, just offering a suggestion when you said you couldn't get it to work | 20:54 |
jgriffith | hemna: if you don't like the tool or seq diagram that's totally cool | 20:54 |
scottda | I think that diagrams are good for getting everyone on the same page about what info needs to be passed around, where it is generated, where it is stored, etc. | 20:55 |
*** salv-orlando has quit IRC | 20:55 | |
scottda | It seems we go round-and-round about implementation details, but we would benefit from a big picture view. | 20:55 |
jgriffith | scottda: I agree with you WRT to diagrams being useful | 20:55 |
*** lpetrut has quit IRC | 20:55 | |
jgriffith | scottda: I wasn't arguing anything... except that using a closed-source tool that costs money probably ain't gonna fly | 20:56 |
*** lpetrut has joined #openstack-cinder | 20:56 | |
*** porrua has quit IRC | 20:56 | |
jgriffith | scottda: sorry if that offends anyone | 20:56 |
scottda | I agree with that part about wanting free+open_source tool | 20:56 |
hemna | yup | 20:56 |
hemna | that's why I spend the last few hours looking for something that actually worked | 20:56 |
scottda | yeah! we agree. #end-meeting | 20:56 |
*** dflorea has joined #openstack-cinder | 20:56 | |
eharney | as a general heads up... i'm looking at some test failures and suspect that a mock 2.0.0 release today broke this thing: https://github.com/openstack/cinder/blob/master/cinder/test.py#L104 | 20:56 |
hemna | but plantuml is a pos | 20:56 |
*** knikolla has quit IRC | 20:57 | |
hemna | so if someone else wants to do it in plantuml, go for it | 20:57 |
hemna | I'm over it | 20:57 |
hemna | dia seems to be able to produce the same output as the gliffy tool | 20:57 |
tbarron | karthikp: look at ^^^^ eharney's remark a couple lines up. | 20:57 |
hemna | but it's not a sequence diagram, so whatever | 20:57 |
smcginnis | eharney: That could explain some things. | 20:57 |
hemna | someone else can do it. | 20:57 |
tbarron | karthikp: if we 'rm -rf .tox' we'll likely see the issue locally. | 20:58 |
eharney | smcginnis: this method of detection looks pretty fishy to me in general, we may need to turn it off unless someone has a better idea for how to do it | 20:58 |
smcginnis | eharney: Yeah _patch_mock just sounds a little odd. :) | 20:58 |
eharney | smcginnis: i'll go ahead and start a patch to just remove it so we don't grind to a halt in the gate | 20:59 |
openstackgerrit | Karthik Prabhu Vinod proposed openstack/cinder: Cleanup for Volume Object https://review.openstack.org/300708 | 21:00 |
tbarron | karthikp: I did that and now it fails. | 21:02 |
*** raildo is now known as raildo-afk | 21:02 | |
openstackgerrit | Nate Potter proposed openstack/python-cinderclient: Implement cinder backend-usage command https://review.openstack.org/301445 | 21:02 |
tbarron | karthikp: now I'll back out that one line and see if it still fails. | 21:02 |
tbarron | karthikp: it still fails, that line has nothing to do with it. mock 2.0 just got released at the same time that you made that update. | 21:04 |
*** jistr|off has quit IRC | 21:04 | |
openstackgerrit | Eric Harney proposed openstack/cinder: Remove "patch mock to raise for invalid assert calls" https://review.openstack.org/302439 | 21:05 |
eharney | smcginnis: that will hopefully keep things rolling ^ | 21:06 |
smcginnis | eharney: Awesome, thanks! | 21:06 |
eharney | smcginnis: whether we should try doing this check in some other way, i'm not too sure. i'm inclined to just drop it, curious what others think | 21:06 |
ntpttr | I was just about to ask about this mock stuff but I see it's a hot topic right now :) | 21:07 |
* ntpttr reads scrollback | 21:07 | |
*** mvk_ has joined #openstack-cinder | 21:08 | |
karthikp | tbarron, Sorry I just read message... I reverted back the line...Thanks. I believe the change was futile | 21:08 |
smcginnis | eharney: Yeah, not sure if there's a good answer there. | 21:08 |
tbarron | karthikp: well I suggested you do it :-) | 21:08 |
tbarron | karthikp: after eharney's patch merges you can submit again with that line :-) | 21:09 |
tbarron | karthikp: then we can see what breaks next ... | 21:09 |
karthikp | tbarron, sure thing | 21:10 |
*** mvk has quit IRC | 21:10 | |
*** knikolla has joined #openstack-cinder | 21:13 | |
*** julim has quit IRC | 21:14 | |
openstackgerrit | Kendall Nelson proposed openstack/os-brick: Split out factory method https://review.openstack.org/302445 | 21:24 |
*** sdague has quit IRC | 21:24 | |
*** mkrcmari__ has joined #openstack-cinder | 21:25 | |
*** rcernin has quit IRC | 21:27 | |
*** jwcroppe has joined #openstack-cinder | 21:28 | |
*** knikolla has quit IRC | 21:28 | |
*** mvk_ has quit IRC | 21:28 | |
*** knikolla has joined #openstack-cinder | 21:29 | |
*** madskier has joined #openstack-cinder | 21:31 | |
*** baumann has quit IRC | 21:32 | |
openstackgerrit | Jacob Gregor proposed openstack/cinder: Storwize: Multiple management IP not raising exceptions https://review.openstack.org/294268 | 21:33 |
*** jwcroppe has quit IRC | 21:33 | |
*** knikolla has quit IRC | 21:36 | |
*** jwcroppe has joined #openstack-cinder | 21:36 | |
*** knikolla has joined #openstack-cinder | 21:37 | |
*** dflorea has quit IRC | 21:40 | |
*** madskier has quit IRC | 21:41 | |
*** rlrossit has quit IRC | 21:44 | |
*** dflorea has joined #openstack-cinder | 21:45 | |
*** dflorea has quit IRC | 21:46 | |
*** jgregor has quit IRC | 21:47 | |
*** dflorea has joined #openstack-cinder | 21:47 | |
*** jgregor has joined #openstack-cinder | 21:49 | |
*** knikolla has quit IRC | 21:53 | |
*** jgregor has quit IRC | 21:55 | |
openstackgerrit | Eric Harney proposed openstack/cinder: Remove "patch mock to raise for invalid assert calls" https://review.openstack.org/302439 | 22:01 |
*** lpetrut has quit IRC | 22:04 | |
*** mriedem has quit IRC | 22:05 | |
*** jgregor has joined #openstack-cinder | 22:05 | |
*** diablo_rojo has left #openstack-cinder | 22:08 | |
*** dustins has joined #openstack-cinder | 22:08 | |
jgriffith | scottda: thanks for hosting | 22:10 |
jgriffith | hemna: I look forward to seeing a WIP with the implementation you're proposing | 22:10 |
jgriffith | hemna: if you're doing anything more than taking my patch and moving the create-attachment entry to reserve volume I'd be interested in hearing more. | 22:12 |
*** dflorea has quit IRC | 22:12 | |
hemna | really, the only difference is 1) returning attachment_id in os-reserve and 2) passing in attachment_id into initialize_connection. | 22:13 |
jgriffith | hemna: in other words there is no difference? | 22:13 |
hemna | well, it's important | 22:13 |
jgriffith | hemna: how come? | 22:13 |
hemna | because initialize_connection can't find the correct attachment w/o it. | 22:14 |
jgriffith | hemna: wait | 22:14 |
jgriffith | first | 22:14 |
jgriffith | answer that question... is there any difference then in the impl? | 22:14 |
hemna | as I mentioned in the meeting, line 920 doesn't get you the correct attachment, you get a random attachment entry. | 22:14 |
jgriffith | hemna: why can't initialize_connection not find something? And what is the "correct attachment" ? | 22:14 |
jgriffith | hemna: umm... that's a stub because multi-attach is broken | 22:15 |
jgriffith | but I can fix that | 22:15 |
jgriffith | hemna: what I'm asking is how are you proposing to fix that? | 22:15 |
hemna | and the attachment entry doesn't exist yet then. | 22:15 |
jgriffith | hemna: sure it does... line 907 | 22:15 |
jgriffith | but regardless.. how are you fixing it? | 22:16 |
jgriffith | hemna: don't get me wrong, you may have the perfect solution | 22:16 |
jgriffith | hemna: I want to understand | 22:16 |
hemna | volume_attachment_get_used_by_volume_id doesn't create the entry | 22:16 |
hemna | it searches for it. | 22:17 |
*** tpsilva has quit IRC | 22:17 | |
jgriffith | volume_attachment_get_used_by_volume_id doesn't create what? | 22:17 |
hemna | the entry isn't created until later | 22:17 |
jgriffith | ?? | 22:17 |
hemna | the volume_attachment row | 22:17 |
hemna | there is no entry in the volume_attachment table yet until later. | 22:17 |
jgriffith | hemna: sure | 22:17 |
jgriffith | hemna: that just means there's no existing attachment | 22:18 |
jgriffith | so who cares? | 22:18 |
hemna | so you aren't going to be working on the right attachment row in the db. | 22:18 |
jgriffith | hemna: if there isn't an entry in the db it means nothings been attached so there is no such thing as a correct row | 22:18 |
hemna | https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L986 | 22:19 |
hemna | that's where the row is created | 22:19 |
jgriffith | hemna: I don't understand your point at all | 22:19 |
jgriffith | hemna: and clearly we already know that the code in master as it is doesn't work | 22:19 |
hemna | line 947 in your code won'd do anything | 22:19 |
hemna | it won't be updating the record in the DB, because it doesn't exist yet. | 22:19 |
hemna | I don't know what else I can say to make that clear. | 22:20 |
jgriffith | hemna: that creates the entry | 22:20 |
*** raunak_ has quit IRC | 22:20 | |
jgriffith | hemna: alright | 22:20 |
hemna | my point is, you won't ever be able to find the correct attachment in initialize_connection because the db doesn't have it at initialize_conneciton time. | 22:20 |
jgriffith | I'll leave it be then | 22:20 |
*** ametts has quit IRC | 22:21 | |
jgriffith | hemna: thanks for trying to explain it to me at least | 22:21 |
hemna | sure, I hope that helped some. | 22:21 |
jgriffith | hemna: honestly... no, not at all | 22:22 |
jgriffith | hemna: in fact I found it more confusing than anything else | 22:22 |
jgriffith | hemna: but that's on me | 22:22 |
hemna | ok so for my understanding | 22:23 |
hemna | the order of operations is | 22:23 |
hemna | os-reserve | 22:23 |
hemna | os-initialize_connection | 22:23 |
hemna | os-attach | 22:23 |
*** haplo37 has quit IRC | 22:23 | |
jgriffith | There's no ambiguity there: http://docs.openstack.org/developer/cinder/devref/attach_detach_conventions.html | 22:23 |
jgriffith | That IS the order of operations | 22:23 |
hemna | yah | 22:23 |
jgriffith | hemna: I don't think we need to waste your time | 22:24 |
hemna | the db record in volume_attachment doesn't get created until os-attach | 22:24 |
jgriffith | hemna: Honestly, I appreciate you trying to explain but I don't think I'll understand your solution without seeing code | 22:24 |
hemna | https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L986 | 22:24 |
hemna | that's where the volume_attachment record is created | 22:24 |
jgriffith | hemna: that's what I'm saying though, my proposal is in fact changing that to create the entry at initialize connection | 22:25 |
hemna | ok I'm not seeing it in your WIP then | 22:26 |
hemna | maybe that's why I'm confused | 22:26 |
*** karthikp has quit IRC | 22:27 | |
*** karthikp has joined #openstack-cinder | 22:28 | |
*** edtubill has quit IRC | 22:29 | |
jgriffith | hemna: line 1471 in the initialize_connection method you mean? | 22:32 |
jgriffith | hemna: where it says "Treat every connection independtly....." | 22:32 |
hemna | omg | 22:32 |
hemna | gerrit completely mangled the diff | 22:33 |
hemna | and had that code as part of attach | 22:33 |
hemna | ! | 22:33 |
hemna | lol | 22:33 |
jgriffith | hemna: ohhhh | 22:33 |
jgriffith | hemna: shit, I was trying to figure out WTF you were talking about | 22:33 |
* hemna hates new gerrit UI | 22:33 | |
jgriffith | hemna: I thought I was loosing what's left of my mind | 22:33 |
hemna | seriously die!!! | 22:33 |
hemna | so did I ! | 22:33 |
hemna | omfg | 22:33 |
jgriffith | hemna: FWIW, I set full context in my settings for this very reason | 22:33 |
hemna | I didn't see it until I launched a new browser and hit the url again. | 22:34 |
hemna | *sigh* | 22:34 |
hemna | sorry man. | 22:34 |
jgriffith | hemna: that's alright, I'm just really happy we figured out the disconnect there | 22:34 |
hemna | man that was awful | 22:34 |
jgriffith | hemna: It's kinda funny now :) | 22:35 |
hemna | :) | 22:35 |
hemna | yah | 22:35 |
eharney | reminder, see this patch re: gate failures: https://review.openstack.org/#/c/302439/ | 22:35 |
eharney | later | 22:35 |
*** eharney has quit IRC | 22:35 | |
*** dflorea has joined #openstack-cinder | 22:35 | |
jgriffith | waiting for jenkins then I'll +2/A | 22:36 |
*** dflorea has quit IRC | 22:37 | |
*** martyturner has quit IRC | 22:39 | |
ildikov | hemna: jgriffith: if you've saved the world please add info to the etherpad for next week :) | 22:42 |
hemna | ildikov, :) | 22:42 |
* ildikov hates new Gerrit UI as well... | 22:43 | |
mtanino | Are there an engineer from Nimble here? | 22:44 |
Swanson | If there is please let their name be Jack. | 22:45 |
*** gcb has quit IRC | 22:45 | |
mtanino | Swanson: Jack? | 22:47 |
*** gcb has joined #openstack-cinder | 22:50 | |
*** sgotliv has quit IRC | 22:51 | |
*** karthikp has quit IRC | 22:52 | |
*** jgregor has quit IRC | 22:52 | |
*** dflorea has joined #openstack-cinder | 22:54 | |
*** dflorea has quit IRC | 22:56 | |
*** bjornar has quit IRC | 22:59 | |
*** lcurtis has quit IRC | 23:09 | |
*** gcb has quit IRC | 23:11 | |
*** gcb has joined #openstack-cinder | 23:11 | |
*** gouthamr has joined #openstack-cinder | 23:16 | |
*** boichev has quit IRC | 23:16 | |
*** dustins has quit IRC | 23:18 | |
*** gcb has quit IRC | 23:33 | |
*** jamielennox|away is now known as jamielennox | 23:34 | |
*** gcb has joined #openstack-cinder | 23:34 | |
*** akshai has quit IRC | 23:34 | |
*** mriedem has joined #openstack-cinder | 23:36 | |
*** salv-orl_ has quit IRC | 23:36 | |
jgriffith | Swanson: or Mr. Quick | 23:37 |
jgriffith | mtanino: it's an old nursery school rhyme | 23:40 |
jgriffith | "Jack be nimble, Jack be quick, Jack jump over the candle stick" | 23:40 |
jgriffith | Swanson: has a very warped sense of humor | 23:40 |
mtanino | jgriffith: Ahh, it's a song. | 23:41 |
mtanino | took one hour to understand that humor... ;) | 23:42 |
*** Julien-zte has joined #openstack-cinder | 23:43 | |
jgriffith | mtanino: :) | 23:45 |
jgriffith | mtanino: it take a LOT longer than that to understand Swanson 's humor :) | 23:45 |
*** Julien-zte has quit IRC | 23:46 | |
*** jwcroppe has quit IRC | 23:46 | |
*** yhayashi has joined #openstack-cinder | 23:51 | |
*** smoriya_ has joined #openstack-cinder | 23:54 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!