Merge pull request #50418 from liggitt/target-godep

Automatic merge from submit-queue (batch tested with PRs 50418, 49830, 49206, 49061, 49912) Target godep script change verifications helps with #50319 `hack/verify-godeps.sh` takes ~15 minutes of the verify job time. We should not run it if not required, especially since the job is regularly timing out at 1 hour when this check is included. https://github.com/kubernetes/kubernetes/pull/48653 added a check to run it if *anything* under `hack` was changed. This targetes just changes to the godep scripts

Merge pull request #50418 from liggitt/target-godep
4d4dd238 · Kubernetes Submit Queue · GitHub · 17ec91c5 · 125a8404 · 4d4dd238
Commit 4d4dd238 authored Aug 09, 2017 by Kubernetes Submit Queue Committed by GitHub Aug 09, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

verify-godeps.sh hack/verify-godeps.sh +1 -1

No files found.
--- a/hack/verify-godeps.sh
+++ b/hack/verify-godeps.sh
@@ -49,7 +49,7 @@ readonly branch=${1:-${KUBE_VERIFY_GIT_BRANCH:-master}}
 if ! [[ ${KUBE_FORCE_VERIFY_CHECKS:-} =~ ^[yY]$ ]] && \
  ! kube::util::has_changes_against_upstream_branch "${branch}" 'Godeps/' && \
  ! kube::util::has_changes_against_upstream_branch "${branch}" 'vendor/' && \
-  ! kube::util::has_changes_against_upstream_branch "${branch}" 'hack/'; then
+  ! kube::util::has_changes_against_upstream_branch "${branch}" 'hack/.*godep'; then
  exit 0
 fi