Skip to content

chore: update patch series to v17 #5340

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

Merged
merged 1 commit into from
Jul 31, 2025

Conversation

roypat
Copy link
Contributor

@roypat roypat commented Jul 31, 2025

Update the mmap support patch series to v17, and update the direct map removal series to apply on top of v17. Rebase all the other series.

v17 is based on kvm-x86/next, which is hosted separately from the kvm tree.

Changes

...

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.

Copy link

codecov bot commented Jul 31, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.05%. Comparing base (c09ce81) to head (a97d054).
⚠️ Report is 1 commits behind head on feature/secret-hiding.

⚠️ Current head a97d054 differs from pull request most recent head f11e341

Please upload reports for the commit f11e341 to get more accurate results.

Additional details and impacted files
@@                    Coverage Diff                    @@
##           feature/secret-hiding    #5340      +/-   ##
=========================================================
+ Coverage                  81.90%   82.05%   +0.14%     
=========================================================
  Files                        250      219      -31     
  Lines                      27612    24849    -2763     
=========================================================
- Hits                       22615    20389    -2226     
+ Misses                      4997     4460     -537     
Flag Coverage Δ
5.10-c5n.metal ?
5.10-m5n.metal ?
5.10-m6a.metal ?
5.10-m6g.metal ?
5.10-m6i.metal ?
5.10-m7a.metal-48xl ?
5.10-m7g.metal ?
5.10-m7i.metal-24xl 82.05% <ø> (+<0.01%) ⬆️
5.10-m7i.metal-48xl ?
5.10-m8g.metal-24xl ?
5.10-m8g.metal-48xl ?
6.1-c5n.metal ?
6.1-m5n.metal ?
6.1-m6a.metal ?
6.1-m6g.metal ?
6.1-m6i.metal ?
6.1-m7a.metal-48xl ?
6.1-m7g.metal ?
6.1-m7i.metal-24xl ?
6.1-m7i.metal-48xl ?
6.1-m8g.metal-24xl ?
6.1-m8g.metal-48xl ?
6.16-c5n.metal ?
6.16-m5n.metal ?
6.16-m6a.metal ?
6.16-m6g.metal ?
6.16-m6i.metal ?
6.16-m7a.metal-48xl ?
6.16-m7g.metal ?
6.16-m7i.metal-24xl ?
6.16-m7i.metal-48xl ?
6.16-m8g.metal-24xl ?
6.16-m8g.metal-48xl ?

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

☔ View full report in Codecov by Sentry.
📢 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.

kalyazin
kalyazin previously approved these changes Jul 31, 2025
Update the mmap support patch series to v17, and update the direct map
removal series to apply on top of v17. Rebase all the other series.

Signed-off-by: Patrick Roy <[email protected]>
@roypat roypat enabled auto-merge (rebase) July 31, 2025 15:18
@roypat roypat requested a review from JackThomson2 July 31, 2025 15:18
@roypat roypat merged commit e296ba2 into firecracker-microvm:feature/secret-hiding Jul 31, 2025
5 checks passed
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.

3 participants