feat(lh-86970): add Terraform resource to add an existing tenant to MSP portal using API token #153

siddhuwarrier · 2024-12-03T12:22:00Z

Make the name field optional; require it only if api_token is not specified, and vice versa. This allows us to add existing tenants.
Add an existing tenant as follows:

resource "cdo_msp_managed_tenant" "existing_tenant" {
  api_token = "existing-tenant-api-token"
}

TODO: unit and acceptance tests

…SP portal using API token

talhazi

small ones, LGTM

client/msp/tenants/add.go

… on branch

…acceptance tests

feat(lh-86970): add Terraform resource to add an existing tenant to M…

da0bc10

…SP portal using API token

siddhuwarrier requested a review from talhazi December 3, 2024 12:22

talhazi approved these changes Dec 3, 2024

View reviewed changes

client/msp/tenants/add.go Outdated Show resolved Hide resolved

client/msp/tenants/add.go Outdated Show resolved Hide resolved

siddhuwarrier added 6 commits December 3, 2024 13:27

fix(lh-86970): address Tal's comments and improve documentation

4ca74d7

test(lh-86970): add more tests, make acceptance tests run temporarily…

6ca92ef

… on branch

revert this before merging

0cf44c6

fix(lh-86970): add ADDED_MSP_MANAGED_TENANT_API_TOKEN to the env for …

e61589b

…acceptance tests

I hate Github

79c00ca

Stop running acceptance tests on branch

0925823

siddhuwarrier merged commit 4364675 into main Dec 3, 2024
7 checks passed

Provide feedback