Skip to content

Commit

Permalink
Merge pull request #3795 from Shopify/optional-be-optional
Browse files Browse the repository at this point in the history
do not warn when optional annotations arent set
  • Loading branch information
k8s-ci-robot authored Feb 21, 2019
2 parents bd61b7a + 420d804 commit fb1df21
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions internal/ingress/annotations/authreq/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -146,12 +146,12 @@ func (a authReq) Parse(ing *extensions.Ingress) (interface{}, error) {
// Optional Parameters
signIn, err := parser.GetStringAnnotation("auth-signin", ing)
if err != nil {
klog.Warning("auth-signin annotation is undefined and will not be set")
klog.V(3).Infof("auth-signin annotation is undefined and will not be set")
}

authSnippet, err := parser.GetStringAnnotation("auth-snippet", ing)
if err != nil {
klog.Warning("auth-snippet annotation is undefined and will not be set")
klog.V(3).Infof("auth-snippet annotation is undefined and will not be set")
}

responseHeaders := []string{}
Expand Down

0 comments on commit fb1df21

Please sign in to comment.