Remove unpythonic execs of system commands
Nova has lots of places where we exec() commands like rm, mv, cp, dd, mkdir, etc instead of using the pythonic alternative. This is slower, and makes unit testing harder. Also, it makes us look silly at social occasions. We should clean this up.
Additionally, now that utils.execute() is just a wrapper around processutils.
Blueprint information
- Status:
- Complete
- Approver:
- Matt Riedemann
- Priority:
- Low
- Drafter:
- Michael Still
- Direction:
- Approved
- Assignee:
- Michael Still
- Definition:
- Approved
- Series goal:
- Accepted for rocky
- Implementation:
- Implemented
- Milestone target:
- rocky-3
- Started by
- Matt Riedemann
- Completed by
- melanie witt
Related branches
Related bugs
Sprints
Whiteboard
Gerrit topic: https:/
Addressed by: https:/
Remove duplicative implementation of temporary directories.
Addressed by: https:/
Use a pythonic delete.
Addressed by: https:/
Use a pythonic delete, with a retry.
Approved as a specless blueprint for Rocky. -- mriedem 20180322
Addressed by: https:/
Use os.rename, not mv.
Addressed by: https:/
Convert fping API to processutils.
Addressed by: https:/
Convert xenapi's xvp console to processutils.
Addressed by: https:/
Convert certificate generation to processutils.
Addressed by: https:/
Convert configdrive to use processutils.
Addressed by: https:/
Convert libvirt's RBD storage to using processutils.
Closing this out for Rocky as all proposed changes have merged and we're coming up on r-3 feature freeze on July 26. -- melwitt 20180713