Skip to content

Commit

Permalink
add changelog
Browse files Browse the repository at this point in the history
Signed-off-by: yy <[email protected]>
  • Loading branch information
yangyy93 committed Jul 24, 2023
1 parent 929e6cb commit 1cf3340
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
3 changes: 3 additions & 0 deletions changelogs/unreleased/5575-yangyy93-minor.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
## Add outlier detection related configuration detection for services

Add [outlier detection](https://www.envoyproxy.io/docs/envoy/v1.26.3/intro/arch_overview/upstream/outlier#arch-overview-outlier-detection) related configuration detection for services, including consecutiveServerErrors and localOriginal errors, and passive health checks can be performed on clusters.
2 changes: 1 addition & 1 deletion internal/dag/policy_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -15,12 +15,12 @@ package dag

import (
"errors"
"github.com/projectcontour/contour/internal/ref"
"io"
"testing"
"time"

contour_api_v1 "github.com/projectcontour/contour/apis/projectcontour/v1"
"github.com/projectcontour/contour/internal/ref"
"github.com/projectcontour/contour/internal/timeout"
"github.com/sirupsen/logrus"
"github.com/stretchr/testify/assert"
Expand Down
2 changes: 0 additions & 2 deletions internal/envoy/v3/cluster.go
Original file line number Diff line number Diff line change
Expand Up @@ -408,8 +408,6 @@ func outlierDetection(policy *dag.OutlierDetectionPolicy) *envoy_cluster_v3.Outl
out.SplitExternalLocalOriginErrors = true
if policy.ConsecutiveLocalOriginFailure > 0 {
out.ConsecutiveLocalOriginFailure = protobuf.UInt32OrNil(policy.ConsecutiveLocalOriginFailure)
// Default to 100% if not specified
//out.EnforcingConsecutiveLocalOriginFailure = protobuf.UInt32OrNil(100)
} else {
// Default to 5 if not specified
out.ConsecutiveLocalOriginFailure = protobuf.UInt32OrNil(5)
Expand Down

0 comments on commit 1cf3340

Please sign in to comment.