| *** crushil_ has joined #openstack-cinder | 00:11 | |
| openstackgerrit | Merged openstack/cinder master: [api-ref] Add snapshot metadata key api https://review.openstack.org/475208 | 00:18 |
|---|---|---|
| *** markstur has quit IRC | 00:24 | |
| *** crushil_ has quit IRC | 00:30 | |
| *** crushil_ has joined #openstack-cinder | 00:31 | |
| *** salv-orl_ has joined #openstack-cinder | 00:37 | |
| *** wanghao has joined #openstack-cinder | 00:39 | |
| *** salv-orlando has quit IRC | 00:40 | |
| *** yangyapeng has joined #openstack-cinder | 00:46 | |
| *** Apoorva_ has joined #openstack-cinder | 00:48 | |
| *** yangyapeng has quit IRC | 00:50 | |
| *** Apoorva has quit IRC | 00:52 | |
| *** Apoorva_ has quit IRC | 00:53 | |
| *** yangyapeng has joined #openstack-cinder | 00:54 | |
| *** Julien-zte has joined #openstack-cinder | 00:59 | |
| *** markstur has joined #openstack-cinder | 01:02 | |
| *** fredli__ has joined #openstack-cinder | 01:02 | |
| *** Shunli has joined #openstack-cinder | 01:05 | |
| *** markstur_ has joined #openstack-cinder | 01:05 | |
| *** yongwc has joined #openstack-cinder | 01:07 | |
| *** markstur has quit IRC | 01:07 | |
| *** crushil_ has quit IRC | 01:08 | |
| openstackgerrit | Wenjun Wang proposed openstack/cinder master: Volume create failed when volume need flatten https://review.openstack.org/477857 | 01:12 |
| *** itlinux has joined #openstack-cinder | 01:13 | |
| *** kaisers__ has joined #openstack-cinder | 01:22 | |
| openstackgerrit | Guy Kim proposed openstack/cinder master: Update configuration names in ITRI DISCO volume driver https://review.openstack.org/461665 | 01:23 |
| *** zhangguoqing has joined #openstack-cinder | 01:24 | |
| *** kaisers_ has quit IRC | 01:25 | |
| *** dhinesh has quit IRC | 01:47 | |
| *** tsuzuki has joined #openstack-cinder | 01:56 | |
| *** itlinux has quit IRC | 02:01 | |
| *** Wenjun has joined #openstack-cinder | 02:01 | |
| openstackgerrit | Matt Riedemann proposed openstack/cinder master: Add ability to extend 'in-use' volume https://review.openstack.org/454287 | 02:06 |
| openstackgerrit | Peter Wang proposed openstack/cinder master: VNX: Add QoS support https://review.openstack.org/441786 | 02:09 |
| *** youzp has joined #openstack-cinder | 02:09 | |
| openstackgerrit | Peter Wang proposed openstack/cinder master: Add VNX replication group support https://review.openstack.org/459205 | 02:12 |
| *** Shunli has quit IRC | 02:33 | |
| *** knikolla has quit IRC | 02:35 | |
| openstackgerrit | qtlu proposed openstack/cinder master: Show user_id when cinder snapshot-show https://review.openstack.org/472545 | 02:37 |
| openstackgerrit | Ashish Billore proposed openstack/cinder master: Include Usage for attachment-delete CLI https://review.openstack.org/476029 | 02:39 |
| *** lucasxu has joined #openstack-cinder | 02:40 | |
| *** knikolla has joined #openstack-cinder | 02:43 | |
| *** abishop has quit IRC | 02:45 | |
| openstackgerrit | Guy Kim proposed openstack/cinder master: Update configuration names in ITRI DISCO volume driver https://review.openstack.org/461665 | 02:48 |
| *** lucasxu has quit IRC | 02:56 | |
| openstackgerrit | Cheng Yong Zhao proposed openstack/cinder master: Add gmcv support in SVC driver https://review.openstack.org/461191 | 02:58 |
| *** Nel1x has quit IRC | 03:03 | |
| *** bswartz has quit IRC | 03:03 | |
| *** lucasxu has joined #openstack-cinder | 03:07 | |
| *** crushil_ has joined #openstack-cinder | 03:14 | |
| *** wanghao has quit IRC | 03:17 | |
| *** wanghao has joined #openstack-cinder | 03:18 | |
| *** gkadam has joined #openstack-cinder | 03:18 | |
| *** crushil_ has quit IRC | 03:18 | |
| *** markstur_ has quit IRC | 03:19 | |
| *** crushil_ has joined #openstack-cinder | 03:21 | |
| *** crushil_ has quit IRC | 03:21 | |
| *** markstur has joined #openstack-cinder | 03:25 | |
| openstackgerrit | zheng yin proposed openstack/cinder master: eventlet.green.subprocess replace subprocess https://review.openstack.org/466607 | 03:25 |
| *** crushil_ has joined #openstack-cinder | 03:25 | |
| *** xyang1 has quit IRC | 03:37 | |
| *** wanghao has quit IRC | 03:37 | |
| *** itlinux has joined #openstack-cinder | 03:38 | |
| *** wanghao has joined #openstack-cinder | 03:39 | |
| *** crushil_ has quit IRC | 03:41 | |
| *** adisky__ has joined #openstack-cinder | 03:51 | |
| *** crushil_ has joined #openstack-cinder | 03:55 | |
| *** links has joined #openstack-cinder | 04:00 | |
| *** fredli__ has quit IRC | 04:01 | |
| *** wangyong has joined #openstack-cinder | 04:03 | |
| *** crushil_ has quit IRC | 04:04 | |
| *** crushil_ has joined #openstack-cinder | 04:04 | |
| *** lucasxu has quit IRC | 04:06 | |
| *** yongwc has quit IRC | 04:07 | |
| *** psachin has joined #openstack-cinder | 04:09 | |
| *** dhinesh has joined #openstack-cinder | 04:10 | |
| *** arnewiebalck_ has joined #openstack-cinder | 04:14 | |
| *** kaisers__ has quit IRC | 04:17 | |
| *** arnewiebalck_ has quit IRC | 04:18 | |
| openstackgerrit | jiamin proposed openstack/cinder master: DS8K: support clone volume asynchronously https://review.openstack.org/467136 | 04:30 |
| *** diablo_rojo has quit IRC | 04:39 | |
| *** kaisers_ has joined #openstack-cinder | 04:47 | |
| *** wangyong has quit IRC | 04:49 | |
| *** kaisers_ has quit IRC | 04:53 | |
| *** Poornima has joined #openstack-cinder | 04:56 | |
| *** kaisers_ has joined #openstack-cinder | 05:02 | |
| openstackgerrit | Guy Kim proposed openstack/cinder master: Update configuration names in ITRI DISCO volume driver https://review.openstack.org/461665 | 05:02 |
| *** lennyb has quit IRC | 05:09 | |
| *** kaisers_ has quit IRC | 05:10 | |
| *** lkuchlan has joined #openstack-cinder | 05:12 | |
| *** lkuchlan has quit IRC | 05:12 | |
| *** lkuchlan has joined #openstack-cinder | 05:12 | |
| *** lkuchlan is now known as lkuchlan_wfh | 05:12 | |
| *** itlinux has quit IRC | 05:15 | |
| openstackgerrit | Ryan Liang proposed openstack/cinder master: Unity: Use thin clone when cloning volume https://review.openstack.org/471250 | 05:16 |
| *** vivsoni_ has joined #openstack-cinder | 05:16 | |
| *** vivsoni has quit IRC | 05:17 | |
| *** lennyb has joined #openstack-cinder | 05:21 | |
| *** vivsoni has joined #openstack-cinder | 05:24 | |
| *** vivsoni_ has quit IRC | 05:25 | |
| smcginnis | jungleboyj: (cc: asettle) I had the doc migration on the cinder meeting agenda to enlist help there. | 05:26 |
| smcginnis | patrickeast: Yeah, geguileo was signed up to write up HA testing criteria, but I think he's been pulled into other things. | 05:27 |
| jungleboyj | smcginnis: Thanks for starting that. I should get that etherpad set up and lead the effort. | 05:29 |
| smcginnis | jungleboyj: Go for it. :) | 05:29 |
| jungleboyj | Have the ha stuff in tomorrow's agenda. | 05:29 |
| jungleboyj | smcginnis: will do. | 05:30 |
| jungleboyj | smcginnis: Need me to run the meeting? | 05:30 |
| smcginnis | jungleboyj: I'm going try to be online for it. | 05:32 |
| smcginnis | jungleboyj: Just need to make sure I can get back to my hotel in time. | 05:32 |
| smcginnis | I've had some late nights, so I'm hoping I'll be there in plenty of time. :) | 05:32 |
| jungleboyj | smcginnis: ok, i will be ready to cover just in case. | 05:33 |
| smcginnis | jungleboyj: Thank you! | 05:33 |
| smcginnis | jungleboyj: Really appreciate not needing to worry too much while I'm on the road. | 05:34 |
| smcginnis | :) | 05:34 |
| *** wangyong has joined #openstack-cinder | 05:35 | |
| jungleboyj | smcginnis: my pleasure, glad to help. | 05:35 |
| *** crushil_ has quit IRC | 05:41 | |
| *** ChubYann has quit IRC | 05:42 | |
| *** crushil_ has joined #openstack-cinder | 05:43 | |
| openstackgerrit | Ryan Liang proposed openstack/cinder master: VNX: wrong size of volume from image cache https://review.openstack.org/476402 | 05:45 |
| *** vivsoni_ has joined #openstack-cinder | 05:45 | |
| *** Shunli has joined #openstack-cinder | 05:45 | |
| openstackgerrit | wanghao proposed openstack/cinder master: Fix driver exception when cascade deleting volume after transferring https://review.openstack.org/474890 | 05:47 |
| *** vivsoni has quit IRC | 05:47 | |
| *** pgadiya has joined #openstack-cinder | 05:50 | |
| *** isaacb has joined #openstack-cinder | 05:51 | |
| *** vivsoni has joined #openstack-cinder | 05:58 | |
| *** vivsoni_ has quit IRC | 06:00 | |
| *** lbragstad_ has joined #openstack-cinder | 06:02 | |
| *** rcernin has joined #openstack-cinder | 06:02 | |
| *** lbragstad has quit IRC | 06:02 | |
| *** vivsoni_ has joined #openstack-cinder | 06:07 | |
| *** salv-orl_ has quit IRC | 06:10 | |
| *** vivsoni_ has quit IRC | 06:10 | |
| *** vivsoni has quit IRC | 06:10 | |
| *** salv-orlando has joined #openstack-cinder | 06:10 | |
| *** crushil_ has quit IRC | 06:11 | |
| *** kaisers_ has joined #openstack-cinder | 06:11 | |
| *** vivsoni has joined #openstack-cinder | 06:12 | |
| *** Julien-zte has quit IRC | 06:14 | |
| *** salv-orlando has quit IRC | 06:15 | |
| *** vivsoni has quit IRC | 06:21 | |
| *** vivsoni has joined #openstack-cinder | 06:21 | |
| *** salv-orlando has joined #openstack-cinder | 06:21 | |
| openstackgerrit | Guy Kim proposed openstack/cinder master: Update configuration names in ITRI DISCO volume driver https://review.openstack.org/461665 | 06:26 |
| openstackgerrit | Shay Halsband proposed openstack/cinder master: XtremIO: Detach all IGs for null connector https://review.openstack.org/477422 | 06:27 |
| *** Julien-zte has joined #openstack-cinder | 06:27 | |
| *** vivsoni_ has joined #openstack-cinder | 06:29 | |
| openstackgerrit | jiamin proposed openstack/cinder master: DS8K driver: specify pool and lss in extra-specs https://review.openstack.org/435857 | 06:30 |
| *** isaacb has quit IRC | 06:31 | |
| *** vivsoni has quit IRC | 06:31 | |
| *** salv-orl_ has joined #openstack-cinder | 06:37 | |
| *** zhonghua has joined #openstack-cinder | 06:37 | |
| *** salv-orlando has quit IRC | 06:40 | |
| *** zhonghua2 has quit IRC | 06:40 | |
| *** dhinesh has quit IRC | 06:52 | |
| *** kaisers_ has quit IRC | 06:53 | |
| *** med_ has quit IRC | 06:53 | |
| *** med_ has joined #openstack-cinder | 06:59 | |
| *** med_ is now known as Guest39045 | 06:59 | |
| openstackgerrit | Cheng Yong Zhao proposed openstack/cinder master: Add gmcv support in SVC driver https://review.openstack.org/461191 | 07:01 |
| *** amito has joined #openstack-cinder | 07:02 | |
| *** amito has left #openstack-cinder | 07:02 | |
| *** Shunli has quit IRC | 07:03 | |
| *** wanghao has quit IRC | 07:03 | |
| *** Jack_Iv has joined #openstack-cinder | 07:04 | |
| *** wanghao has joined #openstack-cinder | 07:05 | |
| *** Jack_Iv has quit IRC | 07:07 | |
| *** tesseract has joined #openstack-cinder | 07:08 | |
| *** markstur has quit IRC | 07:14 | |
| *** markstur has joined #openstack-cinder | 07:15 | |
| *** markstur has quit IRC | 07:15 | |
| *** zhonghua has quit IRC | 07:24 | |
| *** MasterofJOKers_ has joined #openstack-cinder | 07:36 | |
| *** MasterofJOKers has quit IRC | 07:38 | |
| *** MasterofJOKers_ is now known as MasterofJOKers | 07:38 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Add driver interface documentation for list manageable https://review.openstack.org/478415 | 07:47 |
| *** kaisers_ has joined #openstack-cinder | 07:50 | |
| *** gkadam is now known as gkadam-afk | 07:53 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-cinder | 08:00 | |
| openstackgerrit | Guy Kim proposed openstack/cinder master: Update configuration names in ITRI DISCO volume driver https://review.openstack.org/461665 | 08:15 |
| *** Shunli has joined #openstack-cinder | 08:18 | |
| *** kaisers_ has quit IRC | 08:20 | |
| *** Shunli has quit IRC | 08:23 | |
| *** Shunli has joined #openstack-cinder | 08:23 | |
| *** lkuchlan_wfh is now known as lkuchlan_away | 08:27 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder master: Fix some doc issue https://review.openstack.org/478430 | 08:27 |
| *** s-shiono has quit IRC | 08:31 | |
| *** Shunli has quit IRC | 08:31 | |
| openstackgerrit | Jeremy Zhang proposed openstack/cinder master: [api-ref] Update a volume's metadata for a specific key (v2) https://review.openstack.org/478435 | 08:33 |
| *** Poornima has quit IRC | 08:37 | |
| *** isaacb has joined #openstack-cinder | 08:37 | |
| *** lkuchlan_away has quit IRC | 08:41 | |
| *** Poornima has joined #openstack-cinder | 08:41 | |
| *** lkuchlan_away has joined #openstack-cinder | 08:41 | |
| *** lpetrut has joined #openstack-cinder | 08:41 | |
| *** crushil_ has joined #openstack-cinder | 08:43 | |
| *** tsuzuki has quit IRC | 08:44 | |
| *** Jack_Iv has joined #openstack-cinder | 08:52 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder master: [Api-ref] Update quota-set APIs ref https://review.openstack.org/474048 | 08:53 |
| *** zhangguo_ has joined #openstack-cinder | 08:54 | |
| *** zhangguoqing has quit IRC | 08:54 | |
| *** zhangguo_ has quit IRC | 08:55 | |
| *** zhangguoqing has joined #openstack-cinder | 08:58 | |
| *** zhangguoqing has quit IRC | 08:59 | |
| *** zhangguoqing has joined #openstack-cinder | 08:59 | |
| *** dalgaaf has quit IRC | 09:04 | |
| openstackgerrit | ravi edpuganti proposed openstack/cinder master: Added CG capability to volume group in CoprHD https://review.openstack.org/459610 | 09:05 |
| *** crushil_ has quit IRC | 09:05 | |
| *** dalgaaf has joined #openstack-cinder | 09:06 | |
| openstackgerrit | Jeremy Zhang proposed openstack/cinder master: [api-ref] Update a volume's metadata for a specific key (v2) https://review.openstack.org/478435 | 09:11 |
| *** pgadiya has quit IRC | 09:15 | |
| *** kaisers_ has joined #openstack-cinder | 09:17 | |
| openstackgerrit | junbo.li proposed openstack/python-cinderclient master: Fix force delete volume in v3.23 https://review.openstack.org/478447 | 09:19 |
| *** pgadiya has joined #openstack-cinder | 09:23 | |
| *** pcaruana has joined #openstack-cinder | 09:28 | |
| *** wanghao has quit IRC | 09:35 | |
| *** Wenjun has quit IRC | 09:37 | |
| openstackgerrit | jiamin proposed openstack/cinder master: DS8K: add replication consistency group support https://review.openstack.org/465877 | 09:37 |
| *** stakeda has quit IRC | 09:38 | |
| *** mvk has quit IRC | 09:46 | |
| *** kaisers_ has quit IRC | 09:47 | |
| *** rcernin has quit IRC | 09:55 | |
| *** Poornima has quit IRC | 09:57 | |
| *** gkadam-afk is now known as gkadam | 09:59 | |
| *** vivsoni_ has quit IRC | 10:00 | |
| *** vivsoni has joined #openstack-cinder | 10:00 | |
| *** rcernin has joined #openstack-cinder | 10:03 | |
| *** vivsoni_ has joined #openstack-cinder | 10:05 | |
| *** vivsoni has quit IRC | 10:05 | |
| *** sdague has joined #openstack-cinder | 10:07 | |
| *** lkuchlan_away is now known as lkuchlan_wfh | 10:08 | |
| *** rcernin has quit IRC | 10:09 | |
| *** rcernin has joined #openstack-cinder | 10:11 | |
| *** mvk has joined #openstack-cinder | 10:13 | |
| *** kaisers_ has joined #openstack-cinder | 10:43 | |
| *** Poornima has joined #openstack-cinder | 10:52 | |
| *** liverpooler has quit IRC | 10:55 | |
| *** amit213 has quit IRC | 11:02 | |
| *** Jack_Iv has quit IRC | 11:05 | |
| *** vivsoni_ has quit IRC | 11:05 | |
| *** vivsoni has joined #openstack-cinder | 11:05 | |
| *** Jack_Iv has joined #openstack-cinder | 11:06 | |
| *** kaisers_ has quit IRC | 11:07 | |
| *** Jack_Iv has quit IRC | 11:10 | |
| *** adisky__ has quit IRC | 11:25 | |
| *** ericyoung has joined #openstack-cinder | 11:34 | |
| *** dillaman has joined #openstack-cinder | 11:34 | |
| *** Jack_Iv has joined #openstack-cinder | 11:35 | |
| *** youzp has quit IRC | 11:47 | |
| *** Jack_Iv has quit IRC | 11:48 | |
| *** arne_r has joined #openstack-cinder | 11:49 | |
| *** Jack_Iv has joined #openstack-cinder | 11:49 | |
| *** arne_r has quit IRC | 11:49 | |
| *** arne_r has joined #openstack-cinder | 11:49 | |
| *** Jack_Iv_ has joined #openstack-cinder | 11:50 | |
| *** Jack_Iv has quit IRC | 11:50 | |
| *** Julien-zte has quit IRC | 11:51 | |
| *** salv-orl_ has quit IRC | 11:51 | |
| *** salv-orlando has joined #openstack-cinder | 11:51 | |
| *** salv-orlando has quit IRC | 11:56 | |
| *** abishop has joined #openstack-cinder | 11:57 | |
| *** pcaruana has quit IRC | 12:02 | |
| *** pcaruana has joined #openstack-cinder | 12:02 | |
| *** kaisers_ has joined #openstack-cinder | 12:04 | |
| *** Poornima has quit IRC | 12:11 | |
| *** chlong_ has quit IRC | 12:15 | |
| *** pcaruana has quit IRC | 12:16 | |
| *** pcaruana has joined #openstack-cinder | 12:23 | |
| *** pcaruana has quit IRC | 12:23 | |
| *** pcaruana has joined #openstack-cinder | 12:23 | |
| *** pgadiya has quit IRC | 12:24 | |
| *** kaisers_ has quit IRC | 12:28 | |
| *** pcaruana has quit IRC | 12:29 | |
| *** pcaruana has joined #openstack-cinder | 12:37 | |
| *** pcaruana has quit IRC | 12:43 | |
| *** pcaruana has joined #openstack-cinder | 12:44 | |
| *** MVenesio has joined #openstack-cinder | 12:44 | |
| *** youzp has joined #openstack-cinder | 12:45 | |
| *** jmlowe_ has quit IRC | 12:50 | |
| *** gkadam has quit IRC | 12:51 | |
| *** youzp has quit IRC | 12:52 | |
| *** Julien-zte has joined #openstack-cinder | 12:52 | |
| *** pcaruana has quit IRC | 12:53 | |
| *** liverpooler has joined #openstack-cinder | 12:54 | |
| *** pewp has quit IRC | 12:54 | |
| *** pewp has joined #openstack-cinder | 12:56 | |
| *** youzp has joined #openstack-cinder | 12:59 | |
| *** gouthamr has joined #openstack-cinder | 13:01 | |
| *** pcaruana has joined #openstack-cinder | 13:05 | |
| *** wangyong has quit IRC | 13:05 | |
| *** pcaruana has quit IRC | 13:13 | |
| *** lucasxu has joined #openstack-cinder | 13:14 | |
| *** pcaruana has joined #openstack-cinder | 13:14 | |
| openstackgerrit | Merged openstack/cinder master: ScaleIO Driver: Fix for using scheduler specified pool https://review.openstack.org/476116 | 13:14 |
| *** links has quit IRC | 13:16 | |
| *** dustins has joined #openstack-cinder | 13:19 | |
| *** kaisers_ has joined #openstack-cinder | 13:26 | |
| *** adisky__ has joined #openstack-cinder | 13:31 | |
| *** links has joined #openstack-cinder | 13:33 | |
| *** chlong_ has joined #openstack-cinder | 13:34 | |
| *** jmlowe has joined #openstack-cinder | 13:35 | |
| *** vivsoni_ has joined #openstack-cinder | 13:36 | |
| *** vivsoni has quit IRC | 13:36 | |
| *** kaisers_ has quit IRC | 13:37 | |
| *** eharney has joined #openstack-cinder | 13:37 | |
| *** zhurong_ has joined #openstack-cinder | 13:38 | |
| arnewiebalck | Still preparing our upgrade from Mitaka to Ocata, we’re getting errors like “Extension SchedulerHints: Cannot extend resource volumes: No such resource” on c-api startup in Ocata … does anyone know immediately what is causing this (before we start debugging)? | 13:38 |
| *** pcaruana has quit IRC | 13:40 | |
| *** arne_r has quit IRC | 13:46 | |
| *** pcaruana has joined #openstack-cinder | 13:46 | |
| *** anxys has joined #openstack-cinder | 13:52 | |
| *** gouthamr has quit IRC | 13:52 | |
| *** links has quit IRC | 13:56 | |
| openstackgerrit | Vadim Kryvian proposed openstack/cinder master: IBM Storage: Fix for misidentification of QoS type https://review.openstack.org/477424 | 13:56 |
| *** zhangguoqing has quit IRC | 13:57 | |
| *** Poornima has joined #openstack-cinder | 13:58 | |
| *** anxys has quit IRC | 14:02 | |
| *** youzp has quit IRC | 14:03 | |
| *** gouthamr has joined #openstack-cinder | 14:05 | |
| *** lbragstad_ is now known as lbragstad | 14:07 | |
| *** xyang1 has joined #openstack-cinder | 14:10 | |
| *** eharney has quit IRC | 14:16 | |
| *** bswartz has joined #openstack-cinder | 14:18 | |
| *** youzp has joined #openstack-cinder | 14:19 | |
| *** youzp has quit IRC | 14:23 | |
| *** eharney has joined #openstack-cinder | 14:28 | |
| *** felipemonteiro_ has joined #openstack-cinder | 14:33 | |
| *** kaisers_ has joined #openstack-cinder | 14:34 | |
| *** felipemonteiro__ has joined #openstack-cinder | 14:34 | |
| *** gouthamr has quit IRC | 14:34 | |
| *** salv-orlando has joined #openstack-cinder | 14:37 | |
| *** felipemonteiro_ has quit IRC | 14:38 | |
| *** crose has joined #openstack-cinder | 14:38 | |
| eharney | do any third-party CIs support Depends-On: in commit messages? | 14:40 |
| *** gouthamr has joined #openstack-cinder | 14:42 | |
| openstackgerrit | Eric Harney proposed openstack/cinder master: Test new tempest encryption test https://review.openstack.org/478162 | 14:42 |
| *** mtanino__ has joined #openstack-cinder | 14:46 | |
| *** markstur has joined #openstack-cinder | 14:57 | |
| *** isaacb has quit IRC | 14:58 | |
| *** pewp has quit IRC | 15:00 | |
| *** zhurong_ has quit IRC | 15:03 | |
| *** pewp has joined #openstack-cinder | 15:03 | |
| *** KeithMnemonic has joined #openstack-cinder | 15:04 | |
| *** kaisers_ has quit IRC | 15:05 | |
| *** rcernin has quit IRC | 15:06 | |
| *** links has joined #openstack-cinder | 15:07 | |
| *** Poornima has quit IRC | 15:08 | |
| *** lhx__ has joined #openstack-cinder | 15:11 | |
| *** edisonxiang has quit IRC | 15:12 | |
| *** edisonxiang has joined #openstack-cinder | 15:13 | |
| *** gnarld_ is now known as cFouts | 15:14 | |
| *** Julien-zte has quit IRC | 15:34 | |
| *** Jack_Iv_ has quit IRC | 15:35 | |
| *** Julien-zte has joined #openstack-cinder | 15:35 | |
| *** Julien-zte has quit IRC | 15:35 | |
| *** Jack_Iv has joined #openstack-cinder | 15:35 | |
| *** Julien-zte has joined #openstack-cinder | 15:35 | |
| *** Julien-zte has quit IRC | 15:36 | |
| *** links has quit IRC | 15:36 | |
| *** Julien-zte has joined #openstack-cinder | 15:36 | |
| *** Julien-zte has quit IRC | 15:36 | |
| *** Julien-zte has joined #openstack-cinder | 15:37 | |
| *** Julien-zte has quit IRC | 15:37 | |
| *** Julien-zte has joined #openstack-cinder | 15:38 | |
| *** Julien-zte has quit IRC | 15:38 | |
| *** Julien-zte has joined #openstack-cinder | 15:38 | |
| *** Julien-zte has quit IRC | 15:39 | |
| *** Julien-zte has joined #openstack-cinder | 15:39 | |
| *** Julien-zte has quit IRC | 15:40 | |
| *** Julien-zte has joined #openstack-cinder | 15:40 | |
| *** Julien-zte has quit IRC | 15:40 | |
| *** Julien-zte has joined #openstack-cinder | 15:41 | |
| *** Julien-zte has quit IRC | 15:41 | |
| *** salv-orlando has quit IRC | 15:44 | |
| *** salv-orlando has joined #openstack-cinder | 15:45 | |
| *** salv-orlando has quit IRC | 15:49 | |
| *** e0ne has joined #openstack-cinder | 15:51 | |
| openstackgerrit | Merged openstack/cinder master: Add IBM-Storage replication group support https://review.openstack.org/463314 | 15:56 |
| openstackgerrit | Merged openstack/cinder master: Add params for v3 API volume-type-create https://review.openstack.org/476881 | 15:56 |
| *** lkuchlan_wfh has quit IRC | 15:59 | |
| *** salv-orlando has joined #openstack-cinder | 16:00 | |
| *** itlinux has joined #openstack-cinder | 16:01 | |
| *** xyang1 has quit IRC | 16:02 | |
| *** kaisers_ has joined #openstack-cinder | 16:02 | |
| openstackgerrit | Andreas Karis proposed openstack/cinder master: Enable service capabilities update at less than 60 seconds https://review.openstack.org/469936 | 16:02 |
| *** xyang1 has joined #openstack-cinder | 16:03 | |
| *** xyang1 has quit IRC | 16:07 | |
| *** xyang1 has joined #openstack-cinder | 16:07 | |
| *** diablo_rojo has joined #openstack-cinder | 16:07 | |
| *** _alastor1 is now known as _alastor_ | 16:10 | |
| *** psachin has quit IRC | 16:10 | |
| *** admin0 has joined #openstack-cinder | 16:19 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder master: Rename DB Driver param for backup drivers to 'db' https://review.openstack.org/475394 | 16:20 |
| *** kaisers_ has quit IRC | 16:25 | |
| *** Apoorva has joined #openstack-cinder | 16:26 | |
| *** Apoorva has quit IRC | 16:28 | |
| *** dfisher has joined #openstack-cinder | 16:30 | |
| *** lhx__ has quit IRC | 16:32 | |
| *** Apoorva has joined #openstack-cinder | 16:41 | |
| openstackgerrit | Chris M proposed openstack/cinder master: Fix for tempest.api.volume.admin.test_volumes_actions.VolumesActionsTest.test_force_detach_volume https://review.openstack.org/478563 | 16:54 |
| *** eric_young has joined #openstack-cinder | 16:58 | |
| hemna | fwiw, it doesn't look like cinder does anything with the SIGHUP | 17:00 |
| hemna | https://github.com/openstack/cinder/blob/master/cinder/manager.py#L138-L146 | 17:00 |
| hemna | unless I'm missing something | 17:00 |
| lbragstad | hey folks - quick review request from a cinder meeting a few weeks ago | 17:00 |
| hemna | the service calls that afaik | 17:00 |
| lbragstad | we have a series of specs proposed to define and cleaup policy work | 17:00 |
| lbragstad | https://review.openstack.org/#/c/460344/12 | 17:00 |
| lbragstad | https://review.openstack.org/#/c/462733/11 | 17:01 |
| lbragstad | and finally https://review.openstack.org/#/c/464763/ | 17:01 |
| eharney | hemna: yeah, my suspicion is that before that version pin change, it at least attempted to reload the config via oslo.service, and that has maybe now been overridden | 17:01 |
| lbragstad | it'd be great to get opinions from the cinder folks on those before we start working on implementations for queens ^ | 17:01 |
| hemna | eharney, any idea if other projects have tried going this route? | 17:02 |
| eharney | hemna: i haven't looked | 17:02 |
| hemna | seems like the reload of the config should be in oslo | 17:02 |
| eharney | it is | 17:02 |
| eharney | well, in some form, i think | 17:02 |
| e0ne | hemna: oslo.service has some code with conf reloading | 17:02 |
| eharney | but it gets tricky w/ multi-backend etc | 17:02 |
| hemna | and it should be up to cinder's manager.reset() to do something with the new config obj..... re: validate, reject changes and ignore, etc. | 17:02 |
| *** e0ne has quit IRC | 17:03 | |
| hemna | would be nice if there was a way for cinder to declare a validate() or something like that on the config object reload | 17:03 |
| hemna | then we could look at the changes and log stuffs and or reject the reload due to whatever | 17:03 |
| jungleboyj | tbarron: So, what is the concern? | 17:05 |
| jungleboyj | hemna: How could we do that? | 17:05 |
| hemna | well the previous discussion raised some issues about limiting the allowable changes in the conf | 17:05 |
| tbarron | jungleboyj: I guess I want to be able to go to a driverfixes branch and cherry-pick a driverfix w/o pulling in any core code | 17:06 |
| hemna | so it doesn't nuke the running instances | 17:06 |
| jungleboyj | tbarron: Exactly! | 17:06 |
| tbarron | untested core code that may or may not work | 17:06 |
| jungleboyj | A change that goes into driverfixes won't include any core code. It can't. | 17:06 |
| *** eric_young has quit IRC | 17:07 | |
| jungleboyj | We wouldn't merge such a patch. | 17:07 |
| tbarron | but I thought you were saying it would include crit/security fixes from core | 17:07 |
| tbarron | probably I was confused | 17:07 |
| jungleboyj | No, I was saying it wouldn't. | 17:07 |
| tbarron | jungleboyj: nm then - I probably just need lunch. | 17:07 |
| jungleboyj | I was saying that any driver crit/security issues that only touch the driver should be backported. | 17:08 |
| tbarron | jungleboyj: I mis-understood. | 17:08 |
| jungleboyj | tbarron: Cool. I thought that was the case. That was why I pushed forward. | 17:09 |
| tbarron | sorry for all that then. But I do disagree with that fuschia suggestion from jgriffith :D | 17:09 |
| *** salv-orlando has quit IRC | 17:10 | |
| jungleboyj | tbarron: *laughing* | 17:10 |
| jungleboyj | Me too, Too hard to spell. | 17:10 |
| jungleboyj | Ok, so we are all good on that front then. | 17:10 |
| *** salv-orlando has joined #openstack-cinder | 17:10 | |
| *** lpetrut has quit IRC | 17:11 | |
| jungleboyj | hemna: On the config option analysis. | 17:11 |
| *** lpetrut has joined #openstack-cinder | 17:11 | |
| jgriffith | tbarron Oh sureeee!! You probably prefer something like turquoise | 17:11 |
| tbarron | jgriffith: :D | 17:12 |
| jungleboyj | AHHHHHHH | 17:12 |
| jungleboyj | hemna: So, I know we have gone around on the fact that this could cause the end user to shoot themselves in the foot. | 17:12 |
| *** xyang1 has quit IRC | 17:13 | |
| hemna | jungleboyj, yah | 17:13 |
| jungleboyj | hemna: I thought, however, we decided that just get it working at this point and then go from there given that it isn't really that different from when the end user makes a change and then restarts the processes by hand. | 17:13 |
| hemna | there should be some sort of validating in place before allowing the new config object to get propagated into the running service. | 17:14 |
| hemna | well, I disagree with that | 17:14 |
| hemna | you have a running process that can get a new config object with completely different values in the middle of processing things. | 17:14 |
| hemna | many drivers get their config object at instantiation and then make assumptions based off that config object | 17:15 |
| *** salv-orlando has quit IRC | 17:15 | |
| hemna | some drivers thread out things to get stuffs done | 17:15 |
| hemna | config object changes .... | 17:15 |
| hemna | poof... | 17:15 |
| hemna | there is so much that can go wrong with this. | 17:16 |
| hemna | https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/hpe/hpe_3par_fc.py#L125-L126 | 17:17 |
| hemna | how does that work when the config options change after the instance is running? | 17:17 |
| hemna | a user changes those values in the config, and sighup's cinder and it doesn't work right now. | 17:18 |
| hemna | anyway, bleh | 17:18 |
| hemna | this just doesn't seem to be thought out. | 17:18 |
| *** diablo_rojo has quit IRC | 17:21 | |
| *** kaisers_ has joined #openstack-cinder | 17:21 | |
| jungleboyj | hemna: Fair enough. | 17:24 |
| jungleboyj | So, are you thinking that if we support this we just have a subset of variables that we process? | 17:25 |
| hemna | that would be safer to start with | 17:25 |
| hemna | and a way to validate/reject changes before it gets populated into the service | 17:25 |
| jungleboyj | Ok. | 17:26 |
| jungleboyj | So this goes back to looking at the spec again. | 17:26 |
| hemna | It wouldn't be hard to add a new method to the volume driver that driver maintainers could implement to do a check on config options. | 17:26 |
| hemna | they could return True/False for accept/reject | 17:26 |
| hemna | I think john had an idea of a new config section for allowable options to change too. | 17:27 |
| hemna | I don't remember how that ended up | 17:27 |
| *** lpetrut has quit IRC | 17:27 | |
| jungleboyj | :-) | 17:27 |
| hemna | but if we had a way to validate changes, we could flag options as mutable or something like that | 17:28 |
| hemna | and those could get checked automagically | 17:28 |
| jgriffith | hemna so here's my question, other than things like log levels, is mutable config changes really worth all of this? | 17:28 |
| hemna | so by default all of our options could be flagged as immutable | 17:28 |
| hemna | and we'd have to put up a patch to change that to allow said config option to change on sighup | 17:29 |
| jungleboyj | jgriffith: That was where my head was starting to go. | 17:29 |
| jgriffith | I'm sorry, but a lot of this is just sort of like solutions without real problems IMO | 17:29 |
| hemna | jgriffith, if you mean by not allowing SIGUP reloading, then yes I agree | 17:29 |
| jgriffith | hemna I mean, mutable configs in general | 17:29 |
| hemna | I just don't think this will really work IMHO | 17:29 |
| jgriffith | is it really a problem that needs fixing | 17:30 |
| *** reveller_ has joined #openstack-cinder | 17:30 | |
| hemna | I'm not sure what you are asking. | 17:30 |
| jgriffith | hemna yeah, I think you've dug into this more than most so it's awesome to get your feedback | 17:30 |
| hemna | are you asking do we need to allow SIGHUP reloading? | 17:30 |
| jgriffith | hemna really? Hmm... not sure how to be more clear | 17:30 |
| jgriffith | no forget sighup impl details | 17:30 |
| jgriffith | I'm asking "Do we need mutable config"? | 17:31 |
| hemna | oh, well I was just thinking out loud | 17:31 |
| *** reveller has quit IRC | 17:31 | |
| *** reveller_ is now known as reveller | 17:31 | |
| jgriffith | hemna ? | 17:31 |
| hemna | as a mechanism to reject config changes | 17:31 |
| hemna | during the SIGHUP reload process | 17:31 |
| jungleboyj | jgriffith: I think he is saying if we are going to make SIGHUP reloading work that we need to be able to make things mutable/immutable | 17:31 |
| jgriffith | hemna what I'm saying is that based on your analysis and what I've heard from you here it seems to me that the ability to modify configs online may not even be worth doing anyway | 17:32 |
| hemna | heh yah | 17:32 |
| hemna | that's what I think too | 17:32 |
| hemna | and was really why I started looking into what happens if we just blanket allow it | 17:32 |
| jgriffith | jungleboyj oh, ok... but I thought the whole point was that we needed SIGHUP to do the dynamic reloading? | 17:32 |
| jgriffith | jungleboyj so it seemed like we were going in circles there :) | 17:33 |
| jgriffith | My point being, if you let go of dynamic config changes then the SIGHUP thing is irrelevant isn't it? | 17:33 |
| jungleboyj | jgriffith: Right. So, then if we start making things immutable, is there still value here. | 17:33 |
| jungleboyj | jgriffith: Right. I am getting there too. | 17:33 |
| jgriffith | jungleboyj exactly | 17:33 |
| hemna | anyway, so if we want to go down this road of even allowing dynamic reloading, which I'm not sure we do, I believe we should protect config settings from being mutable. | 17:34 |
| jungleboyj | This started as being for log levels, but we have another solution going into play for that. | 17:34 |
| jgriffith | I may be completely wrong, but I'm not sure there's a real value in it | 17:34 |
| hemna | jgriffith, +1 | 17:34 |
| jgriffith | jungleboyj I think that merged didn't it? | 17:34 |
| *** gouthamr has quit IRC | 17:34 | |
| jungleboyj | jgriffith: I need to go look. I am not totally sure. | 17:35 |
| *** jdillaman has quit IRC | 17:36 | |
| *** diablo_rojo has joined #openstack-cinder | 17:36 | |
| hemna | so if we do want to go down the dynamic reload of config, then I think there is a sane way of allowing it, but adding a validation mechanism that checks a config option's mutable flag (new flag). By default all options are immutable (flag set to False). This would protect everything from changing by default. | 17:37 |
| hemna | Then as folks want to allow a specific option to be changed, they put a patch up that changes the mutable flag option to True, and the patch can test for it. | 17:38 |
| jgriffith | I think that's what the OSLO changes do actually | 17:38 |
| jgriffith | let me see if I can find what I'm talking about | 17:38 |
| hemna | jgriffith, heh ok cool :) | 17:38 |
| jgriffith | hemna jungleboyj this. https://review.openstack.org/#/c/464028/2 | 17:39 |
| jgriffith | hemna jungleboyj mutable param added to oslo.config | 17:39 |
| hemna | ok I guess I just designed what has already been implemented then :P | 17:40 |
| jgriffith | hemna isn't that pretty much exactly what you're saying? | 17:40 |
| jgriffith | haha! | 17:40 |
| jgriffith | Well then in that case it must be a good answer :) | 17:40 |
| jungleboyj | jgriffith: Damn! Had not idea that had happened. | 17:41 |
| jgriffith | jungleboyj tooo many patches! | 17:41 |
| hemna | I guess the only thing I am missing is what is the default mutable flag | 17:41 |
| hemna | on options that don't specify it. | 17:41 |
| jungleboyj | Tell me about. I have reviewed like 80 some the last few days. | 17:41 |
| hemna | I'm hoping it's False | 17:41 |
| jgriffith | default is NOT mutable | 17:41 |
| *** dhinesh has joined #openstack-cinder | 17:41 | |
| hemna | ok perfect | 17:41 |
| hemna | so I guess my work here is done. | 17:41 |
| hemna | next | 17:41 |
| jgriffith | hemna that would be train wreck if it were the other way around :) | 17:42 |
| hemna | yah pretty much | 17:42 |
| jungleboyj | jgriffith: Nice. So, that problem is solved. | 17:42 |
| jgriffith | call it a day!! | 17:42 |
| jgriffith | beer time | 17:42 |
| * jungleboyj goes to the fridge. | 17:43 | |
| jungleboyj | I should probably eat something first. | 17:43 |
| jungleboyj | and/or shower | 17:43 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder master: FC: refactor fczm utils decorators to functions https://review.openstack.org/472855 | 17:44 |
| hemna | jgriffith, so changing subjects.... | 17:44 |
| hemna | jgriffith, not sure if you remember a while back discussing the FCZM utils decorators or not | 17:44 |
| openstackgerrit | Kendall Nelson proposed openstack/cinder-specs master: Update Dynamic Reconfig Spec https://review.openstack.org/446132 | 17:44 |
| jgriffith | Yes, and I saw your change that removed it I think | 17:44 |
| diablo_rojo | jungleboyj, hemna eharney ^^ | 17:45 |
| hemna | but I thought the idea was to eliminate the decorators and just have the FCZM drivers call the utils manually when they needed it | 17:45 |
| jungleboyj | diablo_rojo: Will take a look. Have you followed the discussion here the last 45 minutes? | 17:45 |
| hemna | to help standardize on the returns for the terminate call | 17:45 |
| jgriffith | hemna been a long time, but that sounds right | 17:45 |
| hemna | so I put that patch up as a response to it.... | 17:45 |
| diablo_rojo | jungleboyj, no I had a different meeting, but I will read scrollback now. | 17:45 |
| hemna | https://review.openstack.org/#/c/472855/ | 17:45 |
| hemna | if we wanted to go that route | 17:46 |
| jungleboyj | diablo_rojo: Cool. Thanks. | 17:46 |
| jgriffith | hemna ohhh, look at you doing the painful driver updates!!! | 17:46 |
| hemna | yah :( | 17:46 |
| jgriffith | hemna I get it now, this was your sneaky way of asking me to do a review wasn't it :) | 17:46 |
| hemna | :) | 17:46 |
| jgriffith | I'll gladly look at this | 17:46 |
| jgriffith | but I'm still not doing FC :) | 17:47 |
| hemna | haha | 17:47 |
| *** links has joined #openstack-cinder | 17:47 | |
| hemna | still getting out of it eh? | 17:47 |
| hemna | good for you :) | 17:47 |
| jgriffith | until they fire me :) | 17:47 |
| jgriffith | just kidding, I think for us everyone agrees it just doesn't make a ton of sense | 17:47 |
| jgriffith | us==NTAP/SF | 17:48 |
| hemna | the long term outlook on FC in cinder doesn't look good anyway | 17:48 |
| hemna | I'm not sure, but I think the CI for FC hasn't been up for a while | 17:48 |
| jgriffith | :( | 17:48 |
| jgriffith | You know what that means | 17:48 |
| hemna | yah...... | 17:49 |
| hemna | I haven't seen angela from brocade in a long time in here | 17:49 |
| hemna | maybe smcginnis knows what's up. | 17:49 |
| diablo_rojo | jungleboyj, so it seems to me that the changes I made still stand? The first part of the conversation was a conversation we had back a month or so ago where we dove into Oslo and said it all should work how we want it to, but it doesn't and we don't know why. | 17:52 |
| hemna | diablo_rojo it doesn't work now, because our reset method in the service is basically a noop | 17:53 |
| jungleboyj | diablo_rojo: Sounds right. Then the next step is to figure out the where we want things to be mutable going forward. | 17:53 |
| hemna | from what I can tell it does nothing | 17:53 |
| hemna | but I think there are also doubts if we even should do this. | 17:53 |
| diablo_rojo | hemna right. That's what I wanted to chat about last week, but not everyone was around. | 17:54 |
| diablo_rojo | hemna, if you want to explain why you don't think its a good idea on the spec I would appreciate it :) | 17:54 |
| hemna | well, just looking at the drivers | 17:55 |
| hemna | they all make assumptions that the config is set at object instantiation time | 17:55 |
| hemna | and there is no mechanism for safely updating that object in the existing driver object | 17:55 |
| hemna | there can be stuff in flight in the driver | 17:55 |
| hemna | I haven't looked at the spec in a while | 17:56 |
| diablo_rojo | hemna, https://review.openstack.org/446132 | 17:56 |
| hemna | but I'm hoping there is a new mechanism you are adding to the base volume driver to allow for the driver to get notified it has a new config object | 17:56 |
| hemna | and explain what the driver is supposed to do with it, and/or how to reject it | 17:56 |
| diablo_rojo | I don't have anything like that in the spec currently. | 17:57 |
| hemna | :( | 17:57 |
| hemna | ok | 17:57 |
| diablo_rojo | That sounds big enough to be its own spec, but maybe I'm wrong. | 17:57 |
| hemna | well, I'd bet that the config changes would bork most drivers | 17:57 |
| diablo_rojo | hemna, that was what I was afraid of. | 17:57 |
| hemna | there is no mechanism for drivers to be aware of the new config object | 17:58 |
| jungleboyj | hemna: But we are saying the config options are not mutable by default. | 17:58 |
| hemna | I think this needs a lot more thought and protections in place | 17:58 |
| hemna | jungleboyj, sure | 17:58 |
| jungleboyj | So we get the reload working and then choose which things can be mutable. Right? | 17:58 |
| hemna | but, lets say I update my 3PAR driver to allow an option to be mutable | 17:58 |
| hemna | there is no mechanism for the driver to get that change and do something with it | 17:59 |
| hemna | and according to diablo_rojo, that's not even addressed in the spec (again I haven't read it lately) | 17:59 |
| *** Apoorva_ has joined #openstack-cinder | 17:59 | |
| hemna | so it simply won't work | 17:59 |
| hemna | IMHO | 17:59 |
| diablo_rojo | hemna, the closest I have is that it will require some driver updates- I don't go into much if any detail about them. | 17:59 |
| hemna | heh | 18:00 |
| hemna | ok | 18:00 |
| hemna | I'd like to see that flushed out | 18:00 |
| jungleboyj | :-) | 18:00 |
| diablo_rojo | Sounds like the whole spec needs to be rewritten | 18:00 |
| jungleboyj | diablo_rojo: hemna and I will take a look. Would like eharney in there too and smcginnis when he gets back. | 18:00 |
| jungleboyj | Lets work it through the spec. | 18:01 |
| diablo_rojo | Yes please, I feel like I am going around in circles with this and making no progress. A consensus would be spectacular. | 18:01 |
| hemna | well, maybe not rewritten, but at least a detailed discussion how you plan to add the mechanism for updating existing driver objects | 18:01 |
| jungleboyj | At this point I don't think there is any hurry on this but I also don't want it to just be forgotten. | 18:01 |
| jungleboyj | hemna: | 18:01 |
| jungleboyj | Agreed. | 18:01 |
| jungleboyj | diablo_rojo: You aren't going in circles. Our understanding has just changed over time. | 18:02 |
| hemna | diablo_rojo, I just spent a few minutes looking at drivers and stuffs, and this came up | 18:02 |
| *** kaisers_ has quit IRC | 18:02 | |
| *** kaisers_ has joined #openstack-cinder | 18:02 | |
| *** Apoorva has quit IRC | 18:02 | |
| hemna | which caused me to raise this issue and discuss it in the meeting today and here. | 18:02 |
| hemna | I think we need to be very careful with this | 18:03 |
| diablo_rojo | The drivers part is new definitely, but the previous discussion has happened before. | 18:03 |
| hemna | this has the potential to allow operators to really screw things up badly | 18:03 |
| diablo_rojo | Agreed. There is a lot of potential to bomb cinder. | 18:03 |
| hemna | cinder itself and the backends | 18:04 |
| jungleboyj | hemna: Lets note try to swallow an elephant. | 18:04 |
| hemna | jungleboyj, are you threatening me? | 18:04 |
| hemna | @!b jungleboyj | 18:05 |
| pewp | hemna (╯°□°)╯︵ ┻━┻ ︵ ╯(°□° ╯) jungleboyj | 18:05 |
| hemna | :P | 18:05 |
| jungleboyj | 1) Fix SIGHUP. 2) Make sure that things a mutable by default as advertised. | 18:05 |
| jungleboyj | 3) Get basic things like log level working as a PoC. | 18:05 |
| jungleboyj | 4) Address other options 1 by 1. | 18:06 |
| jungleboyj | If drivers want to make things dynamic, then they can do so. | 18:07 |
| *** kaisers_ has quit IRC | 18:07 | |
| jungleboyj | hemna: Make sense? | 18:08 |
| hemna | well, drivers can't do anything right now (with the current spec) | 18:09 |
| hemna | it simply won't work/do anything. | 18:09 |
| hemna | but yah | 18:09 |
| jungleboyj | hemna: Right. So I am going to go comment on the spec that we should do what I have listed above. | 18:10 |
| *** dillaman has quit IRC | 18:10 | |
| hemna | ok | 18:10 |
| jungleboyj | Everyone wants to boil the ocean. We can't do that. | 18:10 |
| hemna | I can comment after that if diablo_rojo wants me to. | 18:10 |
| diablo_rojo | hemna, would you be willing to coauthor this spec/ help with the effort? I don't seem to have the knowledge base that I need to make progress here. | 18:10 |
| diablo_rojo | hemna, I will take all the help I can get :) | 18:10 |
| hemna | I don't see this as boiling the ocean | 18:10 |
| hemna | but making sure it will work | 18:11 |
| jungleboyj | Agreed but we can do that in small bite size pieces. | 18:11 |
| hemna | isn't that what specs are for? :) | 18:11 |
| jungleboyj | Yep. | 18:11 |
| diablo_rojo | It is. | 18:12 |
| *** eharney has quit IRC | 18:12 | |
| jungleboyj | Ok, I am going to add these thoughts. hemna You update after that and we will keep inching this forward. | 18:13 |
| *** eharney has joined #openstack-cinder | 18:15 | |
| *** kaisers_ has joined #openstack-cinder | 18:16 | |
| *** lpetrut has joined #openstack-cinder | 18:20 | |
| *** kaisers_ has quit IRC | 18:22 | |
| *** mvk has quit IRC | 18:22 | |
| *** dillaman has joined #openstack-cinder | 18:23 | |
| *** links has quit IRC | 18:30 | |
| *** adisky__ has quit IRC | 18:34 | |
| *** kaisers_ has joined #openstack-cinder | 18:38 | |
| *** kaisers_ has quit IRC | 18:42 | |
| *** ChubYann has joined #openstack-cinder | 18:42 | |
| *** kaisers_ has joined #openstack-cinder | 18:44 | |
| *** arnewiebalck_ has joined #openstack-cinder | 18:49 | |
| jungleboyj | diablo_rojo: hemna I have responded to the spec. Let me know what you all think. | 18:51 |
| *** jmlowe has quit IRC | 18:53 | |
| *** jmlowe has joined #openstack-cinder | 18:56 | |
| *** dillaman has quit IRC | 18:57 | |
| *** Jack_Iv has quit IRC | 19:05 | |
| *** Jack_Iv has joined #openstack-cinder | 19:05 | |
| *** Apoorva_ has quit IRC | 19:08 | |
| *** Jack_Iv has quit IRC | 19:09 | |
| *** kaisers_ has quit IRC | 19:10 | |
| *** tesseract has quit IRC | 19:13 | |
| *** Jack_Iv has joined #openstack-cinder | 19:14 | |
| *** Jack_Iv has quit IRC | 19:18 | |
| *** dillaman has joined #openstack-cinder | 19:18 | |
| *** Jack_Iv has joined #openstack-cinder | 19:18 | |
| *** Guest28796 is now known as mgagne | 19:19 | |
| *** mgagne has quit IRC | 19:19 | |
| *** mgagne has joined #openstack-cinder | 19:19 | |
| *** xyang1 has joined #openstack-cinder | 19:23 | |
| *** arnewiebalck_ has quit IRC | 19:29 | |
| *** crose has quit IRC | 19:30 | |
| *** markstur has quit IRC | 19:32 | |
| *** salv-orlando has joined #openstack-cinder | 19:44 | |
| openstackgerrit | Eric Harney proposed openstack/os-brick master: Fix ceph incremental backup fail https://review.openstack.org/476503 | 19:49 |
| *** vivsoni_ has quit IRC | 19:55 | |
| *** jmlowe has quit IRC | 19:57 | |
| *** jmlowe has joined #openstack-cinder | 19:57 | |
| *** liverpooler has quit IRC | 19:58 | |
| *** xyang1 has quit IRC | 19:59 | |
| *** kaisers_ has joined #openstack-cinder | 20:03 | |
| *** felipemonteiro__ has quit IRC | 20:03 | |
| *** ericyoung has quit IRC | 20:15 | |
| *** Jack_Iv has quit IRC | 20:15 | |
| *** Jack_Iv has joined #openstack-cinder | 20:16 | |
| *** Jack_Iv has quit IRC | 20:20 | |
| *** Apoorva has joined #openstack-cinder | 20:25 | |
| *** lucasxu has quit IRC | 20:30 | |
| *** samarendra has quit IRC | 20:47 | |
| *** MVenesio has quit IRC | 20:48 | |
| *** Swanson has quit IRC | 20:48 | |
| *** markstur has joined #openstack-cinder | 20:49 | |
| *** markstur has quit IRC | 20:54 | |
| *** dustins has quit IRC | 20:58 | |
| openstackgerrit | Chaynika Saikia proposed openstack/cinder master: Ceph incremental backup uses _snap_exists() to check if a snapshot exists. Here while checking if the name of the snapshot matches the one passed in the method, it uses snap.name where snap is actually a dictionary. Therefore, the correct line should be t https://review.openstack.org/478633 | 21:00 |
| *** felipemonteiro has joined #openstack-cinder | 21:00 | |
| *** jmlowe has quit IRC | 21:01 | |
| *** pcaruana has quit IRC | 21:01 | |
| *** Jack_Iv has joined #openstack-cinder | 21:02 | |
| openstackgerrit | Chaynika Saikia proposed openstack/cinder master: Ceph incremental backup fails with AttributeError in _snap_exists() because it tries to look for name attribute while snap is actually a dictionary. https://review.openstack.org/478633 | 21:03 |
| *** youzp has joined #openstack-cinder | 21:04 | |
| *** felipemonteiro_ has joined #openstack-cinder | 21:07 | |
| *** Jack_Iv has quit IRC | 21:07 | |
| *** youzp has quit IRC | 21:08 | |
| *** felipemonteiro has quit IRC | 21:10 | |
| openstackgerrit | Chaynika Saikia proposed openstack/cinder master: Ceph incremental backup fails because it looks for name attr in snap dict. https://review.openstack.org/478633 | 21:11 |
| *** markstur has joined #openstack-cinder | 21:12 | |
| diablo_rojo | jungleboyj, thanks! When I get comments from hemna I will do another pass and incorporate all the new comments. | 21:17 |
| jungleboyj | diablo_rojo: Welcome. Hope I properly captured todays dicussion. | 21:17 |
| *** Jack_Iv has joined #openstack-cinder | 21:17 | |
| * jungleboyj is celebrating! I have storage to play with!!!!!! | 21:21 | |
| *** Jack_Iv has quit IRC | 21:22 | |
| *** eharney has quit IRC | 21:26 | |
| openstackgerrit | Eric Young proposed openstack/cinder master: [WIP] ScaleIo Driver: refactor config options https://review.openstack.org/478644 | 21:28 |
| *** Jack_Iv has joined #openstack-cinder | 21:33 | |
| *** abishop has quit IRC | 21:35 | |
| *** itlinux has quit IRC | 21:35 | |
| *** Jack_Iv has quit IRC | 21:38 | |
| *** lpetrut has quit IRC | 21:40 | |
| *** felipemonteiro_ has quit IRC | 21:41 | |
| *** felipemonteiro_ has joined #openstack-cinder | 21:41 | |
| openstackgerrit | Chaynika Saikia proposed openstack/python-cinderclient master: Add cinder create --poll https://review.openstack.org/475919 | 21:47 |
| patrickeast | jgriffith: ok, so, i'm *finally* getting around to setting up a test env to play around with the new attach/detach stuff, is there a like tl;dr list of patches i need to apply/instructions to get it working? | 21:50 |
| patrickeast | jgriffith: i was looking at https://etherpad.openstack.org/p/cinder-nova-api-changes but i'm pretty sure i've seen other ones floating around that aren't on there too... | 21:51 |
| openstackgerrit | Chaynika Saikia proposed openstack/python-cinderclient master: Add cinder create --poll https://review.openstack.org/475919 | 21:53 |
| *** markstur has quit IRC | 21:55 | |
| *** Swanson has joined #openstack-cinder | 21:57 | |
| ildikov | patrickeast: do you want to play with the basics only? | 21:57 |
| ildikov | patrickeast: like being able to attach and detach? | 21:58 |
| ildikov | patrickeast: this is the list of open reviews: https://review.openstack.org/#/q/topic:bp/cinder-new-attach-apis | 21:58 |
| hemna | patrickeast, also, the brick cinderclient extension can use the new attach workflow | 21:59 |
| hemna | if you just want to test out cinder and your driver | 21:59 |
| ildikov | patrickeast: you'll need this to be able to attach with the new flow: https://review.openstack.org/#/c/330285/ | 21:59 |
| ildikov | hemna: I temporarily fixed target_lun in the attach PoC ^^ | 21:59 |
| patrickeast | ildikov: no, i've got two arrays, 1 controller and 2 compute nodes | 21:59 |
| patrickeast | ildikov: ready to hit *all* the weird stuff with migrations and all that | 22:00 |
| ildikov | hemna: as I couldn't get the gate to pick your fix up :/ | 22:00 |
| hemna | ildikov, nice | 22:00 |
| hemna | boo | 22:00 |
| patrickeast | ildikov: hemna: thanks for the links! | 22:00 |
| ildikov | hemna: BTW, do you know when brick will be released? | 22:00 |
| ildikov | patrickeast: the attach PoC is now on top of the swap change | 22:00 |
| hemna | ildikov, no, that's kinda up to smcginnis | 22:01 |
| ildikov | patrickeast: tempest is basically happy except rebuild and shelved offloaded for detach | 22:01 |
| ildikov | hemna: ok, tnx | 22:01 |
| patrickeast | we probably should cut a release relatively soon with all the fixes for multipath stuff that has gone in | 22:01 |
| hemna | yah | 22:01 |
| ildikov | patrickeast: there's a separate test for live_migrate | 22:01 |
| hemna | so we have time to see what else is broken due to those big changes | 22:01 |
| * hemna is a bit worried | 22:02 | |
| ildikov | patrickeast: the live_migrate one is your target for the attachment_delete issue for sure | 22:02 |
| ildikov | especially if you have a multi-node env | 22:02 |
| ildikov | you will still need the attach PoC as that one enables to attach with the new flow and unless a volume is attached with the new flow it'll always fall back to the old regardless of versions | 22:02 |
| patrickeast | gotcha | 22:03 |
| ildikov | patrickeast: lemme know if you have more questions, I'll try my best to answer :) | 22:03 |
| ildikov | patrickeast: and thanks much for looking into it! | 22:03 |
| patrickeast | ildikov: thanks, will do | 22:03 |
| *** mriedem has joined #openstack-cinder | 22:09 | |
| openstackgerrit | Chaynika Saikia proposed openstack/python-cinderclient master: Add cinder create --poll https://review.openstack.org/475919 | 22:14 |
| *** salv-orl_ has joined #openstack-cinder | 22:14 | |
| *** dfisher has left #openstack-cinder | 22:16 | |
| *** mvk has joined #openstack-cinder | 22:17 | |
| *** salv-orlando has quit IRC | 22:18 | |
| *** chlong_ has quit IRC | 22:18 | |
| *** Jack_Iv has joined #openstack-cinder | 22:19 | |
| *** Jack_Iv has quit IRC | 22:24 | |
| jungleboyj | patrickeast: Sounds like we are going to get the HA support for the Lenovo driver here pretty soon so you won't be the only one tackling that effort. | 22:29 |
| *** Julien-zte has joined #openstack-cinder | 22:30 | |
| *** Jack_Iv has joined #openstack-cinder | 22:35 | |
| *** Jack_Iv has quit IRC | 22:39 | |
| *** danpawlik has joined #openstack-cinder | 22:39 | |
| *** danpawlik_afk has quit IRC | 22:41 | |
| *** mtanino__ has quit IRC | 22:48 | |
| *** gouthamr has joined #openstack-cinder | 22:50 | |
| *** gouthamr has quit IRC | 22:54 | |
| *** gouthamr has joined #openstack-cinder | 22:55 | |
| *** salv-orl_ has quit IRC | 22:59 | |
| *** gouthamr has quit IRC | 22:59 | |
| *** mriedem has quit IRC | 23:02 | |
| *** felipemonteiro_ has quit IRC | 23:05 | |
| smcginnis | bswartz, tbarron: There was a call by some to explicitly break the driverfixes branch. | 23:10 |
| smcginnis | bswartz, tbarron: The idea was driverfixes definitely should not be an area where all pathes get backported. | 23:10 |
| smcginnis | bswartz, tbarron: Really only have it as a common area where distros can go to pick up driver fixes only. | 23:11 |
| smcginnis | And that's it | 23:11 |
| smcginnis | It definnitely should not be something that a distro would want to take the whole branch and use it as the basis for packaginng. | 23:11 |
| *** KeithMnemonic1 has joined #openstack-cinder | 23:14 | |
| *** KeithMnemonic has quit IRC | 23:17 | |
| *** edmondsw has quit IRC | 23:19 | |
| *** Jack_Iv has joined #openstack-cinder | 23:31 | |
| *** Jack_Iv has quit IRC | 23:36 | |
| *** Julien-zte has quit IRC | 23:36 | |
| tbarron | smcginnis: +1 | 23:37 |
| tbarron | smcginnis: since it's not tested, not under CI, I don't see it as an "integration branch" | 23:37 |
| *** jmlowe has joined #openstack-cinder | 23:38 | |
| tbarron | smcginnis: it's a replacement for the various py-py repos that vendors maintain, a central place to get cherry-picks of *driver* fixes | 23:38 |
| tbarron | smcginnis: then after you (e.g. distro) pick them up, rebase, fix up, test, etc. | 23:39 |
| tbarron | smcginnis: otherwise it just turns into the old proposal to maintain older stable branches | 23:39 |
| tbarron | smcginnis: that's not a bad idea, but there aren't resources committed to do that and maintaining current stable branches is challenging enough | 23:40 |
| *** Jack_Iv has joined #openstack-cinder | 23:47 | |
| *** Jack_Iv has quit IRC | 23:52 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!