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

Need to add macOS Arm64 spec #2501

Closed
gdams opened this issue Apr 13, 2021 · 10 comments · Fixed by #2509
Closed

Need to add macOS Arm64 spec #2501

gdams opened this issue Apr 13, 2021 · 10 comments · Fixed by #2509

Comments

@gdams
Copy link
Member

gdams commented Apr 13, 2021

As shown in https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk/job/jdk-mac-arm64-hotspot_SmokeTests/8/console

java.lang.AssertionError: Cannot find key PLATFORM: arm64_mac in PLATFORM_MAP
@gdams gdams added the mac label Apr 13, 2021
@gdams
Copy link
Member Author

gdams commented Apr 13, 2021

I've temporarily added sw.os.osx hw.arch.arm64 ci.role.test to the build machines until we have more hardware

@smlambert
Copy link
Contributor

https://ci.adoptopenjdk.net/view/Test_grinder/job/Grinder/55/console - will have to add some prints in the setup stage to understand why it (silently) fails...

@smlambert
Copy link
Contributor

smlambert commented Apr 14, 2021

Description
How to address this issue is to add a section in the PLATFORM_MAP in the openjdk_tests file for this platform,

   'arm64_mac' : [
        'SPEC' : 'arm64_mac',
        'LABEL' : 'ci.role.test&&sw.os.osx&&hw.arch.arm64',
    ],

📋 Step by Step

To solve this issue and contribute a fix you should check the following step-by-step list. A more detailed documentation of the workflow can be found here. Note: that you do not need to add yourself to a Contributors.md file as outlined in the detailed instructions.

  • Claim this issue: Comment below.
  • Fork the repository in github by simply clicking the 'fork' button.
  • Check out the forked repository
  • Create a feature branch for the issue. We do not have any naming definition for branches.
  • Commit your changes (to the openjdk_tests file as described above).
  • Start a Pull Request.
  • Done 👍 Ask in comments for a review :)
  • If the reviewer find some missing pieces or a problem they will start a discussion with you and describe the next steps how the problem can be solved.
  • You did it 🎉 We will merge the fix in the master branch.
  • Thanks, thanks, thanks for being part of this project as an open source contributor ❤️

@gdams
Copy link
Member Author

gdams commented Apr 14, 2021

@ShelleyLambert looks like you were pointing it at the wrong job number, trying again here: https://ci.adoptopenjdk.net/view/Test_grinder/job/Grinder/61/console

@gdams
Copy link
Member Author

gdams commented Apr 14, 2021

Another test with JDK_REPO set correctly: https://ci.adoptopenjdk.net/view/Test_grinder/job/Grinder/63/console

@smlambert
Copy link
Contributor

Great, Grinder/63 passed. Though we still need to merge in a PR - those Grinders were running off of my branch, no PR created yet.

@smlambert smlambert reopened this Apr 14, 2021
@gdams
Copy link
Member Author

gdams commented Apr 14, 2021

PR created here #2509

@gdams gdams closed this as completed Apr 14, 2021
@smlambert
Copy link
Contributor

ha, ok thanks for the fix @gdams, but now you owe me a "good first issue" that can be offered to the 50+ Outreachy candidates. ;)

@sophia-guo
Copy link
Contributor

sophia-guo commented Apr 14, 2021

Talked with @renfeiw there are a couple things need to double check so better to keep this issue open.

System.getProperty('os.arch') get x86_64 instead of os.arch=arm64 . For other arm64 the os.arch is aarch64. @gdams what is expected property of os.arch for this macOS Arm64?

With AUTODETECT on SPEC will be set as osx_x86-64 same to macos linux. That is all functional tests will take maxos Arm64 same as maxos linux. With AUTODETECT off SPEC will be set as arm64_mac, which hasn't been added to TKG. That probably will not work well with functional test.

@gdams
Copy link
Member Author

gdams commented Apr 16, 2021

@sophia-guo I was stupidely cross-compiling the x64 version.

An updated build shows the correct output:

System.getProperty('os.arch')=aarch64

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants