Change in ovirt-engine[master]: Revert "packaging: setup: Add to unremovable only files we w...

Code Review gerrit at ovirt.org
Thu Aug 3 07:54:05 UTC 2017


>From Sandro Bonazzola <sbonazzo at redhat.com>:

Sandro Bonazzola has submitted this change and it was merged. ( https://gerrit.ovirt.org/79902 )

Change subject: Revert "packaging: setup: Add to unremovable only files we wrote"
......................................................................


Revert "packaging: setup: Add to unremovable only files we wrote"

This reverts commit c1ee0753624939dcc14ba1741b5390f48eb0c093.

That commit broke cleanup code under certain conditions, yet to be fully
diagnosed.

Change-Id: I83b420cab2142b5d474e10a6fa0ede273afe96ee
Bug-Url: https://bugzilla.redhat.com/1475788
Signed-off-by: Yedidyah Bar David <didi at redhat.com>
---
M packaging/setup/plugins/ovirt-engine-common/base/core/uninstall.py
1 file changed, 14 insertions(+), 18 deletions(-)

Approvals:
  Sandro Bonazzola: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Yedidyah Bar David: Verified



-- 
To view, visit https://gerrit.ovirt.org/79902
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I83b420cab2142b5d474e10a6fa0ede273afe96ee
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yedidyah Bar David <didi at redhat.com>
Gerrit-Reviewer: Ido Rosenzwig <irosenzw at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Lev Veyde <lveyde at redhat.com>
Gerrit-Reviewer: Rafael Martins <rafael at rafaelmartins.eng.br>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos at redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list