| *** aspiers[m] has quit IRC | 00:05 | |
| *** wN_ has quit IRC | 00:05 | |
| *** aspiers[m] has joined #openstack-cinder | 00:13 | |
| *** wN_ has joined #openstack-cinder | 00:17 | |
| *** claudiub has quit IRC | 00:20 | |
| *** masber has joined #openstack-cinder | 00:21 | |
| *** wanghao has joined #openstack-cinder | 00:27 | |
| *** masber has quit IRC | 00:29 | |
| *** Brin has joined #openstack-cinder | 00:39 | |
| *** Apoorva has quit IRC | 00:40 | |
| *** wanghao has quit IRC | 00:42 | |
| *** wanghao has joined #openstack-cinder | 00:44 | |
| *** zhaochao has joined #openstack-cinder | 00:58 | |
| *** hoangcx has joined #openstack-cinder | 01:04 | |
| *** dave-mccowan has joined #openstack-cinder | 01:07 | |
| *** abishop has quit IRC | 01:08 | |
| *** yangyapeng has joined #openstack-cinder | 01:10 | |
| *** masber has joined #openstack-cinder | 01:10 | |
| *** daidv has joined #openstack-cinder | 01:20 | |
| *** daidv_ has joined #openstack-cinder | 01:20 | |
| *** edmondsw has joined #openstack-cinder | 01:21 | |
| *** daidv_ has quit IRC | 01:24 | |
| *** edmondsw has quit IRC | 01:26 | |
| *** dalgaaf has quit IRC | 02:12 | |
| *** dalgaaf has joined #openstack-cinder | 02:14 | |
| *** wanghao_ has joined #openstack-cinder | 02:21 | |
| *** wanghao has quit IRC | 02:22 | |
| *** zhurong has joined #openstack-cinder | 02:46 | |
| *** dave-mccowan has quit IRC | 03:09 | |
| *** edmondsw has joined #openstack-cinder | 03:09 | |
| *** edmondsw has quit IRC | 03:14 | |
| *** tasker has quit IRC | 03:25 | |
| *** abhishekk has joined #openstack-cinder | 03:40 | |
| openstackgerrit | zhurong proposed openstack/cinder master: Backups restore allow name or uuid null https://review.openstack.org/531639 | 03:51 |
|---|---|---|
| *** chhavi has joined #openstack-cinder | 04:08 | |
| *** wanghao has joined #openstack-cinder | 04:08 | |
| *** wanghao_ has quit IRC | 04:10 | |
| *** pooja_jadhav has joined #openstack-cinder | 04:11 | |
| *** moshele has joined #openstack-cinder | 04:39 | |
| *** chhavi has quit IRC | 04:39 | |
| *** gcb has joined #openstack-cinder | 04:41 | |
| *** moshele has quit IRC | 04:44 | |
| *** zhurong has quit IRC | 04:52 | |
| *** edmondsw has joined #openstack-cinder | 04:58 | |
| *** edmondsw has quit IRC | 05:02 | |
| *** moshele has joined #openstack-cinder | 05:04 | |
| *** bhagyashri_s is now known as bhagyashris | 05:05 | |
| *** pooja_jadhav has quit IRC | 05:15 | |
| *** yangyapeng has quit IRC | 05:16 | |
| *** yangyapeng has joined #openstack-cinder | 05:17 | |
| *** ykarel|away has joined #openstack-cinder | 05:17 | |
| *** moshele has quit IRC | 05:18 | |
| *** ykarel|away is now known as ykarel | 05:22 | |
| *** pooja_jadhav has joined #openstack-cinder | 05:25 | |
| openstackgerrit | Yu Zhang proposed openstack/cinder master: Fix: create snapshot failed with no option name is given https://review.openstack.org/531289 | 05:26 |
| *** armaan has joined #openstack-cinder | 05:26 | |
| *** psachin has joined #openstack-cinder | 05:33 | |
| *** mikal has quit IRC | 05:33 | |
| openstackgerrit | jichao zhang proposed openstack/cinder master: Unity: Attach Unity volume via IPv6. https://review.openstack.org/529454 | 05:39 |
| *** monjoshi has joined #openstack-cinder | 05:47 | |
| openstackgerrit | jichao zhang proposed openstack/cinder master: Unity: Attach Unity volume via IPv6. https://review.openstack.org/529454 | 05:48 |
| openstackgerrit | jichao zhang proposed openstack/cinder master: Unity: Attach Unity volume via IPv6. https://review.openstack.org/529454 | 05:50 |
| *** zhurong has joined #openstack-cinder | 05:51 | |
| *** pgadiya has joined #openstack-cinder | 05:53 | |
| *** wanghao_ has joined #openstack-cinder | 05:59 | |
| *** wanghao has quit IRC | 06:03 | |
| *** moshele has joined #openstack-cinder | 06:09 | |
| openstackgerrit | jichao zhang proposed openstack/cinder master: Unity: Attach Unity volume via IPv6. https://review.openstack.org/529454 | 06:12 |
| *** bkopilov has joined #openstack-cinder | 06:29 | |
| *** chhavi has joined #openstack-cinder | 06:33 | |
| *** claudiub has joined #openstack-cinder | 06:43 | |
| *** edmondsw has joined #openstack-cinder | 06:46 | |
| *** edmondsw has quit IRC | 06:50 | |
| *** moshele has quit IRC | 07:03 | |
| *** moshele has joined #openstack-cinder | 07:03 | |
| *** pcaruana has joined #openstack-cinder | 07:04 | |
| *** moshele has quit IRC | 07:06 | |
| *** moshele has joined #openstack-cinder | 07:12 | |
| *** knikolla has quit IRC | 07:13 | |
| *** apopovych has quit IRC | 07:13 | |
| *** jungleboyj has quit IRC | 07:13 | |
| *** diablo_rojo_phon has quit IRC | 07:13 | |
| *** apopovych has joined #openstack-cinder | 07:13 | |
| *** knikolla has joined #openstack-cinder | 07:13 | |
| *** jungleboyj has joined #openstack-cinder | 07:14 | |
| *** diablo_rojo_phon has joined #openstack-cinder | 07:14 | |
| *** gaurangt has quit IRC | 07:15 | |
| *** gaurangt has joined #openstack-cinder | 07:18 | |
| *** moshele has quit IRC | 07:25 | |
| *** moshele has joined #openstack-cinder | 07:27 | |
| openstackgerrit | jichao zhang proposed openstack/cinder master: Unity: Attach Unity volume via IPv6. https://review.openstack.org/529454 | 07:30 |
| *** armaan_ has joined #openstack-cinder | 07:36 | |
| *** armaan has quit IRC | 07:37 | |
| *** arnewiebalck_ has joined #openstack-cinder | 07:41 | |
| *** wanghao_ has quit IRC | 08:02 | |
| *** armaan_ has quit IRC | 08:03 | |
| *** wanghao has joined #openstack-cinder | 08:03 | |
| *** ykarel is now known as ykarel|lunch | 08:06 | |
| *** tesseract has joined #openstack-cinder | 08:09 | |
| *** namnh has joined #openstack-cinder | 08:09 | |
| *** armaan has joined #openstack-cinder | 08:21 | |
| *** ykarel|lunch is now known as ykarel | 08:32 | |
| *** edmondsw has joined #openstack-cinder | 08:34 | |
| *** moshele has quit IRC | 08:37 | |
| *** moshele has joined #openstack-cinder | 08:37 | |
| *** edmondsw has quit IRC | 08:39 | |
| *** armaan_ has joined #openstack-cinder | 09:06 | |
| *** rcernin has quit IRC | 09:07 | |
| *** armaan has quit IRC | 09:09 | |
| *** armaan_ has quit IRC | 09:10 | |
| *** armaan has joined #openstack-cinder | 09:10 | |
| *** sbezverk has quit IRC | 09:11 | |
| *** owalsh_ is now known as owalsh | 09:15 | |
| *** finucannot is now known as stephenfin | 09:16 | |
| *** ircuser-1 has joined #openstack-cinder | 09:18 | |
| *** lkuchlan has joined #openstack-cinder | 09:20 | |
| *** danpawlik has joined #openstack-cinder | 09:23 | |
| openstackgerrit | Jeremy Zhang proposed openstack/python-cinderclient master: Fix v2 volume unit tests https://review.openstack.org/531699 | 09:24 |
| *** armaan has quit IRC | 09:28 | |
| *** armaan has joined #openstack-cinder | 09:29 | |
| *** wanghao has quit IRC | 09:30 | |
| *** wanghao has joined #openstack-cinder | 09:31 | |
| *** wanghao has quit IRC | 09:31 | |
| *** wanghao has joined #openstack-cinder | 09:31 | |
| *** wanghao has quit IRC | 09:32 | |
| *** wanghao has joined #openstack-cinder | 09:32 | |
| *** wanghao has quit IRC | 09:33 | |
| *** wanghao has joined #openstack-cinder | 09:33 | |
| *** e0ne has joined #openstack-cinder | 09:33 | |
| *** wanghao has quit IRC | 09:34 | |
| *** wanghao has joined #openstack-cinder | 09:34 | |
| *** wanghao has quit IRC | 09:34 | |
| *** wanghao has joined #openstack-cinder | 09:35 | |
| *** wanghao has quit IRC | 09:35 | |
| *** wanghao has joined #openstack-cinder | 09:35 | |
| *** wanghao has quit IRC | 09:36 | |
| *** wanghao has joined #openstack-cinder | 09:36 | |
| *** wanghao has quit IRC | 09:37 | |
| *** wanghao has joined #openstack-cinder | 09:37 | |
| *** jiaopengju has joined #openstack-cinder | 09:41 | |
| *** e0ne has quit IRC | 09:47 | |
| *** erlon has joined #openstack-cinder | 09:54 | |
| openstackgerrit | xiaoqin proposed openstack/cinder master: Storwize: correct in-use replication vol status https://review.openstack.org/531357 | 09:56 |
| *** armaan has quit IRC | 09:59 | |
| *** gcb has quit IRC | 10:02 | |
| *** zhurong has quit IRC | 10:02 | |
| *** hoangcx has quit IRC | 10:05 | |
| *** armaan has joined #openstack-cinder | 10:05 | |
| *** hoangcx has joined #openstack-cinder | 10:11 | |
| *** namnh has quit IRC | 10:18 | |
| *** edmondsw has joined #openstack-cinder | 10:22 | |
| -openstackstatus- NOTICE: zuul has been restarted, all queues have been reset. please recheck your patches when appropriate | 10:24 | |
| *** edmondsw has quit IRC | 10:27 | |
| *** ykarel is now known as ykarel|afk | 10:41 | |
| *** simondodsley has quit IRC | 10:50 | |
| *** simondodsley has joined #openstack-cinder | 10:51 | |
| *** wolsen has quit IRC | 10:51 | |
| *** wolsen has joined #openstack-cinder | 10:52 | |
| *** armaan has quit IRC | 10:55 | |
| *** moshele has quit IRC | 10:56 | |
| *** moshele has joined #openstack-cinder | 10:57 | |
| *** abhishekk has quit IRC | 10:58 | |
| *** armaan has joined #openstack-cinder | 11:02 | |
| *** ykarel|afk is now known as ykare | 11:20 | |
| *** ykare is now known as ykarel | 11:20 | |
| *** mvk has quit IRC | 11:20 | |
| *** hoangcx has quit IRC | 11:23 | |
| *** zhurong has joined #openstack-cinder | 11:35 | |
| *** chhavi has quit IRC | 11:38 | |
| *** chhavi has joined #openstack-cinder | 11:39 | |
| *** jistr is now known as jistr|mtg | 11:58 | |
| *** sdague has joined #openstack-cinder | 12:00 | |
| openstackgerrit | Yu Zhang proposed openstack/cinder master: Fix: create snapshot failed with no option name is given https://review.openstack.org/531289 | 12:01 |
| *** zhongjun has quit IRC | 12:06 | |
| *** dave-mccowan has joined #openstack-cinder | 12:09 | |
| *** edmondsw has joined #openstack-cinder | 12:10 | |
| *** gcb has joined #openstack-cinder | 12:13 | |
| *** edmondsw has quit IRC | 12:15 | |
| *** Brin has quit IRC | 12:30 | |
| *** mvk has joined #openstack-cinder | 12:37 | |
| *** dave-mccowan has quit IRC | 12:47 | |
| *** armaan has quit IRC | 12:48 | |
| *** armaan has joined #openstack-cinder | 12:49 | |
| *** dave-mccowan has joined #openstack-cinder | 12:49 | |
| *** armaan has quit IRC | 12:49 | |
| *** armaan has joined #openstack-cinder | 12:50 | |
| openstackgerrit | Jeremy Zhang proposed openstack/python-cinderclient master: Fix v2 volume unit tests https://review.openstack.org/531699 | 12:57 |
| *** moshele has quit IRC | 12:59 | |
| *** moshele has joined #openstack-cinder | 12:59 | |
| *** rcernin has joined #openstack-cinder | 12:59 | |
| *** zhurong has quit IRC | 13:01 | |
| openstackgerrit | Yu Zhang proposed openstack/cinder master: Fix: create snapshot failed with no option name is given https://review.openstack.org/531289 | 13:02 |
| *** moshele has quit IRC | 13:07 | |
| *** chhavi has quit IRC | 13:08 | |
| *** chhavi has joined #openstack-cinder | 13:10 | |
| *** rcernin has quit IRC | 13:16 | |
| *** chhavi has quit IRC | 13:22 | |
| *** chhavi has joined #openstack-cinder | 13:23 | |
| *** pcaruana has quit IRC | 13:25 | |
| *** liverpooler has joined #openstack-cinder | 13:25 | |
| *** pcaruana has joined #openstack-cinder | 13:25 | |
| *** gkadam has joined #openstack-cinder | 13:33 | |
| *** jistr|mtg is now known as jistr | 13:35 | |
| *** mchlumsky has joined #openstack-cinder | 13:39 | |
| *** karmine has joined #openstack-cinder | 13:41 | |
| *** dustins has joined #openstack-cinder | 13:43 | |
| karmine | i need to restrict the cinder function upload-to-image only to the root user, i try to modify the policy.json file without result, anyone can explain me how to? | 13:44 |
| karmine | hi guys | 13:44 |
| *** moshele has joined #openstack-cinder | 13:45 | |
| *** edmondsw has joined #openstack-cinder | 13:51 | |
| *** bhagyashris has quit IRC | 13:52 | |
| *** bhagyashris has joined #openstack-cinder | 14:01 | |
| *** yangyapeng has quit IRC | 14:11 | |
| *** stvnoyes has joined #openstack-cinder | 14:12 | |
| *** mriedem has joined #openstack-cinder | 14:14 | |
| *** jmlowe has quit IRC | 14:19 | |
| *** eharney has joined #openstack-cinder | 14:24 | |
| *** lpetrut has joined #openstack-cinder | 14:31 | |
| *** e0ne has joined #openstack-cinder | 14:37 | |
| *** armaan_ has joined #openstack-cinder | 14:38 | |
| *** armaan has quit IRC | 14:38 | |
| *** armaan_ has quit IRC | 14:38 | |
| *** armaan has joined #openstack-cinder | 14:39 | |
| *** psachin has quit IRC | 14:45 | |
| *** ykarel has quit IRC | 14:49 | |
| *** lhx_ has joined #openstack-cinder | 14:53 | |
| *** AlexeyAbashkin has joined #openstack-cinder | 14:53 | |
| *** armaan has quit IRC | 14:59 | |
| *** yangyapeng has joined #openstack-cinder | 14:59 | |
| *** Guest8485 is now known as dansmith | 15:02 | |
| *** dansmith is now known as Guest62501 | 15:02 | |
| *** sdague has quit IRC | 15:04 | |
| *** Guest62501 is now known as dansmith | 15:05 | |
| *** pgadiya has quit IRC | 15:07 | |
| *** sbezverk has joined #openstack-cinder | 15:09 | |
| *** gouthamr has joined #openstack-cinder | 15:11 | |
| *** AlexeyAbashkin has quit IRC | 15:13 | |
| *** felipemonteiro has joined #openstack-cinder | 15:15 | |
| *** Apoorva has joined #openstack-cinder | 15:17 | |
| openstackgerrit | Yu Zhang proposed openstack/cinder master: Fix: validation error if the optional parameter's value is None https://review.openstack.org/531289 | 15:17 |
| *** zhaochao has quit IRC | 15:24 | |
| *** jmlowe has joined #openstack-cinder | 15:25 | |
| *** e0ne has quit IRC | 15:28 | |
| *** markstur has joined #openstack-cinder | 15:29 | |
| *** eharney has quit IRC | 15:29 | |
| *** SamYaple_ has quit IRC | 15:30 | |
| *** SamYaple has joined #openstack-cinder | 15:30 | |
| *** moshele has quit IRC | 15:30 | |
| *** armaan has joined #openstack-cinder | 15:43 | |
| *** nicolasbock has joined #openstack-cinder | 15:45 | |
| *** gcb has quit IRC | 15:48 | |
| *** _ix has joined #openstack-cinder | 15:49 | |
| *** cfriesen has quit IRC | 15:49 | |
| *** felipemonteiro has quit IRC | 15:54 | |
| *** felipemonteiro has joined #openstack-cinder | 15:55 | |
| *** SamYaple has quit IRC | 15:57 | |
| *** SamYaple has joined #openstack-cinder | 15:57 | |
| *** SamYaple has quit IRC | 15:58 | |
| *** SamYaple has joined #openstack-cinder | 15:58 | |
| *** e0ne has joined #openstack-cinder | 15:58 | |
| *** AlexeyAbashkin has joined #openstack-cinder | 16:02 | |
| *** AlexeyAbashkin has quit IRC | 16:06 | |
| *** Swanson has joined #openstack-cinder | 16:18 | |
| *** armax has joined #openstack-cinder | 16:21 | |
| *** e0ne has quit IRC | 16:24 | |
| *** felipemonteiro has quit IRC | 16:27 | |
| *** lkuchlan has quit IRC | 16:28 | |
| *** jistr is now known as jistr|off|mtg | 16:33 | |
| *** chhavi has quit IRC | 16:38 | |
| *** _ix has quit IRC | 16:38 | |
| *** crushil has joined #openstack-cinder | 16:44 | |
| *** david-lyle has quit IRC | 16:47 | |
| *** david-lyle has joined #openstack-cinder | 16:48 | |
| *** pcaruana has quit IRC | 16:48 | |
| *** _ix has joined #openstack-cinder | 16:49 | |
| *** david-lyle has quit IRC | 16:52 | |
| *** e0ne has joined #openstack-cinder | 16:53 | |
| *** jmlowe has quit IRC | 17:02 | |
| *** lhx_ has quit IRC | 17:08 | |
| *** mvk has quit IRC | 17:09 | |
| *** gman-tx has joined #openstack-cinder | 17:20 | |
| openstackgerrit | John Griffith proposed openstack/cinder master: Add multiattach policy https://review.openstack.org/531569 | 17:26 |
| Swanson | "adds" | 17:26 |
| *** itlinux_ has joined #openstack-cinder | 17:27 | |
| *** Apoorva has quit IRC | 17:29 | |
| *** itlinux_ has quit IRC | 17:30 | |
| *** itlinux_ has joined #openstack-cinder | 17:30 | |
| *** itlinux_ has quit IRC | 17:31 | |
| *** itlinux_ has joined #openstack-cinder | 17:31 | |
| smcginnis | If anyone has not received an email ballot for the board of directors election, you can add your name here: https://etherpad.openstack.org/p/2018_BoD_ballots | 17:34 |
| *** david-lyle has joined #openstack-cinder | 17:34 | |
| smcginnis | They are tracking down why some of these have not been received. | 17:34 |
| *** itlinux_ has quit IRC | 17:34 | |
| *** itlinux_ has joined #openstack-cinder | 17:36 | |
| *** eharney has joined #openstack-cinder | 17:37 | |
| *** itlinux_ has quit IRC | 17:37 | |
| *** itlinux_ has joined #openstack-cinder | 17:38 | |
| *** itlinux_ has quit IRC | 17:42 | |
| *** itlinux_ has joined #openstack-cinder | 17:42 | |
| *** itlinux_ has quit IRC | 17:43 | |
| *** felipemonteiro has joined #openstack-cinder | 17:45 | |
| *** david-lyle has quit IRC | 17:47 | |
| *** itlinux_ has joined #openstack-cinder | 17:49 | |
| *** itlinux_ has quit IRC | 17:49 | |
| *** itlinux has joined #openstack-cinder | 17:50 | |
| *** AlexeyAbashkin has joined #openstack-cinder | 17:54 | |
| *** itlinux has quit IRC | 17:55 | |
| *** e0ne has quit IRC | 17:55 | |
| *** itlinux has joined #openstack-cinder | 17:56 | |
| *** monjoshi has quit IRC | 17:56 | |
| *** Apoorva has joined #openstack-cinder | 17:57 | |
| *** AlexeyAbashkin has quit IRC | 17:58 | |
| *** moshele has joined #openstack-cinder | 18:27 | |
| openstackgerrit | Steve Noyes proposed openstack/cinder master: Improve multiattach error message https://review.openstack.org/531877 | 18:33 |
| *** tesseract has quit IRC | 18:35 | |
| *** moshele has quit IRC | 18:36 | |
| *** armaan has quit IRC | 18:36 | |
| *** armaan has joined #openstack-cinder | 18:36 | |
| *** lpetrut has quit IRC | 18:41 | |
| openstackgerrit | Steve Noyes proposed openstack/cinder master: Improve multiattach error message https://review.openstack.org/531877 | 18:52 |
| *** jmlowe has joined #openstack-cinder | 18:53 | |
| *** harlowja has joined #openstack-cinder | 19:19 | |
| *** e0ne has joined #openstack-cinder | 19:22 | |
| *** e0ne has quit IRC | 19:40 | |
| *** lpetrut has joined #openstack-cinder | 19:41 | |
| *** felipemonteiro_ has joined #openstack-cinder | 19:41 | |
| *** david-lyle has joined #openstack-cinder | 19:43 | |
| *** felipemonteiro has quit IRC | 19:45 | |
| *** diablo_rojo has joined #openstack-cinder | 19:48 | |
| *** jmlowe has quit IRC | 19:59 | |
| *** sdague has joined #openstack-cinder | 20:00 | |
| *** Apoorva has quit IRC | 20:01 | |
| *** Apoorva has joined #openstack-cinder | 20:02 | |
| jgriffith_ | Swanson: add | 20:03 |
| *** jgriffith_ is now known as jgriffith | 20:03 | |
| Swanson | jgriffith, in the commit message? | 20:03 |
| jgriffith | Swanson: I dare you to -1 :) | 20:03 |
| Swanson | jgriffith, I didn't dare to do a +1 with that as a comment! | 20:04 |
| jgriffith | hehe | 20:04 |
| jgriffith | Wait.. that's what I meant: "I dare you to +1" | 20:05 |
| jgriffith | in fact "I triple dog dare you" | 20:05 |
| jgriffith | f'ing zul | 20:05 |
| Swanson | :) | 20:05 |
| Swanson | Zuul working per usual? | 20:06 |
| jgriffith | Swanson: as good as it usually does I guess | 20:06 |
| Swanson | I like when it fails me because it couldn't set up its own tests. | 20:06 |
| jgriffith | still new, working out the kinks | 20:06 |
| jgriffith | Yeah, I keep hitting those | 20:07 |
| smcginnis | if 'Swanson' in user.name: raise | 20:08 |
| jgriffith | smcginnis: I wonder if I could sneak that change in? | 20:09 |
| jgriffith | oh.. hehe; that one is all my fault | 20:10 |
| Swanson | Losing my mind as I switch between fascist pep8 and do-i-look-like-i-care-about-your-formatting javascript. | 20:17 |
| openstackgerrit | John Griffith proposed openstack/cinder master: Add multiattach policy https://review.openstack.org/531569 | 20:17 |
| jgriffith | Swanson: just let it go; join the rest of us and you'll find that life without a mind is MUCH better | 20:18 |
| *** itlinux has quit IRC | 20:19 | |
| ildikov | pep8 saved me before from some long debugging, so I'm ok with it | 20:19 |
| ildikov | it's way faster than running the whole test suite just to figure out I forgot a comma or something dumb like that... | 20:20 |
| ildikov | or well pep8 and whatever that runs along with it | 20:21 |
| Swanson | Oh, yeah, I've got used to pep8. Javascript, on the other hand, just don't care. I think the standards doc for this is "Javascript Standards: Standards Schmandards." | 20:21 |
| *** edmondsw has quit IRC | 20:26 | |
| *** itlinux has joined #openstack-cinder | 20:29 | |
| mriedem | Roamer`: are you going to create an openstack/requirements patch to bump the required version of os-brick>=2.2.0? https://review.openstack.org/#/c/192639/ | 20:30 |
| mriedem | need to update this https://github.com/openstack/requirements/blob/master/global-requirements.txt#L156 | 20:30 |
| *** jmlowe has joined #openstack-cinder | 20:31 | |
| *** imacdonn has joined #openstack-cinder | 20:38 | |
| *** _alastor__ has joined #openstack-cinder | 20:38 | |
| -openstackstatus- NOTICE: The jobs and queues in Zuul between 19:55UTC and 20:20UTC have been lost after recovering from a crash, you might need to re-check your patches if they were being tested during that period. | 20:38 | |
| *** openstack has quit IRC | 20:38 | |
| *** openstack has joined #openstack-cinder | 20:43 | |
| *** ChanServ sets mode: +o openstack | 20:43 | |
| jgriffith | mriedem: that was supposed to be a question... anyway I can look at the test myself and figure that out | 20:44 |
| *** jmlowe has quit IRC | 20:45 | |
| mriedem | jgriffith: looks like the cinder vol manager code can pull it off a 'mode' value in the connector dict passed to attachment_update | 20:48 |
| mriedem | but nova doesn't send that | 20:48 |
| mriedem | would it be nuts to just default the mode if the caller doesn't pass it in, but base it on the readonly settings in the volume metadata? | 20:49 |
| jgriffith | mriedem: yeah, I was wondering how you were setting it, mode vs metadata; apparantly metdata | 20:49 |
| jgriffith | blek! | 20:49 |
| mriedem | in the old flow, nova would pass the mode to the os-attach volume action api | 20:49 |
| jgriffith | mriedem: I think that's the only way to maintain compatbility | 20:49 |
| jgriffith | we can then deprecate the metadata method and remove it in the future | 20:49 |
| jgriffith | that thing is evil | 20:50 |
| mriedem | ah this is the old path nova code https://github.com/openstack/nova/blob/master/nova/virt/block_device.py#L419 | 20:50 |
| *** jmlowe has joined #openstack-cinder | 20:50 | |
| mriedem | hmm, we have a chicken and egg here though | 20:50 |
| mriedem | we don't have the connection_info until we call attachment_update with the host connector | 20:51 |
| mriedem | so nova doesn't know if it should use ro or rw | 20:51 |
| jgriffith | mriedem: we should have that same check in the new code | 20:51 |
| mriedem | this is the attachment_update call in the new flow https://github.com/openstack/nova/blob/master/nova/virt/block_device.py#L466 | 20:51 |
| jgriffith | mriedem: can't we just add the same logic there that's in the legacy_attach call? | 20:52 |
| mriedem | jgriffith: the legacy flow uses the connection_info | 20:52 |
| mriedem | the new flow doesn't have connection_info until we call attachment_update | 20:52 |
| mriedem | which is the thing that fails | 20:52 |
| jgriffith | mriedem: oh!! | 20:53 |
| jgriffith | ummm... silly question | 20:53 |
| jgriffith | why would one not ignore mode until the connection is made? | 20:53 |
| mriedem | not sure i'm following | 20:54 |
| mriedem | but that's what the old flow did, | 20:54 |
| mriedem | it would make the connection https://github.com/openstack/nova/blob/master/nova/virt/block_device.py#L388 | 20:54 |
| mriedem | and then use the resulting connection_info to determine the mode when calling os-attach | 20:54 |
| mriedem | blarg, can't use cinderclient in devstack | 20:56 |
| mriedem | ERROR: 'Namespace' object has no attribute 'os_domain_id' | 20:56 |
| jgriffith | unset OS_AUTH_TYPE | 20:56 |
| mriedem | well alright then :) | 20:57 |
| jgriffith | I guess the statement that it only is borked for me is no longer true | 20:57 |
| jgriffith | There's been a round and round on that bug | 20:58 |
| *** liverpooler has quit IRC | 20:58 | |
| mriedem | don't see anything in https://bugs.launchpad.net/python-cinderclient/+bug/1734945 | 20:58 |
| openstack | Launchpad bug 1734945 in python-cinderclient "AttributeError: 'Namespace' object has no attribute 'os_domain_id' (part 2)" [High,New] | 20:58 |
| mriedem | but ok | 20:58 |
| mriedem | oh, heh, "part 2" | 20:58 |
| ildikov | jgriffith: if you mean OS_AUTH_TYPE I was complaining about it last week too | 20:58 |
| jgriffith | ildikov: ah, didn't see that | 20:59 |
| mriedem | anyway, i got what i wanted http://paste.openstack.org/show/641027/ | 20:59 |
| mriedem | so the volume metadata has "attached_mode" in it | 20:59 |
| mriedem | even though i didn't create the volume with that metadata, it must be set by cinder | 20:59 |
| jgriffith | mriedem: yeah, damn metadata | 20:59 |
| mriedem | so, should nova for the new flow just use that since we don't have the connection_info to make that call? | 20:59 |
| ildikov | jgriffith: I got over it quickly :) | 21:00 |
| jgriffith | mriedem: maybe, but I am looking to see if there's something we could/should do in the attachment-update call to handle it | 21:00 |
| jgriffith | mriedem: what I was getting at earlier was the one source of truth thing | 21:00 |
| mriedem | jgriffith: ack; stvnoyes fyi ^ | 21:00 |
| jgriffith | mriedem: keep it offloaded in Cinder maybe | 21:01 |
| mriedem | mmmm sounds like shelved offloaded, my favorite | 21:02 |
| jgriffith | mriedem: ewwww | 21:02 |
| * jgriffith 's least favorite | 21:03 | |
| jgriffith | sadist | 21:03 |
| *** openstackgerrit has quit IRC | 21:03 | |
| jgriffith | mriedem: fyi https://github.com/openstack/cinder/blob/master/cinder/volume/api.py#L1587 | 21:09 |
| jgriffith | mriedem: curious if your test is calling that or it's setting it directly somewhere | 21:10 |
| mriedem | jgriffith: i don't think nova does that, | 21:12 |
| mriedem | would have to ask the bug reporter | 21:12 |
| Roamer` | mriedem, yes, I already have the repository checked out and the commit prepared, but I thought I'd wait until jungleboyj actually releases os-brick-2.2.0 (he only tagged it on Friday, said he'd release it today) | 21:14 |
| jungleboyj | Roamer`: It merged earlier today. | 21:15 |
| Roamer` | jungleboyj, ah, sorry, must've missed that... thanks | 21:16 |
| *** rcernin has joined #openstack-cinder | 21:17 | |
| jgriffith | lpetrut: ping | 21:18 |
| jgriffith | lpetrut: curious if you get a chance could you add how you set the metadata on this bug: https://bugs.launchpad.net/nova/+bug/1741476 | 21:19 |
| openstack | Launchpad bug 1741476 in OpenStack Compute (nova) "Attaching read-only volumes fails" [Undecided,New] | 21:19 |
| jgriffith | there's the bug bot | 21:19 |
| Roamer` | mriedem, jungleboyj, hm, should we wait for something to automatically pick the change up and update upper-constraints.txt? Otherwise it's up at https://review.openstack.org/531906 | 21:20 |
| stvnoyes | jgriffith: I was just looking at that - it's cinder readonly-mode-update vol1 true | 21:20 |
| jgriffith | stvnoyes: ok, thought it might be | 21:20 |
| jgriffith | stvnoyes: thanks! | 21:20 |
| jgriffith | I think I'll have Cinder check for the metadata setting and use it | 21:21 |
| stvnoyes | it took me a while to figure that out; it's not really obvious... | 21:21 |
| *** gouthamr has quit IRC | 21:22 | |
| *** jmlowe has quit IRC | 21:22 | |
| jgriffith | stvnoyes: yeah, there's only two ways to set that, and only about 10 people that have every done it ;) | 21:22 |
| jgriffith | metadata update directly, or that fancy wrapper | 21:22 |
| smcginnis | Roamer`: That will need to wait for https://review.openstack.org/#/c/531761/ to go through. | 21:23 |
| smcginnis | Roamer`: We typically wait a while after an upper contraint bump to make sure all is good before raising the minimum, but given time constraints and things I think we can probably get that through soon after the cap is raised. | 21:23 |
| Roamer` | smcginnis, right, exactly what I was talking about... should I put a Depends-On or something? | 21:24 |
| Roamer` | or just wiat | 21:24 |
| Roamer` | wait even | 21:24 |
| mriedem | Roamer`: you can use Depends-On or rebase your change on top of that change | 21:24 |
| smcginnis | Roamer`: I'm OK with either. It won't pass Zuul tests until one or other of those conditions is done though. A depends-on might be good. | 21:25 |
| smcginnis | Roamer`: Or yeah, what mriedem said. Actually make it a child of the u-c patch. | 21:25 |
| Roamer` | yeah, my brain turned off for a moment, of course it's in the same repo | 21:25 |
| *** Anticimex has joined #openstack-cinder | 21:27 | |
| Roamer` | mriedem, okay, rebased; https://review.openstack.org/#/c/531906/2 | 21:27 |
| *** r-daneel has joined #openstack-cinder | 21:27 | |
| *** david-lyle has quit IRC | 21:28 | |
| mriedem | i have turned my frown upside down | 21:30 |
| *** rcernin has quit IRC | 21:33 | |
| *** hemna_ has joined #openstack-cinder | 21:37 | |
| *** jmlowe has joined #openstack-cinder | 21:37 | |
| *** gouthamr has joined #openstack-cinder | 21:37 | |
| *** jmlowe has quit IRC | 21:39 | |
| *** itlinux has quit IRC | 21:52 | |
| *** eharney has quit IRC | 21:53 | |
| *** rcernin has joined #openstack-cinder | 22:06 | |
| *** dustins has quit IRC | 22:09 | |
| *** openstackgerrit has joined #openstack-cinder | 22:12 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder master: Dell EMC SC: On None connector terminate_connection unmaps all https://review.openstack.org/531231 | 22:12 |
| lpetrut | jgriffith: stvnoyes: indeed, "cinder readonly-mode-update", that's what I've used | 22:24 |
| *** erlon has quit IRC | 22:24 | |
| lpetrut | I guess that if you'd use 'cinder metadata <vol> set attached_mode=ro', it would be ignored, that seems to set the used metadata | 22:25 |
| lpetrut | user* metadata | 22:25 |
| *** lpetrut has quit IRC | 22:27 | |
| *** e0ne has joined #openstack-cinder | 22:28 | |
| *** chhavi has joined #openstack-cinder | 22:34 | |
| *** dave-mccowan has quit IRC | 22:37 | |
| *** chhavi has quit IRC | 22:38 | |
| *** armax has quit IRC | 22:40 | |
| *** crushil has quit IRC | 22:41 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder master: Dell EMC SC: On None connector terminate_connection unmaps all https://review.openstack.org/531231 | 22:42 |
| *** e0ne has quit IRC | 22:49 | |
| *** itlinux has joined #openstack-cinder | 22:51 | |
| *** david-lyle has joined #openstack-cinder | 22:55 | |
| *** stvnoyes has quit IRC | 22:56 | |
| *** gcb has joined #openstack-cinder | 23:01 | |
| *** mchlumsky has quit IRC | 23:07 | |
| *** david-lyle has quit IRC | 23:12 | |
| *** david-lyle has joined #openstack-cinder | 23:13 | |
| *** _ix has quit IRC | 23:20 | |
| *** gcb has quit IRC | 23:20 | |
| *** felipemonteiro_ has quit IRC | 23:46 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder master: Dell EMC SC: On None connector terminate_connection unmaps all https://review.openstack.org/531231 | 23:49 |
| *** gman-tx has quit IRC | 23:52 | |
| *** itlinux has quit IRC | 23:54 | |
| *** moshele has joined #openstack-cinder | 23:55 | |
| *** stakeda has joined #openstack-cinder | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!