| *** bill_az has joined #openstack-cinder | 00:00 | |
| jgriffith | anish: gotta run | 00:00 |
|---|---|---|
| *** atan8 has joined #openstack-cinder | 00:00 | |
| jgriffith | anish: let's chat tomorrow on those failing unit tests if you don't get it sorted | 00:00 |
| *** dannywil_ has quit IRC | 00:00 | |
| *** annashen has quit IRC | 00:01 | |
| anish | jgriffith: sure, thanks ! | 00:01 |
| *** annashen has joined #openstack-cinder | 00:01 | |
| *** jseiler__ has joined #openstack-cinder | 00:01 | |
| *** dannywilson has joined #openstack-cinder | 00:02 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/145661 | 00:02 |
| *** gtt116_ has joined #openstack-cinder | 00:02 | |
| *** tsekiyam_ has joined #openstack-cinder | 00:02 | |
| *** madskier has joined #openstack-cinder | 00:03 | |
| *** scottda_ has joined #openstack-cinder | 00:04 | |
| *** chlong has joined #openstack-cinder | 00:05 | |
| *** Mandell has joined #openstack-cinder | 00:07 | |
| *** changbl has quit IRC | 00:08 | |
| *** ho has joined #openstack-cinder | 00:08 | |
| *** tsekiyama has quit IRC | 00:09 | |
| *** jseiler_ has quit IRC | 00:09 | |
| *** hypnosb has quit IRC | 00:09 | |
| *** ebalduf has quit IRC | 00:09 | |
| *** tellesnobrega has quit IRC | 00:09 | |
| *** gtt116 has quit IRC | 00:09 | |
| *** annashen has quit IRC | 00:10 | |
| *** annashen has joined #openstack-cinder | 00:10 | |
| *** Longgeek has quit IRC | 00:10 | |
| *** hypnosb has joined #openstack-cinder | 00:11 | |
| *** tellesnobrega has joined #openstack-cinder | 00:11 | |
| *** Mandell has quit IRC | 00:12 | |
| *** scottda_ has quit IRC | 00:12 | |
| *** tsekiyama has joined #openstack-cinder | 00:19 | |
| *** madskier has quit IRC | 00:19 | |
| *** madskier has joined #openstack-cinder | 00:20 | |
| *** tsekiyam_ has quit IRC | 00:21 | |
| *** tsekiyama has quit IRC | 00:23 | |
| *** madskier has quit IRC | 00:25 | |
| *** bswartz has joined #openstack-cinder | 00:26 | |
| *** david-lyle has joined #openstack-cinder | 00:28 | |
| *** tbarron has joined #openstack-cinder | 00:31 | |
| *** tbarron has quit IRC | 00:31 | |
| *** tbarron has joined #openstack-cinder | 00:32 | |
| *** EmilienM is now known as EmilienM|afk | 00:39 | |
| *** annashen has quit IRC | 00:44 | |
| *** ameade has joined #openstack-cinder | 00:44 | |
| *** fandi has joined #openstack-cinder | 00:45 | |
| *** Masahiro has joined #openstack-cinder | 00:46 | |
| *** zhiyan has joined #openstack-cinder | 00:48 | |
| *** cppforlife has joined #openstack-cinder | 00:50 | |
| *** EmilienM|afk has quit IRC | 00:52 | |
| *** sandywalsh_ has joined #openstack-cinder | 00:54 | |
| *** sandywalsh has quit IRC | 00:56 | |
| *** EmilienM has joined #openstack-cinder | 00:56 | |
| *** david-lyle has quit IRC | 00:58 | |
| *** dannywilson has quit IRC | 00:58 | |
| *** david-lyle has joined #openstack-cinder | 00:58 | |
| *** dannywilson has joined #openstack-cinder | 00:59 | |
| *** dannywilson has quit IRC | 01:03 | |
| *** david-lyle has quit IRC | 01:05 | |
| *** EmilienM has quit IRC | 01:07 | |
| *** jdurgin has quit IRC | 01:07 | |
| *** EmilienM has joined #openstack-cinder | 01:08 | |
| *** david-lyle has joined #openstack-cinder | 01:09 | |
| *** jdurgin has joined #openstack-cinder | 01:11 | |
| *** shakamunyi has quit IRC | 01:13 | |
| *** ebalduf has joined #openstack-cinder | 01:21 | |
| *** rwsu has quit IRC | 01:21 | |
| *** david-lyle has quit IRC | 01:24 | |
| *** afazekas has quit IRC | 01:25 | |
| *** ebalduf has quit IRC | 01:26 | |
| *** david-lyle has joined #openstack-cinder | 01:29 | |
| *** _cjones_ has quit IRC | 01:33 | |
| *** _cjones_ has joined #openstack-cinder | 01:33 | |
| *** david-lyle has quit IRC | 01:35 | |
| *** bill_az has quit IRC | 01:37 | |
| *** _cjones_ has quit IRC | 01:40 | |
| *** lcurtis has quit IRC | 01:51 | |
| *** david-lyle has joined #openstack-cinder | 01:52 | |
| *** EmilienM has quit IRC | 01:52 | |
| *** Longgeek has joined #openstack-cinder | 01:55 | |
| *** smoriya has quit IRC | 01:55 | |
| *** EmilienM has joined #openstack-cinder | 01:56 | |
| *** rushil has joined #openstack-cinder | 01:57 | |
| *** david-lyle has quit IRC | 01:59 | |
| *** smoriya has joined #openstack-cinder | 02:01 | |
| *** zhiyan has quit IRC | 02:05 | |
| *** zhiyan has joined #openstack-cinder | 02:07 | |
| *** Masahiro has quit IRC | 02:07 | |
| *** haomaiwang has joined #openstack-cinder | 02:08 | |
| *** tellesnobrega has quit IRC | 02:09 | |
| *** Longgeek has quit IRC | 02:11 | |
| *** vishy has quit IRC | 02:11 | |
| *** vishy has joined #openstack-cinder | 02:14 | |
| *** thingee has quit IRC | 02:15 | |
| *** peluse has quit IRC | 02:16 | |
| *** peluse has joined #openstack-cinder | 02:16 | |
| *** Yogi11 has joined #openstack-cinder | 02:17 | |
| *** mriedem1 has joined #openstack-cinder | 02:20 | |
| *** tellesnobrega has joined #openstack-cinder | 02:20 | |
| *** mriedem has quit IRC | 02:22 | |
| *** ebalduf has joined #openstack-cinder | 02:22 | |
| *** EmilienM has quit IRC | 02:22 | |
| *** EmilienM has joined #openstack-cinder | 02:23 | |
| *** julim has quit IRC | 02:24 | |
| *** IanGovett has quit IRC | 02:25 | |
| *** mtanino has quit IRC | 02:25 | |
| *** ebalduf has quit IRC | 02:27 | |
| *** Masahiro has joined #openstack-cinder | 02:28 | |
| *** Masahiro has quit IRC | 02:30 | |
| *** Masahiro has joined #openstack-cinder | 02:30 | |
| *** Masahiro has quit IRC | 02:34 | |
| *** mberlin1 has joined #openstack-cinder | 02:34 | |
| *** Masahiro has joined #openstack-cinder | 02:34 | |
| *** mberlin has quit IRC | 02:35 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/cinder: Use is_valid_ipv6 from oslo.utils https://review.openstack.org/143611 | 02:37 |
| *** Masahiro has quit IRC | 02:38 | |
| *** Masahiro has joined #openstack-cinder | 02:40 | |
| *** david-lyle has joined #openstack-cinder | 02:40 | |
| *** shakamunyi has joined #openstack-cinder | 02:41 | |
| *** alonmarx_ has joined #openstack-cinder | 02:45 | |
| *** harlowja is now known as harlowja_away | 02:46 | |
| *** git-harry has quit IRC | 02:47 | |
| *** jbernard has quit IRC | 02:47 | |
| *** shakamunyi has quit IRC | 02:47 | |
| *** mgagne has quit IRC | 02:47 | |
| *** russellb has quit IRC | 02:47 | |
| *** alonmarx has quit IRC | 02:48 | |
| *** krtaylor has quit IRC | 02:48 | |
| *** shakamunyi has joined #openstack-cinder | 02:48 | |
| *** git-harry has joined #openstack-cinder | 02:49 | |
| openstackgerrit | Li Yingjun proposed openstack/python-cinderclient: Add missing all-tenants option to transfer-list https://review.openstack.org/114176 | 02:52 |
| *** zhiyan has quit IRC | 02:55 | |
| *** zhiyan has joined #openstack-cinder | 02:57 | |
| *** patrickeast has quit IRC | 03:01 | |
| *** cppforlife has quit IRC | 03:01 | |
| *** cppforlife has joined #openstack-cinder | 03:02 | |
| *** krtaylor has joined #openstack-cinder | 03:02 | |
| *** russellb has joined #openstack-cinder | 03:02 | |
| *** mgagne has joined #openstack-cinder | 03:04 | |
| *** mgagne is now known as Guest33025 | 03:05 | |
| *** shakamunyi has quit IRC | 03:05 | |
| *** tbarron has quit IRC | 03:06 | |
| *** jordandh has quit IRC | 03:06 | |
| *** jordandh has joined #openstack-cinder | 03:06 | |
| *** jcru has joined #openstack-cinder | 03:07 | |
| *** annashen has joined #openstack-cinder | 03:11 | |
| *** Yogi11 has quit IRC | 03:12 | |
| *** fandi has quit IRC | 03:14 | |
| *** rushil has quit IRC | 03:15 | |
| *** russellb has quit IRC | 03:15 | |
| *** junhongl has quit IRC | 03:18 | |
| *** junhongl has joined #openstack-cinder | 03:19 | |
| *** jordandh has quit IRC | 03:21 | |
| *** gtt116__ has joined #openstack-cinder | 03:22 | |
| *** ebalduf has joined #openstack-cinder | 03:23 | |
| *** jordandh has joined #openstack-cinder | 03:23 | |
| *** david-lyle has quit IRC | 03:25 | |
| *** russellb has joined #openstack-cinder | 03:25 | |
| *** gtt116_ has quit IRC | 03:25 | |
| *** bill_az has joined #openstack-cinder | 03:26 | |
| *** ebalduf has quit IRC | 03:28 | |
| *** russellb has quit IRC | 03:38 | |
| *** atan8 has quit IRC | 03:39 | |
| *** russellb has joined #openstack-cinder | 03:39 | |
| openstackgerrit | Trung Trinh proposed openstack/cinder: Delete volume attached to already-deleted VM https://review.openstack.org/142006 | 03:40 |
| *** EmilienM has quit IRC | 03:40 | |
| *** jcru has quit IRC | 03:42 | |
| *** EmilienM has joined #openstack-cinder | 03:44 | |
| *** Masahiro has quit IRC | 03:45 | |
| *** Longgeek has joined #openstack-cinder | 03:51 | |
| *** anshul has joined #openstack-cinder | 03:53 | |
| *** patrickeast has joined #openstack-cinder | 03:56 | |
| *** asselin_ has joined #openstack-cinder | 03:58 | |
| *** dannywilson has joined #openstack-cinder | 03:59 | |
| *** EmilienM has quit IRC | 04:01 | |
| *** annashen has quit IRC | 04:01 | |
| *** Masahiro has joined #openstack-cinder | 04:02 | |
| *** mriedem1 has left #openstack-cinder | 04:03 | |
| *** dannywilson has quit IRC | 04:03 | |
| *** annashen has joined #openstack-cinder | 04:08 | |
| *** EmilienM has joined #openstack-cinder | 04:08 | |
| *** gtt116_ has joined #openstack-cinder | 04:10 | |
| *** ho has quit IRC | 04:10 | |
| *** ho has joined #openstack-cinder | 04:10 | |
| *** rushiagr_away is now known as rushiagr | 04:12 | |
| *** gtt116__ has quit IRC | 04:13 | |
| *** ebalduf has joined #openstack-cinder | 04:25 | |
| *** annashen has quit IRC | 04:28 | |
| *** bill_az has quit IRC | 04:29 | |
| *** ebalduf has quit IRC | 04:29 | |
| *** Guest33025 has quit IRC | 04:32 | |
| openstackgerrit | xing-yang proposed openstack/cinder-specs: Consistency Groups Kilo Update https://review.openstack.org/131024 | 04:32 |
| *** mgagne has joined #openstack-cinder | 04:34 | |
| *** EmilienM has quit IRC | 04:35 | |
| *** mgagne is now known as Guest57605 | 04:35 | |
| openstackgerrit | xing-yang proposed openstack/cinder-specs: Consistency Groups Kilo Update https://review.openstack.org/131024 | 04:36 |
| *** EmilienM has joined #openstack-cinder | 04:38 | |
| *** _cjones_ has joined #openstack-cinder | 04:41 | |
| *** Longgeek has quit IRC | 04:42 | |
| *** Longgeek has joined #openstack-cinder | 04:44 | |
| *** _cjones_ has quit IRC | 04:45 | |
| *** changbl has joined #openstack-cinder | 04:46 | |
| *** teju has joined #openstack-cinder | 04:51 | |
| *** patrickeast has quit IRC | 04:57 | |
| *** vishy has quit IRC | 04:57 | |
| *** vishy has joined #openstack-cinder | 05:04 | |
| *** annashen has joined #openstack-cinder | 05:09 | |
| *** asselin_ has quit IRC | 05:10 | |
| *** EmilienM has quit IRC | 05:12 | |
| *** ho__ has joined #openstack-cinder | 05:13 | |
| *** ho has quit IRC | 05:15 | |
| *** EmilienM has joined #openstack-cinder | 05:16 | |
| *** mwichmann has quit IRC | 05:17 | |
| *** annashen has quit IRC | 05:19 | |
| *** zigo has quit IRC | 05:20 | |
| *** zigo has joined #openstack-cinder | 05:21 | |
| *** dannywilson has joined #openstack-cinder | 05:21 | |
| *** mwichmann has joined #openstack-cinder | 05:22 | |
| *** nkrinner has joined #openstack-cinder | 05:22 | |
| *** ebalduf has joined #openstack-cinder | 05:25 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Remove the useless next link for volumes, transfers and backups https://review.openstack.org/131915 | 05:27 |
| openstackgerrit | xing-yang proposed openstack/cinder: Verify the instance's existance in the VMAX driver https://review.openstack.org/140913 | 05:28 |
| *** Raj_ has joined #openstack-cinder | 05:28 | |
| *** ebalduf has quit IRC | 05:29 | |
| openstackgerrit | Huang Zhiteng proposed openstack/cinder: Cleanup unused DB APIs, part I https://review.openstack.org/147066 | 05:33 |
| *** jbernard has joined #openstack-cinder | 05:38 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Verify the instance's existance in the VMAX driver https://review.openstack.org/140913 | 05:40 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Remove the useless next link for volumes, transfers and backups https://review.openstack.org/131915 | 05:47 |
| *** ebalduf has joined #openstack-cinder | 05:47 | |
| *** russellb has quit IRC | 05:49 | |
| *** russellb has joined #openstack-cinder | 05:49 | |
| *** ebalduf has quit IRC | 05:51 | |
| *** annashen has joined #openstack-cinder | 05:59 | |
| *** lan_ has quit IRC | 06:04 | |
| *** nshaikh has joined #openstack-cinder | 06:06 | |
| openstackgerrit | Dunrong Huang proposed openstack/cinder: Fix eqlx endless loop when server closes the connection https://review.openstack.org/147070 | 06:07 |
| *** lan_ has joined #openstack-cinder | 06:08 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Transifex https://review.openstack.org/144933 | 06:10 |
| openstackgerrit | xing-yang proposed openstack/cinder: Allow multi-attach to happen in VMAX driver https://review.openstack.org/143891 | 06:12 |
| *** annashen has quit IRC | 06:13 | |
| *** annashen has joined #openstack-cinder | 06:14 | |
| *** annashen has quit IRC | 06:14 | |
| *** Longgeek has quit IRC | 06:19 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Remove the useless next link for volumes, transfers and backups https://review.openstack.org/131915 | 06:22 |
| *** anshul has quit IRC | 06:27 | |
| *** cppforlife has quit IRC | 06:28 | |
| *** cppforlife has joined #openstack-cinder | 06:30 | |
| *** Longgeek has joined #openstack-cinder | 06:34 | |
| *** nlevinki has joined #openstack-cinder | 06:41 | |
| *** jcru has joined #openstack-cinder | 06:43 | |
| *** anshul has joined #openstack-cinder | 06:44 | |
| *** Masahiro has quit IRC | 06:45 | |
| *** Masahiro has joined #openstack-cinder | 06:47 | |
| *** jcru has quit IRC | 06:47 | |
| *** ebalduf has joined #openstack-cinder | 06:47 | |
| openstackgerrit | Dave Chen proposed openstack/python-cinderclient: This is Cinder Client specific changes for modifying volume and snapshot's image metadata. https://review.openstack.org/147077 | 06:50 |
| *** sgotliv has quit IRC | 06:51 | |
| *** ebalduf has quit IRC | 06:51 | |
| *** Longgeek has quit IRC | 06:53 | |
| *** Masahiro has quit IRC | 06:54 | |
| *** Masahiro has joined #openstack-cinder | 06:54 | |
| openstackgerrit | Dave Chen proposed openstack/python-cinderclient: Add commands for modifying image metadata https://review.openstack.org/147077 | 06:54 |
| *** Longgeek has joined #openstack-cinder | 06:59 | |
| *** dannywilson has quit IRC | 07:00 | |
| Raj_ | Hi all | 07:07 |
| Raj_ | while migration in cinder | 07:07 |
| Raj_ | from where it will get host[capabilities] ..? | 07:08 |
| *** zhiyan has quit IRC | 07:08 | |
| *** zhiyan has joined #openstack-cinder | 07:11 | |
| *** lan_ has quit IRC | 07:18 | |
| *** oomichi_ has quit IRC | 07:37 | |
| *** russellb has quit IRC | 07:37 | |
| *** Miouge has joined #openstack-cinder | 07:38 | |
| *** jordandh has quit IRC | 07:39 | |
| *** russellb has joined #openstack-cinder | 07:40 | |
| *** TobiasE has joined #openstack-cinder | 07:41 | |
| *** jordandh has joined #openstack-cinder | 07:45 | |
| *** lan__ has joined #openstack-cinder | 07:45 | |
| *** chlong has quit IRC | 07:47 | |
| *** pwehrle has joined #openstack-cinder | 07:58 | |
| *** fandi has joined #openstack-cinder | 08:00 | |
| *** jistr has joined #openstack-cinder | 08:00 | |
| *** Mandell has joined #openstack-cinder | 08:01 | |
| *** dulek has joined #openstack-cinder | 08:05 | |
| openstackgerrit | Abhijeet Malawade proposed openstack/cinder-specs: Return request ID to caller https://review.openstack.org/132161 | 08:09 |
| *** Miouge has quit IRC | 08:11 | |
| *** sgotliv has joined #openstack-cinder | 08:17 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Remove the useless next link for volumes, transfers and backups https://review.openstack.org/131915 | 08:18 |
| *** smoriya has quit IRC | 08:22 | |
| *** tshefi has joined #openstack-cinder | 08:29 | |
| *** russellb has quit IRC | 08:30 | |
| *** Guest57605 has quit IRC | 08:31 | |
| *** mgagne has joined #openstack-cinder | 08:33 | |
| *** mgagne is now known as Guest5538 | 08:33 | |
| *** sgotliv has quit IRC | 08:34 | |
| *** theanalyst has joined #openstack-cinder | 08:38 | |
| *** Masahiro has quit IRC | 08:38 | |
| *** russellb has joined #openstack-cinder | 08:39 | |
| *** Masahiro has joined #openstack-cinder | 08:39 | |
| *** Miouge has joined #openstack-cinder | 08:40 | |
| *** pradipta_away is now known as pradipta | 08:44 | |
| *** Mandell has quit IRC | 08:45 | |
| *** lan__ has quit IRC | 08:49 | |
| *** ebalduf has joined #openstack-cinder | 08:49 | |
| *** lan__ has joined #openstack-cinder | 08:50 | |
| *** ndipanov_ is now known as ndipanov | 08:51 | |
| *** ndipanov has quit IRC | 08:51 | |
| flip214 | argg, my devstack won't start anymore ... because there's no /opt/stack/cinder/bin/cinder-volume anymore. | 08:52 |
| openstackgerrit | Huang Zhiteng proposed openstack/cinder: Raise correct exception when validate_connector failed https://review.openstack.org/146374 | 08:53 |
| *** ebalduf has quit IRC | 08:54 | |
| flip214 | and gate-cinder-python27 failed because of oslo.concurrency. grmpf. | 08:54 |
| *** jpich has joined #openstack-cinder | 09:06 | |
| *** ndipanov has joined #openstack-cinder | 09:09 | |
| *** jordanP has joined #openstack-cinder | 09:10 | |
| *** Miouge has quit IRC | 09:10 | |
| *** Miouge has joined #openstack-cinder | 09:15 | |
| *** nellysmitt has joined #openstack-cinder | 09:25 | |
| *** jamielennox is now known as jamielennox|away | 09:31 | |
| *** Miouge has quit IRC | 09:32 | |
| *** Miouge has joined #openstack-cinder | 09:33 | |
| openstackgerrit | Ph. Marek proposed openstack/cinder: DRBD: Log an error if libraries can't be loaded https://review.openstack.org/146403 | 09:34 |
| *** abhijeetm has joined #openstack-cinder | 09:39 | |
| *** Guest5538 has quit IRC | 09:51 | |
| *** Raj_ has quit IRC | 09:52 | |
| *** mgagne has joined #openstack-cinder | 09:55 | |
| *** mgagne is now known as Guest80979 | 09:55 | |
| *** vishy has quit IRC | 09:59 | |
| *** coolsvap|afk is now known as coolsvap | 09:59 | |
| *** e0ne has joined #openstack-cinder | 10:00 | |
| *** dannywilson has joined #openstack-cinder | 10:01 | |
| *** dulek has quit IRC | 10:04 | |
| *** dannywilson has quit IRC | 10:05 | |
| *** dulek has joined #openstack-cinder | 10:10 | |
| *** vishy has joined #openstack-cinder | 10:10 | |
| *** lpetrut has joined #openstack-cinder | 10:11 | |
| *** vishy has quit IRC | 10:21 | |
| *** aix has joined #openstack-cinder | 10:22 | |
| *** russellb has quit IRC | 10:22 | |
| *** tbarron has joined #openstack-cinder | 10:24 | |
| *** russellb has joined #openstack-cinder | 10:26 | |
| *** sgotliv has joined #openstack-cinder | 10:28 | |
| *** jistr has quit IRC | 10:31 | |
| *** e0ne is now known as e0ne_ | 10:31 | |
| *** tbarron has quit IRC | 10:33 | |
| *** tbarron has joined #openstack-cinder | 10:34 | |
| *** vishy has joined #openstack-cinder | 10:34 | |
| *** karimb has joined #openstack-cinder | 10:35 | |
| *** lpetrut has quit IRC | 10:37 | |
| *** coolsvap is now known as coolsvap|afk | 10:39 | |
| *** e0ne_ has quit IRC | 10:42 | |
| *** Masahiro has quit IRC | 10:42 | |
| *** lpetrut has joined #openstack-cinder | 10:47 | |
| *** jistr has joined #openstack-cinder | 10:48 | |
| *** coolsvap|afk is now known as coolsvap | 10:51 | |
| flip214 | may I offer my opinion that the gerrit reviews are a bit fragile? | 10:51 |
| openstackgerrit | git-harry proposed openstack/cinder: Fix bug in tgt conf for volume https://review.openstack.org/147108 | 10:53 |
| *** e0ne has joined #openstack-cinder | 10:57 | |
| *** russellb has quit IRC | 11:00 | |
| *** coolsvap is now known as coolsvap|afk | 11:02 | |
| *** alexpilotti has joined #openstack-cinder | 11:03 | |
| *** russellb has joined #openstack-cinder | 11:06 | |
| *** coolsvap|afk is now known as coolsvap | 11:09 | |
| *** haomaiwang has quit IRC | 11:11 | |
| *** aix has quit IRC | 11:14 | |
| *** takedakn has joined #openstack-cinder | 11:18 | |
| openstackgerrit | Rushi Agrawal proposed openstack/cinder: WIP: Public snapshots https://review.openstack.org/147122 | 11:23 |
| *** e0ne_ has joined #openstack-cinder | 11:23 | |
| *** takedakn has quit IRC | 11:24 | |
| *** e0ne has quit IRC | 11:26 | |
| *** mdenny has quit IRC | 11:27 | |
| *** asselin has quit IRC | 11:28 | |
| *** aix has joined #openstack-cinder | 11:28 | |
| *** Masahiro has joined #openstack-cinder | 11:43 | |
| *** Masahiro has quit IRC | 11:47 | |
| *** erlon has joined #openstack-cinder | 11:49 | |
| *** ebalduf has joined #openstack-cinder | 11:52 | |
| *** rushiagr is now known as rushiagr_away | 11:53 | |
| *** e0ne_ has quit IRC | 11:55 | |
| *** ganso_ has joined #openstack-cinder | 11:56 | |
| *** ebalduf has quit IRC | 11:57 | |
| *** e0ne has joined #openstack-cinder | 11:58 | |
| *** ho__ has quit IRC | 11:58 | |
| *** takedakn has joined #openstack-cinder | 12:05 | |
| *** chlong has joined #openstack-cinder | 12:06 | |
| *** teju has left #openstack-cinder | 12:08 | |
| *** lan__ has quit IRC | 12:10 | |
| *** takedakn has quit IRC | 12:11 | |
| *** marcusvrn1 has quit IRC | 12:15 | |
| *** miarmak has joined #openstack-cinder | 12:16 | |
| *** lpetrut has quit IRC | 12:18 | |
| *** Raj__ has joined #openstack-cinder | 12:30 | |
| *** pradipta is now known as pradipta_away | 12:30 | |
| *** rushiagr_away is now known as rushiagr | 12:31 | |
| *** Raj__ has quit IRC | 12:31 | |
| *** RAJ__ has joined #openstack-cinder | 12:32 | |
| RAJ__ | Regarding cinder replication support | 12:32 |
| RAJ__ | it is necessary of | 12:32 |
| RAJ__ | Replication relationship db table | 12:32 |
| RAJ__ | and multi back end support ..? | 12:32 |
| *** marcusvrn has joined #openstack-cinder | 12:34 | |
| *** IanGovett has joined #openstack-cinder | 12:38 | |
| *** Masahiro has joined #openstack-cinder | 12:46 | |
| *** sgotliv_ has joined #openstack-cinder | 12:47 | |
| *** sgotliv has quit IRC | 12:47 | |
| *** rushil has joined #openstack-cinder | 12:48 | |
| *** sandywalsh_ has quit IRC | 12:49 | |
| *** Masahiro has quit IRC | 12:50 | |
| *** nlevinki has quit IRC | 12:53 | |
| *** ryanpetrello has joined #openstack-cinder | 12:53 | |
| *** lpetrut has joined #openstack-cinder | 12:55 | |
| *** boris-42 has joined #openstack-cinder | 12:56 | |
| *** e0ne has quit IRC | 12:56 | |
| DuncanT | RAJ__: Can you rephrase the question, please? | 12:56 |
| *** e0ne has joined #openstack-cinder | 12:57 | |
| RAJ__ | DuncaT: replication relationship table is needed for replication support ?? | 12:58 |
| RAJ__ | DuncanT: | 12:58 |
| RAJ__ | ?? | 12:58 |
| DuncanT | Erm, yes... the driver doesn't have to care about it though, the manager handles that. | 12:59 |
| DuncanT | Why? | 12:59 |
| RAJ__ | ohk | 13:01 |
| RAJ__ | DuncanT: hey can you look @my code | 13:02 |
| RAJ__ | ones | 13:02 |
| *** chlong has quit IRC | 13:02 | |
| DuncanT | I can look, but I know very little about the replication code | 13:02 |
| *** zhipeng has joined #openstack-cinder | 13:02 | |
| openstackgerrit | Duncan Thomas proposed openstack/cinder: Fix LVM thin pool creation race https://review.openstack.org/146917 | 13:06 |
| DuncanT | Maybe put up a review and mark it as work-in-progress? | 13:06 |
| *** nlevinki has joined #openstack-cinder | 13:08 | |
| *** lpetrut has quit IRC | 13:10 | |
| *** takedakn has joined #openstack-cinder | 13:11 | |
| *** takedakn has quit IRC | 13:12 | |
| *** takedakn has joined #openstack-cinder | 13:12 | |
| RAJ__ | Link for code ... | 13:12 |
| RAJ__ | http://paste.openstack.org/show/157831/ | 13:12 |
| RAJ__ | DuncanT: http://paste.openstack.org/show/157831/ | 13:13 |
| RAJ__ | take the review and let me knw about that | 13:13 |
| *** e0ne is now known as e0ne_ | 13:14 | |
| RAJ__ | e0ne_: can u take a look | 13:14 |
| RAJ__ | ../ | 13:14 |
| DuncanT | There are a whole bunch of things in there that make me nervous, but what is the problem? Does it work? | 13:17 |
| *** takedakn has quit IRC | 13:19 | |
| *** takedakn has joined #openstack-cinder | 13:20 | |
| *** e0ne_ is now known as e0ne | 13:23 | |
| e0ne | RAJ__: what should i see in this code? i'm a bit out of context | 13:24 |
| *** lpetrut has joined #openstack-cinder | 13:25 | |
| RAJ__ | DuncanT: it is working with volume type which includes : | 13:26 |
| RAJ__ | capabilities=replication now | 13:26 |
| RAJ__ | but | 13:26 |
| RAJ__ | i m confused regarding | 13:26 |
| RAJ__ | DuncanT: promote and reenable ..? | 13:26 |
| RAJ__ | DuncanT: from where storage node 1 knw that which is 2nd storage node ..? | 13:27 |
| rushiagr | DuncanT: saw your comment on public snapshots work. I don't think sharing snapshots has anything to do with glance<->cinder interactions. | 13:27 |
| RAJ__ | e0ne: hi canb you help me to clear doubts related to | 13:27 |
| RAJ__ | promote and reenable replication part ..? | 13:27 |
| rushiagr | i.e., I don't think glance is involved at all.. | 13:27 |
| DuncanT | RAJ__: I don't know the answer to that, and I haven't time right now to go digging through the code, sorry | 13:29 |
| DuncanT | rushiagr: I'm saying we *could* do public snapshots, but they aren't much different to glance images, and glance already does ACLs and a bunch of other useful things, so why not use glance images instead of adding public snapshots | 13:30 |
| rushiagr | DuncanT: oh, so you're just saying that use glance's code to implement same functionality here? | 13:31 |
| rushiagr | that's a good idea actually | 13:32 |
| rushiagr | oh, maybe you mean just copy stuff to glance if one wants to share | 13:32 |
| DuncanT | Yes, I mean actually copy into glance (and make that fast, which has other benefits) | 13:33 |
| *** rushil has quit IRC | 13:33 | |
| *** takedakn has quit IRC | 13:34 | |
| *** RAJ__ has quit IRC | 13:42 | |
| *** nshaikh has quit IRC | 13:42 | |
| DuncanT | rushiagr: If we do public snapshots, I worry we'll have to deal with all of the other things glance has to deal with (access control, lifecycle questions, attestation, metadata, etc) in future. | 13:44 |
| DuncanT | rushiagr: We want glance to be fast for uploading and downloading BfV images anyway, and once that is done then you get public snapshot-like behaviour for free | 13:45 |
| *** nshaikh has joined #openstack-cinder | 13:47 | |
| *** bill_az has joined #openstack-cinder | 13:51 | |
| rushiagr | DuncanT: I submitted a blueprint which talks about sharing snapshots (not only making it public), which is simpler than ACLs | 13:52 |
| rushiagr | https://blueprints.launchpad.net/cinder/+spec/snapshot-sharing | 13:52 |
| rushiagr | (spec not yet up for the BP) | 13:52 |
| *** nlevinki has quit IRC | 13:53 | |
| rushiagr | so this is the blueprint where I was thinking to end the 'sharing' idea, and not go for full ACLs, and other stuff around it | 13:53 |
| DuncanT | rushiagr: Again, I'd say use glance, it already lets you do this | 13:53 |
| DuncanT | rushiagr: It is reinventing the wheel IMO | 13:53 |
| rushiagr | DuncanT: but I'll have to copy snapshot to glance to make this happen | 13:54 |
| rushiagr | while in case of sharing, I don't have to create any more bits. Only when the new tenant creates a volume out of that snapshot. Until that point, sharing means just an additional database entry or two | 13:55 |
| *** takedakn has joined #openstack-cinder | 13:55 | |
| DuncanT | At the moment, yeah. Optimisations like the solidfire one means it would only get copied once (and you can pre-populate the cache on upload so that it never needs to bepulled back down), and you could make cinder a backend for glance if you really wanted | 13:56 |
| DuncanT | I'd rather see the macro case, i.e. the inefficiency of glance<->cinder interactions, fixed, rather than adding hacks that fix sub-cases as they come up | 13:56 |
| *** nlevinki has joined #openstack-cinder | 13:58 | |
| rushiagr | I'm not talking about bfv case, but in general. Someone might want to share a snapshot ('hey I have a DB dump in my vol snapshot, just create a vol out of it and take it!') | 13:58 |
| rushiagr | I am tending towards thinking that the problems are completely different. Optimizing glance-cinder interactions is a valid point, but this snapshot sharing functionality isn't anyway related to it as far as I can see.. | 14:00 |
| *** julim has joined #openstack-cinder | 14:02 | |
| rushiagr | We had a problem with sharing snapshots. Some people here wanted to do that, so I proposed a solution. I never thought it would be a subcase of another bigger issue :) | 14:04 |
| *** sandywalsh has joined #openstack-cinder | 14:04 | |
| *** nlevinki has quit IRC | 14:04 | |
| *** tbarron has quit IRC | 14:04 | |
| *** tbarron has joined #openstack-cinder | 14:05 | |
| *** Yogi1 has joined #openstack-cinder | 14:05 | |
| jgriffith | rushiagr: I still say "who cares" | 14:09 |
| *** fandi has quit IRC | 14:09 | |
| jgriffith | rushiagr: honestly all these "great features we have to have" and most of them are meh | 14:09 |
| *** nlevinki has joined #openstack-cinder | 14:10 | |
| jgriffith | rushiagr: I think ACL's are a horrible idea too FWIW | 14:10 |
| rushiagr | jgriffith: I am with you with all the above points you mentioned :) | 14:10 |
| jgriffith | rushiagr: but I also think the "sharing" thing your proposing for snapshots is "not so good" either | 14:10 |
| *** mkerrin has joined #openstack-cinder | 14:11 | |
| rushiagr | jgriffith: not so big a benefit, but little, and non-zero :) | 14:11 |
| *** fandi has joined #openstack-cinder | 14:11 | |
| rushiagr | jgriffith: do you have any specific concerns? I'm curious to know.. | 14:12 |
| *** dustins has joined #openstack-cinder | 14:12 | |
| jgriffith | DuncanT: by the way what I submitted is NOT a *hack* as you call it. It's an enhancement that take advantage of products specific architecture | 14:12 |
| *** dustins has quit IRC | 14:12 | |
| rushiagr | jgriffith: apart from 'more code to maintain' which I can totally understand.. | 14:12 |
| jgriffith | rushiagr: actually can't say I have any real concern right now since there's no code :) | 14:13 |
| jgriffith | rushiagr: I've never been real big on the idea of ACL's at all.... and as far as this, YES it sounds like it would be more sane | 14:14 |
| rushiagr | jgriffith: :) I didn't want to waste writing code if the idea is going to be rejected straightaway :) | 14:14 |
| jgriffith | rushiagr: but the idea of breaking the relationship betwen snap and volume just to share it seems overkill | 14:14 |
| rushiagr | jgriffith: but if code give more weight to the idea, I'll write it first and then talk :) | 14:14 |
| jgriffith | rushiagr: one is able to share volumes | 14:14 |
| jgriffith | rushiagr: or more accurately "give" | 14:15 |
| jgriffith | rushiagr: which I suppose might be an argument (for consistency sake)... | 14:15 |
| *** dalgaaf has joined #openstack-cinder | 14:17 | |
| DuncanT | jgriffith: Your optimisation I don't consider a hack, adding snapshot sharing I consider a hack. Sorry if that wasn't clear | 14:18 |
| *** takedakn has quit IRC | 14:19 | |
| jgriffith | DuncanT: ahh... thanks, I was very confused | 14:19 |
| DuncanT | jgriffith: I'm saying we should do *more* optimisations like yours, as long as they don't change the behaviour and generally fall back to the old behaviour, which is exactly what your code does | 14:20 |
| *** kaufer has joined #openstack-cinder | 14:20 | |
| *** nlevinki has quit IRC | 14:20 | |
| *** diegows has joined #openstack-cinder | 14:21 | |
| jgriffith | DuncanT: well of course I'd agree with that :) Sorry, I wasn't following the thread correctly. | 14:22 |
| *** Raj2 has joined #openstack-cinder | 14:22 | |
| Raj2 | hey anone seen avishay today ../ | 14:23 |
| Raj2 | ?? | 14:25 |
| *** jungleboyj has quit IRC | 14:25 | |
| *** takedakn has joined #openstack-cinder | 14:26 | |
| *** nkrinner has quit IRC | 14:28 | |
| rushiagr | jgriffith: I think sharing of snapshots is still an alternative way to share data. If you ask me, it is a better way than the current volume transfer. Sharing snapshot just creates a permission for other guy to create a volume out of it. The other guy then can decide if he wants to do that and when and how many times, and charged accordingly | 14:30 |
| jgriffith | rushiagr: sure | 14:30 |
| *** Yogi11 has joined #openstack-cinder | 14:30 | |
| jgriffith | rushiagr: but TBF, xfr volume, create a snap, do whatever you want | 14:30 |
| jgriffith | rushiagr: it's all the same | 14:30 |
| jgriffith | rushiagr: that was a sequence of events BTW | 14:31 |
| jgriffith | xfr-volume--->new user does whatever they want | 14:31 |
| jgriffith | and that's why we have all these crazy "create from snap,image,volume etc" | 14:32 |
| jgriffith | rushiagr: I dunno, maybe I'm not seeing it but maybe others do/will | 14:32 |
| *** nlevinki has joined #openstack-cinder | 14:33 | |
| rushiagr | if I transfer 10 volumes to another tenant, he'll be charged for those 10 right away, starting from the moment they're transferred | 14:33 |
| *** Yogi1 has quit IRC | 14:34 | |
| rushiagr | in case of snapshot, the other user isn't charged, as it's 'my' snapshot, which he can list in 'snapshot list'. He can decide when he wants to create a vol out of it | 14:35 |
| *** Masahiro has joined #openstack-cinder | 14:35 | |
| rushiagr | to be honest, I'm making up these examples inpromptu. Trying to make the idea appear different from (and valid at the same time!) vol xfer | 14:37 |
| *** tbarron has quit IRC | 14:38 | |
| *** tbarron has joined #openstack-cinder | 14:39 | |
| rushiagr | I have added it to today's meeting agenda... Let's see | 14:39 |
| *** Masahiro has quit IRC | 14:39 | |
| Raj2 | jgriffith: | 14:41 |
| Raj2 | Hi | 14:41 |
| *** mwichmann has joined #openstack-cinder | 14:42 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: RemoteFS: move volume/snap locking code to remotefs https://review.openstack.org/147185 | 14:42 |
| openstackgerrit | Eric Harney proposed openstack/cinder: WIP: NFS snapshots https://review.openstack.org/147186 | 14:42 |
| *** akerr has joined #openstack-cinder | 14:42 | |
| *** dustins_ has joined #openstack-cinder | 14:45 | |
| *** tsekiyama has joined #openstack-cinder | 14:47 | |
| DuncanT | rushiagr: He can already do exactly that - upload the volume image to glance, set it public or set an ACL on it | 14:47 |
| *** mtanino has joined #openstack-cinder | 14:49 | |
| DuncanT | rushiagr: Anybody who wants to can create a volume from that image. Not one line of code needed | 14:50 |
| *** miarmak has quit IRC | 14:51 | |
| *** tbarron has quit IRC | 14:53 | |
| *** tbarron has joined #openstack-cinder | 14:53 | |
| rushiagr | agree, but there is a data transfer involved. If glance and cinder are on different backends. | 14:53 |
| rushiagr | I'm completely in agreement that things can be done currently too. I disagree that that's the best way to do. | 14:54 |
| jgriffith | rushiagr: I dunno, in this case I think it might be the better way to do it (the way DuncanT describes it being better) | 14:56 |
| *** ebalduf has joined #openstack-cinder | 14:56 | |
| jgriffith | rushiagr: so you could do that, you could xfr a volume, you could backup/restore.... | 14:57 |
| jgriffith | what's another way | 14:57 |
| jgriffith | rushiagr: oh... attach to an instance and export it that way | 14:57 |
| *** tsekiyam_ has joined #openstack-cinder | 14:58 | |
| *** abhijeetm has quit IRC | 14:59 | |
| rushiagr | wooh | 14:59 |
| *** tsekiya__ has joined #openstack-cinder | 15:00 | |
| *** tsekiyam_ has quit IRC | 15:00 | |
| *** ebalduf has quit IRC | 15:00 | |
| *** Mandell has joined #openstack-cinder | 15:01 | |
| *** tsekiyama has quit IRC | 15:01 | |
| *** dustins_ has quit IRC | 15:02 | |
| *** jungleboyj has joined #openstack-cinder | 15:02 | |
| *** thangp has joined #openstack-cinder | 15:05 | |
| rushiagr | if one has a snapshot, its two steps to the end user for sharing (create a vol from that snapshot, transfer vol). I was just to make it a single-step.. | 15:06 |
| *** rushil has joined #openstack-cinder | 15:07 | |
| jgriffith | rushiagr: yeah, I get ya | 15:07 |
| rushiagr | s/just/just trying/ | 15:07 |
| *** jecarey has joined #openstack-cinder | 15:08 | |
| *** jcru has joined #openstack-cinder | 15:11 | |
| openstackgerrit | Jay Bryant proposed openstack/cinder: Add hacking check for oslo namespace usage https://review.openstack.org/147014 | 15:11 |
| *** jcru has quit IRC | 15:12 | |
| flip214 | can somebody explain me the check-grenade-dsvm FAILURE from https://review.openstack.org/#/c/146403/, please? link is http://logs.openstack.org/03/146403/4/check/check-grenade-dsvm/8859b7e/ | 15:13 |
| *** jcru has joined #openstack-cinder | 15:14 | |
| *** mriedem has joined #openstack-cinder | 15:16 | |
| *** Yogi11 has quit IRC | 15:16 | |
| *** tsekiya__ has quit IRC | 15:17 | |
| nikesh_vedams | jgriffith: good evening | 15:19 |
| nikesh_vedams | jgriffith: sorry morning | 15:19 |
| *** tsekiyama has joined #openstack-cinder | 15:20 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/python-cinderclient: Added formeted view for snapshots in volume details https://review.openstack.org/144740 | 15:20 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Added snapshots list to volume details view https://review.openstack.org/144739 | 15:21 |
| *** Miouge has quit IRC | 15:22 | |
| openstackgerrit | Navneet Singh proposed openstack/cinder: Implement Cinder Manage/Unmanage APIs in NFS driver. https://review.openstack.org/138125 | 15:23 |
| *** tsekiyama has quit IRC | 15:23 | |
| *** tsekiyama has joined #openstack-cinder | 15:24 | |
| dulek | Guys, I'm looking for a way to add periodic_task to cinder-api (is this even possible?) or hook to service startup. Any ideas where I can start? | 15:25 |
| e0ne | dulek: look to the cinder-volume. it already uses periodic tasks | 15:26 |
| dulek | Yeah, I've seen that, but cinder-api doesn't use manager where I would put a task. Or I'm unable to find it. | 15:27 |
| *** fischerw has joined #openstack-cinder | 15:28 | |
| dulek | Same problem with service startup. I mean cinder.wsgi.Server is rather generic and isn't a good place. | 15:28 |
| guitarzan | dulek: you say you need a periodic task in the api... what do you really need? :) | 15:28 |
| dulek | Okay, so I'm looking for a way to resume running TaskFlow's workflow. | 15:29 |
| dulek | I can get them into the DB using taskflow.persistence module. | 15:30 |
| *** dustins has joined #openstack-cinder | 15:30 | |
| *** Yogi11 has joined #openstack-cinder | 15:30 | |
| guitarzan | why are your api tasks stopping? | 15:30 |
| dulek | I'm just killing it, I'm aiming to add more fault tolerance here. | 15:31 |
| dulek | So resuming killed tasks in cinder-volume and cinder-scheduler is easy - periodic task or service startup. | 15:31 |
| guitarzan | the general idea for restarting/continuing tasks was to start up all the tasks on startup I thought | 15:31 |
| dulek | Is there a good way to do it in the API? | 15:31 |
| guitarzan | for example, if your api service or volume service dies, you can restart the tasks when you restart the service | 15:31 |
| guitarzan | s/restart/continue/ | 15:32 |
| DuncanT | Just do it on service startup, if a task dies but the service is still up then you should just fix the bug that caused that to happen | 15:32 |
| *** rhe00_ has joined #openstack-cinder | 15:33 | |
| dulek | DuncanT: That's not that easy in case of scheduler - at startup it doesn't have info about cinder-volume nodes, so it will fail the scheduling. | 15:33 |
| dulek | For cinder-volume this is working. | 15:33 |
| dulek | And for API service - probably will work also, but I can't find decent place to put the restarting code. | 15:34 |
| dulek | Any ideas? | 15:35 |
| DuncanT | dulek: Ok, the scheduler shouldn't do anything except collect stats until at least the bcast interval has passed, resuming tasks included. | 15:35 |
| dulek | DuncanT: Correct. | 15:36 |
| dulek | So my only concern is cinder-api now. | 15:36 |
| *** ndipanov has quit IRC | 15:37 | |
| *** ebalduf has joined #openstack-cinder | 15:38 | |
| *** Miouge has joined #openstack-cinder | 15:38 | |
| *** e0ne is now known as e0ne_ | 15:38 | |
| *** e0ne_ is now known as e0ne | 15:38 | |
| dulek | Hm, okay I think I'm looking at this from the wrong side... | 15:39 |
| *** ndipanov has joined #openstack-cinder | 15:39 | |
| *** xyang has joined #openstack-cinder | 15:40 | |
| e0ne | dulek: actually, it's the same what DuncanT mentioned in a bug | 15:40 |
| *** Mandell has quit IRC | 15:40 | |
| dulek | e0ne: Can you point me to the bug? | 15:40 |
| dulek | I guess the place would be cinder.volume.api.API.__init__ | 15:41 |
| e0ne | dulek: https://bugs.launchpad.net/cinder/+bug/1409012 | 15:41 |
| e0ne | i'm going to provide patch for it later this week | 15:42 |
| dulek | e0ne: Ah, that's cool. I'm doing unrelated work, but will hit this eventually. Your solution will be helpful, thanks! | 15:43 |
| e0ne | dulek: ok. i'll keep you updated on it | 15:44 |
| *** Yogi11 has quit IRC | 15:46 | |
| *** r-daneel has joined #openstack-cinder | 15:48 | |
| *** asselin has joined #openstack-cinder | 15:55 | |
| flip214 | /join #openstack-meeting | 15:57 |
| *** tshefi has quit IRC | 15:57 | |
| *** Guest80979 is now known as mgagne | 15:57 | |
| *** mgagne has joined #openstack-cinder | 15:57 | |
| *** thingee has joined #openstack-cinder | 15:57 | |
| *** jdurgin has quit IRC | 15:58 | |
| *** avishay has joined #openstack-cinder | 16:00 | |
| *** nshaikh has quit IRC | 16:00 | |
| *** AlkaD has joined #openstack-cinder | 16:00 | |
| *** scohen has joined #openstack-cinder | 16:01 | |
| *** diegows has quit IRC | 16:01 | |
| *** rmesta has joined #openstack-cinder | 16:01 | |
| *** deepakcs has joined #openstack-cinder | 16:01 | |
| *** nlevinki has quit IRC | 16:03 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Added snapshots list to volume details view https://review.openstack.org/144739 | 16:03 |
| *** ryanpetrello_ has joined #openstack-cinder | 16:04 | |
| *** patrickeast has joined #openstack-cinder | 16:05 | |
| *** kvidvans has quit IRC | 16:06 | |
| *** Yogi1 has joined #openstack-cinder | 16:06 | |
| *** ryanpetrello has quit IRC | 16:08 | |
| *** ryanpetrello_ is now known as ryanpetrello | 16:08 | |
| *** winston-d_zZZ is now known as winston-d | 16:09 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add tenant validation to quota https://review.openstack.org/143645 | 16:09 |
| *** bruff has joined #openstack-cinder | 16:09 | |
| *** alexpilotti_ has joined #openstack-cinder | 16:10 | |
| *** alexpilotti has quit IRC | 16:12 | |
| *** alexpilotti_ is now known as alexpilotti | 16:12 | |
| *** rushil has quit IRC | 16:13 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add tenant validation to quota https://review.openstack.org/143645 | 16:14 |
| *** jdurgin has joined #openstack-cinder | 16:14 | |
| *** markvoelker has joined #openstack-cinder | 16:15 | |
| *** rakesh___510 has joined #openstack-cinder | 16:17 | |
| *** hypnosb_ has joined #openstack-cinder | 16:20 | |
| *** Raj2 has quit IRC | 16:20 | |
| hypnosb_ | Hi, was there any progress with the iser implementation or was it tossed out? | 16:21 |
| *** kkk has joined #openstack-cinder | 16:23 | |
| *** rakesh___510_ has joined #openstack-cinder | 16:23 | |
| *** krtaylor has quit IRC | 16:23 | |
| *** Masahiro has joined #openstack-cinder | 16:24 | |
| *** pwehrle has quit IRC | 16:25 | |
| *** rakesh___510 has quit IRC | 16:26 | |
| *** ndipanov has quit IRC | 16:27 | |
| *** Masahiro has quit IRC | 16:28 | |
| *** Longgeek has quit IRC | 16:29 | |
| *** takedakn has quit IRC | 16:29 | |
| nikesh_vedams | jgriffith: hi | 16:32 |
| *** juzuluag has joined #openstack-cinder | 16:33 | |
| *** mdenny has joined #openstack-cinder | 16:34 | |
| openstackgerrit | Gary W. Smith proposed openstack/cinder: Add 3PAR extra specs schema into volume stats https://review.openstack.org/147005 | 16:35 |
| *** ndipanov has joined #openstack-cinder | 16:39 | |
| *** markvoelker has quit IRC | 16:40 | |
| *** krtaylor has joined #openstack-cinder | 16:41 | |
| openstackgerrit | Xiaolei Hu proposed openstack/cinder: Implements: blueprint ibm-flashsystem-driver-iscsi Change-Id: I25f8ae0e1b67b1f65e2accebbd9dd978d11a5037 https://review.openstack.org/144627 | 16:43 |
| *** rakesh___510_ has quit IRC | 16:44 | |
| openstackgerrit | Edwin Wang proposed openstack/cinder: Add iSCSI protocol support for IBM FlashSystem. https://review.openstack.org/144627 | 16:47 |
| jgriffith | winston-d: so that's a really good question | 16:49 |
| jgriffith | winston-d: my suspicion is "no" | 16:49 |
| jgriffith | winston-d: but when you look at the code it's easy to get confused on what that var is used for I think | 16:50 |
| winston-d | jgriffith: i'd guess the same | 16:50 |
| jgriffith | winston-d: also, I personally thought that it would be more accurate to update filter-sched info on create/delete rather than rely on periodic as well | 16:50 |
| jgriffith | winston-d: the quota thing is troubling too | 16:51 |
| *** ebalduf has quit IRC | 16:51 | |
| jgriffith | winston-d: but I *think* we have a backup for that in other places | 16:51 |
| *** ebalduf has joined #openstack-cinder | 16:52 | |
| winston-d | jgriffith: fyi, scheduler keeps track of capacity in between two periodic updates in its memory, so scheduler wouldn't allow a create to be placed on a backend without capacity. | 16:52 |
| jgriffith | winston-d: ahh! I didn't see that | 16:53 |
| jgriffith | winston-d: that's good to know | 16:53 |
| jgriffith | winston-d: now I remember you fixed that :) | 16:53 |
| winston-d | but if there are more than one scheduler, well, typical distributed computing problem, we don't get schedulers to sync for now. | 16:53 |
| jgriffith | winston-d: I logged a bug a long time ago for that | 16:53 |
| jgriffith | winston-d: understood (WRT multiple schedulers) | 16:54 |
| *** rushil has joined #openstack-cinder | 16:55 | |
| winston-d | jgriffith: there's another bug about scheduler don't get updates fast enough when it starts, Ivan assigned himself to fix that bug, but I remember we fixed that once. | 16:55 |
| winston-d | I need to take another look. | 16:56 |
| *** madskier has joined #openstack-cinder | 16:56 | |
| winston-d | jgriffith: sorry, I missed the conversation about taskflow/retries, not sure how it messes up with allocated_capacity. | 16:57 |
| *** annashen has joined #openstack-cinder | 16:57 | |
| *** ebalduf has quit IRC | 16:57 | |
| winston-d | jgriffith: I thought no matter how many times taskflow retries, vol manager only increament/decreament once when create/delete succeeded. | 16:58 |
| openstackgerrit | Gary W. Smith proposed openstack/python-cinderclient: Expose cinder's scheduler pool stats API https://review.openstack.org/140142 | 16:59 |
| jgriffith | rushiagr: DuncanT avishay what ever happened with mgagne 's public private stuff? | 17:00 |
| *** kvidvans has joined #openstack-cinder | 17:00 | |
| thingee | DuncanT: I hope you don't mind me taking over the CI enforcing stuff? | 17:01 |
| jgriffith | I think it was mgagne that was working on that | 17:01 |
| DuncanT | thingee: Definitely not. I was getting accused of being obsessed / too pushy with it | 17:01 |
| rushiagr | jgriffith: hmm..Let me check that out. I'm remembering something..but maybe that wasn't what exactly mgagne was doing.. | 17:01 |
| *** avishay has quit IRC | 17:01 | |
| DuncanT | Volumes are different to snapshots because volumes are mutable BTW | 17:02 |
| *** lpetrut has quit IRC | 17:02 | |
| winston-d | jgriffith: that was public/private volume type | 17:02 |
| DuncanT | @avishay ^^^ | 17:02 |
| rushiagr | jgriffith: no, I don't see work from mgagne related to snapshot sharing | 17:02 |
| winston-d | just type | 17:02 |
| jgriffith | winston-d: ahh... yes | 17:03 |
| jgriffith | winston-d: well... now the precedent is set :) | 17:03 |
| jgriffith | I say we extend it | 17:03 |
| *** patrickeast has quit IRC | 17:03 | |
| *** coolsvap is now known as coolsvap|afk | 17:03 | |
| *** dulek has quit IRC | 17:03 | |
| DuncanT | So if we do public snapshots, then how do you catalogue them? How do you filter them? | 17:03 |
| *** AlkaD has quit IRC | 17:03 | |
| jgriffith | DuncanT: I don't know what that means :) | 17:04 |
| jgriffith | DuncanT: you mean in a "snapshot list" | 17:04 |
| *** AlkaD has joined #openstack-cinder | 17:04 | |
| jgriffith | DuncanT: same way we do already with things that have the notion of public acess | 17:04 |
| thingee | jgriffith: I think he means if you have snapshots shared with you, what does snapshot list do to filter those in with your snapshots you own primarily | 17:04 |
| *** anshul has quit IRC | 17:04 | |
| rushiagr | DuncanT: not sure about catalog. Filter should be easy I think | 17:04 |
| jgriffith | DuncanT: you modify the get command to get all tenant owned and all public | 17:04 |
| DuncanT | jgriffith: Yeah. Take a public cloud as a degenerate case. there might be thousands of public snaps | 17:05 |
| jgriffith | DuncanT: better implement a quota :) | 17:05 |
| DuncanT | Do you *really* want them all showing up on every snapshot list | 17:05 |
| DuncanT | ? | 17:05 |
| jgriffith | DuncanT: honestly, maybe it doesn't make sense to turn it on in a public cloud? | 17:05 |
| rushiagr | pass a param, 'owner=me' types, or better to make it the default.. | 17:05 |
| jgriffith | DuncanT: certainly not, but IMO this is kinda of a niche case | 17:06 |
| jgriffith | DuncanT: and if I were running a public cloud I probably wouldn't even turn it on | 17:06 |
| DuncanT | jgriffith: Maybe. I still think glance has already dealt with this usecase, other than the performance, so can we just make it perform better? | 17:06 |
| jgriffith | DuncanT: sure, I'm not saying we should do it one way or other. Just an idea | 17:06 |
| jgriffith | DuncanT: and also consider if consistency is an option | 17:07 |
| DuncanT | jgriffith: Maybe even bite the bullet and make cinder a fully first class glance backend, then you get rid of all the data copying | 17:07 |
| mgagne | jgriffith: only worked on public/private volume types | 17:07 |
| jgriffith | mgagne: yeah, winston-d corrected me :) | 17:07 |
| jgriffith | DuncanT: well, I looked at that again... | 17:08 |
| DuncanT | I'd wonder if there isn't a quota / DoS issue too... create a public snap, let somebody else clone it and write to it, delete the public snap, repeat. 2xdisk space usage for every one of those volumes on many backends | 17:08 |
| hypnosb_ | DuncanT: i vote yes | 17:08 |
| DuncanT | hypnosb_: ? | 17:09 |
| hypnosb_ | cinder glance backender | 17:09 |
| DuncanT | jgriffith: You looked at it again... any road blocks? | 17:10 |
| *** Longgeek has joined #openstack-cinder | 17:10 | |
| jgriffith | DuncanT: yeah, it was a big fat PITA :) | 17:10 |
| DuncanT | jgriffith: :-( | 17:10 |
| jgriffith | DuncanT: seriously though.... I think there's a good way to do it | 17:10 |
| jgriffith | I just hadn't solved it | 17:11 |
| *** asselin has quit IRC | 17:11 | |
| jgriffith | DuncanT: we *should* be able to just leverage what nova does with snapshot bootable instances | 17:11 |
| jgriffith | DuncanT: just modify it a bit and be done | 17:11 |
| jgriffith | DuncanT: everything is apparantly *there* | 17:11 |
| hypnosb_ | i did some silly shit yesterday and exported a n iscsi lun because images are copied from glance | 17:12 |
| hemna | akerr, saw your comments about FC and doing CI | 17:12 |
| jgriffith | but then I thought... "what's the win here" | 17:12 |
| DuncanT | Yeah, feels that way to me too | 17:12 |
| hemna | akerr, we are doing CI for our FC driver | 17:12 |
| hypnosb_ | still broke | 17:12 |
| hypnosb_ | idk why the damn thign is read only | 17:12 |
| hypnosb_ | sigh | 17:12 |
| hypnosb_ | stupid liotarget | 17:12 |
| thingee | hypnosb_, jgriffith: yeah I think we need to deprecate cinder backstore in glance. | 17:13 |
| thingee | no interest in fixing it | 17:13 |
| thingee | there was a ML post about it | 17:13 |
| thingee | I pushed back to give time for people to fix it, but that has proved that no one seems to care. | 17:14 |
| DuncanT | jgriffith: I think the wins are 1) Faster BfV & copy volume to image 2) Fast public snapshots 3) In theory faster ephemeral if you CoW over an iSCSI import of a cinder volume (particularly if you can R/O multi-attach that volume) | 17:14 |
| DuncanT | The existing cinder backingstore appears fundamentally broken... | 17:14 |
| jgriffith | DuncanT: the first two I agree for sure | 17:15 |
| jgriffith | DuncanT: you'll have to help me with number 3 | 17:15 |
| jgriffith | DuncanT: I'll take a look again | 17:15 |
| DuncanT | Number three is a bit next-gen, but: | 17:15 |
| ameade | akerr: he's running back to his desk | 17:16 |
| jgriffith | thingee: yeah, well we just have to do *something* it's sat their incomplete and causing confusion for WAY too long | 17:16 |
| thingee | jgriffith, hypnosb_: I think I'll post that topic for the next cinder meeting, proposal to deprecate it. | 17:16 |
| akerr | hemna: hi | 17:16 |
| thingee | jgriffith, hypnosb_: or maybe just submit a patch to glance | 17:16 |
| ameade | heh put wrong nick | 17:16 |
| jgriffith | thingee: but there's nothing to deprecate :) | 17:16 |
| hemna | akerr, hey | 17:16 |
| thingee | jgriffith: :) | 17:16 |
| DuncanT | If you multi-attach a boot image, as a volume, to a compute node, R/O, then throw a cow layer over the top, you avoid copying any bit of the image that isn't accessed, and can cache on the compute node, giving you faster provisioning and less network usage | 17:16 |
| jgriffith | thingee: just some half written code and some bp's | 17:16 |
| jgriffith | :) | 17:17 |
| hypnosb_ | its really the glance folks that have the cinder backed ... | 17:17 |
| thingee | jgriffith: true | 17:17 |
| akerr | hemna: ya, we think its certainly doable, but I wasn't sure we had the manpower to get it done right before 3-19 | 17:17 |
| *** Mandell has joined #openstack-cinder | 17:17 | |
| hypnosb_ | iirc | 17:17 |
| hemna | akerr, so doing FC in CI is only slightly more difficult than CI | 17:17 |
| jgriffith | DuncanT: interesting | 17:17 |
| *** jpich has quit IRC | 17:17 | |
| jgriffith | DuncanT: so you'd have Nova put a FS and a cow layer on the volume? Maybe as part of BDM? | 17:17 |
| thingee | hypnosb_: right | 17:17 |
| jgriffith | DuncanT: or am I completely missing something | 17:18 |
| DuncanT | jgriffith: Yeah | 17:18 |
| jgriffith | DuncanT: that's pretty cool | 17:18 |
| DuncanT | jgriffith: I've done PoC work on it | 17:18 |
| jgriffith | DuncanT: how do we distinguish that property in Cinder? | 17:18 |
| DuncanT | jgriffith: Make all the glance images owned by a specific tenant maybe? After that it is just an attach | 17:19 |
| DuncanT | jgriffith: I've not yet found an open COW-over-raw device driver, but maybe I'll be able to get ours released | 17:20 |
| jgriffith | interesting | 17:20 |
| *** vbala has joined #openstack-cinder | 17:20 | |
| thingee | rushiagr: your item is first for next meeting https://wiki.openstack.org/wiki/CinderMeetings | 17:20 |
| jgriffith | so 1 and 2 are wins anyway | 17:20 |
| jgriffith | 3 is gravy | 17:20 |
| DuncanT | jgriffith: qcow is an option but that slightly compliicates upload/download | 17:20 |
| rushiagr | DuncanT: I couldn't fully grasp the DoS/quota scenario you meantioned | 17:21 |
| DuncanT | Yeah, 3 is something to shoot for, the other two are easy wins | 17:21 |
| rushiagr | I'll keep a spec ready by next meeting | 17:21 |
| rushiagr | thingee: thanks for that.. | 17:21 |
| jgriffith | alright, I'll look at it again | 17:21 |
| jgriffith | but if I don't have something by end of next week I'm punting or turning it over to someobdy else | 17:21 |
| *** e0ne is now known as e0ne_ | 17:21 | |
| DuncanT | jgriffith: Cool. I'll have a read of the glance backend API myself | 17:21 |
| jgriffith | and if it's not in K3 it needs to get wiped from history :) | 17:21 |
| DuncanT | jgriffith: The existing code should be nuked anyway IMO, it is crap | 17:22 |
| jgriffith | :) | 17:22 |
| hypnosb_ | big problem | 17:22 |
| *** annashen has quit IRC | 17:22 | |
| hypnosb_ | was going to say we could have an image driver.. but then cinder isnt an image solution? | 17:22 |
| hypnosb_ | fffffffffuuuuuuuuuu | 17:22 |
| *** sgotliv_ has quit IRC | 17:23 | |
| hypnosb_ | rage | 17:23 |
| DuncanT | hypnosb_: ? | 17:23 |
| *** xyang has quit IRC | 17:23 | |
| jgriffith | hypnosb_: yo... we're not exactly timid or prudish around here but you might be close to pushing some boundaries | 17:24 |
| *** Longgeek has quit IRC | 17:24 | |
| hypnosb_ | my b | 17:25 |
| jgriffith | hypnosb_: no worries :) | 17:25 |
| kkk | Hi all geek folks | 17:26 |
| hypnosb_ | thats a pretty big boundry push ^ | 17:26 |
| kkk | i need help asap for cinder replication pls help me in thta | 17:26 |
| kkk | i am very new to it pls guys | 17:26 |
| *** TobiasE has quit IRC | 17:26 | |
| DuncanT | kkk: What are you trying to do, exactly? | 17:27 |
| *** juzuluag has quit IRC | 17:27 | |
| *** TobiasE has joined #openstack-cinder | 17:27 | |
| jgriffith | hypnosb_: LOL | 17:27 |
| hypnosb_ | well i was thinking if the cinder backend for glance isnt working out so well | 17:27 |
| *** Raj__ has joined #openstack-cinder | 17:27 | |
| *** kkk has quit IRC | 17:27 | |
| Raj__ | hey its mee only due to another browser its kk nick name | 17:28 |
| Raj__ | i want to report cinder rep with some days only | 17:28 |
| *** rwsu has joined #openstack-cinder | 17:29 | |
| Raj__ | DuncanT: i working on that ... | 17:29 |
| Raj__ | todays issue till now | 17:29 |
| hypnosb_ | maybe the cinder itself should pull from the catalog? | 17:29 |
| hypnosb_ | that could be easier? | 17:29 |
| *** leeantho has joined #openstack-cinder | 17:30 | |
| Raj__ | ?? | 17:30 |
| hypnosb_ | ugh | 17:30 |
| hypnosb_ | idk | 17:30 |
| hypnosb_ | im so confused | 17:31 |
| *** _cjones_ has joined #openstack-cinder | 17:31 | |
| *** TobiasE has quit IRC | 17:31 | |
| *** takedakn has joined #openstack-cinder | 17:31 | |
| *** asselin has joined #openstack-cinder | 17:32 | |
| *** e0ne_ has quit IRC | 17:32 | |
| openstackgerrit | Richard Hagarty proposed openstack/cinder: Set snapCPG when managing existing volumes https://review.openstack.org/147258 | 17:32 |
| *** _cjones_ has quit IRC | 17:32 | |
| *** EmilienM is now known as EmilienM|afk | 17:32 | |
| *** lpetrut has joined #openstack-cinder | 17:32 | |
| hypnosb_ | from what I assume in the implementation of the services is what hardware you place them and the types of IO patters? | 17:33 |
| DuncanT | Raj: You need to say very clearly what your problem is, or nobody is going to be able to help | 17:33 |
| *** deepakcs has quit IRC | 17:34 | |
| hypnosb_ | like for example with cinder I have a 40G 10k sas array, but for glance... i dont need that? | 17:34 |
| *** Raj__ has quit IRC | 17:34 | |
| DuncanT | hypnosb_: None of the service nodes need that much in the way of local disk I/O generally | 17:34 |
| hypnosb_ | aathe sme ith a0 gwhich is a problem wihe va computenodes ahe l disks | 17:35 |
| hypnosb_ | lag | 17:35 |
| *** xyang has joined #openstack-cinder | 17:35 | |
| hypnosb_ | at the same time, its a problem when images are 40GB and compute nodes have lsmall disks | 17:35 |
| *** _cjones_ has joined #openstack-cinder | 17:36 | |
| *** diegows has joined #openstack-cinder | 17:36 | |
| DuncanT | hypnosb_: If you're using large ephemeral boot disks then you need to spec the compute nodes appropriately... otherwise use ceph or a SAN | 17:36 |
| marcusvrn | DuncanT: thingee Hi! I was in another meeting, so I couldn't be in the cinder meeting. I have read the log of the meeting and about CI I have some questions that maybe are related winston-d questions. Sometimes, some job fail because the devstack failed to run. How can we know if the CI is reliable or not? if it fails once, is it not reliable? Looking for another scenario...once the CI is OK and after k-3 the CI stop w | 17:37 |
| DuncanT | marcusvrn: If your CI disagrees with jenkins 5 patches in a row for any reason other than a genuine code bug, it can be considered broken | 17:38 |
| *** annashen has joined #openstack-cinder | 17:38 | |
| DuncanT | marcusvrn: If that is caused by devstack failing on your CI but not on jenkins, it still counts as your CI being broken. Bugs and patches against devstack welcome. | 17:38 |
| *** Raj__ has joined #openstack-cinder | 17:38 | |
| *** jistr has quit IRC | 17:39 | |
| DuncanT | marcusvrn: Running a CI takes some effort. That isn't too unexpected... | 17:39 |
| *** hypnosb_ has quit IRC | 17:40 | |
| marcusvrn | DuncanT: ok...yes! But how many time will we have to fix the CI after infra team have disabled it, before the driver be cut off? | 17:41 |
| DuncanT | marcusvrn: I don't think there will be a simple answer to that. If you have somebody answering emails, trying things and generally staying in contact, there will be a high degree of flexibility. If all contact goes dead, expect less time | 17:43 |
| *** takedakn has quit IRC | 17:43 | |
| DuncanT | marcusvrn: We don't /want/ to remove any drivers, we /do/ want to encourage participation and communication | 17:44 |
| marcusvrn | DuncanT: awesome! it's very nice | 17:44 |
| Raj__ | DuncanT: u got something from net regarding that questions ...? | 17:45 |
| *** e0ne has joined #openstack-cinder | 17:45 | |
| *** scohen has quit IRC | 17:46 | |
| DuncanT | Raj: ? Regarding what questions? | 17:46 |
| *** ganso_ has left #openstack-cinder | 17:47 | |
| *** jordanP has quit IRC | 17:48 | |
| *** Longgeek has joined #openstack-cinder | 17:49 | |
| *** dannywilson has joined #openstack-cinder | 17:50 | |
| *** scohen has joined #openstack-cinder | 17:50 | |
| asselin | marcusvrn, I'm interested to know what kind of devstack failures you get that jenkins isnot | 17:50 |
| *** annegentle_ has quit IRC | 17:51 | |
| nikesh_vedams | jgriffith : now launch instance is working but since i am using trusty image which have a default username "ubuntu" to give in ssh login | 17:52 |
| nikesh_vedams | jgriffith: so in sos-ci do we have provision to login like this ssh -i /home/vedams/cloudkey.pem ubuntu@192.168.2.186 | 17:54 |
| *** ndipanov has quit IRC | 17:55 | |
| *** kbyrne has quit IRC | 17:56 | |
| nikesh_vedams | jgriffith: how can we give absolute path of cloudkey.pem and ubuntu@instance-ip | 17:56 |
| Raj__ | DuncanT: we can discussed regarding | 17:58 |
| *** Mandell has quit IRC | 18:00 | |
| *** dannywilson has quit IRC | 18:00 | |
| *** Raj__ has quit IRC | 18:01 | |
| jgriffith | nikesh_vedams: your questions have nothing to do with CI, whether it be sos-ci or anything else | 18:01 |
| jgriffith | nikesh_vedams: this is openstack user 101 | 18:01 |
| jgriffith | nikesh_vedams: ssh -i keyname ubuntu@xxxxx is how cloud images work! | 18:01 |
| jgriffith | there's no customization there or magic | 18:02 |
| jgriffith | nikesh_vedams: you can set things in your .ssh/config to make key and login usage automatic | 18:02 |
| jgriffith | again, standard ssh stuff, not sos-ci or anything else | 18:02 |
| jgriffith | nikesh_vedams: https://gist.github.com/j-griffith/ae7436cd8fb49610f7eb | 18:03 |
| *** Mandell has joined #openstack-cinder | 18:03 | |
| marcusvrn | asselin: yesterday I was running the CI and an unauthorized error while devstack was trying to upload an image was raised, but running again, it was OK | 18:03 |
| *** e0ne has quit IRC | 18:04 | |
| *** annashen has quit IRC | 18:05 | |
| *** ndipanov has joined #openstack-cinder | 18:08 | |
| *** annashen has joined #openstack-cinder | 18:08 | |
| *** chlong has joined #openstack-cinder | 18:10 | |
| *** harlowja_away is now known as harlowja | 18:10 | |
| harlowja | lol, jgriffith u are GREAT! | 18:12 |
| harlowja | lol | 18:12 |
| harlowja | *form meeting* | 18:12 |
| harlowja | *from | 18:12 |
| thingee | harlowja: heh | 18:12 |
| thingee | jgriffith is great | 18:12 |
| thingee | don't forget to vote! | 18:12 |
| *** Masahiro has joined #openstack-cinder | 18:12 | |
| harlowja | ah, voting | 18:13 |
| harlowja | guess i should do that | 18:13 |
| harlowja | btw; i'm not the only one that understands taskflow i hope, others in #openstack-state-management i hope do :-P | 18:13 |
| harlowja | http://www.rackspace.com/cloud/big-data/ is afaik built on taskflow ;) | 18:13 |
| harlowja | so if that means anything... | 18:14 |
| harlowja | and ML2 blah blah and some other projects are adopting it | 18:14 |
| thingee | harlowja: where's vilobh? | 18:14 |
| thingee | should be back from vacation now? | 18:14 |
| harlowja | should be back soon i thinks | 18:14 |
| harlowja | let me poke someone | 18:14 |
| thingee | harlowja: I have retargeted https://blueprints.launchpad.net/cinder/+spec/cinder-state-enforcer | 18:15 |
| jungleboyj | thingee: or DuncanT any issues with getting this in: https://review.openstack.org/#/c/147014/ Would like to start building the other namespace changes on it today if we can get it in. | 18:15 |
| harlowja | thingee kk | 18:15 |
| harlowja | i'll find out that date; gotta be soon, marriages don't take that long, lol | 18:15 |
| harlowja | but they last for a lifetime i hear, ha | 18:15 |
| *** Guest97290 has joined #openstack-cinder | 18:16 | |
| *** crose has joined #openstack-cinder | 18:16 | |
| *** Masahiro has quit IRC | 18:18 | |
| openstackgerrit | Abel Lopez proposed openstack/cinder: Purge deleted rows https://review.openstack.org/146766 | 18:18 |
| harlowja | jgriffith u guys do know that sqlalchemy is done by like one guy ;) | 18:19 |
| harlowja | :) | 18:19 |
| openstackgerrit | Thang Pham proposed openstack/cinder: WIP: Snapshot object https://review.openstack.org/133566 | 18:19 |
| openstackgerrit | Thang Pham proposed openstack/cinder: WIP: Cinder objects base https://review.openstack.org/131873 | 18:19 |
| harlowja | anyways, here to help | 18:20 |
| harlowja | thingee next week (after mLK) vilobh be back | 18:20 |
| harlowja | or thats what i'm told | 18:20 |
| thingee | jungleboyj: lgtm | 18:21 |
| jungleboyj | thingee: Thanks! Going to spend the afternoon on namespace changes then. | 18:21 |
| *** crose has quit IRC | 18:29 | |
| *** EmilienM|afk is now known as EmilienM | 18:29 | |
| *** patrickeast has joined #openstack-cinder | 18:30 | |
| cebruns_ | assertequal | 18:32 |
| jungleboyj | jgriffith: I would like to get this fixed and pushed back up: https://review.openstack.org/145642 Any concerns with me adding the use of the fixture and pushing it back up? | 18:32 |
| *** rushiagr is now known as rushiagr_away | 18:37 | |
| *** Mandell has quit IRC | 18:37 | |
| *** Mandell has joined #openstack-cinder | 18:39 | |
| nikesh_vedams | jgriffith :ok thanks, | 18:40 |
| nikesh_vedams | jgriffith: in boot_openstack_instance.yml,name: wait for ssh to be ready single nic,timeout=120 is giving ssh timeout for me so increased to 300,that also timeout so now given 600 :) | 18:42 |
| *** e0ne has joined #openstack-cinder | 18:42 | |
| *** e0ne is now known as e0ne_ | 18:48 | |
| openstackgerrit | Corey Bryant proposed openstack/cinder: Deal with PEP-0476 certificate chaining checking https://review.openstack.org/147271 | 18:48 |
| *** jungleboyj has quit IRC | 18:49 | |
| *** e0ne_ has quit IRC | 18:53 | |
| *** Guest97290 is now known as annegentle | 18:53 | |
| *** ndipanov has quit IRC | 18:55 | |
| *** karimb has quit IRC | 18:56 | |
| *** _cjones_ has quit IRC | 18:57 | |
| *** _cjones_ has joined #openstack-cinder | 19:01 | |
| *** bruff has left #openstack-cinder | 19:01 | |
| *** vbala has left #openstack-cinder | 19:04 | |
| *** mudassirlatif has joined #openstack-cinder | 19:05 | |
| *** juzuluag has joined #openstack-cinder | 19:06 | |
| *** e0ne has joined #openstack-cinder | 19:06 | |
| *** _cjones_ has quit IRC | 19:07 | |
| *** _cjones_ has joined #openstack-cinder | 19:08 | |
| *** dannywilson has joined #openstack-cinder | 19:09 | |
| *** alexpilotti has quit IRC | 19:10 | |
| *** csky has joined #openstack-cinder | 19:10 | |
| *** annashen has quit IRC | 19:18 | |
| *** jungleboyj has joined #openstack-cinder | 19:21 | |
| *** aix has quit IRC | 19:22 | |
| *** Mandell has quit IRC | 19:22 | |
| *** Mandell has joined #openstack-cinder | 19:24 | |
| *** esker has joined #openstack-cinder | 19:28 | |
| *** xyang has quit IRC | 19:39 | |
| *** Mandell has quit IRC | 19:40 | |
| hemna | anyone run run_tests.sh lately? | 19:47 |
| hemna | it's failing locally from master on oslo_concurrency lockutils | 19:47 |
| thangp | hemna: i think jungleboyj fixed that | 19:47 |
| thangp | hemna: like last night? | 19:48 |
| hemna | http://paste.openstack.org/show/158007/ | 19:48 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder-specs: cinder-manage should use API instead of direct DB communication https://review.openstack.org/143459 | 19:48 |
| thangp | hemna: yup, saw it | 19:48 |
| jungleboyj | hemna: Yes, I fixed that yesterday. | 19:48 |
| hemna | jungleboyj, but hasn't merged yet? | 19:49 |
| hemna | or has it ? | 19:49 |
| jungleboyj | It has. | 19:49 |
| hemna | ok | 19:49 |
| hemna | I'll pull and see | 19:49 |
| jungleboyj | https://review.openstack.org/#/c/146997/ | 19:49 |
| jungleboyj | You oslo man was on top of that one. | 19:49 |
| hemna | w00t! | 19:49 |
| jungleboyj | *Your | 19:50 |
| hemna | nice job Captain Oslo! | 19:50 |
| jungleboyj | :-) | 19:50 |
| hemna | you'll be in the next Avengers movie. | 19:50 |
| *** zhipeng has quit IRC | 19:54 | |
| *** jecarey has quit IRC | 19:54 | |
| kvidvans | thingee: can enhancements be submited for new drivers added in kilo? | 19:55 |
| anish | hemna: http://ix.io/fKM is that what you are seeing ? | 19:56 |
| *** AlkaD has quit IRC | 19:56 | |
| hemna | anish, no | 19:56 |
| anish | no bueno | 19:57 |
| hemna | I did a pull master and it fixed the run_test.sh problems I was seeing. | 19:57 |
| hemna | thanks to captain oslo | 19:57 |
| * anish tries again | 19:57 | |
| tsekiyama | hemna: please give me a comment on nova-specs of multipath enhancement https://review.openstack.org/#/c/134299/ - about multipathd detection and fallback to single path setup. | 19:58 |
| *** Mandell has joined #openstack-cinder | 19:59 | |
| anish | tsekiyama: I'm doing something only mildly related, but I kinda see the same issue that you might | 20:01 |
| anish | you are saying login failure on a subset of portal when multipathing is enabled is acceptable, yes ? | 20:01 |
| *** Masahiro has joined #openstack-cinder | 20:01 | |
| *** e0ne is now known as e0ne_ | 20:01 | |
| *** AlkaD has joined #openstack-cinder | 20:02 | |
| tsekiyama | anish: yeah, that is one of the use cases of my proposal. | 20:02 |
| anish | tsekiyama: that is great, I was wondering if that would be a side effect of using multipath in conjuction with https://review.openstack.org/#/c/146233/ | 20:03 |
| *** krtaylor has quit IRC | 20:05 | |
| tsekiyama | anish: reading it .. just a moment plz | 20:05 |
| anish | sure, take your time. i'm afk for 10 min | 20:06 |
| *** Masahiro has quit IRC | 20:06 | |
| *** e0ne_ has quit IRC | 20:07 | |
| mudassirlatif | I have a question about provider_location field in the volume ….. is it ok for a driver to use this field to store information about volume location? | 20:08 |
| *** david-lyle has joined #openstack-cinder | 20:10 | |
| mudassirlatif | or in other words … is there a restriction by cinder on using this field to store driver specific data? | 20:11 |
| *** _cjones_ has quit IRC | 20:11 | |
| *** e0ne has joined #openstack-cinder | 20:12 | |
| thingee | kvidvans: yes | 20:14 |
| *** krtaylor has joined #openstack-cinder | 20:18 | |
| anish | tsekiyama: back. any comments ? | 20:22 |
| tsekiyama | How the device file looks like with offloading? something like /dev/disk/by-path/pci-0000:00.00.0-ip-<ip-address>-iscsi-<target>-<lun>? | 20:23 |
| anish | yes | 20:23 |
| tsekiyama | anish: hmm, I need to more closely on the patch, but conceptually this should work with multipath. | 20:24 |
| anish | the main issue there is that on older versions of open-iscsi, discovery does not work over an offloaded transport, though 874 does support it | 20:24 |
| anish | and you're supposed to manually add the interface parameter when you login | 20:24 |
| tsekiyama | anish: ah, then my patch will be useful to omit discovery to tell nova the all the paths to be login | 20:25 |
| anish | which is doable when doing a regular --op new, but transports are bound to interfaces, and multipath may go over multiple interfaces and cause login failures if you use transports | 20:26 |
| anish | yeah,your patch does solve any potential issues for me | 20:26 |
| tsekiyama | anish: although I will have to modify the code to merge with the patch for specifying transport, as these are modifying the same place around volume attach/detach | 20:28 |
| *** crose has joined #openstack-cinder | 20:28 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder-specs: Cinder DB Archiving https://review.openstack.org/135916 | 20:29 |
| *** harlowja is now known as harlowja_away | 20:30 | |
| *** _cjones_ has joined #openstack-cinder | 20:30 | |
| anish | tsekiyama: yes. it might be a bit complicated for you. something like try transport, if it fails then try with tcp, if that also fails, bail | 20:30 |
| anish | (unless you can come up with a better solution) | 20:31 |
| anish | this is partly mitigated when transport is iser, the target config explicitly asks for iser (someone else is looking at the code for this already) | 20:31 |
| anish | but hardware offload transports don't require any changes to remote target unlike iser and hence do not require anything target side | 20:32 |
| erlon | winston-d: hi Huang, can you you give a look in this change? https://review.openstack.org/#/c/141868/ | 20:33 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Add volume multi attach support https://review.openstack.org/85847 | 20:33 |
| erlon | winston-d: the unit tests are in a separete patch: https://review.openstack.org/#/c/146930/2 | 20:33 |
| anish | tsekiyama: the only changes that really clash are the ones in _get_host_device(s) | 20:35 |
| *** elazro has quit IRC | 20:38 | |
| *** sgotliv has joined #openstack-cinder | 20:38 | |
| *** harlowja_away is now known as harlowja | 20:38 | |
| tsekiyama | anish: right, it looks not so difficult to merge with multipath stuff | 20:38 |
| *** Mandell has quit IRC | 20:38 | |
| *** elazro has joined #openstack-cinder | 20:38 | |
| anish | tsekiyama: yep. just that whoever gets merged later needs to make changes | 20:39 |
| openstackgerrit | Abel Lopez proposed openstack/cinder: Purge deleted rows https://review.openstack.org/146766 | 20:48 |
| *** lcurtis has joined #openstack-cinder | 20:54 | |
| *** kmartin has quit IRC | 21:00 | |
| *** david-lyle has quit IRC | 21:01 | |
| *** Mandell has joined #openstack-cinder | 21:03 | |
| jungleboyj | I have a new version of the _lazy fix for stable/juno pushed up if anyone can take a look: https://review.openstack.org/#/c/145642/ | 21:06 |
| anish | jgriffith: run_tests.sh failure no longer seen after a git pull today | 21:07 |
| *** _cjones_ has quit IRC | 21:10 | |
| *** kmartin has joined #openstack-cinder | 21:10 | |
| *** AlkaD has quit IRC | 21:11 | |
| *** rhe00_ has quit IRC | 21:11 | |
| *** _cjones_ has joined #openstack-cinder | 21:13 | |
| *** rhe00_ has joined #openstack-cinder | 21:13 | |
| *** AlkaD has joined #openstack-cinder | 21:14 | |
| *** e0ne is now known as e0ne_ | 21:18 | |
| anish | question, my changeset https://review.openstack.org/#/c/146233/ changes the way the iscsiadm is used in nova. | 21:19 |
| anish | A lot of this code seems to have an exact copy in cinder/brick/initiator/connector.py | 21:19 |
| anish | do I need to change that as well ? | 21:19 |
| jgriffith | anish: yeah, hemna 's plan is to have that be shared code between the two | 21:24 |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix HNAS driver confusing error message (iSCSI driver) https://review.openstack.org/147307 | 21:24 |
| anish | jgriffith: but changes aren't automatically transplanted yet ? | 21:24 |
| *** Mandell has quit IRC | 21:25 | |
| hemna | anish, for now, yes. if changes are made in nova's iscsi/libvirt code, then those changes should get mapped into cinder brick as well. | 21:25 |
| hemna | anish, correct. | 21:25 |
| hemna | anish, I'm actually in the process of extracting cinder's brick initiator code into a separate cinder subproject called 'brick' | 21:25 |
| hemna | after that is done, cinder will get a patch to use that external lib. | 21:25 |
| hemna | and then I'll submit a patch in nova at some point to also use brick | 21:26 |
| anish | so what is the correct workflow ? If my changes get merged to nova, should I open a review for brick as well ? | 21:26 |
| hemna | anish, yes | 21:26 |
| hemna | please :) | 21:26 |
| *** markstur has quit IRC | 21:26 | |
| *** tbarron has quit IRC | 21:26 | |
| anish | easey peasey. will do as soon as nova goes through | 21:26 |
| *** tbarron has joined #openstack-cinder | 21:27 | |
| *** julim has quit IRC | 21:27 | |
| *** tbarron has quit IRC | 21:28 | |
| *** e0ne_ is now known as e0ne | 21:28 | |
| *** tbarron has joined #openstack-cinder | 21:28 | |
| *** Mandell has joined #openstack-cinder | 21:33 | |
| anish | hemna: btw, is there any plan to get things like iscsi_ip_address integrated into neutron ? | 21:35 |
| anish | I ask because of some potential issues with automation of open-iscsi transports. When using the tcp transport, routing is done automatically to figure out correct interface to use. For hardware transports on the other hand, routing information must be known beforehand, and kinda implies that user is aware of what is the correct interface to use and provides this in the iscsiadm command; something that would not be too easy to do in brick as of | 21:35 |
| anish | (sorry bout the long text) | 21:35 |
| jgriffith | anish: no, there's not | 21:36 |
| jgriffith | anish: the design has been that the data network is independent and part of infrastructure | 21:36 |
| jgriffith | anish: keeping in mind that Instances have no real knowledge of iscsi or networking | 21:37 |
| jgriffith | anish: WRT to data path at any rate | 21:37 |
| jgriffith | anish: in other words, don't forget that the iscsi initiator is ONLY on the compute node | 21:37 |
| jgriffith | anish: attach, and pass through via virt-io | 21:37 |
| anish | oh, so some manual sanity for things like iscsi_ip_address is okay then ? | 21:39 |
| jgriffith | anish: honestly I don't really know what you're referring to | 21:40 |
| jgriffith | as the initiator code needs and should handle all of that | 21:40 |
| jgriffith | no, there should not be manual "work" in there | 21:40 |
| *** e0ne is now known as e0ne_ | 21:40 | |
| *** diegows has quit IRC | 21:42 | |
| *** e0ne_ is now known as e0ne | 21:42 | |
| *** david-lyle has joined #openstack-cinder | 21:42 | |
| anish | jgriffith: so a one sentence explanation. if you have 4 interfaces, with only one of them actually connected to a target, the libiscsi kernel modules figure out the correct interface to use for you | 21:42 |
| *** e0ne has quit IRC | 21:43 | |
| hemna | isn't that a routing problem on the host ? | 21:44 |
| anish | now if you use a hardware transport with open-iscsi (via the -I param), then libiscsi will only try the interface you tell it to try | 21:44 |
| hemna | anish, how do you determine which interface to use ? | 21:44 |
| anish | you sorta have to know before hand. As the hardware transports are as the name implies retricted to certain hardware,you give it a 3-tuple of mac address, ip & a name to use for transport | 21:45 |
| hemna | anish, other than keep trying until you find one that successfully creates an iSCSI session ? | 21:45 |
| anish | for the default transport 'tcp' this is the case, yes | 21:46 |
| anish | for an offloaded hardware transport, this is not possible as it is bound to the 3-tuple you provided | 21:46 |
| hemna | but don't you have to tell the target side what 3-tuple you export to as well ? | 21:47 |
| *** chlong has quit IRC | 21:47 | |
| anish | no, that is only needed for iser | 21:47 |
| hemna | almost like how FC works | 21:47 |
| anish | is is transparent to target side when used | 21:48 |
| hemna | who determines which h/w interface to use ? | 21:48 |
| anish | this gets muddy only because iser is also classified as a transport, but it is not transparent | 21:48 |
| anish | hemna: left to user | 21:48 |
| hemna | what 'user' in this case? | 21:48 |
| hemna | user is a loaded term here. | 21:48 |
| hemna | admin? | 21:49 |
| anish | yes | 21:49 |
| anish | sorry, admin | 21:49 |
| hemna | so that would be a cinder.conf entry ? | 21:49 |
| *** david-lyle has quit IRC | 21:49 | |
| anish | well it's a nova.conf entry currently | 21:49 |
| anish | since it's only needed on initiator side | 21:49 |
| hemna | something has to pass to nova/brick the h/w transport to use instead of the default of TCP today | 21:49 |
| *** thangp has quit IRC | 21:50 | |
| *** Masahiro has joined #openstack-cinder | 21:50 | |
| anish | right now, nova.conf does the job | 21:50 |
| hemna | still trying to wrap my head around this | 21:51 |
| hemna | so, you could have N h/w transports over different interfaces | 21:51 |
| anish | yes | 21:51 |
| *** sgotliv has quit IRC | 21:51 | |
| anish | (my current code only allows for one) | 21:51 |
| hemna | and one of those is the right one connected to the cinder array ? | 21:51 |
| anish | yes | 21:52 |
| *** r-daneel has quit IRC | 21:53 | |
| hemna | sounds like the initiator side needs to know which h/w transport is associated with which target host. | 21:54 |
| anish | as tp N h/w transports, the correct way to see this would be every mac-ip combo can have one transport. Most vendors only allow one mac-ip combo per mac | 21:54 |
| hemna | or, you have to try each one until it works. | 21:54 |
| *** Masahiro has quit IRC | 21:54 | |
| hemna | each cinder iSCSI driver returns a target_portal | 21:55 |
| hemna | which contains ip:port # | 21:55 |
| hemna | this sounds like a major pain for admins to maintain | 21:56 |
| anish | you essentially need to know if the ip is reacheable via the transport, yes | 21:56 |
| hemna | if you have to manually know which initiator-target pairs are associated | 21:56 |
| anish | well, you could just put them all on the same subnet | 21:56 |
| hemna | FC gets this for free, by the zone manager lookup service | 21:56 |
| anish | at least that is how I have seen it done | 21:57 |
| hemna | which basically returns a list of initiator-target maps that see each other. | 21:57 |
| anish | I don't think there's an iscsi equivalent, it's always same subnet + ACL as far as I have seen | 21:58 |
| anish | but I'm on the dev side, not deployment | 21:58 |
| *** nellysmitt has quit IRC | 21:59 | |
| hemna | is there a way to inquire the h/w transport which hosts it sees ? | 21:59 |
| *** annashen has joined #openstack-cinder | 21:59 | |
| hemna | yah with TCP, you just let the routing table and ARP do the work | 21:59 |
| anish | ping maybe ? | 21:59 |
| hemna | yah possibly | 21:59 |
| hemna | I don't know enough about the h/w transports | 22:00 |
| hemna | but I'm assuming they would encapsulate iSCSI ( and hence TCP) traffic over the wire | 22:00 |
| hemna | and hope they support ICMP | 22:00 |
| hemna | :P | 22:00 |
| anish | think of it as ifconfig | grep for mac id | 22:00 |
| anish | they do, it's jsut an abstraction over regular nics | 22:01 |
| anish | 'regular' as in 'regular' nics that understand an iscsi packet in hardware | 22:01 |
| hemna | ok then a ping on each interface first prior to calling login | 22:01 |
| hemna | just cycle through each interface until ping with a timeout. | 22:02 |
| hemna | if you get a reply, use that one. if not, next. | 22:02 |
| hemna | no replies = impossible to attach. | 22:02 |
| hemna | but | 22:03 |
| hemna | some network admins disable ICMP | 22:03 |
| hemna | fwiw | 22:03 |
| anish | does not seem like a graceful way either | 22:03 |
| anish | yeah, target arrays may not support ICMP either | 22:03 |
| hemna | but at least this way, the admin doesn't have to pre configure/manage an association dictionary... FOR EACH compute host. | 22:03 |
| hemna | which just seems like an epic fail | 22:04 |
| anish | sadly yes | 22:04 |
| anish | however, even then | 22:04 |
| anish | someone has to make a h/w transport configuration to try | 22:04 |
| hemna | they do? | 22:04 |
| hemna | :P | 22:04 |
| anish | yes | 22:04 |
| anish | otherwise it would be more like this : | 22:05 |
| openstackgerrit | Anthony Lee proposed openstack/cinder: Remove locks from LeftHand driver https://review.openstack.org/141475 | 22:05 |
| anish | find pinging interface | 22:05 |
| anish | lookup pci id of pinging interface to see if it's one of the 7 known transport vendors | 22:05 |
| anish | figure out transport name based on pci id, generate hw transport | 22:05 |
| anish | then proceed | 22:05 |
| anish | I feel like that is instant gerrit rejection | 22:06 |
| hemna | hehe | 22:06 |
| *** rajinir has joined #openstack-cinder | 22:06 | |
| hemna | well you can always try | 22:07 |
| *** rajinir is now known as rajini | 22:07 | |
| *** markstur has joined #openstack-cinder | 22:07 | |
| *** rajini is now known as Guest58524 | 22:07 | |
| *** Guest58524 has quit IRC | 22:08 | |
| boris-42 | jgriffith: hi | 22:09 |
| *** Guest58524 has joined #openstack-cinder | 22:09 | |
| boris-42 | jgriffith: how are you? | 22:09 |
| *** sgotliv has joined #openstack-cinder | 22:09 | |
| jgriffith | boris-42: hey | 22:10 |
| *** _cjones_ has quit IRC | 22:11 | |
| boris-42 | jgriffith: so is there any progress on attach issue?) | 22:11 |
| jgriffith | boris-42: you tell me | 22:12 |
| jgriffith | boris-42: what's rally saying now? | 22:12 |
| boris-42 | jgriffith: so today heat/neutron break rally gates | 22:12 |
| boris-42 | jgriffith: and we are trying to disable that scenario | 22:12 |
| jgriffith | boris-42: well that's not helpful | 22:12 |
| boris-42 | jgriffith: and we can't cause we are facing cinder attach=) | 22:12 |
| jgriffith | boris-42: so I haven't looked since I put the extra debug messages in | 22:12 |
| jgriffith | boris-42: let me do a Kibana search.... | 22:13 |
| boris-42 | jgriffith: http://logs.openstack.org/90/146490/5/gate/gate-rally-dsvm-rally/aead4fe/logs/ | 22:13 |
| boris-42 | jgriffith: this is job that failed | 22:13 |
| *** _cjones_ has joined #openstack-cinder | 22:13 | |
| boris-42 | jgriffith: and another one http://logs.openstack.org/90/146490/5/gate/gate-rally-dsvm-rally/641cfd6/logs/ | 22:14 |
| *** akerr has quit IRC | 22:15 | |
| *** Yogi1 has quit IRC | 22:15 | |
| *** juzuluag has quit IRC | 22:16 | |
| anish | hemna: wait no, that is not 100% correct. you can do it without user intervention. iscsiadm is capabale of making h/w transport configurations on its own. You'll just end up with every possible configuration that the present hardware to test/ping etc | 22:16 |
| jgriffith | boris-42: http://logs.openstack.org/90/146490/5/gate/gate-rally-dsvm-rally/aead4fe/logs/screen-c-vol.txt.gz#_2015-01-14_21_49_53_112 | 22:16 |
| anish | *present hardware is capable of to test/ping and find the relevant one | 22:17 |
| jgriffith | so we sent a create target call... but clearly the target is already there | 22:17 |
| jgriffith | boris-42: hmmm | 22:19 |
| jgriffith | boris-42: ok, so interesting | 22:19 |
| *** esker has quit IRC | 22:19 | |
| jgriffith | boris-42: the persistence file is there | 22:19 |
| jgriffith | but the target itself is never created | 22:20 |
| boris-42 | jgriffith: magic* | 22:20 |
| jgriffith | boris-42: well... not really, just means tgt get's "hosed" | 22:20 |
| jgriffith | boris-42: notice also that the ONLY Lun is the controller lun | 22:21 |
| jgriffith | boris-42: in the "targets after udpate" display | 22:21 |
| jgriffith | boris-42: BUT | 22:21 |
| jgriffith | further down you see the tgt created for 35d9ce.... | 22:22 |
| jgriffith | boris-42: so I *think* we have a race here | 22:22 |
| jgriffith | in other both of these calls are coming at the same time and tgt pukes | 22:22 |
| jgriffith | s/in other/in other words/ | 22:23 |
| *** diegows has joined #openstack-cinder | 22:23 | |
| boris-42 | jgriffith: but why we have few for one tgt? | 22:23 |
| jgriffith | boris-42: not sure what you mean? | 22:24 |
| boris-42 | why there are few calls | 22:24 |
| boris-42 | at all | 22:24 |
| boris-42 | we are attaching different volumes | 22:24 |
| jgriffith | sure, but we're running concurrent tests right? | 22:25 |
| boris-42 | yep | 22:25 |
| boris-42 | we have simultaneouly N attaches | 22:25 |
| jgriffith | boris-42: so we can actually have two create target calls come in at the same time | 22:25 |
| jgriffith | boris-42: which would confirm it's a race and why we don't see 100% of the time | 22:26 |
| jgriffith | boris-42: of course I could be completely wrong, but this is sure what it looks like | 22:26 |
| jgriffith | boris-42: so the good thing is there are easy hacks to try and fix this now that we know what is going on | 22:26 |
| jgriffith | boris-42: It should be simple enough to do a retry, maybe even put a sleep in there | 22:27 |
| jgriffith | boris-42: some might say "put in a lock" | 22:27 |
| jgriffith | so we only do one at a time | 22:27 |
| jgriffith | might not be a bad idea either (as much as I hate locks) | 22:27 |
| jgriffith | boris-42: know what I'm sayin? | 22:28 |
| *** rushil has quit IRC | 22:29 | |
| boris-42 | jgriffith: I would try to avoid locks (cause as far as I understand they should be distributed in case of few drivers instaces?) | 22:30 |
| jgriffith | boris-42: I agree | 22:30 |
| boris-42 | jgriffith: so retries seems a bit better idea | 22:30 |
| *** EmilienM is now known as EmilienM|afk | 22:32 | |
| jgriffith | boris-42: something like this maybe: https://gist.github.com/j-griffith/f3e36a57531259b27a67 | 22:32 |
| jgriffith | boris-42: probably a sleep inbetween wouldn't be a terrible thing | 22:32 |
| *** david-lyle has joined #openstack-cinder | 22:33 | |
| boris-42 | jgriffith: btw | 22:34 |
| boris-42 | jgriffith: we can test this solution easily | 22:34 |
| boris-42 | jgriffith: just make a very very long task for rally | 22:34 |
| boris-42 | in the depending patch | 22:34 |
| jgriffith | Oh... good idea | 22:35 |
| boris-42 | and recheck few times | 22:35 |
| boris-42 | if it pass => good one | 22:35 |
| jgriffith | so let me look at some ideas | 22:35 |
| jgriffith | you be around for a little bit? | 22:35 |
| boris-42 | yep for few hours | 22:35 |
| jgriffith | I'll get a patch ready but I want to look at a couple other things first | 22:35 |
| jgriffith | there might be a better way to do these updates | 22:35 |
| boris-42 | sure | 22:35 |
| jgriffith | like an "update ALL" | 22:35 |
| *** dustins has quit IRC | 22:40 | |
| *** kaufer has quit IRC | 22:45 | |
| openstackgerrit | Gary W. Smith proposed openstack/python-cinderclient: Expose cinder's scheduler pool API https://review.openstack.org/140142 | 22:45 |
| *** olc has quit IRC | 22:45 | |
| openstackgerrit | Tomoki Sekiyama proposed openstack/cinder: TgtAdm: Don't change CHAP username/password on live migration https://review.openstack.org/145391 | 22:45 |
| *** crose has quit IRC | 22:46 | |
| *** Guest58524 has quit IRC | 22:53 | |
| *** jamielennox|away is now known as jamielennox | 22:56 | |
| *** lpetrut has quit IRC | 22:59 | |
| *** ryanpetrello has quit IRC | 23:02 | |
| *** marcusvrn has quit IRC | 23:02 | |
| *** ryanpetrello has joined #openstack-cinder | 23:03 | |
| *** marcusvrn has joined #openstack-cinder | 23:03 | |
| jgriffith | boris-42: let's try and experiment :) | 23:03 |
| jgriffith | boris-42: rather than do updates on specific targets, why not do an update on all targets in the persist file | 23:04 |
| jgriffith | this may have a bad side effect that I'm not thinking of, but I'm not sure what | 23:04 |
| jgriffith | boris-42: it should be *safe* for active targets as long as I don't use forece | 23:04 |
| jgriffith | force | 23:04 |
| boris-42 | jgriffith: let's try | 23:05 |
| boris-42 | jgriffith: i was playing with logstach | 23:05 |
| boris-42 | jgriffith: | 23:05 |
| boris-42 | http://logstash.openstack.org/#eyJzZWFyY2giOiJtZXNzYWdlOlwiY3JlYXRlIHRhcmdldCBiZWNhdXNlIGl0IGFscmVhZHkgZXhpc3RzXCIgYW5kIGJ1aWxkX25hbWU6XCJnYXRlLXJhbGx5LWRzdm0tY2luZGVyXCIiLCJmaWVsZHMiOltdLCJvZmZzZXQiOjAsInRpbWVmcmFtZSI6ImN1c3RvbSIsImdyYXBobW9kZSI6ImNvdW50IiwidGltZSI6eyJmcm9tIjoiMjAxNS0wMS0wMVQyMjo1OTo1MyswMDowMCIsInRvIjoiMjAxNS0wMS0xNFQyMjo1OTo1MyswMDowMCIsI | 23:05 |
| boris-42 | nVzZXJfaW50ZXJ2YWwiOiIwIn0sInN0YW1wIjoxNDIxMjc2NjQyMjUzfQ== | 23:05 |
| boris-42 | jgriffith: interesting that there were no messages 5 Jan | 23:05 |
| *** tbarron has quit IRC | 23:06 | |
| boris-42 | jgriffith: and seems like adding more benchmarks to rally task in cinder start causing more failures of rally job | 23:06 |
| boris-42 | jgriffith: in cinder | 23:06 |
| jgriffith | boris-42: so actually that kinda makes sense (I think) | 23:06 |
| jgriffith | boris-42: since I believe it's an issue of overlapped tgt update calls, increasing the load increases the likliehood of hitting overlaps | 23:07 |
| jgriffith | boris-42: now... | 23:07 |
| jgriffith | boris-42: what would be interesting is if there's a pattern | 23:07 |
| jgriffith | boris-42: like "it's always the n'th target" etc | 23:07 |
| boris-42 | jgriffith: hm that will be hard to check | 23:08 |
| jgriffith | boris-42: sadly it's purely manual :( | 23:09 |
| jgriffith | boris-42: likely not worth thinking about yet | 23:09 |
| boris-42 | jgriffith: we can try to run rally N times and collect info about what itereations failed actually | 23:09 |
| *** _cjones_ has quit IRC | 23:09 | |
| boris-42 | jgriffith: iteration is more or less ID | 23:09 |
| jgriffith | ahh | 23:10 |
| *** _cjones_ has joined #openstack-cinder | 23:11 | |
| boris-42 | jgriffith: going to burn my laptop | 23:11 |
| boris-42 | =) | 23:11 |
| boris-42 | jgriffith: btw | 23:12 |
| boris-42 | jgriffith: I finished really cool stuff | 23:12 |
| boris-42 | jgriffith: for such kind fo testing | 23:12 |
| boris-42 | jgriffith: https://review.openstack.org/#/c/137716/ | 23:12 |
| boris-42 | jgriffith: so you don't need to copy-paste by hands in task, you can write loop=) | 23:12 |
| boris-42 | jgriffith: like goal you would like to test that all images works with all flaovrs | 23:14 |
| boris-42 | jgriffith: you can write such stuff https://gist.github.com/boris-42/2d3ee67d7679b9db4565 | 23:14 |
| jungleboyj | Anyone else seeing that it looks like unit test is broken again? | 23:16 |
| jgriffith | boris-42: let's see what that does | 23:16 |
| jgriffith | oops... I beat it | 23:16 |
| openstackgerrit | John Griffith proposed openstack/cinder: Use tgt-admin --update ALL https://review.openstack.org/147328 | 23:16 |
| jgriffith | I beat the internet... I'm faster than the internet!!!! | 23:16 |
| *** david-lyle has quit IRC | 23:16 | |
| *** lcurtis has quit IRC | 23:17 | |
| jgriffith | jungleboyj: haven't tried a fresh setup today... doing so now | 23:17 |
| jgriffith | jungleboyj: anything in Jenkins? | 23:17 |
| jgriffith | errr...gate? | 23:17 |
| jgriffith | jungleboyj: looks okie-dokie to me | 23:17 |
| jungleboyj | Haven't looked that far yet. Trying a fresh tree pull. | 23:18 |
| boris-42 | jgriffith: lol | 23:18 |
| *** _cjones_ has quit IRC | 23:18 | |
| jungleboyj | jgriffith: Getting 'no such file or directory' looking for cinder-rtstool in a number of test cases. | 23:19 |
| *** erlon has quit IRC | 23:20 | |
| jgriffith | jungleboyj: yup, just hit it | 23:21 |
| jungleboyj | Ok. Good. Couldn't figure out how my namespace changes could cause that. | 23:22 |
| *** AlkaD has quit IRC | 23:24 | |
| jgriffith | jungleboyj: thingee It appears at first glance that rtstool updated | 23:24 |
| jgriffith | and be broken | 23:24 |
| jungleboyj | :-( | 23:25 |
| *** _cjones_ has joined #openstack-cinder | 23:25 | |
| jungleboyj | jgriffith: I think, though, that it is this commit as all the test cases I see failing are in here: https://github.com/openstack/cinder/commit/29f9fa9f2a7a37231398acd442388ad328a5ec50 | 23:26 |
| jgriffith | jungleboyj: thingee 2.1.51 | 23:26 |
| thingee | jgriffith: didn't you have a patch recently that resolved this by adding an additional path of where rtstool could reside | 23:26 |
| jungleboyj | Is rtstool set up as a test requirement? | 23:26 |
| thingee | jgriffith: at least for debian based | 23:27 |
| thingee | jungleboyj: it's in cinder/cmd/cinder-rtstool | 23:27 |
| jgriffith | jungleboyj: it's in requirements | 23:27 |
| thingee | rtslib-fb is part of requirements | 23:27 |
| jgriffith | looking... probably right that it's related to that commit somehow | 23:27 |
| jungleboyj | jgriffith: In each case it is just doing a 'cinder-rtstool verify' and not finding cinder-rtstool . | 23:28 |
| thingee | jungleboyj: link to whatever is failing? | 23:28 |
| jgriffith | thingee: jungleboyj so that commit added lio driver tests | 23:30 |
| *** lcurtis has joined #openstack-cinder | 23:30 | |
| jungleboyj | thingee: http://paste.openstack.org/show/158042/ | 23:30 |
| jgriffith | in cinder.tests.targets.test_lio_driver | 23:30 |
| jgriffith | but doesn't fake the rtstool calls | 23:30 |
| jgriffith | so "boom" | 23:30 |
| jungleboyj | jgriffith: Ah. | 23:30 |
| jgriffith | jungleboyj: thingee verifying now, will push a short term fix for now | 23:31 |
| *** sgotliv has quit IRC | 23:31 | |
| jgriffith | assuming this works :) | 23:31 |
| thingee | ha that's why I couldn't find the commit | 23:31 |
| jgriffith | just removing the test_lio file for now | 23:31 |
| thingee | "Remove unused variables from ensure_export() " | 23:31 |
| jungleboyj | Ok. | 23:31 |
| thingee | that's a really bad summary that this commit introduces tests for targets as well | 23:32 |
| thingee | mtanino: ^ | 23:32 |
| jgriffith | thingee: partially my fault, he pointed out there were no LIO tests and I said.. yeah you should add some :) | 23:32 |
| jgriffith | the tests that are in there are actually to test his change | 23:32 |
| mtanino | thingee: hi | 23:32 |
| thingee | mtanino: read scrollback please. | 23:33 |
| mtanino | thingee: I see. | 23:33 |
| *** _cjones_ has quit IRC | 23:33 | |
| openstackgerrit | Richard Hagarty proposed openstack/cinder: Set snapCPG when managing existing volumes https://review.openstack.org/147258 | 23:33 |
| jungleboyj | Phew, I didn't merge it. :-) | 23:33 |
| jgriffith | mtanino: thingee it's not really a problem, and since we "require" unit tests well... what would you expect :) | 23:33 |
| jgriffith | jungleboyj: well... if you run it on my ubuntu system it works fine :) | 23:34 |
| *** dannywil_ has joined #openstack-cinder | 23:34 | |
| thingee | jgriffith: :) | 23:34 |
| jungleboyj | jgriffith: ? How is that? | 23:34 |
| thingee | jgriffith: I'll add a skip for now on the test class? | 23:34 |
| jgriffith | jungleboyj: just as if you run in gate it's fine | 23:34 |
| thingee | jungleboyj: he has it installed | 23:34 |
| *** dannywil_ has quit IRC | 23:34 | |
| jungleboyj | thingee: jgriffith Ah, yes, then it would work. | 23:35 |
| jgriffith | thingee: mtanino so that's what I had, and yes that solves any issues | 23:35 |
| jgriffith | thingee: I'll let you push that up | 23:35 |
| *** dannywil_ has joined #openstack-cinder | 23:35 | |
| thingee | jgriffith: k | 23:35 |
| mtanino | jgriffith: Thank you. | 23:35 |
| jgriffith | mtanino: you want to work on a fake for that at some point? Maybe open a bug on it? | 23:35 |
| jgriffith | mtanino: if you have time | 23:35 |
| jgriffith | mtanino: if not I'm sure one of us can pick it up as well | 23:35 |
| *** patrickeast_ has joined #openstack-cinder | 23:36 | |
| mtanino | jgriffith: Could you push it? | 23:36 |
| *** patrickeast has quit IRC | 23:36 | |
| *** patrickeast_ is now known as patrickeast | 23:36 | |
| jgriffith | for sure | 23:36 |
| jgriffith | well... I think thingee has it already qued up | 23:36 |
| jgriffith | :) | 23:36 |
| *** mudassirlatif has quit IRC | 23:36 | |
| jgriffith | I'll file a bug and mock out the rts calls | 23:37 |
| *** mudassirlatif has joined #openstack-cinder | 23:37 | |
| jgriffith | no big deal.... not a crisis like the crazy stuff last week | 23:37 |
| jgriffith | :) | 23:37 |
| jgriffith | this one is ez/pz | 23:37 |
| *** dannywilson has quit IRC | 23:37 | |
| thingee | jgriffith: don't worry about writing something to fake it, it's my target afterall...well forked target thing | 23:37 |
| *** madskier has quit IRC | 23:38 | |
| thingee | I' | 23:38 |
| thingee | I'd rather eharney write the fakes :P | 23:38 |
| *** ebalduf has joined #openstack-cinder | 23:38 | |
| *** Masahiro has joined #openstack-cinder | 23:39 | |
| thingee | is anyone already filing a bug? | 23:39 |
| *** _cjones_ has joined #openstack-cinder | 23:40 | |
| jungleboyj | thingee: I can write one. | 23:40 |
| thingee | jungleboyj: thanks | 23:41 |
| jgriffith | thingee: ? | 23:41 |
| jgriffith | all you need to do is mock out a few of the calls in the test | 23:41 |
| jgriffith | shouldn't be a big deal | 23:41 |
| jgriffith | don't know why eharney has to do it but... ok | 23:41 |
| thingee | jgriffith: it was a joke. I was talking about it's my target thing...well the forked version and I was poking fun since it's a redhat fork | 23:42 |
| jgriffith | thingee: lol... ahh, you funny guy | 23:42 |
| jgriffith | bb in a bit | 23:42 |
| *** ebalduf has quit IRC | 23:42 | |
| *** Masahiro has quit IRC | 23:43 | |
| thingee | jungleboyj: let me know when you got a bug # | 23:44 |
| jungleboyj | https://bugs.launchpad.net/cinder/+bug/1411029 | 23:45 |
| jungleboyj | thingee: ^^ | 23:45 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Fix LOG formatting in api initialize_connection https://review.openstack.org/147336 | 23:46 |
| *** ryanpetrello has quit IRC | 23:46 | |
| hemna | we need a ninja for ^^^ | 23:46 |
| hemna | https://bugs.launchpad.net/cinder/+bug/1411026 | 23:47 |
| hemna | oopsie | 23:47 |
| *** kvidvans has quit IRC | 23:48 | |
| openstackgerrit | Mike Perez proposed openstack/cinder: Skip LIO target drivers until properly mocked https://review.openstack.org/147338 | 23:49 |
| thingee | hemna, jungleboyj ^^ | 23:49 |
| *** lcurtis has quit IRC | 23:49 | |
| hemna | doh | 23:50 |
| openstackgerrit | Mike Perez proposed openstack/cinder: Skip LIO target unit tests until mocked https://review.openstack.org/147338 | 23:50 |
| hemna | jgriffith, thanks man | 23:51 |
| thingee | hemna: sorry one more time...fixed summary of commit | 23:51 |
| thingee | hemna: thanks | 23:51 |
| hemna | np | 23:51 |
| *** mriedem has quit IRC | 23:52 | |
| thingee | ninja'd | 23:52 |
| boris-42 | jgriffith: lol | 23:53 |
| *** smoriya has joined #openstack-cinder | 23:53 | |
| boris-42 | jgriffith: today we have happy day | 23:53 |
| jungleboyj | Was about to +2/+A ... You Ninja. | 23:53 |
| mtanino | thingee: Thank you for the patch. | 23:54 |
| boris-42 | jgriffith: 3 times attach race for one patch=) | 23:54 |
| hemna | jungleboyj, that was my first official ninja | 23:56 |
| hemna | heh | 23:56 |
| *** dalgaaf has quit IRC | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!