Yedidyah Bar David has posted comments on this change.

Change subject: packaging: setup: List unreverted changes on cleanup
......................................................................


Patch Set 2:

> Hmmm... I cannot run it now, can you please paste example of output? It is 
> not *THAT* important, but I think that too much information is not doing us 
> great service.

On stdout:

[WARNING] Some changes to /etc/httpd/conf.d/ssl.conf could not be reverted. 
More details can be found in the log.

In the log:

2014-04-28 17:49:23 WARNING 
otopi.plugins.ovirt_engine_remove.ovirt_engine.files.simple 
simple._revertChanges:109 Some changes to /etc/httpd/conf.d/ssl.conf could not 
be reverted. More details can be found in the log.

2014-04-28 17:49:23 DEBUG 
otopi.plugins.ovirt_engine_remove.ovirt_engine.files.simple 
simple._revertChanges:122 The following lines were not found in 
/etc/httpd/conf.d/ssl.conf and so were not removed:

        SSLCACertificateFile /etc/pki/ovirt-engine/apache-ca.pem

I don't have time now to do that again with multiple changes but I did verify 
that then. don't have the logs anymore. The blank lines are not in the log - 
added them for readability in gerrit.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib9555c6d6a2f1652edc1065f9c49154123e8accc
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yedidyah Bar David <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Itamar Heim <[email protected]>
Gerrit-Reviewer: Ofer Schreiber <[email protected]>
Gerrit-Reviewer: Sandro Bonazzola <[email protected]>
Gerrit-Reviewer: Yedidyah Bar David <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to