Change in ovirt-engine[master]: core: [TEXT] restore.sh woes, unable to...

emesika at redhat.com emesika at redhat.com
Tue Nov 5 14:41:40 UTC 2013


Eli Mesika has submitted this change and it was merged.

Change subject: core:  [TEXT] restore.sh woes, unable to...
......................................................................


core:  [TEXT] restore.sh woes, unable to...

[TEXT] restore.sh woes, unable to restore engine DB without using -u postgres

1) removing the internal create/drop of the database, this should be
done manually as explained in the script usage statement
2) adding a comment that this script will work only for DB engine user
3) removing from backup script the -C flag to prevent database creation
statements generation
4) If an old backup having "CREATE DATABASE" or "ALTER DATABASE" is
used, those statements are removed before an attempt to restore the
database is done.

Change-Id: I44921d6fac57cfc565dacca52ed5a6d3b5b57f66
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=976337
Signed-off-by: Eli Mesika <emesika at redhat.com>
---
M packaging/dbscripts/backup.sh
M packaging/dbscripts/restore.sh
2 files changed, 36 insertions(+), 31 deletions(-)

Approvals:
  Eli Mesika: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I44921d6fac57cfc565dacca52ed5a6d3b5b57f66
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Barak Azulay <bazulay at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list