Krzysztof Słonka

Support instances with hostname

4

author: @jakubdyszkiewicz Envoy throws an error when there are instances with hostaname instead of IP. An error thrown by Envoy ``` [2018-11-08 09:29:32.855][23590][warning][config] bazel-out/k8-opt/bin/source/common/config/_virtual_includes/grpc_mux_subscription_lib/common/config/grpc_mux_subscription_impl.h:70] gRPC config for type.googleapis.com/envoy.api.v2.ClusterLoadAssignment rejected: malformed...

Replace portBindings.add with standard way of exposing ports

5

How to expose ports: https://www.testcontainers.org/features/networking/#exposing-container-ports-to-the-host Where to fix: https://github.com/allegro/envoy-control/blob/b74ba3c64f1ea30dd531f8bb98086e7f4b4f6e1d/envoy-control-tests/src/main/kotlin/pl/allegro/tech/servicemesh/envoycontrol/config/consul/ConsulContainer.kt#L30

Check parsing / validation of timeouts and other settings

1

See: https://github.com/allegro/envoy-control/pull/62#discussion_r373955358

Parallelise test by timing data on circle ci

2

https://circleci.com/gh/allegro/envoy-control/tree/run-tests-in-parallel https://circleci.com/blog/how-to-boost-build-time-with-test-parallelism/ (gradle setup)

Document how EC guards against /status/envoy routes (e.g. /status/envoy/config_dump).

I'd add an issue to think about extracting some parts of SnapshotUpdater and breaking things up. _Originally posted by @slonka in https://github.com/allegro/envoy-control/pull/159#discussion_r486260041_

Open ReliabilityTests classes

1

jUnit uses inheritance to allow reusing tests with different configurations. We should add `open` modifier so they can be run with other configurations.

https://github.com/allegro/envoy-control/pull/23#discussion_r336963891

Author: @lukidzi When we run resilience tests in specific order we might observe that consul is in strange state during test `should be resilient to transient unavailability of one DC`:...