| *** haomaiwang has quit IRC | 00:01 | |
| *** haomaiwang has joined #openstack-cinder | 00:01 | |
| *** alonma has quit IRC | 00:03 | |
| *** angela-s has quit IRC | 00:05 | |
| *** ntpttr1 has joined #openstack-cinder | 00:09 | |
| *** ntpttr1 has quit IRC | 00:11 | |
| *** ntpttr has quit IRC | 00:11 | |
| *** garthb_ has joined #openstack-cinder | 00:12 | |
| *** garthb has quit IRC | 00:13 | |
| *** chris_morrell has quit IRC | 00:15 | |
| *** chris_morrell has joined #openstack-cinder | 00:15 | |
| *** pratap has quit IRC | 00:20 | |
| *** lcurtis has quit IRC | 00:21 | |
| *** yhayashi has joined #openstack-cinder | 00:22 | |
| *** pratap has joined #openstack-cinder | 00:25 | |
| *** harlowja_ has joined #openstack-cinder | 00:27 | |
| *** harlowja has quit IRC | 00:30 | |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: LeftHand: Implement v2 replication (unmanaged) https://review.openstack.org/255544 | 00:45 |
|---|---|---|
| *** smoriya_ has joined #openstack-cinder | 00:46 | |
| fernnest | smcginnis, HP XP driver completed successfully on patch https://review.openstack.org/#/c/263515/ | 00:47 |
| *** zhangjn has joined #openstack-cinder | 00:57 | |
| *** alonma has joined #openstack-cinder | 00:59 | |
| *** haomaiwang has quit IRC | 01:01 | |
| *** haomaiwang has joined #openstack-cinder | 01:01 | |
| *** daneyon has joined #openstack-cinder | 01:01 | |
| *** alonma has quit IRC | 01:04 | |
| *** cheneydc has joined #openstack-cinder | 01:07 | |
| *** eharney has quit IRC | 01:08 | |
| *** alonma has joined #openstack-cinder | 01:10 | |
| *** alonma has quit IRC | 01:14 | |
| *** alonma has joined #openstack-cinder | 01:16 | |
| guitarzan | hemna: cool, thanks, I'm guessing my rootwrap stuff is all messed up | 01:16 |
| *** EinstCrazy has joined #openstack-cinder | 01:17 | |
| *** davechen has joined #openstack-cinder | 01:19 | |
| *** pratap has quit IRC | 01:21 | |
| *** alonma has quit IRC | 01:22 | |
| *** alonma has joined #openstack-cinder | 01:24 | |
| *** zhangjn has quit IRC | 01:28 | |
| *** zhangjn has joined #openstack-cinder | 01:29 | |
| *** garthb_ has quit IRC | 01:29 | |
| *** julim has quit IRC | 01:30 | |
| *** julim has joined #openstack-cinder | 01:30 | |
| openstackgerrit | huangtianhua proposed openstack/python-cinderclient: Make _discover_extensions public https://review.openstack.org/259250 | 01:30 |
| *** akerr has quit IRC | 01:31 | |
| openstackgerrit | Merged openstack/cinder: Allow replicated volumes to be recoverable https://review.openstack.org/219900 | 01:31 |
| *** alonma has quit IRC | 01:31 | |
| *** akerr has joined #openstack-cinder | 01:32 | |
| *** haomaiwang has quit IRC | 01:32 | |
| *** manous has quit IRC | 01:32 | |
| *** manous has joined #openstack-cinder | 01:32 | |
| *** [1]Thelo has joined #openstack-cinder | 01:33 | |
| *** alonma has joined #openstack-cinder | 01:34 | |
| *** leeantho has quit IRC | 01:35 | |
| *** Thelo has quit IRC | 01:36 | |
| *** [1]Thelo is now known as Thelo | 01:36 | |
| *** alonma has quit IRC | 01:38 | |
| *** cknight has joined #openstack-cinder | 01:39 | |
| *** akerr has quit IRC | 01:39 | |
| *** alonma has joined #openstack-cinder | 01:40 | |
| *** akerr has joined #openstack-cinder | 01:41 | |
| *** davechen1 has joined #openstack-cinder | 01:43 | |
| *** martyturner has quit IRC | 01:44 | |
| *** alonma has quit IRC | 01:45 | |
| *** davechen has quit IRC | 01:46 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Cleanup in backup reset status https://review.openstack.org/246748 | 01:47 |
| *** davechen has joined #openstack-cinder | 01:48 | |
| *** davechen1 has quit IRC | 01:50 | |
| *** Lee1092 has joined #openstack-cinder | 01:51 | |
| *** mtanino has quit IRC | 02:07 | |
| *** manous has quit IRC | 02:13 | |
| *** rajinir has quit IRC | 02:15 | |
| *** davechen has quit IRC | 02:15 | |
| *** zhangjn has quit IRC | 02:17 | |
| *** zhangjn has joined #openstack-cinder | 02:18 | |
| *** 14WAALZU3 has joined #openstack-cinder | 02:20 | |
| *** pratap has joined #openstack-cinder | 02:20 | |
| *** houming has joined #openstack-cinder | 02:21 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Add restore_volume_id in backup https://review.openstack.org/260903 | 02:31 |
| openstackgerrit | Einst Crazy proposed openstack/cinder: Replace assertEqual(*, None) with assertIsNone in tests https://review.openstack.org/263997 | 02:39 |
| *** zhangjn has quit IRC | 02:39 | |
| *** mudassirlatif has quit IRC | 02:39 | |
| *** alonma has joined #openstack-cinder | 02:41 | |
| *** zhangjn has joined #openstack-cinder | 02:42 | |
| openstackgerrit | Thelo Gaultier proposed openstack/os-brick: Add connector for ITRI DISCO cinder driver https://review.openstack.org/253352 | 02:43 |
| *** akerr_ has joined #openstack-cinder | 02:44 | |
| *** alonma has quit IRC | 02:45 | |
| *** akerr has quit IRC | 02:47 | |
| *** alonma has joined #openstack-cinder | 02:47 | |
| *** daneyon has quit IRC | 02:51 | |
| *** sasukeh has joined #openstack-cinder | 02:51 | |
| *** alonma has quit IRC | 02:52 | |
| *** alonma has joined #openstack-cinder | 02:54 | |
| *** zhangjn has quit IRC | 02:54 | |
| *** pratap has quit IRC | 02:55 | |
| *** rushiagr_away is now known as rushiagr | 02:57 | |
| *** zhangjn has joined #openstack-cinder | 02:58 | |
| *** alonma has quit IRC | 02:59 | |
| *** dims has quit IRC | 02:59 | |
| *** 14WAALZU3 has quit IRC | 03:01 | |
| *** haomaiwang has joined #openstack-cinder | 03:01 | |
| *** alonma has joined #openstack-cinder | 03:01 | |
| *** alonma has quit IRC | 03:06 | |
| *** cknight has quit IRC | 03:07 | |
| *** alonma has joined #openstack-cinder | 03:11 | |
| *** sasukeh_ has joined #openstack-cinder | 03:12 | |
| *** gouthamr has quit IRC | 03:13 | |
| *** sasukeh has quit IRC | 03:15 | |
| *** alonma has quit IRC | 03:16 | |
| *** alonma has joined #openstack-cinder | 03:18 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Scaling backup service https://review.openstack.org/262395 | 03:18 |
| openstackgerrit | LisaLi proposed openstack/cinder: Add restore_volume_id in backup https://review.openstack.org/260903 | 03:18 |
| *** daneyon has joined #openstack-cinder | 03:19 | |
| openstackgerrit | Reedip proposed openstack/python-cinderclient: Fix argument order for assertEqual to (expected, observed) https://review.openstack.org/264006 | 03:20 |
| *** alonma has quit IRC | 03:22 | |
| *** aorourke_ has joined #openstack-cinder | 03:23 | |
| *** bardia has quit IRC | 03:31 | |
| openstackgerrit | Reedip proposed openstack/python-cinderclient: Fix db error when running python34 unit tests https://review.openstack.org/264009 | 03:37 |
| *** zhangjn has quit IRC | 03:38 | |
| *** aorourke_ has quit IRC | 03:41 | |
| *** aorourke_ has joined #openstack-cinder | 03:42 | |
| *** aorourke_ has joined #openstack-cinder | 03:42 | |
| *** aorourke_ has quit IRC | 03:43 | |
| *** aorourke_ has joined #openstack-cinder | 03:43 | |
| openstackgerrit | huanan proposed openstack/cinder: Huawei: Add manage/unmanage snapshot support https://review.openstack.org/261156 | 03:48 |
| *** daneyon has quit IRC | 03:49 | |
| *** rat_ has joined #openstack-cinder | 03:50 | |
| *** links has joined #openstack-cinder | 03:51 | |
| rat_ | general problems faced by a basic guy with cinder? | 03:52 |
| *** haomaiwang has quit IRC | 04:01 | |
| *** haomaiwang has joined #openstack-cinder | 04:01 | |
| *** coolsvap|away is now known as coolsvap | 04:06 | |
| openstackgerrit | Merged openstack/cinder: Remove name_id when creating volume with cache enabled https://review.openstack.org/259937 | 04:11 |
| *** PsionTheory has quit IRC | 04:13 | |
| *** rushiagr is now known as rushiagr_away | 04:14 | |
| *** Pandy has joined #openstack-cinder | 04:16 | |
| *** shyama has joined #openstack-cinder | 04:16 | |
| Pandy | hi all | 04:16 |
| Pandy | am getting colume hangs in creating states forever, only restart of cinder-volume service on compute node helps | 04:17 |
| Pandy | how can i rectify it ? | 04:17 |
| Pandy | m getting Volume hangs in creating states forever, only restart of cinder-volume service on compute node helps | 04:17 |
| *** shyama has quit IRC | 04:18 | |
| *** alonma has joined #openstack-cinder | 04:19 | |
| jgriffith | Pandy: what backend? And do you have a pastebin of any of the logs? | 04:20 |
| *** alonma has quit IRC | 04:23 | |
| *** alonma has joined #openstack-cinder | 04:27 | |
| *** coolsvap is now known as coolsvap|away | 04:29 | |
| *** alonma has quit IRC | 04:31 | |
| *** alonma has joined #openstack-cinder | 04:35 | |
| *** rat_ has quit IRC | 04:35 | |
| *** alonma has quit IRC | 04:40 | |
| Pandy | ceph is backend | 04:40 |
| Pandy | here same launchpad bug : /oslo.utils/+bug/1502092 | 04:48 |
| openstackgerrit | melissaml proposed openstack/cinder: Wrong usage of "an" https://review.openstack.org/264024 | 04:49 |
| *** rushiagr_away is now known as rushiagr | 04:57 | |
| *** haomaiwang has quit IRC | 05:01 | |
| *** haomaiwang has joined #openstack-cinder | 05:01 | |
| *** akerr_ is now known as akerr_away | 05:02 | |
| *** sheel has joined #openstack-cinder | 05:02 | |
| *** aagrawal has quit IRC | 05:13 | |
| *** pratap has joined #openstack-cinder | 05:17 | |
| *** merooney has quit IRC | 05:17 | |
| *** chhavi has joined #openstack-cinder | 05:17 | |
| *** chhavi has quit IRC | 05:26 | |
| *** zhangjn has joined #openstack-cinder | 05:26 | |
| *** chhavi has joined #openstack-cinder | 05:26 | |
| *** ankit_ag has joined #openstack-cinder | 05:27 | |
| *** zhangjn has quit IRC | 05:35 | |
| *** coolsvap|away is now known as coolsvap | 05:35 | |
| *** zhangjn has joined #openstack-cinder | 05:36 | |
| *** Pandy has quit IRC | 05:49 | |
| openstackgerrit | Angela Smith proposed openstack/cinder: Adds friendly zone name support https://review.openstack.org/180518 | 05:50 |
| openstackgerrit | Tomoki Sekiyama proposed openstack/cinder: Fix image volume creation error https://review.openstack.org/263081 | 05:52 |
| *** pratap has quit IRC | 05:52 | |
| *** coolsvap is now known as coolsvap|away | 05:55 | |
| *** gcb has joined #openstack-cinder | 05:56 | |
| *** zhangjn_ has joined #openstack-cinder | 06:00 | |
| *** haomaiwang has quit IRC | 06:01 | |
| *** zhangjn has quit IRC | 06:02 | |
| *** haomaiwa_ has joined #openstack-cinder | 06:03 | |
| *** zhangjn_ has quit IRC | 06:05 | |
| *** zhangjn has joined #openstack-cinder | 06:08 | |
| *** sgotliv has joined #openstack-cinder | 06:11 | |
| *** zhangjn has quit IRC | 06:15 | |
| *** harlowja_at_home has joined #openstack-cinder | 06:16 | |
| *** zhangjn has joined #openstack-cinder | 06:16 | |
| *** coolsvap|away is now known as coolsvap | 06:18 | |
| *** dslevin has quit IRC | 06:20 | |
| *** ChubYann has quit IRC | 06:22 | |
| *** alonma has joined #openstack-cinder | 06:23 | |
| *** alonma has quit IRC | 06:28 | |
| *** enowy has joined #openstack-cinder | 06:28 | |
| *** alonma has joined #openstack-cinder | 06:30 | |
| *** Pandy has joined #openstack-cinder | 06:31 | |
| *** alonma has quit IRC | 06:36 | |
| *** shausy has joined #openstack-cinder | 06:37 | |
| *** jwcroppe has quit IRC | 06:38 | |
| *** isaacb has joined #openstack-cinder | 06:38 | |
| *** alonma has joined #openstack-cinder | 06:42 | |
| *** shausy has quit IRC | 06:46 | |
| *** alonma has quit IRC | 06:47 | |
| *** shausy has joined #openstack-cinder | 06:48 | |
| *** alonma has joined #openstack-cinder | 06:48 | |
| *** chlong has quit IRC | 06:50 | |
| *** harlowja_at_home has quit IRC | 06:51 | |
| *** alonma has quit IRC | 06:53 | |
| *** dslevin has joined #openstack-cinder | 06:54 | |
| *** alonma has joined #openstack-cinder | 06:55 | |
| *** belmoreira has joined #openstack-cinder | 06:59 | |
| *** alonma has quit IRC | 06:59 | |
| *** haomaiwa_ has quit IRC | 07:01 | |
| *** alonma has joined #openstack-cinder | 07:01 | |
| *** haomaiwang has joined #openstack-cinder | 07:01 | |
| *** alonma has quit IRC | 07:05 | |
| *** alonma has joined #openstack-cinder | 07:07 | |
| *** zongliang has quit IRC | 07:09 | |
| *** zhangjn has quit IRC | 07:11 | |
| *** alonma has quit IRC | 07:12 | |
| *** shausy has quit IRC | 07:13 | |
| *** shausy has joined #openstack-cinder | 07:13 | |
| *** alonma has joined #openstack-cinder | 07:14 | |
| *** zhangjn has joined #openstack-cinder | 07:15 | |
| *** yangyapeng has joined #openstack-cinder | 07:19 | |
| *** klkumar has joined #openstack-cinder | 07:19 | |
| *** alonma has quit IRC | 07:19 | |
| *** isaacb has quit IRC | 07:20 | |
| *** alonma has joined #openstack-cinder | 07:21 | |
| *** e0ne has joined #openstack-cinder | 07:25 | |
| *** alonma has quit IRC | 07:25 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Schedule creating encrypted volume https://review.openstack.org/264056 | 07:26 |
| *** alonma has joined #openstack-cinder | 07:27 | |
| *** alonma has quit IRC | 07:31 | |
| openstackgerrit | Chhavi Agarwal proposed openstack/cinder-specs: Allow setting multiattach property https://review.openstack.org/264059 | 07:31 |
| *** enowy has quit IRC | 07:31 | |
| *** alonma has joined #openstack-cinder | 07:33 | |
| *** yangyapeng has quit IRC | 07:33 | |
| *** anshul has joined #openstack-cinder | 07:37 | |
| *** alonma has quit IRC | 07:37 | |
| *** alonma has joined #openstack-cinder | 07:39 | |
| openstackgerrit | Chhavi Agarwal proposed openstack/cinder-specs: Allow setting multiattach property https://review.openstack.org/264059 | 07:41 |
| *** Thelo has quit IRC | 07:41 | |
| *** alonma has quit IRC | 07:44 | |
| *** martyturner has joined #openstack-cinder | 07:44 | |
| *** zhangjn has quit IRC | 07:45 | |
| *** alonma has joined #openstack-cinder | 07:45 | |
| *** martyturner has quit IRC | 07:48 | |
| *** alonma has quit IRC | 07:50 | |
| openstackgerrit | melissaml proposed openstack/cinder: Wrong usage of "an" in the mesages: an volume https://review.openstack.org/264063 | 07:50 |
| *** alonma has joined #openstack-cinder | 07:51 | |
| *** alonma has quit IRC | 07:56 | |
| *** lpetrut has joined #openstack-cinder | 07:56 | |
| *** alonma has joined #openstack-cinder | 07:58 | |
| *** coolsvap is now known as coolsvap|away | 08:00 | |
| *** isaacb has joined #openstack-cinder | 08:00 | |
| *** haomaiwang has quit IRC | 08:01 | |
| *** rushiagr is now known as rushiagr_away | 08:01 | |
| *** haomaiwang has joined #openstack-cinder | 08:01 | |
| *** alonma has quit IRC | 08:02 | |
| *** alonma has joined #openstack-cinder | 08:04 | |
| *** coolsvap|away is now known as coolsvap | 08:06 | |
| *** alonma has quit IRC | 08:09 | |
| *** wiebalck has joined #openstack-cinder | 08:09 | |
| *** chlong has joined #openstack-cinder | 08:11 | |
| *** pratap has joined #openstack-cinder | 08:14 | |
| *** alonma has joined #openstack-cinder | 08:15 | |
| *** alonma has quit IRC | 08:19 | |
| *** apoorvad has joined #openstack-cinder | 08:20 | |
| *** alonma has joined #openstack-cinder | 08:21 | |
| sheel | ankit_ag:hi | 08:22 |
| *** rushiagr_away is now known as rushiagr | 08:22 | |
| ankit_ag | sheel: hi.. | 08:22 |
| sheel | kindly confirm https://bugs.launchpad.net/cinder/+bug/1531400 | 08:23 |
| openstack | Launchpad bug 1531400 in Cinder "Creating volume_types, CGs, encryption-type, backup, QoS with unicode name fails with 500 error" [Undecided,New] - Assigned to Sheel Rana (ranasheel2000) | 08:23 |
| sheel | and confirm if any other API is affected than reported | 08:24 |
| *** wiebalck has quit IRC | 08:25 | |
| *** alonma has quit IRC | 08:26 | |
| *** rushiagr is now known as rushiagr_away | 08:27 | |
| ankit_ag | sheel: One which I have tested and is not there either in the patch under review or the LP bug you have filed is Snapshot update | 08:27 |
| *** alonma has joined #openstack-cinder | 08:27 | |
| ankit_ag | IMO you should test all the remaining APIs in order to fix this issue completely | 08:28 |
| *** rushiagr_away is now known as rushiagr | 08:28 | |
| sheel | ankit_ag:I have tested that also, snapshot edit/update fails | 08:29 |
| sheel | ankit_ag:same is the case with volume update/edit | 08:29 |
| ankit_ag | sheel: Yeah, but volume update is handled in the current patch I think | 08:30 |
| sheel | but missed in patch under review, I would have given this as review comment in patch under review once jenkins passed | 08:30 |
| ankit_ag | I have confirmed the bug as the APIs you mentioned in the bug raises 500 error | 08:31 |
| sheel | ankit_ag: thank you | 08:31 |
| ankit_ag | sheel: welcome | 08:31 |
| *** EinstCra_ has joined #openstack-cinder | 08:32 | |
| *** sgotliv has quit IRC | 08:32 | |
| *** alonma has quit IRC | 08:32 | |
| *** alonma has joined #openstack-cinder | 08:33 | |
| *** rushiagr is now known as rushiagr_away | 08:34 | |
| *** EinstCrazy has quit IRC | 08:35 | |
| *** rushiagr_away is now known as rushiagr | 08:35 | |
| openstackgerrit | Ankit Agrawal proposed openstack/python-cinderclient: Return wrapper classes with request_ids attribute https://review.openstack.org/257180 | 08:35 |
| openstackgerrit | Ankit Agrawal proposed openstack/python-cinderclient: Add request_ids attribute to resource objects https://review.openstack.org/257187 | 08:35 |
| openstackgerrit | Ankit Agrawal proposed openstack/python-cinderclient: Add Wrapper classes for list, dict, tuple https://review.openstack.org/257170 | 08:35 |
| openstackgerrit | Ankit Agrawal proposed openstack/python-cinderclient: Add request_ids attribute to resource objects https://review.openstack.org/257195 | 08:36 |
| openstackgerrit | Ankit Agrawal proposed openstack/python-cinderclient: Add request_ids attribute to resource objects https://review.openstack.org/257199 | 08:36 |
| *** alonma has quit IRC | 08:38 | |
| *** shausy has quit IRC | 08:38 | |
| *** alonma has joined #openstack-cinder | 08:40 | |
| sheel | ankit_ag: updated bug description with - "9. update/edit of entities mentioned in step1-8" | 08:40 |
| ankit_ag | sheel: Thanks | 08:40 |
| *** shausy has joined #openstack-cinder | 08:40 | |
| sheel | ankit_ag: :) | 08:40 |
| ankit_ag | sheel: Please let me know if you need any help to fix this issue | 08:41 |
| *** rushiagr is now known as rushiagr_away | 08:41 | |
| sheel | ankit_ag: I will check and if required will let you know | 08:41 |
| sheel | ankit_ag: thanks | 08:42 |
| ankit_ag | sheel: sure | 08:42 |
| *** rushiagr_away is now known as rushiagr | 08:42 | |
| *** rushiagr is now known as rushiagr_away | 08:43 | |
| *** alonma has quit IRC | 08:44 | |
| *** alonma has joined #openstack-cinder | 08:46 | |
| *** pratap has quit IRC | 08:49 | |
| *** rushiagr_away is now known as rushiagr | 08:49 | |
| *** alonma has quit IRC | 08:51 | |
| *** alonma has joined #openstack-cinder | 08:52 | |
| *** shyama has joined #openstack-cinder | 08:53 | |
| *** rushiagr is now known as rushiagr_away | 08:54 | |
| *** lpetrut has quit IRC | 08:56 | |
| *** jordanP has joined #openstack-cinder | 08:57 | |
| *** rushiagr_away is now known as rushiagr | 08:57 | |
| *** alonma has quit IRC | 08:57 | |
| *** alonma has joined #openstack-cinder | 08:58 | |
| *** haomaiwang has quit IRC | 09:01 | |
| *** haomaiwang has joined #openstack-cinder | 09:01 | |
| *** rushiagr is now known as rushiagr_away | 09:02 | |
| *** yhayashi has quit IRC | 09:02 | |
| *** alonma has quit IRC | 09:03 | |
| *** rushiagr_away is now known as rushiagr | 09:04 | |
| *** shyama has quit IRC | 09:04 | |
| *** shyama has joined #openstack-cinder | 09:04 | |
| *** alonma has joined #openstack-cinder | 09:04 | |
| *** alonma has quit IRC | 09:09 | |
| *** zhangjn has joined #openstack-cinder | 09:12 | |
| *** rushiagr is now known as rushiagr_away | 09:13 | |
| *** rushiagr_away is now known as rushiagr | 09:13 | |
| *** shyama_ has joined #openstack-cinder | 09:14 | |
| *** alonma has joined #openstack-cinder | 09:15 | |
| *** shyama has quit IRC | 09:17 | |
| *** shyama_ is now known as shyama | 09:17 | |
| *** jistr has joined #openstack-cinder | 09:17 | |
| openstackgerrit | Sheel Rana proposed openstack/cinder: Constant defined for sqlAlchemy VARCHAR & INTEGER https://review.openstack.org/264085 | 09:19 |
| *** tiagogomes_ has left #openstack-cinder | 09:19 | |
| *** alonma has quit IRC | 09:20 | |
| *** alonma has joined #openstack-cinder | 09:22 | |
| *** aix has joined #openstack-cinder | 09:23 | |
| *** apoorvad has quit IRC | 09:24 | |
| *** alonma has quit IRC | 09:26 | |
| *** alonma has joined #openstack-cinder | 09:28 | |
| jordanP | hi guys. I have a simple patch here https://review.openstack.org/#/c/250915 that has been here for 2 months. It's simple. Could someone review it please ? :) thanks | 09:28 |
| *** alonma has quit IRC | 09:32 | |
| *** alonma has joined #openstack-cinder | 09:34 | |
| *** ociuhandu has quit IRC | 09:34 | |
| *** ociuhandu has joined #openstack-cinder | 09:35 | |
| *** zhangjn has quit IRC | 09:35 | |
| *** jwcroppe has joined #openstack-cinder | 09:39 | |
| *** alonma has quit IRC | 09:39 | |
| openstackgerrit | melissaml proposed openstack/cinder: Wrong usage of "an" in the mesages: an volume https://review.openstack.org/264063 | 09:40 |
| *** alonma has joined #openstack-cinder | 09:40 | |
| *** EinstCra_ has quit IRC | 09:40 | |
| *** EinstCrazy has joined #openstack-cinder | 09:42 | |
| *** sheel has quit IRC | 09:43 | |
| *** jwcroppe has quit IRC | 09:43 | |
| *** ociuhandu has quit IRC | 09:44 | |
| *** alonma has quit IRC | 09:44 | |
| *** sgotliv has joined #openstack-cinder | 09:46 | |
| *** alonma has joined #openstack-cinder | 09:46 | |
| *** Pandy has quit IRC | 09:47 | |
| *** Pandy has joined #openstack-cinder | 09:47 | |
| *** Pandy has left #openstack-cinder | 09:49 | |
| *** alonma has quit IRC | 09:50 | |
| *** zhangjn has joined #openstack-cinder | 09:51 | |
| *** alonma has joined #openstack-cinder | 09:52 | |
| *** shyama has quit IRC | 09:54 | |
| *** shyama has joined #openstack-cinder | 09:54 | |
| *** alonma has quit IRC | 09:57 | |
| *** alonma has joined #openstack-cinder | 09:59 | |
| *** cheneydc has quit IRC | 10:00 | |
| *** haomaiwang has quit IRC | 10:01 | |
| *** haomaiwang has joined #openstack-cinder | 10:01 | |
| *** alonma has quit IRC | 10:03 | |
| *** EinstCrazy has quit IRC | 10:04 | |
| *** ndipanov has joined #openstack-cinder | 10:05 | |
| *** alonma has joined #openstack-cinder | 10:05 | |
| *** tpsilva has joined #openstack-cinder | 10:07 | |
| *** chhavi has quit IRC | 10:08 | |
| *** alonma has quit IRC | 10:09 | |
| *** alonma has joined #openstack-cinder | 10:11 | |
| *** ndipanov has quit IRC | 10:12 | |
| *** slunkad has quit IRC | 10:15 | |
| *** alonma has quit IRC | 10:15 | |
| *** zhangjn has quit IRC | 10:17 | |
| *** alonma has joined #openstack-cinder | 10:17 | |
| *** chhavi has joined #openstack-cinder | 10:17 | |
| *** zhangjn has joined #openstack-cinder | 10:18 | |
| *** lpetrut has joined #openstack-cinder | 10:19 | |
| *** alonma has quit IRC | 10:22 | |
| *** zhangjn has quit IRC | 10:22 | |
| *** alonma has joined #openstack-cinder | 10:23 | |
| *** slunkad has joined #openstack-cinder | 10:27 | |
| *** alonma has quit IRC | 10:27 | |
| *** alonma has joined #openstack-cinder | 10:29 | |
| *** ociuhandu has joined #openstack-cinder | 10:31 | |
| *** sheel has joined #openstack-cinder | 10:32 | |
| *** alonma has quit IRC | 10:34 | |
| *** ndipanov has joined #openstack-cinder | 10:34 | |
| *** alonma has joined #openstack-cinder | 10:36 | |
| *** sheel has quit IRC | 10:37 | |
| *** sheel has joined #openstack-cinder | 10:38 | |
| *** alonma has quit IRC | 10:40 | |
| openstackgerrit | Yatin Kumbhare proposed openstack/cinder: Use assertTrue/False instead of assertEqual(T/F) https://review.openstack.org/264116 | 10:41 |
| *** alonma has joined #openstack-cinder | 10:42 | |
| *** alonma has quit IRC | 10:46 | |
| *** dims has joined #openstack-cinder | 10:47 | |
| openstackgerrit | melissaml proposed openstack/cinder: Wrong usage of "an" in the mesages: an service https://review.openstack.org/264121 | 10:48 |
| *** alonma has joined #openstack-cinder | 10:49 | |
| *** pratap has joined #openstack-cinder | 10:50 | |
| *** gcb has quit IRC | 10:52 | |
| *** houming_ has joined #openstack-cinder | 10:52 | |
| *** houming has quit IRC | 10:52 | |
| *** houming_ is now known as houming | 10:52 | |
| *** alonma has quit IRC | 10:54 | |
| *** gcb has joined #openstack-cinder | 10:54 | |
| *** alonma has joined #openstack-cinder | 10:56 | |
| *** klkumar has quit IRC | 10:59 | |
| *** alonma has quit IRC | 11:00 | |
| *** haomaiwang has quit IRC | 11:01 | |
| *** haomaiwang has joined #openstack-cinder | 11:01 | |
| *** gcb has quit IRC | 11:01 | |
| *** alonma has joined #openstack-cinder | 11:02 | |
| *** sasukeh_ has quit IRC | 11:03 | |
| *** manous has joined #openstack-cinder | 11:05 | |
| *** alonma has quit IRC | 11:06 | |
| *** alonma has joined #openstack-cinder | 11:08 | |
| *** klkumar has joined #openstack-cinder | 11:12 | |
| *** alonma has quit IRC | 11:13 | |
| *** smoriya_ has quit IRC | 11:15 | |
| *** zhangjn has joined #openstack-cinder | 11:16 | |
| *** alonma has joined #openstack-cinder | 11:17 | |
| *** haomaiwang has quit IRC | 11:17 | |
| *** gcb has joined #openstack-cinder | 11:21 | |
| *** alonma has quit IRC | 11:21 | |
| *** alonma has joined #openstack-cinder | 11:23 | |
| *** jwcroppe has joined #openstack-cinder | 11:24 | |
| *** pratap has quit IRC | 11:25 | |
| *** alonma has quit IRC | 11:27 | |
| *** mriedem has joined #openstack-cinder | 11:28 | |
| *** alonma has joined #openstack-cinder | 11:29 | |
| *** chhavi has quit IRC | 11:30 | |
| *** laurent\ has joined #openstack-cinder | 11:32 | |
| *** wiebalck has joined #openstack-cinder | 11:33 | |
| *** alonma has quit IRC | 11:33 | |
| *** alonma has joined #openstack-cinder | 11:35 | |
| ericksonsantos | morning | 11:38 |
| *** alonma has quit IRC | 11:39 | |
| *** alonma has joined #openstack-cinder | 11:41 | |
| *** jwcroppe has quit IRC | 11:42 | |
| e0ne | hello | 11:45 |
| *** wiebalck has quit IRC | 11:46 | |
| *** alonma has quit IRC | 11:46 | |
| *** chhavi has joined #openstack-cinder | 11:46 | |
| *** klkumar has quit IRC | 11:47 | |
| *** coolsvap is now known as coolsvap|away | 11:47 | |
| *** alonma has joined #openstack-cinder | 11:47 | |
| *** alonma has quit IRC | 11:52 | |
| *** klkumar has joined #openstack-cinder | 11:53 | |
| *** alonma has joined #openstack-cinder | 11:54 | |
| *** lprice has joined #openstack-cinder | 11:56 | |
| *** lprice1 has quit IRC | 11:56 | |
| *** apoorvad has joined #openstack-cinder | 11:57 | |
| *** alonma has quit IRC | 11:58 | |
| openstackgerrit | Harshada Mangesh Kakad proposed openstack/cinder: Replace deprecated library function os.popen() with subprocess https://review.openstack.org/262729 | 11:58 |
| *** alonma has joined #openstack-cinder | 12:00 | |
| *** klkumar has quit IRC | 12:02 | |
| *** klkumar has joined #openstack-cinder | 12:02 | |
| *** skraynev has quit IRC | 12:04 | |
| *** alonma has quit IRC | 12:05 | |
| *** alonma has joined #openstack-cinder | 12:06 | |
| *** apoorvad has quit IRC | 12:06 | |
| *** klkumar has quit IRC | 12:08 | |
| *** alonma has quit IRC | 12:11 | |
| *** alonma has joined #openstack-cinder | 12:12 | |
| *** klkumar has joined #openstack-cinder | 12:13 | |
| *** alonma has quit IRC | 12:17 | |
| *** sheel has quit IRC | 12:18 | |
| *** alonma has joined #openstack-cinder | 12:18 | |
| *** alonma has quit IRC | 12:23 | |
| *** alonma has joined #openstack-cinder | 12:24 | |
| openstackgerrit | Mannu proposed openstack/cinder: volume create with unicode name results in HTTP 500 fix https://review.openstack.org/261290 | 12:25 |
| *** zhangjn has quit IRC | 12:26 | |
| *** alonma has quit IRC | 12:29 | |
| *** martyturner has joined #openstack-cinder | 12:29 | |
| *** martyturner has quit IRC | 12:30 | |
| *** alonma has joined #openstack-cinder | 12:31 | |
| *** timcl has joined #openstack-cinder | 12:31 | |
| *** martyturner has joined #openstack-cinder | 12:32 | |
| *** alonma has quit IRC | 12:35 | |
| *** zhangjn has joined #openstack-cinder | 12:45 | |
| *** zhangjn has quit IRC | 12:45 | |
| *** alonma has joined #openstack-cinder | 12:45 | |
| *** timcl1 has joined #openstack-cinder | 12:46 | |
| *** zhangjn has joined #openstack-cinder | 12:46 | |
| *** zhangjn has quit IRC | 12:46 | |
| *** zhangjn has joined #openstack-cinder | 12:47 | |
| *** timcl has quit IRC | 12:48 | |
| *** akerr_away has quit IRC | 12:48 | |
| *** alonma has quit IRC | 12:50 | |
| *** alonma has joined #openstack-cinder | 12:51 | |
| *** cdelatte has joined #openstack-cinder | 12:53 | |
| *** EinstCrazy has joined #openstack-cinder | 12:53 | |
| *** houming has quit IRC | 12:54 | |
| *** haomaiwang has joined #openstack-cinder | 12:56 | |
| *** alonma has quit IRC | 12:57 | |
| *** gouthamr has joined #openstack-cinder | 12:57 | |
| *** gouthamr is now known as gouthamr_away | 12:58 | |
| *** gouthamr_away is now known as gouthamr | 12:58 | |
| *** gouthamr is now known as gouthamr_away | 12:58 | |
| *** alonma has joined #openstack-cinder | 12:58 | |
| *** klkumar has quit IRC | 12:59 | |
| *** gouthamr_away is now known as gouthamr | 13:00 | |
| *** gouthamr is now known as gouthamr_away | 13:00 | |
| *** haomaiwang has quit IRC | 13:01 | |
| *** zhipeng has joined #openstack-cinder | 13:01 | |
| *** gouthamr_away has quit IRC | 13:02 | |
| *** gouthamr has joined #openstack-cinder | 13:03 | |
| *** alonma has quit IRC | 13:03 | |
| *** gouthamr is now known as gouthamr_away | 13:05 | |
| *** gouthamr_away is now known as gouthamr | 13:05 | |
| *** EinstCrazy has quit IRC | 13:09 | |
| *** alonma has joined #openstack-cinder | 13:09 | |
| *** manous has quit IRC | 13:10 | |
| *** manous has joined #openstack-cinder | 13:10 | |
| *** EinstCrazy has joined #openstack-cinder | 13:11 | |
| *** EinstCrazy has quit IRC | 13:12 | |
| openstackgerrit | Aniket Anikhindi proposed openstack/cinder: Fixes "Wrong usage of 'an' in the message" https://review.openstack.org/264163 | 13:13 |
| *** alonma has quit IRC | 13:14 | |
| *** gouthamr has quit IRC | 13:15 | |
| *** gouthamr has joined #openstack-cinder | 13:16 | |
| *** timcl1 has quit IRC | 13:18 | |
| *** alonma has joined #openstack-cinder | 13:18 | |
| *** timcl has joined #openstack-cinder | 13:19 | |
| *** pratap has joined #openstack-cinder | 13:19 | |
| *** dslev has joined #openstack-cinder | 13:22 | |
| *** alonma has quit IRC | 13:22 | |
| openstackgerrit | Mannu proposed openstack/cinder: volume create with unicode name results in HTTP 500 fix https://review.openstack.org/261290 | 13:24 |
| *** alonma has joined #openstack-cinder | 13:24 | |
| *** akerr has joined #openstack-cinder | 13:29 | |
| *** alonma has quit IRC | 13:29 | |
| *** alonma has joined #openstack-cinder | 13:31 | |
| *** links has quit IRC | 13:32 | |
| *** esker has joined #openstack-cinder | 13:33 | |
| *** zhangjn has quit IRC | 13:33 | |
| *** sheel has joined #openstack-cinder | 13:33 | |
| *** edmondsw has joined #openstack-cinder | 13:34 | |
| *** alonma has quit IRC | 13:35 | |
| openstackgerrit | Aniket Anikhindi proposed openstack/cinder: Fixes wrong usage of "an" https://review.openstack.org/264173 | 13:35 |
| *** alonma has joined #openstack-cinder | 13:37 | |
| *** zhipeng has quit IRC | 13:38 | |
| *** jwcroppe has joined #openstack-cinder | 13:38 | |
| *** zhangjn has joined #openstack-cinder | 13:38 | |
| *** dslev has quit IRC | 13:38 | |
| *** zhangjn has quit IRC | 13:39 | |
| *** shausy has quit IRC | 13:40 | |
| *** rushiagr is now known as rushiagr_away | 13:40 | |
| *** alonma has quit IRC | 13:41 | |
| *** alonma has joined #openstack-cinder | 13:43 | |
| *** eharney has joined #openstack-cinder | 13:43 | |
| *** dave-mccowan has joined #openstack-cinder | 13:45 | |
| *** diablo_rojo has joined #openstack-cinder | 13:46 | |
| *** alonma has quit IRC | 13:48 | |
| *** ankit_ag has quit IRC | 13:48 | |
| *** alonma has joined #openstack-cinder | 13:49 | |
| *** jwcroppe has quit IRC | 13:50 | |
| *** alonma has quit IRC | 13:53 | |
| *** pratap has quit IRC | 13:55 | |
| *** alonma has joined #openstack-cinder | 13:55 | |
| *** zhangjn has joined #openstack-cinder | 13:56 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/python-brick-cinderclient-ext: Attach/detach features implementation https://review.openstack.org/263744 | 13:57 |
| *** jwcroppe has joined #openstack-cinder | 13:59 | |
| *** alonma has quit IRC | 14:00 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/python-brick-cinderclient-ext: Attach/detach features implementation https://review.openstack.org/263744 | 14:01 |
| *** haomaiwang has joined #openstack-cinder | 14:01 | |
| *** cknight has joined #openstack-cinder | 14:01 | |
| *** alonma has joined #openstack-cinder | 14:01 | |
| *** klkumar has joined #openstack-cinder | 14:01 | |
| *** rlrossit has joined #openstack-cinder | 14:03 | |
| *** julim has quit IRC | 14:03 | |
| *** sgotliv has quit IRC | 14:05 | |
| *** alonma has quit IRC | 14:06 | |
| *** alonma has joined #openstack-cinder | 14:07 | |
| smcginnis | fernnest: Thanks for taking care of that quickly! | 14:10 |
| *** timcl has quit IRC | 14:11 | |
| *** porrua has joined #openstack-cinder | 14:12 | |
| *** alonma has quit IRC | 14:12 | |
| *** dslevin_ has joined #openstack-cinder | 14:12 | |
| *** wilson has joined #openstack-cinder | 14:16 | |
| *** wilson is now known as Guest826 | 14:16 | |
| *** skraynev has joined #openstack-cinder | 14:18 | |
| *** changbl has quit IRC | 14:19 | |
| *** alonma has joined #openstack-cinder | 14:19 | |
| *** timcl has joined #openstack-cinder | 14:20 | |
| *** zhangjn has quit IRC | 14:20 | |
| *** chhavi has quit IRC | 14:21 | |
| *** links has joined #openstack-cinder | 14:23 | |
| *** chhavi has joined #openstack-cinder | 14:23 | |
| *** alonma has quit IRC | 14:23 | |
| *** Guest826 has quit IRC | 14:23 | |
| *** wilson_liu has joined #openstack-cinder | 14:24 | |
| *** alonma has joined #openstack-cinder | 14:25 | |
| *** anshul has quit IRC | 14:26 | |
| *** jordan__ has joined #openstack-cinder | 14:26 | |
| *** jgregor has joined #openstack-cinder | 14:26 | |
| *** sgotliv has joined #openstack-cinder | 14:28 | |
| *** links has quit IRC | 14:28 | |
| *** alonma has quit IRC | 14:29 | |
| *** jordanP has quit IRC | 14:29 | |
| *** merooney has joined #openstack-cinder | 14:30 | |
| *** alonma has joined #openstack-cinder | 14:31 | |
| *** esker has quit IRC | 14:32 | |
| *** timcl has quit IRC | 14:33 | |
| *** martyturner has quit IRC | 14:33 | |
| *** cknight has quit IRC | 14:33 | |
| *** alonma has quit IRC | 14:35 | |
| *** cknight has joined #openstack-cinder | 14:36 | |
| *** alonma has joined #openstack-cinder | 14:37 | |
| *** itzdilip1 has joined #openstack-cinder | 14:41 | |
| *** alonma has quit IRC | 14:42 | |
| *** timcl has joined #openstack-cinder | 14:42 | |
| *** alonma has joined #openstack-cinder | 14:43 | |
| *** martyturner has joined #openstack-cinder | 14:44 | |
| *** itzdilip1 has quit IRC | 14:44 | |
| *** rushiagr_away is now known as rushiagr | 14:44 | |
| *** itzdilip has joined #openstack-cinder | 14:46 | |
| *** esker has joined #openstack-cinder | 14:46 | |
| *** alonma has quit IRC | 14:48 | |
| *** julim has joined #openstack-cinder | 14:49 | |
| *** jungleboyj has joined #openstack-cinder | 14:49 | |
| *** alonma has joined #openstack-cinder | 14:50 | |
| *** itzdilip has quit IRC | 14:53 | |
| *** dustins has joined #openstack-cinder | 14:53 | |
| *** diablo_rojo has quit IRC | 14:54 | |
| *** alonma has quit IRC | 14:54 | |
| *** alonma has joined #openstack-cinder | 14:56 | |
| *** lprice has quit IRC | 14:57 | |
| *** itzdilip has joined #openstack-cinder | 14:57 | |
| *** merooney has quit IRC | 15:00 | |
| *** haomaiwang has quit IRC | 15:01 | |
| *** alonma has quit IRC | 15:01 | |
| *** pratap has joined #openstack-cinder | 15:01 | |
| *** alonma has joined #openstack-cinder | 15:02 | |
| *** 77CAAHDEK has joined #openstack-cinder | 15:02 | |
| *** merooney has joined #openstack-cinder | 15:02 | |
| *** aorourke_ has quit IRC | 15:03 | |
| smcginnis | e0ne: The client extension library is official now. Nice! :) | 15:04 |
| e0ne | smcginnis: cool :) | 15:04 |
| *** alonma has quit IRC | 15:07 | |
| *** baumann has joined #openstack-cinder | 15:07 | |
| *** alonma has joined #openstack-cinder | 15:08 | |
| openstackgerrit | Shlomi Avihou proposed openstack/cinder: Add backend driver for Zadara Storage VPSA https://review.openstack.org/263026 | 15:08 |
| *** yusuke has joined #openstack-cinder | 15:11 | |
| *** yusuke has left #openstack-cinder | 15:11 | |
| *** yusuke has joined #openstack-cinder | 15:12 | |
| *** yusuke has left #openstack-cinder | 15:12 | |
| *** alonma has quit IRC | 15:13 | |
| *** yhayashi has joined #openstack-cinder | 15:14 | |
| *** alonma has joined #openstack-cinder | 15:14 | |
| *** shyama has quit IRC | 15:15 | |
| *** itzdilip has left #openstack-cinder | 15:17 | |
| *** xyang has joined #openstack-cinder | 15:18 | |
| *** diablo_rojo has joined #openstack-cinder | 15:19 | |
| *** alonma has quit IRC | 15:19 | |
| *** adrianofr has quit IRC | 15:20 | |
| *** aorourke_ has joined #openstack-cinder | 15:21 | |
| *** alonma has joined #openstack-cinder | 15:21 | |
| *** thangp has joined #openstack-cinder | 15:22 | |
| *** tpsilva has quit IRC | 15:23 | |
| *** adrianofr has joined #openstack-cinder | 15:23 | |
| *** tpsilva has joined #openstack-cinder | 15:24 | |
| *** aorourke_ has quit IRC | 15:24 | |
| *** wilson_liu has quit IRC | 15:25 | |
| *** alonma has quit IRC | 15:26 | |
| *** mtanino has joined #openstack-cinder | 15:26 | |
| *** lprice has joined #openstack-cinder | 15:26 | |
| *** alonma has joined #openstack-cinder | 15:27 | |
| *** tpsilva is now known as _tpsilva | 15:28 | |
| *** tpsilva has joined #openstack-cinder | 15:29 | |
| *** _tpsilva has quit IRC | 15:29 | |
| *** breitz has quit IRC | 15:30 | |
| *** breitz has joined #openstack-cinder | 15:30 | |
| *** alonma has quit IRC | 15:31 | |
| *** alonma has joined #openstack-cinder | 15:33 | |
| *** rs007_ has joined #openstack-cinder | 15:33 | |
| *** rs007 has joined #openstack-cinder | 15:33 | |
| *** rs007_ has left #openstack-cinder | 15:34 | |
| *** rs007 has quit IRC | 15:34 | |
| *** geguileo_phone has joined #openstack-cinder | 15:34 | |
| *** rs007 has joined #openstack-cinder | 15:34 | |
| *** isaacb has quit IRC | 15:35 | |
| *** geguileo_phone has joined #openstack-cinder | 15:36 | |
| *** pratap has quit IRC | 15:36 | |
| *** alonma has quit IRC | 15:37 | |
| *** alonma has joined #openstack-cinder | 15:39 | |
| *** pots has quit IRC | 15:39 | |
| *** jixuepeng has joined #openstack-cinder | 15:39 | |
| *** pots has joined #openstack-cinder | 15:40 | |
| *** merooney has quit IRC | 15:43 | |
| *** alonma has quit IRC | 15:44 | |
| *** merooney has joined #openstack-cinder | 15:44 | |
| *** pratap has joined #openstack-cinder | 15:44 | |
| *** dustins has quit IRC | 15:45 | |
| *** alonma has joined #openstack-cinder | 15:46 | |
| *** lcurtis has joined #openstack-cinder | 15:47 | |
| *** jordanP has joined #openstack-cinder | 15:48 | |
| *** mragupat has joined #openstack-cinder | 15:48 | |
| *** ntpttr has joined #openstack-cinder | 15:49 | |
| *** alonma has quit IRC | 15:50 | |
| *** klkumar has quit IRC | 15:50 | |
| *** klkumar has joined #openstack-cinder | 15:51 | |
| *** jordan__ has quit IRC | 15:51 | |
| *** alonma has joined #openstack-cinder | 15:52 | |
| *** yhayashi has left #openstack-cinder | 15:53 | |
| *** Yogi1 has joined #openstack-cinder | 15:55 | |
| *** alonma has quit IRC | 15:56 | |
| jixuepeng | hello | 15:57 |
| *** shyama has joined #openstack-cinder | 15:57 | |
| *** jseiler has joined #openstack-cinder | 15:58 | |
| *** mragupat has quit IRC | 15:58 | |
| *** krtaylor has quit IRC | 15:58 | |
| *** alonma has joined #openstack-cinder | 15:58 | |
| *** mragupat has joined #openstack-cinder | 15:58 | |
| *** Yogi1 has quit IRC | 15:59 | |
| *** dustins has joined #openstack-cinder | 15:59 | |
| DuncanT | jixuepeng: Hi! | 15:59 |
| *** esker has quit IRC | 16:00 | |
| *** 77CAAHDEK has quit IRC | 16:01 | |
| e0ne | hi | 16:01 |
| *** Guest56749 is now known as cfouts | 16:01 | |
| *** haomaiwang has joined #openstack-cinder | 16:01 | |
| *** rushiagr is now known as rushiagr_away | 16:01 | |
| *** mc_nair_ has quit IRC | 16:02 | |
| *** alonma has quit IRC | 16:03 | |
| *** alonma has joined #openstack-cinder | 16:05 | |
| *** sgotliv has quit IRC | 16:06 | |
| *** rajinir has joined #openstack-cinder | 16:06 | |
| *** haomaiwang has quit IRC | 16:07 | |
| *** cknight has quit IRC | 16:07 | |
| *** alonma has quit IRC | 16:09 | |
| openstackgerrit | Merged openstack/cinder: Don’t log warnings for image cache when disabled https://review.openstack.org/263515 | 16:10 |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Implements Replication V2 https://review.openstack.org/244867 | 16:10 |
| *** krtaylor has joined #openstack-cinder | 16:10 | |
| *** kfox1111 has left #openstack-cinder | 16:11 | |
| *** esker has joined #openstack-cinder | 16:12 | |
| *** salv-orlando has joined #openstack-cinder | 16:12 | |
| *** esker has quit IRC | 16:12 | |
| *** alonma has joined #openstack-cinder | 16:13 | |
| *** mc_nair has joined #openstack-cinder | 16:13 | |
| *** vgridnev has joined #openstack-cinder | 16:13 | |
| *** jgregor has quit IRC | 16:14 | |
| *** esker has joined #openstack-cinder | 16:14 | |
| *** jgregor has joined #openstack-cinder | 16:15 | |
| *** cknight has joined #openstack-cinder | 16:16 | |
| *** alonma has quit IRC | 16:17 | |
| *** sgotliv has joined #openstack-cinder | 16:18 | |
| *** belmoreira has quit IRC | 16:18 | |
| openstackgerrit | Peter Penchev proposed openstack/cinder: Reintroduce the StorPool driver https://review.openstack.org/220155 | 16:19 |
| *** alonma has joined #openstack-cinder | 16:20 | |
| *** garthb has joined #openstack-cinder | 16:22 | |
| *** merooney has quit IRC | 16:23 | |
| *** merooney has joined #openstack-cinder | 16:23 | |
| openstackgerrit | Merged openstack/cinder: Scality SOFS: don't always read /proc/mounts twice https://review.openstack.org/250915 | 16:24 |
| *** alonma has quit IRC | 16:24 | |
| *** alonma has joined #openstack-cinder | 16:26 | |
| *** fernnest has quit IRC | 16:30 | |
| *** alonma has quit IRC | 16:31 | |
| openstackgerrit | Tom Barron proposed openstack/cinder: Add restore_volume_id in backup https://review.openstack.org/260903 | 16:31 |
| *** isaacb has joined #openstack-cinder | 16:32 | |
| *** alonma has joined #openstack-cinder | 16:32 | |
| *** dslevin_ has quit IRC | 16:34 | |
| *** alonma has quit IRC | 16:37 | |
| *** alonma has joined #openstack-cinder | 16:39 | |
| *** merooney has quit IRC | 16:41 | |
| mriedem | hemna: just on the os-brick lvm thing, nova probably wouldn't take that in until N b/c of the privsep implications | 16:44 |
| *** alonma has quit IRC | 16:44 | |
| hemna | yah that's fine | 16:44 |
| *** dslevin_ has joined #openstack-cinder | 16:44 | |
| hemna | Cinder used os-brick by itself for a release before nova used it. | 16:44 |
| hemna | this is just the first step | 16:44 |
| hemna | frankly, I'd like to move to privsep sooner than later, but it's going to be a while. | 16:44 |
| hemna | there is a lot of Cinder plumbing needed to make privsep happen, as would Nova | 16:45 |
| *** alonma has joined #openstack-cinder | 16:47 | |
| *** chhavi has quit IRC | 16:47 | |
| *** jwcroppe has quit IRC | 16:48 | |
| *** dslevin_ has quit IRC | 16:49 | |
| *** jgregor has quit IRC | 16:49 | |
| *** erlon has joined #openstack-cinder | 16:51 | |
| *** alonma has quit IRC | 16:52 | |
| *** jgregor has joined #openstack-cinder | 16:55 | |
| *** markstur has quit IRC | 16:56 | |
| *** salv-orlando has quit IRC | 16:57 | |
| *** salv-orlando has joined #openstack-cinder | 16:58 | |
| *** pratap has quit IRC | 16:59 | |
| *** jordanP has quit IRC | 16:59 | |
| *** markstur has joined #openstack-cinder | 16:59 | |
| *** chhavi has joined #openstack-cinder | 17:00 | |
| *** dslevin_ has joined #openstack-cinder | 17:00 | |
| *** ntpttr has quit IRC | 17:01 | |
| *** rs007 has quit IRC | 17:01 | |
| *** fernnest has joined #openstack-cinder | 17:01 | |
| *** jixuepeng has quit IRC | 17:03 | |
| *** fernnest has quit IRC | 17:03 | |
| *** fernnest has joined #openstack-cinder | 17:03 | |
| scottda | shyama: hemna Hi | 17:04 |
| *** shyama has quit IRC | 17:04 | |
| smcginnis | hemna: Hey about that logging. | 17:04 |
| hemna | hey | 17:04 |
| smcginnis | hemna: That paragraph was actually what I was going to post to you. | 17:05 |
| scottda | I think I see the point about fixing the cinder side of multi-attach detach... | 17:05 |
| smcginnis | hemna: I think you may be misinterpreting that. | 17:05 |
| scottda | https://review.openstack.org/#/c/255595/7 | 17:05 |
| scottda | The question seems to be, we are putting logic in the call to detach() that checks for multiple connections before calling remove_export... | 17:05 |
| smcginnis | hemna: "when the message is only sent directly to the log" is if a string is just being used for logging. | 17:05 |
| scottda | Should we put similar logic in the call to terminate_connection ? | 17:06 |
| smcginnis | hemna: It doesn't matter if the string is assigned to a variable first and passed in or sent as a literal. | 17:06 |
| hemna | scottda, I'm not sure what to do really | 17:06 |
| hemna | if we block terminate_connection, then that will break the 3par driver | 17:06 |
| smcginnis | A logging string needs to be translated for the given level of logging. The only exception is if it's being used for an exception message as well then it's _(). | 17:06 |
| hemna | as the 3par driver creates a new export for every attach, even for the same volume. | 17:07 |
| hemna | so terminate_connection has to be called. | 17:07 |
| smcginnis | And if it's used for anything else other than logging or exception messages, then no translation. | 17:07 |
| hemna | smcginnis, ok I'll change it. | 17:07 |
| smcginnis | hemna: Thanks! | 17:07 |
| scottda | hemna: Yeah, where should the logic go then, for accounting? In the driver itself? | 17:08 |
| hemna | I think so | 17:08 |
| hemna | our driver checks | 17:08 |
| scottda | Figure out whether it's the last attach in manager.terminate_connection(), and then pass that in to the driver call? | 17:08 |
| *** tongli has joined #openstack-cinder | 17:09 | |
| hemna | I think the problem is though that some drivers don't keep track of it. they have a target exported to a host, and if it's already there, do nothing in initialize_connection | 17:09 |
| hemna | but terminate_connection is another story. | 17:09 |
| *** isaacb has quit IRC | 17:09 | |
| hemna | the volume itself has the attachments | 17:09 |
| hemna | and if nova passes in the host during attach time, which I believe ildikov's patch does, then cinder can know how many times a volume is being used on that host. | 17:10 |
| hemna | the driver can decide what to do, for terminate_connection. | 17:10 |
| scottda | Yeah, but if we passed a flag to the driver is_last_connection, then the driver could decide if it needs to do it's work for each call to terminate_connection, or just the last. | 17:10 |
| scottda | ok, I think we are saying the same thing. | 17:10 |
| *** wN has quit IRC | 17:10 | |
| ildikov | hemna: yeah, if you're patch removes that check I can pass it to Cinder | 17:10 |
| johnthetubaguy | its likely to get tricky with live-migrates and migrates, if we have to update the host we pass, I don't think we do that right now | 17:11 |
| ildikov | although it's not a backward compatible solution | 17:11 |
| hemna | scottda, https://review.openstack.org/#/c/255595/7/cinder/volume/manager.py | 17:11 |
| hemna | so basically the same thing that's at line 1027 on. | 17:11 |
| ildikov | johnthetubaguy: no, we don't, I added it here: https://review.openstack.org/#/c/256273/ | 17:12 |
| ildikov | or well, started to add | 17:12 |
| *** pratap has joined #openstack-cinder | 17:12 | |
| hemna | I suppose I could make a util function that gets the existing attaches left for a host. | 17:12 |
| *** wN has joined #openstack-cinder | 17:12 | |
| *** wN has joined #openstack-cinder | 17:12 | |
| scottda | hemna: You're saying add that to terminate_connection? | 17:12 |
| hemna | then the driver can do a if len(host_attaches) == 0: backend.remove_export() | 17:13 |
| scottda | ahh..right | 17:13 |
| hemna | notmorgan, I'm saying the driver can call this util function to get the host_attaches left on a volume. | 17:13 |
| hemna | and then decide what to do. | 17:13 |
| hemna | *sigh* | 17:13 |
| johnthetubaguy | ildikov: ah, OK. that might the the right thing, long term, I guess. | 17:13 |
| scottda | sounds right | 17:13 |
| hemna | friggin xchat | 17:13 |
| hemna | johnthetubaguy, and there is no API to do that now. | 17:13 |
| hemna | there isn't a mechanism for updating a host for an attachment currently. | 17:14 |
| notmorgan | hemna: ;) | 17:14 |
| jgriffith | hemna: didn't you and I talk about this at length in the past? | 17:14 |
| johnthetubaguy | len(host_attaches) == 0 does feel a bit racy, or is that just me? | 17:14 |
| hemna | notmorgan, sorry man. I can't type "no, " in xchat, it automagically converts that to notmorgan, | 17:14 |
| jgriffith | hemna: inparticular replacing a lot of what's going on in there with an attachments list and just using that? | 17:14 |
| hemna | jgriffith, and I keep saying we have an attachments list already | 17:15 |
| jgriffith | hemna: alright, never mind | 17:15 |
| hemna | it just doesn't have the host information. | 17:15 |
| *** jgriffith is now known as jgriffith_away | 17:15 | |
| hemna | as nova isn't passing it. | 17:15 |
| jgriffith_away | You do have the info however | 17:15 |
| *** timcl has quit IRC | 17:15 | |
| jgriffith_away | but I have to jam for a few | 17:15 |
| hemna | johnthetubaguy, the volume is in detaching state and can't get another attachment until it's back into 'in-use' or 'available' | 17:15 |
| jgriffith_away | can chat later if you want | 17:16 |
| hemna | jgriffith_away, ok sounds good. | 17:16 |
| *** merooney has joined #openstack-cinder | 17:16 | |
| johnthetubaguy | hemna: ah, OK, that should work | 17:16 |
| notmorgan | hemna: no worries :) | 17:17 |
| *** leeantho has joined #openstack-cinder | 17:17 | |
| *** sgotliv has quit IRC | 17:18 | |
| *** pratap has quit IRC | 17:18 | |
| *** salv-orlando has quit IRC | 17:19 | |
| *** salv-orlando has joined #openstack-cinder | 17:20 | |
| *** akerr has quit IRC | 17:20 | |
| scottda | hemna: OK, so I guess the path of least resistance is to have some way (flag or util function) for the driver to determine the last attachment during the call to terminate_connection, and the driver can then do whatever is best. | 17:21 |
| scottda | Is there anything besides removing the export that could/should be done? | 17:21 |
| *** akerr has joined #openstack-cinder | 17:22 | |
| scottda | since it still seems that if the driver is doing remove_export in the call to terminate_connection instead of detach(), it's done in the wrong place. | 17:22 |
| hemna | scottda, so there is a live migration issue of the host not being updated after the migration. | 17:23 |
| hemna | which would make that attachments out of sync | 17:23 |
| scottda | yeah, that's bad. | 17:23 |
| hemna | we have the host at initialize_connection time, but we don't know which attachment that's associated with. | 17:23 |
| scottda | any way to fix that without providing a new cinder api to update_host for Nova to call? | 17:23 |
| hemna | if we had the attachment_id, or the instance_id at initialize_connection time, then we could update it | 17:24 |
| scottda | bleh | 17:24 |
| hemna | but nova doesn't pass either of those at initialize_connection time. | 17:24 |
| hemna | so.... | 17:24 |
| *** isaacb has joined #openstack-cinder | 17:24 | |
| *** esker has quit IRC | 17:25 | |
| johnthetubaguy | Just wondering if we need to worry about host+node, for cases like vmware and ironic, etc? | 17:26 |
| scottda | hemna: Sorry, I feel like we've had this conversation more than once in the past. | 17:26 |
| hemna | scottda, https://review.openstack.org/#/c/255595/7 so I -1'd my own review | 17:26 |
| hemna | take a look at see what you think. | 17:26 |
| hemna | scottda, yah IU think so.. | 17:26 |
| *** martyturner has quit IRC | 17:27 | |
| hemna | I think the host should be passed always IMHO | 17:27 |
| *** esker has joined #openstack-cinder | 17:27 | |
| *** cknight has quit IRC | 17:27 | |
| hemna | since the BM node attaches will be done using the new extension to the cinderclient, all it has is the host | 17:27 |
| scottda | hemna: Your comment makes sense. In my mind, that mostly resolves the one issue around needing logic in terminate_connection. | 17:28 |
| hemna | yup | 17:28 |
| *** esker has quit IRC | 17:30 | |
| *** jgriffith_away is now known as jgriffith | 17:30 | |
| jgriffith | ha | 17:30 |
| jgriffith | show up 15 minutes late for a meeting and get out 5 minutes early!! | 17:30 |
| jgriffith | Now that's my kinda meeting! | 17:31 |
| *** timcl has joined #openstack-cinder | 17:31 | |
| hemna | wee! | 17:31 |
| jgriffith | You guys still talking multi-attach? | 17:31 |
| jgriffith | One thing that I've thought about... rather than trying to cram things into the semantics we currently have, should we be considering changing the whole deal? | 17:32 |
| *** e0ne has quit IRC | 17:32 | |
| jgriffith | what I mean is, that since attach, initialize, export etc etc seems to be a bit unclear and open to misinterpretation | 17:32 |
| hemna | jgriffith, I think we started toying with that idea at our Austin meetup ? | 17:32 |
| jgriffith | should we consider taking another look at that entire internal process from start to finish? | 17:32 |
| hemna | jgriffith, yup I think so | 17:33 |
| jgriffith | I've been looking at these patches and just seems like a lot of contortions to try and interpret the right thing to do and make things "work" with the existing model | 17:33 |
| jgriffith | I don't remember if we talked about it in Austin or not | 17:33 |
| hemna | we sketched something on the white board at one of the meetups related to exactly this issue of trying to 'fix' the Nova -> Cinder api workflows | 17:33 |
| jgriffith | but you've been doing all the work here, so just wondering what you thought | 17:33 |
| jgriffith | hemna: ahh... yes, but you guys were just calling it "nova's bad assumptions" which never worked for me | 17:34 |
| jgriffith | now I remember | 17:34 |
| hemna | but we never really followed through on it or figured out exactly what it would end up being. | 17:34 |
| jgriffith | hmmm | 17:34 |
| hemna | I still think it's worth chewing on and seeing if we need a new API/semantics | 17:34 |
| jgriffith | I guess I'd just hate to see everybody kill themeselves to force this in to Nova and Cinder only to end up with something super complex that we look at and say "we should've just cleaned up the whole thing" | 17:35 |
| hemna | scottda, the insanity of how difficult it is to get multi attach in nova is a prime example. | 17:35 |
| jgriffith | FWIW, pulling out the target code was supposed to be a first step in that direction | 17:35 |
| hemna | jgriffith, yah I agree. | 17:36 |
| hemna | it's been painful for a while | 17:36 |
| jgriffith | Not sure if it's helpful to bring any of this up or not | 17:36 |
| *** ntpttr has joined #openstack-cinder | 17:36 | |
| hemna | which is what made me start thinking about doing something more long term for a while. but honestly, I've got nothin | 17:36 |
| jgriffith | certainly wouldn't want to derail the track you're on now if you think it's going to land in M | 17:36 |
| jgriffith | I'm worried about it though from a maintainability perspective | 17:36 |
| openstackgerrit | Harshada Mangesh Kakad proposed openstack/cinder: Replace deprecated library function os.popen() with subprocess https://review.openstack.org/262729 | 17:36 |
| hemna | same here. | 17:37 |
| hemna | it's less than ideal | 17:37 |
| scottda | jgriffith: hemna It seems that there are multiple features/issues that point to a new/improved api for , at least /attach/detach | 17:38 |
| scottda | It's just going to be a ton of work | 17:39 |
| jgriffith | scottda: sadly you may be right | 17:39 |
| jgriffith | scottda: it doesn't seem like the current process is something folks really grok, and are having trouble with in terms of flexibility | 17:39 |
| hemna | I briefly looked at allowing nova to call attach again, to update the host entry, but that changes things quite a bit. | 17:39 |
| jgriffith | hemna: I have a patch that just did that | 17:40 |
| jgriffith | it was actually rather simple, on both the Nova and Cinder sides | 17:40 |
| jgriffith | BUT | 17:40 |
| hemna | ok | 17:40 |
| jgriffith | I was creating a new target even if it's going to the same host | 17:40 |
| hemna | I think that would fix the live migration issue | 17:41 |
| jgriffith | which I think is a good answer | 17:41 |
| hemna | of keeping the host updated for that attachment. | 17:41 |
| jgriffith | right, just treat them as independent attachments | 17:41 |
| jgriffith | that's what I was kinda getting at with the list comment again | 17:41 |
| jgriffith | It's not just "having a list" but relying on and using that list | 17:41 |
| jgriffith | in the DB itself | 17:41 |
| jgriffith | but anyway | 17:42 |
| jgriffith | lemme see if I can find those patches | 17:42 |
| hemna | ok, so I'm probably missing something or we are crossing signals | 17:42 |
| jgriffith | hemna: or I'm missing something :) | 17:42 |
| hemna | but we do in fact have a list of attachments for the volume today. | 17:42 |
| jgriffith | but yeah, quite possibly | 17:42 |
| hemna | in the db | 17:42 |
| jgriffith | hemna: right, but what I'm saying is skip all the semantics on how that works in terms of what to do on a detach etc when there's multiple attachments | 17:43 |
| jgriffith | hemna: you do "everything" you currently do today; you just do it on that single attachment entry | 17:43 |
| hemna | k | 17:43 |
| *** Yogi1 has joined #openstack-cinder | 17:43 | |
| hemna | I guess I'm a bit lost. | 17:43 |
| hemna | I'm slow on the uptake. | 17:43 |
| hemna | heh sorry | 17:43 |
| jgriffith | so if you keep the target independent you can safely do things like disconnect and remote the target etc | 17:43 |
| *** bardia has joined #openstack-cinder | 17:43 | |
| jgriffith | nahh... it's hard to discuss in text | 17:43 |
| hemna | ok I think that's what I'm trying to do | 17:44 |
| scottda | Yeah, that's the problem. The entire API needs a nice pretty diagram | 17:44 |
| jgriffith | So what I'm saying is that if you treat EVERY attachment as a truly independent instantiaton, you can do whatever you want | 17:44 |
| hemna | is keep enough information in each of the attachment entries so that we can know when to remove the single target | 17:44 |
| jgriffith | with the existing api's | 17:44 |
| jgriffith | you just need to modify the api checks | 17:44 |
| *** jistr has quit IRC | 17:45 | |
| hemna | that's what the 3par does in fact. but with the additional overhead of a new target as well. | 17:45 |
| jgriffith | hemna: well, I'm saying maybe you don't need to | 17:45 |
| hemna | so it just 'works' | 17:45 |
| jgriffith | attach-1 ---> target-1 | 17:45 |
| jgriffith | attach-2 ---> target-2 | 17:45 |
| jgriffith | detach-2 ---> clean up and delete target-2 | 17:45 |
| jgriffith | attach-3 --> target-3 | 17:46 |
| *** chhavi has quit IRC | 17:46 | |
| jgriffith | detach-1 ---> clean up and delete target-2 | 17:46 |
| jgriffith | oops!! | 17:46 |
| jgriffith | target-1 | 17:46 |
| hemna | lolz | 17:46 |
| jgriffith | see what I mean? | 17:46 |
| hemna | yah | 17:46 |
| hemna | that's in fact exactly what our driver does | 17:46 |
| jgriffith | Well, I guess what I was getting at was the manager should understand how to do all of that | 17:47 |
| jgriffith | of course the driver might need to do "the work" but you know what I mean? | 17:47 |
| *** alonma has joined #openstack-cinder | 17:47 | |
| jgriffith | if they're independent and unique attachments you already have all the info you need | 17:48 |
| jgriffith | because you have the tgt-iqn. | 17:48 |
| *** jwcroppe has joined #openstack-cinder | 17:48 | |
| hemna | yah completely | 17:48 |
| hemna | I guess that's sorta what my remove_export patch is trying to do. | 17:49 |
| jgriffith | well, what I was thinking was you still call remove export | 17:49 |
| jgriffith | you still do everything you do normally | 17:49 |
| jgriffith | but you only do it on THAT attach instance | 17:49 |
| jgriffith | you have a unique export for every attachment | 17:50 |
| hemna | yah | 17:50 |
| jgriffith | when you get rid of one, you go ahead and just delete it like you do normally | 17:50 |
| hemna | so the lvm driver would have to create a new export for every attachment, and then remove it at detach. | 17:50 |
| jgriffith | yep | 17:50 |
| jgriffith | and later if we wanted to optimize things we could consider it | 17:51 |
| hemna | ok | 17:51 |
| jgriffith | but I'm not sure there's any real value in it | 17:51 |
| hemna | yah that would help a lot I think | 17:51 |
| *** klkumar has quit IRC | 17:51 | |
| jgriffith | I'm happy to have a look at the lvm side if it helps | 17:51 |
| *** timcl has quit IRC | 17:51 | |
| guitarzan | crazy question, is that "optimization" something that really causes any problems? | 17:51 |
| jgriffith | or whatever you might want in terms of help | 17:51 |
| jgriffith | or if you've got it then that's cool | 17:51 |
| jgriffith | guitarzan: which? | 17:51 |
| jgriffith | guitarzan: sharing a target? | 17:52 |
| guitarzan | jgriffith: the difference between one and two targets | 17:52 |
| guitarzan | yeah | 17:52 |
| *** alonma has quit IRC | 17:52 | |
| guitarzan | is that just a "feels better" thing? | 17:52 |
| guitarzan | or does it have a measurable impact | 17:52 |
| jgriffith | yeah, it seems to be causing some heart burn/complication on detach and cleanup | 17:52 |
| hemna | jgriffith, I think that would be another way of fixing the same bug that my patch tries to address. | 17:52 |
| jgriffith | hemna: ^^ | 17:52 |
| guitarzan | jgriffith: right, but we're saying that the "deoptimization" is better right? | 17:52 |
| hemna | might be worth trying out before the nova patches merge | 17:52 |
| hemna | so we can test it out | 17:52 |
| guitarzan | I should've air quoted "better" as well | 17:53 |
| jgriffith | guitarzan: I was saying it would be better because it's cleaner, and right now as I understand it the hang up is there's no way to figure out reliably if/when you can remote the target | 17:53 |
| jgriffith | guitarzan: but I need to defer to hemna and scottda on the details of that | 17:53 |
| guitarzan | jgriffith: I'm in 100% agreement on that | 17:53 |
| hemna | jgriffith, completely correct | 17:53 |
| guitarzan | I may be misunderstanding and hearing the word "optimization" as "sharing targets" | 17:54 |
| jgriffith | guitarzan: no, that's what I meant when I said it | 17:54 |
| guitarzan | I'm basically asking is there any measurable reason to not go 1-1 attachment-export | 17:54 |
| scottda | guitarzan: no, you are right. It's the sharing of targets that's the root of the problem | 17:54 |
| jgriffith | guitarzan: not in my opinion :) | 17:55 |
| guitarzan | cool, I wonder if there's any dissent | 17:55 |
| jgriffith | and technically I don't see a reason | 17:55 |
| hemna | works for me! | 17:55 |
| scottda | I'm sure there's some idealist who would dissent, but to heck with them | 17:55 |
| guitarzan | hah | 17:55 |
| * scottda looks around for DuncanT | 17:55 | |
| jgriffith | guitarzan: so the reality is, like in my devices case; I'm actually going to be using the same target, I'm just going to have another connection/session to it | 17:55 |
| jgriffith | guitarzan: but none of this impacts me because I don't "delete" targets :) | 17:56 |
| jgriffith | they just go away when there are no connections left | 17:56 |
| jgriffith | or "sessions" | 17:56 |
| guitarzan | jgriffith: well that confuses me then :) | 17:56 |
| guitarzan | you're saying it's good to go 1-1, but you're going to keep up 1-N | 17:56 |
| guitarzan | which is fine, I'm just trying to figure out what you're saying :) | 17:56 |
| jgriffith | guitarzan: nahh... I'm saying from a logic perspecitve cinder, nova and cinder-manager will see them as 2 | 17:57 |
| *** isaacb has quit IRC | 17:57 | |
| guitarzan | does this mean we need another construct for exports? | 17:57 |
| jgriffith | guitarzan: internally, my device will just say "yeah... whatever, 1 target 2 sessions" | 17:57 |
| jgriffith | guitarzan: well... that's what I was saying earlier :) | 17:57 |
| jgriffith | guitarzan: exports are abused right now and misinterpretted | 17:57 |
| guitarzan | ok, I admit I'm only half following all of this | 17:57 |
| guitarzan | agreed | 17:57 |
| jgriffith | LVM has exports, because it needs to create/associate a target | 17:58 |
| guitarzan | that's one reason I haven't started working on multiple attach for rax yet | 17:58 |
| *** alonma has joined #openstack-cinder | 17:58 | |
| jgriffith | SolidFire exports are an internal thing... from Cinder's perspective they essentially don't exist | 17:58 |
| jgriffith | guitarzan: I was mentioning earlier the way I look at it is: exports ---> target-mangement | 17:59 |
| guitarzan | either way, I guess I'm firmly in the 1-1 camp | 17:59 |
| guitarzan | sure | 17:59 |
| *** cknight has joined #openstack-cinder | 18:00 | |
| jgriffith | most important thing is what hemna and scottda want to do here | 18:00 |
| guitarzan | since I put acls on each target, I either need cinder to do this (multiple "exports/targets") or do a ton of work myself to manage it | 18:00 |
| jgriffith | they're the ones that have been working on it and putting things together | 18:00 |
| DuncanT | Don't look at me, I know nothing | 18:01 |
| jgriffith | guitarzan: yes | 18:01 |
| guitarzan | sure | 18:01 |
| hemna | hehe | 18:01 |
| hemna | no I think this sounds good jgriffith | 18:01 |
| scottda | jgriffith: I reckon there is the short term question of "can we land multi-attach in Nova with minimal changes to Cinder, and in Mitaka?" and the long term question of "should we make this better and more easily understood, to prevent the complexity from continuing to crush us?" | 18:02 |
| hemna | scottda, +1 | 18:02 |
| *** alonma has quit IRC | 18:03 | |
| jgriffith | scottda: yeah, I think so | 18:03 |
| scottda | I like the 1-1 idea for exports | 18:03 |
| jgriffith | scottda: or the third option, can I bend everyones will to meet mine and agree on the existing API's and semantics :) | 18:03 |
| *** martyturner has joined #openstack-cinder | 18:03 | |
| jgriffith | scottda: I think the answer to that is NO :) | 18:03 |
| jgriffith | scottda: because... well, maybe I"m just flat out wrong :) | 18:03 |
| jgriffith | DuncanT: I'm looking at you, you know lots! | 18:04 |
| scottda | jgriffith: I think you could make a good case for the existing API's and semantics being out there for a long time, with a reference implementation. | 18:04 |
| hemna | off to a beating....bbiab. | 18:04 |
| *** hemna is now known as hemnafk | 18:04 | |
| jgriffith | scottda: yeah, I was half joking about all that | 18:04 |
| *** esker has joined #openstack-cinder | 18:04 | |
| *** timcl has joined #openstack-cinder | 18:04 | |
| *** esker has quit IRC | 18:05 | |
| DuncanT | jgriffith: I was faking that to get invited to parties | 18:05 |
| jgriffith | scottda: so I do think that maybe some good documentation and clear consensus among core might be what we need | 18:05 |
| jgriffith | LOL | 18:05 |
| jgriffith | DuncanT: ^^ | 18:05 |
| jgriffith | scottda: part of why I think we're in a mess right now is every driver implementation has had a slightly different interpretation of what is used for what | 18:06 |
| jgriffith | scottda: which means obviously it wasn't clear and simple enough :) | 18:06 |
| *** alonma has joined #openstack-cinder | 18:06 | |
| scottda | jgriffith: Yes, that seems to come up again and again, i.e. the discussion in the meeting today around some drivers rexmove the eport during terminate_connection, instead of deach-> remove_export | 18:07 |
| *** esker has joined #openstack-cinder | 18:07 | |
| scottda | Ok, sense a mid-cycle topic here | 18:07 |
| scottda | sense/I sense | 18:08 |
| *** nestorf has quit IRC | 18:08 | |
| *** salv-orlando has quit IRC | 18:08 | |
| *** salv-orlando has joined #openstack-cinder | 18:09 | |
| jgriffith | scottda: yeah, maybe something i can contribute this time around | 18:09 |
| *** alonma has quit IRC | 18:11 | |
| *** isaacb has joined #openstack-cinder | 18:11 | |
| *** alonma has joined #openstack-cinder | 18:13 | |
| *** Yogi1 has quit IRC | 18:13 | |
| *** daneyon has joined #openstack-cinder | 18:14 | |
| *** dslevin_ has quit IRC | 18:14 | |
| patrickeast | anyone know if https://review.openstack.org/#/c/196482/ was blocked or waiting on anything? | 18:15 |
| *** daneyon_ has joined #openstack-cinder | 18:15 | |
| * patrickeast wants to have encrypted multipath volumes | 18:15 | |
| patrickeast | hemnafk: mriedem: ^ looks like you guys were the main reviewers on it so far | 18:16 |
| *** isaacb has quit IRC | 18:17 | |
| *** alonma has quit IRC | 18:17 | |
| *** daneyon has quit IRC | 18:19 | |
| mriedem | patrickeast: i had a -1 on ps8 and then it languished | 18:19 |
| mriedem | and i moved to other things | 18:19 |
| openstackgerrit | Merged openstack/cinder: Fix grammatical mistake in defining articles. https://review.openstack.org/263136 | 18:20 |
| *** dslevin_ has joined #openstack-cinder | 18:22 | |
| *** mudassirlatif has joined #openstack-cinder | 18:23 | |
| *** PsionTheory has joined #openstack-cinder | 18:24 | |
| *** alonma has joined #openstack-cinder | 18:27 | |
| *** esker has quit IRC | 18:28 | |
| *** bardia has quit IRC | 18:29 | |
| *** merooney has quit IRC | 18:30 | |
| *** alonma has quit IRC | 18:31 | |
| *** merooney has joined #openstack-cinder | 18:31 | |
| mriedem | hemnafk: are the os-brick unit tests racy? | 18:32 |
| *** wiebalck has joined #openstack-cinder | 18:34 | |
| *** leeantho has quit IRC | 18:35 | |
| *** merooney has quit IRC | 18:36 | |
| *** geguileo_mobile has joined #openstack-cinder | 18:36 | |
| *** wiebalck has left #openstack-cinder | 18:36 | |
| *** leeantho has joined #openstack-cinder | 18:38 | |
| *** geguileo_phone has quit IRC | 18:39 | |
| *** merooney has joined #openstack-cinder | 18:42 | |
| *** e0ne has joined #openstack-cinder | 18:44 | |
| openstackgerrit | Nate Potter proposed openstack/cinder: Added more options while uploading volume as image https://review.openstack.org/230617 | 18:44 |
| *** hemnafk is now known as hemna | 18:51 | |
| hemna | mriedem, sup ? | 18:51 |
| mriedem | os_brick.tests.initiator.test_connector seems very racy | 18:51 |
| mriedem | failing me w/o any changes | 18:52 |
| hemna | hrmm, it shouldn't be racy | 18:52 |
| hemna | do you have attached volumes on the box you are running the tests? | 18:52 |
| hemna | I think there might be some skips on tests for that scenario | 18:53 |
| mriedem | no... | 18:53 |
| *** krtaylor has quit IRC | 18:53 | |
| mriedem | this is a fresh clone | 18:53 |
| mriedem | running with --concurrency=1 didn't help either | 18:53 |
| mriedem | wtf | 18:53 |
| hemna | hrmm | 18:53 |
| mriedem | e.g. http://paste.openstack.org/show/483167/ | 18:54 |
| hemna | let me try in a fresh vagrant | 18:54 |
| mriedem | 'scsi_id --page 0x83 --whitelisted /dev/disk/by-path/pci-0000:00:00.0-ip-10.0.2.15:3260-iscsi-iqn.2010-10.org.openstack:volume-00000001-lun-1', | 18:54 |
| mriedem | shows up when it's not expected | 18:54 |
| *** timcl has quit IRC | 18:54 | |
| sheel | DuncanT:Hi | 18:55 |
| *** timcl has joined #openstack-cinder | 18:55 | |
| DuncanT | sheel: Hi | 18:55 |
| *** alejandrito has joined #openstack-cinder | 18:56 | |
| sheel | DuncanT: Its regarding our yesterday's discussion on "Code Refactoring" in Bug #1528989. | 18:56 |
| openstack | bug 1528989 in Cinder "Refactoring required in cinder/tests/unit/test_migrations.py" [Undecided,In progress] https://launchpad.net/bugs/1528989 - Assigned to Sheel Rana (ranasheel2000) | 18:56 |
| DuncanT | sheel: Yup | 18:56 |
| sheel | DuncanT: Could you please review patch for this @ https://review.openstack.org/#/c/264085/ ? | 18:56 |
| *** martyturner has quit IRC | 18:57 | |
| hemna | mriedem, - Failed: 0 | 18:57 |
| hemna | ah | 18:58 |
| hemna | that test is skipped unless there is a /dev/disk/by-path | 18:58 |
| hemna | you have an attached volume on your system | 18:58 |
| hemna | which is why it is skipped for me | 18:58 |
| hemna | mriedem, http://paste.openstack.org/show/483171/ | 18:59 |
| *** angela-s has joined #openstack-cinder | 18:59 | |
| DuncanT | sheel: Done | 18:59 |
| *** cknight has quit IRC | 18:59 | |
| sheel | DuncanT:Thank you!! | 19:00 |
| *** martyturner has joined #openstack-cinder | 19:00 | |
| *** cknight has joined #openstack-cinder | 19:00 | |
| mriedem | hemna: ah i see | 19:01 |
| mriedem | huh, well it seems the test is wrong then | 19:01 |
| mriedem | since it should expect "scsi_id --page 0x83 --whitelisted /dev/disk/by-path/pci-0000:00:00.0-ip-10.0.2.15:3260-iscsi-iqn.2010-10.org.openstack:volume-00000001-lun-1" | 19:01 |
| *** timcl has quit IRC | 19:01 | |
| *** martyturner has quit IRC | 19:01 | |
| *** sgotliv has joined #openstack-cinder | 19:01 | |
| *** jordanP has joined #openstack-cinder | 19:01 | |
| *** rlrossit has quit IRC | 19:01 | |
| *** martyturner has joined #openstack-cinder | 19:02 | |
| *** merooney has quit IRC | 19:03 | |
| *** bardia has joined #openstack-cinder | 19:03 | |
| hemna | ok I'll see if I can get it to reproduce here | 19:04 |
| hemna | heh that was easy | 19:05 |
| hemna | just created the /dev/disk/by-path dir and it pukes | 19:05 |
| hemna | bleh ok | 19:05 |
| mriedem | :) | 19:05 |
| hemna | mriedem, thanks for that. | 19:05 |
| *** merooney has joined #openstack-cinder | 19:06 | |
| openstackgerrit | Matt Riedemann proposed openstack/os-brick: Actually log the command used in _run_iscsiadm https://review.openstack.org/264310 | 19:06 |
| mriedem | hemna: ^ is what i really wanted | 19:06 |
| *** krtaylor has joined #openstack-cinder | 19:07 | |
| hemna | ah cool | 19:07 |
| hemna | bah, my fault | 19:07 |
| mriedem | i was wondering what was going on with these logs http://logs.openstack.org/64/263664/5/check/gate-manila-tempest-dsvm-neutron/835bf51/logs/screen-n-cpu.txt.gz#_2016-01-06_13_53_38_305 :) | 19:07 |
| mriedem | easy fix | 19:07 |
| *** rlrossit has joined #openstack-cinder | 19:08 | |
| mriedem | i guess it does log stuff, but it's not pretty | 19:08 |
| *** manous has quit IRC | 19:09 | |
| mriedem | you know what, we should just remove that entire line | 19:10 |
| mriedem | http://logs.openstack.org/64/263664/5/check/gate-manila-tempest-dsvm-neutron/835bf51/logs/screen-n-cpu.txt.gz#_2016-01-06_13_53_38_304 | 19:10 |
| *** manous has joined #openstack-cinder | 19:10 | |
| mriedem | processutils already logs the command | 19:10 |
| *** sheel has quit IRC | 19:10 | |
| hemna | ah yah it does | 19:10 |
| mriedem | i'll change that patch to just remove the logging from os-brick | 19:11 |
| hemna | ok sounds good | 19:11 |
| *** edmondsw has quit IRC | 19:14 | |
| *** edmondsw has joined #openstack-cinder | 19:17 | |
| *** harlowja_ has quit IRC | 19:18 | |
| *** jordanP has quit IRC | 19:19 | |
| *** harlowja has joined #openstack-cinder | 19:19 | |
| openstackgerrit | Matt Riedemann proposed openstack/os-brick: Remove redundant command execution debug logging https://review.openstack.org/264310 | 19:19 |
| *** jordanP has joined #openstack-cinder | 19:20 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Fix another unit test failure https://review.openstack.org/264317 | 19:23 |
| hemna | mriedem, ^^ | 19:23 |
| mriedem | will test that out | 19:23 |
| *** mc_nair has quit IRC | 19:25 | |
| *** lpetrut has quit IRC | 19:25 | |
| *** dustins_ has joined #openstack-cinder | 19:25 | |
| *** alonma has joined #openstack-cinder | 19:27 | |
| *** dustins has quit IRC | 19:29 | |
| *** dustins_ has quit IRC | 19:29 | |
| *** dustins_ has joined #openstack-cinder | 19:29 | |
| *** dustins_ is now known as dustins | 19:30 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: os-brick add extend_volume API https://review.openstack.org/243776 | 19:30 |
| mriedem | this looks bad right? http://logs.openstack.org/64/263664/5/check/gate-manila-tempest-dsvm-neutron/835bf51/logs/screen-n-cpu.txt.gz#_2016-01-06_14_05_03_716 | 19:30 |
| mriedem | iscsiadm sessions piling up | 19:30 |
| hemna | huh | 19:31 |
| mriedem | eventually the libvirt driver can't determine if the devices are volumes http://logs.openstack.org/64/263664/5/check/gate-manila-tempest-dsvm-neutron/835bf51/logs/screen-n-cpu.txt.gz#_2016-01-06_14_05_11_412 | 19:31 |
| *** alonma has quit IRC | 19:31 | |
| hemna | depends on what's going on | 19:31 |
| mriedem | and eventually we run out of available devices to use http://logs.openstack.org/64/263664/5/check/gate-manila-tempest-dsvm-neutron/835bf51/logs/screen-n-cpu.txt.gz#_2016-01-06_14_05_17_303 | 19:32 |
| mriedem | this is some manila tempest job | 19:32 |
| hemna | wonder if those are being left behind | 19:32 |
| mriedem | where they are attaching/detaching the same volume to the same instance 26 times | 19:32 |
| mriedem | and it looks like the detach is not clean | 19:32 |
| mriedem | we don't see this in the gate b/c we have tenant isolation, the manila jobs don't use tenant isolation | 19:33 |
| mriedem | so it's the same instance and same volume each time | 19:33 |
| *** alonma has joined #openstack-cinder | 19:33 | |
| mriedem | apparently this started around 1/3 | 19:33 |
| hemna | I've not seen this | 19:33 |
| mriedem | https://bugs.launchpad.net/nova/+bug/1530847 | 19:33 |
| openstack | Launchpad bug 1530847 in OpenStack Compute (nova) "TypeError attaching volume to instance" [High,In progress] - Assigned to Matt Riedemann (mriedem) | 19:33 |
| hemna | last os-brick release was 12-18 | 19:34 |
| mriedem | yeah, checked that already :) | 19:34 |
| hemna | odd that this is popping up only after 1/3 | 19:34 |
| *** ChubYann has joined #openstack-cinder | 19:34 | |
| *** changbl has joined #openstack-cinder | 19:35 | |
| *** ociuhandu has quit IRC | 19:35 | |
| *** ntpttr has quit IRC | 19:36 | |
| *** alonma has quit IRC | 19:37 | |
| mriedem | lots o targets http://logs.openstack.org/64/263664/5/check/gate-manila-tempest-dsvm-neutron/835bf51/logs/screen-c-vol.txt.gz#_2016-01-06_14_05_01_723 | 19:39 |
| *** jordanP has quit IRC | 19:39 | |
| *** alonma has joined #openstack-cinder | 19:40 | |
| hemna | so those devices are actually there | 19:42 |
| *** alonma has quit IRC | 19:44 | |
| *** dslevin_ has quit IRC | 19:47 | |
| *** e0ne has quit IRC | 19:47 | |
| mriedem | hemna: aha | 19:47 |
| mriedem | http://logs.openstack.org/64/263664/5/check/gate-manila-tempest-dsvm-neutron/835bf51/logs/libvirt/libvirtd.txt.gz | 19:47 |
| mriedem | there are many virDomainAttachDeviceFlags calls | 19:48 |
| mriedem | but only 2 virDomainDetachDeviceFlags | 19:48 |
| mriedem | for vdb | 19:48 |
| mriedem | so after that, it looks like libvirt is never detaching anything | 19:48 |
| *** vgridnev has quit IRC | 19:48 | |
| *** alonma has joined #openstack-cinder | 19:49 | |
| hemna | uhh. ok | 19:49 |
| hemna | I'm not sure I follow that | 19:49 |
| hemna | virsh magic ? | 19:50 |
| *** crose has joined #openstack-cinder | 19:51 | |
| mriedem | not sure, there was a change in nova's libvirt driver recently to retry until the device is gone from the domain | 19:51 |
| mriedem | i'm wondering if that screwed something up | 19:51 |
| hemna | did that land 1/3 ? | 19:51 |
| hemna | :) | 19:51 |
| mriedem | nope | 19:51 |
| mriedem | which is the thing that baffles me | 19:51 |
| mriedem | https://github.com/openstack/nova/commit/3a3fb3cfb2c41ad182545e47649ff12a4f3a743e | 19:51 |
| mriedem | we haven't seen a problem with that in the gate, | 19:52 |
| mriedem | however, we run with tenant isolation in the gate | 19:52 |
| mriedem | so i'm wondering if that makes a difference | 19:52 |
| mriedem | this manila job doesn't | 19:52 |
| *** dslevin_ has joined #openstack-cinder | 19:52 | |
| hemna | what does that involve actually | 19:52 |
| *** alonma has quit IRC | 19:53 | |
| *** alonma has joined #openstack-cinder | 19:55 | |
| mriedem | each test in tempest with tenant isolation will have a unique user/project to create volumes and instances | 19:58 |
| mriedem | so if test A and test B are attaching a volume to an instance, they'll at least be separate volumes/instances under different projects | 19:58 |
| mriedem | at least that's my understanding | 19:58 |
| mriedem | with manila it's always the same volume/instance (i think)? | 19:58 |
| hemna | hrmm | 19:58 |
| *** alonma has quit IRC | 19:59 | |
| *** lprice has quit IRC | 20:01 | |
| *** lpetrut has joined #openstack-cinder | 20:02 | |
| *** alonma has joined #openstack-cinder | 20:03 | |
| *** alonma has quit IRC | 20:07 | |
| hemna | so the detachDeviceFlags is trying to remove the device from the guest | 20:07 |
| hemna | I guess that would have to fail in order to leave the host device still around ? | 20:08 |
| hemna | meaning the libvirt volume driver doesn't get called | 20:08 |
| hemna | for disconnect_volume | 20:08 |
| *** ndipanov has quit IRC | 20:09 | |
| *** ociuhandu has joined #openstack-cinder | 20:11 | |
| mriedem | yeah, i checked the n-cpu logs, | 20:12 |
| mriedem | os-detach is only called twice | 20:12 |
| mriedem | so something is messed up in the tests where they aren't detaching | 20:12 |
| mriedem | but keep attaching | 20:12 |
| mriedem | and then kaboom | 20:12 |
| hemna | and that's called after brick's disconnect_volume | 20:12 |
| hemna | at least I'm pretty syre | 20:12 |
| hemna | sure | 20:12 |
| *** krtaylor has quit IRC | 20:13 | |
| jidar | do the controllers need to see the san (FC) for image-to-volume copies? | 20:13 |
| hemna | jidar, yes | 20:14 |
| hemna | and the c-vol nodes need to have FC HBA's as well | 20:14 |
| jidar | creating volumes and attaching them is fine, but this wasn't something I assumed would be needed :( | 20:14 |
| jidar | yea | 20:14 |
| jidar | damn it | 20:14 |
| hemna | heh, another reason it would be nice to have a c-data service. | 20:14 |
| jidar | yea | 20:15 |
| jidar | I was just discussing running the volume service on the compute nodes because of this (though I don't like the idea very much) | 20:15 |
| jidar | I'll have to take everything down and create some HBA's for the controllers from UCS | 20:15 |
| hemna | c-data has been an idea we've tossed around a bit for several reasons | 20:15 |
| hemna | this is a small one of those list of reasons | 20:15 |
| smcginnis | hemna: Where did we leave that (c-data)? I seem to remember most thinking it was a good idea at the last midcycle. | 20:15 |
| hemna | smcginnis, yup | 20:16 |
| hemna | there is some cool stuff we can do if we had that | 20:16 |
| smcginnis | hemna: Yeah, I think so too. | 20:16 |
| jidar | I'm not keen on the idea of mapping a lun to a controller and then unmapping it right away | 20:16 |
| jidar | lots of systems don't like having luns mapped and unmapped often | 20:16 |
| hemna | one of the things on my list to do for os-brick is to report connector capability flags | 20:16 |
| hemna | so we can tell which connectors would work on a given host. | 20:17 |
| hemna | jidar, I don't see a problem with that. | 20:17 |
| jidar | you've never lost data because a host was all pissed off it's lun was unmapped :P | 20:17 |
| smcginnis | jidar: As long as they're cleanly removed, it _shouldn't_ be a problem. :) | 20:18 |
| hemna | you should never have a lun just vanish from a host | 20:18 |
| hemna | cinder doesn't do that | 20:18 |
| hemna | cinder flushes io, then correctly removes the lun and removes the export | 20:18 |
| hemna | as it should | 20:18 |
| jidar | I'm skeptical until I see it work well :) | 20:19 |
| *** dustins has quit IRC | 20:19 | |
| hemna | oooohhhkay. | 20:19 |
| jidar | good news! I'll be testing this in the next day or two | 20:19 |
| hemna | jidar, well, good news is that cinder has been doing this exact thing for oh.....years now. | 20:19 |
| hemna | if you find a bug, feel free to file it. | 20:19 |
| jidar | will do, and thanks for the help :) | 20:20 |
| *** belmoreira has joined #openstack-cinder | 20:22 | |
| *** hemna is now known as hemnafk | 20:22 | |
| openstackgerrit | Merged openstack/cinder: Add metadata aliases to Volume object https://review.openstack.org/247763 | 20:25 |
| *** ntpttr has joined #openstack-cinder | 20:28 | |
| *** e0ne has joined #openstack-cinder | 20:28 | |
| *** e0ne has quit IRC | 20:29 | |
| *** e0ne has joined #openstack-cinder | 20:30 | |
| *** ntpttr has left #openstack-cinder | 20:32 | |
| *** martyturner has quit IRC | 20:33 | |
| *** e0ne has quit IRC | 20:33 | |
| *** vgridnev has joined #openstack-cinder | 20:33 | |
| *** manous has quit IRC | 20:36 | |
| *** merooney has quit IRC | 20:41 | |
| *** krtaylor has joined #openstack-cinder | 20:43 | |
| *** merooney has joined #openstack-cinder | 20:43 | |
| openstackgerrit | Merged openstack/os-brick: remove python 2.6 trove classifier https://review.openstack.org/260815 | 20:49 |
| *** haomaiwang has joined #openstack-cinder | 20:56 | |
| *** lprice has joined #openstack-cinder | 20:58 | |
| *** haomaiwang has quit IRC | 21:01 | |
| *** lprice1 has joined #openstack-cinder | 21:01 | |
| *** dustins has joined #openstack-cinder | 21:01 | |
| openstackgerrit | Merged openstack/cinder: Optimize 3PAR array ID retrieval https://review.openstack.org/263355 | 21:03 |
| *** martyturner has joined #openstack-cinder | 21:03 | |
| *** porrua has quit IRC | 21:03 | |
| *** lprice has quit IRC | 21:04 | |
| openstackgerrit | Merged openstack/cinder: Change minimum 3PAR API version for replication https://review.openstack.org/263458 | 21:04 |
| *** raildo is now known as raildo-afk | 21:05 | |
| *** garthb_ has joined #openstack-cinder | 21:08 | |
| *** garthb has quit IRC | 21:09 | |
| *** alonma has joined #openstack-cinder | 21:11 | |
| *** alonma has quit IRC | 21:15 | |
| *** mragupat has quit IRC | 21:16 | |
| *** mragupat has joined #openstack-cinder | 21:16 | |
| ildikov | can anyone tell me here quickly what the volume['attach_status'] can be? | 21:16 |
| ildikov | I mean it is only 'attached' and 'detached' or there are other states too? | 21:16 |
| openstackgerrit | Anthony Lee proposed openstack/cinder: 3PAR: Adding volume checks to manage snapshot API https://review.openstack.org/260728 | 21:17 |
| *** kragniz is now known as {^-^} | 21:17 | |
| *** thangp has quit IRC | 21:18 | |
| *** {^-^} is now known as Guest46353 | 21:18 | |
| *** Guest46353 is now known as kragniz | 21:18 | |
| openstackgerrit | Chris Morrell proposed openstack/cinder: Enable consisgroups in SolidFire driver https://review.openstack.org/259638 | 21:20 |
| *** alejandrito has quit IRC | 21:20 | |
| *** alonma has joined #openstack-cinder | 21:22 | |
| *** martyturner has quit IRC | 21:22 | |
| *** hemnafk is now known as hemna | 21:26 | |
| *** alonma has quit IRC | 21:26 | |
| hemna | ildikov, attaching | 21:27 |
| hemna | attaching, attached, detached | 21:27 |
| hemna | afaik | 21:27 |
| *** julim has quit IRC | 21:27 | |
| ildikov | hemna: a-ha, ok | 21:28 |
| ildikov | I just try to understand the Nova volume attach code path and the checks | 21:28 |
| *** belmoreira has quit IRC | 21:29 | |
| *** martyturner has joined #openstack-cinder | 21:30 | |
| jgriffith | ildikov: incase it's helpful: http://j-griffith.github.io/2015/07/16/volume-attach-code-flow-in-cinder/ | 21:30 |
| openstackgerrit | Merged openstack/cinder: Constant defined for sqlAlchemy VARCHAR & INTEGER https://review.openstack.org/264085 | 21:30 |
| jgriffith | ildikov: some day we'll move the states into an enum or something so we actually ALL can know what they are :) | 21:31 |
| hemna | crazy talk! | 21:32 |
| jgriffith | Blasphemy!! | 21:32 |
| ildikov | jgriffith: lol :), huge +1 from me | 21:33 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: os-brick add cinder local_dev lvm code https://review.openstack.org/260739 | 21:33 |
| ildikov | hemna: and is that status 'attached' in multiattach case after the first attach or it goes back to attaching when attaching the volume to the second instance? | 21:33 |
| hemna | yes, it's attached | 21:34 |
| hemna | it'll go from attached to attaching to attached | 21:34 |
| hemna | and the volume status field will go from in-use to attaching to in-use | 21:34 |
| *** xyang has quit IRC | 21:35 | |
| ildikov | ok, cool, tnx | 21:35 |
| *** cknight has quit IRC | 21:36 | |
| ildikov | although I realized this will still not help me, but I might understand the code now a bit more, so it's good :) | 21:36 |
| hemna | great, I'll be asking you questions then :) | 21:36 |
| *** merooney has quit IRC | 21:40 | |
| *** gouthamr has quit IRC | 21:41 | |
| *** dave-mccowan has quit IRC | 21:47 | |
| hemna | have I said how much I hate the new gerrit UI yet today...... | 21:47 |
| smcginnis | hemna: It is worth stating daily. | 21:48 |
| smcginnis | Though I think I am slowly getting used to it. | 21:48 |
| hemna | I just ran into a new to me pain | 21:49 |
| hemna | I was trying to vote on a review, by hitting the reply button | 21:50 |
| hemna | but the voting +1, -1 things were gone | 21:50 |
| hemna | w t f!?!?! | 21:50 |
| smcginnis | You looked at an old version, right? | 21:50 |
| hemna | yah | 21:50 |
| hemna | a version of the patchset | 21:50 |
| hemna | so amazingly lame | 21:50 |
| smcginnis | Took me a bit on that one too. | 21:50 |
| smcginnis | Not at all obvious, | 21:50 |
| hemna | nope | 21:50 |
| hemna | who the hell thought this ui was a good idea? | 21:51 |
| hemna | </rant> | 21:51 |
| smcginnis | And no way to switch it either from what I saw other than knowing to remove the /nn from the URL. | 21:51 |
| hemna | that's what I had to do | 21:51 |
| eharney | it's way over on the top right as a "Patch Sets" dropdown | 21:52 |
| smcginnis | Well lookey there. | 21:52 |
| smcginnis | Thanks eharney | 21:52 |
| hemna | oh man | 21:53 |
| smcginnis | Still horrible design though. :) | 21:53 |
| hemna | way over there. | 21:53 |
| hemna | *smh* | 21:53 |
| *** cknight has joined #openstack-cinder | 21:55 | |
| *** cknight has quit IRC | 21:55 | |
| *** vgridnev has quit IRC | 21:55 | |
| *** erlon has quit IRC | 21:56 | |
| scottda | hemna: ildikov I think the only valid attach_status's are "attached" and "detached" | 21:57 |
| scottda | It is the state that can be 'available', 'attaching' ,'detaching', 'in-use' | 21:58 |
| scottda | (or 'creating', 'deleting', 'error*' | 21:58 |
| jgriffith | scottda: hehe... state vs status :) | 21:58 |
| jgriffith | what were we thinking | 21:58 |
| ildikov | scottda: hmm, then I don't understand that Nova code again | 21:58 |
| scottda | yeah, and I've never understood why we need both 'attach_status' and 'state' | 21:59 |
| scottda | but both are used in Nova | 21:59 |
| ildikov | hemna: smcginnis: I hate that new UI too, will take longer for me to get used to :( | 21:59 |
| ildikov | scottda: if attach_status can be attaching, then I think I understand | 21:59 |
| ildikov | if not, then I don't | 22:00 |
| scottda | ildikov: no, I don't think attach_status can be attaching | 22:00 |
| ildikov | scottda: how hard is it find these states out ffrom the code? | 22:00 |
| scottda | just a sec, let me find the code | 22:00 |
| ildikov | not that I don't believe you just I would check the state then too, etc. | 22:01 |
| *** garthb_ has quit IRC | 22:02 | |
| *** akerr_ has joined #openstack-cinder | 22:02 | |
| hemna | ildikov, attach_status can be attaching | 22:02 |
| *** pots3 has joined #openstack-cinder | 22:03 | |
| ildikov | hemna: could you discuss this with scottda? | 22:03 |
| ildikov | hemna: ... he's just wrote the opposite above :) | 22:03 |
| Swanson | smcginnis, I like the UI. | 22:03 |
| scottda | I could be wrong, so go with hemna | 22:03 |
| *** martyturner1 has joined #openstack-cinder | 22:03 | |
| smcginnis | Swanson: Well, I already know you're odd. | 22:03 |
| hemna | hehe | 22:04 |
| *** ociuhandu_ has joined #openstack-cinder | 22:04 | |
| ildikov | :D | 22:04 |
| *** markstur_ has joined #openstack-cinder | 22:04 | |
| *** DuncanT has quit IRC | 22:04 | |
| *** martyturner has quit IRC | 22:04 | |
| *** ociuhandu has quit IRC | 22:04 | |
| *** laurent\ has quit IRC | 22:04 | |
| *** doffm has quit IRC | 22:04 | |
| *** swamireddy has quit IRC | 22:04 | |
| *** ociuhandu_ is now known as ociuhandu | 22:04 | |
| *** MIDENN_ has joined #openstack-cinder | 22:04 | |
| scottda | yeah, I see where it is sometimes attaching, my bad | 22:04 |
| *** aorourke_ has joined #openstack-cinder | 22:04 | |
| *** doffm has joined #openstack-cinder | 22:04 | |
| *** pots1 has quit IRC | 22:04 | |
| *** akerr has quit IRC | 22:05 | |
| *** markstur has quit IRC | 22:05 | |
| *** obutenko has quit IRC | 22:05 | |
| *** liewegas has quit IRC | 22:05 | |
| *** aorourke has quit IRC | 22:05 | |
| *** mdenny has quit IRC | 22:05 | |
| *** cppforlife_ has quit IRC | 22:05 | |
| openstackgerrit | Kedar Vidvans proposed openstack/cinder: Filtering type extra-spec support to ZFSSA drivers https://review.openstack.org/263766 | 22:05 |
| ildikov | scottda: no worries, and thanks for checking | 22:05 |
| *** obutenko has joined #openstack-cinder | 22:05 | |
| hemna | scottda, https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L892 | 22:05 |
| *** cppforlife_ has joined #openstack-cinder | 22:06 | |
| *** liewegas has joined #openstack-cinder | 22:06 | |
| *** laurent\ has joined #openstack-cinder | 22:06 | |
| *** sasukeh has joined #openstack-cinder | 22:06 | |
| scottda | Yeah, my confusion. All part of the Suck of having a status that can be attaching and an attach_status that can be attaching... | 22:06 |
| hemna | yah | 22:07 |
| scottda | someone really should fix all this :) | 22:07 |
| ildikov | yeap that would be nice | 22:08 |
| * hemna waves magic wand. | 22:09 | |
| hemna | hrmm...nope | 22:09 |
| *** salv-orl_ has joined #openstack-cinder | 22:09 | |
| *** martyturner1 has quit IRC | 22:09 | |
| scottda | need a new wand | 22:09 |
| openstackgerrit | Reedip proposed openstack/python-cinderclient: Fix argument order for assertEqual to (expected, observed) https://review.openstack.org/264006 | 22:10 |
| ildikov | ok guys, I call it a day | 22:10 |
| ildikov | thanks for the help! | 22:10 |
| scottda | g'night ildikov | 22:10 |
| hemna | ildikov, night! | 22:10 |
| ildikov | laters :) | 22:11 |
| Swanson | jgriffith, If you get a chance to take a look at this one https://review.openstack.org/#/c/244867/ that would be cool. It is the one I mentioned yesterday. Thanks! | 22:11 |
| *** salv-orlando has quit IRC | 22:12 | |
| *** lprice1 has quit IRC | 22:12 | |
| *** fernnest has quit IRC | 22:14 | |
| *** swamireddy has joined #openstack-cinder | 22:15 | |
| *** crose has quit IRC | 22:15 | |
| openstackgerrit | Anthony Lee proposed openstack/os-brick: Adding a tool for querying volume paths https://review.openstack.org/204191 | 22:15 |
| *** garthb_ has joined #openstack-cinder | 22:17 | |
| *** DuncanT has joined #openstack-cinder | 22:18 | |
| *** fernnest has joined #openstack-cinder | 22:18 | |
| *** fmccrthy has quit IRC | 22:22 | |
| *** dims has quit IRC | 22:22 | |
| *** alonma has joined #openstack-cinder | 22:22 | |
| *** fmccrthy has joined #openstack-cinder | 22:24 | |
| *** alonma has quit IRC | 22:27 | |
| *** alonma has joined #openstack-cinder | 22:28 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: WIP 3PAR fix driver to work with image cache https://review.openstack.org/236143 | 22:31 |
| *** Lee1092 has quit IRC | 22:31 | |
| *** jungleboyj has quit IRC | 22:32 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: WIP 3PAR fix driver to work with image cache https://review.openstack.org/236143 | 22:32 |
| *** alonma has quit IRC | 22:33 | |
| openstackgerrit | Chris Morrell proposed openstack/cinder: Enable consisgroups in SolidFire driver https://review.openstack.org/259638 | 22:38 |
| *** haomaiwang has joined #openstack-cinder | 22:45 | |
| *** mriedem has quit IRC | 22:46 | |
| *** haomaiwang has quit IRC | 22:49 | |
| *** jgregor has quit IRC | 22:50 | |
| *** jgregor has joined #openstack-cinder | 22:51 | |
| *** krtaylor has quit IRC | 22:51 | |
| *** baumann has quit IRC | 22:53 | |
| *** dims has joined #openstack-cinder | 22:53 | |
| *** lpetrut has quit IRC | 22:55 | |
| *** rlrossit has quit IRC | 22:56 | |
| *** mragupat has quit IRC | 22:57 | |
| *** sasukeh has quit IRC | 23:01 | |
| *** dims has quit IRC | 23:04 | |
| *** diablo_rojo has quit IRC | 23:09 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick: Updated from global requirements https://review.openstack.org/262911 | 23:10 |
| *** diablo_rojo has joined #openstack-cinder | 23:11 | |
| *** fernnest_ has joined #openstack-cinder | 23:12 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/264414 | 23:13 |
| *** dustins has quit IRC | 23:14 | |
| *** diablo_rojo has quit IRC | 23:18 | |
| chris_morrell | patrickeast have a second? jgriffith mentioned talking to you re: trim/discard support in nova and cinder. Looks like it went into Nova in Sept 2014? | 23:20 |
| *** alonma has joined #openstack-cinder | 23:29 | |
| *** dims_ has joined #openstack-cinder | 23:29 | |
| *** gouthamr has joined #openstack-cinder | 23:29 | |
| *** dave-mccowan has joined #openstack-cinder | 23:31 | |
| *** alonma has quit IRC | 23:33 | |
| *** dims_ has quit IRC | 23:34 | |
| *** dims_ has joined #openstack-cinder | 23:36 | |
| *** mriedem has joined #openstack-cinder | 23:44 | |
| *** krtaylor has joined #openstack-cinder | 23:45 | |
| openstackgerrit | Matt Riedemann proposed openstack/os-brick: Actually log the command used in _run_iscsiadm https://review.openstack.org/264310 | 23:50 |
| *** yhayashi has joined #openstack-cinder | 23:52 | |
| *** s-moriya has joined #openstack-cinder | 23:54 | |
| *** akerr_ is now known as akerr_away | 23:55 | |
| jgriffith | chris_morrell: Sept 2014? Or 2015? | 23:58 |
| * jgriffith supposes he could go look :) | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!