-
Notifications
You must be signed in to change notification settings - Fork 93
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
Migrate from Zulu OpenJDK Discovery API to Azul Metadata API #514
Comments
For example, here's the discovery API for the JDK version 17 (added newlines for clarity):
Here's the migrated metadata API after being refactored to have the same order in the swagger docs:
Here's the diff after the refactor: - https://api.azul.com/zulu/download/community/v1.0/bundles/latest/\
+ https://api.azul.com/metadata/v1/zulu/packages/\
- ?jdk_version=17\
+ ?java_version=17\
&os=windows\
&arch=x86\
- &ext=zip\
+ &archive_type=zip\
- &javafx=false\
+ &javafx_bundled=false\
+ &crac_supported=false\
+ &release_type=PSU\
+ &latest=true\
+ &page_size=2
- &features= Note that we added CRaC, release type, and latest parameters. I removed the feature parameter ( CRaC support has their own docs here https://docs.azul.com/core/crac/crac-introduction, Release type parameter is, quote from the API:
We use PSU to match latest available versions on the download screen at https://www.azul.com/downloads. I added latest and page size parameter to only list the first x86 and x64, regardless of CRS support. Page size parameter value should be 1 starting from Java version 21 per JEP 449. |
Can be included using |
I have researched the documentation about CRaC and CRS support. CRS (Connected Runtime Service) is not available to 32-bit architecture after October 2023. CRaC (Coordinated Restore at Checkpoint) is only available on 64-bit architecture beginning from Zulu version |
I made a regex to match and substitute the urls here https://regex101.com/r/5ZwIZG/5. |
Using the regex from <ScoopInstaller#514 (comment)>. TODO: fit with json path and regex.
Using the regex from <ScoopInstaller#514 (comment)>. TODO: fit with json path and regex.
Bug Report
Package Name:
zulu*
Current Behaviour
Currently, Azul Zulu packages are using the discovery API (https://api.azul.com/zulu/download/).
Expected Behaviour
Azul Zulu packages should utilize the new metadata API (https://api.azul.com/metadata/v1/)
Additional context/output
In accordance with https://docs.azul.com/core/metadata-api-migration.
Azul now support CRaC on their JDKs beginning on version 17 by default, currently
zulu17-jdk
andzulu-jdk
packages cannot update correctly due to the incorrect regex trying to match the default CRaC (*-ca-crac-*
) support.It's also worth mentioning that the LTS version of 21 is missing from the bucket (#509).
Zulu packages beginning from 21 should drop support for 32-bit architecture in accordance with JEP 449.
Possible Solution
zulu*
packages'checkver
andautoupdate
links with the new metadata API.zulu-*
andzulufx-*
as per JEP 449.We should discuss about making a separate package for the CRaC support.
The text was updated successfully, but these errors were encountered: