Skip to content

Commit

Permalink
Merge pull request kubernetes#10826 from stevekuznetsov/skuznets/supr…
Browse files Browse the repository at this point in the history
…ess-log

Don't warn when no branch protection is set
  • Loading branch information
k8s-ci-robot authored Jan 18, 2019
2 parents 491a8f1 + 27f3b4b commit 7ffd3f1
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions prow/config/tide.go
Original file line number Diff line number Diff line change
Expand Up @@ -442,9 +442,7 @@ func (c Config) GetTideContextPolicy(org, repo, branch string) (*TideContextPoli
bp, err := c.GetBranchProtection(org, repo, branch)
if err != nil {
logrus.WithError(err).Warningf("Error getting branch protection for %s/%s+%s", org, repo, branch)
} else if bp == nil {
logrus.Warningf("branch protection not set for %s/%s+%s", org, repo, branch)
} else if bp.Protect != nil && *bp.Protect && bp.RequiredStatusChecks != nil {
} else if bp != nil && bp.Protect != nil && *bp.Protect && bp.RequiredStatusChecks != nil {
required.Insert(bp.RequiredStatusChecks.Contexts...)
}
}
Expand Down

0 comments on commit 7ffd3f1

Please sign in to comment.