Skip to content
This repository has been archived by the owner on Nov 1, 2022. It is now read-only.

Commit

Permalink
Merge pull request #1840 from 2opremio/tolerate-cluster-forbidden-lis…
Browse files Browse the repository at this point in the history
…t-encore-encore

Improve Forbidden and NotFound error detection
  • Loading branch information
Alfonso Acosta authored Mar 19, 2019
2 parents 2d4cc4d + 431793d commit 9393347
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions cmd/fluxd/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -18,11 +18,11 @@ import (
"github.com/prometheus/client_golang/prometheus/promhttp"
"github.com/spf13/pflag"
crd "k8s.io/apiextensions-apiserver/pkg/client/clientset/clientset"
k8serrors "k8s.io/apimachinery/pkg/api/errors"
k8sruntime "k8s.io/apimachinery/pkg/util/runtime"
k8sclientdynamic "k8s.io/client-go/dynamic"
k8sclient "k8s.io/client-go/kubernetes"
"k8s.io/client-go/rest"
k8serrors "k8s.io/kubernetes/staging/src/k8s.io/apimachinery/pkg/api/errors"

"github.com/weaveworks/flux/checkpoint"
"github.com/weaveworks/flux/cluster"
Expand Down Expand Up @@ -169,7 +169,10 @@ func main() {
// Silence access errors logged internally by client-go
klog := log.With(logger, "type", "internal kubernetes error")
logErrorUnlessAccessRelated := func(err error) {
if k8serrors.IsForbidden(err) || k8serrors.IsNotFound(err) {
errLower := strings.ToLower(err.Error())
if k8serrors.IsForbidden(err) || k8serrors.IsNotFound(err) ||
strings.Contains(errLower, "forbidden") ||
strings.Contains(errLower, "not found") {
return
}
klog.Log("err", err)
Expand Down

0 comments on commit 9393347

Please sign in to comment.