Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[OPENJDK-2722] Split the tests inside openjdk_s2i.feature to couple them with appropriate modules #463

Merged
merged 2 commits into from
Mar 20, 2024

Conversation

jhuttana
Copy link
Contributor

@jhuttana jhuttana commented Mar 15, 2024

Could you please review the changes?
JIRA: https://issues.redhat.com/browse/OPENJDK-2722

@jhuttana jhuttana requested a review from jmtd March 15, 2024 18:12
@jhuttana jhuttana changed the title [OPENJDK-2722] Split the tests inside openjdk_s2i.feature to couple them with appropriate modules [OPENJDK-2177] Split the tests inside openjdk_s2i.feature to couple them with appropriate modules Mar 15, 2024
@jmtd
Copy link
Member

jmtd commented Mar 19, 2024

Please change the subject tag for this PR to OPENJDK-2722 and add a link to the JIRA in the description. Thanks!

@ubi8/openjdk-11
@ubi8/openjdk-17
@ubi8/openjdk-21
Feature: JVM Performance tests
Copy link
Member

Choose a reason for hiding this comment

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

This isn't a performance test. it was originally added as a regression test for https://issues.redhat.com/browse/CLOUD-2070, and relates to permissions issues for a running user.
The test also is broken at the moment for unknown reasons. the code it was testing was related to changing permissions on /etc/passwd and has been removed. I think we should just delete this test please.

@jhuttana jhuttana changed the title [OPENJDK-2177] Split the tests inside openjdk_s2i.feature to couple them with appropriate modules [OPENJDK-2722] Split the tests inside openjdk_s2i.feature to couple them with appropriate modules Mar 19, 2024
Signed-off-by: Jayashree Huttanagoudar <[email protected]>
@jmtd jmtd merged commit eeec32a into rh-openjdk:ubi8 Mar 20, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants