*** dave-mccowan has joined #openstack-cinder | 00:05 | |
*** stvnoyes has quit IRC | 00:09 | |
*** e0ne has quit IRC | 00:14 | |
*** diablo_rojo has quit IRC | 00:19 | |
*** wanghao_ has quit IRC | 00:20 | |
*** wanghao has joined #openstack-cinder | 00:21 | |
*** mriedem has quit IRC | 00:29 | |
*** edmondsw has joined #openstack-cinder | 00:31 | |
*** diablo_rojo has joined #openstack-cinder | 00:34 | |
*** edmondsw has quit IRC | 00:36 | |
*** zhaochao has joined #openstack-cinder | 01:02 | |
*** markvoelker has quit IRC | 01:08 | |
*** catintheroof has quit IRC | 01:10 | |
*** catintheroof has joined #openstack-cinder | 01:11 | |
*** liverpooler has joined #openstack-cinder | 01:13 | |
*** gkadam has quit IRC | 01:15 | |
*** catintheroof has quit IRC | 01:15 | |
*** gkadam has joined #openstack-cinder | 01:16 | |
*** Apoorva_ has joined #openstack-cinder | 01:27 | |
*** Apoorva has quit IRC | 01:30 | |
*** Apoorva_ has quit IRC | 01:32 | |
*** imran_ansari has joined #openstack-cinder | 01:33 | |
*** dave-mccowan has quit IRC | 01:34 | |
*** liverpooler has quit IRC | 01:42 | |
*** namnh has joined #openstack-cinder | 01:46 | |
*** tonyb has quit IRC | 01:47 | |
*** tonyb has joined #openstack-cinder | 01:49 | |
*** yangyapeng has quit IRC | 01:51 | |
*** yangyapeng has joined #openstack-cinder | 01:52 | |
*** dave-mccowan has joined #openstack-cinder | 01:55 | |
*** gkadam has quit IRC | 01:56 | |
*** gkadam has joined #openstack-cinder | 01:59 | |
*** markvoelker has joined #openstack-cinder | 02:05 | |
*** Brin has joined #openstack-cinder | 02:10 | |
*** zhaochao has quit IRC | 02:16 | |
*** dave-mccowan has quit IRC | 02:18 | |
*** lhx_ has joined #openstack-cinder | 02:19 | |
*** dave-mccowan has joined #openstack-cinder | 02:19 | |
*** edmondsw has joined #openstack-cinder | 02:19 | |
openstackgerrit | Merged openstack/cinder master: V2/V3 jsonschema validation: snapshots https://review.openstack.org/520991 | 02:20 |
---|---|---|
*** dalgaaf has quit IRC | 02:22 | |
*** edmondsw has quit IRC | 02:24 | |
*** dalgaaf has joined #openstack-cinder | 02:24 | |
*** gouthamr has quit IRC | 02:27 | |
*** zhaochao has joined #openstack-cinder | 02:28 | |
*** imran_ansari has quit IRC | 02:30 | |
jungleboyj | Swanson: Yes, VMAX needs attention please. | 02:32 |
jungleboyj | Swanson: Whas is vacation? | 02:34 |
*** diablo_rojo has quit IRC | 02:36 | |
*** owalsh_ has joined #openstack-cinder | 02:42 | |
*** owalsh has quit IRC | 02:45 | |
openstackgerrit | Jeremy Zhang proposed openstack/cinder master: Fix api-ref for v3 volume types https://review.openstack.org/530830 | 02:55 |
*** gkadam has quit IRC | 03:07 | |
*** Apoorva has joined #openstack-cinder | 03:23 | |
*** Apoorva has quit IRC | 03:24 | |
*** Apoorva has joined #openstack-cinder | 03:25 | |
*** masber has quit IRC | 03:32 | |
*** abhishekk has joined #openstack-cinder | 03:43 | |
*** gcb has quit IRC | 03:44 | |
*** nicolasbock has quit IRC | 03:44 | |
*** lhx__ has joined #openstack-cinder | 03:47 | |
*** lhx_ has quit IRC | 03:47 | |
*** _ix_ has quit IRC | 03:56 | |
*** _ix has joined #openstack-cinder | 04:03 | |
*** diablo_rojo has joined #openstack-cinder | 04:07 | |
*** edmondsw has joined #openstack-cinder | 04:07 | |
*** _ix_ has joined #openstack-cinder | 04:08 | |
*** _ix has quit IRC | 04:09 | |
*** edmondsw has quit IRC | 04:12 | |
*** gmann has joined #openstack-cinder | 04:19 | |
*** Apoorva has quit IRC | 04:20 | |
*** masber has joined #openstack-cinder | 04:21 | |
*** Apoorva has joined #openstack-cinder | 04:21 | |
openstackgerrit | Felipe Monteiro proposed openstack/cinder master: Fix reserve volume enforcing the wrong policy action https://review.openstack.org/530837 | 04:25 |
*** Apoorva has quit IRC | 04:25 | |
*** hemna_ has quit IRC | 04:29 | |
*** Apoorva has joined #openstack-cinder | 04:30 | |
*** crushil_ has joined #openstack-cinder | 04:46 | |
*** links has joined #openstack-cinder | 04:48 | |
*** crushil_ has quit IRC | 04:51 | |
*** crushil_ has joined #openstack-cinder | 04:51 | |
*** lhx__ has quit IRC | 04:52 | |
*** lhx__ has joined #openstack-cinder | 04:53 | |
*** psachin has joined #openstack-cinder | 04:55 | |
*** links has quit IRC | 04:58 | |
*** chhavi has joined #openstack-cinder | 05:02 | |
*** links has joined #openstack-cinder | 05:03 | |
*** _ix has joined #openstack-cinder | 05:04 | |
*** dave-mccowan has quit IRC | 05:06 | |
*** _ix_ has quit IRC | 05:06 | |
*** pgadiya has joined #openstack-cinder | 05:09 | |
*** Apoorva has quit IRC | 05:14 | |
*** crushil_ has quit IRC | 05:25 | |
*** hoangcx has quit IRC | 05:30 | |
*** namnh has quit IRC | 05:30 | |
*** namnh has joined #openstack-cinder | 05:31 | |
*** hoangcx has joined #openstack-cinder | 05:31 | |
*** crushil_ has joined #openstack-cinder | 05:31 | |
*** crushil_ has quit IRC | 05:33 | |
*** daidv has quit IRC | 05:34 | |
*** crushil_ has joined #openstack-cinder | 05:34 | |
*** daidv has joined #openstack-cinder | 05:35 | |
*** lkuchlan has joined #openstack-cinder | 05:42 | |
*** gcb has joined #openstack-cinder | 05:42 | |
*** markstur has quit IRC | 05:44 | |
*** armaan has joined #openstack-cinder | 05:45 | |
*** daidv has quit IRC | 05:45 | |
*** daidv has joined #openstack-cinder | 05:46 | |
*** zhurong has joined #openstack-cinder | 05:47 | |
*** edmondsw has joined #openstack-cinder | 05:56 | |
*** edmondsw has quit IRC | 06:00 | |
openstackgerrit | wanghao proposed openstack/python-cinderclient master: Support for reporting backend state in service list https://review.openstack.org/530446 | 06:03 |
*** tasker has quit IRC | 06:08 | |
*** tasker has joined #openstack-cinder | 06:09 | |
*** masber has quit IRC | 06:13 | |
openstackgerrit | billsung proposed openstack/cinder master: Re-add Infortrend Cinder volume driver https://review.openstack.org/523659 | 06:24 |
*** markvoelker has quit IRC | 06:28 | |
*** diablo_rojo has quit IRC | 06:29 | |
*** tasker_tel has joined #openstack-cinder | 06:29 | |
*** armax has quit IRC | 06:30 | |
*** armax has joined #openstack-cinder | 06:30 | |
*** armax has joined #openstack-cinder | 06:32 | |
*** armax has quit IRC | 06:32 | |
*** crushil_ has quit IRC | 06:32 | |
*** tasker_tel has left #openstack-cinder | 06:32 | |
*** armax has joined #openstack-cinder | 06:33 | |
*** armax has quit IRC | 06:34 | |
openstackgerrit | Merged openstack/cinder master: Discrepancy in api-ref and code for create group type API https://review.openstack.org/524492 | 06:36 |
*** moshele has joined #openstack-cinder | 06:43 | |
*** pcaruana has joined #openstack-cinder | 07:09 | |
*** david-lyle has quit IRC | 07:12 | |
*** david-lyle has joined #openstack-cinder | 07:12 | |
*** rcernin has quit IRC | 07:25 | |
*** edmondsw has joined #openstack-cinder | 07:44 | |
*** edmondsw has quit IRC | 07:48 | |
*** lkuchlan has quit IRC | 07:51 | |
*** lkuchlan has joined #openstack-cinder | 07:52 | |
*** e0ne has joined #openstack-cinder | 07:57 | |
*** armaan has quit IRC | 08:04 | |
*** armaan has joined #openstack-cinder | 08:05 | |
openstackgerrit | yixuan zhang proposed openstack/cinder master: Storwize: add hyperswap volume support https://review.openstack.org/492414 | 08:05 |
*** tesseract has joined #openstack-cinder | 08:05 | |
*** chandankumar has joined #openstack-cinder | 08:07 | |
openstackgerrit | Jeremy Zhang proposed openstack/cinder master: RBD: get manageable volumes https://review.openstack.org/527903 | 08:09 |
*** medy has joined #openstack-cinder | 08:15 | |
*** medy has quit IRC | 08:16 | |
*** medy has joined #openstack-cinder | 08:16 | |
*** medy has quit IRC | 08:18 | |
*** medy has joined #openstack-cinder | 08:18 | |
*** medy has quit IRC | 08:19 | |
*** medy has joined #openstack-cinder | 08:19 | |
*** medy has quit IRC | 08:21 | |
*** medy has joined #openstack-cinder | 08:21 | |
*** medy has quit IRC | 08:22 | |
*** medy has joined #openstack-cinder | 08:22 | |
*** e0ne has quit IRC | 08:23 | |
*** medy has quit IRC | 08:24 | |
*** medy has joined #openstack-cinder | 08:24 | |
*** medy has quit IRC | 08:25 | |
*** medy has joined #openstack-cinder | 08:25 | |
*** medy has quit IRC | 08:27 | |
*** medy has joined #openstack-cinder | 08:27 | |
*** medy has quit IRC | 08:28 | |
*** medy has joined #openstack-cinder | 08:28 | |
*** markvoelker has joined #openstack-cinder | 08:29 | |
*** medy has quit IRC | 08:29 | |
*** gmann has quit IRC | 08:33 | |
*** moshele has quit IRC | 08:35 | |
*** moshele has joined #openstack-cinder | 08:37 | |
chhavi | Brin: Need some clarification on the review comments : https://review.openstack.org/#/c/518316/ | 08:44 |
chhavi | Brin: can you please elaborate. | 08:45 |
chhavi | Brin: do you foresee any issues making this change, | 08:45 |
*** armaan has quit IRC | 08:48 | |
*** armaan has joined #openstack-cinder | 08:48 | |
*** moshele has quit IRC | 08:50 | |
*** moshele has joined #openstack-cinder | 08:50 | |
*** arnewiebalck_ has joined #openstack-cinder | 08:54 | |
openstackgerrit | TommyLike proposed openstack/cinder-specs master: Support restore backup to volumes simultaneously https://review.openstack.org/524441 | 08:58 |
*** gmann has joined #openstack-cinder | 08:59 | |
*** markvoelker has quit IRC | 09:03 | |
*** erlon has joined #openstack-cinder | 09:20 | |
*** e0ne has joined #openstack-cinder | 09:21 | |
openstackgerrit | melissaml proposed openstack/os-brick master: Cleanup test-requirements https://review.openstack.org/530878 | 09:26 |
*** edmondsw has joined #openstack-cinder | 09:32 | |
*** wanghao has quit IRC | 09:33 | |
*** wanghao has joined #openstack-cinder | 09:33 | |
*** wanghao has quit IRC | 09:34 | |
*** wanghao has joined #openstack-cinder | 09:34 | |
*** wanghao has quit IRC | 09:35 | |
*** wanghao has joined #openstack-cinder | 09:35 | |
*** Brin has quit IRC | 09:35 | |
*** wanghao has quit IRC | 09:35 | |
*** wanghao has joined #openstack-cinder | 09:36 | |
openstackgerrit | Xiaojun Liao proposed openstack/cinder master: Fix v3 api-ref for showing API details url error https://review.openstack.org/522492 | 09:36 |
*** wanghao has quit IRC | 09:36 | |
*** wanghao has joined #openstack-cinder | 09:36 | |
*** edmondsw has quit IRC | 09:37 | |
*** wanghao has quit IRC | 09:37 | |
*** wanghao has joined #openstack-cinder | 09:37 | |
*** wanghao has quit IRC | 09:38 | |
*** wanghao has joined #openstack-cinder | 09:38 | |
*** erlon has quit IRC | 09:38 | |
*** wanghao has quit IRC | 09:39 | |
*** wanghao has joined #openstack-cinder | 09:39 | |
*** wanghao has quit IRC | 09:39 | |
*** wanghao has joined #openstack-cinder | 09:40 | |
*** wanghao has quit IRC | 09:40 | |
*** erlon has joined #openstack-cinder | 09:41 | |
*** wanghao has joined #openstack-cinder | 09:41 | |
*** markvoelker has joined #openstack-cinder | 10:01 | |
*** mvk has quit IRC | 10:06 | |
*** mvk has joined #openstack-cinder | 10:07 | |
*** pcaruana has quit IRC | 10:09 | |
*** pcaruana has joined #openstack-cinder | 10:10 | |
*** namnh has quit IRC | 10:11 | |
*** armaan has quit IRC | 10:12 | |
*** armaan has joined #openstack-cinder | 10:13 | |
*** owalsh has joined #openstack-cinder | 10:21 | |
*** owalsh_ has quit IRC | 10:22 | |
*** Trident has joined #openstack-cinder | 10:24 | |
*** armaan has quit IRC | 10:29 | |
*** armaan has joined #openstack-cinder | 10:30 | |
*** markvoelker has quit IRC | 10:34 | |
*** armaan has quit IRC | 10:43 | |
*** armaan has joined #openstack-cinder | 10:44 | |
*** abhishekk has quit IRC | 10:49 | |
*** moshele has quit IRC | 11:01 | |
*** moshele has joined #openstack-cinder | 11:01 | |
*** zhurong has quit IRC | 11:06 | |
*** edmondsw has joined #openstack-cinder | 11:20 | |
*** mvk has quit IRC | 11:23 | |
*** edmondsw has quit IRC | 11:24 | |
*** markvoelker has joined #openstack-cinder | 11:31 | |
*** moshele has quit IRC | 11:32 | |
*** moshele has joined #openstack-cinder | 11:35 | |
*** nicolasbock has joined #openstack-cinder | 11:36 | |
*** edmondsw has joined #openstack-cinder | 11:36 | |
*** moshele has quit IRC | 11:40 | |
*** chhavi has quit IRC | 11:40 | |
*** arnewiebalck_ has quit IRC | 11:48 | |
*** armaan has quit IRC | 11:54 | |
*** armaan has joined #openstack-cinder | 11:54 | |
*** mvk has joined #openstack-cinder | 11:59 | |
*** markvoelker has quit IRC | 12:05 | |
*** yikun has quit IRC | 12:05 | |
*** yikun has joined #openstack-cinder | 12:06 | |
*** yikun has quit IRC | 12:09 | |
*** gcb has quit IRC | 12:09 | |
*** gcb has joined #openstack-cinder | 12:12 | |
*** edmondsw has quit IRC | 12:15 | |
*** gcb has quit IRC | 12:28 | |
*** gcb has joined #openstack-cinder | 12:31 | |
*** laurent\ has quit IRC | 12:40 | |
openstackgerrit | Lucian Petrut proposed openstack/cinder master: Windows: allow multiple c-vol backends per service https://review.openstack.org/522806 | 12:44 |
*** yangyapeng has quit IRC | 12:45 | |
*** yangyapeng has joined #openstack-cinder | 12:46 | |
*** laurent\ has joined #openstack-cinder | 12:47 | |
*** sdague has joined #openstack-cinder | 12:49 | |
*** Jack_Iv has joined #openstack-cinder | 12:50 | |
*** yangyapeng has quit IRC | 12:50 | |
*** chenying has joined #openstack-cinder | 12:51 | |
*** moshele has joined #openstack-cinder | 12:52 | |
*** lkuchlan has quit IRC | 12:55 | |
*** laurent\ has quit IRC | 12:55 | |
*** lkuchlan has joined #openstack-cinder | 12:56 | |
*** catintheroof has joined #openstack-cinder | 12:58 | |
*** catintheroof has quit IRC | 12:59 | |
*** catintheroof has joined #openstack-cinder | 12:59 | |
*** lkuchlan has quit IRC | 13:00 | |
*** laurent\ has joined #openstack-cinder | 13:01 | |
*** markvoelker has joined #openstack-cinder | 13:02 | |
*** lhx__ has quit IRC | 13:04 | |
*** arnewiebalck_ has joined #openstack-cinder | 13:05 | |
*** pgadiya has quit IRC | 13:05 | |
*** abishop has joined #openstack-cinder | 13:09 | |
*** yangyapeng has joined #openstack-cinder | 13:12 | |
*** armaan has quit IRC | 13:13 | |
*** armaan has joined #openstack-cinder | 13:13 | |
*** edmondsw has joined #openstack-cinder | 13:13 | |
*** pgadiya has joined #openstack-cinder | 13:15 | |
*** yangyapeng has quit IRC | 13:17 | |
*** Jack_Iv has quit IRC | 13:18 | |
*** edmondsw has quit IRC | 13:18 | |
*** Jack_Iv has joined #openstack-cinder | 13:19 | |
*** markvoelker has quit IRC | 13:19 | |
*** markvoelker has joined #openstack-cinder | 13:20 | |
*** yangyapeng has joined #openstack-cinder | 13:20 | |
*** psachin has quit IRC | 13:29 | |
*** psachin has joined #openstack-cinder | 13:39 | |
*** psachin has quit IRC | 13:43 | |
*** dave-mccowan has joined #openstack-cinder | 13:49 | |
*** mriedem has joined #openstack-cinder | 13:52 | |
*** links has quit IRC | 13:57 | |
*** sdake has quit IRC | 14:12 | |
*** sdake has joined #openstack-cinder | 14:13 | |
*** sdake has quit IRC | 14:13 | |
*** sdake has joined #openstack-cinder | 14:13 | |
*** wanghao has quit IRC | 14:16 | |
*** mriedem has quit IRC | 14:17 | |
*** pgadiya has quit IRC | 14:25 | |
*** wanghao has joined #openstack-cinder | 14:29 | |
*** edmondsw has joined #openstack-cinder | 14:37 | |
*** mriedem has joined #openstack-cinder | 14:41 | |
*** _ix has quit IRC | 14:45 | |
*** eharney has joined #openstack-cinder | 14:59 | |
*** wanghao_ has joined #openstack-cinder | 15:01 | |
*** wanghao has quit IRC | 15:02 | |
*** liverpooler has joined #openstack-cinder | 15:05 | |
*** lkuchlan has joined #openstack-cinder | 15:06 | |
*** Apoorva has joined #openstack-cinder | 15:06 | |
*** armax has joined #openstack-cinder | 15:19 | |
*** armax has quit IRC | 15:23 | |
*** mchlumsky has quit IRC | 15:23 | |
*** mchlumsky has joined #openstack-cinder | 15:25 | |
*** links has joined #openstack-cinder | 15:25 | |
*** cz2 has joined #openstack-cinder | 15:25 | |
*** chhavi has joined #openstack-cinder | 15:29 | |
*** lhx_ has joined #openstack-cinder | 15:29 | |
*** links has quit IRC | 15:30 | |
*** moshele has quit IRC | 15:32 | |
*** gmann has quit IRC | 15:43 | |
*** lyarwood has joined #openstack-cinder | 15:54 | |
*** armaan has quit IRC | 15:55 | |
*** armaan has joined #openstack-cinder | 15:55 | |
lyarwood | eharney: https://bugs.launchpad.net/cinder/+bug/1739442 - we spoke about this before the break, any thoughts? Trying to work out where we can actually do this if not in c-vol. | 15:57 |
openstack | Launchpad bug 1739442 in Cinder "c-vol should always format encrypted volumes during creation" [Undecided,New] | 15:57 |
*** armax has joined #openstack-cinder | 15:57 | |
*** arnewiebalck_ has quit IRC | 15:58 | |
*** imacdonn has quit IRC | 15:58 | |
*** imacdonn has joined #openstack-cinder | 15:58 | |
*** lkuchlan has quit IRC | 16:00 | |
*** lkuchlan has joined #openstack-cinder | 16:01 | |
*** bswartz has joined #openstack-cinder | 16:03 | |
*** markstur has joined #openstack-cinder | 16:03 | |
*** markstur has quit IRC | 16:03 | |
*** markstur has joined #openstack-cinder | 16:03 | |
*** zhaochao has quit IRC | 16:04 | |
*** _ix has joined #openstack-cinder | 16:05 | |
*** dims has quit IRC | 16:09 | |
jungleboyj | Reminder for anyone who didn't see the mailing list note. No meeting this week. We will meet again next week. | 16:09 |
*** dims has joined #openstack-cinder | 16:10 | |
*** Jack_Iv has quit IRC | 16:18 | |
*** openstackgerrit has quit IRC | 16:19 | |
*** dave-mccowan has quit IRC | 16:24 | |
*** gouthamr has joined #openstack-cinder | 16:24 | |
*** mvk has quit IRC | 16:24 | |
*** lpetrut has joined #openstack-cinder | 16:24 | |
*** dave-mccowan has joined #openstack-cinder | 16:26 | |
*** lkuchlan has quit IRC | 16:31 | |
ericyoung | I've been out a few weeks... any updates on where cinder/nova stand with multi-attach and what drivers need to do? | 16:34 |
*** pcaruana has quit IRC | 16:35 | |
*** mvk has joined #openstack-cinder | 16:37 | |
Swanson | Oh, is there some expectation that'll land this release and that there might be changes for drivers? | 16:40 |
*** hemna_ has joined #openstack-cinder | 16:40 | |
Swanson | Need to devote more cycles... | 16:40 |
ericyoung | swanson: I don't know if it will land but I heard driver work is 'minimal'. I'm looking for verification as we have customers asking | 16:41 |
Swanson | ericyoung, I see. We don't have a ton of time before beta 3, right? | 16:42 |
*** moshele has joined #openstack-cinder | 16:42 | |
ericyoung | Swanson. correct. hence my question so I can plan :) | 16:42 |
*** mvk has quit IRC | 16:43 | |
*** niraj_singh has quit IRC | 16:46 | |
*** stvnoyes has joined #openstack-cinder | 16:47 | |
*** niraj_singh has joined #openstack-cinder | 16:50 | |
jungleboyj | ericyoung: The spec has merged and ildikov has started working on implementing based on the spec. | 16:53 |
jungleboyj | ericyoung: We are still hopeful that it will land in Queens. The driver impact is minimal. | 16:53 |
ericyoung | jungleboyj, thanks. will check the spec for specifics and keep fingers crossed | 16:53 |
jungleboyj | I think it is still just a matter of updating a flag to say if your driver supports multiattach. | 16:53 |
ildikov | ericyoung: so I'm looking into what I need to do | 16:53 |
jungleboyj | ericyoung: :-) Sounds good. | 16:54 |
*** moshele has quit IRC | 16:54 | |
ericyoung | ildikov, is it something like.. 'and then a miracle occurred'? | 16:55 |
ildikov | ericyoung: the target is Queens, we have weekly meetings and myself completely freaked out, I'm not fully knowledgable on the driver part, but it's something like whether or not it can handle it on the first place, reporting on shared targets if the driver does share them and turn on the capability if everything looks fine | 16:55 |
*** catintheroof has quit IRC | 16:55 | |
jgriffith | ildikov: jungleboyj ericyoung I am planning on trying to get something started and posted later this week | 16:55 |
*** mvk has joined #openstack-cinder | 16:55 | |
jgriffith | the needed changes should be fairly minimal | 16:56 |
jgriffith | driver requirements as per the spec will be a new capabilities setting being added | 16:56 |
jgriffith | I can't speak to the Nova side however | 16:56 |
jungleboyj | jgriffith: Awesome! Going to tackle it after a restful break? :-) | 16:56 |
ericyoung | excellent. I'll keep watching ad be ready to jump in (review code, change the ScaleIO driver, whatever). | 16:57 |
jgriffith | jungleboyj: "restful" is a relative term :) | 16:57 |
jgriffith | https://photos.app.goo.gl/jYUqiXzwoPV232Gr2 | 16:57 |
jgriffith | :) | 16:57 |
ildikov | jgriffith: I'm handling that part, I might need to bump the compute version once more, but should be fine, fingers crossed | 16:57 |
ildikov | jgriffith: and I'm hoping the ceiling will not land on my head after saying 'it should be fine'... :) | 16:58 |
jungleboyj | jgriffith: That is a beautiful photo. | 16:59 |
jgriffith | jungleboyj: it's a beautiful place :) | 16:59 |
jungleboyj | jgriffith: Looks relaxing to me. | 17:00 |
jgriffith | hardest part about visiting MT for me is leaving :) | 17:00 |
*** lhx_ has quit IRC | 17:00 | |
*** Apoorva has quit IRC | 17:01 | |
jungleboyj | jgriffith: That is how I feel when having to leave Florida. | 17:02 |
jgriffith | jungleboyj: nice | 17:03 |
*** chhavi has quit IRC | 17:05 | |
*** felipemonteiro has joined #openstack-cinder | 17:06 | |
*** kaisers has quit IRC | 17:10 | |
*** e0ne has quit IRC | 17:12 | |
*** kaisers has joined #openstack-cinder | 17:15 | |
*** moshele has joined #openstack-cinder | 17:23 | |
*** slunkad has quit IRC | 17:32 | |
*** Jack_Iv has joined #openstack-cinder | 17:32 | |
*** tesseract has quit IRC | 17:38 | |
*** mvk has quit IRC | 17:43 | |
edmondsw | niraj_singh why does send_service_user_token default to False? I hope you're planning on switching that to True... ? | 17:47 |
edmondsw | I'm not even sure we really need a CONF option for that... why not just always use service tokens, period? | 17:48 |
edmondsw | oh, I see the code isn't checking to see if the new CONF group has anything in it | 17:54 |
edmondsw | so you need it for backward compat unless you start checking that | 17:55 |
*** Apoorva has joined #openstack-cinder | 17:55 | |
*** Apoorva has quit IRC | 17:56 | |
*** lpetrut has quit IRC | 17:57 | |
*** Apoorva has joined #openstack-cinder | 18:00 | |
*** Apoorva has quit IRC | 18:02 | |
*** Apoorva_ has joined #openstack-cinder | 18:02 | |
*** Jack_Iv has quit IRC | 18:03 | |
*** openstackgerrit has joined #openstack-cinder | 18:05 | |
openstackgerrit | Hagen Finley proposed openstack/cinder master: Test document change to cinder install index.rst file to test my ability to upload files for review in Gerrit. https://review.openstack.org/530986 | 18:05 |
openstackgerrit | Hagen Finley proposed openstack/cinder master: 2nd Attempt to push revised install documentation to gerrit https://review.openstack.org/530987 | 18:05 |
*** armaan has quit IRC | 18:06 | |
openstackgerrit | Merged openstack/python-cinderclient master: Deprecate multiattach flag on volume create https://review.openstack.org/529375 | 18:07 |
openstackgerrit | Merged openstack/os-brick master: Cleanup test-requirements https://review.openstack.org/530878 | 18:12 |
*** armaan has joined #openstack-cinder | 18:13 | |
*** Elidx10 has joined #openstack-cinder | 18:13 | |
*** ChanServ sets mode: +r | 18:19 | |
openstackgerrit | Hagen Finley proposed openstack/cinder master: mend https://review.openstack.org/530990 | 18:23 |
*** e0ne has joined #openstack-cinder | 18:27 | |
*** felipemonteiro has quit IRC | 18:44 | |
*** pchavva has joined #openstack-cinder | 18:59 | |
*** armax has quit IRC | 19:14 | |
*** wanghao_ has quit IRC | 19:39 | |
*** yangyapeng has quit IRC | 19:44 | |
openstackgerrit | Matt Riedemann proposed openstack/cinder master: Enable multiattach capability for lvm driver https://review.openstack.org/530996 | 19:45 |
*** pcaruana has joined #openstack-cinder | 19:45 | |
openstackgerrit | Vipin Balachandran proposed openstack/cinder master: VMware: Support volume adapter type change https://review.openstack.org/530998 | 19:48 |
openstackgerrit | Merged openstack/os-brick master: Recover node.startup values after discovering https://review.openstack.org/466146 | 19:50 |
mriedem | edmondsw: that's likely copied from nova, and it defaults to False because it was originally considered experimental when it was added | 19:50 |
mriedem | and yeah, you have to opt into it by configuring a new service user | 19:51 |
*** dave-mccowan has quit IRC | 19:56 | |
*** pcaruana has quit IRC | 19:59 | |
*** e0ne has quit IRC | 20:09 | |
mriedem | ildikov: https://bugs.launchpad.net/cinder/+bug/1741112 | 20:15 |
openstack | Launchpad bug 1741112 in Cinder "500 response received when trying to create volume attachment when volume is in-use" [Undecided,New] | 20:15 |
ildikov | mriedem: Tnx! | 20:17 |
edmondsw | mriedem yeah. Not what I would have done, but not so bad when I think about it, since you have to go in and add all the auth properties anyway | 20:25 |
mriedem | are there any fake volume drivers that would allow using the api in the functional tests to create a volume and create attachments without actually needing a real volume backend? | 20:28 |
mriedem | unit tests don't really cut it for testing the entire wsgi stack | 20:28 |
patrickeast | mriedem: this thing might be close https://github.com/openstack/cinder/blob/master/cinder/tests/fake_driver.py#L29 | 20:32 |
patrickeast | iirc we do top down wsgi tests from c-api through to driver with that | 20:32 |
mriedem | yeah maybe, i saw the FakeGateDriver | 20:32 |
patrickeast | i think the gate driver is closer to a "real" backend, inheriting from lvm iscsi | 20:34 |
patrickeast | err nvm | 20:34 |
patrickeast | so is the logging one | 20:34 |
mriedem | ildikov: now i'm asking myself why we added that explicit check on the nova side for attaching the same volume to the same instance | 20:40 |
mriedem | oh i see https://github.com/openstack/cinder/blob/master/cinder/volume/api.py#L2042 | 20:43 |
mriedem | the volume api explicitly checks if we're attaching more than once to the same instance | 20:43 |
mriedem | for live migrate | 20:43 |
mriedem | oh duh, i see the issue | 20:45 |
mriedem | https://github.com/openstack/cinder/blob/master/cinder/volume/api.py#L2042 | 20:45 |
mriedem | ha | 20:45 |
mriedem | oops | 20:45 |
mriedem | raise webob.exc.HTTPInternalServerError(explanation=err_msg) | 20:45 |
ildikov | yeah, these checks got a little messy... | 20:51 |
openstackgerrit | Matt Riedemann proposed openstack/cinder master: Handle InvalidVolume when creating a volume attachment https://review.openstack.org/531011 | 20:53 |
mriedem | ^ | 20:53 |
*** ociuhandu has joined #openstack-cinder | 20:55 | |
*** eharney has quit IRC | 20:55 | |
*** ociuhandu has quit IRC | 20:56 | |
*** armax has joined #openstack-cinder | 20:59 | |
*** hemna_ has quit IRC | 21:02 | |
mriedem | ildikov: patched my devstack with that and now i get a 400 from nova when trying to attach the volume to a 2nd instance | 21:04 |
mriedem | so yay | 21:04 |
ildikov | \o/ :) | 21:05 |
*** Apoorva_ has quit IRC | 21:16 | |
*** pchavva has quit IRC | 21:21 | |
openstackgerrit | Merged openstack/cinder master: Fix reserve volume enforcing the wrong policy action https://review.openstack.org/530837 | 21:22 |
openstackgerrit | Merged openstack/cinder master: Add descriptions of base policies https://review.openstack.org/530797 | 21:22 |
*** edmondsw has quit IRC | 21:22 | |
openstackgerrit | Merged openstack/cinder master: Add driver-requirements entry for storpool https://review.openstack.org/530751 | 21:29 |
*** gouthamr has quit IRC | 21:42 | |
*** e0ne has joined #openstack-cinder | 21:43 | |
openstackgerrit | Vipin Balachandran proposed openstack/cinder master: VMware: Support volume adapter type change https://review.openstack.org/530998 | 21:43 |
mriedem | ildikov: debugging the multiattach test, there appears to be a bug on i think the cinder side when creating the 2nd attachment; it never shows up as a registered attachment for the volume | 21:57 |
mriedem | maybe nova's not calling attachment_update on it... | 21:57 |
mriedem | 2nd attachment to the 2nd server i should say | 21:57 |
ildikov | You mean with a multi-attach volume it gets attached only once? | 21:59 |
*** e0ne has quit IRC | 21:59 | |
mriedem | yeah create a multiattach volume, attach it to server1, the volume goes in-use as expected, | 21:59 |
mriedem | attach the volume to server2 and the volume goes from in-use->reserved->attaching->in-use, but the 2nd attachment doesn't show up in the volume's attachments list in the GET /volumes/{id}/detail response | 22:00 |
mriedem | and when the test tears down and tries to detach from server2, nova returns a 400 because the volume isn't actually attached | 22:00 |
ildikov | I know that Cinder needs updates on how the attachments get displayed | 22:02 |
*** e0ne has joined #openstack-cinder | 22:02 | |
ildikov | I guess it includes the volume details too, I've never checked that one explicitly TBH | 22:02 |
mriedem | i'm assuming either nova isn't calling attachment_update on the 2nd server / attachment, or it is and cinder is ignoring it | 22:03 |
ildikov | if the volume goes through those state changes then the calls should happen | 22:04 |
*** eharney has joined #openstack-cinder | 22:04 | |
mriedem | this is where the attach starts for the 2nd server http://logs.openstack.org/33/266633/4/check/tempest-full/1c4979f/job-output.txt.gz#_2018-01-03_20_56_26_602927 | 22:04 |
*** rcernin has joined #openstack-cinder | 22:05 | |
mriedem | the attachment is created in n-api here | 22:05 |
mriedem | Jan 03 20:42:28.525284 ubuntu-xenial-citycloud-sto2-0001671881 devstack@n-api.service[2197]: DEBUG cinderclient.v3.client [None req-3a39b6a5-517e-46a1-ad52-bff9b89080ab tempest-AttachVolumeMultiAttachTest-640168740 tempest-AttachVolumeMultiAttachTest-640168740] REQ: curl -g -i --cacert "/opt/stack/data/ca-bundle.pem" -X POST https://10.0.1.154/volume/v3/09a2df8b29f847f1a844be562fc917a1/attachments -H "X-Service-Token: {SHA1 | 22:05 |
mriedem | 1ef9ca173dbb52d2a33caafcdcf0bf326dde0" -H "Accept: application/json" -H "User-Agent: python-cinderclient" -H "OpenStack-API-Version: volume 3.44" -H "X-Auth-Token: {SHA1}e052150045e8c201aa1757408757c98ce1bfbdcf" -H "Content-Type: application/json" -H "X-OpenStack-Request-ID: req-3a39b6a5-517e-46a1-ad52-bff9b89080ab" -d '{"attachment": {"instance_uuid": "719979d8-9fe5-4854-9bf8-d6ce96fb7295", "connector": null, "volume_uuid" | 22:05 |
mriedem | 8fc8c1d-b39d-4365-b370-a068c5d99d81"}}' {{(pid=2199) _http_log_request /usr/local/lib/python2.7/dist-packages/keystoneauth1/session.py:372}} | 22:05 |
mriedem | RESP BODY: {"attachment": {"status": "reserved", "detached_at": "", "connection_info": {}, "attached_at": "", "attach_mode": null, "instance": "719979d8-9fe5-4854-9bf8-d6ce96fb7295", "volume_id": "68fc8c1d-b39d-4365-b370-a068c5d99d81", "id": "dd5101f2-ef06-4284-9c9b-96da2e38be26"}} | 22:05 |
mriedem | in n-cpu, it looks like the attachment is updated with the host connector | 22:06 |
mriedem | REQ: curl -g -i --cacert "/opt/stack/data/ca-bundle.pem" -X PUT https://10.0.1.154/volume/v3/09a2df8b29f847f1a844be562fc917a1/attachments/dd5101f2-ef06-4284-9c9b-96da2e38be26 -H "Accept: application/json" -H "User-Agent: python-cinderclient" -H "OpenStack-API-Version: volume 3.44" -H "X-Auth-Token: {SHA1}e052150045e8c201aa1757408757c98ce1bfbdcf" -H "Content-Type: application/json" -H "X-OpenStack-Request-ID: req-3a39b6a5-51 | 22:06 |
mriedem | 6a1-ad52-bff9b89080ab" -d '{"attachment": {"connector": {"initiator": "iqn.1993-08.org.debian:01:87ca90688668", "ip": "10.0.1.154", "platform": "x86_64", "host": "ubuntu-xenial-citycloud-sto2-0001671881", "do_local_attach": false, "mountpoint": "/dev/vdb", "os_type": "linux2", "multipath": false}}}' {{(pid=11175) _http_log_request /usr/local/lib/python2.7/dist-packages/keystoneauth1/session.py:372}} | 22:06 |
mriedem | RESP BODY: {"attachment": {"status": "reserved", "detached_at": "", "connection_info": {"access_mode": "rw", "attachment_id": "dd5101f2-ef06-4284-9c9b-96da2e38be26", "target_discovered": false, "encrypted": false, "driver_volume_type": "iscsi", "qos_specs": null, "target_iqn": "iqn.2010-10.org.openstack:volume-68fc8c1d-b39d-4365-b370-a068c5d99d81", "target_portal": "10.0.1.154:3260", "volume_id": "68fc8c1d-b39d-4365-b370-a0 | 22:07 |
mriedem | d99d81", "target_lun": 1, "auth_password": "eiBrJ2K8Qhe7UV4U", "auth_username": "AVsQ8sULVhPS5Louystc", "auth_method": "CHAP"}, "attached_at": "", "attach_mode": null, "instance": "719979d8-9fe5-4854-9bf8-d6ce96fb7295", "volume_id": "68fc8c1d-b39d-4365-b370-a068c5d99d81", "id": "dd5101f2-ef06-4284-9c9b-96da2e38be26"}} | 22:07 |
mriedem | aha! | 22:07 |
mriedem | libvirtError: internal error: unable to execute QEMU command 'device_add': Failed to get "write" lock | 22:07 |
mriedem | Driver failed to attach volume 68fc8c1d-b39d-4365-b370-a068c5d99d81 at /dev/vdb: libvirtError: internal error: unable to execute QEMU command 'device_add': Failed to get "write" lock | 22:08 |
*** e0ne has quit IRC | 22:08 | |
mriedem | i can't link to these n-cpu logs because of how infra has changed where the logs live now, but that's the issue | 22:08 |
ildikov | hmm, don't recall seeing that one before... | 22:09 |
mriedem | and when the attach_volume call in the compute fails, nova destroys the bdm record, | 22:09 |
mriedem | that's why the detach fails in tempest cleanup | 22:10 |
ildikov | at least the cleanup works... | 22:11 |
mriedem | well, the cleanup won't work until we add attachment counting to tempest, | 22:11 |
mriedem | but i just haven't hit that failure yet | 22:11 |
*** e0ne has joined #openstack-cinder | 22:12 | |
mriedem | so this should be used https://review.openstack.org/#/c/267587/74/nova/virt/libvirt/volume/volume.py | 22:12 |
mriedem | so yeah i'm not sure what "libvirtError: internal error: unable to execute QEMU command 'device_add': Failed to get "write" lock" means | 22:12 |
mriedem | from the libvirt domain docs | 22:14 |
mriedem | " shareable If present, this indicates the device is expected to be shared between domains (assuming the hypervisor and OS support this), which means that caching should be deactivated for that device. " | 22:14 |
*** e0ne has quit IRC | 22:14 | |
ildikov | does the test patch depend on the chain? | 22:14 |
mriedem | yes | 22:14 |
mriedem | this is the devstack patch at the top that enables the multiattach test in tempest: https://review.openstack.org/#/c/266633/ | 22:14 |
mriedem | this is the tempest test that depends on the nova series https://review.openstack.org/#/c/266605/ | 22:15 |
ildikov | sigh | 22:16 |
mriedem | i think this is a behavior change in qemu 2.10 | 22:16 |
mriedem | because it sounds like https://bugs.launchpad.net/nova/+bug/1718295 | 22:16 |
openstack | Launchpad bug 1718295 in OpenStack Compute (nova) "Live migration fails with qemu-img >= 2.10: "Failed to get shared "write" lock\nIs another process using the image?"" [High,Fix released] - Assigned to Sean Dague (sdague) | 22:16 |
ildikov | yeah, I did a quick search too and it looks like | 22:17 |
ildikov | as it used to work fine | 22:17 |
mriedem | melwitt: any idea off the top of your head where a cache mode is set for disk devices in libvirt guests? | 22:18 |
ildikov | we tested multi-attach before and we only had detach problems with lvm | 22:18 |
mriedem | ah there is a driver element | 22:20 |
mriedem | "The optional cache attribute controls the cache mechanism, possible values are "default", "none", "writethrough", "writeback", "directsync" (like "writethrough", but it bypasses the host page cache) and "unsafe" (host may cache all disk io, and sync requests from guest are ignored). Since 0.6.0, "directsync" since 0.9.5, | 22:20 |
mriedem | "unsafe" since 0.9.7 " | 22:20 |
melwitt | mriedem: this is where we set it in the driver https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L416 | 22:21 |
mriedem | but we explicitly set it to none for the volume block device | 22:22 |
mriedem | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/volume/volume.py#L51 | 22:23 |
mriedem | ok, so looking at https://libvirt.org/html/libvirt-libvirt-domain.html#virDomainAttachDeviceFlags | 22:24 |
mriedem | https://libvirt.org/html/libvirt-libvirt-domain.html#virDomainDeviceModifyFlags | 22:24 |
mriedem | i bet if shareable is True, we need to add the VIR_DOMAIN_DEVICE_MODIFY_FORCE flag | 22:24 |
mriedem | i'll update the patch to try that out | 22:26 |
mriedem | huzzah! | 22:27 |
*** e0ne has joined #openstack-cinder | 22:28 | |
mriedem | melwitt: if you're looking to move a couple things along, the bottom 2 changes in this series should be pretty easy https://review.openstack.org/#/c/529656/ | 22:31 |
mriedem | mostly plumbing working up toward multiattach | 22:31 |
melwitt | mriedem: yup, will look at those. I have to run out for awhile right now but can do when I get back | 22:32 |
*** abishop has quit IRC | 22:38 | |
*** moshele has quit IRC | 22:48 | |
*** e0ne has quit IRC | 23:09 | |
*** jdurgin has quit IRC | 23:14 | |
*** _ix has quit IRC | 23:16 | |
*** jdurgin has joined #openstack-cinder | 23:18 | |
openstackgerrit | Merged openstack/cinder master: Dell EMC SC: API 4.0 incompatibilities https://review.openstack.org/507260 | 23:19 |
*** erlon has quit IRC | 23:40 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!