Revert "Fix passing positional args to ES in Docker (#88502)"

This reverts commit 3776923.
elastic · Jul 14, 2022 · 0f052d1 · 0f052d1
1 parent 5f5a48c
commit 0f052d1
Show file tree

Hide file tree

Showing 4 changed files with 2 additions and 27 deletions.
diff --git a/distribution/docker/src/docker/bin/docker-entrypoint.sh b/distribution/docker/src/docker/bin/docker-entrypoint.sh
@@ -81,4 +81,4 @@ fi
 
 # Signal forwarding and child reaping is handled by `tini`, which is the
 # actual entrypoint of the container
-exec /usr/share/elasticsearch/bin/elasticsearch "$@" $POSITIONAL_PARAMETERS <<<"$KEYSTORE_PASSWORD"
+exec /usr/share/elasticsearch/bin/elasticsearch $POSITIONAL_PARAMETERS <<<"$KEYSTORE_PASSWORD"
diff --git a/docs/changelog/88502.yaml b/docs/changelog/88502.yaml
diff --git a/qa/os/src/test/java/org/elasticsearch/packaging/test/DockerTests.java b/qa/os/src/test/java/org/elasticsearch/packaging/test/DockerTests.java
@@ -1096,18 +1096,6 @@ public void test170DefaultShellIsBash() {
         }
     }
 
-    /**
-     * Ensure that it is possible to apply CLI options when running the image.
-     */
-    public void test171AdditionalCliOptionsAreForwarded() throws Exception {
-        runContainer(distribution(), builder().runArgs("bin/elasticsearch", "-Ecluster.name=kimchy").envVar("ELASTIC_PASSWORD", PASSWORD));
-        waitForElasticsearch(installation, "elastic", PASSWORD);
-
-        final JsonNode node = getJson("/", "elastic", PASSWORD, ServerUtils.getCaCert(installation));
-
-        assertThat(node.get("cluster_name").textValue(), equalTo("kimchy"));
-    }
-
     /**
      * Check that the UBI images has the correct license information in the correct place.
      */
@@ -1205,7 +1193,7 @@ private List<String> listPlugins() {
     /**
      * Check that readiness listener works
      */
-    public void test500Readiness() throws Exception {
+    public void testReadiness001() throws Exception {
         assertFalse(readinessProbe(9399));
         // Disabling security so we wait for green
         installation = runContainer(

diff --git a/qa/os/src/test/java/org/elasticsearch/packaging/util/docker/DockerRun.java b/qa/os/src/test/java/org/elasticsearch/packaging/util/docker/DockerRun.java
@@ -34,7 +34,6 @@ public class DockerRun {
     private Integer uid;
     private Integer gid;
     private final List<String> extraArgs = new ArrayList<>();
-    private final List<String> runArgs = new ArrayList<>();
     private String memory = "2g"; // default to 2g memory limit
 
     private DockerRun() {}
@@ -96,11 +95,6 @@ public DockerRun extraArgs(String... args) {
         return this;
     }
 
-    public DockerRun runArgs(String... args) {
-        Collections.addAll(this.runArgs, args);
-        return this;
-    }
-
     String build() {
         final List<String> cmd = new ArrayList<>();
 
@@ -150,8 +144,6 @@ String build() {
         // Image name
         cmd.add(getImageName(distribution));
 
-        cmd.addAll(this.runArgs);
-
         return String.join(" ", cmd);
     }