[OPENJDK-2805] WIP install s2i scripts to libexec

Signed-off-by: Jonathan Dowland <[email protected]>
jmtd · Nov 15, 2024 · 1d42286 · 1d42286
1 parent 179ce26
commit 1d42286
Show file tree

Hide file tree

Showing 14 changed files with 14 additions and 13 deletions.
diff --git a/...aven/s2i/artifacts/usr/local/s2i/assemble → ...en/s2i/artifacts/usr/libexec/s2i/assemble b/...aven/s2i/artifacts/usr/local/s2i/assemble → ...en/s2i/artifacts/usr/libexec/s2i/assemble
diff --git a/...2i/artifacts/usr/local/s2i/save-artifacts → .../artifacts/usr/libexec/s2i/save-artifacts b/...2i/artifacts/usr/local/s2i/save-artifacts → .../artifacts/usr/libexec/s2i/save-artifacts
diff --git a/modules/maven/s2i/configure.sh b/modules/maven/s2i/configure.sh
@@ -8,7 +8,7 @@ ARTIFACTS_DIR=${SCRIPT_DIR}/artifacts
 chown -R $USER:root $SCRIPT_DIR
 chmod -R ug+rwX $SCRIPT_DIR
 chmod ug+x ${ARTIFACTS_DIR}/opt/jboss/container/maven/s2i/*
-chmod ug+x ${ARTIFACTS_DIR}/usr/local/s2i/*
+chmod ug+x ${ARTIFACTS_DIR}/usr/libexec/s2i/*
 
 pushd ${ARTIFACTS_DIR}
 cp -pr * /

diff --git a/modules/run/tests/features/java.runtime.feature b/modules/run/tests/features/java.runtime.feature
@@ -10,5 +10,5 @@ Feature: Openshift OpenJDK Runtime tests (OPENJDK-474)
        | JAVA_DIAGNOSTICS | true   |
     Then container log should contain /deployments/undertow-servlet.jar unique
      And container log should contain -XX:NativeMemoryTracking=summary
-     And file /usr/local/s2i/run should not contain JVM_ARGS
+     And file /usr/libexec/s2i/run should not contain JVM_ARGS
      And container log should not contain unique unique
diff --git a/...s2i/bash/artifacts/usr/local/s2i/assemble → ...i/bash/artifacts/usr/libexec/s2i/assemble b/...s2i/bash/artifacts/usr/local/s2i/assemble → ...i/bash/artifacts/usr/libexec/s2i/assemble
diff --git a/modules/s2i/bash/artifacts/usr/local/s2i/run → ...es/s2i/bash/artifacts/usr/libexec/s2i/run b/modules/s2i/bash/artifacts/usr/local/s2i/run → ...es/s2i/bash/artifacts/usr/libexec/s2i/run
diff --git a/...es/s2i/bash/artifacts/usr/local/s2i/usage → .../s2i/bash/artifacts/usr/libexec/s2i/usage b/...es/s2i/bash/artifacts/usr/local/s2i/usage → .../s2i/bash/artifacts/usr/libexec/s2i/usage
diff --git a/modules/s2i/bash/configure.sh b/modules/s2i/bash/configure.sh
@@ -8,7 +8,7 @@ ARTIFACTS_DIR=${SCRIPT_DIR}/artifacts
 chown -R $USER:root $SCRIPT_DIR
 chmod -R ug+rwX $SCRIPT_DIR
 chmod ug+x ${ARTIFACTS_DIR}/opt/jboss/container/java/s2i/*
-chmod ug+x ${ARTIFACTS_DIR}/usr/local/s2i/*
+chmod ug+x ${ARTIFACTS_DIR}/usr/libexec/s2i/*
 
 pushd ${ARTIFACTS_DIR}
 cp -pr * /

diff --git a/modules/s2i/core/configure.sh b/modules/s2i/core/configure.sh
@@ -13,10 +13,6 @@ pushd ${ARTIFACTS_DIR}
 cp -pr * /
 popd
 
-mkdir -p /usr/local/s2i \
- && chmod 775 /usr/local/s2i \
- && chown -R $USER:root /usr/local/s2i
-
 mkdir -p /deployments \
  && chmod -R "ug+rwX" /deployments \
  && chown -R $USER:root /deployments
diff --git a/modules/s2i/core/module.yaml b/modules/s2i/core/module.yaml
@@ -12,8 +12,9 @@ packages:
   - rsync
 
 labels:
+  # XXX this modules does not install files in this path
 - name: "io.openshift.s2i.scripts-url"
-  value: "image:///usr/local/s2i"
+  value: "image:///usr/libexec/s2i"
 - name: io.openshift.s2i.destination
   value: "/tmp"
 - name: org.jboss.container.deployments-dir
@@ -114,7 +115,8 @@ envs:
 
 run:
   cmd:
-  - "/usr/local/s2i/run"
+    # XXX this script is in a different module
+  - "/usr/libexec/s2i/run"
 
 modules:
   install:

diff --git a/tests/features/imagebasic.feature b/tests/features/imagebasic.feature
@@ -14,4 +14,4 @@ Feature: Tests for all openshift images
   @ubi9/openjdk-21
   Scenario: Check that builder labels are correctly set
     Given image is built
-    Then the image should contain label io.openshift.s2i.scripts-url with value image:///usr/local/s2i
+    Then the image should contain label io.openshift.s2i.scripts-url with value image:///usr/libexec/s2i
diff --git a/ubi9-openjdk-11.yaml b/ubi9-openjdk-11.yaml
@@ -30,8 +30,9 @@ labels:
   value: *version
 
 envs:
+  # XXX should this move to an s2i module?
 - name: PATH
-  value: $PATH:"/usr/local/s2i"
+  value: $PATH:"/usr/libexec/s2i"
 - name: "JBOSS_IMAGE_NAME"
   value: *name
 - name: "JBOSS_IMAGE_VERSION"

diff --git a/ubi9-openjdk-17.yaml b/ubi9-openjdk-17.yaml
@@ -30,8 +30,9 @@ labels:
   value: *version
 
 envs:
+  # XXX should this move to an s2i module?
 - name: PATH
-  value: $PATH:"/usr/local/s2i"
+  value: $PATH:"/usr/libexec/s2i"
 - name: "JBOSS_IMAGE_NAME"
   value: *name
 - name: "JBOSS_IMAGE_VERSION"

diff --git a/ubi9-openjdk-21.yaml b/ubi9-openjdk-21.yaml
@@ -30,8 +30,9 @@ labels:
   value: *version
 
 envs:
+  # XXX should this move to an s2i module?
 - name: PATH
-  value: $PATH:"/usr/local/s2i"
+  value: $PATH:"/usr/libexec/s2i"
 - name: "JBOSS_IMAGE_NAME"
   value: *name
 - name: "JBOSS_IMAGE_VERSION"