gateway error detail is not passed along in raised exception
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
python-etcd3gw (Ubuntu) |
Fix Released
|
Medium
|
Heather Lemon | ||
Bionic |
Fix Released
|
Medium
|
Heather Lemon | ||
Focal |
Fix Released
|
Medium
|
Heather Lemon | ||
Groovy |
Won't Fix
|
Medium
|
Heather Lemon | ||
Hirsute |
Fix Released
|
Medium
|
Heather Lemon |
Bug Description
[impact]
when the gateway reports an error, it is not passed along in the exception raised by python-etcd3gw
[test case]
Upstream added unit tests for this bug, which are included in the backport, so a successful package build (which runs the unit tests at build time) is enough to verify the fix.
[regression potential]
* Any regression would likely occur in handling errors sent from the gateway to python-etcd3gw, or in handling or later processing the exception(s) generated from the gateway error(s).
* This patchset introduces 'import mock' in a test unit which requires python3-mock as a Build-Depends.
For now, it is indirectly covered by python3-oslotest as follows:
d/control:
18 Build-Depends-
19 python-coverage,
20 python-futurist,
21 python-hacking,
22 python-nose,
23 python-oslosphinx,
24 python-oslotest,
25 python-pytest,
26 python-requests,
27 python-six,
28 python-
29 python-testtools,
30 python-urllib3,
31 python3-futurist,
32 python3-nose,
=> 33 python3-oslotest,
~# apt-cache depends python3-oslotest
python3-oslotest
Depends: python3-fixtures
=> Depends: python3-mock
Ideally, it would be best to have an implicit instruction for it, but since in Stable release, adding new build dependencies is not the preference and that python3-oslotest will unlikely be removed from the list. It shouldn't have any effect in the future. Worse case, python-etcd3gw will stop building, and we will have to implicitly instruct d/control to have python3-mock as Build-Depends.
[scope]
three commits are needed for b/f/g, and one commit is needed in h.
this is fixed upstream with commits:
483a37e28a59e29
19abd85b710682b
which are included in v2.5, and commit:
5a3157a122368c2
which is included in v2.6
Debian currently has v2.5, and MR to add the last commit needed is:
https:/
Linking to related LP https:/
Changed in python-etcd3gw (Ubuntu Bionic): | |
status: | New → In Progress |
assignee: | nobody → Dan Streetman (ddstreet) |
Changed in python-etcd3gw (Ubuntu Focal): | |
assignee: | nobody → Dan Streetman (ddstreet) |
Changed in python-etcd3gw (Ubuntu Groovy): | |
assignee: | nobody → Dan Streetman (ddstreet) |
Changed in python-etcd3gw (Ubuntu Bionic): | |
importance: | Undecided → Medium |
Changed in python-etcd3gw (Ubuntu Focal): | |
importance: | Undecided → Medium |
Changed in python-etcd3gw (Ubuntu Groovy): | |
importance: | Undecided → Medium |
Changed in python-etcd3gw (Ubuntu Focal): | |
status: | New → In Progress |
Changed in python-etcd3gw (Ubuntu Groovy): | |
status: | New → In Progress |
description: | updated |
Changed in python-etcd3gw (Ubuntu Groovy): | |
assignee: | Dan Streetman (ddstreet) → Heather Lemon (hypothetical-lemon) |
Changed in python-etcd3gw (Ubuntu Focal): | |
assignee: | Dan Streetman (ddstreet) → Heather Lemon (hypothetical-lemon) |
Changed in python-etcd3gw (Ubuntu Bionic): | |
assignee: | Dan Streetman (ddstreet) → Heather Lemon (hypothetical-lemon) |
Changed in python-etcd3gw (Ubuntu): | |
assignee: | Dan Streetman (ddstreet) → nobody |
status: | In Progress → Fix Released |
Changed in python-etcd3gw (Ubuntu Hirsute): | |
assignee: | nobody → Heather Lemon (hypothetical-lemon) |
description: | updated |
description: | updated |
tags: | added: sts sts-sponsor-slashd |
description: | updated |
description: | updated |
Changed in python-etcd3gw (Ubuntu Hirsute): | |
status: | Fix Released → In Progress |
tags: | added: sts-sponsor |
description: | updated |
description: | updated |
description: | updated |
description: | updated |
tags: |
added: verification-done-bionic verification-done-focal removed: verification-needed-focal |
attached debdiff