Skip to content

Commit b8ffd1d

Browse files
authored
Remove --bootstrap-version from args in envoy test container, (envoyproxy#207)
since this has been removed from the availble command line args. Signed-off-by: rulex123 <[email protected]>
1 parent a8f1f51 commit b8ffd1d

File tree

4 files changed

+4
-15
lines changed

4 files changed

+4
-15
lines changed

server/src/test/java/io/envoyproxy/controlplane/server/EnvoyContainer.java

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ class EnvoyContainer extends GenericContainer<EnvoyContainer> {
1212

1313
private static final Logger LOGGER = LoggerFactory.getLogger(EnvoyContainer.class);
1414

15-
private static final int DEFAULT_API_VERSION = 2;
1615
private static final String CONFIG_DEST = "/etc/envoy/envoy.yaml";
1716
private static final String HOST_IP_SCRIPT = "docker/host_ip.sh";
1817
private static final String HOST_IP_SCRIPT_DEST = "/usr/local/bin/host_ip.sh";
@@ -23,19 +22,13 @@ class EnvoyContainer extends GenericContainer<EnvoyContainer> {
2322

2423
private final String config;
2524
private final Supplier<Integer> controlPlanePortSupplier;
26-
private final int apiVersion;
2725

2826
EnvoyContainer(String config, Supplier<Integer> controlPlanePortSupplier) {
29-
this(config, controlPlanePortSupplier, DEFAULT_API_VERSION);
30-
}
31-
32-
EnvoyContainer(String config, Supplier<Integer> controlPlanePortSupplier, int apiVersion) {
3327
// this version is changed automatically by /tools/update-sha.sh:57
3428
// if you change it make sure to reflect changes there
3529
super("envoyproxy/envoy-alpine-dev:bef18019d8fc33a4ed6aca3679aff2100241ac5e");
3630
this.config = config;
3731
this.controlPlanePortSupplier = controlPlanePortSupplier;
38-
this.apiVersion = apiVersion;
3932
}
4033

4134
@Override
@@ -50,8 +43,7 @@ protected void configure() {
5043
"/bin/sh", "/usr/local/bin/launch_envoy.sh",
5144
Integer.toString(controlPlanePortSupplier.get()),
5245
CONFIG_DEST,
53-
"-l", "debug",
54-
"--bootstrap-version", Integer.toString(apiVersion)
46+
"-l", "debug"
5547
);
5648

5749
getExposedPorts().add(0, ADMIN_PORT);

server/src/test/java/io/envoyproxy/controlplane/server/V3DiscoveryServerAdsIT.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@ public class V3DiscoveryServerAdsIT {
2323
private static final String CONFIG = "envoy/ads.v3.config.yaml";
2424
private static final String GROUP = "key";
2525
private static final Integer LISTENER_PORT = 10000;
26-
private static final int API_VERSION = 3;
2726

2827
private static final CountDownLatch onStreamOpenLatch = new CountDownLatch(1);
2928
private static final CountDownLatch onStreamRequestLatch = new CountDownLatch(1);
@@ -68,7 +67,7 @@ protected void configureServerBuilder(NettyServerBuilder builder) {
6867

6968
private static final Network NETWORK = Network.newNetwork();
7069

71-
private static final EnvoyContainer ENVOY = new EnvoyContainer(CONFIG, () -> ADS.getServer().getPort(), API_VERSION)
70+
private static final EnvoyContainer ENVOY = new EnvoyContainer(CONFIG, () -> ADS.getServer().getPort())
7271
.withExposedPorts(LISTENER_PORT)
7372
.withNetwork(NETWORK);
7473

server/src/test/java/io/envoyproxy/controlplane/server/V3DiscoveryServerAdsWarmingClusterIT.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,6 @@ public class V3DiscoveryServerAdsWarmingClusterIT {
3838
private static final String CONFIG = "envoy/ads.v3.config.yaml";
3939
private static final String GROUP = "key";
4040
private static final Integer LISTENER_PORT = 10000;
41-
private static final int API_VERSION = 3;
4241
private static final SimpleCache<String> cache = new SimpleCache<>(new NodeGroup<String>() {
4342
@Override public String hash(Node node) {
4443
throw new IllegalStateException("Unexpected v2 request in v3 test");
@@ -104,7 +103,7 @@ public void onV3StreamResponse(long streamId, DiscoveryRequest request,
104103

105104
private static final Network NETWORK = Network.newNetwork();
106105

107-
private static final EnvoyContainer ENVOY = new EnvoyContainer(CONFIG, () -> ADS.getServer().getPort(), API_VERSION)
106+
private static final EnvoyContainer ENVOY = new EnvoyContainer(CONFIG, () -> ADS.getServer().getPort())
108107
.withExposedPorts(LISTENER_PORT)
109108
.withNetwork(NETWORK);
110109

server/src/test/java/io/envoyproxy/controlplane/server/V3DiscoveryServerXdsIT.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@ public class V3DiscoveryServerXdsIT {
2323
private static final String CONFIG = "envoy/xds.v3.config.yaml";
2424
private static final String GROUP = "key";
2525
private static final Integer LISTENER_PORT = 10000;
26-
private static final int API_VERSION = 3;
2726

2827
private static final CountDownLatch onStreamOpenLatch = new CountDownLatch(2);
2928
private static final CountDownLatch onStreamRequestLatch = new CountDownLatch(2);
@@ -69,7 +68,7 @@ protected void configureServerBuilder(NettyServerBuilder builder) {
6968

7069
private static final Network NETWORK = Network.newNetwork();
7170

72-
private static final EnvoyContainer ENVOY = new EnvoyContainer(CONFIG, () -> XDS.getServer().getPort(), API_VERSION)
71+
private static final EnvoyContainer ENVOY = new EnvoyContainer(CONFIG, () -> XDS.getServer().getPort())
7372
.withExposedPorts(LISTENER_PORT)
7473
.withNetwork(NETWORK);
7574

0 commit comments

Comments
 (0)