Skip to content

[BACKPORT 1.16] skip flaky hotplug tests on x86_64 5.10 hosts#6010

Merged
not4s merged 2 commits into
firecracker-microvm:firecracker-v1.16from
not4s:backport/v1.16-skip-hotplug-and-audit
Jun 30, 2026
Merged

[BACKPORT 1.16] skip flaky hotplug tests on x86_64 5.10 hosts#6010
not4s merged 2 commits into
firecracker-microvm:firecracker-v1.16from
not4s:backport/v1.16-skip-hotplug-and-audit

Conversation

@not4s

@not4s not4s commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Changes

Backport the two merged main commits to unblock v1.16.1 release QA (hotplug tests fail on x86_64 5.10; cargo-audit fails on RUSTSEC-2026-0190) #6007

Reason

...

License Acceptance

By submitting this pull request, I confirm that my contribution is made under
the terms of the Apache 2.0 license. For more information on following Developer
Certificate of Origin and signing off your commits, please check
CONTRIBUTING.md.

PR Checklist

  • I have read and understand CONTRIBUTING.md.
  • I have run tools/devtool checkbuild --all to verify that the PR passes
    build checks on all supported architectures.
  • I have run tools/devtool checkstyle to verify that the PR passes the
    automated style checks.
  • I have described what is done in these changes, why they are needed, and
    how they are solving the problem in a clear and encompassing way.
  • I have updated any relevant documentation (both in code and in the docs)
    in the PR.
  • I have mentioned all user-facing changes in CHANGELOG.md.
  • If a specific issue led to this PR, this PR closes the issue.
  • When making API changes, I have followed the
    Runbook for Firecracker API changes.
  • I have tested all new and changed functionalities in unit tests and/or
    integration tests.
  • I have linked an issue to every new TODO.

  • This functionality cannot be added in rust-vmm.

not4s added 2 commits June 30, 2026 14:45
test_virtio_mem_hotplug_hotunplug and test_memory_hotplug_latency
intermittently fail on x86_64 5.10 hosts, where the GET /hotplug/memory
API call exceeds the maximum duration assertion. Skip both there until
the latency is investigated.

(cherry picked from commit ac7510c)
Signed-off-by: Jay Chung <jaehoc@amazon.com>
Signed-off-by: Jay Chung <jaehoc@amazon.com>
cargo-audit flags RUSTSEC-2026-0190, failing the security audit on all
branches. anyhow is not a direct dependency of any Firecracker crate;
it is only pulled in transitively by build-time WASM tooling
(wit-bindgen), so we are not affected. Ignore it to unblock CI.

(cherry picked from commit 573d249)
Signed-off-by: Jay Chung <jaehoc@amazon.com>
Signed-off-by: Jay Chung <jaehoc@amazon.com>
@not4s not4s added the Status: Awaiting review Indicates that a pull request is ready to be reviewed label Jun 30, 2026
@not4s not4s marked this pull request as ready for review June 30, 2026 14:51
@codecov

codecov Bot commented Jun 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.97%. Comparing base (2038188) to head (c1c6708).
⚠️ Report is 2 commits behind head on firecracker-v1.16.

Additional details and impacted files
@@                Coverage Diff                 @@
##           firecracker-v1.16    #6010   +/-   ##
==================================================
  Coverage              82.97%   82.97%           
==================================================
  Files                    277      277           
  Lines                  30092    30092           
==================================================
  Hits                   24970    24970           
  Misses                  5122     5122           
Flag Coverage Δ
5.10-m5n.metal 83.27% <ø> (-0.01%) ⬇️
5.10-m6a.metal 82.62% <ø> (-0.01%) ⬇️
5.10-m6g.metal 79.91% <ø> (ø)
5.10-m6i.metal 83.27% <ø> (-0.01%) ⬇️
5.10-m7a.metal-48xl 82.62% <ø> (+<0.01%) ⬆️
5.10-m7g.metal 79.91% <ø> (ø)
5.10-m7i.metal-24xl 83.24% <ø> (-0.02%) ⬇️
5.10-m7i.metal-48xl 83.25% <ø> (ø)
5.10-m8g.metal-24xl 79.91% <ø> (-0.01%) ⬇️
5.10-m8g.metal-48xl 79.91% <ø> (ø)
5.10-m8i.metal-48xl 83.25% <ø> (ø)
5.10-m8i.metal-96xl 83.25% <ø> (ø)
6.1-m5n.metal 83.31% <ø> (+<0.01%) ⬆️
6.1-m6a.metal 82.65% <ø> (-0.01%) ⬇️
6.1-m6g.metal 79.91% <ø> (ø)
6.1-m6i.metal 83.30% <ø> (-0.01%) ⬇️
6.1-m7a.metal-48xl 82.63% <ø> (-0.01%) ⬇️
6.1-m7g.metal 79.90% <ø> (-0.01%) ⬇️
6.1-m7i.metal-24xl 83.31% <ø> (-0.01%) ⬇️
6.1-m7i.metal-48xl 83.31% <ø> (-0.01%) ⬇️
6.1-m8g.metal-24xl 79.90% <ø> (-0.01%) ⬇️
6.1-m8g.metal-48xl 79.91% <ø> (-0.01%) ⬇️
6.1-m8i.metal-48xl 83.31% <ø> (-0.01%) ⬇️
6.1-m8i.metal-96xl 83.32% <ø> (ø)
6.18-m5n.metal 83.30% <ø> (ø)
6.18-m6a.metal 82.64% <ø> (-0.01%) ⬇️
6.18-m6g.metal 79.91% <ø> (ø)
6.18-m6i.metal 83.30% <ø> (ø)
6.18-m7a.metal-48xl 82.63% <ø> (ø)
6.18-m7g.metal 79.91% <ø> (-0.01%) ⬇️
6.18-m7i.metal-24xl 83.31% <ø> (-0.02%) ⬇️
6.18-m7i.metal-48xl 83.31% <ø> (-0.01%) ⬇️
6.18-m8g.metal-24xl 79.91% <ø> (-0.01%) ⬇️
6.18-m8g.metal-48xl 79.91% <ø> (ø)
6.18-m8i.metal-48xl 83.32% <ø> (+<0.01%) ⬆️
6.18-m8i.metal-96xl 83.32% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@not4s not4s enabled auto-merge (rebase) June 30, 2026 15:12
@not4s not4s merged commit 52cbb76 into firecracker-microvm:firecracker-v1.16 Jun 30, 2026
8 checks passed
@not4s not4s deleted the backport/v1.16-skip-hotplug-and-audit branch June 30, 2026 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Status: Awaiting review Indicates that a pull request is ready to be reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants