| openstackgerrit | Shunei Shiono proposed openstack/cinder master: NEC driver: add automatic configuration of SAN access control. https://review.openstack.org/529430 | 00:08 |
|---|---|---|
| *** moshele has joined #openstack-cinder | 00:11 | |
| *** itlinux__ has quit IRC | 00:15 | |
| *** salv-orlando has joined #openstack-cinder | 00:18 | |
| *** itlinux_ has joined #openstack-cinder | 00:19 | |
| *** yangyape_ has quit IRC | 00:27 | |
| *** yangyapeng has joined #openstack-cinder | 00:27 | |
| *** yangyapeng has quit IRC | 00:32 | |
| *** moshele has quit IRC | 00:35 | |
| *** wanghao has joined #openstack-cinder | 00:39 | |
| *** felipemonteiro__ has quit IRC | 00:53 | |
| *** itlinux_ has quit IRC | 00:55 | |
| *** peter-wang has joined #openstack-cinder | 00:59 | |
| *** hoangcx has quit IRC | 00:59 | |
| *** salv-orlando has quit IRC | 01:03 | |
| *** wanghao_ has joined #openstack-cinder | 01:04 | |
| *** salv-orlando has joined #openstack-cinder | 01:04 | |
| *** wanghao has quit IRC | 01:07 | |
| *** wanghao_ has quit IRC | 01:08 | |
| *** hoangcx has joined #openstack-cinder | 01:09 | |
| *** peter-wang has quit IRC | 01:09 | |
| *** wanghao has joined #openstack-cinder | 01:09 | |
| *** yangyapeng has joined #openstack-cinder | 01:12 | |
| *** psachin has joined #openstack-cinder | 01:20 | |
| *** edmondsw has joined #openstack-cinder | 01:21 | |
| *** edmondsw has quit IRC | 01:26 | |
| *** dalgaaf has quit IRC | 01:32 | |
| *** dalgaaf has joined #openstack-cinder | 01:32 | |
| *** crushil has joined #openstack-cinder | 01:33 | |
| *** stakeda has joined #openstack-cinder | 01:34 | |
| *** crushil has quit IRC | 01:42 | |
| *** rmcall has quit IRC | 01:44 | |
| *** salv-orlando has quit IRC | 01:46 | |
| *** Apoorva_ has joined #openstack-cinder | 01:46 | |
| *** Apoorva has quit IRC | 01:50 | |
| *** Apoorva_ has quit IRC | 01:51 | |
| *** threestrands has joined #openstack-cinder | 01:51 | |
| *** threestrands has quit IRC | 01:51 | |
| *** threestrands has joined #openstack-cinder | 01:51 | |
| *** jappleii__ has quit IRC | 01:52 | |
| *** gcb has joined #openstack-cinder | 02:03 | |
| *** tommylikehu_ has joined #openstack-cinder | 02:04 | |
| *** rajinir_ has joined #openstack-cinder | 02:04 | |
| gcb | eharney: can you revisit https://review.openstack.org/#/c/528618/ ? the requirement update patch was merged | 02:05 |
| *** dalgaaf has quit IRC | 02:07 | |
| *** mvk has quit IRC | 02:07 | |
| *** harlowja has quit IRC | 02:07 | |
| *** v12aml has quit IRC | 02:07 | |
| *** obre has quit IRC | 02:07 | |
| *** tommylikehu has quit IRC | 02:07 | |
| *** Gorian has quit IRC | 02:07 | |
| *** dansmith has quit IRC | 02:07 | |
| *** guyr-infinidat has quit IRC | 02:07 | |
| *** thurloat has quit IRC | 02:07 | |
| *** evrardjp has quit IRC | 02:07 | |
| *** dulek has quit IRC | 02:07 | |
| *** rajinir has quit IRC | 02:07 | |
| *** obre_ has joined #openstack-cinder | 02:07 | |
| *** tommylikehu_ is now known as tommylikehu | 02:07 | |
| *** rajinir_ is now known as rajinir | 02:07 | |
| *** v12aml has joined #openstack-cinder | 02:07 | |
| *** Gorian has joined #openstack-cinder | 02:07 | |
| *** thurloat has joined #openstack-cinder | 02:07 | |
| *** mvk has joined #openstack-cinder | 02:08 | |
| *** dulek has joined #openstack-cinder | 02:08 | |
| *** dansmith has joined #openstack-cinder | 02:08 | |
| *** dansmith is now known as Guest99680 | 02:08 | |
| *** evrardjp has joined #openstack-cinder | 02:09 | |
| *** dave-mccowan has joined #openstack-cinder | 02:09 | |
| *** andreykurilin has quit IRC | 02:10 | |
| *** mriedem has quit IRC | 02:10 | |
| *** dalgaaf has joined #openstack-cinder | 02:11 | |
| *** andreykurilin has joined #openstack-cinder | 02:12 | |
| *** yangyapeng has quit IRC | 02:14 | |
| *** yangyapeng has joined #openstack-cinder | 02:15 | |
| *** wanghao_ has joined #openstack-cinder | 02:16 | |
| *** wanghao has quit IRC | 02:20 | |
| openstackgerrit | jian.song proposed openstack/cinder master: Follow the new PTI for document build https://review.openstack.org/529006 | 02:24 |
| *** gouthamr has quit IRC | 02:29 | |
| openstackgerrit | jichao zhang proposed openstack/cinder master: Unity: Attach Unity volume via IPv6. https://review.openstack.org/529454 | 02:29 |
| *** _pewp_ has quit IRC | 02:46 | |
| *** _pewp_ has joined #openstack-cinder | 02:46 | |
| *** salv-orlando has joined #openstack-cinder | 02:47 | |
| *** salv-orlando has quit IRC | 02:47 | |
| *** Apoorva has joined #openstack-cinder | 03:01 | |
| *** masuberu has joined #openstack-cinder | 03:03 | |
| *** masber has quit IRC | 03:04 | |
| *** edmondsw has joined #openstack-cinder | 03:09 | |
| openstackgerrit | Merged openstack/os-brick master: Updated from global requirements https://review.openstack.org/528423 | 03:12 |
| openstackgerrit | Merged openstack/python-cinderclient master: Updated from global requirements https://review.openstack.org/528901 | 03:13 |
| *** edmondsw has quit IRC | 03:14 | |
| *** masuberu has quit IRC | 03:17 | |
| *** wanghao_ has quit IRC | 03:18 | |
| *** wanghao has joined #openstack-cinder | 03:18 | |
| *** dave-mccowan has quit IRC | 03:20 | |
| *** Apoorva has quit IRC | 03:22 | |
| *** Apoorva has joined #openstack-cinder | 03:23 | |
| *** Apoorva has quit IRC | 03:27 | |
| *** markstur has quit IRC | 03:35 | |
| *** pooja-jadhav has joined #openstack-cinder | 03:38 | |
| *** pooja_jadhav has joined #openstack-cinder | 03:38 | |
| pooja_jadhav | tommylikehu: Hi | 03:38 |
| tommylikehu | pooja_jadhav: hi | 03:38 |
| *** threestrands_ has joined #openstack-cinder | 03:39 | |
| *** threestrands_ has quit IRC | 03:39 | |
| *** threestrands_ has joined #openstack-cinder | 03:39 | |
| pooja_jadhav | tommylikehu: I have commented on patch https://review.openstack.org/#/c/524528/. that "As cinderclient patch[1] is got merged, But the cinderclient version is not yet released so until and unless cinderclient is get released we should not merge this patch, otherwise it will affact on jobs. | 03:39 |
| pooja_jadhav | [1] https://review.openstack.org/#/c/527637/" | 03:39 |
| pooja_jadhav | tommylikehu: but workflow is +1 | 03:40 |
| tommylikehu | pooja_jadhav: ok! | 03:40 |
| *** threestrands has quit IRC | 03:41 | |
| *** namnh has joined #openstack-cinder | 03:42 | |
| *** guyr-infinidat has joined #openstack-cinder | 04:00 | |
| *** namnh has quit IRC | 04:12 | |
| *** salv-orlando has joined #openstack-cinder | 04:15 | |
| *** pooja-jadhav has quit IRC | 04:15 | |
| *** masber has joined #openstack-cinder | 04:17 | |
| *** salv-orlando has quit IRC | 04:20 | |
| *** links has joined #openstack-cinder | 04:35 | |
| *** links has quit IRC | 04:35 | |
| *** markstur has joined #openstack-cinder | 04:52 | |
| *** psachin has quit IRC | 04:55 | |
| *** markstur has quit IRC | 04:57 | |
| *** edmondsw has joined #openstack-cinder | 04:57 | |
| openstackgerrit | Merged openstack/cinder master: Remove deprecated 'nas_ip' config option https://review.openstack.org/528299 | 04:58 |
| *** edmondsw has quit IRC | 05:02 | |
| *** gmann has joined #openstack-cinder | 05:08 | |
| *** storshoo___ has joined #openstack-cinder | 05:12 | |
| *** a_storshoo has quit IRC | 05:13 | |
| *** salv-orlando has joined #openstack-cinder | 05:16 | |
| *** avishay has joined #openstack-cinder | 05:16 | |
| *** salv-orlando has quit IRC | 05:21 | |
| *** crushil has joined #openstack-cinder | 05:30 | |
| *** chhavi has joined #openstack-cinder | 05:39 | |
| *** markstur has joined #openstack-cinder | 05:41 | |
| *** markstur has quit IRC | 05:42 | |
| *** psachin has joined #openstack-cinder | 05:43 | |
| *** armaan has joined #openstack-cinder | 05:43 | |
| *** pgadiya has joined #openstack-cinder | 05:49 | |
| openstackgerrit | Guoqiang Ding proposed openstack/cinder-specs master: Fix the misspelling of "migration" https://review.openstack.org/529487 | 05:54 |
| *** armaan has quit IRC | 05:57 | |
| *** storshoo___ has quit IRC | 05:59 | |
| *** a_storshoo has joined #openstack-cinder | 05:59 | |
| *** moshele has joined #openstack-cinder | 05:59 | |
| *** psachin has quit IRC | 06:02 | |
| openstackgerrit | Merged openstack/cinder-specs master: Fix the misspelling of "migration" https://review.openstack.org/529487 | 06:05 |
| *** moshele has quit IRC | 06:07 | |
| *** lkuchlan has joined #openstack-cinder | 06:11 | |
| *** armax has quit IRC | 06:12 | |
| *** armax has joined #openstack-cinder | 06:12 | |
| *** armax has quit IRC | 06:13 | |
| *** armax has joined #openstack-cinder | 06:13 | |
| *** armax has quit IRC | 06:14 | |
| *** armax has joined #openstack-cinder | 06:14 | |
| *** salv-orlando has joined #openstack-cinder | 06:14 | |
| *** armax has quit IRC | 06:14 | |
| *** armax has joined #openstack-cinder | 06:15 | |
| *** armax has quit IRC | 06:15 | |
| openstackgerrit | jichao zhang proposed openstack/cinder master: Unity: Attach Unity volume via IPv6. https://review.openstack.org/529454 | 06:16 |
| *** salv-orl_ has joined #openstack-cinder | 06:17 | |
| *** chhavi__ has joined #openstack-cinder | 06:17 | |
| *** threestrands_ has quit IRC | 06:21 | |
| *** salv-orlando has quit IRC | 06:26 | |
| *** lkuchlan has quit IRC | 06:26 | |
| *** chhavi has quit IRC | 06:26 | |
| *** crushil has quit IRC | 06:26 | |
| *** v12aml has quit IRC | 06:26 | |
| *** rajinir has quit IRC | 06:26 | |
| *** tommylikehu has quit IRC | 06:26 | |
| *** itlinux has quit IRC | 06:26 | |
| *** afazekas has quit IRC | 06:26 | |
| *** robcresswell has quit IRC | 06:26 | |
| *** ameade has quit IRC | 06:26 | |
| *** dirk has quit IRC | 06:26 | |
| *** TheJulia has quit IRC | 06:26 | |
| *** Roamer` has quit IRC | 06:26 | |
| *** zhongjun has quit IRC | 06:26 | |
| *** knikolla has quit IRC | 06:26 | |
| *** tbarron has quit IRC | 06:26 | |
| *** catmando has quit IRC | 06:26 | |
| *** y_storshoo has quit IRC | 06:26 | |
| *** y_storshoo has joined #openstack-cinder | 06:28 | |
| *** itlinux has joined #openstack-cinder | 06:30 | |
| *** v12aml has joined #openstack-cinder | 06:31 | |
| *** rajinir has joined #openstack-cinder | 06:31 | |
| *** tommylikehu has joined #openstack-cinder | 06:31 | |
| *** afazekas has joined #openstack-cinder | 06:31 | |
| *** catmando has joined #openstack-cinder | 06:31 | |
| *** robcresswell has joined #openstack-cinder | 06:31 | |
| *** ameade has joined #openstack-cinder | 06:31 | |
| *** dirk has joined #openstack-cinder | 06:31 | |
| *** TheJulia has joined #openstack-cinder | 06:31 | |
| *** Roamer` has joined #openstack-cinder | 06:31 | |
| *** zhongjun has joined #openstack-cinder | 06:31 | |
| *** knikolla has joined #openstack-cinder | 06:31 | |
| *** lkuchlan has joined #openstack-cinder | 06:32 | |
| *** tbarron has joined #openstack-cinder | 06:33 | |
| *** lkuchlan has quit IRC | 06:41 | |
| *** lkuchlan has joined #openstack-cinder | 06:41 | |
| kwathore | @Team, Please review patch:https://review.openstack.org/#/c/528266/ | 06:45 |
| *** edmondsw has joined #openstack-cinder | 06:45 | |
| *** v12aml has quit IRC | 06:48 | |
| *** rajinir has quit IRC | 06:48 | |
| *** tommylikehu has quit IRC | 06:48 | |
| *** afazekas has quit IRC | 06:48 | |
| *** robcresswell has quit IRC | 06:48 | |
| *** ameade has quit IRC | 06:48 | |
| *** dirk has quit IRC | 06:48 | |
| *** TheJulia has quit IRC | 06:48 | |
| *** Roamer` has quit IRC | 06:48 | |
| *** zhongjun has quit IRC | 06:48 | |
| *** knikolla has quit IRC | 06:48 | |
| *** catmando has quit IRC | 06:48 | |
| openstackgerrit | Ryan Liang proposed openstack/cinder master: [VNX] Restore snapshot to volume https://review.openstack.org/529497 | 06:49 |
| *** edmondsw has quit IRC | 06:50 | |
| *** e0ne has joined #openstack-cinder | 06:57 | |
| *** e0ne has quit IRC | 06:58 | |
| *** zhenguo has quit IRC | 07:04 | |
| *** psachin has joined #openstack-cinder | 07:18 | |
| *** Roamer` has joined #openstack-cinder | 07:24 | |
| *** afazekas has joined #openstack-cinder | 07:24 | |
| *** v12aml has joined #openstack-cinder | 07:24 | |
| *** dirk has joined #openstack-cinder | 07:24 | |
| *** TheJulia has joined #openstack-cinder | 07:24 | |
| *** catmando has joined #openstack-cinder | 07:24 | |
| *** robcresswell has joined #openstack-cinder | 07:25 | |
| *** zhongjun has joined #openstack-cinder | 07:25 | |
| *** knikolla has joined #openstack-cinder | 07:25 | |
| *** ameade has joined #openstack-cinder | 07:26 | |
| *** tommylikehu has joined #openstack-cinder | 07:26 | |
| *** rajinir has joined #openstack-cinder | 07:27 | |
| openstackgerrit | Kushal Wathore proposed openstack/cinder master: HPE 3PAR - Implement Tiramisu feature on 3PAR https://review.openstack.org/528266 | 07:35 |
| *** rcernin has quit IRC | 07:37 | |
| *** AlexeyAbashkin has joined #openstack-cinder | 07:41 | |
| *** pcaruana has joined #openstack-cinder | 07:58 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Update tgt iSCSI target on volume extend https://review.openstack.org/529500 | 08:09 |
| *** amoralej|off is now known as amoralej | 08:12 | |
| *** gcb has quit IRC | 08:16 | |
| *** imran_ansari has joined #openstack-cinder | 08:25 | |
| *** gmann has quit IRC | 08:27 | |
| imran_ansari | smcginnis: Hi, I've addressed your review comments related to patch https://review.openstack.org/#/c/524185/10 . Request you to have a look at those when you get time and suggest if there is anything else that needs to be addressed. | 08:29 |
| *** pgadiya has quit IRC | 08:31 | |
| *** gkadam has joined #openstack-cinder | 08:32 | |
| *** edmondsw has joined #openstack-cinder | 08:33 | |
| *** salv-orl_ has quit IRC | 08:35 | |
| *** salv-orlando has joined #openstack-cinder | 08:35 | |
| *** edmondsw has quit IRC | 08:38 | |
| *** armaan has joined #openstack-cinder | 08:38 | |
| *** linkmark has joined #openstack-cinder | 08:39 | |
| *** alexchadin has joined #openstack-cinder | 08:39 | |
| *** salv-orlando has quit IRC | 08:40 | |
| *** namnh has joined #openstack-cinder | 08:41 | |
| *** e0ne has joined #openstack-cinder | 08:43 | |
| *** Brin has joined #openstack-cinder | 08:43 | |
| *** salv-orlando has joined #openstack-cinder | 08:44 | |
| *** pgadiya has joined #openstack-cinder | 08:46 | |
| openstackgerrit | wanghao proposed openstack/cinder master: Report backend state in service list https://review.openstack.org/526825 | 08:53 |
| *** armaan has quit IRC | 08:58 | |
| *** pgadiya has quit IRC | 08:59 | |
| *** armaan has joined #openstack-cinder | 09:00 | |
| *** alexchadin has quit IRC | 09:00 | |
| openstackgerrit | Imran Ansari proposed openstack/cinder master: 3PAR - Fix temp snapshot that remains post online copy https://review.openstack.org/524185 | 09:00 |
| *** armaan has quit IRC | 09:00 | |
| *** armaan has joined #openstack-cinder | 09:01 | |
| *** gcb has joined #openstack-cinder | 09:03 | |
| *** alexchadin has joined #openstack-cinder | 09:04 | |
| *** yikun_jiang is now known as yikun | 09:09 | |
| *** pgadiya has joined #openstack-cinder | 09:13 | |
| *** imran_ansari_ has joined #openstack-cinder | 09:17 | |
| *** pgadiya has quit IRC | 09:17 | |
| *** pgadiya has joined #openstack-cinder | 09:18 | |
| *** imran_ansari has quit IRC | 09:19 | |
| *** pgadiya has quit IRC | 09:23 | |
| *** armaan has quit IRC | 09:26 | |
| *** gmann has joined #openstack-cinder | 09:27 | |
| *** wanghao has quit IRC | 09:34 | |
| *** wanghao_ has joined #openstack-cinder | 09:34 | |
| *** pgadiya has joined #openstack-cinder | 09:36 | |
| *** imran_ansari_ has quit IRC | 09:44 | |
| *** imran_ansari_ has joined #openstack-cinder | 09:44 | |
| *** salv-orlando has quit IRC | 09:44 | |
| *** salv-orlando has joined #openstack-cinder | 09:45 | |
| *** salv-orlando has quit IRC | 09:49 | |
| *** pgadiya has quit IRC | 09:54 | |
| *** imran_ansari_ has quit IRC | 09:57 | |
| *** imran_ansari has joined #openstack-cinder | 09:59 | |
| *** e0ne has quit IRC | 10:03 | |
| *** pgadiya has joined #openstack-cinder | 10:07 | |
| *** lpetrut has joined #openstack-cinder | 10:18 | |
| *** namnh has quit IRC | 10:20 | |
| *** edmondsw has joined #openstack-cinder | 10:22 | |
| *** mayur_ind has joined #openstack-cinder | 10:22 | |
| *** edmondsw has quit IRC | 10:26 | |
| *** mayur_ind has quit IRC | 10:27 | |
| *** alexchadin has quit IRC | 10:31 | |
| *** wanghao has joined #openstack-cinder | 10:32 | |
| *** wanghao_ has quit IRC | 10:36 | |
| *** chhavi__ has quit IRC | 10:36 | |
| *** alexchadin has joined #openstack-cinder | 10:37 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - Fix AttributeError of dell emc driver https://review.openstack.org/525913 | 10:39 |
| *** alexchadin has quit IRC | 10:41 | |
| *** pcaruana has quit IRC | 10:43 | |
| *** wanghao has quit IRC | 10:45 | |
| *** wanghao has joined #openstack-cinder | 10:45 | |
| *** salv-orlando has joined #openstack-cinder | 10:45 | |
| *** e0ne has joined #openstack-cinder | 10:46 | |
| *** salv-orlando has quit IRC | 10:50 | |
| *** wanghao_ has joined #openstack-cinder | 11:00 | |
| *** AlexeyAbashkin has quit IRC | 11:01 | |
| *** wanghao has quit IRC | 11:01 | |
| *** stakeda has quit IRC | 11:06 | |
| *** bhagyashris has quit IRC | 11:07 | |
| *** alexchadin has joined #openstack-cinder | 11:08 | |
| *** AlexeyAbashkin has joined #openstack-cinder | 11:08 | |
| *** chhavi__ has joined #openstack-cinder | 11:10 | |
| kwathore | @Team:Please review : https://review.openstack.org/#/c/528266/ | 11:11 |
| *** imran_ansari has quit IRC | 11:22 | |
| *** hoonetorg has quit IRC | 11:22 | |
| *** salv-orlando has joined #openstack-cinder | 11:24 | |
| *** bhagyashris has joined #openstack-cinder | 11:34 | |
| *** hoonetorg has joined #openstack-cinder | 11:34 | |
| *** Brin has quit IRC | 11:44 | |
| *** rarora has joined #openstack-cinder | 11:50 | |
| *** liverpooler has quit IRC | 11:54 | |
| mdbooth | Quick security-related question: can connection_info contain secrets? | 12:01 |
| mdbooth | IoW, is it safe for Nova to log it without redaction? | 12:01 |
| * mdbooth would guess no | 12:01 | |
| *** imran_ansari has joined #openstack-cinder | 12:04 | |
| *** edmondsw has joined #openstack-cinder | 12:10 | |
| *** dave-mccowan has joined #openstack-cinder | 12:10 | |
| *** erlon has joined #openstack-cinder | 12:12 | |
| *** edmondsw has quit IRC | 12:14 | |
| *** imran_ansari has quit IRC | 12:23 | |
| *** imran_ansari has joined #openstack-cinder | 12:23 | |
| *** dave-mccowan has quit IRC | 12:24 | |
| *** wanghao_ has quit IRC | 12:30 | |
| *** raghadq has joined #openstack-cinder | 12:30 | |
| *** wanghao has joined #openstack-cinder | 12:30 | |
| *** wanghao has quit IRC | 12:31 | |
| *** wanghao has joined #openstack-cinder | 12:31 | |
| *** raghadq has quit IRC | 12:31 | |
| *** wanghao has quit IRC | 12:31 | |
| *** wanghao has joined #openstack-cinder | 12:32 | |
| *** aseela has joined #openstack-cinder | 12:32 | |
| *** wanghao has quit IRC | 12:32 | |
| *** wanghao has joined #openstack-cinder | 12:33 | |
| *** wanghao has quit IRC | 12:33 | |
| *** wanghao has joined #openstack-cinder | 12:33 | |
| *** wanghao has quit IRC | 12:34 | |
| *** kwathore_ has joined #openstack-cinder | 12:34 | |
| *** wanghao has joined #openstack-cinder | 12:34 | |
| *** wanghao has quit IRC | 12:35 | |
| *** wanghao has joined #openstack-cinder | 12:35 | |
| *** wanghao has quit IRC | 12:35 | |
| *** wanghao has joined #openstack-cinder | 12:36 | |
| *** wanghao has quit IRC | 12:36 | |
| *** wanghao has joined #openstack-cinder | 12:36 | |
| *** kwathore has quit IRC | 12:37 | |
| *** wanghao has quit IRC | 12:37 | |
| *** wanghao has joined #openstack-cinder | 12:38 | |
| *** wanghao has quit IRC | 12:42 | |
| *** imran_ansari has quit IRC | 12:57 | |
| *** psachin has quit IRC | 12:58 | |
| *** dave-mccowan has joined #openstack-cinder | 12:58 | |
| *** crushil has joined #openstack-cinder | 12:59 | |
| openstackgerrit | Merged openstack/cinder master: Add service_token for cinder-nova interaction https://review.openstack.org/524497 | 13:05 |
| *** psachin has joined #openstack-cinder | 13:06 | |
| *** dave-mcc_ has joined #openstack-cinder | 13:07 | |
| *** dave-mccowan has quit IRC | 13:08 | |
| *** yangyapeng has quit IRC | 13:10 | |
| *** yangyapeng has joined #openstack-cinder | 13:15 | |
| *** yangyapeng has quit IRC | 13:19 | |
| *** salv-orl_ has joined #openstack-cinder | 13:20 | |
| *** eharney has quit IRC | 13:20 | |
| *** salv-orlando has quit IRC | 13:22 | |
| *** crushil has quit IRC | 13:25 | |
| *** alexchadin has quit IRC | 13:31 | |
| *** pgadiya has quit IRC | 13:32 | |
| *** gnufied has joined #openstack-cinder | 13:33 | |
| *** gnufied__ has joined #openstack-cinder | 13:33 | |
| *** gnufied__ has quit IRC | 13:34 | |
| *** catintheroof has joined #openstack-cinder | 13:34 | |
| *** catintheroof has quit IRC | 13:36 | |
| *** alexchadin has joined #openstack-cinder | 13:36 | |
| *** catintheroof has joined #openstack-cinder | 13:36 | |
| *** salv-orl_ has quit IRC | 13:39 | |
| *** salv-orlando has joined #openstack-cinder | 13:39 | |
| *** salv-orlando has quit IRC | 13:44 | |
| *** mriedem has joined #openstack-cinder | 13:46 | |
| *** abhishekk has joined #openstack-cinder | 13:51 | |
| Roamer` | hmm, our third-party CI (still silent) just failed tempest.api.volume.test_volumes_list.VolumesListTestJSON.test_volume_list_with_details because test_volume_list_details_by_name was invoked before it, so the volume's "updated_at" property changed | 13:54 |
| Roamer` | test_volume_list_details_by_name was invoked two seconds after the volume's creation, so the updated_at field cached just after creation was no longer valid by the time test_volume_list_with_details checked it | 13:55 |
| Roamer` | if anybody wants to take a look, that's http://logs.ci-openstack.storpool.com/96/529396/1/silent/dsvm-tempest-storpool/afa6b8a/ | 13:55 |
| Roamer` | it's possible that our CI is a bit slow at times, thus the two-second interval, but still... | 13:56 |
| kwathore_ | @Team:Please review : https://review.openstack.org/#/c/528266/ | 13:57 |
| Roamer` | should the tests update the cached copy's updated_at field? If people think they should, I could try my hand at it | 13:58 |
| *** edmondsw has joined #openstack-cinder | 13:58 | |
| *** psachin has quit IRC | 13:58 | |
| *** gouthamr has joined #openstack-cinder | 14:00 | |
| jungleboyj | mdbooth There was discussion about that somewhere in the last few days. I believe the answer was that it shouldn't be logged without being sanitized. Not sure if there has been a patch. | 14:01 |
| mdbooth | jungleboyj: Thanks. | 14:02 |
| *** edmondsw has quit IRC | 14:02 | |
| *** mchlumsky has joined #openstack-cinder | 14:04 | |
| *** dustins has joined #openstack-cinder | 14:07 | |
| *** eharney has joined #openstack-cinder | 14:19 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Update tgt iSCSI target on volume extend https://review.openstack.org/529500 | 14:19 |
| smcginnis | mdbooth: It could contain CHAP secrets, so probably not the best to log it. | 14:21 |
| mnaser | question: does connection_info contain the volume size (or does nova at any point have access to that information in attach_volume() if anyone knows) | 14:22 |
| mdbooth | smcginnis: Thanks. I was pretty sure it was dodgy and deleted it. | 14:22 |
| mnaser | i'm trying to implement a per-gb iops limit (rather than an absolute one) | 14:22 |
| mdbooth | mnaser: Are you thinking in the context of Nova? | 14:22 |
| mdbooth | mnaser: Or maybe os-brick? | 14:23 |
| mnaser | mdbooth: yes, for now.. if that doesnt work, then maybe we can think in context of cinder sending a "precalculated" value | 14:23 |
| mnaser | mdbooth: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/volume/volume.py#L65-L78 was thinking of adding something small here | 14:24 |
| smcginnis | mnaser: Like this? https://review.openstack.org/#/c/451565/ | 14:24 |
| mdbooth | mnaser: I don't know the answer to your question, but from a Nova pov please make sure that anything which looks inside connection_info is at least volume-driver specific, and preferrably in os-brick. | 14:24 |
| mnaser | smcginnis: yes, exactly that spec smcginnis ! | 14:24 |
| mdbooth | We abuse that all over the place, unfortunately. | 14:24 |
| mnaser | smcginnis: that is terrifyingly close to what i did | 14:24 |
| mnaser | (or what im trying to do) | 14:25 |
| mnaser | http://paste.openstack.org/show/629586/ | 14:25 |
| mnaser | but obviously i dont know what the volume_size is tehre | 14:25 |
| mnaser | what | 14:25 |
| mnaser | this mergeD?! | 14:25 |
| mnaser | okay well.. thank you smcginnis .. i'll ask next time | 14:25 |
| smcginnis | mnaser: Yep, looks like that did make it into Pike - https://review.openstack.org/#/c/447127/ | 14:25 |
| mnaser | this is awesome! | 14:26 |
| *** liverpooler has joined #openstack-cinder | 14:26 | |
| mnaser | time to see how well it works i guess! | 14:26 |
| smcginnis | mnaser: It's an early Christmas for you. Or $HOLIDAY of choice. :) | 14:26 |
| smcginnis | mnaser: So that one actually confused me for awhile because I couldn't grok the use case. Is it easy for you describe your need for it in a couple sentences? | 14:27 |
| mnaser | smcginnis: yes! and i was thinking how i was going to convince teams get this to merge for queens and how it would be a simple multiplication :p | 14:27 |
| smcginnis | If not, that's OK. Just curious, | 14:27 |
| mnaser | smcginnis: sure! so picture this - we have a big ceph cluster, as the ceph cluster grows in capacity, it grows in performance as well | 14:27 |
| mnaser | so at 1000GB, it has 1000 iops, 2000GB, it has 2000 iops | 14:28 |
| mnaser | (not exactly, but you could say that it grows almost linearly) | 14:28 |
| mnaser | the idea is that if i have 1000GB, someone creates a 10GB volume of 1000 iops type, now i have 990 GB of wasted capacity that i cannot guarantee performance on | 14:28 |
| smcginnis | OK, I think I get it now. So it's kind of performance/capacity slicing. | 14:29 |
| mnaser | smcginnis: correct and it maximizes the efficency of our cluster | 14:29 |
| mnaser | because then that way we wont have to overprovision disks and have them sit empty just to "maintain" performance | 14:29 |
| smcginnis | mnaser: OK, thanks. I see how that could be useful for some backends then. | 14:29 |
| mnaser | and in the context of a public cloud this stuff is critical | 14:30 |
| smcginnis | We had a few internal discussions about that, and for some reason it was difficult for me to fully grasp. But I think I actually get it now. :) | 14:30 |
| mnaser | well i'll let you know how it works! | 14:31 |
| *** avishay has quit IRC | 14:32 | |
| smcginnis | mnaser: That would be great to get some real world feedback! | 14:33 |
| *** aseela has quit IRC | 14:34 | |
| *** liverpooler has quit IRC | 14:38 | |
| *** crushil has joined #openstack-cinder | 14:38 | |
| *** liverpooler has joined #openstack-cinder | 14:39 | |
| *** alexchadin has quit IRC | 14:40 | |
| *** salv-orlando has joined #openstack-cinder | 14:40 | |
| *** salv-orlando has quit IRC | 14:44 | |
| *** yangyapeng has joined #openstack-cinder | 14:44 | |
| *** yangyapeng has quit IRC | 14:47 | |
| *** crushil has quit IRC | 14:48 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Update tgt iSCSI target on volume extend https://review.openstack.org/529500 | 14:49 |
| *** abhishekk has quit IRC | 14:49 | |
| *** gkadam has quit IRC | 14:50 | |
| *** gouthamr has quit IRC | 14:51 | |
| *** yangyapeng has joined #openstack-cinder | 14:51 | |
| *** gouthamr has joined #openstack-cinder | 14:52 | |
| *** gouthamr has quit IRC | 14:55 | |
| *** mayur_ind has joined #openstack-cinder | 14:58 | |
| mayur_ind | Hi, | 15:02 |
| mayur_ind | I need to write Cinder ISCSI driver , for our storage backend. But I dont know actual ciner flow for third party driver. From where should I begin??? | 15:02 |
| smcginnis | mayur_ind: Best is to start reading here: https://wiki.openstack.org/wiki/Cinder/how-to-contribute-a-driver | 15:02 |
| smcginnis | mayur_ind: And follow all links to understand the requirements. | 15:03 |
| *** gouthamr_ has joined #openstack-cinder | 15:03 | |
| smcginnis | mayur_ind: Then there is also very pertinent information under here: https://docs.openstack.org/cinder/latest/contributor/drivers.html | 15:03 |
| smcginnis | mayur_ind: Then when you are ready, create a new blueprint with just some basic info stating something like "Add driver for XXX storage" here: https://blueprints.launchpad.net/cinder/+addspec | 15:05 |
| *** danpawlik has quit IRC | 15:05 | |
| smcginnis | mayur_ind: All that's assuming you want to add it to the cinder repo. If you just want to write a driver to maintain out of the Cinder tree (in you own repo) then that second link is probably the most useful. | 15:05 |
| mayur_ind | Thanks @smcginnis.. :-) | 15:06 |
| smcginnis | mayur_ind: No problem. Feel free to ask questions here if there is anything unclear in the docs. | 15:07 |
| smcginnis | mayur_ind: And definitely feel free to propose any patches updating the docs for anything unclear. ;) | 15:07 |
| mayur_ind | sure @smcginnis... | 15:08 |
| *** armax has joined #openstack-cinder | 15:09 | |
| *** dave-mcc_ has quit IRC | 15:09 | |
| mayur_ind | We already have NFS based driver. But for ISCSI Iam little confuse what configurations should I maintain in cinder.conf | 15:09 |
| *** dave-mccowan has joined #openstack-cinder | 15:10 | |
| smcginnis | mayur_ind: I guess that all depends on what configuration options your driver would need. | 15:11 |
| *** gouthamr has joined #openstack-cinder | 15:11 | |
| *** crose has joined #openstack-cinder | 15:11 | |
| *** dave-mcc_ has joined #openstack-cinder | 15:12 | |
| *** liverpooler has quit IRC | 15:13 | |
| *** dave-mccowan has quit IRC | 15:15 | |
| mayur_ind | @smcginnis I wrote dummy functions and tried to connect driver to backend.. | 15:15 |
| mayur_ind | But Iam getting this log -- > | 15:16 |
| mayur_ind | ERROR cinder.service [-] Manager for service cinder-volume host@type-2 is reporting problems, not sending heartbeat. Service will appear "down". | 15:16 |
| smcginnis | mayur_ind: You probably need to check further back in the logs, but I would guess it's not returning anything for update_stats and therefore getting marked as down. | 15:16 |
| mriedem | this is new to me, anyone else seen this? http://logs.openstack.org/79/527479/3/gate/legacy-tempest-dsvm-py35/96944d0/job-output.txt.gz#_2017-12-21_13_32_44_398136 | 15:18 |
| mriedem | AssertionError: Could not find volumes ['fd31d767-e079-4c6f-a9f4-42d4e82bba08:tempest-VolumesListTestJSON-Volume-120871556'] in expected list | 15:18 |
| Roamer` | mriedem, could it be related to what I wrote above? | 15:18 |
| mriedem | Roamer`: hmm maybe | 15:19 |
| Roamer` | mriedem, one of the tests fetches the info for a volume anew, so the "updated_at" field for the volume gets, well, updated :) | 15:19 |
| mriedem | is this a new test? | 15:19 |
| mriedem | or a recent change to an existing test? | 15:19 |
| mayur_ind | thanks @smcginnis. I will start working on this. Will get back if got stuck anywhere ;) | 15:20 |
| smcginnis | A fetch should not update the updated_at value. Only modifications. | 15:20 |
| Roamer` | smcginnis, maybe that's the root of the problem, I didn't look much deeper, but it happened here too | 15:20 |
| Roamer` | let me take a look at what mriedem pointed at, to see if this is the case | 15:21 |
| mriedem | looks like something changed around 12/19 | 15:25 |
| mriedem | oh maybe not | 15:25 |
| mriedem | http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22AssertionError%3A%20Could%20not%20find%20volumes%5C%22%20AND%20message%3A%5C%22in%20expected%20list%5C%22%20AND%20tags%3A%5C%22console%5C%22&from=10d | 15:26 |
| Roamer` | yep, it is the same problem: "updated_at" is 17 seconds later | 15:26 |
| *** itlinux has quit IRC | 15:27 | |
| mriedem | https://bugs.launchpad.net/cinder/+bug/1739629 | 15:27 |
| openstack | Launchpad bug 1739629 in Cinder "VolumesListTestJSON fails with "AssertionError: Could not find volumes ... in expected list"" [Undecided,New] | 15:27 |
| *** imran_ansari has joined #openstack-cinder | 15:27 | |
| openstackgerrit | Imran Ansari proposed openstack/cinder master: 3PAR - Fix temp snapshot that remains post online copy https://review.openstack.org/524185 | 15:28 |
| *** gouthamr_ has quit IRC | 15:31 | |
| mriedem | Roamer`: smcginnis: maybe a periodic task running and hitting the volume around the same time which updates the updated_at? | 15:31 |
| mriedem | logstash says this is only happening on master, so any recent changes in queens for periodic tasks? | 15:32 |
| *** Apoorva has joined #openstack-cinder | 15:34 | |
| Roamer` | mriedem, it could be, because in your log there is no informational query at the time pointed to by the first change to updated_at, so my hypothesis about the other test fetching the info is wrong | 15:38 |
| *** lkuchlan has quit IRC | 15:44 | |
| Roamer` | mriedem, smcginnis, actually I think it might be at creation time... screen-c-vol contains log entries for the volume *after* the timestamp that was first reported as "available" to tempest, and some of them are +/- 1 second from the time later reported as updated_at to another tempest query | 15:44 |
| Roamer` | it looks like the volume's updated_at changes several times during creation, and, yeah, it also changes later, too | 15:45 |
| *** avishay has joined #openstack-cinder | 15:45 | |
| *** mayur_ind has quit IRC | 15:48 | |
| *** imran_ansari has quit IRC | 15:49 | |
| *** hemna_ has joined #openstack-cinder | 15:54 | |
| *** AlexeyAbashkin has quit IRC | 15:55 | |
| *** avishay has quit IRC | 15:58 | |
| *** liverpooler has joined #openstack-cinder | 16:00 | |
| *** lpetrut has quit IRC | 16:09 | |
| *** avishay has joined #openstack-cinder | 16:10 | |
| *** itlinux has joined #openstack-cinder | 16:14 | |
| *** lpetrut has joined #openstack-cinder | 16:17 | |
| jgriffith | xyang: sorry, I didn't make any progress yesterday as there was a breakage in the wheel pkg related to latest requirements update | 16:20 |
| jgriffith | xyang: I'm trying to build again now (I think it's all fixed on the loci side now) | 16:20 |
| jgriffith | ipaddress==1.0.19 :) | 16:20 |
| jgriffith | and we're building again W00T | 16:20 |
| xyang | jgriffith: no problem. I also ran into this problem with wheel pkg. Glad that's fixed now. | 16:20 |
| xyang | jgriffith: I did some debugging into os-brick. that works. So I suspect there's a issue with the new attach api | 16:21 |
| jgriffith | xyang: yeah, that wheels change is now reverted https://review.openstack.org/#/c/529412/ | 16:22 |
| xyang | jgriffith: I wonder if you used the legacy code path or the new attach path when you tested yours | 16:22 |
| jgriffith | xyang: hmm... interesting; I have never tried using the new attach api's :) | 16:22 |
| xyang | jgriffith: great. will try that | 16:22 |
| jgriffith | xyang: and I dind't know the brick extension knew how to use it | 16:22 |
| jgriffith | but that might explain the problem | 16:22 |
| ildikov | jgriffith: if you have two minutes we're having our joyful meeting now to chat about shared_target urban legends, etc :) | 16:22 |
| xyang | jgriffith: so it appears that no one is changing the status back to in-use | 16:22 |
| jgriffith | xyang: an attachment-complete API was added that would explain what you're seeing | 16:23 |
| jgriffith | ildikov: oh.. meeting, yes I'll be right there | 16:23 |
| xyang | jgriffith: ok:) | 16:23 |
| *** chhavi__ has quit IRC | 16:23 | |
| *** lpetrut has quit IRC | 16:23 | |
| *** giodeco has joined #openstack-cinder | 16:24 | |
| *** markstur has joined #openstack-cinder | 16:27 | |
| ildikov | xyang: sorry, you'll get jgriffith back in a bit :) | 16:32 |
| xyang | ildikov: no problem:) | 16:35 |
| *** salv-orlando has joined #openstack-cinder | 16:40 | |
| jungleboyj | ildikov: Why did my singing make you nauseous ? | 16:50 |
| ildikov | jungleboyj: singing about multi-attach made me :) | 16:50 |
| jungleboyj | I thought it was appropriate! :-) | 16:50 |
| jungleboyj | "I'm dreaming of multi-attach for Christmas, just like the other vendors I know" | 16:51 |
| ildikov | fair enough :) | 16:52 |
| jungleboyj | Yep, I went there: https://twitter.com/jungleboyj/status/943887804823719936 | 16:56 |
| jgriffith | haha! | 16:56 |
| jungleboyj | :-) | 16:57 |
| xyang | jungleboyj: :) | 16:57 |
| ildikov | jungleboyj: no feature felt this welcomed ever before :) | 16:58 |
| xyang | jgriffith: I made a switch in brick cinderclient ext to use the legacy attach code path and that worked | 16:58 |
| * jungleboyj just hopes I don't end up regretting it like replication. | 16:58 | |
| xyang | https://www.irccloud.com/pastebin/TlwshTSF/ | 16:58 |
| ildikov | jungleboyj: we're all already regretting it... :) | 16:58 |
| jungleboyj | jgriffith: brick-cinderclient-ext is broken in the V3 API path. I am working on a fix for that. I have attach working. The detach is a bit baffling but I am getting there. | 16:59 |
| *** giodeco has quit IRC | 16:59 | |
| *** avishay has quit IRC | 17:03 | |
| *** pcaruana has joined #openstack-cinder | 17:04 | |
| xyang | jungleboyj: you are fixing it? excellent! send me the patch when you are done | 17:05 |
| jgriffith | jungleboyj: yeah, I just sucesfully ran with my *old* version still installed. | 17:05 |
| jgriffith | thanks for confirming! | 17:05 |
| xyang | jungleboyj: detach in the new attach api works for me though. | 17:05 |
| *** e0ne has quit IRC | 17:06 | |
| jungleboyj | xyang: Working on it. Detach isn't right in the brick-cinderclient for v3. | 17:06 |
| jgriffith | jungleboyj: let me know if you need another pair of eyes or have any questions on the detach | 17:06 |
| jungleboyj | Fixing attach was a piece of cake, the detach is going to take a little more work. I just have other fires at the moment. :-) | 17:06 |
| jungleboyj | jgriffith: Will do. | 17:07 |
| xyang | jungleboyj: ok. I manually changed the status from "attaching" to "in-use" and did a detach. It cleaned up everything for me with the new attach code :) | 17:07 |
| jungleboyj | Really? With brick-cinderclient-ext? | 17:07 |
| jungleboyj | xyang: ^^^ | 17:07 |
| xyang | jungleboyj: yes:) | 17:07 |
| *** ntpttr_laptop has joined #openstack-cinder | 17:07 | |
| xyang | jungleboyj: I'll try your new patch when it's ready | 17:08 |
| jgriffith | jungleboyj: it should be fairly simple, just `attachment-delete` | 17:08 |
| jgriffith | should do all the messy stuff for you | 17:08 |
| jungleboyj | Weird. Ok, let me look at that again here in a minute. I was working on a brick-cinderclient that I had hacked up for other stuff. I bet I broke something. | 17:08 |
| xyang | jungleboyj, jgriffith: it changed the status from in-use to available and cleaned up all the targets for me | 17:08 |
| jgriffith | xyang: but you're just using the API directly, not the "local-attach" command correct? | 17:09 |
| jungleboyj | Give me a few minutes and if it fixes things then I can have a patch up in a few. | 17:09 |
| xyang | jgriffith: I use local-detach | 17:09 |
| jgriffith | xyang: oh, nice! | 17:09 |
| xyang | jgriffith: so I used local-attach first. everything else works but the status is stuck in "attaching" . After manually changing the status to in-use in db, I did local-detach and that worked fine | 17:10 |
| jungleboyj | xyang: Ok, let me recreate that and then see if I can get it all to work. | 17:10 |
| xyang | jungleboyj: what did you change to fix the attach status? | 17:11 |
| jungleboyj | Just had to add a call to the end of '_attach' to attachment.complete(attachment_id) | 17:12 |
| *** imacdonn has quit IRC | 17:12 | |
| xyang | jungleboyj: okay, thanks. let me try that | 17:12 |
| *** imacdonn has joined #openstack-cinder | 17:12 | |
| jungleboyj | device_info = brick_connector.connect_volume(connection) | 17:12 |
| jungleboyj | self.volumes_client.attachments.complete(info['connection_info']['attachment_id']) | 17:12 |
| jungleboyj | xyang: If that works then you have the patch. | 17:12 |
| xyang | jungleboyj: thanks! | 17:13 |
| Roamer` | mriedem, smcginnis, everyone: I have to run now, I'll be back later, but what do you guys think about the workaround I proposed in https://bugs.launchpad.net/cinder/+bug/1739629/comments/1 - let Tempest ignore the updated_at field in its comparisons? | 17:15 |
| openstack | Launchpad bug 1739629 in Cinder "VolumesListTestJSON fails with "AssertionError: Could not find volumes ... in expected list"" [High,Confirmed] | 17:15 |
| mriedem | Roamer`: definitely seems like a busted test http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/volumes/test_volumes_list.py#n80 | 17:18 |
| mriedem | why doesn't it just check that the volume id is in the fetched list? | 17:18 |
| mriedem | comparing on updated_at is obviously a bad idea | 17:18 |
| mriedem | looks like there are tests specifically to compare details vs non-details, | 17:19 |
| mriedem | but i think in all cases that yes you'd want to ignore updated_at | 17:19 |
| Roamer` | mriedem, good point, too | 17:19 |
| Roamer` | mriedem (the one about just comparing the volume id, I mean)... OK, I really have to run now, ttyl | 17:19 |
| *** Apoorva has quit IRC | 17:20 | |
| *** Apoorva has joined #openstack-cinder | 17:21 | |
| xyang | jungleboyj: what api version is this attachment complete api? | 17:24 |
| jungleboyj | xyang: 3.44 | 17:24 |
| *** Apoorva has quit IRC | 17:25 | |
| xyang | jungleboyj, jgriffith: the fix works:) | 17:30 |
| jungleboyj | Awesome! I must have mucked around with something else in my environment. | 17:30 |
| xyang | both local-attach and local-detach works now after applying jungleboyj's fix. | 17:30 |
| jungleboyj | xyang: Awesome. Are you going to push that fix up then? | 17:31 |
| xyang | jungleboyj: I can if you want me to. It is your fix | 17:31 |
| *** armax has quit IRC | 17:31 | |
| jungleboyj | :-) I can do it. I want to figure out what I mucked up so it will be a good exercise. | 17:31 |
| jungleboyj | You can apporve. | 17:31 |
| xyang | jungleboyj: sure:). are you going to push it up soon? | 17:32 |
| *** armax has joined #openstack-cinder | 17:34 | |
| *** edmondsw has joined #openstack-cinder | 17:39 | |
| jungleboyj | Yeah, give me a little bit. | 17:44 |
| jungleboyj | Working on it now. | 17:44 |
| *** edmondsw has quit IRC | 17:44 | |
| xyang | jungleboyj: no problem. thanks! | 17:46 |
| mriedem | jungleboyj: https://review.openstack.org/529659 | 17:52 |
| jungleboyj | xyang: You are right! Ok. Sweet. Fix on its way. | 17:55 |
| *** chhavi__ has joined #openstack-cinder | 17:55 | |
| xyang | jungleboyj: awesome! | 17:55 |
| *** daidv has quit IRC | 17:56 | |
| *** hoangcx has quit IRC | 17:56 | |
| *** daidv has joined #openstack-cinder | 17:56 | |
| *** hoangcx has joined #openstack-cinder | 17:56 | |
| *** liverpooler has quit IRC | 17:59 | |
| *** imran_ansari has joined #openstack-cinder | 18:08 | |
| imran_ansari | smcginnis: this is regarding your review comments related to 3PAR - Fix temp snapshot that remains post online copy https://review.openstack.org/524185 | 18:10 |
| imran_ansari | I responded to your comment related to moving oslo_utils after oslo_config | 18:15 |
| hemna_ | imran_ansari, the online copy capability needs to get removed from the driver | 18:15 |
| hemna_ | I had a patch up a little while ago that did that | 18:15 |
| hemna_ | prior to the refactoring | 18:15 |
| hemna_ | imran_ansari, https://review.openstack.org/#/c/499806/ | 18:16 |
| *** harlowja has joined #openstack-cinder | 18:16 | |
| *** ntpttr_laptop has quit IRC | 18:16 | |
| openstackgerrit | Jay Bryant proposed openstack/python-brick-cinderclient-ext master: Update to support new attachment API mv 3.44 https://review.openstack.org/529668 | 18:17 |
| jungleboyj | xyang: ^^ | 18:18 |
| xyang | jungleboyj: see my comments:) | 18:18 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder master: StorageCenter: Fix volume mapping for API v3.1 https://review.openstack.org/529669 | 18:18 |
| xyang | jungleboyj: need a rebase? | 18:19 |
| imran_ansari | smcginnis: Could you please have a look at my response related to moving the import down and suggest what should be done? | 18:19 |
| xyang | jgriffith: do you know if the gophercloud support the new attachment APIs yet? | 18:20 |
| *** catintheroof has quit IRC | 18:22 | |
| imran_ansari | hemna_: William is our Tech Lead - IMHO it would help if you two could have a call to conclude the online copy issue. | 18:24 |
| imran_ansari | pbrush | 18:26 |
| imran_ansari | Sorry | 18:27 |
| jgriffith | xyang: nope, it does not | 18:27 |
| jgriffith | xyang: handling mv's is a bit of a task | 18:27 |
| jungleboyj | xyang: OMG. One of those days. WTF. | 18:29 |
| xyang | jgriffith: okay | 18:29 |
| jungleboyj | xyang: Fixing. | 18:29 |
| xyang | jungleboyj: no problem | 18:29 |
| jungleboyj | The VM I did the work on whas having git/gerrit issues that I didn't want to deal with so I just SCPd the file and apparently messed that up. | 18:29 |
| xyang | jungleboyj: :) | 18:30 |
| openstackgerrit | Jay Bryant proposed openstack/python-brick-cinderclient-ext master: Update to support new attachment API mv 3.44 https://review.openstack.org/529668 | 18:32 |
| jungleboyj | That looks better. | 18:32 |
| * jungleboyj is embarrassed | 18:32 | |
| xyang | jgriffith: I assume we have to wait until all multiattach issues have been resolved before adding the new attach api support there? | 18:32 |
| *** e0ne has joined #openstack-cinder | 18:33 | |
| jgriffith | xyang: nah, the problem there is figuring out a good strategy for supporting mv's | 18:34 |
| jgriffith | xyang: you can browse some history here: https://github.com/gophercloud/gophercloud/issues/412 | 18:34 |
| xyang | jgriffith: oh, I thought you mean multiattach:) | 18:35 |
| jgriffith | xyang: oh... adding new attach API's to gophercloud? meh... not really but might as well | 18:36 |
| xyang | jgriffith: yes. I thought V3 API is already supported there | 18:36 |
| *** chhavi__ has quit IRC | 18:37 | |
| jgriffith | V3 is, but MV's are still a bit of a challenge | 18:37 |
| xyang | jgriffith: I just couldn't find the new attachments apis yet | 18:37 |
| xyang | jgriffith: ok | 18:37 |
| jgriffith | xyang: in other words V3==V2 so there ya go | 18:38 |
| xyang | jgriffith: so any new APIs added in V3 won't work yet, not just attach api? | 18:39 |
| jgriffith | xyang: that would be correct | 18:39 |
| xyang | jgriffith: but things under contrib will work, right? | 18:39 |
| jgriffith | xyang: and then keep in mind even if you pull one of my branches with an MV in in it; you then need to propogate through to the provider | 18:39 |
| jgriffith | xyang: yup, contrib is good | 18:40 |
| xyang | jgriffith: ok | 18:40 |
| jgriffith | xyang: although I suspect that's actually what you're working on :) | 18:40 |
| jgriffith | for OSDS | 18:40 |
| xyang | jgriffith: I have not tested that part. my co-worker said he used the new attachment api. I'll have to double check how that is possible:) | 18:41 |
| jgriffith | xyang: Yeah, love to know if there was some way that was done through GC | 18:41 |
| jgriffith | xyang: I suspect maybe she just *thought* she did :) | 18:41 |
| xyang | jgriffith: I'll find out and let you know:) | 18:42 |
| *** catintheroof has joined #openstack-cinder | 18:47 | |
| *** rmcall has joined #openstack-cinder | 18:48 | |
| e0ne | jungleboyj, jgriffith, xyang: please, let me know if you need any help with brick-cinderclient-ext | 18:50 |
| e0ne | I'm ready to help and/or fix it if needed | 18:50 |
| jungleboyj | e0ne: Will do. I have been getting to know it pretty well though. :-) | 18:50 |
| xyang | e0ne: hi, thanks! you are still up | 18:50 |
| e0ne | xyang: hi. it's only 9pm:) | 18:51 |
| *** crose has quit IRC | 18:51 | |
| xyang | e0ne: only 9pm:). right, very early working hour | 18:51 |
| xyang | e0ne: will ping you next time when I'm stuck with brick-cinderclient-ext! | 18:52 |
| e0ne | xyang: ok | 18:52 |
| *** imran_ansari has quit IRC | 18:57 | |
| jungleboyj | Man, I managed to get some of my volumes in bad state trying to switch between the two different API versions. | 18:58 |
| xyang | jungleboyj: :( | 18:59 |
| jungleboyj | Something to look into in the future. | 18:59 |
| *** gouthamr has quit IRC | 19:02 | |
| jgriffith | jungleboyj: | 19:05 |
| *** Apoorva has joined #openstack-cinder | 19:05 | |
| jungleboyj | jgriffith: | 19:05 |
| jgriffith | sorry, got disconnected :) | 19:06 |
| jungleboyj | jgriffith: The anticipation is killing me. | 19:06 |
| jgriffith | so I'd be curious about what you ran into there; I actually had done a fair bit of work to make that scenario work | 19:07 |
| jgriffith | there have been a number of changes since then however that may require some work added back to it | 19:07 |
| jgriffith | jungleboyj: the attachment-delete was a sledge-hammer that pretty much worked all the time regardless of how the attachment was created (old vs new) | 19:08 |
| jgriffith | there may be some additional steps now though, I'd be curious what you saw | 19:08 |
| jungleboyj | When I went back to trying to use V2 I couldn't delete because it things the volume still had attachments. | 19:09 |
| jgriffith | jungleboyj: oh, well I make no efforts to go from V3 back to V2 | 19:10 |
| jgriffith | although, if you run begin-detaching, terminate it should probably be possible | 19:10 |
| jungleboyj | jgriffith: Yeah. I am thinking that wasn't a good idea. :-) | 19:10 |
| jgriffith | actually, that was a use case I had at one point because of multiple API servers rolling upgrades etc | 19:11 |
| jgriffith | What problem did you see? | 19:11 |
| jgriffith | jungleboyj: "cinder delete <volume-x>" didn't work even though it was available because the attachment-status was still "attached"? | 19:12 |
| jgriffith | jungleboyj: I'd be curious of the steps you used if you happen to remember them | 19:13 |
| jgriffith | since we now allow a disconnect that keeps a volume "attached" the model is probably going to need rethinking | 19:13 |
| jungleboyj | jgriffith: Yes, that was what happened. | 19:13 |
| jgriffith | hmm | 19:14 |
| xyang | jungleboyj: pep8:) | 19:14 |
| jungleboyj | WHAT!?! | 19:14 |
| xyang | jungleboyj: line too long:) | 19:14 |
| jgriffith | jungleboyj: she said "pep8" | 19:14 |
| xyang | jungleboyj: I meant your patch | 19:14 |
| jungleboyj | jgriffith: Smart Ass. | 19:15 |
| jungleboyj | xyang: Look at me making all the mistakes I yell at people for. | 19:15 |
| xyang | jungleyboyj: sorry, I didn't mean to scare you:) | 19:16 |
| jungleboyj | xyang: You didn't scare me. Just annoyed with myself. | 19:18 |
| xyang | jungleboyj: :) | 19:18 |
| openstackgerrit | Jay Bryant proposed openstack/python-brick-cinderclient-ext master: Update to support new attachment API mv 3.44 https://review.openstack.org/529668 | 19:18 |
| *** rmcall has quit IRC | 19:19 | |
| * jungleboyj crosses fingers that I didn't do something else stupid. | 19:19 | |
| jungleboyj | Don't suppose the fact I had a needle in my spine at this point yesterday is a valid excuse? | 19:20 |
| *** dave-mcc_ has quit IRC | 19:21 | |
| e0ne | jungleboyj: unit tests? | 19:22 |
| jungleboyj | e0ne: Don't have time to get that in today. | 19:23 |
| e0ne | fair enouph | 19:23 |
| jungleboyj | :-) Can add those after I get this demo done for work. | 19:24 |
| jgriffith | effing attachment-complete | 19:25 |
| jgriffith | sigh | 19:25 |
| jungleboyj | A complete mess. ;-) | 19:25 |
| jgriffith | mistakes were made | 19:26 |
| *** edmondsw has joined #openstack-cinder | 19:28 | |
| jgriffith | sorry jungleboyj https://review.openstack.org/#/c/529668/3 | 19:29 |
| jungleboyj | Oh, good catch. I didn't even know that was in there. | 19:29 |
| jungleboyj | jgriffith: Fixing. | 19:30 |
| jgriffith | https://github.com/openstack/cinder/blob/master/cinder/api/microversions.py#L128 | 19:30 |
| jgriffith | jungleboyj: that's what I was complaining about :) | 19:30 |
| jgriffith | although the old 3.27 should work without it though, not sure what happened there | 19:30 |
| jgriffith | doesn't matter, moving forward | 19:30 |
| jgriffith | ildikov: jungleboyj ok, caught up; grabbing lunch and I'll get that spec updated | 19:32 |
| *** edmondsw has quit IRC | 19:33 | |
| *** armaan has joined #openstack-cinder | 19:46 | |
| *** armaan has quit IRC | 19:49 | |
| *** diablo_rojo has quit IRC | 19:51 | |
| openstackgerrit | Jay Bryant proposed openstack/python-brick-cinderclient-ext master: Update to support new attachment API mv 3.44 https://review.openstack.org/529668 | 19:52 |
| jungleboyj | ildikov: Which spec? | 19:53 |
| *** gmann has quit IRC | 19:57 | |
| *** Apoorva has quit IRC | 19:58 | |
| *** e0ne_ has joined #openstack-cinder | 20:05 | |
| *** e0ne has quit IRC | 20:08 | |
| *** pcaruana has quit IRC | 20:12 | |
| ildikov | jungleboyj: I was harassing you with 3.48... :) | 20:12 |
| ildikov | jungleboyj: do you need the link again? | 20:13 |
| *** amoralej is now known as amoralej|off | 20:14 | |
| jgriffith | jungleboyj: one last thing on that extension patch :) | 20:18 |
| jgriffith | jungleboyj: do we have some documentation on how that sucks in the requested version from the client settings and for example regardless of supported version allows you to specify a lower version? | 20:19 |
| jgriffith | because it *does* allow that right :) | 20:19 |
| jgriffith | and if there isn't docs explaining that, there probably should be don't ya think? | 20:19 |
| jungleboyj | jgriffith: I am not sure if there are docks. | 20:20 |
| jgriffith | Or I guess more of the use the default 3.0 unless I specify a 3.xx version | 20:20 |
| jgriffith | docks? What are you on a boat? :) | 20:20 |
| jgriffith | just kidding | 20:20 |
| jungleboyj | Looking at the snow covered lake. | 20:20 |
| jungleboyj | Could use a doc. | 20:20 |
| jgriffith | hehe... | 20:20 |
| jungleboyj | ildikov: Yes, I need a link again. | 20:20 |
| * jgriffith is kinda jealous, that sounds picturesque | 20:20 | |
| *** ntpttr_laptop has joined #openstack-cinder | 20:21 | |
| jungleboyj | ildikov: My brain is going in many directions today. | 20:21 |
| jgriffith | Anyway... maybe something we should open a bug on etc; adding some usage docs to the brick ext; and making sure we get base version by default | 20:21 |
| jgriffith | probably needs some tests somewhere too | 20:22 |
| smcginnis | Docks and kubes. | 20:22 |
| jgriffith | hehe | 20:22 |
| jungleboyj | jgriffith: http://www.ustream.tv/channel/nw-rochester-mn-weather-feed | 20:23 |
| jgriffith | jungleboyj: I thought you were down in Raleigh? | 20:23 |
| jungleboyj | No, I just work for Raleigh. Still live in Rochester. | 20:24 |
| jgriffith | Well I knew that, just thought you were on a trip this week :) | 20:24 |
| jungleboyj | No, that is the beginning of January. | 20:24 |
| jgriffith | Looks like a nice view for a Winter Solstice in Minnesota | 20:24 |
| jungleboyj | https://www.wunderground.com/personal-weather-station/dashboard?ID=KMNORONO7#history | 20:25 |
| ildikov | jungleboyj: https://review.openstack.org/#/c/523608/ | 20:25 |
| smcginnis | Oh right, sun should be going down any minute now. :) | 20:25 |
| jungleboyj | Yeah. | 20:25 |
| jgriffith | ha! | 20:25 |
| jgriffith | That would be an exceptionally short day | 20:25 |
| jungleboyj | jgriffith: Yes, we definitely need some documentation on the brick-cinderclient-ext | 20:25 |
| jgriffith | if it was that bad I think I'd just jump off a bridge | 20:25 |
| jungleboyj | jgriffith: Same here. | 20:26 |
| jgriffith | not a very tall one though | 20:26 |
| smcginnis | jgriffith: Just give me until mid-February. | 20:26 |
| jgriffith | and into a warm lazily flowing river | 20:26 |
| jungleboyj | ildikov: I am waiting for jgriffith to update that patch. | 20:26 |
| jgriffith | smcginnis: well, hopefully you'll get to travel to better places for work and stave off the tempation | 20:26 |
| jungleboyj | smcginnis: And you are down in the basement. | 20:26 |
| ildikov | jungleboyj: then why are you asking me for the link? Or you asked for another one? | 20:27 |
| jungleboyj | ildikov: Why are you asking me to have the spec done? | 20:28 |
| jungleboyj | :-) | 20:28 |
| jungleboyj | I am so confused. | 20:28 |
| smcginnis | jgriffith: Dublin is the next work trip, at least that I know about so far. But New Years in the Pacific Northwest, so that should be fun. | 20:28 |
| jgriffith | cold and gray, to less cold but more gray, to colder and grayer | 20:29 |
| jgriffith | That should help :P | 20:29 |
| *** pcaruana has joined #openstack-cinder | 20:29 | |
| smcginnis | Heh, pretty much. | 20:29 |
| *** AlexeyAbashkin has joined #openstack-cinder | 20:29 | |
| ildikov | jungleboyj: you asked me for the spec link I just sent it :) | 20:30 |
| ildikov | jgriffith: smcginnis: Dublin/Ireland is awesome! :) | 20:31 |
| *** AlexeyAbashkin has quit IRC | 20:33 | |
| smcginnis | ildikov: I do really like Dublin. I'm looking forward to it, even though it will be February. | 20:34 |
| *** catintheroof has quit IRC | 20:37 | |
| ildikov | smcginnis: I was there at the same time as the PTG will be and it was pretty nice, very rainbowy :) | 20:38 |
| *** catintheroof has joined #openstack-cinder | 20:38 | |
| smcginnis | I've been there in March and I think April. You would think I would go somewhere warm in the winter, but somehow I always stay cold. | 20:39 |
| *** owalsh_ has joined #openstack-cinder | 20:39 | |
| ildikov | smcginnis: :) | 20:41 |
| *** pcaruana has quit IRC | 20:42 | |
| *** owalsh has quit IRC | 20:42 | |
| *** catintheroof has quit IRC | 20:42 | |
| jungleboyj | I can't wait for Dublin. | 20:49 |
| *** diablo_rojo has joined #openstack-cinder | 20:51 | |
| openstackgerrit | John Griffith proposed openstack/cinder-specs master: Propose multi-attach using V3 Attachment API's https://review.openstack.org/523608 | 21:00 |
| jgriffith | mriedem: ildikov jungleboyj smcginnis ^^ | 21:00 |
| mriedem | ack, after nova meeting | 21:00 |
| jungleboyj | jgriffith: Yep, will take a look after I get the boys from school. | 21:01 |
| jungleboyj | jgriffith: Somehow I ended up with a volume like this: | 21:02 |
| jungleboyj | 6d015fd8-86da-4412-9d06-003ef959c8d5 | in-use | LenovoVol | 5 | lenovo-b | false | e71677c5-7da4-482a-892a-26d04c6789fd,380ebc44-0b13-4f26-9e39-a11cc03d00f4 | 21:02 |
| jungleboyj | Says it is attached to two locations but it should only be one. | 21:02 |
| jungleboyj | I was using a heavily hacked brick-cinderclient-ext though so god knows what I had wrong in there. | 21:03 |
| ildikov | jgriffith: thank you! | 21:03 |
| jgriffith | jungleboyj: nope :) | 21:03 |
| jgriffith | jungleboyj: attachments are disposable and free | 21:04 |
| jgriffith | jungleboyj: if you call attach-create again, you get another | 21:04 |
| jgriffith | nothing prevents that | 21:04 |
| jungleboyj | jgriffith: Yeah, I think that is what happened there. | 21:04 |
| jgriffith | and also warning; the last one wins | 21:04 |
| jgriffith | I believe there's a bug for that out there that I'm supposed to fix some day | 21:05 |
| ildikov | jgriffith: you mean having two creates work? | 21:06 |
| jgriffith | ildikov: yeah | 21:08 |
| jgriffith | ildikov: becuase they're completely independent things | 21:08 |
| jgriffith | ildikov: but in the case of non multiattach the first one just gets implicitly *disconnected* | 21:09 |
| ildikov | We need that in Nova if it's the same instance to leave it reserved | 21:09 |
| jgriffith | maybe | 21:09 |
| jgriffith | I don't know | 21:09 |
| ildikov | And it's now fixed in Nova I believe to not have it attached twice | 21:10 |
| ildikov | By checking the BDM | 21:10 |
| ildikov | In the attach flow and we said it could be checked in update in Cinder | 21:12 |
| ildikov | jgriffith: ^^ | 21:12 |
| jgriffith | sure | 21:13 |
| openstackgerrit | John Griffith proposed openstack/cinder-specs master: Propose multi-attach using V3 Attachment API's https://review.openstack.org/523608 | 21:15 |
| *** e0ne has joined #openstack-cinder | 21:16 | |
| ildikov | jgriffith: good then :) | 21:16 |
| *** e0ne_ has quit IRC | 21:17 | |
| openstackgerrit | Merged openstack/python-brick-cinderclient-ext master: Update to support new attachment API mv 3.44 https://review.openstack.org/529668 | 21:18 |
| *** gouthamr has joined #openstack-cinder | 21:22 | |
| *** Apoorva has joined #openstack-cinder | 21:24 | |
| *** eharney has quit IRC | 21:29 | |
| *** e0ne has quit IRC | 21:34 | |
| *** threestrands_ has joined #openstack-cinder | 21:35 | |
| *** e0ne has joined #openstack-cinder | 21:39 | |
| *** e0ne has quit IRC | 21:40 | |
| openstackgerrit | Sean McGinnis proposed openstack/python-cinderclient master: Deprecate multiattach flag on volume create https://review.openstack.org/529375 | 21:52 |
| mriedem | ok comments in the spec https://review.openstack.org/#/c/523608/ | 21:58 |
| smcginnis | Oui oui | 21:59 |
| mriedem | WEE!!! | 22:00 |
| openstackgerrit | John Griffith proposed openstack/cinder master: Enable fail back in SolidFire driver https://review.openstack.org/529182 | 22:01 |
| mriedem | smcginnis: wonder if you want to use DeprecationWarning in https://review.openstack.org/#/c/529375/ | 22:02 |
| smcginnis | mriedem: Meh | 22:03 |
| smcginnis | mriedem: Would it buy us anything really? | 22:05 |
| *** markstur has quit IRC | 22:05 | |
| mriedem | i just think that's what the oslo deprecation utils use | 22:06 |
| mriedem | so that's where i know it | 22:06 |
| *** diablo_rojo has quit IRC | 22:07 | |
| mriedem | http://git.openstack.org/cgit/openstack/debtcollector/tree/debtcollector/removals.py#n66 | 22:07 |
| smcginnis | I guess if someone sets warnings.simplefilter('once', DeprecationWarning) it might help. | 22:07 |
| mriedem | you'd have to ask someone more pythonic than myself | 22:07 |
| mriedem | like dhellmann | 22:07 |
| openstackgerrit | John Griffith proposed openstack/cinder-specs master: Propose multi-attach using V3 Attachment API's https://review.openstack.org/523608 | 22:08 |
| openstackgerrit | Sean McGinnis proposed openstack/python-cinderclient master: Deprecate multiattach flag on volume create https://review.openstack.org/529375 | 22:09 |
| *** dustins has quit IRC | 22:11 | |
| openstackgerrit | Matt Riedemann proposed openstack/cinder-specs master: Propose multi-attach using V3 Attachment API's https://review.openstack.org/523608 | 22:13 |
| *** diablo_rojo has joined #openstack-cinder | 22:14 | |
| mriedem | smcginnis: looks like the releasenotes job failed due to this http://logs.openstack.org/75/529375/3/check/build-openstack-releasenotes/6e198c3/job-output.txt.gz#_2017-12-21_22_12_35_564720 | 22:17 |
| mriedem | which is...odd | 22:17 |
| smcginnis | mriedem: Jobs were changed to not install a project to run release notes. I thought the fix for that went through in cinderclient. Let me check. | 22:21 |
| smcginnis | This will need to get through first. | 22:22 |
| smcginnis | https://review.openstack.org/#/c/528997/ | 22:22 |
| smcginnis | Actually, this one first: https://review.openstack.org/#/c/520645/ | 22:23 |
| smcginnis | jungleboyj, jgriffith: hint, hint ^^ | 22:23 |
| *** rcernin has joined #openstack-cinder | 22:24 | |
| *** e0ne has joined #openstack-cinder | 22:24 | |
| *** mchlumsky has quit IRC | 22:33 | |
| *** dalgaaf has quit IRC | 22:34 | |
| *** dalgaaf has joined #openstack-cinder | 22:36 | |
| *** catintheroof has joined #openstack-cinder | 22:37 | |
| *** catintheroof has quit IRC | 22:37 | |
| *** e0ne has quit IRC | 22:39 | |
| *** salv-orlando has quit IRC | 22:49 | |
| *** diablo_rojo has quit IRC | 22:49 | |
| *** salv-orlando has joined #openstack-cinder | 22:51 | |
| *** salv-orlando has quit IRC | 22:54 | |
| *** edmondsw has joined #openstack-cinder | 23:04 | |
| *** stvnoyes has quit IRC | 23:06 | |
| ildikov | jungleboyj: so I believe this is waiting for you now: https://review.openstack.org/#/c/523608/ | 23:07 |
| *** crushil has joined #openstack-cinder | 23:08 | |
| ildikov | jgriffith: thanks for the updates on the spec | 23:08 |
| jungleboyj | ildikov: In the coffee shop reading it now. :-) | 23:08 |
| *** edmondsw has quit IRC | 23:08 | |
| ildikov | jgriffith: once it gets merged, hopefully very soon, if you have anything I can use as a starting point I would be very thankful if you could share that :) | 23:09 |
| ildikov | jungleboyj: nice! :) thank you | 23:09 |
| jungleboyj | ildikov: It is off to the Gate. :-) | 23:15 |
| jungleboyj | and the people rejoiced. | 23:15 |
| jungleboyj | jgriffith: Thanks for all the updates in there. | 23:15 |
| *** crushil has quit IRC | 23:16 | |
| ildikov | jungleboyj: saw it, thank you | 23:17 |
| jungleboyj | ildikov: Welcome. Thank you! | 23:18 |
| openstackgerrit | Merged openstack/cinder-specs master: Propose multi-attach using V3 Attachment API's https://review.openstack.org/523608 | 23:22 |
| *** AlexeyAbashkin has joined #openstack-cinder | 23:34 | |
| *** AlexeyAbashkin has quit IRC | 23:39 | |
| *** salv-orlando has joined #openstack-cinder | 23:55 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!