Clean up unused exceptions
Bug #1271283 reported by
aeva black
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Ironic |
Fix Released
|
Low
|
aeva black |
Bug Description
We should clean up unused exceptions from ironic.
$ cd ironic/ironic
$ for e in $(grep -P '^class ' common/exception.py | sed 's/(/ /' | awk '{print $2}'); do l=$(grep -rP "raise .*\.$e" . | wc -l); if [ $l == 0 ]; then l=$(grep "($e):" common/exception.py | wc -l); fi; if [ $l == 0 ]; then echo $e; fi ; done
AdminRequired
PolicyNotAuthorized
InvalidCPUInfo
InvalidIpAddres
InvalidDiskFormat
DiskNotFound
HostNotFound
ConsoleNotFound
FileNotFound
NoValidHost
NodeInUse
NodeNotConfigured
ImageNotAuthorized
HTTPException
InvalidEndpoint
CommunicationError
Changed in ironic: | |
status: | New → Triaged |
importance: | Undecided → Low |
tags: | added: low-hanging-fruit |
description: | updated |
Changed in ironic: | |
assignee: | nobody → Michael Davies (mrda) |
status: | Triaged → In Progress |
description: | updated |
Changed in ironic: | |
assignee: | Michael Davies (mrda) → Joshua Hesketh (joshua.hesketh) |
Changed in ironic: | |
assignee: | Joshua Hesketh (joshua.hesketh) → Devananda van der Veen (devananda) |
Changed in ironic: | |
milestone: | none → icehouse-3 |
status: | Fix Committed → Fix Released |
Changed in ironic: | |
milestone: | icehouse-3 → 2014.1 |
To post a comment you must log in.
I've reviewed the current code and this exception list, and I don't think there's any immediate use for these exceptions.
I'm recommending deleting them.