Skip to content

Add resource to handle customized bare metal with quote#151

Open
steven4ibm wants to merge 2 commits intosoftlayer:masterfrom
steven4ibm:bare_metal_quote
Open

Add resource to handle customized bare metal with quote#151
steven4ibm wants to merge 2 commits intosoftlayer:masterfrom
steven4ibm:bare_metal_quote

Conversation

@steven4ibm
Copy link

Actual bare metal resource handles only fixed configuration of bare metal.

This resource allows to manage customized bare metal where price and configuration are stored in IBM Bluemix Infrastructure (ex-SoftLayer) quote system.

@minsikl
Copy link
Contributor

minsikl commented May 26, 2017

The plugin already has bare_metal resource. It is necessary to extend current bare_metal resource to support custom bare metal servers.

Utilizing quote ID is a good idea and can simplify a resource definition and validation process. However, If the terraform resource definition does not contain specifications, it cannot be reused later. To fully support CRUD, it is necessary to have individual parameters in the terraform resource definition. One idea I'm thinking about is implementing a quote data source. Then, users can define custom bare metal server quotes on SL portal and import quote data sources. The quote data sources will retrieve bare metal server specifications, and users will define custom bare metal server with the information.

@renier
Copy link
Contributor

renier commented Jun 21, 2017

At a minimum, this would have to work as an additional parameter to the existing bare metal resource, and not as a separate resource.

I also like Minsik's idea of creating a data source for a bare metal quote. So in order for this to be merged, it would have to align with one of these two thoughts.

@steven4ibm
Copy link
Author

I have merge the bare_metal_quote resource with the bare_metal resource as requested.

I don't have the Go skill neither the time to implement the second idea. But I am interested by seeing it implemented.

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

Successfully merging this pull request may close these issues.

3 participants