Exceptions in meta-mock.py need more care
Bug #1701527 reported by
Joonas Kylmälä
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| cloud-init |
Fix Released
|
Low
|
Unassigned | ||
Bug Description
The tools/mock-meta.py has one error in exception handling (it excepts KeyError which is made for dicts instead of IndexError that is made for lists) and also it returns non-describing http code. I have attached a patch that fixes this.
Related branches
~j.kylmala/cloud-init:exceptions
- Server Team CI bot: Approve (continuous-integration)
- cloud-init Commiters: Pending requested
-
Diff: 15 lines (+2/-2)1 file modifiedtools/mock-meta.py (+2/-2)
Revision history for this message
| Joonas Kylmälä (j.kylmala) wrote | #1 |
| Changed in cloud-init: | |
| status: | New → Confirmed |
| importance: | Undecided → Low |
| status: | Confirmed → Fix Committed |
Revision history for this message
| Scott Moser (smoser) wrote Fixed in Cloud-init 17.1 | #2 |
| Changed in cloud-init: | |
| status: | Fix Committed → Fix Released |
Revision history for this message
| James Falcon (falcojr) wrote | #3 |
To post a comment you must log in.
This bug is believed to be fixed in cloud-init in 17.1. If this is still a problem for you, please make a comment and set the state back to New
Thank you.