Skip to content
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

Windows: Add is_virtual and virtual support for virtual machines hosted on Nutanix AHV #2692

Merged
merged 3 commits into from
Apr 11, 2024

Conversation

kenyon
Copy link
Contributor

@kenyon kenyon commented Mar 22, 2024

  • Add is_virtual and virtual support for Nutanix AHV
  • windows/virtualization: reformat MODEL_HASH for readability
  • windows/virtualization_spec: add Nutanix AHV test case

This PR just adds a test to #2589 and is rebased on current main.

@kenyon kenyon requested a review from a team as a code owner March 22, 2024 20:31
@puppetlabs-jenkins
Copy link
Collaborator

Can one of the admins verify this patch?

eman-koch and others added 3 commits April 9, 2024 16:28
…x AHV

Prior to this change facter returned:

    $ facter is_virtual
    false
    $ facter virtual
    physical

After this change facter returns:

    $ facter is_virtual
    true
    $ facter virtual
    ahv

Systems running on Nutanix AHV are correctly recognized now.
This formatting style makes the hash easier to read, and makes diffs
smaller.
Copy link
Contributor

@joshcooper joshcooper left a comment

Choose a reason for hiding this comment

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

Thanks @kenyon and @eman-koch!

@joshcooper joshcooper merged commit 859d888 into puppetlabs:main Apr 11, 2024
17 checks passed
@joshcooper joshcooper added the bug Something isn't working label Apr 11, 2024
@joshcooper
Copy link
Contributor

Marking this as a bug since it used to work in facter 3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants