Change in ovirt-engine[master]: db: do not execute sql files as shell scripts

Code Review gerrit at ovirt.org
Wed Jul 5 08:19:36 UTC 2017


>From Eli Mesika <emesika at redhat.com>:

Eli Mesika has submitted this change and it was merged. ( https://gerrit.ovirt.org/78778 )

Change subject: db: do not execute sql files as shell scripts
......................................................................


db: do not execute sql files as shell scripts

This patch insures that id SQL scripts have the executable flag set that
the upgrade will not try to execute them as shell scripts.

Change-Id: If0e38818de1b53f07e57ccacbb1924d534869ece
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1464735
Signed-off-by: emesika <emesika at redhat.com>
---
M packaging/dbscripts/dbfunc-common.sh
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
  Eli Mesika: Verified
  Jenkins CI: Passed CI tests
  Moti Asayag: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: If0e38818de1b53f07e57ccacbb1924d534869ece
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Ravi Nori <rnori at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list