Cinder outputs traceback in c-api service logs if taskflow raises exception.InvalidInput and exception.QuotaError
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Cinder |
Fix Released
|
Undecided
|
Dinesh Bhor |
Bug Description
Cinder uses '_format_failure' method to suppress the traceback in logs [1]. But this method no
longer exists in taskflow. It is removed in this patch [2]. Thus if exception.
Command:
exception.
$ cinder create --name new_volume 0
ERROR: Invalid input received: Volume size '0' must be an integer and greater than 0 (HTTP 400) (Request-ID: req-0ef0583c-
C-api logs:
2016-12-08 19:48:03.332 WARNING cinder.volume.api [req-0ef0583c-
1 predecessors (most recent first):
Flow 'volume_create_api'
2016-12-08 19:48:03.332 TRACE cinder.volume.api Traceback (most recent call last):
2016-12-08 19:48:03.332 TRACE cinder.volume.api File "/usr/local/
2016-12-08 19:48:03.332 TRACE cinder.volume.api result = task.execute(
2016-12-08 19:48:03.332 TRACE cinder.volume.api File "/opt/stack/
2016-12-08 19:48:03.332 TRACE cinder.volume.api size = self._extract_
2016-12-08 19:48:03.332 TRACE cinder.volume.api File "/opt/stack/
2016-12-08 19:48:03.332 TRACE cinder.volume.api func(size)
2016-12-08 19:48:03.332 TRACE cinder.volume.api File "/opt/stack/
2016-12-08 19:48:03.332 TRACE cinder.volume.api raise exception.
2016-12-08 19:48:03.332 TRACE cinder.volume.api InvalidInput: Invalid input received: Volume size '0' must be an integer and greater than 0
2016-12-08 19:48:03.332 TRACE cinder.volume.api
2016-12-08 19:48:03.335 DEBUG cinder.volume.api [req-0ef0583c-
exception.
$ cinder create 1111111111111111
ERROR: VolumeSizeExcee
C-api logs:
|__Flow 'volume_create_api'
2016-12-14 15:17:35.417 TRACE cinder.volume.api Traceback (most recent call last):
2016-12-14 15:17:35.417 TRACE cinder.volume.api File "/usr/local/
st-packages/
2016-12-14 15:17:35.417 TRACE cinder.volume.api result = task.execute(
ts)
2016-12-14 15:17:35.417 TRACE cinder.volume.api File "/opt/stack/
lume/flows/
2016-12-14 15:17:35.417 TRACE cinder.volume.api size=size)
2016-12-14 15:17:35.417 TRACE cinder.volume.api File "/opt/stack/
ota_utils.py", line 283, in process_
2016-12-14 15:17:35.417 TRACE cinder.volume.api quota=quotas[over])
2016-12-14 15:17:35.417 TRACE cinder.volume.api VolumeSizeExcee
2016-12-14 15:17:35.417 TRACE cinder.volume.api
2016-12-14 15:17:35.424 DEBUG cinder.volume.api [req-ee0d47b0-
Taskflow has introduced a new class 'taskflow.
[1] https:/
[2] https:/
Changed in cinder: | |
assignee: | nobody → Dinesh Bhor (dinesh-bhor) |
Fix proposed to branch: master /review. openstack. org/412819
Review: https:/