Statuses - 3 classes with the same name
Bug #1522733 reported by
Vincent Françoise
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| watcher |
Fix Released
|
Low
|
Vincent Françoise | ||
Bug Description
As we can see in the codebase, we have 3 "Status" enums which are located at:
- watcher/
- watcher/
- watcher/
To make it more consistent with the DB, the objective would be rename them as "State" and prefix each one of them (e.g. ActionState) as they are meant to be different although it looks like duplicated code.
Adding a docstring to each one of them would be appreciated.
| Changed in watcher: | |
| importance: | Undecided → Low |
| Changed in watcher: | |
| assignee: | nobody → Steve Wilkerson (sw5822) |
| Changed in watcher: | |
| assignee: | Steve Wilkerson (sw5822) → nobody |
| assignee: | nobody → Steve Wilkerson (sw5822) |
| status: | New → In Progress |
| Changed in watcher: | |
| milestone: | none → mitaka-2 |
| Changed in watcher: | |
| assignee: | Steve Wilkerson (sw5822) → nobody |
| Changed in watcher: | |
| assignee: | nobody → Vincent Françoise (vincent-francoise) |
| description: | updated |
Revision history for this message
| OpenStack Infra (hudson-openstack) wrote Fix proposed to watcher (master) | #1 |
Revision history for this message
| OpenStack Infra (hudson-openstack) wrote Fix merged to watcher (master) | #2 |
| Changed in watcher: | |
| status: | In Progress → Fix Released |
To post a comment you must log in.
Fix proposed to branch: master
Review: https:/