Project

General

Profile

Bug #19326

Compute Resource - Make URL optional for certain providers(example: VMware)

Added by Swapnil Abnave over 1 year ago. Updated 3 months ago.

Status:
Closed
Priority:
Normal
Category:
Compute resources
Target version:
Difficulty:
trivial
Triaged:
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

Related issues

Related to Hammer CLI - Bug #17568: Not able to create the VMware compute resource without --url flag in hammerClosed2016-12-03
Related to Docker - Bug #20429: Docker Compute Resource - URL should be requiredClosed2017-07-27

Associated revisions

Revision 18439248 (diff)
Added by Swapnil Abnave about 1 year ago

Fixes #19326 - create compute-resource - URL optional

History

#1 Updated by Swapnil Abnave over 1 year ago

  • Related to Bug #17568: Not able to create the VMware compute resource without --url flag in hammer added

#2 Updated by Dominic Cleal over 1 year ago

Why would the URL be optional on a VMware compute resource? How would the CR be accessed, is there a well-known default?

(Fog::Compute::Vsphere specifies the server address as required.)

#3 Updated by The Foreman Bot over 1 year ago

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

#4 Updated by Swapnil Abnave over 1 year ago

Dominic Cleal wrote:

Why would the URL be optional on a VMware compute resource? How would the CR be accessed, is there a well-known default?

(Fog::Compute::Vsphere specifies the server address as required.)

I suppose server attribute is an alias for url and it is required in Foreman::Model::Vmware https://github.com/theforeman/foreman/blob/develop/app/models/compute_resources/foreman/model/vmware.rb#L9

Attribute server is mandatory per API and Web. I think making URL not required for VMware would be fine.

#5 Updated by Swapnil Abnave over 1 year ago

  • Target version changed from 177 to 1.17.0-RC1

#6 Updated by Swapnil Abnave about 1 year ago

  • Related to Bug #20429: Docker Compute Resource - URL should be required added

#7 Updated by Timo Goebel about 1 year ago

  • Legacy Backlogs Release (now unused) set to 240

#8 Updated by Swapnil Abnave about 1 year ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

Also available in: Atom PDF