Wednesday, 2018-01-17

*** gman-tx has quit IRC00:00
*** ntpttr_ has quit IRC00:02
*** bigdogstl has joined #openstack-cinder00:03
*** dalgaaf has quit IRC00:08
*** bigdogstl has quit IRC00:09
*** dalgaaf has joined #openstack-cinder00:19
openstackgerritMerged openstack/cinder master: Synology: Rename iSCSI target config option  https://review.openstack.org/53202100:20
openstackgerritMerged openstack/cinder master: Updated from global requirements  https://review.openstack.org/53304000:20
*** bigdogstl has joined #openstack-cinder00:26
*** yangyapeng has quit IRC00:38
openstackgerritMerged openstack/cinder master: Check create from image policy when image not none  https://review.openstack.org/53274700:38
*** yangyapeng has joined #openstack-cinder00:38
*** pchavva has quit IRC00:39
*** yangyapeng has quit IRC00:43
*** zhurong has joined #openstack-cinder00:45
*** dave-mccowan has joined #openstack-cinder00:49
*** Dinesh_Bhor has joined #openstack-cinder00:57
*** zhaochao has joined #openstack-cinder00:58
tommylikehuhey jungleboyj  could you take a look when you free? https://review.openstack.org/#/c/534216/01:00
*** hoangcx has quit IRC01:01
*** caisan has joined #openstack-cinder01:01
jungleboyjtommylikehu:  Done.  :-)01:04
*** bigdogstl has quit IRC01:08
*** Brin has joined #openstack-cinder01:08
*** yangyapeng has joined #openstack-cinder01:11
tommylikehujungleboyj:  thanks!01:12
jungleboyjWelcome!01:12
BrinThis review lasts for too long, who can review it, or is there any suggestion to change?01:13
BrinThank you Tommy , can you do it?01:14
smcginnisBrin: Can you provide a link to the review?01:14
Brinhttps://review.openstack.org/#/c/522482/01:14
*** hoangcx has joined #openstack-cinder01:14
*** bigdogstl has joined #openstack-cinder01:15
*** bigdogstl has quit IRC01:17
*** bigdogstl has joined #openstack-cinder01:17
*** bigdogstl has quit IRC01:19
*** bigdogstl has joined #openstack-cinder01:19
*** bigdogstl has quit IRC01:21
*** david-lyle has quit IRC01:23
*** markvoelker has quit IRC01:25
*** bigdogstl has joined #openstack-cinder01:27
*** caisan_ has joined #openstack-cinder01:28
openstackgerritMerged openstack/cinder master: Dell EMC SC: On None connector terminate_connection unmaps all  https://review.openstack.org/53123101:30
openstackgerritMerged openstack/cinder master: Revert volume consumed capacity when rescheduling  https://review.openstack.org/53134501:30
*** caisan has quit IRC01:32
*** bigdogstl has quit IRC01:34
*** gman-tx has joined #openstack-cinder01:36
*** Dinesh_Bhor has quit IRC01:39
*** Dinesh_Bhor has joined #openstack-cinder01:41
*** dave-mccowan has quit IRC01:42
*** dave-mccowan has joined #openstack-cinder01:42
*** edmondsw has joined #openstack-cinder01:45
*** edmondsw has quit IRC01:50
*** caisan has joined #openstack-cinder01:52
*** caisan_ has quit IRC01:55
*** dalgaaf has quit IRC01:56
*** dalgaaf has joined #openstack-cinder01:57
openstackgerritJeremy Zhang proposed openstack/cinder master: Fix api-ref for v3 volume types  https://review.openstack.org/53458401:58
*** bigdogstl has joined #openstack-cinder02:00
*** bigdogstl has quit IRC02:06
*** bigdogstl has joined #openstack-cinder02:13
*** zhaochao has quit IRC02:14
*** dave-mccowan has quit IRC02:19
*** caisan_ has joined #openstack-cinder02:21
*** caisan has quit IRC02:24
*** zhaochao has joined #openstack-cinder02:32
tommylikehuBrin: this one? https://review.openstack.org/#/c/522482/, waiting for your next patch :)02:33
openstackgerritxiaoqin proposed openstack/cinder master: Storwize: use DLM locks  https://review.openstack.org/53315602:33
openstackgerritBrin Zhang proposed openstack/cinder master: Add enough notification for QoS  https://review.openstack.org/52248202:39
Brintommy: The new patch has been submitted, and so on zuul test passed. Thank you.02:39
*** caisan has joined #openstack-cinder02:45
*** caisan_ has quit IRC02:48
*** jdillaman has quit IRC02:49
*** dave-mccowan has joined #openstack-cinder02:52
*** jdillaman has joined #openstack-cinder02:54
*** crushil_ has joined #openstack-cinder02:57
*** crushil has joined #openstack-cinder02:57
tommylikehuBrin:  thanks for working on this :)02:59
*** moshele has joined #openstack-cinder02:59
*** crushil_ has quit IRC03:03
*** jdillaman has quit IRC03:07
*** caisan_ has joined #openstack-cinder03:10
*** jdillaman has joined #openstack-cinder03:10
*** caisan has quit IRC03:13
*** gcb has quit IRC03:14
*** gcb has joined #openstack-cinder03:18
*** moshele has quit IRC03:19
*** itlinux has joined #openstack-cinder03:25
*** markvoelker has joined #openstack-cinder03:26
Brintommy: thank you too.03:31
*** bigdogstl has quit IRC03:35
*** ykarel has joined #openstack-cinder03:45
*** caisan_ has quit IRC03:46
*** caisan has joined #openstack-cinder03:46
*** caisan has quit IRC03:47
*** caisan has joined #openstack-cinder03:48
*** daidv has joined #openstack-cinder03:50
*** caisan has quit IRC03:50
*** caisan has joined #openstack-cinder03:51
*** caisan has quit IRC03:55
*** vivsoni_ has quit IRC03:57
*** vivsoni_ has joined #openstack-cinder03:58
*** vivsoni_ has quit IRC03:59
*** abhishekk has joined #openstack-cinder04:00
*** markvoelker has quit IRC04:01
*** vivsoni_ has joined #openstack-cinder04:01
*** vivsoni_ has quit IRC04:01
*** gman-tx has quit IRC04:02
*** bigdogstl has joined #openstack-cinder04:03
*** Dinesh_Bhor has quit IRC04:05
*** fishbone_ has quit IRC04:05
*** nicolasbock has quit IRC04:09
*** bigdogstl has quit IRC04:14
*** david-lyle has joined #openstack-cinder04:23
*** moshele has joined #openstack-cinder04:31
*** psachin has joined #openstack-cinder04:31
*** psachin has quit IRC04:31
*** caisan has joined #openstack-cinder04:32
*** dave-mccowan has quit IRC04:32
*** psachin has joined #openstack-cinder04:33
*** bigdogstl has joined #openstack-cinder04:34
*** zhurong has quit IRC04:36
openstackgerritJohn Griffith proposed openstack/cinder master: Enable multiattach capability  https://review.openstack.org/53228004:39
*** bigdogstl has quit IRC04:40
*** bigdogstl has joined #openstack-cinder04:43
*** caisan has quit IRC04:44
*** bigdogstl has quit IRC04:47
*** links has joined #openstack-cinder04:49
*** gouthamr has joined #openstack-cinder04:49
*** gouthamr has quit IRC04:53
*** caisan has joined #openstack-cinder04:54
*** chhagarw has joined #openstack-cinder04:56
*** chhavi has joined #openstack-cinder04:56
*** markvoelker has joined #openstack-cinder04:57
*** caisan has quit IRC04:59
*** bigdogstl has joined #openstack-cinder05:06
*** bigdogstl has quit IRC05:12
*** bigdogstl has joined #openstack-cinder05:14
*** markvoelker has quit IRC05:16
*** caisan has joined #openstack-cinder05:16
*** bswartz has quit IRC05:18
*** bigdogstl has quit IRC05:20
*** edmondsw has joined #openstack-cinder05:21
openstackgerritPeter Wang proposed openstack/cinder master: Unity: Add revert_to_snapshot support  https://review.openstack.org/53207105:25
*** edmondsw has quit IRC05:26
*** crushil_ has joined #openstack-cinder05:29
openstackgerritMerged openstack/cinder master: Report backend state in service list  https://review.openstack.org/52682505:30
*** bswartz has joined #openstack-cinder05:36
*** bigdogstl has joined #openstack-cinder05:38
*** bigdogstl has quit IRC05:43
*** dillaman has quit IRC05:47
*** jdillaman has quit IRC05:47
*** zhurong has joined #openstack-cinder05:48
BrinHi Sean McGinnis, are you online?05:48
*** bigdogstl has joined #openstack-cinder05:50
*** jdillaman has joined #openstack-cinder05:53
*** dillaman has joined #openstack-cinder05:54
*** bigdogstl has quit IRC06:01
*** caisan_ has joined #openstack-cinder06:08
*** caisan has quit IRC06:11
*** Dinesh_Bhor has joined #openstack-cinder06:17
*** Dinesh_Bhor has quit IRC06:22
*** Dinesh_Bhor has joined #openstack-cinder06:23
*** markvoelker has joined #openstack-cinder06:28
*** Dinesh_Bhor has quit IRC06:28
*** Dinesh_Bhor has joined #openstack-cinder06:28
*** caisan has joined #openstack-cinder06:36
*** sapd__ has quit IRC06:38
*** pgadiya has joined #openstack-cinder06:39
*** sapd__ has joined #openstack-cinder06:39
*** caisan_ has quit IRC06:40
*** lkuchlan has joined #openstack-cinder06:47
*** prometheanfire has joined #openstack-cinder06:52
prometheanfiredoes http://logs.openstack.org/45/534345/1/check/neutron-grenade/9dc031a/logs/grenade.sh.txt.gz#_2018-01-17_02_32_31_746 look like openstacksdk's new version is breaking grenade?  (from https://review.openstack.org/532420 )06:52
openstackgerritNeha Alhat proposed openstack/cinder master: Fix BadRequest for 'ref' as string value  https://review.openstack.org/53407506:57
*** Dinesh_Bhor has quit IRC06:58
*** Dinesh_Bhor has joined #openstack-cinder06:59
*** caisan_ has joined #openstack-cinder07:02
*** markvoelker has quit IRC07:02
*** Dinesh_Bhor has quit IRC07:03
*** bigdogstl has joined #openstack-cinder07:03
*** caisan has quit IRC07:04
*** Dinesh_Bhor has joined #openstack-cinder07:06
*** edmondsw has joined #openstack-cinder07:09
openstackgerritJeremy Zhang proposed openstack/cinder master: Fix api-ref for v3 volume types  https://review.openstack.org/53458407:11
*** Dinesh_Bhor has quit IRC07:12
*** bigdogstl has quit IRC07:12
openstackgerritxiaoqin proposed openstack/cinder master: Storwize: use DLM locks  https://review.openstack.org/53315607:13
*** edmondsw has quit IRC07:14
*** lkuchlan has quit IRC07:15
*** Dinesh_Bhor has joined #openstack-cinder07:17
openstackgerritDou Rui Yuan proposed openstack/cinder master: Update nfs driver doc for snapshot support  https://review.openstack.org/53465007:18
*** threestrands_ has quit IRC07:20
*** armaan has joined #openstack-cinder07:22
*** gnufied has quit IRC07:28
*** crushil has quit IRC07:29
*** caisan has joined #openstack-cinder07:30
*** caisan_ has quit IRC07:33
*** gnufied has joined #openstack-cinder07:33
*** ykarel has quit IRC07:36
*** Dinesh_Bhor has quit IRC07:37
*** armaan has quit IRC07:38
*** Dinesh_Bhor has joined #openstack-cinder07:38
*** ykarel has joined #openstack-cinder07:38
*** claudiub has joined #openstack-cinder07:40
*** Dinesh_Bhor has quit IRC07:41
*** Dinesh_Bhor has joined #openstack-cinder07:42
*** ykarel has quit IRC07:43
*** ykarel has joined #openstack-cinder07:43
*** lkuchlan has joined #openstack-cinder07:45
*** crushil has joined #openstack-cinder07:47
*** pcaruana has joined #openstack-cinder07:48
*** Dinesh_Bhor has quit IRC07:53
*** pcaruana has quit IRC07:55
*** pcaruana has joined #openstack-cinder07:55
*** AlexeyAbashkin has joined #openstack-cinder07:57
*** quotaq has quit IRC07:57
*** markvoelker has joined #openstack-cinder07:58
*** caisan_ has joined #openstack-cinder07:58
openstackgerrityixuan zhang proposed openstack/cinder master: Storwize: fix an incorrect temporary parameter name  https://review.openstack.org/53466507:59
*** danpawlik has quit IRC08:00
*** caisan has quit IRC08:01
*** kholkina has joined #openstack-cinder08:02
*** danpawlik has joined #openstack-cinder08:04
*** cristicalin has joined #openstack-cinder08:04
*** lkuchlan has quit IRC08:07
*** hoonetorg has quit IRC08:10
prometheanfiredoes http://logs.openstack.org/45/534345/1/check/neutron-grenade/9dc031a/logs/grenade.sh.txt.gz#_2018-01-17_02_32_31_746 look like openstacksdk's new version is breaking grenade?  (from https://review.openstack.org/532420 )08:12
*** bigdogstl has joined #openstack-cinder08:12
prometheanfiresince eu may be waking up08:12
*** cristicalin has quit IRC08:14
*** cristicalin has joined #openstack-cinder08:14
*** Dinesh_Bhor has joined #openstack-cinder08:16
*** lkuchlan has joined #openstack-cinder08:19
*** gkadam has joined #openstack-cinder08:21
*** bigdogstl has quit IRC08:21
*** cristicalin has quit IRC08:22
*** cristicalin has joined #openstack-cinder08:22
*** alexchadin has joined #openstack-cinder08:22
*** Dinesh_Bhor has quit IRC08:24
*** hoonetorg has joined #openstack-cinder08:27
*** crushil has quit IRC08:27
*** tesseract has joined #openstack-cinder08:29
*** markvoelker has quit IRC08:32
*** Dinesh_Bhor has joined #openstack-cinder08:34
*** ykarel has quit IRC08:36
*** crushil has joined #openstack-cinder08:37
openstackgerritPooja Jadhav proposed openstack/cinder master: Remove leading and trailing spaces from parameters  https://review.openstack.org/53418408:38
*** crushil__ has joined #openstack-cinder08:39
*** crushil_ has quit IRC08:41
*** cristicalin has quit IRC08:41
*** Dinesh_Bhor has quit IRC08:41
*** crushil has quit IRC08:43
*** mvk has joined #openstack-cinder08:44
*** Dinesh_Bhor has joined #openstack-cinder08:45
*** rcernin has quit IRC08:45
*** crushil__ has quit IRC08:45
openstackgerritJeremy Zhang proposed openstack/cinder master: Fix api-ref for v3 volume types  https://review.openstack.org/53458408:48
pooja_jadhavtommylikehu: Hi08:51
*** psachin` has joined #openstack-cinder08:54
*** psachin has quit IRC08:54
*** lkuchlan has quit IRC08:56
*** edmondsw has joined #openstack-cinder08:58
*** caisan has joined #openstack-cinder08:59
*** daidv has quit IRC09:01
*** daidv has joined #openstack-cinder09:01
*** caisan_ has quit IRC09:02
*** edmondsw has quit IRC09:02
openstackgerritRyan Liang proposed openstack/cinder master: [Unity] Add `force detach` support  https://review.openstack.org/53469109:03
*** ykarel has joined #openstack-cinder09:04
*** ktibi has joined #openstack-cinder09:05
*** lkuchlan has joined #openstack-cinder09:08
*** psachin` has quit IRC09:09
*** psachin has joined #openstack-cinder09:13
*** caisan_ has joined #openstack-cinder09:16
*** chhagarw has quit IRC09:16
*** chhavi has quit IRC09:16
openstackgerritRyan Liang proposed openstack/cinder master: [VNX]Add `force detach` support  https://review.openstack.org/53415709:17
*** caisan has quit IRC09:19
*** bigdogstl has joined #openstack-cinder09:19
tommylikehupooja_jadhav:  hi09:28
openstackgerritJeremy Zhang proposed openstack/cinder master: Fix api-ref for v3 volume types  https://review.openstack.org/53458409:28
*** stakeda has quit IRC09:28
*** markvoelker has joined #openstack-cinder09:29
*** bigdogstl has quit IRC09:30
*** Dinesh_Bhor has quit IRC09:35
*** caisan has joined #openstack-cinder09:40
*** rcernin has joined #openstack-cinder09:40
*** caisan_ has quit IRC09:44
*** e0ne has joined #openstack-cinder09:47
*** mvk has quit IRC09:55
*** bigdogstl has joined #openstack-cinder09:57
*** markvoelker has quit IRC10:02
*** Brin has quit IRC10:02
*** caisan has quit IRC10:03
*** bigdogstl has quit IRC10:06
*** ktibi has quit IRC10:07
*** ganso has joined #openstack-cinder10:16
*** lkuchlan has quit IRC10:19
*** aspiers has quit IRC10:27
openstackgerritMerged openstack/cinder master: VMAX driver - Fix error handling and checks for generic volume groups  https://review.openstack.org/52108010:28
*** bigdogstl has joined #openstack-cinder10:33
*** lkuchlan has joined #openstack-cinder10:38
*** bigdogstl has quit IRC10:38
*** zhurong has quit IRC10:43
*** mvk has joined #openstack-cinder10:43
*** belmoreira has joined #openstack-cinder10:56
*** erlon has joined #openstack-cinder10:57
*** dims has quit IRC10:57
*** dims has joined #openstack-cinder10:58
*** markvoelker has joined #openstack-cinder11:00
*** bigdogstl has joined #openstack-cinder11:04
openstackgerritMoshe Levi proposed openstack/cinder master: Adding abstract class NVMeOF  https://review.openstack.org/50494411:11
openstackgerritMoshe Levi proposed openstack/cinder master: Adding NVMET target for NVMeOF  https://review.openstack.org/50555611:11
*** alexchadin has quit IRC11:13
*** bigdogstl has quit IRC11:14
*** alexchadin has joined #openstack-cinder11:18
*** caisan has joined #openstack-cinder11:21
*** caisan has quit IRC11:26
*** markvoelker has quit IRC11:32
*** nicolasbock has joined #openstack-cinder11:33
*** lpetrut has joined #openstack-cinder11:36
*** AlexeyAbashkin has quit IRC11:36
*** abhishekk has quit IRC11:37
*** bigdogstl has joined #openstack-cinder11:45
*** caisan has joined #openstack-cinder11:46
openstackgerritMerged openstack/python-brick-cinderclient-ext master: Updated from global requirements  https://review.openstack.org/53304911:49
*** caisan has quit IRC11:51
*** bigdogstl has quit IRC11:52
*** AlexeyAbashkin has joined #openstack-cinder12:02
*** alexchadin has quit IRC12:02
*** alexchadin has joined #openstack-cinder12:03
*** AlexeyAbashkin has quit IRC12:06
*** caisan has joined #openstack-cinder12:10
*** AlexeyAbashkin has joined #openstack-cinder12:12
*** pgadiya has quit IRC12:12
*** bigdogstl has joined #openstack-cinder12:13
*** rcernin has quit IRC12:14
*** caisan has quit IRC12:14
*** pgadiya has joined #openstack-cinder12:15
*** caisan has joined #openstack-cinder12:17
*** pgadiya has quit IRC12:21
*** caisan has quit IRC12:22
*** bigdogstl has quit IRC12:29
*** mvenesio has joined #openstack-cinder12:29
*** markvoelker has joined #openstack-cinder12:29
*** bigdogstl has joined #openstack-cinder12:31
*** Brin has joined #openstack-cinder12:31
*** pgadiya has joined #openstack-cinder12:34
*** edmondsw has joined #openstack-cinder12:34
pooja_jadhavgeguileo: Hi12:35
geguileopooja_jadhav: hi12:35
pooja_jadhavgeguileo: Actually i am facing one issue like, I have configured nfs backend and now when i tried to create volume, cinder scheduler fails with "Failed to run task cinder.scheduler.flows.create_volume.ScheduleCreateVolumeTask;volume:create: No valid backend was found. No weighed backends available: NoValidBackend: No valid backend was found. No weighed backends available". Can you please help me if u are aware about this scenario?12:37
*** yangyapeng has quit IRC12:38
geguileopooja_jadhav: what cinder version? master?12:38
pooja_jadhavon current master12:38
geguileopooja_jadhav: because in recent releases you should have additional information12:38
geguileopooja_jadhav: is the service up?12:38
*** edmondsw has quit IRC12:38
geguileopooja_jadhav: cinder service-list --binary cinder-volume12:39
pooja_jadhavgeguileo: all services are up12:39
pooja_jadhavgeguileo: yes up12:39
geguileopooja_jadhav: If your logs are set to debug you should see which filters discarded your backends in the scheduler log12:39
pooja_jadhavgeguileo: ok12:40
*** bigdogstl has quit IRC12:42
*** abishop has joined #openstack-cinder12:45
*** dave-mccowan has joined #openstack-cinder12:45
*** eharney has joined #openstack-cinder12:49
*** alexchadin has quit IRC12:53
*** ykarel has quit IRC12:57
*** alexchadin has joined #openstack-cinder12:57
*** aspiers has joined #openstack-cinder13:02
*** markvoelker has quit IRC13:03
*** Brin has quit IRC13:06
*** yangyapeng has joined #openstack-cinder13:11
*** nicolasbock has quit IRC13:11
*** pchavva has joined #openstack-cinder13:13
*** jmlowe_ has quit IRC13:14
*** caisan has joined #openstack-cinder13:15
*** jmlowe has joined #openstack-cinder13:18
*** caisan has quit IRC13:19
*** markvoelker has joined #openstack-cinder13:21
*** lkuchlan has quit IRC13:21
*** csaikia has joined #openstack-cinder13:29
*** bigdogstl has joined #openstack-cinder13:29
*** jmlowe has quit IRC13:30
*** jmlowe has joined #openstack-cinder13:31
*** moshele has quit IRC13:36
*** lkuchlan has joined #openstack-cinder13:37
kaisersHey Cinders! Q: Can somebody pls give me a hint what snapshot._context should contain and where it should be set? I'm trying to understand what seems to be a bug in the remotefs driver where a tmp_snapshot is missing it's _context field...13:37
*** ykarel has joined #openstack-cinder13:39
*** bigdogstl has quit IRC13:40
e0nekaisers: could you please share a traceback?13:42
kaiserse0ne: of course: http://paste.openstack.org/show/646511/13:44
kaisersIt's just i'm trying to understand what the context entity itself actually is.13:45
*** edmondsw has joined #openstack-cinder13:45
*** alexchadin has quit IRC13:46
kaisersmmh, blind me, found the class...13:46
*** liverpooler has joined #openstack-cinder13:46
*** jmlowe has quit IRC13:47
*** quotaq has joined #openstack-cinder13:48
*** jmlowe has joined #openstack-cinder13:48
quotaqHi, question about quotas. I have project that is frequently hitting quota maxTotalVolumeGigabytes=6000 but lots of space is available in cinder backend(ceph, used ~700GB). Do I need to tune some overcommitment? Using openstack-ansible Pike.13:49
*** pgadiya has quit IRC13:58
*** gman-tx has joined #openstack-cinder14:02
*** armaan has joined #openstack-cinder14:03
*** dustins has joined #openstack-cinder14:04
*** bigdogstl has joined #openstack-cinder14:09
*** caisan has joined #openstack-cinder14:11
e0nekaisers: I'm sorry, was afk. do you still need any help?14:15
*** lkuchlan has quit IRC14:15
e0nequotaq: looks like it depends on thin provisioning14:15
e0nequotaq: AFAIR, cinder doesn't have quotas overcommit option14:16
*** caisan has quit IRC14:16
*** liverpooler has quit IRC14:17
*** bigdogstl has quit IRC14:19
*** liverpooler has joined #openstack-cinder14:19
*** belmoreira has quit IRC14:20
*** thewanderer1 has joined #openstack-cinder14:22
*** caisan has joined #openstack-cinder14:22
*** crushil__ has joined #openstack-cinder14:23
*** caisan_ has joined #openstack-cinder14:23
*** crushil_ has joined #openstack-cinder14:24
*** caisan_ has quit IRC14:27
*** caisan has quit IRC14:27
thewanderer1Hi. I've noticed that some Cinder resources (volume list and backup list) support an ?all_tenants=1 query option. However, this is scarcely documented, yet the CLI util (`openstack volume backup list --all-tenants`, for example) does make use of it. Is it a "well-known" option universal to all Cinder REST API? Is it "hidden" on purpose?14:28
*** crushil__ has quit IRC14:28
*** josecastroleon has joined #openstack-cinder14:32
*** lkuchlan has joined #openstack-cinder14:33
*** belmoreira has joined #openstack-cinder14:33
chandankumareharney: Hello14:33
quotaqe0ne: can OpenStack look at "really" used space in cinder backend not sum of volume virtual sizes? Need to use used space, because ceph does thin provisioning and whatever I set quota to if volumes are big quota will get exhausted14:34
*** bigdogstl has joined #openstack-cinder14:35
*** gman-tx has quit IRC14:36
*** david-lyle has quit IRC14:37
*** _ix has joined #openstack-cinder14:38
smcginnisthewanderer1: I don't think that's on purpose, but I see we do not have it in our api-ref.14:41
smcginnisthewanderer1: If you want to file a bug, we can try to get that added.14:42
smcginnisLooks like we only have it listed on /v3/{project_id}/volumes/summary, but I believe it is supported on all of those GET calls.14:43
thewanderer1smcginnis: it would appear so, as the source of cinder/api and cinder/backup/api indicates14:43
smcginnisthewanderer1: Unfortunately our api-ref if a slowly evolving work in progress in some areas. Though it has gotten much, much better over the last cycle or so.14:44
thewanderer1I'm not sure about the option itself, though - weren't those supposed to be called "projects" by now?14:44
smcginnisthewanderer1: Yes, but this existed before then, so it's already baked into the api. We can't really change it at this point.14:45
smcginnisOr we can, but we would need to microversion it, which would be kind of silly IMO.14:45
*** bigdogstl has quit IRC14:46
thewanderer1got it - it's probably best to document it, then, as it could save folks like me some work the next time they need to write a volume crawler :P14:46
smcginnisDefinitely.14:46
*** moshele has joined #openstack-cinder14:48
kaiserse0ne: Probably yes. I've somewhat understood what the context is. However, as there haven't been recent code changes to the remotefs.py parts involved, the question why this is failing. As i cannot see so far where the context for the tmp_snapshot should be set i'm wondering if this is related to the devstack env this has been running in... Any ideas appreciated14:48
amitosmcginnis: hey14:54
amitosmcginnis: I uploaded the second patchset yesterday with the unit-tests and the new commit message. Had some issues with our CI, hopefully it will pass soon.14:55
smcginnisamito: o/14:55
smcginnisamito: OK, great.14:55
*** felipemonteiro has joined #openstack-cinder14:55
*** felipemonteiro_ has joined #openstack-cinder14:57
eharneykaisers: when does this failure happen?14:58
*** jmlowe has quit IRC15:01
*** felipemonteiro has quit IRC15:01
kaiserse0ne: Reproducably in our Cinder CI when running the Tempest test tempest.api.volume.test_volumes_backup.VolumesBackupsTest.test_backup_create_attached_volume15:02
kaisersWeirdly _only_ in this test15:02
kaisersRunning our normal set of tests has lot's of working snapshots but not for this one. That's why i'm trying to understand the context object and where it does come from15:03
*** mdbooth has quit IRC15:03
*** mdbooth has joined #openstack-cinder15:04
*** ykarel is now known as ykarel|away15:04
*** itlinux has quit IRC15:07
*** felipemonteiro_ has quit IRC15:09
quotaqcan one force cinder to look at used space in ceph for quota evaluation?15:10
*** felipemonteiro has joined #openstack-cinder15:10
*** bigdogstl has joined #openstack-cinder15:10
*** _ix has quit IRC15:10
*** links has quit IRC15:11
*** gman-tx has joined #openstack-cinder15:14
kaisersoops, reposting (just in case) eharney: Reproducably in our Cinder CI when running the Tempest test tempest.api.volume.test_volumes_backup.VolumesBackupsTest.test_backup_create_attached_volume15:15
*** ykarel|away has quit IRC15:17
jungleboyjjbernard:  You around?15:18
*** jmlowe has joined #openstack-cinder15:19
openstackgerritEric Harney proposed openstack/cinder master: Add context to driver.create_volume()  https://review.openstack.org/53481015:20
openstackgerritEric Harney proposed openstack/cinder master: RBD: Create encrypted volumes  https://review.openstack.org/53481115:20
eharneykaisers: it looks like a bug related to snapshot objects, but not sure why it would only be showing up now... i can take a closer look later today15:22
kaiserseharney: Might already be present for some time, the CI was stumbling for quite some time due to several issues and i just now came around to clear out and take a look at things. I'll go afk but will follow up in here tomorrow. Thanks for looking!15:23
*** mvenesio has quit IRC15:28
*** mchlumsky has quit IRC15:29
*** hemna_ has quit IRC15:29
*** csaikia has quit IRC15:30
*** mchlumsky has joined #openstack-cinder15:31
*** bigdogstl has quit IRC15:32
*** csaikia has joined #openstack-cinder15:32
*** zhaochao has quit IRC15:33
jgriffithsmcginnis: ping15:33
smcginnisHey15:33
jgriffithsmcginnis: hola!15:33
jgriffithsmcginnis: Hey, I'm not sure I understand that comment on the release notes...15:34
jgriffithdo you mean that whole section is irrelevant there?15:34
smcginnisjgriffith: Maybe relevant information, but it will need more context to be understood in the generated output for release notes.15:34
jgriffithsmcginnis: https://review.openstack.org/#/c/531569/ is what I'm talking about BTW :)15:34
smcginnisRight now it makes sense because they are grouped together, but they won't be in the output.15:34
jgriffithsmcginnis: ahh... you mean the upgrade note will be a seperate *thing*15:35
smcginnisYep, exactly.15:35
jgriffithderp15:35
jgriffithOk, I'll add the context around that, thanks!15:35
smcginnisSo it either needs a little more context, or just condense it into the one note.15:35
smcginnisCool. :)15:35
jgriffithI finally got a doc build job to run locally too15:36
jgriffithso what the heck, maybe I'll even test things :)15:36
ildikovjgriffith: new year new habits? :)15:37
smcginnisHaha. Let's not get too crazy now.15:37
ildikovjgriffith: as for docs, Sphinx is a fastidious cat so be careful :)15:39
*** bigdogstl has joined #openstack-cinder15:40
*** bigdogstl has quit IRC15:45
*** gman-tx has quit IRC15:52
jgriffithsmcginnis: so then it seems like the features section I have should just be copy/pasted into an upgrade section.15:53
jgriffithThat way all the bases are covered?15:53
*** moshele has quit IRC15:54
*** alexchadin has joined #openstack-cinder15:54
smcginnisjgriffith: Unfortunately, that's sometimes the best option if you want to make sure the info gets to the right person.15:54
jgriffithez/pz15:54
openstackgerritJohn Griffith proposed openstack/cinder master: Add multiattach policy  https://review.openstack.org/53156915:55
openstackgerritJohn Griffith proposed openstack/cinder master: Enable multiattach capability  https://review.openstack.org/53228015:56
*** itlinux has joined #openstack-cinder15:57
*** kholkina has quit IRC15:57
smcginnisjgriffith: Whaa whaa whaaaaa16:00
jgriffithNow I remember why I don't run tox -edoc16:01
jgriffithsmcginnis: whaaaaaat now?16:01
jgriffithThere can't possibly be anything wrong :)16:01
smcginnisjgriffith: I think it failed right away. Merge conflict or something.16:02
jgriffithOh, yeah... the second one; I'm working on that POS now16:02
*** itlinux_ has joined #openstack-cinder16:03
ildikovgood to see you guys are having fun :)16:06
*** belmoreira has quit IRC16:06
*** itlinux has quit IRC16:06
smcginnisNothing more fun than merge conflicts. :D16:07
*** lkuchlan has quit IRC16:07
*** quotaq has quit IRC16:07
ildikovlol16:11
*** bigdogstl has joined #openstack-cinder16:11
*** tpsilva has joined #openstack-cinder16:11
*** _ix has joined #openstack-cinder16:14
*** _ix has quit IRC16:19
*** _ix_ has joined #openstack-cinder16:19
*** _ix_ has quit IRC16:21
*** bigdogstl has quit IRC16:21
*** prometheanfire has left #openstack-cinder16:24
*** _ix has joined #openstack-cinder16:24
*** csaikia has quit IRC16:27
jgriffithGRRRRR SEEEEE!!!!!16:27
jgriffithMV conflict!!!16:27
jgriffithBAAAAHHHHHHHH *BOOOM*16:27
* jgriffith 's head just exploded16:27
*** csaikia has joined #openstack-cinder16:28
ildikovjgriffith: I'm sorry, I feel your pain :(16:28
ildikovjgriffith: feel free to blame me for it if that helps16:29
ildikovjgriffith: not fully my fault, just a little :)16:29
openstackgerritJohn Griffith proposed openstack/cinder master: Add multiattach policy  https://review.openstack.org/53156916:30
openstackgerritJohn Griffith proposed openstack/cinder master: Enable multiattach capability  https://review.openstack.org/53228016:30
jgriffithildikov: I'm bighting my tongue :)16:30
ildikovjgriffith: lol16:30
smcginnis:)16:31
jgriffiththe blood tastes good16:31
jgriffithhmmmm16:31
ildikovjgriffith: I would take it for the team though16:31
jgriffithThere's no I in team16:31
jgriffithThere's also no W, X, Y or Z16:31
ildikovLOL16:32
jgriffithwell.. there are actually 21 letters that aren't in team; don't know why people point out I inparticular all the time16:32
*** moshele has joined #openstack-cinder16:32
ildikovjgriffith: they are all self-centered, don't care about them16:33
jgriffithhaha16:33
ildikovjgriffith: you seem to be a vampire anyway :)16:33
jgriffithHa!16:33
jgriffithNow if I could turn into a bat and fly around that would be super neat!16:33
*** alexchadin has quit IRC16:34
ildikovjust a little practice :)16:34
*** rwellum_ has joined #openstack-cinder16:35
smcginnishttps://www.youtube.com/watch?v=Hp6dJRAwxKM16:35
ildikovAwwwwww :)16:36
jgriffithLOL smcginnis how the heck did you dig that up!!!16:37
jgriffithBTW bela lugosi's dead16:38
jgriffithhttps://www.youtube.com/watch?v=OKRJfIPiJGY16:38
smcginnisMy mind went to Dracula, so I looked for a Bela Lugosi bat video clip and hit that. Now I'm entertained for the next hour. :)16:38
jgriffithhaha, the shit I used to listen to16:38
smcginnisHah, me too!16:39
jgriffith1:08:20 to be exact16:39
ildikovsmcginnis: I have a call in 20 minutes :/16:39
smcginnisildikov: You'll have to put it on fast-play. :)16:39
jgriffith56:28 is the best BTW16:40
jgriffithha!  Now wait, 56:45 is even better16:40
ildikovsmcginnis: a Dracula movie with Bela Lugosi???16:40
*** AlexeyAbashkin has quit IRC16:41
smcginnisildikov: You've seen that classic, haven't you? Please tell me you haven't only been exposed to Gary Oldman.16:41
jgriffithGary Oldman... Pissshhhhh16:42
jgriffithThe only real dracula is Bela Lugosi16:42
ildikovsmcginnis: as a Hungarian that would've been a real shame!16:42
smcginnis:)16:42
ildikovsmcginnis: sorry, half-Transilvanian even16:42
smcginnisOooh.16:42
ildikovso yeah, I would be careful if I were you ;)16:44
*** thewanderer1 has quit IRC16:45
*** pcaruana has quit IRC16:46
*** david-lyle has joined #openstack-cinder16:49
*** bigdogstl has joined #openstack-cinder16:50
*** diablo_rojo has quit IRC16:52
jbernardjungleboyj: o/16:53
*** bigdogstl has quit IRC16:55
ildikovjgriffith: and the docs build finally succeeded on the gate :)16:56
*** armaan has quit IRC16:56
openstackgerritMerged openstack/cinder master: VMAX driver - Errors extending replicated volumes  https://review.openstack.org/52421516:56
openstackgerritMerged openstack/cinder master: VMAX driver - Fix AttributeError of dell emc driver  https://review.openstack.org/52591316:56
*** diablo_rojo has joined #openstack-cinder16:57
jgriffithildikov: well yeah, of course it did16:59
jgriffith;)16:59
ildikovjgriffith: ;)16:59
*** david-lyle has quit IRC17:02
jungleboyjjbernard:  I am assuming that I should move out the spec for 'Update cheesecake promotion specification' out to Rocky as I see no code in queens for that?17:02
*** david-lyle has joined #openstack-cinder17:03
jungleboyjjgriffith: So, it sounds like we have at least one vote for the new logo with 'Who manages your backend' on a T-Shirt.17:03
jungleboyjAnyone else like that?17:03
jungleboyjObviously I think it is cool.17:03
*** david-lyle has quit IRC17:03
*** dklyle has joined #openstack-cinder17:03
jgriffithjungleboyj: yeah, I like it17:04
jbernardjungleboyj: yeah, i expect to have some code posted soon, but not in time for the cutoff17:06
jungleboyjOk.  I will do some pricing and see what options are out there.  Put together some examples before next week's meeting and see what we can do.17:09
*** e0ne has quit IRC17:09
jungleboyjjbernard: I will move that spec out.17:12
*** ntpttr_laptop has joined #openstack-cinder17:14
jbernardjungleboyj: thanks17:16
*** tesseract has quit IRC17:21
*** bigdogstl has joined #openstack-cinder17:27
*** r-daneel has joined #openstack-cinder17:31
*** moshele has quit IRC17:32
*** bigdogstl has quit IRC17:36
*** gkadam has quit IRC17:37
openstackgerritMerged openstack/cinder master: Fix glance create client unit tests  https://review.openstack.org/53429117:38
*** mvenesio has joined #openstack-cinder17:45
openstackgerritErlon R. Cruz proposed openstack/cinder master: Overprovisioning Improvements  https://review.openstack.org/53485417:45
openstackgerritErlon R. Cruz proposed openstack/cinder master: Netapp Ontap: Adds support for auto-max-over-subscription  https://review.openstack.org/53485517:45
erlonjungleboyj: smcginnis: geguileo: manage to push that directly17:48
smcginniserlon: Great, thanks!17:59
erlonnp18:00
*** diablo_rojo has quit IRC18:00
jungleboyjerlon:  Good news.  Thank yoU!18:01
*** dklyle has quit IRC18:02
*** sbra has joined #openstack-cinder18:04
geguileoerlon: are the ut still failing?18:04
*** felipemonteiro has quit IRC18:06
openstackgerritMerged openstack/python-cinderclient master: Migrate to keystoneauth identity cli opts.  https://review.openstack.org/53221718:06
*** bigdogstl has joined #openstack-cinder18:07
*** itlinux_ has quit IRC18:10
*** itlinux has joined #openstack-cinder18:10
*** moshele has joined #openstack-cinder18:10
*** e0ne has joined #openstack-cinder18:15
*** bigdogstl has quit IRC18:16
erlongeguileo: so, they are passing upstream, but the error continue to happens in our downstream CI, and happens intermittently running locally. Downstream failure:  http://logs.openstack.netapp.com/logs/53/553/10/downstream-check/check-cinder-pep8-and-py27/3e68421/ , http://logs.openstack.netapp.com/logs/53/553/10/downstream-check/check-cinder-py35/63025b2/18:17
* geguileo looking18:18
erlongeguileo: same tests (py27 and py35 from upsrteam zuul): http://logs.openstack.org/54/534854/1/check/openstack-tox-py35/c484cc7/18:18
smcginniserlon: That's a known issue that geguileo's just-landed patch should get rid of.18:19
erlonso, we are wondering if that is a bug upstream not related with the patch that happened to be happening for some reason only in our internal CI18:19
tpsilva:(18:19
geguileoerlon: yup upstream bug https://review.openstack.org/#/c/534291/18:19
erlonsmcginnis: \o/18:19
geguileoerlon: merged now18:19
*** dulek_mobile has joined #openstack-cinder18:20
smcginniserlon: It did happen upstream too, but it was intermittent.18:20
*** AlexeyAbashkin has joined #openstack-cinder18:20
erlonsmcginnis: oww, and I was blamming myself18:20
smcginnisYou're off the hook. :)18:21
erlonsmcginnis: thanks18:21
erlon:)18:21
geguileoerlon: you should always blame upstream  };-)18:21
smcginnishehe18:21
erlongeguileo: haha18:21
*** moshele has quit IRC18:22
*** bigdogstl has joined #openstack-cinder18:22
*** AlexeyAbashkin has quit IRC18:25
*** e0ne has quit IRC18:30
*** bigdogstl has quit IRC18:31
*** felipemonteiro has joined #openstack-cinder18:31
*** pooja_jadhav_ has joined #openstack-cinder18:33
*** pooja_jadhav_ has quit IRC18:34
*** pooja_jadhav_ has joined #openstack-cinder18:35
*** bigdogstl has joined #openstack-cinder18:39
*** openstackstatus has quit IRC18:40
jungleboyjgeguileo:  Good work.  Thanks for helping erlon  :-)18:40
geguileolol18:41
*** openstackstatus has joined #openstack-cinder18:41
*** ChanServ sets mode: +v openstackstatus18:41
jungleboyjWhether intentional or not.  :-)18:41
geguileoyup, smcginnis actually beat me to tell him that it was already fixed18:41
geguileobut it was good to see him being able to post it today :-)18:42
erlongeguileo: haha, I was crazilly working since from friday looking the error in my patches18:42
geguileoouch, that hurts  :-(18:43
erlongeguileo: and the flow from the glance code was jumping right after a CONF.auth_strategy, and since I was adding some tests on conf, so 'it only could be my changes' lol18:45
geguileo:-(18:45
erlonmurphy's faults18:46
*** markstur has joined #openstack-cinder18:46
*** AlexeyAbashkin has joined #openstack-cinder18:47
*** dustins has quit IRC18:51
*** AlexeyAbashkin has quit IRC18:52
*** psachin has quit IRC18:52
*** pooja_jadhav_ has quit IRC18:53
*** stvnoyes has quit IRC18:55
*** mvenesio has quit IRC18:55
*** ntpttr_laptop has quit IRC18:57
*** stvnoyes has joined #openstack-cinder18:59
*** mvk has quit IRC19:02
*** steve-noyes has joined #openstack-cinder19:02
*** david-lyle has joined #openstack-cinder19:02
*** adisky__ has quit IRC19:04
*** jmlowe has quit IRC19:06
*** jmlowe has joined #openstack-cinder19:07
*** david-lyle has quit IRC19:08
*** e0ne has joined #openstack-cinder19:13
*** bigdogst_ has joined #openstack-cinder19:18
*** armaan has joined #openstack-cinder19:21
*** bigdogstl has quit IRC19:22
*** mvenesio has joined #openstack-cinder19:23
*** bigdogst_ has quit IRC19:26
*** ntpttr_laptop has joined #openstack-cinder19:26
*** caisan has joined #openstack-cinder19:28
*** lpetrut has quit IRC19:28
*** lpetrut has joined #openstack-cinder19:31
*** bigdogstl has joined #openstack-cinder19:31
*** diablo_rojo has joined #openstack-cinder19:32
*** caisan has quit IRC19:32
*** bigdogstl has quit IRC19:36
*** e0ne has quit IRC19:38
*** mvenesio has quit IRC19:40
*** mvenesio has joined #openstack-cinder19:40
*** ykarel|away has joined #openstack-cinder19:42
*** david-lyle has joined #openstack-cinder19:43
*** bigdogstl has joined #openstack-cinder19:48
*** e0ne has joined #openstack-cinder19:49
*** mvk has joined #openstack-cinder19:50
*** lpetrut has quit IRC19:55
*** lpetrut has joined #openstack-cinder19:55
*** ykarel|away has quit IRC19:58
*** bigdogstl has quit IRC19:59
*** ntpttr_laptop has quit IRC20:01
*** moshele has joined #openstack-cinder20:04
*** moshele has quit IRC20:08
*** gkadam has joined #openstack-cinder20:10
smcginnisPSA - zuul restart coming in about 20 minutes20:12
*** bigdogstl has joined #openstack-cinder20:15
* ildikov hates code freeze period20:17
e0neildikov: it's only feature freeze20:17
ildikove0ne: it's enough to kill the gate :)20:18
e0ne:)20:18
*** ntpttr_laptop has joined #openstack-cinder20:21
*** bigdogstl has quit IRC20:28
openstackgerritOpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements  https://review.openstack.org/53491020:29
openstackgerritOpenStack Proposal Bot proposed openstack/cinder-tempest-plugin master: Updated from global requirements  https://review.openstack.org/53395720:29
*** lkuchlan has joined #openstack-cinder20:37
-openstackstatus- NOTICE: Zuul will be offline for a few minutes; existing changes will be re-enqueued; approvals during the downtime will need to be re-added.20:38
*** jdillaman has quit IRC20:41
*** lpetrut has quit IRC20:42
*** KeithMnemonic has joined #openstack-cinder20:43
*** dustins has joined #openstack-cinder20:45
*** bigdogstl has joined #openstack-cinder20:48
*** csaikia has quit IRC20:48
SwansonZuul! Zuul! Zuul!20:49
smcginnisSwanson: I was waiting for you to say something.20:49
*** KeithMnemonic has quit IRC20:52
*** bigdogstl has quit IRC20:53
*** csaikia has joined #openstack-cinder20:53
*** bigdogstl has joined #openstack-cinder20:56
*** dulek_mobile_ has joined #openstack-cinder20:57
*** dustins has quit IRC20:58
*** dulek_mobile has quit IRC21:00
*** diablo_rojo has quit IRC21:03
*** eharney has quit IRC21:03
*** bigdogstl has quit IRC21:04
SwansonZuul! It's so cool!21:05
*** lkuchlan has quit IRC21:08
*** dustins has joined #openstack-cinder21:10
*** dulek_mobile_ has quit IRC21:13
*** dustins has quit IRC21:15
*** eharney has joined #openstack-cinder21:16
*** dustins has joined #openstack-cinder21:16
*** csaikia has quit IRC21:19
*** bigdogstl has joined #openstack-cinder21:20
openstackgerritSean McGinnis proposed openstack/os-brick master: adding VZSTORAGE connector for PPC64  https://review.openstack.org/53373921:21
openstackgerritSean McGinnis proposed openstack/os-brick master: adding VERITAS_HYPERSCALE connector for PPC64  https://review.openstack.org/53374121:21
openstackgerritSean McGinnis proposed openstack/os-brick master: adding iSER connector for PPC64  https://review.openstack.org/53372721:22
*** bigdogstl has quit IRC21:27
*** csaikia has joined #openstack-cinder21:33
*** mvenesio has quit IRC21:36
*** pchavva has quit IRC21:40
*** threestrands_ has joined #openstack-cinder21:44
*** threestrands_ has quit IRC21:45
*** threestrands_ has joined #openstack-cinder21:45
*** threestrands_ has joined #openstack-cinder21:45
*** bigdogstl has joined #openstack-cinder21:56
*** david-lyle has quit IRC22:00
*** rcernin has joined #openstack-cinder22:02
*** bigdogstl has quit IRC22:08
*** sbra has quit IRC22:08
*** bigdogstl has joined #openstack-cinder22:11
*** claudiub has quit IRC22:15
*** bigdogstl has quit IRC22:16
*** itlinux has quit IRC22:17
*** steve-noyes has quit IRC22:19
*** e0ne has quit IRC22:20
*** abishop has quit IRC22:25
*** e0ne has joined #openstack-cinder22:26
*** e0ne_ has joined #openstack-cinder22:27
*** gkadam has quit IRC22:29
*** csaikia has quit IRC22:30
*** liverpooler has quit IRC22:30
*** e0ne has quit IRC22:30
*** e0ne_ has quit IRC22:32
*** diablo_rojo has joined #openstack-cinder22:34
*** csaikia has joined #openstack-cinder22:48
*** david-lyle has joined #openstack-cinder22:51
*** dustins has quit IRC22:52
jungleboyjsmcginnis: Thanks for re proposing the os-brick changes.22:52
jungleboyjsmcginnis:  I will watch for those to merge and propose the release patch later.22:53
*** csaikia has left #openstack-cinder22:55
*** edmondsw has quit IRC23:02
*** _ix has quit IRC23:02
*** bigdogstl has joined #openstack-cinder23:07
*** _ix has joined #openstack-cinder23:10
*** crushil has joined #openstack-cinder23:10
*** r-daneel has quit IRC23:11
*** bigdogstl has quit IRC23:15
*** gnufied has quit IRC23:24
jungleboyjjgriffith:  Ping.23:24
*** ganso has quit IRC23:26
*** felipemonteiro has quit IRC23:27
*** felipemonteiro has joined #openstack-cinder23:27
jgriffithjungleboyj: pong23:31
jungleboyjjgriffith:  Hey, hopefully this is a silly question ... but.23:32
jgriffithI hope so23:32
jungleboyjhttps://review.openstack.org/#/c/531569/12/cinder/policies/attachments.py23:32
jgriffithI seem to recall that patch :)23:33
jungleboyjSo, why is the check_str just "" there instead of base.RULE_ADMIN_OR_OWNER23:33
jgriffithcuz I'm dumb :(23:33
jungleboyj*Phew*  Ok.  So we need to fix that?23:34
jgriffithWell, I would think so; but I'm unclear how that policy stuff works now.23:34
jgriffithBecause notice I have a test in place23:34
jgriffithand the test works23:34
jungleboyjHmmm, I am baffled by the Policy stuff too.23:35
jgriffithI'll add it, just a sec23:35
jgriffithbut if it blows up the unit test I'm going to be sad23:35
jungleboyjBut I thought that indicated the default.23:35
jgriffithshit that'll be another rebase too :(23:35
jgriffithjungleboyj: yeah, it does23:36
jungleboyjYou have it for the other Policy.23:36
jungleboyjOk.  That was waht I thought.23:36
jungleboyjWhich is obvious from the name of the field.  *Sigh*23:36
jgriffithrunning unit tests now, see how it goes23:37
jungleboyjOk.  Sorry to rain on the parade.23:37
jgriffithNahh... I've got an umbrella23:40
jungleboyjOk good.23:40
*** moshele has joined #openstack-cinder23:41
openstackgerritJohn Griffith proposed openstack/cinder master: Add multiattach policy  https://review.openstack.org/53156923:41
openstackgerritJohn Griffith proposed openstack/cinder master: Enable multiattach capability  https://review.openstack.org/53228023:41
jungleboyjjgriffith: I take it that UT passed?23:43
jgriffithjungleboyj: WRT to the log.error23:43
jgriffithyes23:43
jungleboyjjgriffith:  Good.23:43
jungleboyjYeah ...23:43
openstackgerritMarc Koderer proposed openstack/cinder master: Adapt cinder backup swift help test  https://review.openstack.org/53497123:43
jgriffiththat shouldn't actually be an error because it's just an invalid op ting23:43
jgriffiththing23:43
jgriffithMatt pointed out in a prior review that I should think of "errors" only as things that cause someone's pager to go off in the middle of the night23:44
jungleboyjjgriffith:  Looking again.23:44
jungleboyjjgriffith:  Ok.  That makes sense.23:46
jgriffithon that note, I'm heading on out of here23:47
jgriffithping if something else comes up, I'll try to log on later tonight23:47
jungleboyjOk.  Have a good night buddy.23:47
jgriffiththanks man!  You too23:47
jungleboyjWill do.23:47

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!