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] UBI8: Suitably rename the test #480

Merged
merged 1 commit into from
Apr 15, 2024

Conversation

jhuttana
Copy link
Contributor

@jhuttana jhuttana commented Apr 10, 2024

Could you please take a look?

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

@jhuttana jhuttana changed the title [OPENJDK-2837] Suitably rename the test [OPENJDK-2837] UBI8: Suitably rename the test Apr 12, 2024
@jmtd
Copy link
Member

jmtd commented Apr 12, 2024

Please include a link to the relevant JIRA issue in all PRs (I've edited it into this one).

The referenced issue is for UBI9 but this issue is targetting UBI8.

What's the motivation for renaming the file? Please include that in the commit message.

Copy link
Member

@jmtd jmtd left a comment

Choose a reason for hiding this comment

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

pls see previous comment

@jhuttana jhuttana changed the title [OPENJDK-2837] UBI8: Suitably rename the test [OPENJDK-2722] UBI8: Suitably rename the test Apr 12, 2024
@jhuttana
Copy link
Contributor Author

jhuttana commented Apr 12, 2024

Please include a link to the relevant JIRA issue in all PRs (I've edited it into this one).

Actually I don't have JIRA tracker specific to this. The suitable one I found is OPENJDK-2722 so adding that to the title.

The referenced issue is for UBI9 but this issue is targetting UBI8.

Sorry that was misleading.

What's the motivation for renaming the file? Please include that in the commit message.

Sure I will add that

The rationale is, these are not openshift tests. Instead checks for some common label and image labels are correct.
So its good to rename the test.

Signed-off-by: Jayashree Huttanagoudar <[email protected]>
@jmtd jmtd merged commit 682cb1e into rh-openjdk:ubi8 Apr 15, 2024
0 of 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