Add feature to check the account type #337
Conversation
| # Account Check | ||
|
|
||
| This module determines whether a given IBM Cloud account is part of an `Enterprise` or is a `Standalone (Normal)` account. | ||
| It uses the IBM Cloud Enterprise Management API and can be easily integrated into Terraform configurations via the external data source. |
There was a problem hiding this comment.
Add the usage and required permissions here
| required_providers { | ||
| external = { | ||
| source = "hashicorp/external" | ||
| version = "2.3.5" |
There was a problem hiding this comment.
modules should use a range. EG: >=2.3.5, <3.0.0
|
|
||
| # --- Output for Terraform --- | ||
| echo "{\"account_type\": \"${ACCOUNT_TYPE}\"}" | ||
| exit 0 |
There was a problem hiding this comment.
This script needs to be updated so the endpoints can be overridden by environment variables, incase someone is using the to deploy to stage for example. See example
| source = "./modules/account_check" | ||
| target_account_id = local.target_account_id | ||
| ibmcloud_api_key = var.ibmcloud_api_key | ||
| } |
There was a problem hiding this comment.
The output of this has to be used in the restapi_object block as a new parameter. This should also have a count around it as its only needed if CSPM is enabled
| # Input Variables | ||
| ############################################################################## | ||
|
|
||
| variable "ibmcloud_api_key" { |
There was a problem hiding this comment.
Instead of added a new required input here, maybe it would be better to get the IAM token using a data lookup in the new submodule and pass the token value directly to the script?
2 participants
Description
Release required?
x.x.X)x.X.x)X.x.x)Release notes content
Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
Checklist for reviewers
For mergers