Oved Ourfali has posted comments on this change.

Change subject: dbutils: Enable unlock utility handle all entities
......................................................................


Patch Set 3:

(1 comment)

https://gerrit.ovirt.org/#/c/38946/3/packaging/dbscripts/unlock_entity.sh
File packaging/dbscripts/unlock_entity.sh:

Line 220:             entity_unlock "${TYPE}" "" "$(whoami)" ${RECURSIVE}
Line 221:         else
Line 222:           echo "Caution, this operation may lead to data corruption 
and should be used with care. Please contact support prior to running this 
command"
Line 223:           echo "Are you sure you want to proceed? [y/n]"
Line 224:           read answer
> Could we also have a way to bypass interactive confirmation from engine-set
+1.
Eli - can you add a "-y" here to approve without user interaction?
Line 225:           [ "${answer}" = "y" ] || die "Please contact support for 
further assistance."
Line 226: 
Line 227:           for ID in ${IDS} ; do
Line 228:               entity_unlock "${TYPE}" "${ID}" "$(whoami)" ${RECURSIVE}


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I494dece6ba871727750f3fccacd95490294bea6e
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <emes...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Eli Mesika <emes...@redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourf...@redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stira...@redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <d...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to