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

Warn on java over max supported version #3429

Open
wants to merge 4 commits into
base: v2.x/staging
Choose a base branch
from

Conversation

1000TurquoisePogs
Copy link
Member

This PR adds a simple test on java version.
Currently we error out if a java under our supported version is used.
But, if someone uses a java version newer than we know works, the code does run and then could have some error later, without people having any clue java could be the reason for the error.
So, this code just adds a warning if we detect java is newer than what we know works. The execution still continues, but now people will be warned.

Warn on java over 8

Signed-off-by: 1000TurquoisePogs <[email protected]>
Signed-off-by: 1000TurquoisePogs <[email protected]>
Signed-off-by: 1000TurquoisePogs <[email protected]>
Signed-off-by: 1000TurquoisePogs <[email protected]>
@github-actions
Copy link

github-actions bot commented May 23, 2023

PAX build 3128 SUCCEEDED.
Link to workflow run: https://github.com/zowe/zowe-install-packaging/actions/runs/5058882714

@zowe-robot zowe-robot temporarily deployed to marist-zzow02 May 23, 2023 15:05 — with GitHub Actions Inactive
@github-actions
Copy link

github-actions bot commented May 23, 2023

Test workflow 2643 is started.
Running install test: Convenience Pax
The zowe artifact being used by this test workflow: libs-snapshot-local/org/zowe/2.9.0-PR-3429/zowe-2.9.0-pr-3429-3128-20230523150412.pax
Running on machine: zzow02
Result: SUCCESS
Link to workflow run: https://github.com/zowe/zowe-install-packaging/actions/runs/5058948246

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