Skip to content

Commit

Permalink
CA-227605: fix inconsistent check in the CA-227605 patch
Browse files Browse the repository at this point in the history
The check on signed contained a typo that made it pass some tests but
failed on more careful ones.

Signed-off-by: Marcello Seri <[email protected]>
  • Loading branch information
Marcello Seri committed Nov 25, 2016
1 parent 319f6a9 commit afa13bf
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions scripts/setup-pvs-proxy-rules
Original file line number Diff line number Diff line change
Expand Up @@ -51,15 +51,15 @@ fi
# The VIF UUID is maintained across the migration.
# Furthermore, a proxied VIF can be in only one PVS site at once.
pvs_prefix="/xapi/pvs-proxy"
started=false
started="false"
for path in $($XSLIST -p "$pvs_prefix"); do
PVS_PROXY_STATE=$($XSREAD "$path/$VIF/state")
if [ $? -eq 0 ] && [ "$PVS_PROXY_STATE" = "started" ]; then
started=true
started="true"
break
fi
done
if [ ! started ]; then
if [ "$started" = "false" ]; then
handle_error "PVS proxy daemon not configured for this proxy - not installing OVS rules."
fi

Expand Down

0 comments on commit afa13bf

Please sign in to comment.