| ildikov | jungleboyj: sounds good to me, Tnx | 00:01 |
|---|---|---|
| *** aspiers[m] has joined #openstack-cinder | 00:02 | |
| *** stapler118 has left #openstack-cinder | 00:04 | |
| jungleboyj | ildikov: Welcome. :-) | 00:04 |
| *** salv-orlando has quit IRC | 00:06 | |
| *** salv-orlando has joined #openstack-cinder | 00:06 | |
| *** liverpooler has quit IRC | 00:06 | |
| *** wN_ has joined #openstack-cinder | 00:08 | |
| *** salv-orlando has quit IRC | 00:11 | |
| *** david-lyle has quit IRC | 00:13 | |
| *** yangyapeng has quit IRC | 00:21 | |
| *** wanghao has joined #openstack-cinder | 00:29 | |
| *** armaan has quit IRC | 00:30 | |
| tommylikehu | jungleboyj, jgriffith what if the 3.47 has been merged before 3.45..... | 00:30 |
| jungleboyj | tommylikehu: They should go through the gate in the order I +W'ed them. | 00:31 |
| tommylikehu | jungleboyj: some of them would fail when veriyfing and the rest will have merged conflicts when any one of them has been merged. | 00:36 |
| jungleboyj | tommylikehu: Oy. | 00:37 |
| jungleboyj | tommylikehu: Ok, so I guess the first one will merge and then the second will fail? | 00:37 |
| openstackgerrit | Merged openstack/python-cinderclient master: Support list with 'with_count' in client https://review.openstack.org/525120 | 00:38 |
| tommylikehu | jungleboyj: let's see. | 00:39 |
| *** itlinux has joined #openstack-cinder | 00:39 | |
| jungleboyj | tommylikehu: Yeah, there is the first one. | 00:39 |
| jungleboyj | tommylikehu: *Sigh* You are right. They need to be rebased. Do you want me to do that? | 00:41 |
| tommylikehu | jungleboyj: need your approval again :) | 00:42 |
| openstackgerrit | TommyLike proposed openstack/python-cinderclient master: Bump Max version to 3.46 https://review.openstack.org/525035 | 00:42 |
| openstackgerrit | Merged openstack/python-cinderclient master: Updated from global requirements https://review.openstack.org/525391 | 00:42 |
| jungleboyj | tommylikehu: Done. | 00:43 |
| jungleboyj | When that one goes through rebase the last one and then ping me. | 00:43 |
| *** linkmark has quit IRC | 00:45 | |
| tommylikehu | jungleboyj: will do | 00:45 |
| jungleboyj | Sorry for the trouble. I guess this was going to have to happen regardless since they didn't have dependencies. | 00:47 |
| *** liverpooler has joined #openstack-cinder | 00:54 | |
| *** wanghao has quit IRC | 00:55 | |
| *** yangyapeng has joined #openstack-cinder | 01:12 | |
| *** edmondsw has quit IRC | 01:12 | |
| *** zhurong has joined #openstack-cinder | 01:19 | |
| *** itlinux has quit IRC | 01:19 | |
| *** takedakn has joined #openstack-cinder | 01:20 | |
| *** salv-orlando has joined #openstack-cinder | 01:27 | |
| *** psachin has joined #openstack-cinder | 01:30 | |
| *** salv-orlando has quit IRC | 01:34 | |
| *** salv-orlando has joined #openstack-cinder | 01:35 | |
| *** salv-orlando has quit IRC | 01:39 | |
| *** itlinux has joined #openstack-cinder | 01:45 | |
| *** david-lyle has joined #openstack-cinder | 01:48 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 01:50 | |
| *** dalgaaf has quit IRC | 01:58 | |
| *** david-lyle has quit IRC | 02:00 | |
| *** dalgaaf has joined #openstack-cinder | 02:00 | |
| *** markstur has quit IRC | 02:03 | |
| *** salv-orlando has joined #openstack-cinder | 02:08 | |
| *** rcernin has quit IRC | 02:12 | |
| *** gnufied has quit IRC | 02:15 | |
| *** salv-orlando has quit IRC | 02:18 | |
| *** salv-orlando has joined #openstack-cinder | 02:19 | |
| *** rcernin has joined #openstack-cinder | 02:19 | |
| *** harlowja has quit IRC | 02:22 | |
| *** salv-orlando has quit IRC | 02:23 | |
| *** Apoorva_ has joined #openstack-cinder | 02:24 | |
| *** Apoorva has quit IRC | 02:27 | |
| *** Apoorva_ has quit IRC | 02:28 | |
| *** wanghao has joined #openstack-cinder | 02:29 | |
| *** edmondsw has joined #openstack-cinder | 02:33 | |
| *** edmondsw has quit IRC | 02:37 | |
| *** dhinesh has quit IRC | 02:38 | |
| *** markstur has joined #openstack-cinder | 02:51 | |
| *** yikun has joined #openstack-cinder | 02:53 | |
| *** takedakn has quit IRC | 03:08 | |
| *** wanghao has quit IRC | 03:08 | |
| *** itlinux has quit IRC | 03:10 | |
| *** armax has joined #openstack-cinder | 03:11 | |
| *** hemna_ has quit IRC | 03:13 | |
| *** markstur has quit IRC | 03:16 | |
| openstackgerrit | Merged openstack/python-cinderclient master: Bump Max version to 3.46 https://review.openstack.org/525035 | 03:19 |
| *** wanghao has joined #openstack-cinder | 03:23 | |
| *** abhishekk has joined #openstack-cinder | 03:31 | |
| openstackgerrit | TommyLike proposed openstack/python-cinderclient master: Support create volume from backup in client https://review.openstack.org/516175 | 03:32 |
| *** itlinux has joined #openstack-cinder | 03:40 | |
| tommylikehu | ping jungleboyj , needs your attention on this:) https://review.openstack.org/#/c/516175/ | 03:48 |
| *** threestrands has joined #openstack-cinder | 03:52 | |
| *** threestrands has quit IRC | 03:52 | |
| *** threestrands has joined #openstack-cinder | 03:52 | |
| *** markstur has joined #openstack-cinder | 03:58 | |
| *** markstur_ has joined #openstack-cinder | 04:01 | |
| *** markstu__ has joined #openstack-cinder | 04:02 | |
| *** markstur has quit IRC | 04:04 | |
| *** markstur_ has quit IRC | 04:06 | |
| jungleboyj | Ok, will do when I get home. | 04:09 |
| *** Apoorva has joined #openstack-cinder | 04:15 | |
| *** dave-mccowan has quit IRC | 04:17 | |
| *** adisky_ has joined #openstack-cinder | 04:21 | |
| *** pgadiya has joined #openstack-cinder | 04:21 | |
| *** zhurong has quit IRC | 04:22 | |
| *** links has joined #openstack-cinder | 04:26 | |
| *** moshele has joined #openstack-cinder | 04:26 | |
| *** chhavi has joined #openstack-cinder | 04:37 | |
| *** harlowja has joined #openstack-cinder | 04:39 | |
| openstackgerrit | Shunei Shiono proposed openstack/cinder master: NEC driver: implement manage/unmanage functions. https://review.openstack.org/526268 | 04:42 |
| *** moshele has quit IRC | 04:50 | |
| openstackgerrit | Shunei Shiono proposed openstack/cinder master: NEC driver: implement manage/unmanage functions. https://review.openstack.org/526268 | 04:51 |
| jungleboyj | tommylikehu: Ok, I have +A'd that one and then I can rebase John's once that goes through. | 04:51 |
| *** Dinesh_Bhor has quit IRC | 05:00 | |
| *** armax has quit IRC | 05:06 | |
| *** itlinux has quit IRC | 05:10 | |
| chhavi | tommylikehu: thanks for the review. | 05:16 |
| *** moshele has joined #openstack-cinder | 05:20 | |
| *** moshele has quit IRC | 05:21 | |
| *** salv-orlando has joined #openstack-cinder | 05:23 | |
| *** gouthamr has quit IRC | 05:24 | |
| *** harlowja has quit IRC | 05:30 | |
| *** zhurong has joined #openstack-cinder | 05:30 | |
| *** salv-orlando has quit IRC | 05:31 | |
| *** sticker has quit IRC | 05:32 | |
| openstackgerrit | Shunei Shiono proposed openstack/cinder master: NEC driver: implement manage/unmanage functions. https://review.openstack.org/526268 | 05:40 |
| *** hoangcx has quit IRC | 05:42 | |
| *** hoangcx has joined #openstack-cinder | 05:42 | |
| *** lkuchlan_wfh has joined #openstack-cinder | 05:45 | |
| *** Apoorva has quit IRC | 05:53 | |
| *** Apoorva has joined #openstack-cinder | 05:54 | |
| *** takedakn has joined #openstack-cinder | 05:58 | |
| *** Apoorva has quit IRC | 05:58 | |
| *** markstu__ has quit IRC | 06:02 | |
| *** takedakn has quit IRC | 06:02 | |
| *** moshele has joined #openstack-cinder | 06:03 | |
| *** markstur has joined #openstack-cinder | 06:03 | |
| *** markstur_ has joined #openstack-cinder | 06:08 | |
| *** gcb has quit IRC | 06:08 | |
| *** gcb has joined #openstack-cinder | 06:10 | |
| *** lkuchlan_wfh has quit IRC | 06:10 | |
| *** markstur has quit IRC | 06:12 | |
| openstackgerrit | Merged openstack/python-cinderclient master: Support create volume from backup in client https://review.openstack.org/516175 | 06:13 |
| *** AlexeyAbashkin has joined #openstack-cinder | 06:15 | |
| *** AlexeyAbashkin has quit IRC | 06:26 | |
| *** salv-orlando has joined #openstack-cinder | 06:28 | |
| *** kristaps_ has quit IRC | 06:31 | |
| chhavi | wxy,smcginnis: please review https://review.openstack.org/#/c/524091/ | 06:32 |
| *** threestrands has quit IRC | 06:32 | |
| *** sapd_ has joined #openstack-cinder | 06:32 | |
| *** sapd has quit IRC | 06:32 | |
| *** gk__ has joined #openstack-cinder | 06:34 | |
| *** gk__ has quit IRC | 06:39 | |
| *** e0ne has joined #openstack-cinder | 06:41 | |
| openstackgerrit | Merged openstack/cinder master: Update the documentation links https://review.openstack.org/523282 | 06:45 |
| *** e0ne has quit IRC | 06:46 | |
| *** salv-orlando has quit IRC | 06:49 | |
| *** lkuchlan_wfh has joined #openstack-cinder | 07:04 | |
| *** rcernin has quit IRC | 07:11 | |
| *** zhurong has quit IRC | 07:16 | |
| *** markstur_ has quit IRC | 07:32 | |
| *** alexchadin has joined #openstack-cinder | 07:51 | |
| *** salv-orlando has joined #openstack-cinder | 08:00 | |
| *** salv-orlando has quit IRC | 08:04 | |
| *** wanghao has quit IRC | 08:05 | |
| *** AlexeyAbashkin has joined #openstack-cinder | 08:07 | |
| *** tesseract has joined #openstack-cinder | 08:15 | |
| *** rcernin has joined #openstack-cinder | 08:19 | |
| *** sapd_ has quit IRC | 08:24 | |
| *** sapd_ has joined #openstack-cinder | 08:26 | |
| *** armaan has joined #openstack-cinder | 08:27 | |
| *** gkadam has joined #openstack-cinder | 08:27 | |
| *** imacdonn has quit IRC | 08:39 | |
| *** imacdonn has joined #openstack-cinder | 08:40 | |
| *** e0ne has joined #openstack-cinder | 08:42 | |
| evrardjp | hello everyone | 08:43 |
| e0ne | hi | 08:44 |
| openstackgerrit | Merged openstack/cinder master: Fix policy documentation for os-show_image_metadata endpoint https://review.openstack.org/521915 | 08:56 |
| *** salv-orlando has joined #openstack-cinder | 09:01 | |
| *** salv-orlando has quit IRC | 09:06 | |
| evrardjp | jungleboyj: smcginnis: in order for OSA to do a m2 queens release, we have a few things to do... and one relies on this patch in cinder to merge: https://review.openstack.org/#/c/525981/1 | 09:19 |
| evrardjp | if you could have a look that would be great | 09:20 |
| evrardjp | thank you in advance! | 09:20 |
| evrardjp | or if there is any other core that can help, that would be great too :D | 09:21 |
| *** linkmark has joined #openstack-cinder | 09:23 | |
| *** stakeda has quit IRC | 09:25 | |
| openstackgerrit | Neha Alhat proposed openstack/cinder master: V3 jsonschema validation: Volume-types https://review.openstack.org/520561 | 09:28 |
| *** moshele has quit IRC | 09:29 | |
| *** ganso has joined #openstack-cinder | 09:31 | |
| *** nikeshm has joined #openstack-cinder | 09:33 | |
| *** alexchadin has quit IRC | 09:35 | |
| openstackgerrit | Neha Alhat proposed openstack/cinder master: Fix discrepancy in api-ref for create volume manage api https://review.openstack.org/526326 | 09:36 |
| *** pooja-jadhav has joined #openstack-cinder | 09:38 | |
| *** alexchad_ has joined #openstack-cinder | 09:38 | |
| *** bhagyashri_s has joined #openstack-cinder | 09:39 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: Don't raise 'NotAuthorized' when soft authorization https://review.openstack.org/526327 | 09:39 |
| *** Tahvok has quit IRC | 09:40 | |
| e0ne | tommylikehu: hi. what is a soft authorization? | 09:40 |
| *** niraj_singh has joined #openstack-cinder | 09:40 | |
| *** pooja_jadhav has quit IRC | 09:41 | |
| *** nsingh has quit IRC | 09:41 | |
| *** bhagyashris has quit IRC | 09:41 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: Don't raise 'NotAuthorized' when soft authorization https://review.openstack.org/526327 | 09:42 |
| tommylikehu | e0ne: kind of this one: https://github.com/openstack/cinder/blob/master/cinder/api/contrib/volume_mig_status_attribute.py#L31 | 09:42 |
| tommylikehu | e0ne: looks like my commit message is confusing..... | 09:42 |
| *** Tahvok has joined #openstack-cinder | 09:43 | |
| e0ne | tommylikehu: I'm more confused now | 09:44 |
| openstackgerrit | TommyLike proposed openstack/cinder master: Don't raise 'NotAuthorized' inside when soft authorization https://review.openstack.org/526327 | 09:44 |
| e0ne | tommylikehu: why 'show' and 'detail' methods of VolumeMigStatusAttributeController do not return anything? | 09:44 |
| nikeshm | any idea why zuul is not happy on https://review.openstack.org/#/c/522723/ | 09:44 |
| tommylikehu | e0ne: oh, that if statement only used to decide whether we should display the additonal columns | 09:47 |
| *** armaan has quit IRC | 09:47 | |
| tommylikehu | e0ne: I mean the 'if' statement in that line | 09:48 |
| *** armaan has joined #openstack-cinder | 09:48 | |
| *** sri_ has quit IRC | 09:51 | |
| evrardjp | nikeshm: probably worth checking at openstack-tox-py27 logs ;) | 09:53 |
| evrardjp | <-- helper, I know it ;p | 09:53 |
| evrardjp | e0ne: tommylikehu could you help review this https://review.openstack.org/#/c/525981/1 , too? That would help me tremendously. | 09:54 |
| evrardjp | thank you | 09:55 |
| *** armaan has quit IRC | 09:55 | |
| *** armaan has joined #openstack-cinder | 09:56 | |
| tommylikehu | evrardjp: sure | 09:56 |
| evrardjp | thanks a ton! | 09:56 |
| evrardjp | not my patch, but I can help if need be. | 09:56 |
| openstackgerrit | xiaoqin proposed openstack/cinder master: Storwize: fix group creation restrict rules https://review.openstack.org/526331 | 09:58 |
| *** moshele has joined #openstack-cinder | 10:00 | |
| *** salv-orlando has joined #openstack-cinder | 10:02 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: Don't raise 'NotAuthorized' inside when soft authorization https://review.openstack.org/526327 | 10:04 |
| *** gmann is now known as gmann_afk | 10:05 | |
| *** salv-orlando has quit IRC | 10:06 | |
| *** rcernin has quit IRC | 10:21 | |
| *** alexchad_ has quit IRC | 10:27 | |
| *** alexchadin has joined #openstack-cinder | 10:28 | |
| *** gcb has quit IRC | 10:29 | |
| *** alexchadin has quit IRC | 10:39 | |
| *** alexchadin has joined #openstack-cinder | 10:40 | |
| *** abhishekk has quit IRC | 10:44 | |
| *** pgadiya has quit IRC | 10:47 | |
| *** hoangcx has quit IRC | 10:48 | |
| *** hoangcx has joined #openstack-cinder | 10:49 | |
| *** sdague has joined #openstack-cinder | 10:56 | |
| *** sdague has quit IRC | 10:57 | |
| *** laurent\ has quit IRC | 10:59 | |
| openstackgerrit | Merged openstack/cinder master: Remove v1 API reference https://review.openstack.org/519474 | 11:00 |
| openstackgerrit | Merged openstack/cinder master: Disallow unmanaging encrypted volumes https://review.openstack.org/523522 | 11:00 |
| *** salv-orlando has joined #openstack-cinder | 11:02 | |
| *** salv-orlando has quit IRC | 11:07 | |
| *** armaan has quit IRC | 11:15 | |
| *** armaan has joined #openstack-cinder | 11:15 | |
| *** abishop has joined #openstack-cinder | 11:18 | |
| openstackgerrit | Neha Alhat proposed openstack/cinder master: Fix discrepancy in api-ref for volume_types apis https://review.openstack.org/526344 | 11:21 |
| openstackgerrit | TommyLike proposed openstack/cinder master: Don't raise 'NotAuthorized' inside when soft authorization https://review.openstack.org/526327 | 11:22 |
| *** sdague has joined #openstack-cinder | 11:22 | |
| *** armaan has quit IRC | 11:22 | |
| *** armaan has joined #openstack-cinder | 11:23 | |
| *** liverpooler has quit IRC | 11:25 | |
| openstackgerrit | Ciara Stacke proposed openstack/cinder master: VMAX driver - Errors extending replicated volumes https://review.openstack.org/524215 | 11:35 |
| openstackgerrit | Ciara Stacke proposed openstack/cinder master: VMAX driver - Errors extending replicated volumes https://review.openstack.org/524215 | 11:39 |
| *** alexchadin has quit IRC | 11:43 | |
| *** alexchadin has joined #openstack-cinder | 11:43 | |
| *** salv-orlando has joined #openstack-cinder | 12:03 | |
| *** salv-orlando has quit IRC | 12:07 | |
| *** armaan has quit IRC | 12:51 | |
| *** armaan has joined #openstack-cinder | 12:51 | |
| *** armaan has quit IRC | 12:55 | |
| *** armaan has joined #openstack-cinder | 12:55 | |
| *** armaan has quit IRC | 12:55 | |
| *** armaan has joined #openstack-cinder | 12:56 | |
| *** psachin has quit IRC | 12:59 | |
| *** armaan has quit IRC | 13:00 | |
| *** armaan has joined #openstack-cinder | 13:00 | |
| *** lkuchlan_wfh has quit IRC | 13:03 | |
| *** salv-orlando has joined #openstack-cinder | 13:04 | |
| *** liverpooler has joined #openstack-cinder | 13:05 | |
| *** erlon has joined #openstack-cinder | 13:09 | |
| *** salv-orl_ has joined #openstack-cinder | 13:09 | |
| *** salv-orlando has quit IRC | 13:11 | |
| *** yangyapeng has quit IRC | 13:16 | |
| *** yangyapeng has joined #openstack-cinder | 13:17 | |
| *** dave-mccowan has joined #openstack-cinder | 13:18 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/526367 | 13:19 |
| *** yangyapeng has quit IRC | 13:21 | |
| *** alexchadin has quit IRC | 13:21 | |
| *** salv-orl_ has quit IRC | 13:33 | |
| *** lhx_ has joined #openstack-cinder | 13:33 | |
| *** salv-orlando has joined #openstack-cinder | 13:33 | |
| *** salv-orlando has quit IRC | 13:39 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder master: WIP. Create backups via scheduler https://review.openstack.org/519355 | 13:45 |
| *** yangyapeng has joined #openstack-cinder | 13:45 | |
| *** dustins has joined #openstack-cinder | 13:46 | |
| *** alexchadin has joined #openstack-cinder | 13:52 | |
| *** abhishekk has joined #openstack-cinder | 13:54 | |
| *** alexchadin has quit IRC | 13:56 | |
| *** mriedem has joined #openstack-cinder | 13:57 | |
| *** takedakn has joined #openstack-cinder | 13:58 | |
| *** armaan has quit IRC | 13:59 | |
| *** salv-orlando has joined #openstack-cinder | 13:59 | |
| *** links has quit IRC | 14:01 | |
| *** armaan has joined #openstack-cinder | 14:05 | |
| *** AlexeyAbashkin has quit IRC | 14:12 | |
| *** chhavi has quit IRC | 14:12 | |
| *** chhavi has joined #openstack-cinder | 14:13 | |
| *** chhavi__ has joined #openstack-cinder | 14:17 | |
| *** chhavi has quit IRC | 14:20 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: Don't raise 'NotAuthorized' inside when soft authorization https://review.openstack.org/526327 | 14:21 |
| *** lkuchlan_wfh has joined #openstack-cinder | 14:25 | |
| *** salv-orlando has quit IRC | 14:27 | |
| *** salv-orlando has joined #openstack-cinder | 14:28 | |
| *** armaan has quit IRC | 14:28 | |
| *** edmondsw has joined #openstack-cinder | 14:30 | |
| *** salv-orlando has quit IRC | 14:33 | |
| *** takedakn has quit IRC | 14:39 | |
| *** yangyapeng has quit IRC | 14:48 | |
| *** niraj_singh has quit IRC | 14:48 | |
| *** bhagyashri_s has quit IRC | 14:48 | |
| *** pooja-jadhav has quit IRC | 14:48 | |
| *** openstackgerrit has quit IRC | 14:48 | |
| *** _pewp_ has quit IRC | 14:48 | |
| *** winston-d has quit IRC | 14:48 | |
| *** ericyoung has quit IRC | 14:48 | |
| *** lifeless has quit IRC | 14:48 | |
| *** mfisch has quit IRC | 14:48 | |
| *** afazekas has quit IRC | 14:48 | |
| *** diegows has quit IRC | 14:48 | |
| *** dustins has quit IRC | 14:48 | |
| *** gkadam has quit IRC | 14:48 | |
| *** adisky_ has quit IRC | 14:48 | |
| *** MeltedLux has quit IRC | 14:48 | |
| *** mmethot has quit IRC | 14:48 | |
| *** enriquetaso has quit IRC | 14:48 | |
| *** fungi has quit IRC | 14:48 | |
| *** twouters has quit IRC | 14:48 | |
| *** thingee has quit IRC | 14:48 | |
| *** ameade has quit IRC | 14:48 | |
| *** TheJulia has quit IRC | 14:48 | |
| *** wxy has quit IRC | 14:48 | |
| *** robcresswell has quit IRC | 14:48 | |
| *** logan- has quit IRC | 14:48 | |
| *** DuncanT has quit IRC | 14:48 | |
| *** johnthetubaguy has quit IRC | 14:48 | |
| *** apopovych has quit IRC | 14:48 | |
| *** dirk has quit IRC | 14:48 | |
| *** wiggin15 has quit IRC | 14:48 | |
| *** yuval has quit IRC | 14:48 | |
| *** nikeshm has quit IRC | 14:48 | |
| *** ChanServ has quit IRC | 14:48 | |
| *** yikun has quit IRC | 14:48 | |
| *** chenying has quit IRC | 14:48 | |
| *** mvk has quit IRC | 14:48 | |
| *** slunkad_ has quit IRC | 14:48 | |
| *** niedbalski has quit IRC | 14:48 | |
| *** gnarld_ has quit IRC | 14:48 | |
| *** cargonza has quit IRC | 14:48 | |
| *** moshele has quit IRC | 14:48 | |
| *** Chealion has quit IRC | 14:48 | |
| *** BlackDex has quit IRC | 14:48 | |
| *** netapp has quit IRC | 14:48 | |
| *** spotz has quit IRC | 14:48 | |
| *** sdague has quit IRC | 14:48 | |
| *** Tahvok has quit IRC | 14:48 | |
| *** imacdonn has quit IRC | 14:48 | |
| *** jose-phillips has quit IRC | 14:48 | |
| *** masber has quit IRC | 14:48 | |
| *** hoonetorg has quit IRC | 14:48 | |
| *** zigo has quit IRC | 14:48 | |
| *** lkwan has quit IRC | 14:48 | |
| *** abhi has quit IRC | 14:48 | |
| *** tomh__ has quit IRC | 14:48 | |
| *** swamireddy has quit IRC | 14:48 | |
| *** chhavi__ has quit IRC | 14:48 | |
| *** erlon has quit IRC | 14:48 | |
| *** sapd_ has quit IRC | 14:48 | |
| *** mikal has quit IRC | 14:48 | |
| *** zhaochao has quit IRC | 14:48 | |
| *** gmann_afk has quit IRC | 14:48 | |
| *** zhongjun has quit IRC | 14:48 | |
| *** kwathore has quit IRC | 14:48 | |
| *** knikolla has quit IRC | 14:48 | |
| *** tbarron has quit IRC | 14:48 | |
| *** v12aml has quit IRC | 14:48 | |
| *** Swanson has quit IRC | 14:48 | |
| *** a_storshoo has quit IRC | 14:48 | |
| *** dave-mccowan has quit IRC | 14:48 | |
| *** uberjay has quit IRC | 14:48 | |
| *** serverascode has quit IRC | 14:48 | |
| *** amito has quit IRC | 14:48 | |
| *** melwitt has quit IRC | 14:48 | |
| *** oomichi_afk has quit IRC | 14:48 | |
| *** tobasco has quit IRC | 14:48 | |
| *** pocketprotector has quit IRC | 14:48 | |
| *** pots has quit IRC | 14:48 | |
| *** mriedem has quit IRC | 14:48 | |
| *** abishop has quit IRC | 14:48 | |
| *** wN has quit IRC | 14:48 | |
| *** storshoo has quit IRC | 14:48 | |
| *** t_storshoo has quit IRC | 14:48 | |
| *** shaner has quit IRC | 14:48 | |
| *** aspiers has quit IRC | 14:48 | |
| *** masayukig has quit IRC | 14:48 | |
| *** jbernard has quit IRC | 14:48 | |
| *** hemna has quit IRC | 14:48 | |
| *** bswartz has quit IRC | 14:48 | |
| *** lennyb has quit IRC | 14:48 | |
| *** amrith has quit IRC | 14:48 | |
| *** lhx_ has quit IRC | 14:48 | |
| *** hoangcx has quit IRC | 14:48 | |
| *** cfriesen_ has quit IRC | 14:48 | |
| *** EmilienM has quit IRC | 14:48 | |
| *** jdillaman has quit IRC | 14:48 | |
| *** 17SAA4WSA has quit IRC | 14:48 | |
| *** tonyb has quit IRC | 14:48 | |
| *** strigazi has quit IRC | 14:48 | |
| *** rmk has quit IRC | 14:48 | |
| *** cebruns has quit IRC | 14:48 | |
| *** mdbooth has quit IRC | 14:48 | |
| *** lhinds has quit IRC | 14:48 | |
| *** stephenfin has quit IRC | 14:48 | |
| *** freerunner has quit IRC | 14:48 | |
| *** andreykurilin has quit IRC | 14:48 | |
| jgriffith | tommylikehu: if 47 lands 45 isn't needed was the point, since there's no implementation associated with it. But either way it's all fine | 14:48 |
| *** mmethot has joined #openstack-cinder | 14:49 | |
| tommylikehu | jgriffith: thanks, and would you please take a look at these two patches? https://review.openstack.org/526327 and https://review.openstack.org/#/c/524432/ | 14:51 |
| *** wN_ has quit IRC | 14:51 | |
| *** aspiers[m] has quit IRC | 14:51 | |
| jgriffith | tommylikehu: np, yeah I noticed those; will have a closer look when I get my day started. First glance looked good | 14:52 |
| jgriffith | tommylikehu: thanks for grabbing that policy bug | 14:52 |
| tommylikehu | jgriffith: great! | 14:53 |
| *** adisky_ has joined #openstack-cinder | 14:54 | |
| *** MeltedLux has joined #openstack-cinder | 14:54 | |
| *** enriquetaso has joined #openstack-cinder | 14:54 | |
| *** fungi has joined #openstack-cinder | 14:54 | |
| *** twouters has joined #openstack-cinder | 14:54 | |
| *** thingee has joined #openstack-cinder | 14:54 | |
| *** ameade has joined #openstack-cinder | 14:54 | |
| *** wxy has joined #openstack-cinder | 14:54 | |
| *** TheJulia has joined #openstack-cinder | 14:54 | |
| *** robcresswell has joined #openstack-cinder | 14:54 | |
| *** logan- has joined #openstack-cinder | 14:54 | |
| *** DuncanT has joined #openstack-cinder | 14:54 | |
| *** johnthetubaguy has joined #openstack-cinder | 14:54 | |
| *** apopovych has joined #openstack-cinder | 14:54 | |
| *** dirk has joined #openstack-cinder | 14:54 | |
| *** wiggin15 has joined #openstack-cinder | 14:54 | |
| *** yuval has joined #openstack-cinder | 14:54 | |
| *** yikun has joined #openstack-cinder | 14:54 | |
| *** chenying has joined #openstack-cinder | 14:54 | |
| *** mvk has joined #openstack-cinder | 14:54 | |
| *** slunkad_ has joined #openstack-cinder | 14:54 | |
| *** niedbalski has joined #openstack-cinder | 14:54 | |
| *** nug has joined #openstack-cinder | 14:54 | |
| *** cargonza has joined #openstack-cinder | 14:54 | |
| *** diegows has joined #openstack-cinder | 14:54 | |
| *** afazekas has joined #openstack-cinder | 14:54 | |
| *** mfisch has joined #openstack-cinder | 14:54 | |
| *** lifeless has joined #openstack-cinder | 14:54 | |
| *** ericyoung has joined #openstack-cinder | 14:54 | |
| *** winston-d has joined #openstack-cinder | 14:54 | |
| *** _pewp_ has joined #openstack-cinder | 14:54 | |
| *** openstackgerrit has joined #openstack-cinder | 14:54 | |
| *** pooja-jadhav has joined #openstack-cinder | 14:54 | |
| *** bhagyashri_s has joined #openstack-cinder | 14:54 | |
| *** niraj_singh has joined #openstack-cinder | 14:54 | |
| *** yangyapeng has joined #openstack-cinder | 14:54 | |
| *** KeithMnemonic has joined #openstack-cinder | 14:54 | |
| *** gouthamr has joined #openstack-cinder | 14:54 | |
| *** chhavi__ has joined #openstack-cinder | 14:54 | |
| *** mriedem has joined #openstack-cinder | 14:54 | |
| *** lhx_ has joined #openstack-cinder | 14:54 | |
| *** dave-mccowan has joined #openstack-cinder | 14:54 | |
| *** erlon has joined #openstack-cinder | 14:54 | |
| *** sdague has joined #openstack-cinder | 14:54 | |
| *** abishop has joined #openstack-cinder | 14:54 | |
| *** hoangcx has joined #openstack-cinder | 14:54 | |
| *** moshele has joined #openstack-cinder | 14:54 | |
| *** Tahvok has joined #openstack-cinder | 14:54 | |
| *** imacdonn has joined #openstack-cinder | 14:54 | |
| *** sapd_ has joined #openstack-cinder | 14:54 | |
| *** mikal has joined #openstack-cinder | 14:54 | |
| *** uberjay has joined #openstack-cinder | 14:54 | |
| *** zhaochao has joined #openstack-cinder | 14:54 | |
| *** gmann_afk has joined #openstack-cinder | 14:54 | |
| *** zhongjun has joined #openstack-cinder | 14:54 | |
| *** kwathore has joined #openstack-cinder | 14:54 | |
| *** knikolla has joined #openstack-cinder | 14:54 | |
| *** tbarron has joined #openstack-cinder | 14:54 | |
| *** cfriesen_ has joined #openstack-cinder | 14:54 | |
| *** v12aml has joined #openstack-cinder | 14:54 | |
| *** jose-phillips has joined #openstack-cinder | 14:54 | |
| *** masber has joined #openstack-cinder | 14:54 | |
| *** Swanson has joined #openstack-cinder | 14:54 | |
| *** wN has joined #openstack-cinder | 14:54 | |
| *** hoonetorg has joined #openstack-cinder | 14:54 | |
| *** a_storshoo has joined #openstack-cinder | 14:54 | |
| *** storshoo has joined #openstack-cinder | 14:54 | |
| *** t_storshoo has joined #openstack-cinder | 14:54 | |
| *** tomh__ has joined #openstack-cinder | 14:54 | |
| *** abhi has joined #openstack-cinder | 14:54 | |
| *** lkwan has joined #openstack-cinder | 14:54 | |
| *** zigo has joined #openstack-cinder | 14:54 | |
| *** EmilienM has joined #openstack-cinder | 14:54 | |
| *** shaner has joined #openstack-cinder | 14:54 | |
| *** jdillaman has joined #openstack-cinder | 14:54 | |
| *** Chealion has joined #openstack-cinder | 14:54 | |
| *** 17SAA4WSA has joined #openstack-cinder | 14:54 | |
| *** aspiers has joined #openstack-cinder | 14:54 | |
| *** swamireddy has joined #openstack-cinder | 14:54 | |
| *** tonyb has joined #openstack-cinder | 14:54 | |
| *** BlackDex has joined #openstack-cinder | 14:54 | |
| *** masayukig has joined #openstack-cinder | 14:54 | |
| *** netapp has joined #openstack-cinder | 14:54 | |
| *** spotz has joined #openstack-cinder | 14:54 | |
| *** jbernard has joined #openstack-cinder | 14:54 | |
| *** strigazi has joined #openstack-cinder | 14:54 | |
| *** hemna has joined #openstack-cinder | 14:54 | |
| *** rmk has joined #openstack-cinder | 14:54 | |
| *** bswartz has joined #openstack-cinder | 14:54 | |
| *** lennyb has joined #openstack-cinder | 14:54 | |
| *** amrith has joined #openstack-cinder | 14:54 | |
| *** cebruns has joined #openstack-cinder | 14:54 | |
| *** serverascode has joined #openstack-cinder | 14:54 | |
| *** mdbooth has joined #openstack-cinder | 14:54 | |
| *** lhinds has joined #openstack-cinder | 14:54 | |
| *** stephenfin has joined #openstack-cinder | 14:54 | |
| *** andreykurilin has joined #openstack-cinder | 14:54 | |
| *** amito has joined #openstack-cinder | 14:54 | |
| *** melwitt has joined #openstack-cinder | 14:54 | |
| *** freerunner has joined #openstack-cinder | 14:54 | |
| *** oomichi_afk has joined #openstack-cinder | 14:54 | |
| *** tobasco has joined #openstack-cinder | 14:54 | |
| *** pocketprotector has joined #openstack-cinder | 14:54 | |
| *** pots has joined #openstack-cinder | 14:54 | |
| *** mtreinish has quit IRC | 14:55 | |
| *** chhavi__ has quit IRC | 14:56 | |
| *** salv-orlando has joined #openstack-cinder | 14:57 | |
| *** wanghao has joined #openstack-cinder | 14:58 | |
| *** mtreinish has joined #openstack-cinder | 14:58 | |
| *** abhishekk has quit IRC | 15:00 | |
| *** alexchadin has joined #openstack-cinder | 15:11 | |
| openstackgerrit | Jay Bryant proposed openstack/python-cinderclient master: Bump Max API version to 3.48 https://review.openstack.org/524700 | 15:11 |
| *** alexchadin has quit IRC | 15:13 | |
| jungleboyj | tommylikehu: jgriffith ildikov ^^^ | 15:14 |
| jungleboyj | If we can merge that I will spin a new cinderclient and our MV problems will be temporarily resolved. | 15:14 |
| *** nikeshm has joined #openstack-cinder | 15:16 | |
| *** ChanServ has joined #openstack-cinder | 15:16 | |
| *** barjavel.freenode.net sets mode: +o ChanServ | 15:16 | |
| *** moshele has quit IRC | 15:18 | |
| *** nug is now known as Guest20148 | 15:20 | |
| *** dustins_ has joined #openstack-cinder | 15:20 | |
| *** gnufied has joined #openstack-cinder | 15:20 | |
| *** dustins_ is now known as dustins | 15:21 | |
| *** itlinux has joined #openstack-cinder | 15:23 | |
| *** links has joined #openstack-cinder | 15:25 | |
| *** hemna_ has joined #openstack-cinder | 15:27 | |
| *** alexchadin has joined #openstack-cinder | 15:28 | |
| *** wN_ has joined #openstack-cinder | 15:33 | |
| *** aspiers[m] has joined #openstack-cinder | 15:38 | |
| imacdonn | Hey guys ... noticing some Oracle ZFSSA CI runs are failing on test_snapshot_create_volume_description_non_ascii_code, whilst others are passing, and don't seem to have that test ... is it something new ? | 15:39 |
| *** itlinux has quit IRC | 15:41 | |
| *** yangyapeng has quit IRC | 15:42 | |
| *** yangyapeng has joined #openstack-cinder | 15:43 | |
| *** yangyapeng has quit IRC | 15:48 | |
| *** itlinux has joined #openstack-cinder | 15:49 | |
| *** itlinux has quit IRC | 15:54 | |
| *** links has quit IRC | 15:59 | |
| *** itlinux has joined #openstack-cinder | 15:59 | |
| *** markstur has joined #openstack-cinder | 16:02 | |
| *** itlinux has quit IRC | 16:04 | |
| *** itlinux has joined #openstack-cinder | 16:06 | |
| *** alexchadin has quit IRC | 16:09 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder master: ≈WIP. Create backups via scheduler https://review.openstack.org/519355 | 16:17 |
| *** thiagolib has joined #openstack-cinder | 16:22 | |
| *** Apoorva has joined #openstack-cinder | 16:24 | |
| *** AlexeyAbashkin has joined #openstack-cinder | 16:24 | |
| *** AlexeyAbashkin has quit IRC | 16:31 | |
| *** crose has joined #openstack-cinder | 16:35 | |
| *** aspiers[m] has quit IRC | 16:39 | |
| *** wN_ has quit IRC | 16:39 | |
| *** yangyapeng has joined #openstack-cinder | 16:43 | |
| *** cebruns_ has joined #openstack-cinder | 16:45 | |
| *** cebruns has quit IRC | 16:47 | |
| *** yangyapeng has quit IRC | 16:48 | |
| *** itlinux has quit IRC | 16:50 | |
| *** moshele has joined #openstack-cinder | 16:51 | |
| *** itlinux has joined #openstack-cinder | 16:52 | |
| *** salv-orlando has quit IRC | 16:53 | |
| *** yangyapeng has joined #openstack-cinder | 16:56 | |
| *** yangyapeng has quit IRC | 17:00 | |
| *** Apoorva has quit IRC | 17:02 | |
| *** felipemonteiro_ has joined #openstack-cinder | 17:06 | |
| *** harlowja has joined #openstack-cinder | 17:08 | |
| *** markstur_ has joined #openstack-cinder | 17:11 | |
| *** markstur has quit IRC | 17:13 | |
| *** salv-orlando has joined #openstack-cinder | 17:15 | |
| mriedem | jungleboyj: here is my summary https://bugs.launchpad.net/cinder/+bug/1736773/comments/9 | 17:19 |
| openstack | Launchpad bug 1736773 in Cinder "attachment-show is including `connection_info` for non-admin callers, it shouldn't" [High,Triaged] - Assigned to John Griffith (john-griffith) | 17:19 |
| *** lkuchlan_wfh has quit IRC | 17:19 | |
| *** crose has quit IRC | 17:21 | |
| jungleboyj | mriedem: Good summary and I concur with your analysis. | 17:23 |
| *** itlinux has quit IRC | 17:23 | |
| mriedem | \o/ | 17:23 |
| *** salv-orlando has quit IRC | 17:24 | |
| *** moshele has quit IRC | 17:24 | |
| *** KeithMnemonic has quit IRC | 17:27 | |
| *** e0ne has quit IRC | 17:29 | |
| jungleboyj | Touchdown! | 17:35 |
| *** Apoorva has joined #openstack-cinder | 17:39 | |
| *** salv-orlando has joined #openstack-cinder | 17:42 | |
| *** moshele has joined #openstack-cinder | 17:51 | |
| *** wN_ has joined #openstack-cinder | 17:52 | |
| *** salv-orlando has quit IRC | 17:53 | |
| *** moshele has quit IRC | 17:53 | |
| *** salv-orlando has joined #openstack-cinder | 17:53 | |
| *** aspiers[m] has joined #openstack-cinder | 17:57 | |
| *** salv-orlando has quit IRC | 17:57 | |
| *** salv-orlando has joined #openstack-cinder | 18:00 | |
| *** salv-orlando has quit IRC | 18:04 | |
| *** salv-orlando has joined #openstack-cinder | 18:04 | |
| *** wN_ has quit IRC | 18:05 | |
| *** aspiers[m] has quit IRC | 18:05 | |
| *** yangyapeng has joined #openstack-cinder | 18:05 | |
| *** salv-orlando has quit IRC | 18:09 | |
| *** yangyapeng has quit IRC | 18:09 | |
| *** salv-orlando has joined #openstack-cinder | 18:10 | |
| *** dustins has quit IRC | 18:11 | |
| *** mvk has quit IRC | 18:11 | |
| *** tesseract has quit IRC | 18:13 | |
| *** salv-orlando has quit IRC | 18:16 | |
| *** salv-orlando has joined #openstack-cinder | 18:16 | |
| *** salv-orlando has quit IRC | 18:19 | |
| *** salv-orlando has joined #openstack-cinder | 18:20 | |
| *** felipemonteiro_ has quit IRC | 18:22 | |
| *** e0ne has joined #openstack-cinder | 18:23 | |
| *** vivsoni__ has joined #openstack-cinder | 18:24 | |
| *** vivsoni_ has quit IRC | 18:24 | |
| *** salv-orlando has quit IRC | 18:24 | |
| *** monjoshi has joined #openstack-cinder | 18:27 | |
| *** wN_ has joined #openstack-cinder | 18:35 | |
| *** aspiers[m] has joined #openstack-cinder | 18:40 | |
| *** moshele has joined #openstack-cinder | 18:40 | |
| *** lhx_ has quit IRC | 18:41 | |
| mriedem | jungleboyj: smcginnis: looks like that new snapshot + non-ascii fails the ceph job all the time https://bugs.launchpad.net/cinder/+bug/1737015 | 18:53 |
| openstack | Launchpad bug 1737015 in Cinder "ceph: test_snapshot_create_volume_description_non_ascii_code fails with: MismatchError: u'\u05e7\u05d9\u05d9\u05e4\u05e9' != None" [Undecided,New] | 18:53 |
| *** itlinux has joined #openstack-cinder | 18:53 | |
| mriedem | melwitt: ^ | 18:53 |
| *** dhinesh has joined #openstack-cinder | 18:54 | |
| melwitt | ack, will look | 18:54 |
| mriedem | not sure why this would be backend-specific at all | 18:57 |
| *** dhinesh_ has joined #openstack-cinder | 18:57 | |
| *** dhinesh has quit IRC | 18:59 | |
| *** edmondsw has quit IRC | 19:00 | |
| mriedem | should probably have eharney look at that | 19:04 |
| mriedem | since he was involved in the red hat bugzilla | 19:04 |
| mriedem | said it was caused by enabling osprofiler, but the ceph ci job doesn't have osprofiler set | 19:04 |
| melwitt | hm | 19:05 |
| mriedem | this is all i see in the cinder volume logs http://logs.openstack.org/85/330285/176/check/legacy-tempest-dsvm-full-devstack-plugin-ceph/da266b0/logs/screen-c-vol.txt.gz#_Dec_07_16_32_48_483781 | 19:06 |
| mriedem | just that it created the snapshot successfully | 19:06 |
| mriedem | but i'd expect the backend has nothing to do with storing the snapshot description | 19:06 |
| *** moshele has quit IRC | 19:07 | |
| *** dhinesh_ has quit IRC | 19:08 | |
| *** felipemonteiro_ has joined #openstack-cinder | 19:10 | |
| *** adisky_ has quit IRC | 19:10 | |
| mriedem | yeah this doesn't make any sense | 19:11 |
| *** felipemonteiro__ has joined #openstack-cinder | 19:11 | |
| jungleboyj | That is odd. I wouldn't expect the backend to put anything in the description. Child be wrong though. | 19:13 |
| jungleboyj | eharney had been awol for a while. :-( | 19:13 |
| mriedem | cinder.volume.api.create_snapshot_in_db is what creates it in the db | 19:14 |
| mriedem | the description is passed through to the object and db code, no filtering from what i can see | 19:14 |
| melwitt | according to the assert it means the backend is wiping out the description? | 19:14 |
| *** itlinux has quit IRC | 19:14 | |
| mriedem | no, it's just creating a volume snapshot with a non-ascii description and asserting that the description is actually set on the snapshot | 19:14 |
| mriedem | which from what i can tell is all db / api stuff | 19:15 |
| *** felipemonteiro_ has quit IRC | 19:15 | |
| mriedem | what is a cleanable object? | 19:16 |
| mriedem | i bet this is the problem right here https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L1054-L1057 | 19:17 |
| *** itlinux has joined #openstack-cinder | 19:18 | |
| mriedem | but the rbd driver doesn't return anything from the create_snapshot method... | 19:19 |
| melwitt | so shouldn't model_update be None and not do that save? | 19:19 |
| mriedem | yeah | 19:19 |
| *** harlowja has quit IRC | 19:21 | |
| *** armaan has joined #openstack-cinder | 19:23 | |
| *** dustins has joined #openstack-cinder | 19:26 | |
| *** armaan has quit IRC | 19:28 | |
| *** monjoshi has quit IRC | 19:30 | |
| *** cloudnull has joined #openstack-cinder | 19:32 | |
| evrardjp | mriedem: there is already a bug with that | 19:33 |
| evrardjp | and a fix | 19:33 |
| evrardjp | https://review.openstack.org/#/c/525981/1 | 19:33 |
| *** aspiers[m] has quit IRC | 19:33 | |
| *** wN_ has quit IRC | 19:33 | |
| evrardjp | jungleboyj: melwitt ^ | 19:34 |
| evrardjp | I am not sure if that's the right solution, but at least it passes tempest for me. | 19:34 |
| melwitt | evrardjp: very cool. but do you know why does that only fail with the ceph backend? | 19:34 |
| *** edmondsw has joined #openstack-cinder | 19:35 | |
| *** dhinesh has joined #openstack-cinder | 19:35 | |
| imacdonn | melwitt: It also fails Oracle ZFSSA CI - I just commented on the other bug | 19:35 |
| evrardjp | not yet I haven't got the chance to look at it to be honest. I was just in emergency. | 19:35 |
| imacdonn | (other bug => https://bugs.launchpad.net/cinder/+bug/1737015) | 19:36 |
| openstack | Launchpad bug 1737015 in Cinder "ceph: test_snapshot_create_volume_description_non_ascii_code fails with: MismatchError: u'\u05e7\u05d9\u05d9\u05e4\u05e9' != None" [High,Confirmed] | 19:36 |
| * melwitt nods | 19:36 | |
| *** e0ne has quit IRC | 19:36 | |
| mriedem | it fails a few backends | 19:38 |
| mriedem | drbd, sheepdog also | 19:38 |
| mriedem | it's very weird that for some backends the snapshot description was being inherited by the volume? | 19:38 |
| *** edmondsw has quit IRC | 19:39 | |
| *** e0ne has joined #openstack-cinder | 19:40 | |
| *** edmondsw has joined #openstack-cinder | 19:41 | |
| mriedem | evrardjp: thanks, commented on the patch | 19:41 |
| melwitt | mriedem: here the next arg after volume is 'name' so I wonder if some backends copy name to description or something? https://github.com/openstack/cinder/blob/master/cinder/volume/api.py#L1037-L1038 | 19:41 |
| melwitt | if description is empty that is | 19:42 |
| melwitt | though again, not seeing how particular backends are involved in the name/description yet | 19:42 |
| *** crose has joined #openstack-cinder | 19:44 | |
| *** edmondsw has quit IRC | 19:45 | |
| *** edmondsw has joined #openstack-cinder | 19:52 | |
| *** mikal_ has joined #openstack-cinder | 19:55 | |
| *** e0ne has quit IRC | 19:55 | |
| *** moshele has joined #openstack-cinder | 19:55 | |
| *** edmondsw has quit IRC | 19:56 | |
| evrardjp | mriedem: it's not my patch :p | 19:57 |
| *** mikal has quit IRC | 19:57 | |
| *** edmondsw has joined #openstack-cinder | 19:58 | |
| *** edmondsw has quit IRC | 19:58 | |
| *** edmondsw has joined #openstack-cinder | 19:58 | |
| *** mikal has joined #openstack-cinder | 20:00 | |
| mriedem | evrardjp: i know, i'm just shooting the messenger | 20:01 |
| evrardjp | I can give you a few bullets. | 20:01 |
| evrardjp | Please don't use them :D | 20:01 |
| *** mikal_ has quit IRC | 20:01 | |
| *** edmondsw has quit IRC | 20:02 | |
| *** armaan has joined #openstack-cinder | 20:05 | |
| *** edmondsw has joined #openstack-cinder | 20:07 | |
| *** edmondsw has quit IRC | 20:12 | |
| melwitt | mriedem: I have a theory. before the proposed patch, the non-ascii was going to 'name' which several backends access. some of them blew up processing it and raised an exception in driver.create_snapshot. those ones would have description empty | 20:13 |
| evrardjp | mriedem: share with others when you know more | 20:13 |
| mriedem | melwitt: from my understanding of the bugzilla, the issue is actually in osprofiler | 20:14 |
| mriedem | if you have that enabled | 20:14 |
| mriedem | it doesn't handle unicode | 20:14 |
| melwitt | mriedem: the backends that pass back a model_update from create_snapshot must be copying 'name' to 'description' somehow. I don't see it yet for certain tho | 20:14 |
| mriedem | that was my thinking as well, the model_update thing | 20:15 |
| melwitt | okay. I don't know how that relates to snapshots | 20:15 |
| evrardjp | let me check at the rbd driver, never looked at all that but maybe it would help my understanding | 20:15 |
| mriedem | the rbd driver doesn't return anything from create_snapshot | 20:15 |
| melwitt | yeah. some of the other backends do though | 20:16 |
| melwitt | I guess obviously | 20:16 |
| mriedem | the lvm one doesn't either though, | 20:16 |
| *** moshele has quit IRC | 20:16 | |
| mriedem | and lvm is what we test in the gate | 20:16 |
| melwitt | and that one is passing | 20:16 |
| melwitt | okay. nevermind that then | 20:16 |
| openstackgerrit | Matt Riedemann proposed openstack/cinder master: Check snapshot flag for test_snapshot_create_volume_description_non_ascii_code https://review.openstack.org/526502 | 20:18 |
| mriedem | ^ is my semi-related contribution | 20:18 |
| *** salv-orlando has joined #openstack-cinder | 20:31 | |
| *** dustins has quit IRC | 20:33 | |
| jungleboyj | mriedem: melwitt I am going to go out on a limb and say that it might be a copypasta situation. I am guessing that ZFSSA, drbd, etc may have used CEPH as the example. | 20:34 |
| mriedem | copy/paste of what though? | 20:35 |
| imacdonn | yeah, I dunno what it'd be ... ZFSSA's create_snapshot() doesn't try to return anything either | 20:37 |
| imacdonn | (or do anything with descriptions) | 20:38 |
| *** dustins has joined #openstack-cinder | 20:38 | |
| *** salv-orlando has quit IRC | 20:39 | |
| *** salv-orlando has joined #openstack-cinder | 20:39 | |
| *** itlinux has quit IRC | 20:40 | |
| jungleboyj | mriedem: The driver. I know a people have taken one driver and copied it/updated it for their backend. | 20:40 |
| mriedem | the rbd create_snapshot() method is 3 LOC | 20:41 |
| mriedem | so unless there is something else going on outside of that call, i'm not sure that has anything to do with it | 20:41 |
| mriedem | this is why i was asking about the cleanable object stuff | 20:41 |
| mriedem | that all looks like magic | 20:41 |
| jungleboyj | Not familiar with the cleanable object stuff. | 20:42 |
| jungleboyj | smcginnis: ^^ | 20:42 |
| *** salv-orlando has quit IRC | 20:44 | |
| *** moshele has joined #openstack-cinder | 20:44 | |
| *** salv-orlando has joined #openstack-cinder | 20:48 | |
| openstackgerrit | Jay Bryant proposed openstack/python-cinderclient master: Bump Max API version to 3.48 https://review.openstack.org/524700 | 20:48 |
| *** salv-orlando has quit IRC | 20:53 | |
| *** armaan has quit IRC | 20:56 | |
| *** itlinux has joined #openstack-cinder | 21:00 | |
| *** ganso has quit IRC | 21:07 | |
| abishop | hey folks, just a reminder that the encryption key migration code is ready for review | 21:13 |
| abishop | https://review.openstack.org/524720 | 21:13 |
| *** hemna_ has quit IRC | 21:15 | |
| *** crose has quit IRC | 21:15 | |
| *** linkmark has quit IRC | 21:18 | |
| *** moshele has quit IRC | 21:23 | |
| *** sticker has joined #openstack-cinder | 21:39 | |
| *** salv-orlando has joined #openstack-cinder | 21:40 | |
| *** moshele has joined #openstack-cinder | 21:41 | |
| *** salv-orlando has quit IRC | 21:43 | |
| *** yikun has quit IRC | 21:43 | |
| *** chenying has quit IRC | 21:43 | |
| *** slunkad_ has quit IRC | 21:43 | |
| *** niedbalski has quit IRC | 21:43 | |
| *** Guest20148 has quit IRC | 21:43 | |
| *** cargonza has quit IRC | 21:43 | |
| *** salv-orlando has joined #openstack-cinder | 21:48 | |
| *** yikun has joined #openstack-cinder | 21:48 | |
| *** chenying has joined #openstack-cinder | 21:48 | |
| *** slunkad_ has joined #openstack-cinder | 21:48 | |
| *** niedbalski has joined #openstack-cinder | 21:48 | |
| *** Guest20148 has joined #openstack-cinder | 21:48 | |
| *** cargonza has joined #openstack-cinder | 21:48 | |
| *** openstackgerrit has quit IRC | 21:48 | |
| *** salv-orlando has quit IRC | 21:49 | |
| *** salv-orlando has joined #openstack-cinder | 21:49 | |
| *** armax has joined #openstack-cinder | 21:53 | |
| *** salv-orlando has quit IRC | 21:54 | |
| *** felipemonteiro__ has quit IRC | 21:57 | |
| *** yangyapeng has joined #openstack-cinder | 21:57 | |
| *** yangyapeng has quit IRC | 22:02 | |
| *** threestrands has joined #openstack-cinder | 22:04 | |
| *** threestrands has quit IRC | 22:04 | |
| *** threestrands has joined #openstack-cinder | 22:04 | |
| *** dustins has quit IRC | 22:05 | |
| *** hemna_ has joined #openstack-cinder | 22:06 | |
| *** rcernin has joined #openstack-cinder | 22:09 | |
| *** rcernin has quit IRC | 22:09 | |
| *** rcernin has joined #openstack-cinder | 22:09 | |
| *** armaan has joined #openstack-cinder | 22:09 | |
| *** rhedlind has quit IRC | 22:13 | |
| *** rhe00 has quit IRC | 22:13 | |
| *** crose has joined #openstack-cinder | 22:16 | |
| *** armaan has quit IRC | 22:17 | |
| *** crose has quit IRC | 22:23 | |
| *** dustins has joined #openstack-cinder | 22:24 | |
| *** mvk has joined #openstack-cinder | 22:33 | |
| *** rcernin has quit IRC | 22:36 | |
| *** hemna_ has quit IRC | 22:36 | |
| *** yikun has quit IRC | 22:36 | |
| *** chenying has quit IRC | 22:36 | |
| *** slunkad_ has quit IRC | 22:36 | |
| *** niedbalski has quit IRC | 22:36 | |
| *** Guest20148 has quit IRC | 22:36 | |
| *** cargonza has quit IRC | 22:36 | |
| *** prg3 has quit IRC | 22:39 | |
| *** dustins has quit IRC | 22:40 | |
| *** rcernin has joined #openstack-cinder | 22:42 | |
| *** hemna_ has joined #openstack-cinder | 22:42 | |
| *** yikun has joined #openstack-cinder | 22:42 | |
| *** chenying has joined #openstack-cinder | 22:42 | |
| *** slunkad_ has joined #openstack-cinder | 22:42 | |
| *** niedbalski has joined #openstack-cinder | 22:42 | |
| *** Guest20148 has joined #openstack-cinder | 22:42 | |
| *** cargonza has joined #openstack-cinder | 22:42 | |
| *** salv-orlando has joined #openstack-cinder | 22:50 | |
| *** wanghao has quit IRC | 22:52 | |
| *** wanghao has joined #openstack-cinder | 22:53 | |
| *** moshele has quit IRC | 22:54 | |
| *** itlinux has quit IRC | 22:56 | |
| *** salv-orlando has quit IRC | 22:57 | |
| *** felipemonteiro_ has joined #openstack-cinder | 22:57 | |
| *** wanghao has quit IRC | 22:57 | |
| *** gouthamr has quit IRC | 22:59 | |
| *** markvoelker has quit IRC | 23:03 | |
| *** markvoelker has joined #openstack-cinder | 23:04 | |
| *** harlowja has joined #openstack-cinder | 23:07 | |
| *** yangyapeng has joined #openstack-cinder | 23:07 | |
| *** yangyapeng has quit IRC | 23:11 | |
| *** imacdonn has quit IRC | 23:11 | |
| *** imacdonn has joined #openstack-cinder | 23:14 | |
| *** salv-orlando has joined #openstack-cinder | 23:16 | |
| *** felipemonteiro_ has quit IRC | 23:22 | |
| *** salv-orlando has quit IRC | 23:26 | |
| *** salv-orlando has joined #openstack-cinder | 23:26 | |
| *** erlon has quit IRC | 23:28 | |
| *** salv-orlando has quit IRC | 23:30 | |
| jgriffith | mriedem: jungleboyj melwitt I only have partial context in my scroll back but I think what you're seeing with the model return stuff is the NFS add on junk | 23:35 |
| mriedem | jgriffith: the ceph driver doesn't do anything with the nfs driver though | 23:36 |
| jgriffith | mriedem: so that's part of the context I'm missing :) | 23:36 |
| mriedem | fair enough | 23:36 |
| jgriffith | mriedem: so you're seeing a ceph failure when there's unicode in the description? (me is guessing) | 23:37 |
| *** liverpooler has quit IRC | 23:37 | |
| mriedem | the unicode isn't what caused the test to fail in the ceph job | 23:37 |
| mriedem | https://bugs.launchpad.net/cinder/+bug/1737015 | 23:37 |
| openstack | Launchpad bug 1736676 in Cinder "duplicate for #1737015 test case in test_volume_unicode.py breaks legacy-tempest-dsvm-full-devstack-plugin-ceph job" [Undecided,In progress] - Assigned to Jeremy Zhang (jeremy.zhang) | 23:37 |
| jgriffith | mriedem: k, I'll have a peek and see if anything jumps out that I can help | 23:38 |
| mriedem | the problem is the test creates a volume with a unicode description, then creates a snapshot of that volume and asserts the snapshot description has the unicode in it from the volume | 23:38 |
| mriedem | totally API / DB stuff from what i can tell | 23:38 |
| mriedem | but apparently some backends "inherit" the volume description on the snapshot and some don't? | 23:38 |
| jgriffith | mriedem: ahhh | 23:38 |
| mriedem | i didn't see anything like that in the code though | 23:38 |
| mriedem | could be a timing issue, but i don't see where | 23:39 |
| mriedem | it all seems like control plane stuff | 23:39 |
| smcginnis | All of the backends I saw referred to I think inherit from the RemoteFS base. | 23:39 |
| mriedem | is there any kind of callback code where the storage backend provides details about what it thinks the description should be and cinder updates its db? | 23:39 |
| mriedem | smcginnis: rbd doesn't | 23:39 |
| mriedem | class RBDDriver(driver.CloneableImageVD, | 23:40 |
| mriedem | driver.MigrateVD, driver.ManageableVD, driver.BaseVD): | 23:40 |
| *** gouthamr has joined #openstack-cinder | 23:42 | |
| jgriffith | GD extended attributes | 23:45 |
| *** yangyapeng has joined #openstack-cinder | 23:47 | |
| *** yangyapeng has quit IRC | 23:51 | |
| jgriffith | anyway, so it looks that test has never worked since lkuchlan added it | 23:51 |
| *** stakeda has joined #openstack-cinder | 23:53 | |
| jgriffith | There's no rule that a snapshot should copy the parent volumes description so the test isn't really valid anyway. I'll try and track down where that magic is when I get back from dinner | 23:54 |
| *** threestrands has quit IRC | 23:55 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!