Project

General

Profile

Bug #35836

BIOS UUID on host detail page is incorrect

Added by Ewoud Kohl van Wijngaarden about 2 months ago. Updated about 2 months ago.

Status:
Closed
Priority:
Normal
Category:
Inventory
Target version:
-
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

In the system properties card on the host detail page there is a field BIOS UUID. If this really was the BIOS UUID I'd expect it to be on the BIOS card instead.

However, the UUID field doesn't come from the BIOS. For managed hosts it's the UUID in the compute resource (if any). It doesn't exist for unmanaged hosts.

I'd suggest to not show it at all and treat it as a system internal property. Instead, show the VM as is done in #35649.


Related issues

Related to Foreman - Feature #34901: Details tab - System properties card updatesClosed
Related to Foreman - Feature #35649: Details tab - Virtualization cardClosed

Associated revisions

Revision 47800cd6 (diff)
Added by Ewoud Kohl van Wijngaarden about 2 months ago

Fixes #35836 - Drop "BIOS" UUID field

The host.uuid property is actually the virtualization UUID (for managed
hosts). This has nothing to do with the BIOS and is only confusing.

The UUID should also be considered an internal property and users
shouldn't be bothered by this. Instead, they should see a card showing
the virtual machine's details.

Fixes: 4e7ac8c11cb1f936fc6103bd837492e152ccfe65

History

#1 Updated by Ewoud Kohl van Wijngaarden about 2 months ago

  • Related to Feature #34901: Details tab - System properties card updates added

#2 Updated by Ewoud Kohl van Wijngaarden about 2 months ago

#3 Updated by Ewoud Kohl van Wijngaarden about 2 months ago

  • Description updated (diff)

#4 Updated by The Foreman Bot about 2 months ago

  • Assignee set to Ewoud Kohl van Wijngaarden
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/9540 added

#5 Updated by The Foreman Bot about 2 months ago

  • Fixed in Releases 3.6.0 added

#6 Updated by Ewoud Kohl van Wijngaarden about 2 months ago

  • Status changed from Ready For Testing to Closed

#7 Updated by The Foreman Bot about 2 months ago

  • Pull request https://github.com/theforeman/foreman/pull/9552 added

#8 Updated by The Foreman Bot about 2 months ago

  • Fixed in Releases 3.5.0 added

#9 Updated by Ewoud Kohl van Wijngaarden about 2 months ago

  • Fixed in Releases deleted (3.6.0)

#10 Updated by Ewoud Kohl van Wijngaarden about 2 months ago

  • Triaged changed from No to Yes

Also available in: Atom PDF