Skip to content

ci: harden firecracker workflow diagnostics - #956

Closed
AlexMikhalev wants to merge 1 commit into
task/2668-terraphim-lsp-foundationfrom
release-guardian/firecracker-ci-diagnostics
Closed

ci: harden firecracker workflow diagnostics#956
AlexMikhalev wants to merge 1 commit into
task/2668-terraphim-lsp-foundationfrom
release-guardian/firecracker-ci-diagnostics

Conversation

@AlexMikhalev

Copy link
Copy Markdown
Contributor

Summary

  • log Firecracker VM create HTTP status/body and curl transport failures before JSON parsing
  • clean up Create VM temp response files reliably
  • run cargo nextest directly instead of through rch exec so the filter expression quoting survives

Evidence

  • yaml.safe_load parsed .github/workflows/ci-firecracker.yml
  • extracted Create VM shell block passed bash -n
  • simulated VM create success (201), HTTP 500, and curl transport failure paths
  • independent review passed: no security concerns, no logic errors

Tracks Gitea #3173 and unblocks diagnosis of the shared Firecracker CI failures affecting PR #953/#954/#955.

Improve Firecracker VM creation diagnostics by logging HTTP status/body and handling curl transport errors before JSON parsing.

Run cargo nextest directly instead of through rch exec because rch's non-compilation path loses shell quoting around nextest filter expressions.

Verification:
- workflow YAML parsed
- extracted Create VM shell passed bash -n
- simulated 201, HTTP 500, and curl transport failure paths
- independent review passed
@AlexMikhalev
AlexMikhalev force-pushed the release-guardian/firecracker-ci-diagnostics branch from 7c5c6d2 to 7e2ed4a Compare August 7, 2026 18:44
@AlexMikhalev

Copy link
Copy Markdown
Contributor Author

Closing as superseded: the complete local CI unblocker set has been collapsed into #955 to avoid circular failing checks between the RLM format/MSRV branch and the Firecracker workflow branch.

@AlexMikhalev

Copy link
Copy Markdown
Contributor Author

Superseded by #955.

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.

1 participant