| *** kragniz_ is now known as kragniz | 00:00 | |
| *** markvoelker has joined #openstack-cinder | 00:02 | |
| *** smoriya has joined #openstack-cinder | 00:02 | |
| *** markvoelker has quit IRC | 00:08 | |
| *** ebalduf has quit IRC | 00:10 | |
| *** Yogi1 has quit IRC | 00:16 | |
| *** ho has joined #openstack-cinder | 00:43 | |
| *** Lee1092 has joined #openstack-cinder | 00:46 | |
| *** ho has quit IRC | 00:47 | |
| *** ho has joined #openstack-cinder | 00:51 | |
| *** markvoelker has joined #openstack-cinder | 00:52 | |
| *** markvoelker has quit IRC | 00:57 | |
| *** liusheng has quit IRC | 01:02 | |
| *** Yogi1 has joined #openstack-cinder | 01:09 | |
| *** ebalduf has joined #openstack-cinder | 01:11 | |
| *** ebalduf has quit IRC | 01:16 | |
| *** Yogi1 has quit IRC | 01:26 | |
| openstackgerrit | wanghao proposed openstack/cinder: non-admin user to query volume filter by az https://review.openstack.org/156034 | 01:37 |
|---|---|---|
| *** scottda_ has joined #openstack-cinder | 01:42 | |
| *** scottda_ has quit IRC | 01:46 | |
| *** markvoelker has joined #openstack-cinder | 01:54 | |
| *** markvoelker has quit IRC | 01:58 | |
| *** thrawn01 has quit IRC | 02:03 | |
| *** thrawn01 has joined #openstack-cinder | 02:07 | |
| *** ebalduf has joined #openstack-cinder | 02:12 | |
| *** ebalduf has quit IRC | 02:17 | |
| *** jamielennox is now known as jamielennox|away | 02:19 | |
| *** jamielennox|away is now known as jamielennox | 02:27 | |
| *** ebalduf has joined #openstack-cinder | 02:39 | |
| *** kaufer has joined #openstack-cinder | 02:39 | |
| *** ebalduf has quit IRC | 02:39 | |
| *** ebalduf has joined #openstack-cinder | 02:41 | |
| openstackgerrit | wanghao proposed openstack/cinder: Implement function of import/export snapshots https://review.openstack.org/144590 | 02:42 |
| *** Yogi1 has joined #openstack-cinder | 02:42 | |
| *** markvoelker has joined #openstack-cinder | 02:55 | |
| *** mberlin has joined #openstack-cinder | 02:56 | |
| *** kaisers1 has joined #openstack-cinder | 02:56 | |
| *** kaisers has quit IRC | 02:57 | |
| *** mberlin1 has quit IRC | 02:58 | |
| *** Yogi1 has quit IRC | 03:00 | |
| *** markvoelker has quit IRC | 03:01 | |
| *** jaypipes has joined #openstack-cinder | 03:10 | |
| *** Yogi1 has joined #openstack-cinder | 03:24 | |
| *** harlowja_at_home has joined #openstack-cinder | 03:34 | |
| *** Yogi1 has quit IRC | 03:44 | |
| *** markvoelker has joined #openstack-cinder | 03:57 | |
| *** markvoelker has quit IRC | 04:02 | |
| *** harlowja_at_home has quit IRC | 04:07 | |
| openstackgerrit | wanghao proposed openstack/cinder: Implement function of import/export snapshots https://review.openstack.org/144590 | 04:11 |
| openstackgerrit | Steven Kaufer proposed openstack/cinder: Replication status periodic task optimization https://review.openstack.org/154673 | 04:12 |
| *** Mandell has joined #openstack-cinder | 04:12 | |
| *** kaufer has quit IRC | 04:21 | |
| *** ebalduf has quit IRC | 04:24 | |
| *** Teal has left #openstack-cinder | 04:39 | |
| *** pradipta has joined #openstack-cinder | 04:40 | |
| *** markvoelker has joined #openstack-cinder | 04:58 | |
| *** theanalyst has quit IRC | 05:03 | |
| *** markvoelker has quit IRC | 05:03 | |
| *** theanalyst has joined #openstack-cinder | 05:05 | |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Huawei driver check before associating LUN to a LUN group https://review.openstack.org/148144 | 05:09 |
| *** Ilja has joined #openstack-cinder | 05:16 | |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Huawei driver fix problems under multipath https://review.openstack.org/150767 | 05:16 |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Huawei driver remove LUN controller change https://review.openstack.org/152116 | 05:24 |
| *** annashen has joined #openstack-cinder | 05:34 | |
| *** anshul has joined #openstack-cinder | 05:44 | |
| *** markvoelker has joined #openstack-cinder | 05:59 | |
| *** markvoelker has quit IRC | 06:04 | |
| *** sgotliv has joined #openstack-cinder | 06:06 | |
| *** ankit_ag has joined #openstack-cinder | 06:08 | |
| *** Mandell has quit IRC | 06:09 | |
| *** Mandell has joined #openstack-cinder | 06:19 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Transifex https://review.openstack.org/152401 | 06:29 |
| *** Mandell has quit IRC | 06:40 | |
| *** Murugan_ has joined #openstack-cinder | 06:42 | |
| *** annashen has quit IRC | 06:46 | |
| openstackgerrit | Amit Saha proposed openstack/cinder: Fixed typo https://review.openstack.org/155213 | 06:47 |
| *** vnagpal has joined #openstack-cinder | 06:47 | |
| openstackgerrit | Amit Saha proposed openstack/cinder: Fixed typo https://review.openstack.org/155213 | 06:50 |
| *** vilobhmm has joined #openstack-cinder | 06:58 | |
| openstackgerrit | Pranali Deore proposed openstack/cinder: Snapshot of bootable volume goes in error state https://review.openstack.org/153170 | 07:00 |
| *** markvoelker has joined #openstack-cinder | 07:01 | |
| *** nkrinner has joined #openstack-cinder | 07:02 | |
| openstackgerrit | Amit Saha proposed openstack/cinder: Fixed a couple of Typos https://review.openstack.org/156140 | 07:03 |
| *** markvoelker has quit IRC | 07:06 | |
| *** theanalyst has quit IRC | 07:10 | |
| *** sgotliv has quit IRC | 07:12 | |
| openstackgerrit | Amit Saha proposed openstack/cinder: Fixed typo https://review.openstack.org/155213 | 07:13 |
| *** theanalyst has joined #openstack-cinder | 07:13 | |
| *** Mandell has joined #openstack-cinder | 07:18 | |
| *** avishay has joined #openstack-cinder | 07:24 | |
| *** pradipta has quit IRC | 07:31 | |
| *** afazekas_ has joined #openstack-cinder | 07:35 | |
| *** junhongl has quit IRC | 07:40 | |
| *** zhipeng has joined #openstack-cinder | 07:45 | |
| *** chlong has quit IRC | 07:51 | |
| *** nellysmitt has joined #openstack-cinder | 07:53 | |
| *** vilobhmm has quit IRC | 07:57 | |
| *** markvoelker has joined #openstack-cinder | 08:02 | |
| *** avishay has quit IRC | 08:04 | |
| *** Mandell has quit IRC | 08:05 | |
| *** Mandell has joined #openstack-cinder | 08:06 | |
| *** markvoelker has quit IRC | 08:07 | |
| *** Mandell has quit IRC | 08:07 | |
| *** Mandell has joined #openstack-cinder | 08:07 | |
| *** jistr has joined #openstack-cinder | 08:08 | |
| *** takedakn has joined #openstack-cinder | 08:13 | |
| *** mberlin has quit IRC | 08:14 | |
| *** takedakn has quit IRC | 08:19 | |
| *** mberlin has joined #openstack-cinder | 08:25 | |
| *** dulek has joined #openstack-cinder | 08:40 | |
| *** jordanP has joined #openstack-cinder | 08:42 | |
| *** ronis has joined #openstack-cinder | 08:42 | |
| *** takedakn has joined #openstack-cinder | 08:42 | |
| *** jordanP has quit IRC | 08:44 | |
| *** jordanP has joined #openstack-cinder | 08:44 | |
| *** annashen has joined #openstack-cinder | 08:47 | |
| *** mberlin has quit IRC | 08:50 | |
| *** annashen has quit IRC | 08:52 | |
| *** deepakcs has joined #openstack-cinder | 08:53 | |
| *** takedakn has quit IRC | 08:54 | |
| *** nlevinki has joined #openstack-cinder | 08:56 | |
| *** openstackgerrit has quit IRC | 08:57 | |
| *** openstackgerrit has joined #openstack-cinder | 08:58 | |
| *** ndipanov has joined #openstack-cinder | 08:59 | |
| *** karimb has joined #openstack-cinder | 08:59 | |
| *** jistr has quit IRC | 08:59 | |
| *** Longgeek has joined #openstack-cinder | 09:00 | |
| *** markvoelker has joined #openstack-cinder | 09:03 | |
| *** markvoelker has quit IRC | 09:08 | |
| *** ho has quit IRC | 09:14 | |
| *** boichev has quit IRC | 09:21 | |
| *** Ilja has quit IRC | 09:29 | |
| *** dalgaaf has joined #openstack-cinder | 09:40 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Fix comments style according to the Hacking Rules https://review.openstack.org/145821 | 09:40 |
| *** zhipeng has quit IRC | 09:42 | |
| *** Mandell has quit IRC | 09:43 | |
| *** theanalyst has quit IRC | 09:47 | |
| *** theanalyst has joined #openstack-cinder | 09:47 | |
| *** alecv has joined #openstack-cinder | 09:48 | |
| *** yuriy_n17 has joined #openstack-cinder | 09:56 | |
| *** takedakn has joined #openstack-cinder | 10:01 | |
| *** obutenko_ has quit IRC | 10:03 | |
| *** markvoelker has joined #openstack-cinder | 10:04 | |
| *** alecv has quit IRC | 10:04 | |
| *** markvoelker has quit IRC | 10:09 | |
| *** takedakn has quit IRC | 10:14 | |
| *** lpetrut has joined #openstack-cinder | 10:16 | |
| *** e0ne has joined #openstack-cinder | 10:20 | |
| *** rushiagr_away is now known as rushiagr | 10:23 | |
| *** pradipta has joined #openstack-cinder | 10:25 | |
| *** rushiagr is now known as rushiagr_away | 10:40 | |
| *** aix has joined #openstack-cinder | 10:44 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Import only modules: H302 https://review.openstack.org/145780 | 10:46 |
| *** annashen has joined #openstack-cinder | 10:48 | |
| *** annashen has quit IRC | 10:53 | |
| *** e0ne is now known as e0ne_ | 10:57 | |
| openstackgerrit | Julien Danjou proposed openstack/cinder: Remove useless requirement on wsgiref https://review.openstack.org/156182 | 10:58 |
| *** e0ne_ is now known as e0ne | 11:00 | |
| *** markvoelker has joined #openstack-cinder | 11:05 | |
| *** markvoelker has quit IRC | 11:11 | |
| *** casusbelli has joined #openstack-cinder | 11:20 | |
| *** aix has quit IRC | 11:27 | |
| *** sgotliv has joined #openstack-cinder | 11:30 | |
| *** Ilja has joined #openstack-cinder | 11:30 | |
| openstackgerrit | Ronen Mesonzhnik proposed openstack/cinder: Provide backup service that uses ProtecTIER as the backend https://review.openstack.org/153969 | 11:32 |
| *** Ilja has quit IRC | 11:35 | |
| openstackgerrit | Marc Koderer proposed openstack/cinder: Adapt volume manager to new driver structure https://review.openstack.org/138999 | 11:36 |
| *** Ilja has joined #openstack-cinder | 11:36 | |
| *** Ilja1 has joined #openstack-cinder | 11:39 | |
| *** Ilja has quit IRC | 11:39 | |
| *** sgotliv has quit IRC | 11:40 | |
| openstackgerrit | Ronen Mesonzhnik proposed openstack/cinder: Provide backup service that uses ProtecTIER as the backend https://review.openstack.org/153969 | 11:41 |
| *** e0ne is now known as e0ne_ | 11:45 | |
| *** boichev has joined #openstack-cinder | 11:48 | |
| *** sgotliv has joined #openstack-cinder | 11:54 | |
| *** e0ne_ has quit IRC | 11:55 | |
| *** e0ne has joined #openstack-cinder | 11:57 | |
| openstackgerrit | wang peter proposed openstack/cinder: CG Modification Support in EMC VNX Cinder Driver https://review.openstack.org/154784 | 12:04 |
| *** markvoelker has joined #openstack-cinder | 12:08 | |
| *** smcginni1 is now known as smcginnis | 12:08 | |
| *** markvoelker has quit IRC | 12:12 | |
| *** smoriya has quit IRC | 12:25 | |
| *** aix has joined #openstack-cinder | 12:32 | |
| *** e0ne is now known as e0ne_ | 12:35 | |
| *** IanGovett has joined #openstack-cinder | 12:36 | |
| *** kbyrne has joined #openstack-cinder | 12:37 | |
| *** e0ne_ is now known as e0ne | 12:39 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Fix comments style according to the Hacking Rules https://review.openstack.org/145821 | 12:47 |
| *** annashen has joined #openstack-cinder | 12:49 | |
| *** humble_ has joined #openstack-cinder | 12:51 | |
| *** annashen has quit IRC | 12:54 | |
| *** takedakn has joined #openstack-cinder | 12:56 | |
| *** takedakn has quit IRC | 12:56 | |
| openstackgerrit | Edwin Wang proposed openstack/cinder: Add iSCSI protocol support for IBM FlashSystem https://review.openstack.org/156206 | 12:58 |
| openstackgerrit | Edwin Wang proposed openstack/cinder: Add iSCSI protocol support for IBM FlashSystem https://review.openstack.org/156206 | 13:00 |
| openstackgerrit | Edwin Wang proposed openstack/cinder: Add iSCSI protocol support for IBM FlashSystem https://review.openstack.org/156206 | 13:01 |
| *** e0ne is now known as e0ne_ | 13:04 | |
| *** takedakn has joined #openstack-cinder | 13:05 | |
| *** takedakn has quit IRC | 13:07 | |
| *** markvoelker has joined #openstack-cinder | 13:09 | |
| *** deepakcs has quit IRC | 13:14 | |
| *** markvoelker has quit IRC | 13:14 | |
| *** e0ne_ has quit IRC | 13:14 | |
| *** e0ne has joined #openstack-cinder | 13:16 | |
| *** ankit_ag has quit IRC | 13:19 | |
| *** EmilienM is now known as EmilienM|afk | 13:20 | |
| *** coolsvap- is now known as coolsvap | 13:27 | |
| *** Longgeek has quit IRC | 13:32 | |
| openstackgerrit | Ph. Marek proposed openstack/cinder: DRBD for NOVA. https://review.openstack.org/156212 | 13:33 |
| *** rushiagr_away is now known as rushiagr | 13:34 | |
| *** jcru has joined #openstack-cinder | 13:38 | |
| openstackgerrit | Ronen Mesonzhnik proposed openstack/cinder: Provide backup service that uses ProtecTIER as the backend https://review.openstack.org/153969 | 13:38 |
| *** e0ne is now known as e0ne_ | 13:40 | |
| *** sgotliv has quit IRC | 13:44 | |
| *** Mandell has joined #openstack-cinder | 13:44 | |
| *** e0ne_ has quit IRC | 13:45 | |
| *** sgotliv has joined #openstack-cinder | 13:47 | |
| *** sgotliv has quit IRC | 13:47 | |
| *** sgotliv has joined #openstack-cinder | 13:47 | |
| *** Mandell has quit IRC | 13:49 | |
| *** e0ne has joined #openstack-cinder | 13:50 | |
| *** e0ne is now known as e0ne_ | 13:51 | |
| *** e0ne_ is now known as e0ne | 13:51 | |
| *** e0ne is now known as e0ne_ | 13:51 | |
| *** e0ne_ is now known as e0ne | 13:52 | |
| *** e0ne is now known as e0ne_ | 13:52 | |
| *** sandywalsh_ has quit IRC | 13:52 | |
| *** e0ne_ is now known as e0ne | 13:52 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Delay requests in scheduler to wait for metrics https://review.openstack.org/156219 | 13:52 |
| *** e0ne is now known as e0ne_ | 13:53 | |
| *** e0ne_ is now known as e0ne | 13:53 | |
| dulek | e0ne: Hi | 13:54 |
| dulek | e0ne: You may want to take a look at my attempt to fix https://bugs.launchpad.net/cinder/+bug/1409012 | 13:54 |
| openstack | Launchpad bug 1409012 in Cinder "Volume becomes in 'error' state after scheduler starts" [High,In progress] - Assigned to Michal Dulko (michal-dulko-f) | 13:54 |
| e0ne | dulek: hi | 13:54 |
| e0ne | dulek: sure, i'll do! thanks | 13:55 |
| dulek | e0ne: I hope this idea doesn't share your problems with instability. | 13:55 |
| dulek | your idea's problems ;) | 13:55 |
| e0ne | :) | 13:56 |
| *** nlevinki has quit IRC | 13:57 | |
| *** boichev2 has joined #openstack-cinder | 13:57 | |
| *** anshul has quit IRC | 13:57 | |
| *** boichev has quit IRC | 13:59 | |
| *** Guest59076 is now known as dansmith | 14:05 | |
| *** dansmith is now known as Guest28456 | 14:06 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Delay requests in scheduler to wait for metrics https://review.openstack.org/156219 | 14:08 |
| *** Guest28456 is now known as dansmith_ | 14:09 | |
| *** markvoelker has joined #openstack-cinder | 14:10 | |
| *** markvoelker has quit IRC | 14:15 | |
| *** kaufer has joined #openstack-cinder | 14:18 | |
| *** Murugan_ has quit IRC | 14:20 | |
| *** vnagpal has quit IRC | 14:21 | |
| *** afazekas_ has quit IRC | 14:23 | |
| *** markvoelker has joined #openstack-cinder | 14:28 | |
| *** scottda_ has joined #openstack-cinder | 14:28 | |
| *** mriedem has joined #openstack-cinder | 14:31 | |
| *** Longgeek has joined #openstack-cinder | 14:33 | |
| *** markvoelker has quit IRC | 14:34 | |
| dulek | DuncanT: What are the symptoms that Jenkins isn't fine with review 156219? | 14:35 |
| *** scottda_ has quit IRC | 14:37 | |
| DuncanT | dulek: I'm sorry, I entirely misread my screen there, it is the IBM CI system that isn't happy, jenkins hasn't got to it yet. Sorry for the false alarm | 14:38 |
| *** primechuck has joined #openstack-cinder | 14:39 | |
| dulek | DuncanT: np, thanks for clarifying, I've thought that there's some tool to check that I'm not aware of. :) | 14:39 |
| *** Longgeek has quit IRC | 14:40 | |
| *** afazekas has joined #openstack-cinder | 14:40 | |
| *** Longgeek has joined #openstack-cinder | 14:40 | |
| DuncanT | dulek: Nope, just me being blind. Thanks for looking at this, it is a nasty issue and we're unlikely to get a clean fix through any of the necessary hooks with oslo in time for Kilo | 14:41 |
| dulek | DuncanT: I've commented on the patchset, but I we can discuss here | 14:41 |
| dulek | Is it considered to make volume save metrics to the DB? | 14:41 |
| dulek | This is how Nova is tackling this problem | 14:42 |
| DuncanT | I think we used to do that but we were trying to reduce the load on the DB (nova folks have also commented many times that the db approach has issues, the 'no db scheduler' proposal has floated and sunk many, many times) | 14:43 |
| DuncanT | I /think/ the right solution is for the scheduler to listen for incoming stats messages from drivers, but not for work, for the first thirty seconds that it is up | 14:44 |
| dulek | Hm, compromise would be to save metrics but read it only on the start and then rely on the RPC casts | 14:44 |
| dulek | for updates | 14:44 |
| DuncanT | The volume-managers would still have to write to the db every update, since they don't know when scheduler(s) are starting | 14:45 |
| dulek | DuncanT: True | 14:45 |
| DuncanT | And it seems rather ugly to my mind to have two communications methods | 14:45 |
| dulek | DuncanT: You're right, when you have DB load on your mind then RPC solution seems to be the best | 14:46 |
| DuncanT | I suppose if it went in its own table, we can always back out the db scheduler cleanly if needed. The data coming back from the cinder-volume service is getting very large though, with pools, schema, help strings etc being in there - 10s of k is looking likely | 14:47 |
| DuncanT | I've been worrying for a while if MQ load isn't a problem too :-) | 14:47 |
| DuncanT | Need to run some benchmarks though, not just guess, and I haven't time for that at the moment | 14:48 |
| dulek | I guess MQ is easier to scale than DB. | 14:48 |
| DuncanT | We're trying to get a release out of the door this week, so I'm rather distracted by that TBH | 14:48 |
| dulek | What do you mean by out of the door? | 14:49 |
| DuncanT | Well, you can shard the db easily enough if you think hard about it, just as you can MQs, so they scale pretty much horizontally either way, though neither is easy to setup/manage | 14:49 |
| DuncanT | HP have a Helion version release nominally scheduled for this Friday | 14:50 |
| *** annashen has joined #openstack-cinder | 14:50 | |
| *** jungleboyj has quit IRC | 14:50 | |
| DuncanT | By 'out the door' I mean we're in the final stages of the release process | 14:50 |
| dulek | Ah, okay, thought of Cinder and that confused me | 14:51 |
| *** tsekiyama has joined #openstack-cinder | 14:51 | |
| *** Ilja1 has quit IRC | 14:52 | |
| *** Ilja has joined #openstack-cinder | 14:52 | |
| DuncanT | No, if cinder was my only responsibility I'd be happier and more productive :-) | 14:53 |
| *** annashen has quit IRC | 14:55 | |
| *** scottda_ has joined #openstack-cinder | 14:59 | |
| *** tsekiyam_ has joined #openstack-cinder | 15:00 | |
| *** sigmavirus24_awa is now known as sigmavirus24 | 15:01 | |
| *** humble_ has quit IRC | 15:03 | |
| *** tsekiyama has quit IRC | 15:04 | |
| *** eharney has joined #openstack-cinder | 15:04 | |
| *** dansmith_ is now known as dansmith | 15:05 | |
| *** scottda_ has quit IRC | 15:05 | |
| *** scottda_ has joined #openstack-cinder | 15:10 | |
| *** patrickeast has joined #openstack-cinder | 15:11 | |
| openstackgerrit | Marc Koderer proposed openstack/cinder: Adapt volume manager to new driver structure https://review.openstack.org/138999 | 15:12 |
| openstackgerrit | Marc Koderer proposed openstack/cinder: Port rbd driver to use new driver model https://review.openstack.org/138661 | 15:12 |
| *** beekneemech is now known as bnemec | 15:13 | |
| *** scottda_ has quit IRC | 15:14 | |
| *** EmilienM|afk is now known as EmilienM | 15:16 | |
| *** thangp has joined #openstack-cinder | 15:16 | |
| *** scottda_ has joined #openstack-cinder | 15:19 | |
| openstackgerrit | Ronen Mesonzhnik proposed openstack/cinder: Provide backup service that uses ProtecTIER as the backend https://review.openstack.org/153969 | 15:26 |
| *** Ilja has quit IRC | 15:27 | |
| *** e0ne is now known as e0ne_ | 15:28 | |
| *** thangp has quit IRC | 15:29 | |
| openstackgerrit | Patrick East proposed openstack/cinder-specs: Add DB table for driver private data https://review.openstack.org/153333 | 15:30 |
| *** ebalduf has joined #openstack-cinder | 15:30 | |
| *** markvoelker has joined #openstack-cinder | 15:31 | |
| *** xyang1 has joined #openstack-cinder | 15:34 | |
| *** markvoelker has quit IRC | 15:35 | |
| *** patrickeast has quit IRC | 15:35 | |
| *** patrickeast_ has joined #openstack-cinder | 15:35 | |
| *** e0ne_ has quit IRC | 15:38 | |
| *** jungleboyj has joined #openstack-cinder | 15:39 | |
| openstackgerrit | Jeegn Chen proposed openstack/cinder: Pool-aware scheduler support in EMC VNX Cinder driver https://review.openstack.org/147408 | 15:45 |
| *** tsekiyam_ has quit IRC | 15:46 | |
| *** sandywalsh has joined #openstack-cinder | 15:47 | |
| *** haomaiwang has joined #openstack-cinder | 15:47 | |
| *** tsekiyama has joined #openstack-cinder | 15:50 | |
| *** nellysmitt has quit IRC | 15:51 | |
| *** Mandell has joined #openstack-cinder | 15:52 | |
| *** markvoelker has joined #openstack-cinder | 15:53 | |
| *** mtanino has joined #openstack-cinder | 15:53 | |
| *** rmesta has joined #openstack-cinder | 15:54 | |
| *** jistr has joined #openstack-cinder | 15:58 | |
| *** markvoelker has quit IRC | 15:58 | |
| *** david-lyle_afk is now known as david-lyle | 16:01 | |
| *** ronis has quit IRC | 16:01 | |
| *** scottda_ has quit IRC | 16:01 | |
| *** thangp has joined #openstack-cinder | 16:02 | |
| jgriffith | hemna: gary-smith around? | 16:05 |
| *** anshul has joined #openstack-cinder | 16:07 | |
| *** nkrinner has quit IRC | 16:08 | |
| *** pradipta has quit IRC | 16:09 | |
| *** hodos has joined #openstack-cinder | 16:14 | |
| openstackgerrit | Tomoki Sekiyama proposed openstack/cinder: Limit volume copy bandwidth per backend https://review.openstack.org/149446 | 16:14 |
| openstackgerrit | Tomoki Sekiyama proposed openstack/cinder: Limit volume copy bandwidth per backend https://review.openstack.org/149446 | 16:15 |
| *** xyang has joined #openstack-cinder | 16:16 | |
| *** scottda_ has joined #openstack-cinder | 16:18 | |
| *** scottda_ has quit IRC | 16:20 | |
| openstackgerrit | Mikhail Khodos proposed openstack/cinder: Fix Nexenta NFS driver mounts https://review.openstack.org/149380 | 16:22 |
| patrickeast_ | jgriffith: ping | 16:24 |
| *** zigo has quit IRC | 16:26 | |
| jgriffith | patrickeast_: hey | 16:27 |
| *** annashen has joined #openstack-cinder | 16:27 | |
| patrickeast_ | jgriffith: hey, just wanted to follow up on the replication thing about replicating to hosts outside of cinder | 16:28 |
| jgriffith | patrickeast_: yeah.. | 16:28 |
| patrickeast_ | i think what you have works well, i actually lean towards not allowing that and making sure this doesn’t allow for it | 16:28 |
| jgriffith | patrickeast_: so when you say outside of Cinder to me that implies you won't fail over to it either right? | 16:28 |
| patrickeast_ | the current replication does and its a pain | 16:28 |
| patrickeast_ | our driver then essentially has to manage multiple arrays | 16:28 |
| patrickeast_ | no it does fail over | 16:29 |
| jgriffith | patrickeast_: so it fails over in the driver only? | 16:29 |
| jgriffith | patrickeast_: yeah, not a fan of that | 16:29 |
| patrickeast_ | but the initialize_connection and all that needs to know which array to connect to any point after | 16:29 |
| patrickeast_ | yea its gross | 16:29 |
| patrickeast_ | caused a lot of issues (not sure if you saw the review our team put up) | 16:29 |
| jgriffith | patrickeast_: FWIW my first iteration did that | 16:29 |
| jgriffith | patrickeast_: I don't know that I did... | 16:30 |
| jgriffith | patrickeast_: so my take on it would be Cinder needs to know about the device to do things properly | 16:30 |
| jgriffith | patrickeast_: the "no_schedule" flag I think is a good thing to have | 16:30 |
| *** delattec has joined #openstack-cinder | 16:30 | |
| patrickeast_ | jgriffith: ok cool, we’re on the same page then ;) | 16:31 |
| jgriffith | but no knowledge of the target makes things ugly | 16:31 |
| *** zigo has joined #openstack-cinder | 16:31 | |
| jgriffith | patrickeast_: awesome | 16:31 |
| jgriffith | patrickeast_: and FWIW, my main thing about that is if you want to implement a replication target all in your driver with Cinder not having any real knowledge... | 16:31 |
| jgriffith | That's fine, but you can do that today with extra-spec keys | 16:31 |
| patrickeast_ | yea exactly | 16:32 |
| *** jistr has quit IRC | 16:32 | |
| jgriffith | I have a customer doing that with a "custom" driver on Icehouse today | 16:32 |
| jgriffith | no changes to Cinder at all | 16:32 |
| jgriffith | Ok... cool | 16:32 |
| *** delattec has quit IRC | 16:33 | |
| *** delatte has quit IRC | 16:33 | |
| patrickeast_ | jgriffith: so one other thing, not sure i saw it in the spec, are you thinking that the two drivers for the replicated backends will know about eachother? like for ours we need to know about their management ip’s and such | 16:33 |
| patrickeast_ | i don’t think anywhere else in cinder a driver actually has access to values from another one | 16:34 |
| jgriffith | patrickeast_: so that's the nice thing about having it in the conf and putting some sort of pairing together | 16:34 |
| jgriffith | patrickeast_: the driver could use the conf file to obtain that info if it wants | 16:34 |
| patrickeast_ | ooo right, gotcha | 16:34 |
| jgriffith | patrickeast_: or ignore it if it doesn't care | 16:34 |
| jgriffith | patrickeast_: although I don't know how it would ignore it | 16:34 |
| jgriffith | patrickeast_: but regardless, it accomodates both scenarios | 16:34 |
| patrickeast_ | perfect | 16:35 |
| jgriffith | patrickeast_: awesome... I'll get some more feedback from folks and update that spec later today | 16:35 |
| jgriffith | I'd like to make sure hemna, smcginnis, xyang, and jungleboyj inparticular have a good look at it | 16:36 |
| xyang | jgriffith: So Cinder will know only 1 volume, even though there are primary, secondary hosts | 16:36 |
| jgriffith | xyang: right | 16:36 |
| jgriffith | xyang: so rather than have multiple volumes we have multiple hosts for a volume | 16:37 |
| jgriffith | xyang: and that's what we change around instead of "volume swapping" magic | 16:37 |
| xyang | jgriffith: ok, I think that is easier to manage | 16:37 |
| jgriffith | xyang: yeah, I think it's a lot easier to manage | 16:37 |
| patrickeast_ | ^ i think that better represents what actually happens (at least for my array) | 16:37 |
| *** e0ne has joined #openstack-cinder | 16:38 | |
| jgriffith | xyang: my thoughts are that it also makes it more flexible and less intrusive to the Cinder code | 16:38 |
| xyang | jgriffith: we'll need to store primary lun id and secondar lun id somewhere and swap it when we do promote | 16:38 |
| jgriffith | and at the same time has solid structure around it | 16:38 |
| jgriffith | xyang: my thought was when you promote it just sends the initialize_con call to the "other" backend | 16:38 |
| jgriffith | xyang: so we avoid all of that tracking and swapping business altogether | 16:39 |
| patrickeast_ | jgriffith: oh that reminds me, the plan is to still only support failover once a volume is detached and then requires a manual attach call, right? | 16:39 |
| jgriffith | patrickeast_: Yes, I think it's more accurate for the replication use-case in general | 16:39 |
| xyang | jgriffith: will both primary and secondary hosts be active? | 16:39 |
| *** Mandell has quit IRC | 16:39 | |
| *** sgotliv has quit IRC | 16:40 | |
| jgriffith | xyang: depends on what you mean by that... but I'd say "no" | 16:40 |
| jgriffith | xyang: it's an active/passive HA | 16:40 |
| xyang | jgriffith: ok, that's what I refer to | 16:40 |
| xyang | jgriffith: as active/active HA doesn't work for cinder yet | 16:40 |
| jgriffith | patrickeast_: so TBH I think so at least first pass. Only because without things like muti-attach I haven't figured otu a clever way to make it work yet :) | 16:40 |
| jgriffith | patrickeast_: and that goes back to active/passive | 16:40 |
| jgriffith | xyang: right, and there's no way to get that without multi-attach | 16:41 |
| patrickeast_ | ok cool | 16:41 |
| jgriffith | xyang: the good thing is my proposal does take it into consideration | 16:41 |
| xyang | jgriffith: so basically one cinder volume host will be managing a local-remote pair underneath a volume uuid | 16:41 |
| jgriffith | xyang: so that at some point in the future it could be added | 16:41 |
| jgriffith | xyang: well... no | 16:41 |
| jgriffith | xyang: actually kinda the opposite | 16:42 |
| jgriffith | xyang: one volume in Cinder | 16:42 |
| jgriffith | xyang: that can be managed/service by two hosts (backend-drivers) | 16:42 |
| jgriffith | xyang: it flips the model around | 16:42 |
| xyang | jgriffith: yes, one volume, and you only have one entry in db for each volume | 16:43 |
| jgriffith | xyang: but honestly, it's up to your implementation on exactly what that looks like | 16:43 |
| jgriffith | xyang: yes, exactly | 16:43 |
| jungleboyj | jgriffith: Which spec is this that you are talking about? | 16:43 |
| jgriffith | jungleboyj: https://review.openstack.org/#/c/155644/ | 16:43 |
| e0ne | hi all! i know that https://review.openstack.org/131182 is not targeted for K, but spec is merged and i'll be happy if anybody will have a time to review it | 16:44 |
| jungleboyj | Whoa, didn't know you had started this. | 16:44 |
| jungleboyj | jgriffith: Let me read through this. | 16:44 |
| jgriffith | xyang: so just saw your etherpad regarding CG's | 16:44 |
| jungleboyj | jgriffith: Thanks for getting it started. | 16:45 |
| jgriffith | xyang: I'll have to think through that a bit... | 16:45 |
| xyang | jgriffith: sure | 16:45 |
| jgriffith | xyang: but, my first thought is it's just an workflow | 16:45 |
| jgriffith | xyang: in other words, it just means as you say that CG's have to be on replicated volumes | 16:45 |
| xyang | jgriffith: I think it should work together, just need think thru it | 16:45 |
| jgriffith | xyang: depending on the backend that could be made implicit if required | 16:45 |
| jgriffith | xyang: FWIW, if it's required by the back-end I like the idea of making it implicit | 16:46 |
| jgriffith | xyang: the fewer moving parts for the cloud-admin the better | 16:46 |
| xyang | jgriffith: so the promote is currently on volume, we need to add one for group if we integrate the two | 16:46 |
| patrickeast_ | jgriffith: xyang: i can only speak to my knowledge of how we have groups, but can’t we just treat the cg’s the same way we do volumes in your spec? Just let the commands specify a volume or cg id? | 16:47 |
| jgriffith | patrickeast_: yeah.. interesting. So the CG stuff I haven't really looked at with this. | 16:47 |
| jgriffith | patrickeast_: xyang in my case I don't do CG's across devices currently | 16:47 |
| jgriffith | so that was a blind-spot for me | 16:47 |
| jgriffith | not that I can't, I just don't :) | 16:47 |
| *** xyang1 has quit IRC | 16:47 | |
| xyang | jgriffith: patrickeast_: I think the scheduler can take care of that if you have both cg_support and replication_support as true | 16:47 |
| jgriffith | xyang: +1 | 16:48 |
| *** xyang1 has joined #openstack-cinder | 16:48 | |
| jgriffith | xyang: patrickeast_ so... just to make sure I'm clear | 16:48 |
| jgriffith | xyang: patrickeast_ do your backends require replication for CG's? | 16:48 |
| jgriffith | xyang: patrickeast_ or is that an "option" | 16:48 |
| xyang | jgriffith: there's one problem we need to make sure. if volume is in CG, then user can't operate on it individually, so we could have a replication_cg_support flag | 16:49 |
| xyang | if you want both | 16:49 |
| jgriffith | xyang: makes sense | 16:49 |
| jgriffith | xyang: so it's an "option" that can be setup | 16:49 |
| jgriffith | xyang: I think we could do that using the CG API's | 16:49 |
| patrickeast_ | so for us replication is on a cg level, we can do a single volume cg for volume level replication or a whole cg, its treated the same way | 16:49 |
| jgriffith | patrickeast_: right... but the question is "can you do a CG on a single backend" | 16:50 |
| xyang | jgriffith, patrickeast_, ya, pure is the first to put the two together. It is interesting | 16:50 |
| jgriffith | patrickeast_: I've always assumed that answer to be "yes" | 16:50 |
| patrickeast_ | jgriffith: yea we do cg’s on a single backend | 16:50 |
| jgriffith | patrickeast_: so given that, I think we're good still | 16:51 |
| patrickeast_ | things get weird for us in the case of trying to replicate a single volume that belongs to a larger group | 16:51 |
| patrickeast_ | since if you fail it over the group is split | 16:51 |
| xyang | jgriffith: we can do both group level and volume level replication | 16:51 |
| jgriffith | patrickeast_: xyang Ok, perfect. That was my assumption | 16:51 |
| jgriffith | patrickeast_: xyang so I think this works fairly well still | 16:51 |
| *** dulek has quit IRC | 16:52 | |
| jgriffith | patrickeast_: xyang we may impose some limits to start with depending on how things look | 16:52 |
| jgriffith | patrickeast_: xyang but IMO I'd rather do that and iterate on the implementation than try and solve every use case up front | 16:52 |
| patrickeast_ | jgriffith: that sounds good to me, id rather have it be a bit more restrictive than have a weird support matrix of what works on each backend | 16:53 |
| jgriffith | patrickeast_: xyang but I think this is def doable | 16:53 |
| jgriffith | patrickeast_: +1 on the weird support matrix | 16:53 |
| jgriffith | patrickeast_: that only ends badly | 16:53 |
| xyang | jgriffith: sure | 16:53 |
| jgriffith | xyang: patrickeast_ I'll get some stuff cleaned in the spec barring any huge concerns from HP, IBM or others | 16:54 |
| xyang | sounds good | 16:54 |
| jgriffith | and then start working on some prototypes that I can share with everyone | 16:54 |
| jgriffith | we can all collaborate on this as it goes along is my thought | 16:54 |
| *** markvoelker has joined #openstack-cinder | 16:54 | |
| jgriffith | beauty of github | 16:54 |
| jgriffith | and TBF the initial implementation was done the same way | 16:55 |
| jgriffith | but I think many of us were too distracted with other things to give it as much attention as it deserved | 16:55 |
| jgriffith | especially me :( | 16:55 |
| *** e0ne is now known as e0ne_ | 16:56 | |
| patrickeast_ | jgriffith: speaking of other things to distract from this… have you had a chance to look through my spec for the initiator db table meta data thingy? | 16:57 |
| patrickeast_ | ^ https://review.openstack.org/#/c/153333/ | 16:58 |
| jgriffith | I have as a matter of fact | 16:58 |
| jgriffith | patrickeast_: for the most part I think it looks really sane | 16:58 |
| jgriffith | patrickeast_: one question I had for you... that may be in the spec and I just missed it | 16:58 |
| *** nileshb has joined #openstack-cinder | 16:58 | |
| jgriffith | patrickeast_: how does the driver access the info? | 16:59 |
| *** markvoelker has quit IRC | 16:59 | |
| jgriffith | patrickeast_: ie, how does a driver know that an entry in that table belongs to it? | 16:59 |
| *** annashen has quit IRC | 16:59 | |
| jgriffith | patrickeast_: FK? | 16:59 |
| patrickeast_ | jgriffith: ah yea | 16:59 |
| patrickeast_ | jgriffith: so that part is a little vauge | 16:59 |
| patrickeast_ | jgriffith: my thought was the manager looks up data for the initiator and passes it to the driver, at which point the driver can decide what data it cares about | 17:00 |
| patrickeast_ | the reasoning being that it would be much more restrictive/hard to have the manger/db query make that decision | 17:00 |
| *** e0ne_ is now known as e0ne | 17:00 | |
| patrickeast_ | there is of course a performance hit | 17:01 |
| patrickeast_ | if there is a ton of junk saved for some initiator | 17:01 |
| patrickeast_ | but i’m not sure if thats really too big of a deal | 17:01 |
| jgriffith | patrickeast_: so not to be a PITA but I'd like to avoid that if we could | 17:01 |
| jgriffith | patrickeast_: let me look at something... just a sec | 17:01 |
| patrickeast_ | i’m all for that, but i couldn’t think of a better solution that still worked for HA down the road | 17:02 |
| jgriffith | patrickeast_: hmm... that's tricky | 17:03 |
| jgriffith | patrickeast_: the only place that it would work is an FK in the service table | 17:03 |
| jgriffith | patrickeast_: I think | 17:03 |
| jgriffith | patrickeast_: as far as the HA piece, you can have the same info for both hosts no? | 17:03 |
| patrickeast_ | jgriffith: hehe yea, but this goes back down the road of both drivers needing to be able to get the same value | 17:04 |
| jgriffith | patrickeast_: oh... understood | 17:04 |
| jgriffith | patrickeast_: but in HA I'd say just like replication... | 17:05 |
| jgriffith | patrickeast_: create some association between the two that the driver knows about | 17:05 |
| jgriffith | patrickeast_: build on that foundation | 17:05 |
| jgriffith | patrickeast_: so iterating through items in a DB return has bit us in the past | 17:05 |
| jgriffith | patrickeast_: it's just really inefficient IMO and it's arbitrary | 17:06 |
| patrickeast_ | jgriffith: yea, I agree | 17:06 |
| jgriffith | patrickeast_: I like explicit when possible | 17:06 |
| jgriffith | patrickeast_: so I think the FK in the Service table might be a good way to do this | 17:06 |
| patrickeast_ | jgriffith: i kind of like the idea of the driver being able to specify a query parameter, almost like a vendor prefix | 17:06 |
| patrickeast_ | jgriffith: so like you said they can be linked through the config files | 17:06 |
| jgriffith | patrickeast_: and when we do get to HA we can easily put something in the conf file to let two backends know they're linked | 17:07 |
| jgriffith | patrickeast_: yeah... that sounds good to me | 17:07 |
| patrickeast_ | jgriffith: its a bit more free-form then the service name | 17:07 |
| jgriffith | patrickeast_: honestly, the way that shapes up it goes back to the possibility of just "custom" KV pairs for driver usage being possible | 17:07 |
| jgriffith | patrickeast_: ohhh... | 17:08 |
| jgriffith | patrickeast_: interesting | 17:08 |
| jgriffith | patrickeast_: so you're saying use a label in the conf file | 17:08 |
| jgriffith | patrickeast_: skip the FK altogether and query on that label | 17:08 |
| patrickeast_ | jgriffith: yep | 17:08 |
| jgriffith | patrickeast_: hmm... that's interesting | 17:08 |
| jgriffith | patrickeast_: I kinda like that | 17:09 |
| jgriffith | patrickeast_: and default for that label could be backend_name | 17:09 |
| jgriffith | patrickeast_: haha | 17:09 |
| patrickeast_ | jgriffith: yea, thats even where i started on that path | 17:09 |
| jgriffith | patrickeast_: actually, if you do that you're covered | 17:09 |
| jgriffith | patrickeast_: if it's backend name you're going to have the same entry no matter how many backends you have | 17:10 |
| jgriffith | patrickeast_: in other words... second Pure backend comes online, checks the table and says "hey, I've already got the initiator info, move along" | 17:10 |
| jgriffith | patrickeast_: let's do that :) | 17:10 |
| patrickeast_ | jgriffith: sounds good to me | 17:10 |
| jgriffith | patrickeast_: awesome! | 17:10 |
| patrickeast_ | jgriffith: ill update the spec with that | 17:11 |
| jgriffith | patrickeast_: cool, thanks | 17:11 |
| patrickeast_ | jgriffith: no no, thank you | 17:11 |
| jgriffith | patrickeast_: the only other thing worth considering is going back to the "generic" data | 17:11 |
| patrickeast_ | solving the hard parts for me ;) | 17:11 |
| jgriffith | patrickeast_: but I don't want to throw that wrench into the works | 17:11 |
| jgriffith | patrickeast_: it's likely to get all sorts of scrutiny :) | 17:11 |
| patrickeast_ | jgriffith: isn’t the general concensous that generic data is bad? | 17:11 |
| jgriffith | patrickeast_: and there's my point :) | 17:11 |
| jgriffith | patrickeast_: just depends on who you ask | 17:12 |
| jgriffith | patrickeast_: so go with what you have | 17:12 |
| jgriffith | patrickeast_: I'm in a different camp on the generic data, BUT the problem is abuse | 17:12 |
| jgriffith | patrickeast_: and no way to prevent it other than review which is WAY too subjective | 17:12 |
| jgriffith | patrickeast_: update that spec and I'm happy to approve it | 17:13 |
| patrickeast_ | jgriffith: yea i agree, its kind of a scary can of worms | 17:13 |
| jgriffith | indeed | 17:13 |
| jgriffith | can o'worms, slippery slope, road to hell | 17:13 |
| jgriffith | popular quotes that have come up on the topic | 17:13 |
| patrickeast_ | haha yep | 17:14 |
| patrickeast_ | jgriffith: i’ll post an update a little later this afternoon… going to go enjoy some sunshine while i can | 17:14 |
| jgriffith | haha! Rare moments in Seattle eh | 17:14 |
| *** coolsvap is now known as coolsvap_ | 17:14 | |
| jgriffith | enjoy | 17:14 |
| *** jdurgin has joined #openstack-cinder | 17:21 | |
| *** _cjones_ has joined #openstack-cinder | 17:23 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Fix logging guideline violations in volume/api.py https://review.openstack.org/155894 | 17:23 |
| *** _cjones_ has quit IRC | 17:24 | |
| *** _cjones_ has joined #openstack-cinder | 17:24 | |
| *** anshul has quit IRC | 17:25 | |
| *** pck has quit IRC | 17:25 | |
| *** scottda has quit IRC | 17:25 | |
| *** tellesnobrega has quit IRC | 17:25 | |
| *** number80 has quit IRC | 17:25 | |
| *** rmstar has quit IRC | 17:25 | |
| *** mkoderer has quit IRC | 17:25 | |
| *** mtreinish has quit IRC | 17:25 | |
| *** strictlyb has quit IRC | 17:25 | |
| *** Anticimex has quit IRC | 17:25 | |
| *** guitarzan has quit IRC | 17:25 | |
| *** gugl2 has quit IRC | 17:25 | |
| *** SamYaple has quit IRC | 17:25 | |
| *** anish has quit IRC | 17:25 | |
| *** SergeyLukjanov has quit IRC | 17:25 | |
| *** eikke has quit IRC | 17:25 | |
| *** gpocentek has quit IRC | 17:25 | |
| *** anshul has joined #openstack-cinder | 17:31 | |
| *** pck has joined #openstack-cinder | 17:31 | |
| *** scottda has joined #openstack-cinder | 17:31 | |
| *** number80 has joined #openstack-cinder | 17:31 | |
| *** tellesnobrega has joined #openstack-cinder | 17:31 | |
| *** rmstar has joined #openstack-cinder | 17:31 | |
| *** mkoderer has joined #openstack-cinder | 17:31 | |
| *** mtreinish has joined #openstack-cinder | 17:31 | |
| *** 64MABZ0GB has joined #openstack-cinder | 17:31 | |
| *** Anticimex has joined #openstack-cinder | 17:31 | |
| *** guitarzan has joined #openstack-cinder | 17:31 | |
| *** gugl2 has joined #openstack-cinder | 17:31 | |
| *** SamYaple has joined #openstack-cinder | 17:31 | |
| *** anish has joined #openstack-cinder | 17:31 | |
| *** SergeyLukjanov has joined #openstack-cinder | 17:31 | |
| *** eikke has joined #openstack-cinder | 17:31 | |
| *** gpocentek has joined #openstack-cinder | 17:31 | |
| *** ronis has joined #openstack-cinder | 17:35 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder-specs: Volume get and list should be able get detailed views https://review.openstack.org/156292 | 17:35 |
| *** harlowja_at_home has joined #openstack-cinder | 17:37 | |
| jungleboyj | eharney: Ping | 17:38 |
| eharney | jungleboyj: hi | 17:38 |
| jungleboyj | eharney: Hey, long time no chat. | 17:38 |
| eharney | jungleboyj: indeed :) | 17:39 |
| jungleboyj | eharney: Have you done anything with RHEL 7 and sharing out volumes via iSCSI? | 17:39 |
| eharney | jungleboyj: well i know a bit about LVM+LIO on it | 17:39 |
| *** mkerrin has quit IRC | 17:39 | |
| *** nellysmitt has joined #openstack-cinder | 17:40 | |
| jungleboyj | eharney: Ok, you are further than me. I have a HyperV node that seems to be failing because I think CHAP is enabled, but I don't see anywhere that stuff like that is cmnfigured. | 17:40 |
| jungleboyj | *configured | 17:40 |
| eharney | jungleboyj: so when you create an export in the LVM driver, chap credentials are created for the LIO target | 17:41 |
| eharney | jungleboyj: Cinder does this with the rtstool script in Cinder, as a user you'd look at these things w/ targetcli | 17:41 |
| jungleboyj | eharney: Ok, so that is happening by default. That was what it looked like to me. | 17:41 |
| jungleboyj | eharney: Right. | 17:41 |
| *** nileshb has quit IRC | 17:42 | |
| *** karimb has quit IRC | 17:42 | |
| eharney | jungleboyj: from the point of view of the compute node, it shouldn't be different from tgtd AFAIK | 17:43 |
| jungleboyj | Was tgt setting chap credentials by default? | 17:43 |
| jungleboyj | I don't know that it was. | 17:44 |
| eharney | yes | 17:45 |
| eharney | well, i thought... | 17:45 |
| jgriffith | jungleboyj: what version of cinder are you working with? | 17:45 |
| jungleboyj | eharney: Regardless, there isn't an option to disable setting up CHAP? | 17:45 |
| eharney | jungleboyj: nope | 17:45 |
| jungleboyj | jgriffith: This is Kilo. | 17:46 |
| jungleboyj | Looks like a late January build though. | 17:46 |
| jgriffith | jungleboyj: put a debug in volume/targets/lio at the iscsi target create | 17:47 |
| jgriffith | jungleboyj: you can see the chap setting that way | 17:47 |
| jgriffith | jungleboyj: or instrument the code with a log statement | 17:47 |
| jungleboyj | jgriffith: Ok. | 17:47 |
| jgriffith | jungleboyj: you need to make sure you have the iqn's set up from initiator too | 17:47 |
| jgriffith | IIRC | 17:47 |
| jgriffith | but eharney knows way more than I do about LIO | 17:48 |
| jungleboyj | jgriffith: Well, from the debug I was getting it looked like CHAP was being set up. Which explains why I am getting an error from the HyperV node. It can't do CHAP. | 17:49 |
| jgriffith | jungleboyj: hehe | 17:49 |
| jgriffith | sorry... so your question was if you can disable use of chap | 17:50 |
| jungleboyj | jgriffith: Yes, I wasn't seeing an option for that. Was trying to confirm. | 17:50 |
| jungleboyj | jgriffith: And you wouldn't laugh if you had to deal with HyperV. :-) | 17:50 |
| eharney | jungleboyj: you'll see in cinder/cmd/rtstool that it sets the "authentication" attribute to 1, so it's always required | 17:50 |
| jgriffith | jungleboyj: eharney we should clean things up a bit, as the conf file has an entry to set chap to None | 17:52 |
| jgriffith | http://54.164.167.86/solidfire-ci-logs/cinder-conf-samples/cinder.conf.sample.txt | 17:52 |
| jgriffith | CHAP username to use for iSCSI Targets | 17:52 |
| *** briancline has quit IRC | 17:52 | |
| *** briancline has joined #openstack-cinder | 17:52 | |
| jgriffith | added for scst I think (at my request) | 17:53 |
| jgriffith | although just documenting some drivers always use chap is probably fine too | 17:54 |
| *** markvoelker has joined #openstack-cinder | 17:56 | |
| eharney | jgriffith: hmm, seems like a good idea to clean things up for sure. I think LIO is a little different in that it uses different CHAP creds for every target, too | 17:56 |
| jgriffith | eharney: yeah, there are other drivers that do that, but I think it's unique in the "volume/targets" space | 17:57 |
| eharney | er, yeah, i meant per-volume actually | 17:57 |
| jgriffith | eharney: yeah, I'm with ya | 17:57 |
| jgriffith | eharney: my driver does the same thing... but that's completely different | 17:58 |
| jgriffith | anyway, just a potential clean up | 17:58 |
| *** markvoelker has quit IRC | 18:01 | |
| jungleboyj | eharney: jgriffith Thanks for the pointers. Makes sense to have CHAP enabled by default. HyperV is just limited there. | 18:04 |
| *** e0ne has quit IRC | 18:05 | |
| *** yuriy_n17 has quit IRC | 18:07 | |
| *** EmilienM is now known as EmilienM|afk | 18:07 | |
| *** rushiagr is now known as rushiagr_away | 18:11 | |
| openstackgerrit | Patrick East proposed openstack/cinder-specs: Add DB table for driver private data https://review.openstack.org/153333 | 18:12 |
| *** afazekas has quit IRC | 18:13 | |
| *** coolsvap_ is now known as coolsvap | 18:25 | |
| casusbelli | jgriffith: Hi! Do you have a min regarding sos-ci setup questions? | 18:29 |
| casusbelli | jgriffith: (no is a valid answer. :) ) | 18:30 |
| openstackgerrit | Thang Pham proposed openstack/cinder: Snapshot object https://review.openstack.org/133566 | 18:32 |
| openstackgerrit | Thang Pham proposed openstack/cinder: Cinder objects base https://review.openstack.org/131873 | 18:32 |
| *** patrickeast_ has quit IRC | 18:34 | |
| *** MasterPiece has joined #openstack-cinder | 18:38 | |
| jgriffith | casusbelli: hey... sure | 18:39 |
| *** jordanP has quit IRC | 18:40 | |
| *** ebalduf has quit IRC | 18:40 | |
| *** Mandell has joined #openstack-cinder | 18:45 | |
| *** ybathia has joined #openstack-cinder | 18:48 | |
| ybathia | Hi, I have a small query.. Why are quotas expected to be in gigabytes in this file https://github.com/openstack/cinder/blob/master/cinder/exception.py? Can the quotas not be smaler or way bigger than gigabytes? | 18:51 |
| jgriffith | ybathia: the resolution of a volume size is Gigabytes | 18:51 |
| jgriffith | ybathia: so it sort of seems natural | 18:51 |
| jgriffith | ybathia: so no smaller, but sure on bigger (104857600000 Gigabytes) | 18:52 |
| jgriffith | ybathia: it's just the default unit of measure | 18:52 |
| ybathia | Ok..Got it. Submitted a patch in horizon and got asked this question. So, wanted to know the reason from the cinder guys.. | 18:53 |
| ybathia | Thanks | 18:53 |
| jgriffith | ybathia: sure | 18:54 |
| *** e0ne has joined #openstack-cinder | 18:56 | |
| *** markvoelker has joined #openstack-cinder | 18:57 | |
| *** annashen has joined #openstack-cinder | 19:00 | |
| *** markvoelker has quit IRC | 19:02 | |
| casusbelli | jgriffith: ick, was afk, sry. | 19:04 |
| casusbelli | jgriffith: I'm doing a sos-ci setup but i'm a bit confused about which OpenStack, especially neutron / networks setup it expects. | 19:05 |
| *** annashen has quit IRC | 19:05 | |
| casusbelli | jgriffith: What setup is your default scenario. sos-ci running on a controller? In my setup the VMs require a floating ip to be contactable from the ci but that does not seem to be the case in your default setup... | 19:06 |
| casusbelli | jgriffith: questions questions questions... :) | 19:07 |
| *** Yogi1 has joined #openstack-cinder | 19:07 | |
| jgriffith | casusbelli: so look in templates | 19:08 |
| jgriffith | casusbelli: that the local.conf file I send over and use for stack.sh | 19:08 |
| jgriffith | casusbelli: it doesn't use neutron in mine | 19:08 |
| jgriffith | casusbelli: but you could modify it to do so if you like of course | 19:08 |
| jgriffith | casusbelli: you're correct, I don't require floating IP's either | 19:09 |
| jgriffith | casusbelli: and as such I don't register those as a variable | 19:09 |
| jgriffith | casusbelli: you can modify that pretty easily in the "boot_openstack_instance.yml" task | 19:09 |
| casusbelli | jgriffith: yep, already tampered with the yml scripts | 19:10 |
| jgriffith | casusbelli: either add a task to assing a floating ip, or set your nova.conf on the cloud you're running against to always assing floating IP and just parse itout | 19:10 |
| jgriffith | it out | 19:10 |
| jgriffith | casusbelli: the networking info in the instance object is kind of atrocious though | 19:10 |
| casusbelli | jgriffith: yep, that's why i wanted to ask. :-D | 19:10 |
| jgriffith | casusbelli: you'll want to run the task by itself and dump the object to figure out how to parse it | 19:11 |
| casusbelli | jgriffith: hmm, ok | 19:11 |
| jgriffith | casusbelli: hehe.. yeah, it's somewhat madening | 19:11 |
| casusbelli | jgriffith: How do you avoid this in your setup? | 19:11 |
| jgriffith | casusbelli: I don't need floating IP's for anything | 19:11 |
| *** Yogi1 has quit IRC | 19:11 | |
| *** anshul has quit IRC | 19:11 | |
| jgriffith | casusbelli: everything is run "on" the machine I have my cloud deployed on | 19:11 |
| jgriffith | casusbelli: devstack inside of devstack | 19:11 |
| jgriffith | so to speak | 19:11 |
| casusbelli | jgriffith: that's what i gathered. Are the private IPs of your VMs reachable from the sos-ci node? | 19:11 |
| *** aix has quit IRC | 19:11 | |
| jgriffith | so private IP's are all I need | 19:12 |
| jgriffith | casusbelli: yeah, so the sos-ci node IS the devstack node that's running my CI | 19:12 |
| casusbelli | Jgriffith: hmm, irc hickup | 19:12 |
| jgriffith | casusbelli: I "used" to have it split out and running on a VM in my OpenStack cloud | 19:12 |
| casusbelli | jgriffith: aaah, ok | 19:12 |
| jgriffith | casusbelli: that would solve the problem for you as well | 19:12 |
| jgriffith | casusbelli: ie instead of an sos-ci node, you have an sos-ci instance | 19:13 |
| jgriffith | casusbelli: which honestly for a number of reasons was actually a "better" way to go | 19:13 |
| jgriffith | casusbelli: but I changed it a while back while working on some things and never switched it back :) | 19:13 |
| jgriffith | casusbelli: so what's nice about using instances other than the whole "IP access" thing | 19:14 |
| jgriffith | is you can have several of them doing different things | 19:14 |
| casusbelli | jgriffith: hmm, ok. That give's me some food for thought... | 19:14 |
| *** Yogi1 has joined #openstack-cinder | 19:14 | |
| jgriffith | I had one running LVM, one with LVM thin and one with SolidFire | 19:15 |
| jgriffith | casusbelli: I highly recommend the instance model FWIW | 19:15 |
| jgriffith | casusbelli: plus you can can do things like "experimental" setups easily that way | 19:15 |
| jgriffith | running things in parallel | 19:15 |
| jgriffith | casusbelli: not sure if that answered your questions or not | 19:16 |
| casusbelli | jgriffith: when running a sos-ci node, where does it access the nova client? I tried that but the ansible nova module didn't work mit RDO based controller | 19:16 |
| jgriffith | casusbelli: I should probably draw out the infrastructure I'm using in teh README | 19:16 |
| casusbelli | jgriffith: THat's definitely a good idea | 19:16 |
| jgriffith | casusbelli: it *should* because if you look I'm just specifying the rc file | 19:16 |
| jgriffith | casusbelli: should work with RDO although I haven't tried it | 19:17 |
| jgriffith | casusbelli: my setup is an all in one OpenStack deployment on a server | 19:17 |
| casusbelli | jgriffith: i was surprised, too. It works when run against localhost but not when run from a VM instance vs. the controller node | 19:17 |
| jgriffith | casusbelli: I set all of the openstack creds in the sos-ci code (keystone endpoint, login, user, tenant) | 19:17 |
| jgriffith | casusbelli: OH! | 19:17 |
| jgriffith | casusbelli: did you install the clients and everything? | 19:18 |
| jgriffith | casusbelli: and are your endpoints accessible from the instance? | 19:18 |
| casusbelli | jgriffith: on the vm? no | 19:18 |
| casusbelli | but on the controller? yep | 19:18 |
| jgriffith | casusbelli: there's your problem | 19:18 |
| jgriffith | casusbelli: it's just "openstack" at that point | 19:18 |
| casusbelli | jgriffith: Well, normally the point of an ansible module is to be executable on the remote host, not on the local one. and i ran the ansible scripts with the controller as the remote hsot | 19:19 |
| jgriffith | casusbelli: wherever sos-ci is running, you have to have the clients installed and be able to access with your openstack creds | 19:19 |
| casusbelli | jgriffith: *host | 19:19 |
| casusbelli | jgriffith: OH | 19:19 |
| jgriffith | casusbelli: no, no... that's what I'm saying you don't *have* to be on localhost | 19:19 |
| *** ebalduf has joined #openstack-cinder | 19:19 | |
| jgriffith | casusbelli: but you have to enable it with the tools you need to do it's thing | 19:19 |
| jgriffith | casusbelli: and I'd argue about the localhost part | 19:19 |
| casusbelli | jgriffith: ok, i see that now | 19:20 |
| jgriffith | casusbelli: so localhost is great for doing things like calling "nova boot --image-id xxxx....." | 19:20 |
| jgriffith | casusbelli: localhost is where your python-novaclient is etc | 19:20 |
| jgriffith | casusbelli: remote tasks are installing devstack on the resultant instance, configuring/running tempest and gathering results | 19:21 |
| casusbelli | jgriffith: ok, i'll review the whole setup with this info. | 19:21 |
| casusbelli | jgriffith: Thanks for the help!! | 19:21 |
| jgriffith | casusbelli: np | 19:21 |
| jgriffith | lemme know how it goes | 19:21 |
| casusbelli | jgriffith: will do! | 19:21 |
| jgriffith | casusbelli: ya know.. maybe I'll write an ansible task to "deploy" and setup an sos-ci machine | 19:22 |
| jgriffith | casusbelli: that way it will do all of this "for you" | 19:23 |
| jgriffith | lunchtime.. bbl | 19:23 |
| casusbelli | jgriffith: i'm working on that, actually | 19:23 |
| casusbelli | jgriffith: bon appetite | 19:23 |
| *** coolsvap is now known as coolsvap_ | 19:31 | |
| *** lpetrut has quit IRC | 19:41 | |
| *** kaufer has quit IRC | 19:48 | |
| *** xyang has quit IRC | 19:48 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Kilo Consistency Group API update https://review.openstack.org/144561 | 19:49 |
| *** MasterPiece has quit IRC | 19:49 | |
| *** Lee1092 has quit IRC | 19:50 | |
| *** Mandell has quit IRC | 19:53 | |
| *** markvoelker has joined #openstack-cinder | 19:58 | |
| *** diemt has joined #openstack-cinder | 19:58 | |
| *** kaufer has joined #openstack-cinder | 19:59 | |
| *** markvoelker has quit IRC | 20:03 | |
| *** EmilienM|afk is now known as EmilienM | 20:04 | |
| e0ne | jgriffith: hi! one small question about API and cinder client from me | 20:06 |
| e0ne | any thouts why do we have 'detatiled=True' by default in client code: https://github.com/openstack/python-cinderclient/blob/master/cinderclient/v2/volumes.py#L236? | 20:07 |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Clear migration_status from a destination volume if migration fails https://review.openstack.org/154739 | 20:10 |
| guitarzan | e0ne: probably because the non detail view is very boring | 20:12 |
| guitarzan | e0ne: and you'd just want to make a detail call after that anyway | 20:12 |
| e0ne | guitarzan: in such case, having 2 api methods don't make sence | 20:13 |
| guitarzan | oh, I'm not going to get into any arguments about whether "show" is useful :) | 20:13 |
| guitarzan | or index I suppose | 20:14 |
| e0ne | guitarzan: my question is related not only for 'volume get' method | 20:14 |
| *** _cjones_ has quit IRC | 20:15 | |
| guitarzan | hmm, create returns the summary view as well? | 20:16 |
| guitarzan | nah, I must be reading something wrong | 20:17 |
| *** xyang has joined #openstack-cinder | 20:19 | |
| openstackgerrit | Anish Bhatt proposed openstack/cinder: Add support for chiscsi iscsi helper https://review.openstack.org/147758 | 20:22 |
| *** lpetrut has joined #openstack-cinder | 20:22 | |
| *** Mandell has joined #openstack-cinder | 20:23 | |
| anish | jgriffith: e0ne slight change in code, can I get the +2's back please :) | 20:23 |
| *** patrickeast has joined #openstack-cinder | 20:24 | |
| e0ne | anish: lets wait for Jenkins | 20:24 |
| anish | yes, o'course | 20:26 |
| jgriffith | anish: sorry... that was my last one | 20:29 |
| e0ne | :) | 20:29 |
| jgriffith | anish: e0ne think this is good to go no? | 20:29 |
| jgriffith | minor changes, was pretty much done prior to them | 20:30 |
| anish | yes, it reused more code that mtanino added instead of duplicating and moved a debug line around | 20:30 |
| mtanino | anish: Thanks! | 20:30 |
| jgriffith | yeah, I'll +2/A after jenkins passes | 20:31 |
| anish | woot | 20:31 |
| e0ne | jgriffith: +2 from me | 20:31 |
| *** harlowja_at_home has quit IRC | 20:31 | |
| e0ne | jgriffith: did you see my question about client and api above? | 20:32 |
| jgriffith | e0ne: no... sorry, just got back from lunch | 20:32 |
| * jgriffith scrolls back | 20:32 | |
| e0ne | jgriffith: np. i'm confused a bit how we use our api | 20:33 |
| jgriffith | e0ne: hmmm... | 20:34 |
| jgriffith | e0ne: well that's kinda lame | 20:34 |
| e0ne | jgriffith: i hope, i've missed something. looks like we've got only 'detailed views' | 20:34 |
| jgriffith | e0ne: yeah, it's a pointless option it appears | 20:36 |
| jgriffith | oh... wait | 20:36 |
| jgriffith | looking at the wrong thing | 20:36 |
| jgriffith | e0ne: nope, I wasn't | 20:36 |
| jgriffith | e0ne: it's always True | 20:36 |
| jgriffith | e0ne: suppose you could set it if you were using something other than the client | 20:37 |
| e0ne | "grep -r 'detailed=False' . | wc -l" - 6 lines found | 20:38 |
| jgriffith | ie curl | 20:38 |
| jgriffith | e0ne: yeah, but there's no mechanism in the client to set it that I can see | 20:38 |
| guitarzan | anyone sane will use their own client ;) | 20:38 |
| jgriffith | guitarzan: ouch | 20:38 |
| jgriffith | guitarzan: so you're saying our client is no good? | 20:39 |
| guitarzan | can't paste in the curl debugged commands anymore :( | 20:39 |
| e0ne | jgriffith: i want to added in scope of https://review.openstack.org/#/c/156292/ | 20:39 |
| guitarzan | jgriffith: nah, just griping on the keystoneclient "security" features that keep getting added | 20:39 |
| jgriffith | guitarzan: ahhh | 20:39 |
| e0ne | jgriffith: but not sure that it is good K | 20:39 |
| guitarzan | those folks obviously never have to use openstack | 20:39 |
| jgriffith | guitarzan: there's a lot of that going on in OpenStack these days IMO | 20:39 |
| jgriffith | guitarzan: including in Cinder :( | 20:39 |
| guitarzan | I was trying to see the /index vs /detail but can't just use cinder --debug to try it | 20:40 |
| e0ne | jgriffith: thanks for clarification it. i'm a sad a bit: we've got a lot of unused code in client:( | 20:40 |
| jgriffith | e0ne: so that spec you have I'm in favor of, think you and I discussed this before | 20:41 |
| jgriffith | e0ne: only difference is I said show, you say detailed | 20:41 |
| jgriffith | e0ne: which if we make detailed actually "be" something is good with me | 20:41 |
| e0ne | jgriffith: yes. but for now we use only 'detailed' views. | 20:42 |
| jgriffith | e0ne: right | 20:42 |
| e0ne | jgriffith: so changing it requires changes both in server and client APIs | 20:42 |
| jgriffith | e0ne: yeah, as awful as this sounds... maybe we change the default, expose detailed and move on? | 20:44 |
| e0ne | jgriffith: sounds reasonable to leave current behaviour as default and move everything else to detailed | 20:45 |
| *** dulek has joined #openstack-cinder | 20:45 | |
| jgriffith | e0ne: yeah, that sounds good since it's what we expose anyway | 20:47 |
| jgriffith | e0ne: the only catch I tought of is people like guitarzan that have their own clients | 20:47 |
| jgriffith | e0ne: where detailed can actually be used for something ;) | 20:47 |
| *** Longgeek has quit IRC | 20:48 | |
| anish | huh, something up with the third party CIs ? | 20:50 |
| anish | normally I'm flooded within the first 20 min | 20:50 |
| anish | today, nothing | 20:50 |
| guitarzan | jgriffith: nah, don't worry about me, I don't really use my own client for much :) | 20:51 |
| jgriffith | guitarzan: lol... you just use Jedi Mind Trick for Cinder calls :) | 20:52 |
| openstackgerrit | Patrick East proposed openstack/cinder-specs: Add DB table for driver private data https://review.openstack.org/153333 | 20:52 |
| guitarzan | jgriffith: my "client" for that is called requests :) | 20:52 |
| guitarzan | also, TIL that the volume manager exits if there are no periodic tasks | 20:53 |
| guitarzan | I didn't really expect that | 20:53 |
| guitarzan | jgriffith: doing the "maintenance mode" signal handler thing is really easy | 20:53 |
| guitarzan | jgriffith: but I can't cancel the periodic tasks or the launcher keeps restarting the service :) | 20:54 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder-specs: Volume get and list should be able get detailed views https://review.openstack.org/156292 | 20:54 |
| e0ne | jgriffith, guitarzan: "3rd party clients" are always risky until API freezed | 20:55 |
| *** patrickeast has quit IRC | 20:55 | |
| sigmavirus24 | guitarzan: I'm glad to hear someone likes requests =P | 20:56 |
| jgriffith | guitarzan: you might be on to something :) | 20:56 |
| *** _cjones_ has joined #openstack-cinder | 20:56 | |
| *** dulek has quit IRC | 20:56 | |
| guitarzan | jgriffith: I'll have a simple POC diff for you and DuncanT to look at :) | 20:56 |
| *** patrickeast has joined #openstack-cinder | 20:56 | |
| guitarzan | I'm curious if he thinks I'm crazy with this idea | 20:56 |
| openstackgerrit | Anish Bhatt proposed openstack/cinder: Add support for chiscsi iscsi helper https://review.openstack.org/147758 | 20:57 |
| jgriffith | guitarzan: I'll be curious to see it. He'll be a better person to ask if it's crazy or not | 20:57 |
| *** hodos has quit IRC | 20:58 | |
| jgriffith | anish: my +2/A is now withdrawn since you added copyright | 20:58 |
| *** markvoelker has joined #openstack-cinder | 20:59 | |
| anish | this is the part where I really wish I knew if you were joking :) | 20:59 |
| jgriffith | anish: don't worry, I was joking :) | 21:01 |
| anish | fwiw, I have zero idea what copyright + reserved + apache even implies | 21:01 |
| *** annashen has joined #openstack-cinder | 21:01 | |
| * anish is following directions from above | 21:01 | |
| guitarzan | anish: "looks like all the other files" == good :) | 21:01 |
| guitarzan | imo | 21:01 |
| *** devlaps has joined #openstack-cinder | 21:01 | |
| anish | guitarzan: I'm pretty sure that's what "above" went with as well | 21:02 |
| *** patrickeast has quit IRC | 21:02 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder-specs: Volume get and list should be able get detailed views https://review.openstack.org/156292 | 21:04 |
| *** markvoelker has quit IRC | 21:04 | |
| *** annashen has quit IRC | 21:06 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder-specs: Volume get and list should be able get detailed views https://review.openstack.org/156292 | 21:12 |
| e0ne | jgriffith: is any change to get spec ^^ approved for K? | 21:13 |
| anish | haha. you should see nova right now, it's a stream of "spec not approved for K" | 21:14 |
| anish | and here.. | 21:14 |
| e0ne | :) | 21:15 |
| jgriffith | e0ne: you're going to have to talk to thingee and other cores on that | 21:16 |
| jgriffith | e0ne: IMO I'd be ok with it | 21:16 |
| e0ne | thanks! | 21:16 |
| jgriffith | e0ne: because it's more of a "fix" and "improvement" on something we have currently that isn't quite right | 21:16 |
| e0ne | thingee, hemna, jungleboyj, winston-d, xyang: could you please take a look on https://review.openstack.org/156292? | 21:18 |
| e0ne | jgriffith: agree. we spent a lot of time with it while fixint a 'bug' instead of implemented it like a bp | 21:18 |
| *** xyang has quit IRC | 21:18 | |
| *** xyang has joined #openstack-cinder | 21:19 | |
| openstackgerrit | xing-yang proposed openstack/python-cinderclient: Kilo Consistency Group CLI update https://review.openstack.org/144567 | 21:19 |
| jgriffith | e0ne: minor point on that spec if you don't mind | 21:21 |
| *** _cjones_ has quit IRC | 21:22 | |
| jgriffith | e0ne: I'd like it if we were very explicity about what happens to the existing views vs the new views etc | 21:22 |
| jgriffith | e0ne: make sense? | 21:22 |
| *** jwcroppe has joined #openstack-cinder | 21:22 | |
| e0ne | jgriffith: absolutely, i'll add it | 21:23 |
| *** _cjones_ has joined #openstack-cinder | 21:23 | |
| *** xyang has quit IRC | 21:23 | |
| jgriffith | e0ne: great! | 21:24 |
| jgriffith | e0ne: thanks! | 21:24 |
| *** xyang has joined #openstack-cinder | 21:31 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder-specs: Volume get and list should be able get detailed views https://review.openstack.org/156292 | 21:34 |
| *** annashen has joined #openstack-cinder | 21:36 | |
| e0ne | jgriffith: thanks for very detailed and reasonable feedback | 21:37 |
| *** lpetrut has quit IRC | 21:38 | |
| *** jungleboyj has quit IRC | 21:42 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder-specs: Volume get and list should be able get detailed views https://review.openstack.org/156292 | 21:42 |
| jgriffith | e0ne: for sure, thanks for the great spec | 21:46 |
| *** xyang has quit IRC | 21:47 | |
| *** Mandell has quit IRC | 21:47 | |
| e0ne | jgriffith: to be honest: it's only summarizing of irc and gerrit discussions | 21:48 |
| *** xyang has joined #openstack-cinder | 21:50 | |
| *** annashen has quit IRC | 21:52 | |
| jgriffith | e0ne: IMO those are the best specs | 21:53 |
| jgriffith | e0ne: and somebody following through and taking on the work is the hard part | 21:53 |
| jgriffith | e0ne: especially when it comes to writing specs, that's not *easy* IMHO | 21:53 |
| e0ne | jgriffith: imo, it means that we had a good community work | 21:54 |
| e0ne | jgriffith: and maybe it is because nobody likes to write specs:) | 21:54 |
| jgriffith | e0ne: haha... agree on both points | 21:55 |
| e0ne | :) | 21:55 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder-specs: Volume get and list should be able get detailed views https://review.openstack.org/156292 | 21:58 |
| kaufer | e0ne: I'm reading your spec, time for a question on it? | 21:59 |
| e0ne | kaufer: sure. but i'll be available about 15-20min more today | 22:00 |
| *** zzzeek has joined #openstack-cinder | 22:00 | |
| kaufer | Did you see my question in the Alternatives section about adding a --minimal param? | 22:00 |
| kaufer | I'm just curious if you considered that | 22:00 |
| openstackgerrit | Jon Bernard proposed openstack/cinder: Add support for generic volume migration https://review.openstack.org/135002 | 22:01 |
| zzzeek | jbernard / eharney : I’m running cinder-volume from a fresh devstack, and this is my current console output: http://paste.openstack.org/show/175492/. it doesn’t fail at all across mulitple runs, can you give me a clue what dependencies are needed to cause the failure seen in https://bugs.launchpad.net/cinder/+bug/1417018? OS, plugins, cinder.conf, existing state ? | 22:01 |
| openstack | Launchpad bug 1417018 in Cinder "Cinder encounters dbapi error: NoSuchColumnError: "Could not locate column in row for column '%(140070887032400 anon)s.volumes_id'"" [Undecided,New] | 22:01 |
| *** markvoelker has joined #openstack-cinder | 22:01 | |
| *** Mandell has joined #openstack-cinder | 22:02 | |
| *** aix has joined #openstack-cinder | 22:02 | |
| e0ne | kaufer: i'm sorry, i missed it | 22:02 |
| *** jamielennox is now known as jamielennox|away | 22:03 | |
| *** primechuck has quit IRC | 22:03 | |
| e0ne | kaufer: i didn't added anything about '--minimal' to spec because not sure that is is needed right now | 22:04 |
| e0ne | kaufer: i mean, i haven't got real use-cases where it is needed | 22:04 |
| kaufer | e0ne: Is the goal of the spec to execute non-detailed queries when the caller knows that they do not need all of the information? | 22:05 |
| *** markvoelker has quit IRC | 22:05 | |
| e0ne | kaufer: yes | 22:06 |
| e0ne | kaufer: and i don't want to break api compatibility with a current master | 22:06 |
| kaufer | e0ne: So are you proposed that the user would need to add --detailed when they needed all of that information? And if it did not supply it then it would not make a detailed query? | 22:07 |
| e0ne | kaufer: w/o --detailed flag user will got the same data as it is available in the current implementation | 22:08 |
| e0ne | kaufer: for now, i'm planning to add only snapshots info in a detailed view | 22:08 |
| *** thangp has quit IRC | 22:08 | |
| *** Longgeek has joined #openstack-cinder | 22:08 | |
| kaufer | e0ne: So then on the server the non-detailed view handler would become the detailed view handler? And then detailed view handler would add in the 'child_snapshots' field? | 22:10 |
| e0ne | kaufer: detailed and default views are mixed now | 22:10 |
| anish | e0ne: that was fast. do you get notified when jenkins passes ? | 22:10 |
| e0ne | kaufer: on the server-side, current detailed view will be moved to default and detailed will be extended with 'child_snapshots' | 22:12 |
| jbernard | eharney: did you hit the same issue, or was it just me? (nosuchcolumn error) | 22:12 |
| kaufer | e0ne: By default view you mean the summary function here? https://github.com/openstack/cinder/blob/master/cinder/api/v2/views/volumes.py#L41 | 22:12 |
| eharney | jbernard: i haven't hit it | 22:14 |
| *** jamielennox|away is now known as jamielennox | 22:14 | |
| *** kaufer has quit IRC | 22:15 | |
| e0ne | e0ne: yes, it isn't used via cinderclient now and should be safe | 22:15 |
| jbernard | zzzeek: maybe it's just my hacked up tree… i can give you my local.conf, a patch to apply, and an operation to perform and you should see it | 22:16 |
| e0ne | kaufer: ^^ | 22:16 |
| zzzeek | jbernard: ahha. OK, please do :) | 22:16 |
| *** kaufer has joined #openstack-cinder | 22:16 | |
| *** primechuck has joined #openstack-cinder | 22:16 | |
| e0ne | kaufer: also there is no reference to it in a api docs: http://developer.openstack.org/api-ref-blockstorage-v2.html | 22:16 |
| zzzeek | jbernard: the reporter of the issue was someone other than yourself, correct? so more than one person is getting it | 22:16 |
| e0ne | kaufer: (coped message) yes, it isn't used via cinderclient now and should be safe | 22:17 |
| *** diemt has quit IRC | 22:17 | |
| kaufer | e0ne: Sorry, I have a bad connection, what isn't used via cinderclient? | 22:17 |
| jbernard | zzzeek: yes, but im not completely sure they are the same issue | 22:18 |
| e0ne | kaufer: "the summary function" that you referenced | 22:18 |
| jbernard | zzzeek: this local.conf: http://sprunge.us/WSYa | 22:18 |
| zzzeek | jbernard: OK. yes your issue is something different symptom wise | 22:18 |
| jbernard | zzzeek: this patch: https://review.openstack.org/#/c/135002/ | 22:18 |
| zzzeek | jbernard: undo that patch and the error goes away ? | 22:19 |
| jbernard | zzzeek: i haven't had time to bisect or try different commit yet | 22:19 |
| zzzeek | jbernard: OK :) | 22:19 |
| kaufer | e0ne: The server will use the view handler to format any non-detailed API query, which is what the client would invoke | 22:19 |
| *** ronis has quit IRC | 22:19 | |
| zzzeek | jbernard: if i could have a clueless moment with devstack, for this local.conf do I have to run stack.sh again ? | 22:20 |
| jbernard | zzzeek: it doens't appear to happen in the gate for that patch, so it could very well be a problem with my setup | 22:20 |
| jbernard | zzzeek: yep, you'll have to rstack | 22:20 |
| e0ne | kaufer: didn't get yout point in the last message | 22:20 |
| zzzeek | jbernard: OK, but you are hitting one of those creepy mysql driver issues, woudl be good to know | 22:20 |
| kaufer | e0ne: The cinderclient flow uses that function, that's all | 22:21 |
| jbernard | zzzeek: i hit two differtn (but posibly related errors) 1: no such column error and 2: database connection reset error | 22:21 |
| zzzeek | jbernard: yeah | 22:21 |
| jbernard | zzzeek: every now and then :) | 22:21 |
| e0ne | kaufer: how does cinderclient use it? | 22:22 |
| jbernard | zzzeek: this particular vm is ubuntu 14.04, that may be relevant | 22:22 |
| zzzeek | jbernard: it could be | 22:22 |
| e0ne | kaufer: it calls /volumes/vol_id rest api method | 22:23 |
| e0ne | kaufer: wich renders detailed view on the server-side | 22:24 |
| kaufer | e0ne: Oh, for a show, yes | 22:24 |
| kaufer | e0ne: I thought that you meant for a list when removing the detail suffix from the API call | 22:25 |
| e0ne | kaufer: list works good on the server-side | 22:26 |
| e0ne | kaufer: but client calls it with 'detail' suffux everytime | 22:26 |
| *** jcru has quit IRC | 22:26 | |
| kaufer | yes | 22:27 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder-specs: Volume get and list should be able get detailed views https://review.openstack.org/156292 | 22:30 |
| e0ne | kaufer: so what is the benefit to leave it as is? are there a lot of people that use API directly w/o client? | 22:34 |
| e0ne | kaufer: i believe that only non-python users do it | 22:34 |
| kaufer | TBH, I'm not sure, I was just surprised to see that the non-detailed view would be going away | 22:35 |
| *** Yogi1 has quit IRC | 22:36 | |
| kaufer | e0ne: That is what you are proposing, right? | 22:36 |
| e0ne | kaufer: i'm also don't have stats for it but it has backward compatibility and hasn't performance impact on the server side | 22:37 |
| e0ne | kaufer: yes, it is | 22:37 |
| *** nellysmitt has quit IRC | 22:37 | |
| *** nellysmitt has joined #openstack-cinder | 22:38 | |
| kaufer | e0ne: So, to back up, you are trying to define a framework where more details (ie, child_snapshots) can be retrieved for a volume, right? | 22:39 |
| e0ne | kaufer: i'm really sorry. must go offline and sleep for a while. i'll be available tomorrow | 22:39 |
| kaufer | e0ne: sure, np, I'll collect my thoughts and put something into the spec review, g'night! | 22:40 |
| e0ne | kaufer: i didn't thanks that it is a framework, but it is something like you said | 22:40 |
| e0ne | s/thanks/thins | 22:41 |
| e0ne | think | 22:41 |
| *** annashen has joined #openstack-cinder | 22:41 | |
| *** Longgeek has quit IRC | 22:41 | |
| *** nellysmitt has quit IRC | 22:42 | |
| *** jdurgin has quit IRC | 22:43 | |
| *** ybathia has quit IRC | 22:44 | |
| *** mriedem has quit IRC | 22:46 | |
| *** afazekas has joined #openstack-cinder | 22:47 | |
| *** e0ne has quit IRC | 22:50 | |
| *** sdague has joined #openstack-cinder | 22:51 | |
| *** kaufer has quit IRC | 22:56 | |
| *** tsekiyama has quit IRC | 22:58 | |
| *** afazekas has quit IRC | 23:01 | |
| sdague | so... what do I need to change in default devstack to make this go away - http://logs.openstack.org/10/155410/2/check/check-tempest-dsvm-full/ccc45f8/logs/screen-c-vol.txt.gz?level=WARNING#_2015-02-13_15_50_36_282 ? | 23:02 |
| *** markvoelker has joined #openstack-cinder | 23:02 | |
| *** tsekiyama has joined #openstack-cinder | 23:03 | |
| *** markvoelker has quit IRC | 23:07 | |
| *** annashen has quit IRC | 23:08 | |
| *** chlong has joined #openstack-cinder | 23:09 | |
| *** annashen has joined #openstack-cinder | 23:10 | |
| *** ebalduf has quit IRC | 23:12 | |
| openstackgerrit | xing-yang proposed openstack/python-cinderclient: Add support to incremental backups in cinder https://review.openstack.org/112101 | 23:15 |
| *** xyang has quit IRC | 23:15 | |
| openstackgerrit | Jay Bryant proposed openstack/cinder: Sync policy module from oslo-incubator https://review.openstack.org/156412 | 23:15 |
| *** primechuck has quit IRC | 23:20 | |
| *** xyang has joined #openstack-cinder | 23:25 | |
| *** afazekas has joined #openstack-cinder | 23:30 | |
| *** annashen has quit IRC | 23:42 | |
| *** ebalduf has joined #openstack-cinder | 23:46 | |
| *** scottda_ has joined #openstack-cinder | 23:52 | |
| *** Mandell has quit IRC | 23:52 | |
| *** Mandell has joined #openstack-cinder | 23:54 | |
| *** scottda_ has quit IRC | 23:58 | |
| *** Mandell has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!