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

Remove obsolete test paths for non-Adopt test builds #2638

Merged

Conversation

AdamBrousseau
Copy link
Contributor

  • We no longer run JDK9 so we can remove this case.
    If we need to run 9 at some point in the future, test
    jobs will need to be created using the Adopt test scripts.
  • Windows is now supported using the Adopt test scripts

[skip ci]
Issue #1450 #1779 adoptium/aqa-tests#506

Signed-off-by: Adam Brousseau adam.brousseau88@gmail.com

- We no longer run JDK9 so we can remove this case.
  If we need to run 9 at some point in the future, test
  jobs will need to be created using the Adopt test scripts.
- Windows is now supported using the Adopt test scripts

[skip ci]
Issue eclipse-openj9#1450 eclipse-openj9#1779 adoptium/aqa-tests#506

Signed-off-by: Adam Brousseau <adam.brousseau88@gmail.com>
@AdamBrousseau
Copy link
Contributor Author

@llxia for review
fyi @vsebe @jdekonin

@AdamBrousseau
Copy link
Contributor Author

Confirmed all the test jobs for Windows 8 (64&32), 10, 11 are created across functional & system cross sanity & extended. Still missing 9 (unsupported) and "next" (WIP)

@llxia
Copy link
Contributor

llxia commented Aug 17, 2018

Did not know you are also working on this, but we cannot switch wins atm due to system test failure. For detail, please see #2037

@llxia
Copy link
Contributor

llxia commented Aug 17, 2018

Also, win32 script still references wrong label, which needs to be fixed.

https://github.com/AdoptOpenJDK/openjdk-tests/blob/2424010ee6835e99b0e9066602929dd2740feb54/buildenv/jenkins/openjdk8_x86-32_windows#L6

Issue opened for this adoptium/aqa-tests#517

@llxia
Copy link
Contributor

llxia commented Aug 22, 2018

Systemtest further failed during native test compilation on win: adoptium/openj9-systemtest#38

@llxia
Copy link
Contributor

llxia commented Aug 23, 2018

system test failure: adoptium/aqa-systemtest#165

@llxia
Copy link
Contributor

llxia commented Aug 29, 2018

Going to merge this PR, so we can completely switch to use AdoptOpenJDK test pipeline.
As discussed, Adam will change the systemtest wins jobs to do nothing (print helloworld) for now. Once the systemtest issue on wins is fixed, we can re-enable systemtest on wins.

@llxia llxia merged commit e3745a6 into eclipse-openj9:master Aug 29, 2018
AdamBrousseau added a commit to AdamBrousseau/openj9 that referenced this pull request Jun 11, 2019
- We no longer run our own testing so this
  code is no longer needed.
- Remove PullRequest pipelines as they are no longer in use.
- Combine 'build' and 'pullRequest' variable setup cases
  as they are identical.
- Also remove temp reverse spec map used prior to eclipse-openj9#2836.

Related eclipse-openj9#1450
Depends eclipse-openj9#2728 eclipse-openj9#2638 eclipse-openj9#2467 eclipse-openj9#2272 eclipse-openj9#2836 eclipse-openj9#4443
[skip ci]
Signed-off-by: Adam Brousseau <adam.brousseau88@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants