| *** jmlowe has quit IRC | 00:00 | |
| *** edmondsw has quit IRC | 00:00 | |
| *** jmlowe has joined #openstack-cinder | 00:00 | |
| geguileo | erlon: unfortunately this is not a directory, this is a pool in Ceph :-( | 00:03 |
|---|---|---|
| *** diablo_rojo has quit IRC | 00:05 | |
| *** diablo_rojo has joined #openstack-cinder | 00:06 | |
| *** hoonetorg has quit IRC | 00:15 | |
| *** hoonetorg has joined #openstack-cinder | 00:16 | |
| *** felipemonteiro has joined #openstack-cinder | 00:19 | |
| *** sbra has joined #openstack-cinder | 00:22 | |
| *** erlon has quit IRC | 00:27 | |
| *** armax has joined #openstack-cinder | 00:31 | |
| *** esker has quit IRC | 00:35 | |
| openstackgerrit | John Griffith proposed openstack/cinder master: Add ability to specify mode to attachment-create https://review.openstack.org/532702 | 00:40 |
| *** wanghao has quit IRC | 00:42 | |
| *** wanghao has joined #openstack-cinder | 00:42 | |
| *** claudiub has quit IRC | 00:43 | |
| *** rmcall has quit IRC | 00:46 | |
| *** rmcall has joined #openstack-cinder | 00:46 | |
| *** sbra has quit IRC | 00:46 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 00:46 | |
| openstackgerrit | Sean McGinnis proposed openstack/os-brick master: Remove the unnecessary pv_list assign during LVM object init https://review.openstack.org/481630 | 00:59 |
| *** mvk has quit IRC | 01:01 | |
| *** threestrands has quit IRC | 01:03 | |
| *** zhaochao has joined #openstack-cinder | 01:04 | |
| *** caisan has joined #openstack-cinder | 01:10 | |
| *** stakeda has joined #openstack-cinder | 01:12 | |
| *** Dinesh_Bhor has quit IRC | 01:19 | |
| *** markstur has quit IRC | 01:26 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 01:26 | |
| openstackgerrit | John Griffith proposed openstack/cinder master: Enable multiattach capability https://review.openstack.org/532280 | 01:27 |
| *** masayukig_ has quit IRC | 01:28 | |
| *** masayukig has joined #openstack-cinder | 01:30 | |
| *** zhurong has joined #openstack-cinder | 01:33 | |
| *** felipemonteiro has quit IRC | 01:35 | |
| *** rcernin has quit IRC | 01:42 | |
| *** caisan_ has joined #openstack-cinder | 01:43 | |
| *** edmondsw has joined #openstack-cinder | 01:44 | |
| *** caisan has quit IRC | 01:47 | |
| *** edmondsw has quit IRC | 01:48 | |
| *** markvoelker has joined #openstack-cinder | 01:50 | |
| *** rcernin has joined #openstack-cinder | 01:50 | |
| *** Dinesh_Bhor has quit IRC | 01:52 | |
| *** Apoorva has quit IRC | 01:58 | |
| *** moshele has joined #openstack-cinder | 02:01 | |
| openstackgerrit | zheng yin proposed openstack/cinder master: backup/ceph: remove redundant pool name command line option https://review.openstack.org/489949 | 02:04 |
| mriedem | jgriffith: ildikov: noticed something tonight, the nova spec for multiattach says cinder will add a policy rule for "Enable/Disable multiattach=True + bootable=True" | 02:07 |
| mriedem | but i don't see that called out in the cinder spec, only a policy rule for enabling/disabling multiattach altogether | 02:07 |
| mriedem | was that just an oversight? | 02:07 |
| mriedem | https://specs.openstack.org/openstack/cinder-specs/specs/queens/enable-multiattach.html#multiattach-policy | 02:07 |
| *** dalgaaf has quit IRC | 02:07 | |
| *** wanghao_ has joined #openstack-cinder | 02:08 | |
| *** wanghao has quit IRC | 02:08 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 02:08 | |
| *** dalgaaf has joined #openstack-cinder | 02:09 | |
| *** gouthamr has joined #openstack-cinder | 02:10 | |
| *** caisan has joined #openstack-cinder | 02:14 | |
| *** esker has joined #openstack-cinder | 02:14 | |
| *** Apoorva has joined #openstack-cinder | 02:14 | |
| *** gouthamr_ has joined #openstack-cinder | 02:15 | |
| openstackgerrit | zheng yin proposed openstack/python-cinderclient master: Add snapshot_id param note for backup-create https://review.openstack.org/489931 | 02:15 |
| *** gouthamr has quit IRC | 02:15 | |
| *** caisan_ has quit IRC | 02:17 | |
| *** namnh has joined #openstack-cinder | 02:18 | |
| *** gouthamr_ has quit IRC | 02:19 | |
| *** Apoorva has quit IRC | 02:19 | |
| openstackgerrit | Ryan Liang proposed openstack/cinder master: VNX: fix options incorrect behavior https://review.openstack.org/529202 | 02:20 |
| *** rmcall has quit IRC | 02:20 | |
| *** gouthamr has joined #openstack-cinder | 02:23 | |
| openstackgerrit | liuyamin proposed openstack/cinder master: Fix url error about showing API v2 details https://review.openstack.org/532710 | 02:28 |
| *** kwathore has quit IRC | 02:28 | |
| *** kwathore has joined #openstack-cinder | 02:29 | |
| *** Dinesh_Bhor has quit IRC | 02:36 | |
| *** markvoelker has quit IRC | 02:36 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 02:36 | |
| openstackgerrit | Peter Wang proposed openstack/cinder master: Unity: Add revert_to_snapshot support https://review.openstack.org/532071 | 02:37 |
| openstackgerrit | Ryan Liang proposed openstack/cinder master: [VNX] Restore snapshot to volume https://review.openstack.org/529497 | 02:39 |
| *** gouthamr has quit IRC | 02:40 | |
| mriedem | https://developer.openstack.org/api-ref/block-storage/v2/#show-volume-details - is multiattach really optional in the GET /volumes/id/detail response? | 02:42 |
| mriedem | nope https://github.com/openstack/cinder/blob/master/cinder/api/v2/views/volumes.py#L91 | 02:43 |
| mriedem | unless None is optional | 02:43 |
| *** threestrands has joined #openstack-cinder | 02:47 | |
| openstackgerrit | Ryan Liang proposed openstack/cinder master: [VNX] Restore snapshot to volume https://review.openstack.org/529497 | 02:48 |
| *** caisan_ has joined #openstack-cinder | 02:48 | |
| openstackgerrit | Matt Riedemann proposed openstack/cinder master: api-ref: fix the response parameter for multiattach https://review.openstack.org/532715 | 02:49 |
| *** caisan has quit IRC | 02:51 | |
| *** wanghao has joined #openstack-cinder | 02:54 | |
| *** wanghao_ has quit IRC | 02:54 | |
| *** mriedem has quit IRC | 02:56 | |
| *** moshele has quit IRC | 02:58 | |
| *** wanghao_ has joined #openstack-cinder | 02:59 | |
| *** wanghao has quit IRC | 03:00 | |
| *** Dinesh_Bhor has quit IRC | 03:01 | |
| *** masber has quit IRC | 03:02 | |
| *** masber has joined #openstack-cinder | 03:05 | |
| *** rmcall has joined #openstack-cinder | 03:10 | |
| *** caisan_ has quit IRC | 03:15 | |
| *** caisan has joined #openstack-cinder | 03:15 | |
| *** esker has quit IRC | 03:16 | |
| *** felipemonteiro has joined #openstack-cinder | 03:18 | |
| *** wanghao has joined #openstack-cinder | 03:19 | |
| *** wanghao has quit IRC | 03:20 | |
| *** wanghao has joined #openstack-cinder | 03:20 | |
| *** wanghao_ has quit IRC | 03:21 | |
| *** harlowja_ has quit IRC | 03:21 | |
| *** caisan_ has joined #openstack-cinder | 03:22 | |
| *** caisan has quit IRC | 03:26 | |
| *** Brin has joined #openstack-cinder | 03:33 | |
| *** sbra has joined #openstack-cinder | 03:42 | |
| *** caisan has joined #openstack-cinder | 03:51 | |
| *** caisan_ has quit IRC | 03:53 | |
| *** sbra has quit IRC | 03:54 | |
| *** abhishekk has joined #openstack-cinder | 03:54 | |
| *** caisan has quit IRC | 03:57 | |
| *** felipemonteiro has quit IRC | 03:57 | |
| *** caisan has joined #openstack-cinder | 03:58 | |
| openstackgerrit | John Griffith proposed openstack/cinder master: Enable multiattach capability https://review.openstack.org/532280 | 04:02 |
| *** caisan has quit IRC | 04:02 | |
| *** gkadam has joined #openstack-cinder | 04:03 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 04:05 | |
| *** zhurong has quit IRC | 04:12 | |
| *** felipemonteiro has joined #openstack-cinder | 04:15 | |
| *** daidv has joined #openstack-cinder | 04:16 | |
| *** armax has quit IRC | 04:24 | |
| *** links has joined #openstack-cinder | 04:25 | |
| *** ykarel|away has joined #openstack-cinder | 04:33 | |
| *** Dinesh_Bhor has quit IRC | 04:38 | |
| *** felipemonteiro has quit IRC | 04:39 | |
| *** chhavi__ has joined #openstack-cinder | 04:46 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 04:53 | |
| *** rmcall has quit IRC | 04:54 | |
| *** Dinesh_Bhor has quit IRC | 04:56 | |
| *** links has quit IRC | 04:56 | |
| *** links has joined #openstack-cinder | 04:57 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 04:58 | |
| *** caisan has joined #openstack-cinder | 04:59 | |
| *** caisan has quit IRC | 05:01 | |
| *** caisan has joined #openstack-cinder | 05:01 | |
| *** itlinux has quit IRC | 05:14 | |
| *** claudiub has joined #openstack-cinder | 05:15 | |
| *** gkadam has quit IRC | 05:16 | |
| *** edmondsw has joined #openstack-cinder | 05:20 | |
| *** psachin has joined #openstack-cinder | 05:20 | |
| *** edmondsw has quit IRC | 05:24 | |
| *** psachin has quit IRC | 05:27 | |
| *** zhurong has joined #openstack-cinder | 05:28 | |
| *** rcernin has quit IRC | 05:29 | |
| *** harlowja has joined #openstack-cinder | 05:29 | |
| *** gcb has quit IRC | 05:30 | |
| *** gcb has joined #openstack-cinder | 05:31 | |
| *** psachin has joined #openstack-cinder | 05:35 | |
| openstackgerrit | Chhavi Agarwal proposed openstack/cinder master: Run backup-restore operations on native thread https://review.openstack.org/518316 | 05:42 |
| openstackgerrit | Chhavi Agarwal proposed openstack/cinder master: Run backup-restore operations on native thread https://review.openstack.org/518316 | 05:45 |
| *** markvoelker has joined #openstack-cinder | 05:47 | |
| *** Dinesh_Bhor has quit IRC | 05:52 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 05:56 | |
| *** moshele has joined #openstack-cinder | 05:58 | |
| *** caisan has quit IRC | 05:58 | |
| *** sandanar has joined #openstack-cinder | 05:58 | |
| *** caisan has joined #openstack-cinder | 05:59 | |
| *** ykarel|away is now known as ykarel | 05:59 | |
| *** Dinesh_Bhor has quit IRC | 06:04 | |
| *** caisan_ has joined #openstack-cinder | 06:07 | |
| *** hoonetorg has quit IRC | 06:08 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: Validate metadata before reserve quota https://review.openstack.org/532427 | 06:08 |
| *** caisan has quit IRC | 06:10 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 06:12 | |
| *** armaan has quit IRC | 06:17 | |
| *** Dinesh_Bhor has quit IRC | 06:17 | |
| *** hoonetorg has joined #openstack-cinder | 06:21 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 06:32 | |
| *** ntpttr_laptop has quit IRC | 06:36 | |
| *** moshele has quit IRC | 06:37 | |
| *** markvoelker has quit IRC | 06:40 | |
| *** markvoelker has joined #openstack-cinder | 06:42 | |
| *** caisan_ has quit IRC | 06:46 | |
| *** alexchadin has joined #openstack-cinder | 06:46 | |
| *** caisan has joined #openstack-cinder | 06:46 | |
| *** harlowja has quit IRC | 06:47 | |
| *** lkuchlan has joined #openstack-cinder | 06:51 | |
| *** caisan has quit IRC | 06:52 | |
| *** caisan has joined #openstack-cinder | 06:53 | |
| *** caisan has quit IRC | 06:54 | |
| *** Dinesh_Bhor has quit IRC | 06:59 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 06:59 | |
| *** Dinesh_Bhor has quit IRC | 06:59 | |
| *** threestrands has quit IRC | 07:00 | |
| *** sbra has joined #openstack-cinder | 07:00 | |
| *** threestrands has joined #openstack-cinder | 07:01 | |
| *** threestrands has quit IRC | 07:01 | |
| *** threestrands has joined #openstack-cinder | 07:01 | |
| *** threestrands has quit IRC | 07:02 | |
| *** threestrands has joined #openstack-cinder | 07:03 | |
| *** threestrands has quit IRC | 07:03 | |
| *** threestrands has joined #openstack-cinder | 07:03 | |
| *** threestrands has quit IRC | 07:04 | |
| *** threestrands has joined #openstack-cinder | 07:04 | |
| *** caisan has joined #openstack-cinder | 07:07 | |
| *** edmondsw has joined #openstack-cinder | 07:08 | |
| *** kholkina has joined #openstack-cinder | 07:10 | |
| *** caisan has quit IRC | 07:11 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: Check create from image policy when image not none https://review.openstack.org/532747 | 07:13 |
| *** edmondsw has quit IRC | 07:13 | |
| *** armaan has joined #openstack-cinder | 07:13 | |
| *** pcaruana has joined #openstack-cinder | 07:21 | |
| *** dims_ has quit IRC | 07:21 | |
| *** hemna_ has quit IRC | 07:21 | |
| openstackgerrit | Chandan Kumar proposed openstack/cinder-tempest-plugin master: Added cinder tempest plugin zuulv3 jobs to execute tempest https://review.openstack.org/531206 | 07:23 |
| openstackgerrit | Zhao Chao proposed openstack/python-cinderclient master: Migrate to keystoneauth identity cli opts. https://review.openstack.org/532217 | 07:24 |
| openstackgerrit | Yu Zhang proposed openstack/cinder master: [Doc] Add replication group support in doc for Storwize https://review.openstack.org/532749 | 07:24 |
| *** dims has joined #openstack-cinder | 07:25 | |
| *** caisan has joined #openstack-cinder | 07:29 | |
| *** moshele has joined #openstack-cinder | 07:35 | |
| *** armaan has quit IRC | 07:41 | |
| *** armaan has joined #openstack-cinder | 07:41 | |
| *** armaan has quit IRC | 07:46 | |
| *** armaan has joined #openstack-cinder | 07:47 | |
| *** caisan_ has joined #openstack-cinder | 08:00 | |
| *** caisan has quit IRC | 08:03 | |
| *** evin has joined #openstack-cinder | 08:04 | |
| *** sbra has quit IRC | 08:06 | |
| *** AlexeyAbashkin has joined #openstack-cinder | 08:08 | |
| *** threestrands has quit IRC | 08:08 | |
| *** tesseract has joined #openstack-cinder | 08:17 | |
| *** gkadam has joined #openstack-cinder | 08:18 | |
| *** alexchadin has quit IRC | 08:28 | |
| *** caisan has joined #openstack-cinder | 08:30 | |
| *** caisan has quit IRC | 08:32 | |
| *** caisan has joined #openstack-cinder | 08:33 | |
| *** caisan_ has quit IRC | 08:33 | |
| *** caisan has quit IRC | 08:34 | |
| *** caisan has joined #openstack-cinder | 08:34 | |
| *** alexchadin has joined #openstack-cinder | 08:41 | |
| *** pcaruana has quit IRC | 08:44 | |
| *** links has quit IRC | 08:46 | |
| *** armaan has quit IRC | 08:47 | |
| *** armaan has joined #openstack-cinder | 08:48 | |
| *** esker has joined #openstack-cinder | 08:53 | |
| *** esker has quit IRC | 08:56 | |
| *** edmondsw has joined #openstack-cinder | 08:56 | |
| *** wanghao has quit IRC | 08:58 | |
| *** wanghao has joined #openstack-cinder | 08:58 | |
| *** wanghao has quit IRC | 08:59 | |
| *** wanghao has joined #openstack-cinder | 08:59 | |
| *** links has joined #openstack-cinder | 08:59 | |
| *** wanghao has quit IRC | 09:00 | |
| *** wanghao has joined #openstack-cinder | 09:00 | |
| *** wanghao has quit IRC | 09:00 | |
| *** wanghao has joined #openstack-cinder | 09:01 | |
| *** edmondsw has quit IRC | 09:01 | |
| *** wanghao has quit IRC | 09:03 | |
| *** wanghao has joined #openstack-cinder | 09:04 | |
| *** wanghao has quit IRC | 09:04 | |
| *** wanghao has joined #openstack-cinder | 09:04 | |
| *** wanghao has quit IRC | 09:05 | |
| *** wanghao has joined #openstack-cinder | 09:05 | |
| *** caisan_ has joined #openstack-cinder | 09:05 | |
| *** wanghao has quit IRC | 09:06 | |
| *** wanghao has joined #openstack-cinder | 09:06 | |
| *** wanghao has quit IRC | 09:07 | |
| *** wanghao has joined #openstack-cinder | 09:07 | |
| *** wanghao has quit IRC | 09:07 | |
| *** wanghao has joined #openstack-cinder | 09:08 | |
| *** caisan has quit IRC | 09:08 | |
| *** wanghao has quit IRC | 09:08 | |
| *** wanghao has joined #openstack-cinder | 09:08 | |
| *** Brin has quit IRC | 09:09 | |
| *** wanghao has quit IRC | 09:09 | |
| *** wanghao has joined #openstack-cinder | 09:09 | |
| *** wanghao has quit IRC | 09:10 | |
| *** wanghao has joined #openstack-cinder | 09:10 | |
| *** wanghao has quit IRC | 09:10 | |
| *** e0ne has joined #openstack-cinder | 09:21 | |
| *** ktibi has joined #openstack-cinder | 09:22 | |
| *** elmaciej has joined #openstack-cinder | 09:25 | |
| *** armaan has quit IRC | 09:27 | |
| *** armaan has joined #openstack-cinder | 09:27 | |
| *** armaan has quit IRC | 09:36 | |
| *** caisan has joined #openstack-cinder | 09:36 | |
| *** caisan_ has quit IRC | 09:39 | |
| *** links has quit IRC | 09:51 | |
| *** stakeda has quit IRC | 09:57 | |
| *** kholkina has quit IRC | 10:02 | |
| *** armaan has joined #openstack-cinder | 10:03 | |
| *** kholkina has joined #openstack-cinder | 10:03 | |
| *** pcaruana has joined #openstack-cinder | 10:04 | |
| *** links has joined #openstack-cinder | 10:05 | |
| *** caisan_ has joined #openstack-cinder | 10:07 | |
| *** caisan has quit IRC | 10:10 | |
| *** moshele has quit IRC | 10:12 | |
| *** markvoelker has quit IRC | 10:13 | |
| *** namnh has quit IRC | 10:14 | |
| *** mvk has joined #openstack-cinder | 10:14 | |
| *** caisan_ has quit IRC | 10:28 | |
| *** alexchadin has quit IRC | 10:34 | |
| *** zhurong has quit IRC | 10:35 | |
| *** lkuchlan has quit IRC | 10:36 | |
| *** e0ne has quit IRC | 10:38 | |
| *** e0ne has joined #openstack-cinder | 10:39 | |
| *** sandanar has quit IRC | 10:39 | |
| *** sandanar has joined #openstack-cinder | 10:40 | |
| *** edmondsw has joined #openstack-cinder | 10:44 | |
| *** danpawlik_ has joined #openstack-cinder | 10:46 | |
| *** danpawlik has quit IRC | 10:48 | |
| *** edmondsw has quit IRC | 10:49 | |
| openstackgerrit | Merged openstack/python-cinderclient master: Fix 'cluster' paramter using for v3 volume manage https://review.openstack.org/531344 | 10:49 |
| *** moshele has joined #openstack-cinder | 11:03 | |
| *** abhishekk has quit IRC | 11:03 | |
| *** AlexeyAbashkin has quit IRC | 11:06 | |
| openstackgerrit | Lucian Petrut proposed openstack/cinder master: Windows: allow multiple c-vol backends per service https://review.openstack.org/522806 | 11:21 |
| *** AlexeyAbashkin has joined #openstack-cinder | 11:25 | |
| *** tpsilva_ has joined #openstack-cinder | 11:31 | |
| *** tpsilva_ is now known as tpsilva | 11:34 | |
| *** lhx_ has joined #openstack-cinder | 11:36 | |
| *** sdague has joined #openstack-cinder | 11:39 | |
| *** e0ne has quit IRC | 11:58 | |
| *** icey has quit IRC | 12:02 | |
| *** pcaruana has quit IRC | 12:02 | |
| *** pcaruana|afk| has joined #openstack-cinder | 12:02 | |
| *** icey has joined #openstack-cinder | 12:02 | |
| *** lkuchlan has joined #openstack-cinder | 12:13 | |
| *** markvoelker has joined #openstack-cinder | 12:14 | |
| *** sandanar has quit IRC | 12:18 | |
| *** mmethot_ has quit IRC | 12:23 | |
| *** mmethot_ has joined #openstack-cinder | 12:23 | |
| tommylikehu | hey anyone could take a look at these bug fix patches :) thanks! | 12:24 |
| tommylikehu | https://review.openstack.org/#/c/532427/ | 12:24 |
| tommylikehu | https://review.openstack.org/#/c/532747/ | 12:24 |
| tommylikehu | https://review.openstack.org/#/c/532781/ | 12:24 |
| tommylikehu | https://review.openstack.org/#/c/531345/ | 12:24 |
| *** mvk has quit IRC | 12:25 | |
| *** jose-phi_ has joined #openstack-cinder | 12:26 | |
| *** jose-phillips has quit IRC | 12:27 | |
| *** geguileo has quit IRC | 12:27 | |
| openstackgerrit | Eric Young proposed openstack/cinder master: Modify tests to account for ScaleIO creating volumes = 8GB https://review.openstack.org/532684 | 12:27 |
| *** geguileo has joined #openstack-cinder | 12:28 | |
| *** geguileo is now known as Guest69243 | 12:28 | |
| *** e0ne has joined #openstack-cinder | 12:30 | |
| *** edmondsw has joined #openstack-cinder | 12:33 | |
| *** e0ne has quit IRC | 12:33 | |
| *** edmondsw has quit IRC | 12:38 | |
| *** e0ne has joined #openstack-cinder | 12:42 | |
| *** moshele has quit IRC | 12:46 | |
| *** markvoelker has quit IRC | 12:48 | |
| *** pgadiya has joined #openstack-cinder | 12:49 | |
| *** pcaruana|afk| has quit IRC | 12:49 | |
| *** moshele has joined #openstack-cinder | 12:50 | |
| openstackgerrit | Chandan Kumar proposed openstack/cinder-tempest-plugin master: Added cinder tempest plugin zuulv3 jobs to execute tempest https://review.openstack.org/531206 | 12:51 |
| *** e0ne has quit IRC | 12:52 | |
| *** e0ne has joined #openstack-cinder | 12:53 | |
| *** pcaruana|afk| has joined #openstack-cinder | 13:02 | |
| *** kholkina has quit IRC | 13:02 | |
| *** kholkina has joined #openstack-cinder | 13:03 | |
| *** lhx_ has quit IRC | 13:09 | |
| *** abishop has joined #openstack-cinder | 13:17 | |
| *** e0ne has quit IRC | 13:20 | |
| *** e0ne has joined #openstack-cinder | 13:24 | |
| *** mvk has joined #openstack-cinder | 13:26 | |
| *** e0ne has quit IRC | 13:30 | |
| *** e0ne has joined #openstack-cinder | 13:33 | |
| *** e0ne has quit IRC | 13:34 | |
| *** lhx_ has joined #openstack-cinder | 13:37 | |
| *** markvoelker has joined #openstack-cinder | 13:44 | |
| openstackgerrit | Eric Young proposed openstack/cinder master: ScaleIO: Enable multiattach for volumes https://review.openstack.org/531511 | 13:49 |
| *** liverpooler has joined #openstack-cinder | 13:51 | |
| *** abhishekk has joined #openstack-cinder | 13:51 | |
| *** masber has quit IRC | 13:52 | |
| *** dustins has joined #openstack-cinder | 13:55 | |
| *** pgadiya has quit IRC | 13:56 | |
| *** e0ne has joined #openstack-cinder | 13:58 | |
| *** alexchadin has joined #openstack-cinder | 13:59 | |
| *** lkuchlan has quit IRC | 14:01 | |
| *** eharney has joined #openstack-cinder | 14:07 | |
| chandankumar | smcginnis: eharney can we get this merged https://review.openstack.org/#/c/529824/ so that i can finish packaging on RDO side | 14:11 |
| eharney | chandankumar: i think so, but i'm not sure i follow smcginnis's comment there | 14:13 |
| *** lkuchlan has joined #openstack-cinder | 14:14 | |
| chandankumar | eharney: https://review.openstack.org/#/c/531206/ regarding jobs which tests we want to run, complete cinder_temest_plugin + volume tests | 14:14 |
| *** yangyapeng has quit IRC | 14:15 | |
| *** dave-mccowan has joined #openstack-cinder | 14:16 | |
| *** markvoelker has quit IRC | 14:18 | |
| *** edmondsw has joined #openstack-cinder | 14:21 | |
| *** moshele has quit IRC | 14:22 | |
| *** evin has quit IRC | 14:24 | |
| *** edmondsw_ has joined #openstack-cinder | 14:24 | |
| *** psachin has quit IRC | 14:24 | |
| *** lkuchlan has quit IRC | 14:24 | |
| *** pchavva has joined #openstack-cinder | 14:24 | |
| *** edmondsw has quit IRC | 14:28 | |
| *** lkuchlan has joined #openstack-cinder | 14:33 | |
| *** zhaochao has quit IRC | 14:40 | |
| *** mchlumsky has joined #openstack-cinder | 14:42 | |
| *** kholkina has quit IRC | 14:43 | |
| *** kholkina has joined #openstack-cinder | 14:43 | |
| *** mchlumsky has quit IRC | 14:44 | |
| *** mchlumsky has joined #openstack-cinder | 14:45 | |
| *** mchlumsky has quit IRC | 14:45 | |
| *** edmondsw_ is now known as edmondsw | 14:45 | |
| *** abhishekk has quit IRC | 14:46 | |
| openstackgerrit | Zhao Chao proposed openstack/cinder master: Remove unused '_execute' variable in ceph backup driver. https://review.openstack.org/532839 | 14:47 |
| smcginnis | eharney: We haven't merged the removal of the in-tree tempest tests yet from openstack/cinder. So there have been a few test updates slip by that I have had to cherry-pick over to the new plugin repo. | 14:48 |
| smcginnis | eharney: So my only concern on that one is if we rename the paths now, it will cause any further migrations to be a little more difficult. | 14:48 |
| smcginnis | eharney: Not a huge deal though. | 14:48 |
| smcginnis | eharney: I would like to see that removal patch go through though. | 14:49 |
| eharney | smcginnis: well, we can't remove the tests from Cinder until the plugin is up and running (and we are close to turning it on in some jobs) -- do we know how close we are to that? | 14:49 |
| smcginnis | eharney: Sure we can. The in-tree tempest tests were only ever picked up by a few third party CI's as far as I saw. So any additions now are just bonus. | 14:50 |
| smcginnis | eharney: I do have the LIO patch out there. | 14:50 |
| smcginnis | https://review.openstack.org/#/c/528511/ | 14:51 |
| eharney | hmm | 14:52 |
| eharney | i need to create a similar patch for the Ceph job which is running these tests now | 14:52 |
| eharney | what's the plan as far as requiring third-party CIs to run the plugin? | 14:53 |
| *** moshele has joined #openstack-cinder | 14:54 | |
| *** mchlumsky has joined #openstack-cinder | 14:55 | |
| *** moshele has quit IRC | 14:56 | |
| openstackgerrit | Steve Noyes proposed openstack/cinder master: Improve multiattach error message https://review.openstack.org/531877 | 14:57 |
| *** bkopilov has quit IRC | 14:58 | |
| *** eharney has quit IRC | 15:00 | |
| smcginnis | Well, we had stated in the past that all third party CIs should run the plugin when we added the in-tree tests, but I am only aware of a small handfull that ever added the all_plugin tox target. | 15:00 |
| *** yangyapeng has joined #openstack-cinder | 15:00 | |
| *** gouthamr has joined #openstack-cinder | 15:01 | |
| *** gman-tx has joined #openstack-cinder | 15:02 | |
| *** markvoelker has joined #openstack-cinder | 15:02 | |
| *** markvoelker has quit IRC | 15:02 | |
| *** mriedem has joined #openstack-cinder | 15:02 | |
| *** eharney has joined #openstack-cinder | 15:03 | |
| openstackgerrit | John Griffith proposed openstack/cinder master: Deprecate the allow-multiattach option to create https://review.openstack.org/531570 | 15:03 |
| *** Apoorva has joined #openstack-cinder | 15:12 | |
| *** edmondsw has quit IRC | 15:18 | |
| *** markvoelker has joined #openstack-cinder | 15:24 | |
| *** crose has joined #openstack-cinder | 15:24 | |
| *** edmondsw has joined #openstack-cinder | 15:24 | |
| openstackgerrit | Jeremy Zhang proposed openstack/python-cinderclient master: Add api_version wraps for generic volume groups https://review.openstack.org/532858 | 15:25 |
| *** rmcall has joined #openstack-cinder | 15:26 | |
| *** alexchad_ has joined #openstack-cinder | 15:28 | |
| openstackgerrit | Lee Yarwood proposed openstack/cinder master: WIP rbd: Allow encrypted volumes https://review.openstack.org/527979 | 15:30 |
| *** alexchadin has quit IRC | 15:31 | |
| *** yangyapeng has quit IRC | 15:33 | |
| *** itlinux has joined #openstack-cinder | 15:37 | |
| *** felipemonteiro has joined #openstack-cinder | 15:37 | |
| *** alexchad_ has quit IRC | 15:37 | |
| *** links has quit IRC | 15:38 | |
| *** felipemonteiro_ has joined #openstack-cinder | 15:38 | |
| *** yangyapeng has joined #openstack-cinder | 15:38 | |
| *** felipemonteiro has quit IRC | 15:43 | |
| *** gman-tx has quit IRC | 15:45 | |
| *** itlinux has quit IRC | 15:47 | |
| *** kholkina has quit IRC | 15:50 | |
| *** kholkina has joined #openstack-cinder | 15:51 | |
| *** bkopilov has joined #openstack-cinder | 15:51 | |
| *** alexchadin has joined #openstack-cinder | 15:53 | |
| *** kholkina has quit IRC | 15:55 | |
| *** ykarel is now known as ykarel|away | 15:57 | |
| *** markstur has joined #openstack-cinder | 16:01 | |
| *** armaan has quit IRC | 16:04 | |
| *** ykarel|away has quit IRC | 16:05 | |
| *** sbra has joined #openstack-cinder | 16:05 | |
| *** itlinux has joined #openstack-cinder | 16:08 | |
| *** armaan has joined #openstack-cinder | 16:17 | |
| *** armaan has quit IRC | 16:21 | |
| *** e0ne has quit IRC | 16:24 | |
| *** alexchadin has quit IRC | 16:24 | |
| *** alexchadin has joined #openstack-cinder | 16:25 | |
| *** alexchadin has quit IRC | 16:29 | |
| openstackgerrit | Unana Okpoyo proposed openstack/cinder master: VMAX driver - QOS settings incorrectly applied on Default storage group https://review.openstack.org/532911 | 16:49 |
| openstackgerrit | Tom Swanson proposed openstack/cinder master: Dell EMC SC: On None connector terminate_connection unmaps all https://review.openstack.org/531231 | 16:53 |
| *** AlexeyAbashkin has quit IRC | 16:53 | |
| openstackgerrit | Unana Okpoyo proposed openstack/cinder master: VMAX driver - QOS settings incorrectly applied on Default storage group https://review.openstack.org/532911 | 16:55 |
| *** armaan has joined #openstack-cinder | 16:55 | |
| *** Guest69243 is now known as geguileo | 16:57 | |
| *** geguileo is now known as Guest65830 | 16:57 | |
| *** pcaruana|afk| has quit IRC | 16:57 | |
| *** felipemonteiro_ has quit IRC | 16:58 | |
| Swanson | 350 jobs in the old zuul queue today. Going to be stone cold dead before that crap finishes. | 17:06 |
| *** lkuchlan has quit IRC | 17:11 | |
| *** lhx_ has quit IRC | 17:11 | |
| *** links has joined #openstack-cinder | 17:17 | |
| *** links has quit IRC | 17:17 | |
| *** elmaciej has quit IRC | 17:20 | |
| openstackgerrit | John Griffith proposed openstack/cinder master: Add multiattach policy https://review.openstack.org/531569 | 17:21 |
| jgriffith | mriedem: ^^ added in bootable policy | 17:21 |
| mriedem | ack | 17:22 |
| mriedem | jgriffith: how about a release note? | 17:22 |
| jungleboyj | mriedem: ++ | 17:22 |
| jgriffith | mriedem: yeah, I had that in the last enable patch actually | 17:22 |
| jgriffith | but I can/should break those out | 17:23 |
| jgriffith | working on cleaning the dependency chain now | 17:23 |
| *** chhavi__ has quit IRC | 17:26 | |
| *** Apoorva has quit IRC | 17:28 | |
| *** ntpttr_laptop has joined #openstack-cinder | 17:32 | |
| openstackgerrit | John Griffith proposed openstack/cinder master: Add multiattach policy https://review.openstack.org/531569 | 17:36 |
| *** gkadam has quit IRC | 17:37 | |
| openstackgerrit | John Griffith proposed openstack/cinder master: Enable multiattach capability https://review.openstack.org/532280 | 17:44 |
| *** KeithMnemonic has joined #openstack-cinder | 17:44 | |
| -openstackstatus- NOTICE: Due to an unexpected issue with zuulv3.o.o, we were not able to preserve running jobs for a restart. As a result, you'll need to recheck your previous patchsets | 17:46 | |
| *** Apoorva has joined #openstack-cinder | 17:53 | |
| *** jistr is now known as jistr|afk | 17:55 | |
| *** liverpooler has quit IRC | 17:59 | |
| *** jdillaman has quit IRC | 17:59 | |
| *** nicolasbock has joined #openstack-cinder | 18:00 | |
| *** Guest65830 is now known as geguileo | 18:00 | |
| *** liverpooler has joined #openstack-cinder | 18:02 | |
| *** raunak has joined #openstack-cinder | 18:04 | |
| *** crose has quit IRC | 18:07 | |
| *** ntpttr_laptop has quit IRC | 18:11 | |
| *** crose has joined #openstack-cinder | 18:13 | |
| *** jdillaman has joined #openstack-cinder | 18:16 | |
| cburgess | @jgriffith Quick question about cinder + SF specific behaviour. If I have a volume on a SolidFire and via the SolidFire UI I see in "attributes" that the volume has a "migrate_uuid" in its list of attributes, does that mean the volume was previously re-typed or migrated? | 18:20 |
| *** AlexeyAbashkin has joined #openstack-cinder | 18:20 | |
| jgriffith | cburgess: yes, that's exactly what it means | 18:21 |
| *** rmcall has quit IRC | 18:21 | |
| cburgess | jgriffith OK, so that migration_uuid appears in the cinder volume table as well. Is that the original volume record from before the retype/migration or is that a temporary record that gets created to for tracking purposes during the migration? | 18:22 |
| jgriffith | That's the temporary value that was used before the "magic switch-aroo" | 18:22 |
| jgriffith | I added that prior to the entry in the cinder db becuase it was absolutely maddening not having any history tracking of that | 18:23 |
| cburgess | OK so the actual UUID of the volume doesn't change. We just create a new record with a new UUID to hold temporary values during the migration and then update the original UUID record? | 18:23 |
| jgriffith | cburgess: si | 18:23 |
| cburgess | OK.. finally question... | 18:23 |
| cburgess | Final even | 18:23 |
| jgriffith | cburgess: migration creates a new volume, migrates the data, then swaps uuid's and deletes the temp record | 18:23 |
| jgriffith | I prefer *finally* questions when possible :) | 18:24 |
| cburgess | So a volume basicly has "old state" then we start a migration and create "migration UUID" then at the end we update the old UUID with the new info. Does the migration UUID hold the new info or the old info in the case of the migration? | 18:24 |
| cburgess | LOL | 18:24 |
| cburgess | As in if I'm looking at the deleted migration UUID is that that the connection info of the original volume, or that is the new connection info that we copied into the old UUID? | 18:24 |
| *** AlexeyAbashkin has quit IRC | 18:24 | |
| jgriffith | cburgess: hmm... you saved the hardest question for last :) | 18:25 |
| jgriffith | cburgess: I'm not sure I completely grock what you're asking | 18:25 |
| jgriffith | cburgess: we do a model/provider update if that's what you're getting at | 18:26 |
| jgriffith | cburgess: honestly my advice is don't do migrations, it'll just make your head hurt :) | 18:27 |
| cburgess | jgriffith I didn't.. the client did.. and yes now my head hurts. | 18:27 |
| jgriffith | :( Yeah, sorry about that | 18:27 |
| cburgess | jgriffith So what I mean is.. if I do a select from the DB on the migration UUID and I look at say... the provider_location field in the migration UUID record. Is that the original provider_location from before the migration? Or is that the new one? | 18:27 |
| jgriffith | cburgess: that should be the original | 18:28 |
| cburgess | jgriffith Oh this isn't your fault. This a series of the client doing things we told them not to.. it breaking.. us fixing (but not correctly because we didn't fully understand) and now we are in an end-state that I'm trying to re-construct what happened. | 18:28 |
| jgriffith | cburgess: ie the id that was actually used | 18:28 |
| jgriffith | cburgess: it kinda is my fault; I gave that feature a +2/A against my own better judegment and it's haunted me ever since :) | 18:29 |
| jgriffith | cburgess: FWIW I've dealt with a number of customers trying to get that whole thing sorted/mapped out | 18:29 |
| jgriffith | anyhow... so yeah; if you do the select on that migration-uuid in the db and get the deleted volume's provider info that will be the tgt-iqn info that was used for the migration process | 18:30 |
| cburgess | OK so the migration_uuid can be used to figure out what was there before and construct a history then. Thanks | 18:30 |
| jgriffith | cburgess:yeah | 18:30 |
| cburgess | jgriffith Oh for the migration process.. | 18:30 |
| cburgess | So.. that is the new one then on the new dest? | 18:30 |
| jgriffith | cburgess: it's hard to reconstruct all fo this | 18:30 |
| jgriffith | cburgess: well it's what I call that "temporary" volume that was used before swapping UUID's | 18:31 |
| cburgess | Yeah I see that.. just trying to figure it out as best I can. | 18:31 |
| cburgess | OK. | 18:31 |
| cburgess | So that volume should not exist on the SF anymore then? | 18:31 |
| cburgess | Which it does not so that makes sense. | 18:31 |
| cburgess | OK. | 18:31 |
| jgriffith | That's correct; on SF you should only see the new volume-name | 18:31 |
| jgriffith | thus the migration entry in the attributes to leave a bread-crumb | 18:31 |
| *** mvk has quit IRC | 18:32 | |
| cburgess | Cool | 18:32 |
| cburgess | OK I think I understand now. | 18:32 |
| jgriffith | cburgess: great; let me know if you get stuck or see something wonky | 18:32 |
| jgriffith | happy to help if I can | 18:33 |
| cburgess | yeah well do.. like I said some of this is us previously doing things wrong.. so just trying to figure that out. | 18:33 |
| jgriffith | no worries; it's kind of a tricky one IMHO | 18:34 |
| openstackgerrit | John Griffith proposed openstack/cinder master: Enable multiattach capability https://review.openstack.org/532280 | 18:39 |
| ildikov | jgriffith: are you certain that 3.49 will land before 3.50? | 18:41 |
| jgriffith | ildikov: nope | 18:41 |
| jgriffith | ildikov: but I'm not certain 3.50 will land before 3.49 either | 18:41 |
| jgriffith | :) | 18:41 |
| ildikov | as in my understanding we need 3.50 in Queens and can do 3.49 at the beginning of Rocky | 18:42 |
| ildikov | jgriffith: fair enough :) | 18:42 |
| jgriffith | I disagree, but ok | 18:42 |
| Swanson | Big cat just barfed. Sounded like a blackhawk going down. | 18:42 |
| jgriffith | I think you need both | 18:42 |
| jgriffith | but I'll go change it and start mv chasing game | 18:42 |
| jgriffith | it's much easier to do now thanks to smcginnis 's variable file thingy majigy | 18:43 |
| ildikov | both works too, but if you modify the mocroversion then I think it should either be a chain of patches or have the same microversion in both and then update the losing one when you need to rebase | 18:43 |
| jungleboyj | Yeah, that has been a good improvement. Glad we did that. | 18:43 |
| openstackgerrit | John Griffith proposed openstack/cinder master: Enable multiattach capability https://review.openstack.org/532280 | 18:44 |
| jgriffith | ildikov: ^^ there ya go | 18:44 |
| *** tesseract has quit IRC | 18:45 | |
| ildikov | jgriffith: <3 | 18:45 |
| ildikov | jgriffith: thank you :) | 18:45 |
| jgriffith | sure, just make it end :) | 18:45 |
| jgriffith | of course who knows if all works any more :) | 18:45 |
| ildikov | make what end exactly? :) | 18:45 |
| jgriffith | feeling lucky? | 18:46 |
| jgriffith | ildikov: haha... make the multiattach-challenge end :) | 18:46 |
| jgriffith | time to move on to bigger/better things | 18:46 |
| ildikov | no bigger thing than that :) | 18:46 |
| ildikov | and it won't solve the microversion headache | 18:47 |
| ildikov | but, yes, believe me, I would really love to move on now | 18:47 |
| ildikov | and if feeling lucky helps, then I will try my best :) | 18:48 |
| jgriffith | I would appreciate some thoughts/input from people on these two: | 18:51 |
| jgriffith | https://review.openstack.org/#/c/532645/1 | 18:51 |
| jgriffith | ^^ That ones fairly straight forward | 18:51 |
| jgriffith | https://review.openstack.org/#/c/532702/1 | 18:52 |
| jgriffith | ^^ is possibly contentious | 18:52 |
| jgriffith | because by eliminating the bug and eliminating the error condition I may "change" behavior for somebody | 18:52 |
| jgriffith | but I think that's the right thing to do personally | 18:52 |
| jgriffith | some day I'll figure out the mv stuff in the client shell :) | 18:53 |
| jgriffith | on that note, BBIAB | 18:53 |
| *** felipemonteiro has joined #openstack-cinder | 18:55 | |
| ildikov | I'm back in an ~hour and will check | 18:56 |
| ildikov | and yes, client patches are needed too :) | 18:56 |
| *** felipemonteiro_ has joined #openstack-cinder | 18:56 | |
| *** AlexeyAbashkin has joined #openstack-cinder | 18:57 | |
| *** hemna_ has joined #openstack-cinder | 18:59 | |
| *** felipemonteiro has quit IRC | 19:00 | |
| *** AlexeyAbashkin has quit IRC | 19:01 | |
| *** raunak has quit IRC | 19:07 | |
| openstackgerrit | John Griffith proposed openstack/cinder-specs master: Update the multiattach spec https://review.openstack.org/532944 | 19:11 |
| *** liverpooler has quit IRC | 19:15 | |
| *** gkadam has joined #openstack-cinder | 19:17 | |
| *** itlinux has quit IRC | 19:26 | |
| *** raunak has joined #openstack-cinder | 19:28 | |
| *** alexchadin has joined #openstack-cinder | 19:29 | |
| *** liverpooler has joined #openstack-cinder | 19:38 | |
| eharney | smcginnis: so i'm trying to figure out how we should merge these driverfixes/mitaka fixes... | 19:39 |
| eharney | it's kind of a mess since we can't merge them individually without toggling the job to non-voting | 19:40 |
| eharney | maybe we should just do that temporarily | 19:41 |
| *** eharney has quit IRC | 19:42 | |
| *** harlowja has joined #openstack-cinder | 19:43 | |
| *** eharney has joined #openstack-cinder | 19:44 | |
| *** edmondsw_ has joined #openstack-cinder | 19:48 | |
| *** edmonds__ has joined #openstack-cinder | 19:48 | |
| *** alexchadin has quit IRC | 19:51 | |
| *** edmondsw has quit IRC | 19:52 | |
| *** edmondsw_ has quit IRC | 19:52 | |
| *** itlinux has joined #openstack-cinder | 19:54 | |
| *** itlinux has quit IRC | 19:55 | |
| *** jistr|afk is now known as jistr | 19:57 | |
| *** Apoorva has quit IRC | 20:01 | |
| *** armaan has quit IRC | 20:02 | |
| *** armaan has joined #openstack-cinder | 20:03 | |
| smcginnis | eharney: I was thinking either that, or just take the changes from each separate patch and just squash them into one. | 20:04 |
| smcginnis | eharney: Just add some "Co-authored by:" tags in the commit to be fair. | 20:04 |
| eharney | smcginnis: squashing them into one really makes a mess for anyone wanting to cherry-pick things from this tree though, which is a lot of its goal... so i'm leaning toward the non-voting idea | 20:05 |
| smcginnis | Then we can abandon the individual changes. Cuz yeah, we'll need all of them to get anything to pass unless we flip the job. | 20:05 |
| smcginnis | eharney: You think someone would want to cherry pick just some of it and not all? | 20:05 |
| eharney | well, that's how i normally manage these things | 20:06 |
| *** mvk has joined #openstack-cinder | 20:08 | |
| eharney | otherwise you have to manage a lot of unrelated changes when trying to backport a specific fix | 20:08 |
| smcginnis | Fair enough. Want to propose a patch to toggle that? | 20:08 |
| eharney | yeah, i just have to figure out how to do it again | 20:09 |
| smcginnis | eharney: Or we could proceed with your revert of those patches and have everyone resubmit with these new updates included. | 20:10 |
| *** gkadam has quit IRC | 20:12 | |
| patrickeast | speaking of the driver fixes... did anything ever happen with the test_matrix.py thing not liking that branch name? | 20:13 |
| patrickeast | iirc there was a fix for it but i'm not seeing it in devstack-gate | 20:14 |
| smcginnis | test_matrix.py? | 20:14 |
| eharney | i think i was looking at that once, but ended up not needing to change it | 20:14 |
| patrickeast | smcginnis: for this type of error http://paste.openstack.org/show/643053/ | 20:14 |
| smcginnis | patrickeast: Is that a CI thing? | 20:15 |
| patrickeast | yea | 20:15 |
| patrickeast | its how devstack-gate figures out what things to install and configure | 20:15 |
| patrickeast | https://github.com/openstack-infra/devstack-gate/blob/master/roles/test-matrix/library/test_matrix.py | 20:16 |
| *** crose has quit IRC | 20:16 | |
| smcginnis | patrickeast: Why would CI be running on driverfixes branches? Or it sees the branch name on normal branches and pukes? | 20:17 |
| *** mchlumsky has quit IRC | 20:17 | |
| patrickeast | i mean, i can just turn it off | 20:17 |
| patrickeast | but if like i was going to back-port some pure change i'd run my ci on it | 20:17 |
| smcginnis | patrickeast: Please don't. | 20:17 |
| patrickeast | then again if its fubar i can just not | 20:17 |
| smcginnis | We were going to intentionally disable driverfixes code from actually being able to run, so cut the noise. | 20:18 |
| patrickeast | sure, easy fix | 20:18 |
| patrickeast | i like less work | 20:18 |
| smcginnis | ;) | 20:18 |
| eharney | smcginnis: similar to the discussion the other day with the irrelevant-files stuff... i'm not sure how to make it a non-voting job now that the unit tests are provided by a template instead of a cinder-specific job | 20:23 |
| eharney | hrmm | 20:24 |
| smcginnis | eharney: I really don't like that "feature". | 20:24 |
| eharney | maybe you can just override the voting setting anyway | 20:24 |
| smcginnis | Yeah, actually I think voting applies, just not irrelevant-files. | 20:24 |
| smcginnis | Because voting is something at the end, but job selection is done up front, so once it's selected it's too late. | 20:25 |
| eharney | smcginnis: actually, ajaeger said it can't be done here the last time: https://review.openstack.org/#/c/498868/1/zuul/layout.yaml | 20:26 |
| eharney | so i guess we can do the messy approach | 20:26 |
| smcginnis | pewp | 20:28 |
| eharney | or just disable the job for that branch and define our own in .zuul.yaml | 20:28 |
| eharney | so many fun options :) | 20:28 |
| smcginnis | Once we get past this, we should be in good shape going forward. I'd rather not have to carry our own job definition. | 20:29 |
| eharney | yeah, i'll see if squashing everything sounds good again after i fix the last 2 failures | 20:30 |
| *** elmaciej has joined #openstack-cinder | 20:30 | |
| *** pchavva has quit IRC | 20:31 | |
| *** imacdonn has quit IRC | 20:33 | |
| *** imacdonn has joined #openstack-cinder | 20:33 | |
| *** eharney has quit IRC | 20:41 | |
| *** eharney has joined #openstack-cinder | 20:51 | |
| *** eharney has quit IRC | 20:52 | |
| *** e0ne has joined #openstack-cinder | 20:55 | |
| *** bigdogstl has joined #openstack-cinder | 20:56 | |
| *** Apoorva has joined #openstack-cinder | 21:02 | |
| *** david-lyle has quit IRC | 21:03 | |
| *** david-lyle has joined #openstack-cinder | 21:04 | |
| *** Apoorva has quit IRC | 21:06 | |
| *** gouthamr has quit IRC | 21:07 | |
| *** edmonds__ is now known as edmondsw | 21:07 | |
| *** e0ne has quit IRC | 21:09 | |
| *** liverpooler has quit IRC | 21:10 | |
| *** masber has joined #openstack-cinder | 21:10 | |
| *** gouthamr has joined #openstack-cinder | 21:15 | |
| *** bigdogstl has quit IRC | 21:15 | |
| *** raunak has left #openstack-cinder | 21:17 | |
| *** e0ne has joined #openstack-cinder | 21:32 | |
| *** sbra has quit IRC | 21:38 | |
| ildikov | jgriffith: as for which microversion patch needs to land, I forgot about the bug for setting the mode so plz ignore me on that one | 21:42 |
| *** bigdogstl has joined #openstack-cinder | 21:45 | |
| *** eharney has joined #openstack-cinder | 21:46 | |
| *** rcernin has joined #openstack-cinder | 21:46 | |
| *** eharney has quit IRC | 21:48 | |
| *** bigdogstl has quit IRC | 21:53 | |
| *** e0ne has quit IRC | 21:57 | |
| *** threestrands has joined #openstack-cinder | 21:58 | |
| *** threestrands has quit IRC | 21:58 | |
| *** threestrands has joined #openstack-cinder | 21:58 | |
| *** threestrands has quit IRC | 21:59 | |
| *** threestrands has joined #openstack-cinder | 21:59 | |
| *** threestrands has quit IRC | 21:59 | |
| *** threestrands has joined #openstack-cinder | 21:59 | |
| *** dustins has quit IRC | 22:00 | |
| *** threestrands has quit IRC | 22:00 | |
| *** KeithMnemonic has quit IRC | 22:00 | |
| *** threestrands has joined #openstack-cinder | 22:01 | |
| *** threestrands has quit IRC | 22:02 | |
| *** threestrands has joined #openstack-cinder | 22:02 | |
| *** Apoorva has joined #openstack-cinder | 22:03 | |
| jgriffith | ildikov: no worries; I probably won't be doing much with any of those patches for a while | 22:05 |
| jgriffith | ildikov: so everybody can feel free to pile on with their recommendations, changes, and redesigns :) | 22:06 |
| ildikov | jgriffith: do you mean the set rw/ro mode or all of them? | 22:06 |
| ildikov | jgriffith: and for a how long while exactly? | 22:06 |
| jgriffith | Not sure, might be able to find some time tomorrow but not much. Not sure what next week looks like; and I'm not wasting my week-end on it :) | 22:08 |
| *** abishop has quit IRC | 22:08 | |
| *** tpsilva has quit IRC | 22:09 | |
| *** nicolasbock has quit IRC | 22:10 | |
| *** ktibi has quit IRC | 22:12 | |
| *** jgriffith has left #openstack-cinder | 22:12 | |
| *** bigdogstl has joined #openstack-cinder | 22:12 | |
| *** jgriffith has joined #openstack-cinder | 22:13 | |
| jgriffith | oops | 22:13 |
| *** dave-mccowan has quit IRC | 22:16 | |
| *** dave-mccowan has joined #openstack-cinder | 22:17 | |
| ildikov | jgriffith: pretty please, sugar on top, not the weekend part but tomorrow and next week :) | 22:22 |
| *** Apoorva_ has joined #openstack-cinder | 22:23 | |
| smcginnis | ildikov: You need to fly back to Denver and feed jgriffith coffee until he finished the work or passes out from a caffiene overdose. | 22:25 |
| *** dave-mccowan has quit IRC | 22:25 | |
| *** Apoorva has quit IRC | 22:26 | |
| *** bigdogstl has quit IRC | 22:26 | |
| *** bigdogstl has joined #openstack-cinder | 22:26 | |
| ildikov | smcginnis: I found a good brand for him while I was there | 22:26 |
| smcginnis | ildikov: Which one? | 22:27 |
| ildikov | smcginnis: but well, I liked Denver and the area, so I'm always happy to find an excuse to go back | 22:27 |
| smcginnis | Me too. | 22:27 |
| ildikov | smcginnis: not sure how much jgriffith wants to see me ever again after all this multi-attach torture though... :/ | 22:27 |
| ildikov | smcginnis: Kicking Horse | 22:28 |
| smcginnis | ildikov: That sounds like an appropriate one. | 22:29 |
| smcginnis | ildikov: Alternate between that and the Richardo's liquer and the patches will be done in no time. | 22:29 |
| ildikov | smcginnis: that was my first thought as well :) | 22:29 |
| ildikov | smcginnis: I don't see why we need to alternate :) | 22:29 |
| *** Apoorva_ has quit IRC | 22:30 | |
| *** itlinux has joined #openstack-cinder | 22:30 | |
| smcginnis | You'll have a shorter window if you just stick with one or the other, but the two would balance things out for awhile. ;) | 22:30 |
| *** Apoorva has joined #openstack-cinder | 22:31 | |
| ildikov | smcginnis: makes sense, we can just make a cocktail out of them when we're close enough to the finish line :) | 22:35 |
| *** sandk has joined #openstack-cinder | 22:48 | |
| *** markvoelker has quit IRC | 22:49 | |
| *** markvoelker has joined #openstack-cinder | 22:49 | |
| *** sandk has quit IRC | 22:51 | |
| *** sandk has joined #openstack-cinder | 22:51 | |
| *** markvoelker has quit IRC | 22:54 | |
| *** edmondsw has quit IRC | 22:54 | |
| *** edmondsw has joined #openstack-cinder | 22:55 | |
| *** sandk has quit IRC | 22:55 | |
| *** felipemonteiro_ has quit IRC | 22:55 | |
| *** itlinux has quit IRC | 22:55 | |
| *** sandk has joined #openstack-cinder | 22:56 | |
| *** markvoelker has joined #openstack-cinder | 22:56 | |
| *** itlinux has joined #openstack-cinder | 22:57 | |
| *** edmondsw has quit IRC | 22:59 | |
| *** markvoelker has quit IRC | 22:59 | |
| *** markvoelker has joined #openstack-cinder | 23:00 | |
| *** markvoelker has quit IRC | 23:05 | |
| *** itlinux has quit IRC | 23:12 | |
| jgriffith | ildikov: mriedem ok, I've updated my opinions and responses on https://review.openstack.org/#/c/531569 when you have a chance | 23:17 |
| jgriffith | I'll get the obvious ones that I agree with fixed up and submitted shortly but wanted to give every one a chance to pile some more comments on there | 23:18 |
| *** bigdogstl has quit IRC | 23:19 | |
| *** chhavi__ has joined #openstack-cinder | 23:22 | |
| jgriffith | smcginnis: not too mention I'd get ADD and constantly context switch between the two... then I'd just chase my tail in circles :) | 23:23 |
| jungleboyj | Oh the mental image of jgriffith chasing his tail just made a rough day slightly more bearable. | 23:25 |
| ildikov | jgriffith: I can give my old bartending skills a try so you don't need context switching at least :) | 23:25 |
| ildikov | jgriffith: no guarantee on the consequences though still :) | 23:25 |
| *** chhavi__ has quit IRC | 23:26 | |
| jgriffith | jungleboyj: https://i.chzbgr.com/full/7456715520/hEA32C279/ | 23:28 |
| jungleboyj | :-) That boor dog looks so ashamed. | 23:29 |
| jgriffith | sadly I know how he feels :) | 23:30 |
| jungleboyj | :-) | 23:30 |
| ildikov | hmm, I better not to ask follow up questions here... :) | 23:31 |
| *** bigdogstl has joined #openstack-cinder | 23:31 | |
| *** gnufied has quit IRC | 23:32 | |
| mriedem | jgriffith: tried to reply where i could | 23:32 |
| mriedem | i didn't know cinder automatically set bootable everywhere | 23:32 |
| mriedem | also left comments in https://review.openstack.org/#/c/531570/ | 23:32 |
| jgriffith | mriedem: yeah, there are several things that will auto-update that for us (or against us) :) | 23:32 |
| jgriffith | mriedem: excellent, thanks!! I appreciate it | 23:33 |
| mriedem | so are you guys planning on deprecating out the multiattach flag on volume create via a microversion where you just no longer accept that value, and stop showing it in GET /volumes responses? | 23:33 |
| mriedem | i guess you could show it in the volume details response based on the volume type associated with the volume | 23:34 |
| jgriffith | mriedem: oh, hmmm... yeah; so I looked at doing an MV bump on that and just moving on without it but got annoyed at the implementation scattered trhoughout everywhere (ie having to maintain it at all) | 23:34 |
| jgriffith | but I suppose that might just be the answer | 23:34 |
| jgriffith | mriedem: my plan was to deprecate and just remove it all together | 23:35 |
| mriedem | well, can't really just drop a request parameter out of the volume create API w/o a microversion | 23:35 |
| mriedem | i mean you can, but you shouldn't | 23:35 |
| jgriffith | mriedem: I know what you meant :) | 23:35 |
| jgriffith | you just don't understand my disdain for that option and the code associated with it | 23:36 |
| jgriffith | but you're correct | 23:36 |
| *** bigdogstl has quit IRC | 23:36 | |
| jgriffith | and IMHO I'd count that as a "critical bug" that MUST be fixed asap and the only way to fix it is to remove it :) | 23:36 |
| jgriffith | but seriously; I'll go the mv route and see how ugly things get. Maybe I just didn't see a clear path in there | 23:37 |
| ildikov | and the 'multiattach' field in the volume details would go too? | 23:37 |
| jgriffith | ildikov: ? | 23:38 |
| jgriffith | ildikov: you mean volume_ref.multiattach? | 23:38 |
| jgriffith | NO | 23:38 |
| jgriffith | I certainly wouldn't want that to go away | 23:38 |
| ildikov | jgriffith: yep, I thought that's what mriedem meant regarding the GET /volume response | 23:38 |
| mriedem | it can't be removed from the response othewrise nova is f'ed | 23:38 |
| jgriffith | Oh... wait, sorry; I just glossed over that and meant the post | 23:39 |
| jgriffith | POST | 23:39 |
| mriedem | deprecating it out of POST is fine | 23:39 |
| mriedem | since it could be based on the type | 23:39 |
| jgriffith | I have no desire to remove the attribute from the response | 23:39 |
| jgriffith | I'm relying on it in the new code | 23:39 |
| mriedem | ok | 23:39 |
| jgriffith | sorry... my bad | 23:39 |
| jgriffith | I shoud've read more carefully | 23:39 |
| openstackgerrit | Michael Latchmansingh proposed openstack/cinder master: Adds documentation for DataCore volume driver https://review.openstack.org/533011 | 23:40 |
| ildikov | mriedem: jgriffith: great, then we're on the same page, thanks for clarifying | 23:40 |
| *** bigdogstl has joined #openstack-cinder | 23:46 | |
| *** armaan has quit IRC | 23:53 | |
| ildikov | jgriffith: what's the disagreement regarding the meeting logs on the boot from volume case? | 23:54 |
| jgriffith | ildikov: sorry, stepped away... | 23:54 |
| jgriffith | so it's more just about implementation than aything else | 23:55 |
| jgriffith | I dont' understand why we shouldn't allow a bootable volume with the multiattach type; and just restrict attaching it more than once like we do today | 23:55 |
| jgriffith | it mostly just goes back to just easier/single point of control | 23:56 |
| jungleboyj | jgriffith: That is a good question. | 23:56 |
| jungleboyj | I feel like we discussed this before though. | 23:56 |
| jgriffith | the other thing is looking forward; maybe you want to change that rule? | 23:56 |
| ildikov | jgriffith: fair enough, I think I'm just taking things too literally regarding that one | 23:57 |
| jgriffith | I am also concerned because I seem to recall that some changes were made to the new attachment api's where we're back to doing the behind the scenes two attach thing | 23:57 |
| jgriffith | I believe it was part of shelve, or maybe live-migration (not sure) | 23:57 |
| ildikov | shelve | 23:57 |
| jgriffith | ok... | 23:58 |
| ildikov | because leaving the volume in 'reserved' state was breaking the API and it seemed easier to work around than adding a new Nova microversion | 23:58 |
| jgriffith | so at some point; when ReadOnly mode options land in attachment-create; I would see a valuable use-case/implementation in "hot standby" instance type of thing | 23:58 |
| ildikov | I added the topic to the PTG etherpad | 23:58 |
| jgriffith | in which case I'm going to want multiple attaches of a bootable volume | 23:59 |
| jgriffith | That may be 6 months away, but no harm in planning | 23:59 |
| jgriffith | The other thing is; the other approach just doesn't make sense to me at all | 23:59 |
| ildikov | yeah, I completely agree | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!