Dan Kenigsberg has submitted this change and it was merged.
Change subject: utils: atomic file write
..
utils: atomic file write
Atomic writes are needed for safe file editation. This function will
be used in following network
gerrit-hooks has posted comments on this change.
Change subject: utils: atomic file write
..
Patch Set 5:
* #1364081::Update tracker: OK
* Set MODIFIED::bug 1364081#1364081IGNORE, not all related patches are
closed,
gerrit-hooks has posted comments on this change.
Change subject: utils: atomic file write
..
Patch Set 4:
* #1364081::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1364081::OK, public bug
* Check
Edward Haas has posted comments on this change.
Change subject: utils: atomic file write
..
Patch Set 3:
Per the current limited usage, I do not think it is worth importing a 3rd party
library (like python-atomicwrites).
That
Yaniv Bronhaim has posted comments on this change.
Change subject: utils: atomic file write
..
Patch Set 3: Code-Review+2
(2 comments)
https://gerrit.ovirt.org/#/c/61950/3/lib/vdsm/utils.py
File lib/vdsm/utils.py:
Line 949:
Yaniv Kaul has posted comments on this change.
Change subject: utils: atomic file write
..
Patch Set 3:
(2 comments)
Why not use something like https://github.com/untitaker/python-atomicwrites ?
gerrit-hooks has posted comments on this change.
Change subject: utils: atomic file write
..
Patch Set 3:
* #1364081::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1364081::OK, public bug
* Check
Edward Haas has posted comments on this change.
Change subject: utils: atomic file write
..
Patch Set 2: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/61950
To unsubscribe, visit https://gerrit.ovirt.org/settings
Petr Horáček has posted comments on this change.
Change subject: utils: atomic file write
..
Patch Set 2: Verified+1
Passed network/*_test.py, utilsTests.py and functional/networkTests.py.
--
To view, visit
gerrit-hooks has posted comments on this change.
Change subject: utils: atomic file write
..
Patch Set 2:
* #1364081::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1364081::OK, public bug
* Check
gerrit-hooks has posted comments on this change.
Change subject: utils: atomic file write
..
Patch Set 1:
* #1364081::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1364081::OK, public bug
* Check
Hello Yaniv Bronhaim, Dan Kenigsberg, Edward Haas,
I'd like you to do a code review. Please visit
https://gerrit.ovirt.org/61950
to review the following change.
Change subject: utils: atomic file write
..
utils: atomic
12 matches
Mail list logo