Merge pull request #22329 from stevekuznetsov/skuznets/check-empty-sha

prow: check for empty refs on fetch call
kubernetes · May 27, 2021 · ae26658 · ae26658
2 parents 88fab4f + 036f593
commit ae26658
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/prow/repoowners/repoowners.go b/prow/repoowners/repoowners.go
@@ -270,6 +270,9 @@ func (c *Client) LoadRepoOwners(org, repo, base string) (RepoOwner, error) {
 	if err != nil {
 		return nil, fmt.Errorf("failed to get current SHA for %s: %v", fullName, err)
 	}
+	if sha == "" {
+		return nil, fmt.Errorf("got an empty SHA for %s@heads/%s", fullName, base)
+	}
 	log.WithField("duration", time.Since(start).String()).Debugf("Completed ghc.GetRef(%s, %s, %s)", org, repo, fmt.Sprintf("heads/%s", base))
 
 	entry, err := c.cacheEntryFor(org, repo, base, cloneRef, fullName, sha, log)