| *** ddixit has quit IRC | 00:03 | |
| *** itlinux has quit IRC | 00:08 | |
| *** yangyape_ has quit IRC | 00:09 | |
| *** yangyapeng has joined #openstack-cinder | 00:10 | |
| *** yangyapeng has quit IRC | 00:14 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/503206 | 00:17 |
|---|---|---|
| *** abhitechie has joined #openstack-cinder | 00:18 | |
| *** ddixit has joined #openstack-cinder | 00:23 | |
| *** ddixit has quit IRC | 00:28 | |
| *** crushil_ has joined #openstack-cinder | 00:28 | |
| *** ddixit has joined #openstack-cinder | 00:30 | |
| *** Shunli has joined #openstack-cinder | 00:31 | |
| *** mikal_ has quit IRC | 00:32 | |
| *** mikal has joined #openstack-cinder | 00:34 | |
| *** crushil_ has quit IRC | 00:38 | |
| *** zhurong has joined #openstack-cinder | 00:41 | |
| *** raunak has quit IRC | 00:50 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: Support count info in List&Detail APIs https://review.openstack.org/505101 | 00:50 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/503206 | 00:57 |
| *** Apoorva_ has joined #openstack-cinder | 00:57 | |
| openstackgerrit | yenai proposed openstack/cinder master: capacity filter: fix scheduler failed https://review.openstack.org/507325 | 01:00 |
| *** Apoorva has quit IRC | 01:00 | |
| *** Apoorva_ has quit IRC | 01:02 | |
| *** yangyapeng has joined #openstack-cinder | 01:12 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/503206 | 01:13 |
| *** ddixit_ has joined #openstack-cinder | 01:16 | |
| *** ddixit_ has quit IRC | 01:16 | |
| zengyingzhe_ | Hi all, I see the jenkins check of stable/pike and driverfixes/ocata branches keep failing, is there something wrong with them? | 01:16 |
| *** ddixit has quit IRC | 01:19 | |
| *** yangyapeng has quit IRC | 01:40 | |
| *** yangyapeng has joined #openstack-cinder | 01:40 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: [policy in code] Add support for attachment resource https://review.openstack.org/506976 | 01:41 |
| *** tpsilva has quit IRC | 01:42 | |
| *** ddixit has joined #openstack-cinder | 01:45 | |
| *** crushil has quit IRC | 01:50 | |
| *** yangyapeng has quit IRC | 01:50 | |
| *** yangyapeng has joined #openstack-cinder | 01:51 | |
| *** wanghao_ has quit IRC | 01:51 | |
| *** wanghao has joined #openstack-cinder | 01:51 | |
| *** gcb has joined #openstack-cinder | 01:52 | |
| *** yangyape_ has joined #openstack-cinder | 01:56 | |
| *** yangyapeng has quit IRC | 01:56 | |
| *** wanghao_ has joined #openstack-cinder | 01:58 | |
| *** wanghao has quit IRC | 02:02 | |
| openstackgerrit | Merged openstack/cinder master: Check for outstanding attachments during reserve https://review.openstack.org/504467 | 02:03 |
| *** raunak has joined #openstack-cinder | 02:10 | |
| *** yangyape_ has quit IRC | 02:12 | |
| *** yangyapeng has joined #openstack-cinder | 02:13 | |
| *** yangyapeng has quit IRC | 02:15 | |
| *** yangyapeng has joined #openstack-cinder | 02:15 | |
| *** yangyapeng has quit IRC | 02:20 | |
| *** abhitechie has quit IRC | 02:21 | |
| *** abhitechie has joined #openstack-cinder | 02:21 | |
| *** abhinavtechie has joined #openstack-cinder | 02:22 | |
| openstackgerrit | wanghao proposed openstack/cinder-specs master: Report backend state in service list https://review.openstack.org/252921 | 02:23 |
| *** abhitechie has quit IRC | 02:26 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/503206 | 02:28 |
| openstackgerrit | wanghao proposed openstack/cinder master: Support to query volume filter by changes since/before https://review.openstack.org/352688 | 02:32 |
| *** wanghao_ has quit IRC | 02:33 | |
| *** wanghao has joined #openstack-cinder | 02:34 | |
| *** wanghao_ has joined #openstack-cinder | 02:35 | |
| *** wanghao has quit IRC | 02:38 | |
| *** lhx__ has joined #openstack-cinder | 02:40 | |
| *** erlon has quit IRC | 02:46 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/503206 | 02:48 |
| *** bhagyashris has joined #openstack-cinder | 02:54 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 02:54 | |
| *** pooja_jadhav has joined #openstack-cinder | 02:54 | |
| *** bhagyashris has quit IRC | 02:56 | |
| *** bhagyashris has joined #openstack-cinder | 02:56 | |
| *** ps_jadhav has joined #openstack-cinder | 02:57 | |
| *** Dinesh_Bhor has quit IRC | 02:57 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 02:57 | |
| *** mikal has quit IRC | 02:59 | |
| *** pooja_jadhav has quit IRC | 02:59 | |
| *** gnufied has quit IRC | 03:00 | |
| *** yangyapeng has joined #openstack-cinder | 03:01 | |
| *** mikal has joined #openstack-cinder | 03:02 | |
| *** yangyape_ has joined #openstack-cinder | 03:03 | |
| *** yangyapeng has quit IRC | 03:07 | |
| *** dave-mcc_ has quit IRC | 03:09 | |
| openstackgerrit | lihaijing proposed openstack/cinder master: Api-ref: fix v2/v3 hosts extension api doc https://review.openstack.org/477798 | 03:12 |
| *** crushil has joined #openstack-cinder | 03:14 | |
| *** crushil has quit IRC | 03:17 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: [policy in code] Add support for attachment resource https://review.openstack.org/506976 | 03:19 |
| *** itlinux has joined #openstack-cinder | 03:19 | |
| *** ps_jadhav has quit IRC | 03:20 | |
| *** bhagyashris has quit IRC | 03:20 | |
| *** itlinux has quit IRC | 03:22 | |
| *** itlinux has joined #openstack-cinder | 03:23 | |
| *** Dinesh_Bhor has quit IRC | 03:24 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/503206 | 03:24 |
| *** pooja_jadhav has joined #openstack-cinder | 03:27 | |
| *** bhagyashris has joined #openstack-cinder | 03:27 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 03:28 | |
| *** pooja_jadhav has quit IRC | 03:30 | |
| *** Apoorva has joined #openstack-cinder | 03:30 | |
| *** bhagyashris has quit IRC | 03:34 | |
| *** Dinesh_Bhor has quit IRC | 03:34 | |
| *** itlinux has quit IRC | 03:35 | |
| *** gouthamr has quit IRC | 03:35 | |
| *** itlinux has joined #openstack-cinder | 03:38 | |
| *** chhavi has joined #openstack-cinder | 03:38 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 03:46 | |
| *** bhagyashris has joined #openstack-cinder | 03:47 | |
| *** pooja_jadhav has joined #openstack-cinder | 03:49 | |
| *** itlinux has quit IRC | 04:00 | |
| *** gkadam has joined #openstack-cinder | 04:02 | |
| *** lkuchlan has joined #openstack-cinder | 04:02 | |
| *** mtreinish has quit IRC | 04:24 | |
| *** psachin has joined #openstack-cinder | 04:26 | |
| *** mtreinish has joined #openstack-cinder | 04:34 | |
| *** Apoorva has quit IRC | 04:36 | |
| *** Apoorva has joined #openstack-cinder | 04:37 | |
| *** bhagyashri_s has joined #openstack-cinder | 04:37 | |
| *** crushil has joined #openstack-cinder | 04:38 | |
| *** abhitechie has joined #openstack-cinder | 04:38 | |
| *** bhagyashris has quit IRC | 04:40 | |
| *** Apoorva has quit IRC | 04:41 | |
| *** abhinavtechie has quit IRC | 04:41 | |
| *** Apoorva has joined #openstack-cinder | 04:45 | |
| *** gcb has quit IRC | 04:48 | |
| openstackgerrit | Merged openstack/cinder master: Api-ref: fix v2/v3 hosts extension api doc https://review.openstack.org/477798 | 05:00 |
| *** Apoorva has quit IRC | 05:02 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Compact Newton database migrations https://review.openstack.org/507298 | 05:03 |
| *** Apoorva has joined #openstack-cinder | 05:05 | |
| *** Apoorva has quit IRC | 05:05 | |
| *** Apoorva has joined #openstack-cinder | 05:08 | |
| *** yangyape_ has quit IRC | 05:09 | |
| *** ganso has quit IRC | 05:09 | |
| *** yangyapeng has joined #openstack-cinder | 05:09 | |
| *** mriedem has quit IRC | 05:12 | |
| *** yangyapeng has quit IRC | 05:14 | |
| *** Apoorva has quit IRC | 05:18 | |
| *** gcb has joined #openstack-cinder | 05:24 | |
| *** crushil_ has joined #openstack-cinder | 05:27 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Add indexes to SQLAlchemy models https://review.openstack.org/507288 | 05:29 |
| *** crushil has quit IRC | 05:29 | |
| *** abhitechie has quit IRC | 05:30 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Compact Newton database migrations https://review.openstack.org/507298 | 05:32 |
| *** felipemonteiro has joined #openstack-cinder | 05:35 | |
| *** chhavi has quit IRC | 05:40 | |
| *** chhavi_ has joined #openstack-cinder | 05:40 | |
| *** lhx__ has quit IRC | 05:52 | |
| openstackgerrit | Vivek Soni proposed openstack/cinder master: 3PAR: Cinder volume revert to snapshot support https://review.openstack.org/506072 | 05:52 |
| *** ChubYann has quit IRC | 05:55 | |
| openstackgerrit | Vivek Soni proposed openstack/cinder master: 3PAR: Cinder volume revert to snapshot support https://review.openstack.org/506072 | 05:58 |
| *** armax has joined #openstack-cinder | 05:59 | |
| *** felipemonteiro has quit IRC | 06:00 | |
| openstackgerrit | TommyLike proposed openstack/cinder-specs master: Update backup's size when backup is created https://review.openstack.org/506141 | 06:00 |
| *** zhaochao has quit IRC | 06:02 | |
| *** pgadiya has joined #openstack-cinder | 06:04 | |
| *** zhaochao has joined #openstack-cinder | 06:10 | |
| openstackgerrit | Dongdong Pei proposed openstack/cinder master: Fix calculate_virtual_free_capacity https://review.openstack.org/507359 | 06:14 |
| chhavi_ | Request to any of the core review to please review this: https://review.openstack.org/#/c/504482/ | 06:15 |
| *** raunak has quit IRC | 06:16 | |
| *** pcaruana has joined #openstack-cinder | 06:20 | |
| *** zhurong has quit IRC | 06:22 | |
| *** arnewiebalck_ has joined #openstack-cinder | 06:23 | |
| *** zhaochao has quit IRC | 06:30 | |
| *** zhaochao has joined #openstack-cinder | 06:36 | |
| *** lkuchlan has quit IRC | 06:38 | |
| *** lhx__ has joined #openstack-cinder | 06:43 | |
| *** crushil_ has quit IRC | 06:45 | |
| openstackgerrit | Merged openstack/cinder master: Updated from global requirements https://review.openstack.org/503206 | 06:45 |
| *** rcernin has joined #openstack-cinder | 06:47 | |
| *** markstur has quit IRC | 06:50 | |
| *** markstur has joined #openstack-cinder | 06:51 | |
| *** diablo_rojo has quit IRC | 06:53 | |
| *** markstur has quit IRC | 06:55 | |
| *** lhx__ has quit IRC | 07:00 | |
| *** lhx__ has joined #openstack-cinder | 07:00 | |
| *** gkadam has quit IRC | 07:02 | |
| *** gkadam has joined #openstack-cinder | 07:02 | |
| *** belmoreira has joined #openstack-cinder | 07:11 | |
| *** tesseract has joined #openstack-cinder | 07:14 | |
| openstackgerrit | Dinesh Bhor proposed openstack/cinder-specs master: Add json schema validation for V3 API's https://review.openstack.org/507386 | 07:16 |
| *** belmoreira has quit IRC | 07:19 | |
| *** abhi has joined #openstack-cinder | 07:21 | |
| *** e0ne has joined #openstack-cinder | 07:22 | |
| *** lkuchlan has joined #openstack-cinder | 07:24 | |
| openstackgerrit | TommyLike proposed openstack/cinder-specs master: Inspection Mechanism For Capacity Limited Host https://review.openstack.org/308869 | 07:25 |
| *** belmoreira has joined #openstack-cinder | 07:26 | |
| *** armax has quit IRC | 07:29 | |
| *** belmoreira has quit IRC | 07:29 | |
| *** markstur has joined #openstack-cinder | 07:32 | |
| *** markstur has quit IRC | 07:37 | |
| *** belmoreira has joined #openstack-cinder | 07:45 | |
| *** lkuchlan_ has joined #openstack-cinder | 07:46 | |
| *** lkuchlan has quit IRC | 07:47 | |
| openstackgerrit | TommyLike proposed openstack/cinder-specs master: Proposal to count resources to check quota in API https://review.openstack.org/496562 | 07:57 |
| *** belmoreira has quit IRC | 07:59 | |
| *** alexchadin has joined #openstack-cinder | 08:00 | |
| *** markvoelker has quit IRC | 08:06 | |
| *** alexchadin has quit IRC | 08:08 | |
| *** markstur has joined #openstack-cinder | 08:14 | |
| *** markstur has quit IRC | 08:19 | |
| *** lkuchlan_ is now known as lkuchlan | 08:21 | |
| *** zhaochao has quit IRC | 08:21 | |
| *** zhurong has joined #openstack-cinder | 08:21 | |
| *** mvk has quit IRC | 08:26 | |
| *** Shunli has quit IRC | 08:27 | |
| *** Shunli has joined #openstack-cinder | 08:28 | |
| *** belmoreira has joined #openstack-cinder | 08:29 | |
| *** belmoreira has quit IRC | 08:30 | |
| *** pgadiya has quit IRC | 08:30 | |
| *** alexchadin has joined #openstack-cinder | 08:31 | |
| *** zhaochao has joined #openstack-cinder | 08:40 | |
| kwathore | @Team: Requesting you to review below cherry pick: | 08:44 |
| kwathore | https://review.openstack.org/#/c/507366/ | 08:44 |
| kwathore | jenkin CI are passed..:) | 08:45 |
| *** pgadiya has joined #openstack-cinder | 08:46 | |
| *** mvk has joined #openstack-cinder | 08:54 | |
| *** belmoreira has joined #openstack-cinder | 08:59 | |
| *** yangyapeng has joined #openstack-cinder | 09:10 | |
| *** yangyapeng has quit IRC | 09:12 | |
| *** belmoreira has quit IRC | 09:13 | |
| *** yangyapeng has joined #openstack-cinder | 09:13 | |
| *** mvk has quit IRC | 09:15 | |
| *** lkuchlan has quit IRC | 09:17 | |
| openstackgerrit | yixuan zhang proposed openstack/cinder master: Storwize: add hyperswap volume support https://review.openstack.org/492414 | 09:27 |
| *** mvk has joined #openstack-cinder | 09:28 | |
| *** yangyapeng has quit IRC | 09:30 | |
| *** stakeda has quit IRC | 09:30 | |
| *** wanghao_ has quit IRC | 09:30 | |
| *** Shunli has quit IRC | 09:31 | |
| *** yangyapeng has joined #openstack-cinder | 09:31 | |
| *** yangyapeng has quit IRC | 09:34 | |
| *** yangyapeng has joined #openstack-cinder | 09:35 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: Remove timestamp when updating service capabilities https://review.openstack.org/507453 | 09:37 |
| *** dalgaaf has quit IRC | 09:37 | |
| *** yangyapeng has quit IRC | 09:39 | |
| *** dalgaaf has joined #openstack-cinder | 09:39 | |
| *** pgadiya has quit IRC | 09:45 | |
| *** markstur has joined #openstack-cinder | 09:45 | |
| *** markstur has quit IRC | 09:50 | |
| *** zhurong has quit IRC | 09:52 | |
| *** sdague has joined #openstack-cinder | 10:00 | |
| *** markvoelker has joined #openstack-cinder | 10:07 | |
| *** zhaochao has quit IRC | 10:09 | |
| *** zhaochao has joined #openstack-cinder | 10:09 | |
| *** markstur has joined #openstack-cinder | 10:12 | |
| *** armax has joined #openstack-cinder | 10:15 | |
| *** markstur has quit IRC | 10:17 | |
| *** yangyapeng has joined #openstack-cinder | 10:22 | |
| *** yangyapeng has quit IRC | 10:26 | |
| *** zhurong has joined #openstack-cinder | 10:35 | |
| *** markvoelker has quit IRC | 10:41 | |
| *** savvas_ has quit IRC | 10:44 | |
| *** savvas has joined #openstack-cinder | 10:45 | |
| *** lkuchlan has joined #openstack-cinder | 10:48 | |
| *** markstur has joined #openstack-cinder | 10:50 | |
| *** markstur has quit IRC | 10:54 | |
| *** alexchadin has quit IRC | 10:58 | |
| *** alexchadin has joined #openstack-cinder | 10:59 | |
| *** nicolasbock has joined #openstack-cinder | 11:07 | |
| *** armax has quit IRC | 11:12 | |
| *** liverpooler has quit IRC | 11:13 | |
| *** alexchadin has quit IRC | 11:14 | |
| *** abishop_ has quit IRC | 11:16 | |
| *** e0ne has quit IRC | 11:21 | |
| *** markstur has joined #openstack-cinder | 11:27 | |
| *** e0ne has joined #openstack-cinder | 11:28 | |
| *** kong has left #openstack-cinder | 11:29 | |
| *** alexchadin has joined #openstack-cinder | 11:30 | |
| *** markstur has quit IRC | 11:31 | |
| *** zhurong has quit IRC | 11:35 | |
| *** yangyapeng has joined #openstack-cinder | 11:38 | |
| *** zhurong has joined #openstack-cinder | 11:39 | |
| *** markvoelker has joined #openstack-cinder | 11:39 | |
| *** lkuchlan has quit IRC | 11:59 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder master: Fix backup compression unit tests https://review.openstack.org/507509 | 11:59 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder master: Run backup compression on native thread https://review.openstack.org/507510 | 11:59 |
| *** abishop has joined #openstack-cinder | 11:59 | |
| *** tpsilva has joined #openstack-cinder | 12:01 | |
| *** markstur has joined #openstack-cinder | 12:04 | |
| *** markstur has quit IRC | 12:08 | |
| *** dave-mcc_ has joined #openstack-cinder | 12:09 | |
| *** dustins has joined #openstack-cinder | 12:10 | |
| *** MVenesio has joined #openstack-cinder | 12:11 | |
| *** markvoelker has quit IRC | 12:12 | |
| *** edmondsw has joined #openstack-cinder | 12:14 | |
| *** lkuchlan has joined #openstack-cinder | 12:21 | |
| *** armax has joined #openstack-cinder | 12:30 | |
| *** markvoelker has joined #openstack-cinder | 12:32 | |
| *** hemna__ has joined #openstack-cinder | 12:34 | |
| *** armax has quit IRC | 12:37 | |
| *** zhaochao has quit IRC | 12:39 | |
| *** markstur has joined #openstack-cinder | 12:41 | |
| *** liverpooler has joined #openstack-cinder | 12:45 | |
| *** markstur has quit IRC | 12:45 | |
| *** zhurong has quit IRC | 12:46 | |
| *** ganso has joined #openstack-cinder | 12:48 | |
| *** gkadam has quit IRC | 12:48 | |
| *** zhaochao has joined #openstack-cinder | 12:53 | |
| *** bhagyashri_s has quit IRC | 12:55 | |
| *** pooja_jadhav has quit IRC | 12:56 | |
| *** erlon has joined #openstack-cinder | 12:57 | |
| *** lhx__ has quit IRC | 12:58 | |
| *** Dinesh_Bhor has quit IRC | 12:59 | |
| *** alexchadin has quit IRC | 13:01 | |
| *** pooja_jadhav has joined #openstack-cinder | 13:06 | |
| *** gnufied has joined #openstack-cinder | 13:06 | |
| *** bhagyashris has joined #openstack-cinder | 13:07 | |
| *** lucasxu has joined #openstack-cinder | 13:07 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 13:08 | |
| *** mriedem has joined #openstack-cinder | 13:11 | |
| *** markstur has joined #openstack-cinder | 13:17 | |
| *** shaner has quit IRC | 13:21 | |
| *** amoralej is now known as amoralej|lunch | 13:21 | |
| *** mchlumsky has joined #openstack-cinder | 13:22 | |
| *** markstur has quit IRC | 13:22 | |
| *** zhaochao has quit IRC | 13:27 | |
| * zzzeek sends little anonymous letters to geguileo that say...."drop ... eventlet......drop...EVENTLET !!!" | 13:29 | |
| geguileo | rofl | 13:29 |
| zzzeek | look at all the problems it causes | 13:29 |
| zzzeek | greenthreads are great if you have like 800 sleepy clients . 10 threads, not so much | 13:30 |
| geguileo | zzzeek: you suggest moving to mod_wsgi, but the service with the error is not a WSGI service | 13:31 |
| zzzeek | geguileo: interesting, it's not an HTTP service ? | 13:31 |
| geguileo | zzzeek: no it isn't | 13:31 |
| zzzeek | geguileo: weird | 13:32 |
| geguileo | zzzeek: the only cinder service doing API is cinder-api, the rest use AMQP | 13:32 |
| geguileo | zzzeek: but the issue was the same, too many greenthreads doing CPU bound operations (compression) | 13:32 |
| zzzeek | geguileo: the database calls are in an AMQP server process ? | 13:32 |
| *** felipemonteiro has joined #openstack-cinder | 13:32 | |
| geguileo | zzzeek: the service communicates using AMQP with the other services | 13:33 |
| geguileo | so it has a client | 13:33 |
| *** zhaochao has joined #openstack-cinder | 13:33 | |
| geguileo | using oslo service and the whole RPC sheband | 13:34 |
| *** felipemonteiro_ has joined #openstack-cinder | 13:34 | |
| zzzeek | geguileo: OK, in that case the ideal woudl still be that the database-centric business logic would be under traditional threads but that's not any easier than what you are doing already | 13:35 |
| zzzeek | geguileo: if it were http, going to mod_Wsgi means you get to throw away all that service code, but you're not, so feh | 13:35 |
| geguileo | yup, in Cinder we end up using subprocessed or native threads for non interruptable code | 13:35 |
| geguileo | and thanks for you analysis, I know it took you a while to figure this one out (judging by the tweet it was something like 8 hours)... | 13:36 |
| zzzeek | geguileo: it took weeks b.c. the client kept sending me incomplete sosreports, then we did the haproxy logging wrong, etc | 13:37 |
| zzzeek | geguileo: i went off and tried for a week to reproduce myself w/ no luck, etc | 13:37 |
| geguileo | I hear you on the incomplete sos reports... :-( | 13:37 |
| *** felipemonteiro has quit IRC | 13:38 | |
| *** armax has joined #openstack-cinder | 13:41 | |
| *** markstur has joined #openstack-cinder | 13:54 | |
| *** belmoreira has joined #openstack-cinder | 13:55 | |
| *** zhaochao has quit IRC | 13:57 | |
| *** eharney has joined #openstack-cinder | 13:58 | |
| *** armax has quit IRC | 13:58 | |
| *** markstur has quit IRC | 13:59 | |
| *** shaner has joined #openstack-cinder | 14:01 | |
| *** belmoreira has quit IRC | 14:01 | |
| *** crushil has joined #openstack-cinder | 14:02 | |
| *** belmoreira has joined #openstack-cinder | 14:04 | |
| *** armax has joined #openstack-cinder | 14:07 | |
| *** amoralej|lunch is now known as amoralej | 14:12 | |
| *** zhaochao has joined #openstack-cinder | 14:16 | |
| *** felipemonteiro_ has quit IRC | 14:22 | |
| *** belmoreira has quit IRC | 14:28 | |
| *** lucasxu has quit IRC | 14:30 | |
| *** markstur has joined #openstack-cinder | 14:31 | |
| *** markstur has quit IRC | 14:36 | |
| *** lbragstad has quit IRC | 14:44 | |
| *** markstur has joined #openstack-cinder | 14:52 | |
| *** lbragstad has joined #openstack-cinder | 14:54 | |
| *** eharney has quit IRC | 14:56 | |
| *** eharney has joined #openstack-cinder | 15:11 | |
| *** lucasxu has joined #openstack-cinder | 15:13 | |
| *** felipemonteiro has joined #openstack-cinder | 15:15 | |
| *** felipemonteiro_ has joined #openstack-cinder | 15:16 | |
| *** felipemonteiro has quit IRC | 15:20 | |
| *** lkuchlan has quit IRC | 15:21 | |
| *** felipemonteiro_ has quit IRC | 15:21 | |
| *** tonyb has quit IRC | 15:22 | |
| *** tonyb has joined #openstack-cinder | 15:24 | |
| *** e0ne has quit IRC | 15:28 | |
| *** wes_dillingham has joined #openstack-cinder | 15:31 | |
| *** psachin has quit IRC | 15:32 | |
| *** jdillaman has joined #openstack-cinder | 15:38 | |
| *** armax has quit IRC | 15:38 | |
| *** felipemonteiro has joined #openstack-cinder | 15:42 | |
| *** raunak has joined #openstack-cinder | 15:46 | |
| *** zhaochao has quit IRC | 15:57 | |
| *** zhaochao has joined #openstack-cinder | 15:58 | |
| *** sbezverk has quit IRC | 16:00 | |
| *** jistr is now known as jistr|mtg | 16:01 | |
| *** jdillaman has quit IRC | 16:02 | |
| *** rcernin has quit IRC | 16:05 | |
| *** Apoorva has joined #openstack-cinder | 16:09 | |
| *** mvk has quit IRC | 16:11 | |
| *** armax has joined #openstack-cinder | 16:15 | |
| *** jistr|mtg is now known as jistr | 16:18 | |
| *** armax has quit IRC | 16:20 | |
| *** r-daneel has joined #openstack-cinder | 16:27 | |
| *** moshele has joined #openstack-cinder | 16:34 | |
| *** jdillaman has joined #openstack-cinder | 16:34 | |
| *** ntpttr_laptop has joined #openstack-cinder | 16:54 | |
| *** crose has joined #openstack-cinder | 17:00 | |
| *** rcernin has joined #openstack-cinder | 17:11 | |
| *** arnewiebalck_ has quit IRC | 17:14 | |
| *** raunak has quit IRC | 17:17 | |
| *** crushil has quit IRC | 17:27 | |
| *** crushil has joined #openstack-cinder | 17:27 | |
| *** moshele has quit IRC | 17:32 | |
| *** ntpttr_laptop has quit IRC | 17:35 | |
| *** ntpttr_laptop has joined #openstack-cinder | 17:37 | |
| *** tesseract has quit IRC | 17:37 | |
| *** raunak has joined #openstack-cinder | 17:44 | |
| *** jmlowe has quit IRC | 17:46 | |
| *** lbragstad has quit IRC | 17:47 | |
| *** moshele has joined #openstack-cinder | 17:48 | |
| *** lbragstad has joined #openstack-cinder | 17:51 | |
| openstackgerrit | Merged openstack/cinder master: nimble: fix for parent volume delete https://review.openstack.org/506421 | 17:52 |
| *** crose has quit IRC | 17:53 | |
| *** Apoorva_ has joined #openstack-cinder | 18:02 | |
| *** raunak_ has joined #openstack-cinder | 18:04 | |
| *** vivsoni has quit IRC | 18:05 | |
| *** Apoorva has quit IRC | 18:06 | |
| *** raunak has quit IRC | 18:06 | |
| *** raunak_ is now known as raunak | 18:06 | |
| *** raunak has quit IRC | 18:07 | |
| *** Apoorva_ has quit IRC | 18:08 | |
| *** Apoorva has joined #openstack-cinder | 18:08 | |
| *** moshele has quit IRC | 18:09 | |
| *** lucasxu has quit IRC | 18:11 | |
| *** dave-mcc_ is now known as dave-mccowan | 18:12 | |
| *** moshele has joined #openstack-cinder | 18:17 | |
| *** jmlowe has joined #openstack-cinder | 18:18 | |
| openstackgerrit | Kaitlin Farr proposed openstack/cinder master: Remove deprecated keymgr code https://review.openstack.org/468252 | 18:20 |
| *** moshele has quit IRC | 18:20 | |
| *** amoralej is now known as amoralej|off | 18:21 | |
| -openstackstatus- NOTICE: The infra team is continuing work to bring Zuul v3 online; expect service disruptions and please see https://docs.openstack.org/infra/manual/zuulv3.html for more information. | 18:23 | |
| *** xyang1 has joined #openstack-cinder | 18:24 | |
| *** markstur has quit IRC | 18:26 | |
| lbragstad | does anyone here happen to know where the cinder spec for a capabilities API went? | 18:26 |
| lbragstad | did that merge? or get abandon? | 18:26 |
| *** xyang1 has quit IRC | 18:27 | |
| *** e0ne has joined #openstack-cinder | 18:28 | |
| *** moshele has joined #openstack-cinder | 18:31 | |
| *** Apoorva has quit IRC | 18:33 | |
| smcginnis | lbragstad: I seem to remember multiple different ones working towards "capabilities". Let me see what I can find? | 18:33 |
| *** Apoorva has joined #openstack-cinder | 18:33 | |
| smcginnis | lbragstad: This one? http://specs.openstack.org/openstack/cinder-specs/specs/newton/discovering-system-capabilities.html | 18:33 |
| lbragstad | yeah - that might be it | 18:34 |
| lbragstad | thanks smcginnis! | 18:34 |
| smcginnis | lbragstad: Looking at the bp, doesn't look like that was actually fully implemented. | 18:35 |
| lbragstad | i remember it being a lot of work - but the long term vision made a lot of sense | 18:36 |
| *** e0ne has quit IRC | 18:36 | |
| smcginnis | I agree. | 18:37 |
| *** markstur has joined #openstack-cinder | 18:46 | |
| *** moshele has quit IRC | 18:47 | |
| *** markstur has quit IRC | 18:51 | |
| *** lucasxu has joined #openstack-cinder | 18:52 | |
| openstackgerrit | Rajini Karthik proposed openstack/cinder master: Dell PS: Fix over-subscription ratio stats https://review.openstack.org/507647 | 19:02 |
| jungleboyj | smcginnis: lbragstad Doesn't that go back to the MeetUp we had at IBM in Austin? | 19:03 |
| jungleboyj | I would love to see that path improved and more integration with Horizon to make it easier to actually create useful volume_types. | 19:04 |
| lbragstad | yeah - with all the policy work going on right now - i have an idea in mind to make that easier | 19:05 |
| lbragstad | i just need to write it down | 19:05 |
| jungleboyj | lbragstad: Cool. | 19:05 |
| *** mvk has joined #openstack-cinder | 19:06 | |
| *** markstur has joined #openstack-cinder | 19:08 | |
| *** ChubYann has joined #openstack-cinder | 19:09 | |
| *** moshele has joined #openstack-cinder | 19:10 | |
| *** markstur has quit IRC | 19:13 | |
| *** markstur has joined #openstack-cinder | 19:15 | |
| openstackgerrit | John Griffith proposed openstack/cinder master: Add shared_targets and backend_name to volumes https://review.openstack.org/507649 | 19:16 |
| jgriffith | smcginnis: ildikov ^^ | 19:16 |
| jgriffith | jungleboyj: ^^ | 19:16 |
| smcginnis | jgriffith: ack | 19:16 |
| jungleboyj | jgriffith: Looking. | 19:16 |
| openstackgerrit | John Griffith proposed openstack/python-cinderclient master: Bump MAX version of client to 3.45 https://review.openstack.org/507650 | 19:17 |
| jungleboyj | OMG!?! Are we following process!?! | 19:17 |
| jgriffith | jungleboyj: :) | 19:17 |
| jgriffith | jungleboyj: just means that now I'll have two patches to deal with when everybody rips my proposal apart :) | 19:17 |
| jgriffith | smcginnis: your mv change made life MUCH simpler BTW!! I like it!! | 19:18 |
| smcginnis | jgriffith: Awesome! | 19:18 |
| jungleboyj | smcginnis: Yay! | 19:18 |
| openstackgerrit | Gorka Eguileor proposed openstack/os-brick master: Protect against race within os.path.realpath https://review.openstack.org/507653 | 19:22 |
| openstackgerrit | Gorka Eguileor proposed openstack/os-brick master: Fix _remove_scsi_symlinks_no_links test https://review.openstack.org/507654 | 19:22 |
| jgriffith | jungleboyj: if you think that warrants a spec let me know and I'll get one written up. | 19:24 |
| jgriffith | I wasn't sure after it was done if it needed it or not | 19:24 |
| *** chhavi_ has quit IRC | 19:25 | |
| jungleboyj | There is one Typo I found. How topical. :-) | 19:29 |
| jungleboyj | As far as needing a Spec. | 19:29 |
| smcginnis | Only one? I've seen more than that. :P | 19:30 |
| jungleboyj | Hmmm, that is a good question. So, we discussed this pretty well at the Summit and I feel good about that. The only reason I hedge is because this has an impact on all of the drivers and having the spec would give something to point the driver developers at as far as this release and the change being made. | 19:30 |
| jungleboyj | jgriffith: ^^ | 19:30 |
| *** chenying has quit IRC | 19:33 | |
| *** chenying has joined #openstack-cinder | 19:34 | |
| *** rcernin has quit IRC | 19:34 | |
| *** chenying_ has joined #openstack-cinder | 19:37 | |
| jgriffith | I have no problem writing one | 19:37 |
| jgriffith | I'll work something up so long as there's no glaring issue with this approach | 19:37 |
| *** chenying has quit IRC | 19:40 | |
| smcginnis | jgriffith: I passed! :D | 19:40 |
| jgriffith | smcginnis: :) | 19:41 |
| *** armax has joined #openstack-cinder | 19:43 | |
| *** Apoorva has quit IRC | 19:44 | |
| jungleboyj | jgriffith: Yeah, if you don't mind doing that lets do it. | 19:44 |
| jungleboyj | I am going to add these patches to the priority review list. | 19:44 |
| *** arnewiebalck_ has joined #openstack-cinder | 19:45 | |
| jgriffith | cool; I've got the update for the things you and smcginnis pointed out, will let it sit for a bit to see if there's other feedback | 19:45 |
| jgriffith | I was going to mark it WIP but I fear if I do that it won't get reviewed :( | 19:46 |
| jgriffith | slackers such as myself tend to skip items with a red mark | 19:46 |
| jungleboyj | :-) | 19:47 |
| smcginnis | Note to self, flavor creation does RAM in MB and Ubuntu doesn't like only having 8MB. | 19:50 |
| jgriffith | smcginnis: :) | 19:50 |
| jungleboyj | Ooops. | 19:50 |
| *** jmlowe has quit IRC | 19:56 | |
| *** wes_dillingham has quit IRC | 19:56 | |
| *** lucasxu has quit IRC | 19:57 | |
| *** pcaruana has quit IRC | 20:05 | |
| *** dave-mcc_ has joined #openstack-cinder | 20:10 | |
| *** arnewiebalck_ has quit IRC | 20:11 | |
| *** dave-mccowan has quit IRC | 20:12 | |
| *** arnewiebalck_ has joined #openstack-cinder | 20:13 | |
| *** Apoorva has joined #openstack-cinder | 20:17 | |
| openstackgerrit | John Griffith proposed openstack/cinder-specs master: Add shared_targets column to volume table https://review.openstack.org/507670 | 20:23 |
| *** liverpooler has quit IRC | 20:23 | |
| *** dave-mccowan has joined #openstack-cinder | 20:24 | |
| *** belmoreira has joined #openstack-cinder | 20:25 | |
| *** dave-mcc_ has quit IRC | 20:25 | |
| *** moshele has quit IRC | 20:25 | |
| *** jmlowe has joined #openstack-cinder | 20:30 | |
| ericyoung | jungleboyj, I addressed your comments on https://review.openstack.org/#/c/502473/ can you have another look when you have a chance | 20:31 |
| *** amoralej|off has quit IRC | 20:32 | |
| jgriffith | patrickeast: I responded on that stripping out host, yeah, valid case | 20:33 |
| jgriffith | patrickeast: there's another little problem in there though I think | 20:33 |
| * patrickeast reads the review comments | 20:34 | |
| jgriffith | patrickeast: if users don't select the same backend_name in their config it's still not going to work | 20:34 |
| jgriffith | patrickeast: host-a@pure-1 host-b@pure-1 Then cool either way.... | 20:35 |
| patrickeast | jgriffith: not sure i follow | 20:35 |
| *** amoralej has joined #openstack-cinder | 20:35 | |
| jgriffith | But host-a@pure-1 and host-b@pure-2 no bueano | 20:35 |
| patrickeast | jgriffith: whys that? | 20:35 |
| jgriffith | what I mean is even if they're using the same backend, there's nothing that forces them to name it the same | 20:35 |
| patrickeast | oh | 20:35 |
| patrickeast | well yea | 20:35 |
| jgriffith | a lock on pure-q won't do any good on pure-2 | 20:35 |
| jgriffith | with or without the host prefix | 20:36 |
| patrickeast | if they're doing the ghetto HA c-vol's all bets are off | 20:36 |
| jgriffith | s/pure-q/pure-1/ | 20:36 |
| jgriffith | :) | 20:36 |
| jgriffith | patrickeast: actually that still works :) | 20:36 |
| patrickeast | maybe we just put that as a disclaimer | 20:36 |
| *** felipemonteiro has quit IRC | 20:36 | |
| patrickeast | oh yea, its in production use :*( | 20:36 |
| jgriffith | yeah, I'm cool with thinking on it more | 20:36 |
| jgriffith | I would like to avoid it only because of the admin requirement it will introduce | 20:37 |
| patrickeast | but some people like to live dangerously... for the rest we can give some protection if they follow our new HA guidance (and assuming we can turn on HA for drivers) | 20:37 |
| jgriffith | It's questionable if we need that now anyway, but if we keep the host there we'll def need it | 20:37 |
| jgriffith | :) | 20:37 |
| jgriffith | honestly it may be good to just make it admin visisble only and keep it in place | 20:38 |
| patrickeast | yea, that might be enough | 20:38 |
| jgriffith | patrickeast: but I'm still concerned that it might make things more difficult | 20:38 |
| patrickeast | i guess if we *really* want to give people a long rope we can add a generic driver config option that is like the string to lock on | 20:38 |
| jgriffith | patrickeast: your use case inparticular is problematic | 20:38 |
| patrickeast | so the deployer who (in theory) knows best can just say its the same, and we maybe just default to hostname@backend otherwise | 20:39 |
| patrickeast | or something | 20:39 |
| jgriffith | patrickeast: yeah, I was trying to avoid adding another db field but that might be the best answer | 20:39 |
| jgriffith | That way no internal data leaks, and it's controllable | 20:39 |
| patrickeast | yea :( | 20:39 |
| jgriffith | default if not set is the standard backend-name thing like I have it | 20:39 |
| patrickeast | whats another column on the volume table, eh? | 20:40 |
| jgriffith | else use config | 20:40 |
| patrickeast | seems reasonable to me | 20:40 |
| jgriffith | actually, I might be able to do that without the column | 20:40 |
| jgriffith | just extend the same dirty little hack I already have in there | 20:40 |
| patrickeast | what if we go back to the shared flag just being a string? | 20:40 |
| patrickeast | in our db at least | 20:40 |
| patrickeast | kind of a hack i guess | 20:40 |
| jgriffith | yeah... that might be worth reconsidering | 20:40 |
| jgriffith | ummm... have you looked at the code :) | 20:41 |
| patrickeast | haha | 20:41 |
| jgriffith | I mean ALL OF IT, not just mine inparticular. All of OpenStack | 20:41 |
| jgriffith | :) | 20:41 |
| jgriffith | so an empty string means "NOT-SHARED" | 20:41 |
| patrickeast | some parts are very well thought out and clearly coded | 20:41 |
| patrickeast | yea | 20:41 |
| jgriffith | anything else means "here's a handy lock string to use" | 20:42 |
| patrickeast | exactly | 20:42 |
| *** crushil has quit IRC | 20:42 | |
| jgriffith | Yeah, that's what we talked about in Denver.... | 20:42 |
| patrickeast | i guess the tricky part is defaulting to something smart | 20:42 |
| patrickeast | not sure how to tackle that | 20:42 |
| jgriffith | but I don't wat to rewrite that stupid spec :) | 20:42 |
| jgriffith | patrickeast: default to backend-name | 20:42 |
| jgriffith | that's easy | 20:42 |
| patrickeast | so then operators with SF backends need to set it to ""? | 20:42 |
| patrickeast | thats less than ideal no? | 20:43 |
| jgriffith | nah | 20:43 |
| jgriffith | I'd put it in capabilities still | 20:43 |
| patrickeast | ohh i see, yea if the driver says its shared | 20:43 |
| jgriffith | driver would do it, and ignore anything that they put in there anyway | 20:43 |
| patrickeast | gotcha | 20:43 |
| jgriffith | dang it | 20:43 |
| jgriffith | I think that might be better | 20:43 |
| jgriffith | All I wanted was for this attach stuff to end :) | 20:44 |
| jgriffith | smcginnis: jungleboyj thoughts on the above proposal? | 20:44 |
| *** armax has quit IRC | 20:50 | |
| smcginnis | Sheesh, step away for a snack and there's a wall of text. :) | 20:50 |
| *** armax has joined #openstack-cinder | 20:50 | |
| smcginnis | jgriffith: So the idea is to move back to string so we can give the consumer a consistent thing to lock on? Able to be overridden per backend by config setting? Did I parse all that right? | 20:50 |
| *** armax has quit IRC | 20:50 | |
| *** armax has joined #openstack-cinder | 20:51 | |
| *** armax has quit IRC | 20:51 | |
| *** armax has joined #openstack-cinder | 20:52 | |
| *** ntpttr_laptop has quit IRC | 20:52 | |
| *** armax has quit IRC | 20:52 | |
| *** armax has joined #openstack-cinder | 20:52 | |
| *** armax has quit IRC | 20:53 | |
| *** itlinux has joined #openstack-cinder | 20:53 | |
| *** kfarr has joined #openstack-cinder | 20:56 | |
| *** kfarr has quit IRC | 20:57 | |
| *** hemna__ has quit IRC | 21:00 | |
| *** arnewiebalck_ has quit IRC | 21:04 | |
| *** jmlowe has quit IRC | 21:07 | |
| jgriffith | smcginnis: well... yeah, for the most part | 21:08 |
| jgriffith | although I'm still not fully convinced which approach is better | 21:08 |
| jgriffith | I'm kinda leaning towards a hybrid | 21:08 |
| *** abishop has quit IRC | 21:09 | |
| *** jmlowe has joined #openstack-cinder | 21:09 | |
| jgriffith | take what I have posted now, and add a config option to specify the lock name | 21:09 |
| jgriffith | so change backend_name to something like "lockable_name" | 21:09 |
| jgriffith | defaults to the backend-name like I do now and overrides if you have it in the config | 21:09 |
| jgriffith | patrickeast: thoughts on that ^^ | 21:10 |
| jgriffith | seems like the best of both worlds | 21:10 |
| jgriffith | and I don't have to rewrite all the things :) | 21:10 |
| jgriffith | I suppose it does mean I need to add a valid column to the DB though | 21:10 |
| smcginnis | jgriffith: I like that plan. | 21:11 |
| smcginnis | jgriffith: But what's the column needed for? | 21:11 |
| jgriffith | smcginnis: well I don't think I can get the config variable in the API layer like I'm doing currently | 21:12 |
| *** belmoreira has quit IRC | 21:12 | |
| *** armax has joined #openstack-cinder | 21:12 | |
| jgriffith | smcginnis: since I'm just plucking the host field out and massaging it in v3/views/volumes.py | 21:12 |
| smcginnis | Oh right, not for a backend I think. | 21:12 |
| *** dustins has quit IRC | 21:12 | |
| jgriffith | I won't be able to translate that into a backend config for that volume | 21:12 |
| jgriffith | yeah | 21:13 |
| jgriffith | would have to have it stored somewhere | 21:13 |
| jgriffith | I could cheat and use provider info :) | 21:13 |
| smcginnis | jgriffith: I think you need to add another metadata field. :P | 21:13 |
| jgriffith | smcginnis: why I oughta!!!! | 21:14 |
| *** MVenesio has quit IRC | 21:14 | |
| *** MVenesio has joined #openstack-cinder | 21:15 | |
| *** jmlowe has quit IRC | 21:18 | |
| *** MVenesio has quit IRC | 21:19 | |
| *** belmoreira has joined #openstack-cinder | 21:20 | |
| *** belmoreira has quit IRC | 21:22 | |
| jungleboyj | I try to get the kids from school and get a wall of text. | 21:25 |
| jungleboyj | So I think it's good to let the user configure the lock name and default to using the host if they don't specify it. I think trying to use the shared volume field for multiple things gets confusing. | 21:27 |
| jungleboyj | I will have to look at the code again when we get to a point where I'm by at my computer. | 21:28 |
| jungleboyj | jgriffith: ^^ | 21:28 |
| *** ntpttr_laptop has joined #openstack-cinder | 21:29 | |
| *** eharney has quit IRC | 21:30 | |
| jgriffith | jungleboyj: thanks... I'm working on an update that adds another column and allows the config option | 21:30 |
| jgriffith | I'm leaning away from the single column for the shared purpose thing | 21:31 |
| jgriffith | besides, who knows, maybe some of these things will be useful for *future* things | 21:31 |
| *** wes_dillingham has joined #openstack-cinder | 21:35 | |
| openstackgerrit | Merged openstack/cinder master: [DOC] Update index page for Install tutorial https://review.openstack.org/505357 | 21:35 |
| jungleboyj | jgriffith: sounds good. I'll be back at my computer in about an hour and will take a look if you have something up then. | 21:41 |
| *** r-daneel has quit IRC | 21:43 | |
| *** armax has quit IRC | 21:43 | |
| *** edmondsw has quit IRC | 21:46 | |
| *** edmondsw_ has joined #openstack-cinder | 21:49 | |
| *** edmondsw_ has quit IRC | 21:54 | |
| jgriffith | patrickeast: jungleboyj So I'm workign on making these FK's that point to the service rather than try and deal with independent columns in every object | 21:59 |
| jgriffith | I'll get an update finished tonight and post it for folks to vomit over | 22:00 |
| ildikov | jgriffith: you should aim at merge it over :) | 22:06 |
| *** sdague has quit IRC | 22:13 | |
| patrickeast | jgriffith: sounds good to me | 22:18 |
| *** jmlowe has joined #openstack-cinder | 22:24 | |
| *** tpsilva has quit IRC | 22:31 | |
| *** dave-mccowan has quit IRC | 22:36 | |
| *** itlinux has quit IRC | 22:37 | |
| *** lbragstad has quit IRC | 22:40 | |
| *** moshele has joined #openstack-cinder | 22:41 | |
| jungleboyj | jgriffith: Sounds like a plan. Will watch for the patch. | 23:00 |
| jungleboyj | ildikov: I like that goal. I don't feel like vomiting tonight. | 23:00 |
| ildikov | jungleboyj: +1 | 23:08 |
| jungleboyj | ildikov: :-) How'd things go at OSD UK? | 23:08 |
| ildikov | ildikov: it was a good event, got a few people with questions, talked to johnthetubaguy about the multi-attach spec | 23:09 |
| ildikov | jungleboyj: ^^ | 23:09 |
| ildikov | jungleboyj: and I'm tired :) | 23:09 |
| *** wes_dillingham has quit IRC | 23:10 | |
| ildikov | jungleboyj: we will need to talk a bit about policies this Thursday on the meeting as we got confused on r-w vs r-o | 23:10 |
| jungleboyj | ildikov :-) Well, good that there was some interest. I bet you are tired! | 23:10 |
| ildikov | jungleboyj: like whether it's per volume or per attachment | 23:10 |
| jungleboyj | ildikov: Ok, sounds good. | 23:10 |
| jungleboyj | ildikov: Oy, that is a good question ... My first thought was per volume, but really that is something that happens at attach time I think. | 23:11 |
| ildikov | jungleboyj: nova doesn't really specify it, I think it comes with the connector | 23:11 |
| ildikov | jungleboyj: and according to the API docs it's in metadata in the volume info | 23:11 |
| ildikov | jungleboyj: the new attach API has it in the attachment record though | 23:12 |
| ildikov | jgriffith: how much are you knowledgable on r-w and r-o settings? | 23:12 |
| jungleboyj | ildikov: So, if it is in the volume, then all places where the volume is attached it would need to be R/O or R/W . | 23:12 |
| jungleboyj | ildikov: We wouldn't want that behavior. For multi-attach we are likely to want R/W on one attachment and R/O on one or more attachments. | 23:13 |
| ildikov | jungleboyj: well, we said we would like to have policies | 23:14 |
| ildikov | jungleboyj: and if we say it's one R/W and the rest is R/O that is also a setting that we cannot set through Nova today for instance and it would also require to set it per attachment | 23:15 |
| jungleboyj | ildikov: I don't honestly know how read only versis R/W works right now. | 23:15 |
| ildikov | jungleboyj: which is slightly unclear from the currently existing API's regarding how, if, it is doable | 23:16 |
| ildikov | jungleboyj: it should be though I would think | 23:16 |
| jungleboyj | ildikov: I was going to say that ideally we would only allow one R/W and multiple R/O. | 23:17 |
| jungleboyj | Though, I guess that might not be true for all backends. | 23:17 |
| ildikov | and if we allow that case, it's also a question that who's responsible for forcing it | 23:18 |
| jungleboyj | So, I think figuring out a way where it can be handled by policy is best. Set it by default to something safe and then admins can change it based on what works for their env. | 23:18 |
| ildikov | we need to look into what's doable first | 23:19 |
| ildikov | having a go/no-go version first is honestly fine with me and I think with others as well | 23:19 |
| ildikov | I mean turn it full yon and off | 23:20 |
| jungleboyj | Right, I think getting multi-attach working with something safe is best. We can work out the R/O and R/W details later. | 23:20 |
| ildikov | but if we come up with mixtures like the above we need to have an idea on how it is doable and which side the responsibility lands on | 23:20 |
| jungleboyj | It is your responsibility. ;-) | 23:21 |
| ildikov | the annoying people with it part is | 23:22 |
| ildikov | which I'm already doing :) | 23:22 |
| ildikov | so it's now on others to chime in ;) | 23:22 |
| *** wes_dillingham has joined #openstack-cinder | 23:23 | |
| jungleboyj | ildikov: You never annoy us. Thankful you keep this moving. | 23:23 |
| jungleboyj | ildikov: I am pretty swamped this week. Not sure if I will get a chance to look into things before Thursday. Will be thinking about it and hopefully we can get a game plan put together to go forward. | 23:25 |
| ildikov | jungleboyj: at least we should look into it and then we will see where we end up | 23:25 |
| ildikov | jungleboyj: do you know any policy expert in the team? | 23:25 |
| jungleboyj | ildikov: smcginnis and tommylikehu Are the ones that have been looking to the policy code to move towards default policy in code comliance. | 23:26 |
| jungleboyj | *compliance. | 23:26 |
| ildikov | ok | 23:27 |
| jungleboyj | I have an ok understanding of how it works. | 23:27 |
| ildikov | will keep in touch with them on this | 23:27 |
| ildikov | thanks! | 23:27 |
| jungleboyj | ildikov: No problem. Thank you. When do you head back home? | 23:27 |
| ildikov | Saturday I think | 23:29 |
| ildikov | I have another conference this week in Amsterdam | 23:29 |
| jungleboyj | ildikov: Wow. Well, at least you get a little time to adjust before changing timezones again. | 23:30 |
| ildikov | jungleboyj: well, I'm kind of skipping sleep tonight it seems, so who knows what I get to adjust to... :) | 23:31 |
| jungleboyj | ildikov: I was going to ask if you should be sleeping now. :-) | 23:31 |
| ildikov | jungleboyj: I should :) | 23:32 |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Remove Hitachi volume drivers https://review.openstack.org/507709 | 23:45 |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Remove Hitachi volume drivers https://review.openstack.org/507709 | 23:52 |
| erlon | smcginnis: maan, that will boost your contribution stats :) | 23:54 |
| smcginnis | erlon: Haha, I guess so! +6, -19000 | 23:54 |
| smcginnis | erlon: Too bad there aren't uncontribution stats. | 23:54 |
| smcginnis | decontribution? Something like that. :D | 23:55 |
| erlon | smcginnis: haha, yeah, it only counts changed lines | 23:55 |
| smcginnis | erlon: I've gotta beat e0ne with amount of code removed. ;) | 23:55 |
| erlon | so, is mod(+6) + mod(-19000) = 19006 | 23:56 |
| smcginnis | Sweet! :D | 23:56 |
| jungleboyj | There goes smcginnis trying to pad his stats! | 23:56 |
| jungleboyj | I refuse to review your patches. | 23:56 |
| smcginnis | Yah, you know me. | 23:56 |
| jungleboyj | Minnesotans always have hidden agendas in their contributions. ;-) | 23:57 |
| smcginnis | Abandom them all, resubmit as your own, +2A it through. Hahah | 23:57 |
| smcginnis | *Abandon | 23:57 |
| jungleboyj | Hmmm, ok. | 23:58 |
| jungleboyj | Oh wait, I already have all my suspicious Doc patches. Just keeping the MN stereotype going. | 23:59 |
| smcginnis | LOL | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!