Skip to content

Commit

Permalink
fix OutlierDetectionTest
Browse files Browse the repository at this point in the history
  • Loading branch information
kozjan committed Jun 25, 2024
1 parent 46037f7 commit ffc1ac0
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ import org.junit.jupiter.api.extension.RegisterExtension
import pl.allegro.tech.servicemesh.envoycontrol.assertions.isFrom
import pl.allegro.tech.servicemesh.envoycontrol.assertions.isOk
import pl.allegro.tech.servicemesh.envoycontrol.assertions.untilAsserted
import pl.allegro.tech.servicemesh.envoycontrol.config.RandomConfigFile
import pl.allegro.tech.servicemesh.envoycontrol.config.consul.ConsulExtension
import pl.allegro.tech.servicemesh.envoycontrol.config.service.EchoServiceExtension
import pl.allegro.tech.servicemesh.envoycontrol.config.envoy.EnvoyExtension
Expand Down Expand Up @@ -33,9 +34,18 @@ class OutlierDetectionTest {
@RegisterExtension
val unhealthyService = EchoServiceExtension()

// necessary since Envoy 1.28.0 to keep the same behaviour (https://www.envoyproxy.io/docs/envoy/latest/version_history/v1.28/v1.28.0)
private val outlierEjectionConfig = """
layered_runtime:
layers:
- name: static_layer
static_layer:
envoy.reloadable_features.check_mep_on_first_eject: false
""".trimIndent()

@JvmField
@RegisterExtension
val envoy = EnvoyExtension(envoyControl)
val envoy = EnvoyExtension(envoyControl, config = RandomConfigFile.copy(configOverride = outlierEjectionConfig))
}

@Test
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ class EnvoyContainer(
private const val ADMIN_PORT = 10000

private const val MIN_SUPPORTED_ENVOY_VERSION = "v1.22.7"
private const val MAX_SUPPORTED_ENVOY_VERSION = "v1.30.2" // todo: v1.28.0+ - OutlierDetectionTest breaks
private const val MAX_SUPPORTED_ENVOY_VERSION = "v1.30.2"

val DEFAULT_IMAGE = run {
val version =
Expand Down

0 comments on commit ffc1ac0

Please sign in to comment.