Alon Bar-Lev has posted comments on this change.

Change subject: packaging: updated handling of pgpass file between versions
......................................................................


Patch Set 5:

Let's sum it up...

setup should only use new location.

so the issue is upgrade only

As soon as we start upgrade, we should copy the old location to new location.

This means that during upgrade and setup we always have the file at new 
location.

If upgrade succeed commit by removing the old location.

If upgrade fails rollback by removing the new location.

If I understand correctly, it is simpler than this implementation...

--
To view, visit http://gerrit.ovirt.org/8135
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I1d44297725d7270982e8724e4aeefda8bc7a88e2
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alex Lourie <[email protected]>
Gerrit-Reviewer: Alex Lourie <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Juan Hernandez <[email protected]>
Gerrit-Reviewer: Kiril Nesenko <[email protected]>
Gerrit-Reviewer: Moran Goldboim <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to