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

feat(platforms): Adds s390x, ppc64le and armv7 architectures #1701

Merged
merged 4 commits into from
Sep 13, 2023

Conversation

gounthar
Copy link
Contributor

@gounthar gounthar commented Sep 13, 2023

After discussing with @ksalerno99 yesterday during the Platform SIG meeting, I considered that adding support for a few platforms for JDK21 would be beneficial.

I then propose to add "linux/s390x", "linux/ppc64le" and "linux/arm/v7" as new target platforms for our JDK21 preview images.

I have also updated the bookworm tag and the JDK build number, as they are not yet tracked with updatecli.

Testing done

docker buildx bake --file docker-bake.hcl debian_slim_jdk21
docker buildx bake --file docker-bake.hcl debian_jdk21

Submitter checklist

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests - that demonstrates feature works or fixes the issue

@gounthar gounthar requested review from ksalerno99 and a team as code owners September 13, 2023 08:04
@gounthar gounthar changed the title feat(platforms): Adds s390x architecture feat(platforms): Adds s390x, ppc64le and armv7 architectures Sep 13, 2023
--no-man-pages \
--compress=zip-6 \
--output /javaruntime; \
# It is acceptable to have a larger image in arm/v7 (arm 32 bits) environment.
Copy link
Member

Choose a reason for hiding this comment

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

have you checked this is still needed on 21?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes. If you know of a trick with qemu to get rid of that, please let me know.

@timja timja merged commit ca3e63b into jenkinsci:master Sep 13, 2023
@gounthar gounthar deleted the exotic-architectures branch September 13, 2023 16:07
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