Meant to comment on the os.sep thing - since the existing code already uses '/':
absolute_path = os.path.join(fs, path.lstrip('/'))
and we don't support compute nodes on any platforms where '/' isn't the path separator, I don't think it's an issue.
No problem with a later general "switch to os.sep" cleanup in Nova, though.
Meant to comment on the os.sep thing - since the existing code already uses '/':
absolute_path = os.path.join(fs, path.lstrip('/'))
and we don't support compute nodes on any platforms where '/' isn't the path separator, I don't think it's an issue.
No problem with a later general "switch to os.sep" cleanup in Nova, though.