Skip to content

Commit

Permalink
Merge pull request #29920 from thockin/verify-godeps-results
Browse files Browse the repository at this point in the history
Automatic merge from submit-queue

Make verify-godeps check the results

Verify the results of godep restore/save.  When forced to run this finds a handful of real deltas that need to be resolved.

Fixes #29402

@rmmh because you wrote  #29571 which is the major culprit
  • Loading branch information
Kubernetes Submit Queue authored Aug 4, 2016
2 parents 10b0f58 + b9d68ab commit 0e55753
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions hack/verify-godeps.sh
Original file line number Diff line number Diff line change
Expand Up @@ -104,5 +104,12 @@ if ! _out="$(diff -Naupr --ignore-matching-lines='^\s*\"GoVersion\":' --ignore-m
exit 1
fi

if ! _out="$(diff -Naupr ${KUBE_ROOT}/vendor ${_kubetmp}/vendor)"; then
echo "Your vendored results are different:"
echo "${_out}"
echo "Godeps Verify failed."
exit 1
fi

echo "Godeps Verified."
# ex: ts=2 sw=2 et filetype=sh

0 comments on commit 0e55753

Please sign in to comment.