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

Newer API does not support "type" #17

Closed
Rocco83 opened this issue Aug 9, 2017 · 2 comments
Closed

Newer API does not support "type" #17

Rocco83 opened this issue Aug 9, 2017 · 2 comments

Comments

@Rocco83
Copy link
Contributor

Rocco83 commented Aug 9, 2017

As by

Newer API seems to not have the "type" filled.

  • which is the root cause?
  • how can we identify in the newer API the kind of server (Hypervisor, Physical, Virtual)
  • should we block the yaml to run if type is set? -> reason why, we expect that what is written in the yaml is respected
@mossicrue
Copy link
Contributor

With Hosts and Systems API unification, type field is currently missing for Satellite >= 6.2

Currently there is an open BugZilla for fix this problem, BZ1635861, that is linked to Bug 11914 of Foreman

Also, in the BZ above is present a possibile workaround that can't fix at 100% the problem because assign the host type as hypervisor checking if name match '*virt-who*' regexp.

Another possible workaround (that can't fix the problem at 100% too) is to check the combination of this facts fields: facts.virt, facts.virt::host_type and facts.virt::is_guest values.

Depending on the servers type, this values are setted in this way:

Physical Server:

facts.virt::host_type : "Not Applicable"
facts.virt::is_guest  : false
facts.virt            : null

Guest:

facts.virt::host_type : "vmware"/"rhev, kvm"
facts.virt::is_guest  : true
facts.virt            : null

Hypervisor:

facts.virt::host_type : not present
facts.virt::is_guest  : not present
facts.virt            : not present

@evgeni
Copy link
Member

evgeni commented Dec 7, 2018

fixed in 3821d6a

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

No branches or pull requests

3 participants