Skip to content

Conversation

@zulinx86
Copy link
Contributor

@zulinx86 zulinx86 commented Jun 14, 2023

Changes

Backports PR #3813 and #3816 to v1.4

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

  • [ ] If a specific issue led to this PR, this PR closes the issue.
  • The description of changes is clear and encompassing.
  • [ ] Any required documentation changes (code and docs) are included in this PR.
  • [ ] API changes follow the Runbook for Firecracker API changes.
  • [ ] User-facing changes are mentioned in CHANGELOG.md.
  • All added/changed functionality is tested.
  • [ ] New TODOs link to an issue.
  • Commits meet contribution quality standards.

  • This functionality cannot be added in rust-vmm.

@zulinx86 zulinx86 self-assigned this Jun 14, 2023
@zulinx86 zulinx86 requested review from kalyazin and pb8o June 14, 2023 08:22
kalyazin
kalyazin previously approved these changes Jun 14, 2023
@zulinx86 zulinx86 changed the title chore(tool): Remove message for debug Backport PR #3816 Jun 14, 2023
roypat
roypat previously approved these changes Jun 14, 2023
zulinx86 added 4 commits June 14, 2023 08:53
The host information (kernel version and CPU info) always has to be
displayed even when the test is aborted unexpectedly in the middle.

Signed-off-by: Takahiro Itazuri <itazur@amazon.com>
Adds a missing dash "-" to the fingerprint comparison command of the CPU
template helper tool.

Signed-off-by: Takahiro Itazuri <itazur@amazon.com>
Regathers the fingerprint baseline files where some kernel versions,
microcode versions and guest CPU configs are updated.

Signed-off-by: Takahiro Itazuri <itazur@amazon.com>
Backports PR firecracker-microvm#3816.

Removes a message used for debug in the fingerprint dump command of the
CPU template helper tool.

```
$ sudo cpu-template-helper fingerprint dump --config config.json
Some(0)
```

Signed-off-by: Takahiro Itazuri <itazur@amazon.com>
@zulinx86 zulinx86 dismissed stale reviews from roypat and kalyazin via fc7536f June 14, 2023 08:53
@zulinx86 zulinx86 changed the title Backport PR #3816 Backport PR #3813 and #3816 Jun 14, 2023
@zulinx86 zulinx86 changed the title Backport PR #3813 and #3816 Backport PR #3813 and #3816 to v1.4 Jun 14, 2023
@zulinx86 zulinx86 merged commit 34747d2 into firecracker-microvm:firecracker-v1.4 Jun 14, 2023
@zulinx86 zulinx86 deleted the backport-v1.4 branch June 14, 2023 09:32
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