Refactor TaskFlow implementation
Code that's using TaskFlow in Cinder is almost 2 years old. There's a lot of outdated comments and redundant code paths. Additionally some limitations of TaskFlow led to ugly workarounds that are making the code hard to understand. At the summit team agreed to have time for refactoring till around Liberty-1 and then decide if we want to integrate with TaskFlow further.
This BP aims to clean up and refactor each of create_volume workflows (c-api, c-sch, c-vol). To cover TaskFlow limitations problem we want to step up and work with TaskFlow community to address the gaps and refactor Cinder's code to use latest features.
All these efforts should make current code easier to understand and make further TaskFlow adoption possible.
[1] https:/
[2] https:/
Blueprint information
- Status:
- Complete
- Approver:
- Mike Perez
- Priority:
- Medium
- Drafter:
- Michal Dulko
- Direction:
- Approved
- Assignee:
- Michal Dulko
- Definition:
- Approved
- Series goal:
- Accepted for liberty
- Implementation:
- Implemented
- Milestone target:
- 7.0.0
- Started by
- Mike Perez
- Completed by
- Michal Dulko
Related branches
Related bugs
Sprints
Whiteboard
Gerrit topic: https:/
Addressed by: https:/
Refactor API create_volume flow
Addressed by: https:/
WIP: Refactoring of manager's create_volume flow
Gerrit topic: https:/
Addressed by: https:/
Remove unused arguments from c-vol's create_volume
Addressed by: https:/
Refactoring of manager's manage_existing flow