Yedidyah Bar David has uploaded a new change for review.

Change subject: packaging: Do not overwrite 10-setup-database.conf if not 
required
......................................................................

packaging: Do not overwrite 10-setup-database.conf if not required

Without this change, engine-backup --mode=restore and without
--change-db-credentials will empty 10-setup-database.conf .

Bug-Url: https://bugzilla.redhat.com/1002401
Change-Id: I5b2e4d0e291e8c5a8afabd7d8cca29f289154b0a
Signed-off-by: Yedidyah Bar David <[email protected]>
---
M packaging/bin/engine-backup.sh
1 file changed, 3 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/91/21791/1

diff --git a/packaging/bin/engine-backup.sh b/packaging/bin/engine-backup.sh
index 469c3dc..4aea0ad 100755
--- a/packaging/bin/engine-backup.sh
+++ b/packaging/bin/engine-backup.sh
@@ -306,7 +306,9 @@
        verifyConnection
        log "Restoring database backup at 
${TEMP_FOLDER}/db/${DB_BACKUP_FILE_NAME}"
        restoreDB "${TEMP_FOLDER}/db/${DB_BACKUP_FILE_NAME}"
-       changeDBConf
+       if [ -n "${CHANGE_DB_CREDENTIALS}" ]; then
+               changeDBConf
+       fi
        output "Note: you might need to manually fix:"
        output "- iptables/firewalld configuration"
        output "- autostart of ovirt-engine service"


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5b2e4d0e291e8c5a8afabd7d8cca29f289154b0a
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.3
Gerrit-Owner: Yedidyah Bar David <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to