Fetch and use compressed manifest.json #246
Open
shiv-tyagi wants to merge 3 commits into
Open
Conversation
Contributor
|
Using the .xz would be better. Pretty sure Python supports it? |
Member
Author
|
We only generate manifest.json.xz on custom build server today. I would be happy to change it to xz after we do the firmware server side of changes to serve the xz format. |
Fetch the xz-compressed manifest from firmware.ardupilot.org and decompress before writing the on-disk cache. Keep If-None-Match and If-Modified-Since revalidation unchanged.
Switch the default CBS manifest source from the 67 MB JSON file to the 0.7 MB xz file on firmware.ardupilot.org.
Add ManifestClient tests for xz decompression and conditional cache hits. Assert the default manifest URL ends with .xz.
shiv-tyagi
force-pushed
the
feat/manifest-gz
branch
from
July 18, 2026 15:02
4d2d51c to
b48e415
Compare
Member
Author
|
Thanks to @peterbarker we do emit xz on firmware server now. I have changed the PR to use the xz manifest. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
#244 added support to use manifest.json for listing versions on custom build server.
This PR does an enhancement to use the compressed manifest.json instead of downloading the whole big file.
PS.: I somehow was under an impression that we currently do not host gz manifest in firmware server and we need to make the change there first before we start using it here, but looks like we already do. @peterbarker