Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add ability to bring your own EM project id or name the created project #55

Merged
merged 4 commits into from
Mar 10, 2021

Conversation

displague
Copy link
Member

@displague displague commented Mar 10, 2021

Adds the ability to bring your own Equinix Metal Project ID.

  • Organization ID is now required, If creating a project. The project name can now be specified or use the existing randomly generated name.

  • Project ID can now be specified. When specified, set metal_create_project to false.

  • Fixes the integration test by defining the needed GitHub secrets

  • Makes the test ephemeral. Hashicorp's external state is no longer used, a project creation and sweeping action is used.

Fixes #54 (not sure how, but the tests should help confirm)
Fixes #50

Signed-off-by: Marques Johansson <mjohansson@equinix.com>
@displague displague force-pushed the project-control branch 8 times, most recently from bcd3e92 to 06295a6 Compare March 10, 2021 15:53
Integration tests will now uses lighter resources
The project will be managed by GH actions
External state (Hashicorp Enterprise) is no longer needed, the
integration test resources are ephemeral.

Signed-off-by: Marques Johansson <mjohansson@equinix.com>
Signed-off-by: Marques Johansson <mjohansson@equinix.com>
Signed-off-by: Marques Johansson <mjohansson@equinix.com>
@displague
Copy link
Member Author

The automation now gets significantly further:

Error: Could not find matching reserved block, all IPs were []

  on ProviderNetwork.tf line 6, in data "metal_precreated_ip_block" "private_ipv4":
   6: data "metal_precreated_ip_block" "private_ipv4" {



Error: Could not find matching reserved block, all IPs were []

  on ProviderNetwork.tf line 24, in data "metal_precreated_ip_block" "public_ipv6":
  24: data "metal_precreated_ip_block" "public_ipv6" {

I'll open a new issue for this. Merging this PR.

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.

Error Not Found CI needs to be configured with an API token
1 participant