Skip to content

Conversation

@mismithhisler
Copy link
Member

@mismithhisler mismithhisler commented Dec 1, 2025

Description

Links

Code PR: hashicorp/nomad#27179

Contributor checklists

Review urgency:

  • ASAP: Bug fixes, broken content, imminent releases
  • 3 days: Small changes, easy reviews
  • 1 week: Default expectation
  • Best effort: No urgency

Pull request:

  • Verify that the PR is set to merge into the correct base branch
  • Verify that all status checks passed
  • Verify that the preview environment deployed successfully
  • Add additional reviewers if they are not part of assigned groups

Content:

  • I added redirects for any moved or removed pages
  • I followed the Education style guide
  • I looked at the local or Vercel build to make sure the content rendered correctly

Reviewer checklist

  • This PR is set to merge into the correct base branch.
  • The content does not contain technical inaccuracies.
  • The content follows the Education content and style guides.
  • I have verified and tested changes to instructions for end users.

@mismithhisler mismithhisler self-assigned this Dec 1, 2025
@mismithhisler mismithhisler requested review from a team as code owners December 1, 2025 21:36
@mismithhisler mismithhisler requested a review from aimeeu December 1, 2025 21:36
@github-actions
Copy link
Contributor

github-actions bot commented Dec 1, 2025

Vercel Previews Deployed

Name Status Preview Updated (UTC)
Dev Portal ✅ Ready (Inspect) Visit Preview Tue Dec 2 22:26:36 UTC 2025
Unified Docs API ✅ Ready (Inspect) Visit Preview Tue Dec 2 22:21:07 UTC 2025

@github-actions
Copy link
Contributor

github-actions bot commented Dec 1, 2025

Broken Link Checker

No broken links found! 🎉

@aimeeu aimeeu changed the title Nomad Docs Nomad Docs: Add new QEMU driver parameters Dec 2, 2025
Copy link
Contributor

@aimeeu aimeeu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! I left a few style guide suggestions.

Comment on lines 17 to 23
In Nomad 1.11.1, `emulator` and `machine_type` were added to the task config,
and `fingerprint_emulator` was added to the driver config. These default to the
previously used values of `qemu-system-x86_64`, and `pc`. Previously, when using
the `kvm` accelerator, the machine type `host` was forced. This is no longer true,
the value for `machine_type` will be used. Additionally, if using resources.cores,
with the `kvm` accelerator, the `-smp` was hardcoded to that number of cores. This
is now only done if the user has not specified a custom -smp flag.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
In Nomad 1.11.1, `emulator` and `machine_type` were added to the task config,
and `fingerprint_emulator` was added to the driver config. These default to the
previously used values of `qemu-system-x86_64`, and `pc`. Previously, when using
the `kvm` accelerator, the machine type `host` was forced. This is no longer true,
the value for `machine_type` will be used. Additionally, if using resources.cores,
with the `kvm` accelerator, the `-smp` was hardcoded to that number of cores. This
is now only done if the user has not specified a custom -smp flag.
In Nomad 1.11.1, we added `emulator` and `machine_type` to the task config and
`fingerprint_emulator` to the driver config. These default to the previously
used values of `qemu-system-x86_64` and `pc`. Previously, when using the `kvm`
accelerator, Nomad forced the machine type `host`. This is no longer true. Nomad
now uses the value for `machine_type` . Additionally, if using `resources.cores`
with the `kvm` accelerator, the `-smp` used that number of cores. Nomad now only
uses the `resources.cores` value when you have not specified a custom `-smp`
flag.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I had to refactor this paragraph a good bit after refactoring my PR

@aimeeu aimeeu added the Nomad-Minor PR for Nomad minor release label Dec 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Nomad Nomad-Minor PR for Nomad minor release Runtime

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants