Project

General

Profile

Bug #28885

bug in virtual machine api in compute resource

Added by yifat makias 10 months ago. Updated 9 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Compute resources
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

2 things needed to be fixed in virtual machine API:
1. adding provider field to show_vm - in order to be able to identify which compute resource, we are dealing with in hammer.
2. fixing the API for GCE - find_vm_by_uuid needs to get the full name in GCE and not the id.


Related issues

Related to Hammer CLI - Bug #20451: Hammer compute-resource lacks commands for virtual machineClosed

Associated revisions

Revision fd0b39ac (diff)
Added by yifat makias 9 months ago

Fixes #28885 - added provider field to virtual machine api

History

#1 Updated by The Foreman Bot 9 months ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/7495 added

#2 Updated by The Foreman Bot 9 months ago

  • Fixed in Releases 2.1.0 added

#3 Updated by yifat makias 9 months ago

  • Status changed from Ready For Testing to Closed

#4 Updated by yifat makias 9 months ago

  • Related to Bug #20451: Hammer compute-resource lacks commands for virtual machine added

#5 Updated by yifat makias 9 months ago

2 things needed to be fixed in virtual machine API:
1. adding provider field to show_vm - in order to be able to identify which compute resource, we are dealing with in hammer.
2. fixing the API for GCE - find_vm_by_uuid needs to get the full name in GCE and not the id.

The second issue is not really an issue. Apparently it didn't work before because of the problematic fields that were excluded in this fix. In addition, a comment was added to the hammer help documentation so that users will know how to use this command properly when dealing with GCE compute resource (use VM name instead of VM id).

Also available in: Atom PDF