Skip to content

Conversation

jmtd
Copy link
Member

@jmtd jmtd commented Feb 18, 2025

This is the conventional path for the scripts.

https://issues.redhat.com/browse/OPENJDK-3653

@jmtd jmtd requested a review from sefroberg February 18, 2025 14:09
@sefroberg
Copy link

@jmtd I need to review the existing regression tests to see if they check for the s2i path.

@sefroberg
Copy link

@jmtd Yes, the OCP testing checks for this. So we will need to make sure to update this for the Rhel 10 images.
https://github.com/rh-openjdk/jdkContainerOcpTests/blob/main/test-openjdk/src/test/java/com/redhat/qe/openjdk/image/DockerImageTest.java#L93

This is the conventional path for the scripts.

Signed-off-by: Jonathan Dowland <[email protected]>
Copy link

@sefroberg sefroberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@jmtd jmtd merged commit 8b9b410 into rh-openjdk:ubi10 Feb 25, 2025
1 of 2 checks passed
@jmtd jmtd deleted the 3653-mv-s2i-scripts branch February 25, 2025 16:11
@jmtd jmtd added the ubi10 RHEL UBI10 label Aug 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ubi10 RHEL UBI10
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants