Skip to content

[jdk] Resolve EA OpenJDK builds to our JDK archive #131237

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

brianseeders
Copy link
Contributor

See ES-12044

  • Use our new GCS-based JDK archive for early access OpenJDK builds
  • Add -Druntime.java.build=<build_number> to the test failure reproduction line when using an early access build

@brianseeders brianseeders marked this pull request as ready for review July 14, 2025 21:15
@brianseeders brianseeders requested a review from a team as a code owner July 14, 2025 21:15
@elasticsearchmachine elasticsearchmachine added the Team:Delivery Meta label for Delivery team label Jul 14, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-delivery (Team:Delivery)

Copy link
Member

@rjernst rjernst left a comment

Choose a reason for hiding this comment

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

LGTM

One nit: should we have a reverse proxy in front of the url so we aren't strongly tied to GCP?

@mark-vieira
Copy link
Contributor

LGTM

One nit: should we have a reverse proxy in front of the url so we aren't strongly tied to GCP?

I don't even think we need that. We could just create a CNAME record for this.

@brianseeders
Copy link
Contributor Author

Yep, just need a CNAME. I'll get one set up for it

@brianseeders
Copy link
Contributor Author

Okay, it's using a custom domain now. The downside is that it's not using SSL/HTTPS. We can set that up, but it will cost more because it requires a load balancer (as well as being a more complex setup) or CDN. I think that a CDN that only caches the build artifact files (and not the JSON manifests I create) would be an effective solution, but I need to look at it some more. For now, this is fine and unblocks any other related work.

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.

4 participants