Edward Haas has posted comments on this change.
Change subject: utils: atomic write ......................................................................
Patch Set 8:
(1 comment)
https://gerrit.ovirt.org/#/c/61482/8/lib/vdsm/utils.py File lib/vdsm/utils.py:
Line 932: return rget(dict.get(keys[0]), keys[1:], default) Line 933: Line 934: Line 935: @contextmanager Line 936: def atomic_write(path, flag):
this implies about the usage imo
As I see it, it's like asking to specify the type name in the variable (or the returned value type of a func). The func is marked as a contextmanager, that is enough imo.
Leaving it to you guys to solve this out, I won't fight over it. Line 937: """Atomically write into a file. Line 938: Line 939: Usage: Line 940: