Skip to content

Conversation

@ValentaTomas
Copy link
Member

@ValentaTomas ValentaTomas commented Dec 11, 2025

Note

Build pipeline now uses a custom Firecracker repo, parses versions into a matrix, builds in parallel, checks upstream CI, and conditionally uploads to GCS and creates per-version releases.

  • CI/Workflows:
    • Add reusable github/workflows/build.yml to build a single version/hash/version_name and upload artifact.
    • Revamp fc-versions.yml:
      • Add concurrency control.
      • Prepare matrix from firecracker_versions.txt via script; build using reusable workflow.
      • Check upstream CI for each commit; gate publishing on success and main.
      • Collect artifacts, upload to GCS, and create tagged GitHub releases per version_name.
  • Build:
    • build.sh now clones e2b-dev/firecracker, validates tag+hash, builds firecracker binary, and outputs to builds/<version_name>/.
  • Scripts:
    • scripts/parse-versions-with-hash.sh resolves versions to full hashes and normalized version_name JSON.
    • scripts/check-fc-ci.sh inspects commit statuses/check-runs via GitHub API and reports pass/fail.
  • Docs:
    • Update README.md to describe version parsing, matrix builds, CI gating, GCS upload, and releases; rename project to fc-versions.

Written by Cursor Bugbot for commit d7b3db0. This will update automatically on new commits. Configure here.

@ValentaTomas
Copy link
Member Author

ValentaTomas commented Dec 11, 2025

Not sure if FC tests should be run here or in the FC repo—in the FC repo, there might not be a way to enforce this as we will have multiple branches for versions, that won't be merged.

But somehow requiring that all checks for the specific hash passed might work.

EDIT: Adding CI status checks here and triggering tests on each push to the FC repo.

@ValentaTomas ValentaTomas requested a review from jakubno December 11, 2025 20:41
@tomassrnka
Copy link
Member

@ValentaTomas yes, maybe we should run all FC checks with the build, i will revert the question - why not here? we want to have stable / tested version done, maybe we should actually add it here in next PR.

@ValentaTomas
Copy link
Member Author

@jakubno lgty?

@ValentaTomas ValentaTomas merged commit eeb5244 into main Dec 18, 2025
6 checks passed
@ValentaTomas ValentaTomas deleted the refactor branch December 18, 2025 10:25
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.

4 participants