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

Log routing tenant creation can fail if instance name does not meet certain criteria #628

Closed
ocofaigh opened this issue Feb 6, 2025 · 1 comment · Fixed by #630
Closed

Comments

@ocofaigh
Copy link
Member

ocofaigh commented Feb 6, 2025

The logic in the cloud logs module is currently set up to use the value of the instance name as the log routing target name too. However we found that the target name has restrictions in what the value can be:

The value for 'targets.name' is not valid. The value must be between 1 and 35 characters in length and can only contain these characters: ^[a-zA-Z][a-zA-Z0-9-]*$."

When you do not enter a value that meets the above criteria, you get an error like this:

 2025/02/06 07:20:31 Terraform apply | Error: ---
 2025/02/06 07:20:31 Terraform apply | id: terraform-b13d0bdb
 2025/02/06 07:20:31 Terraform apply | summary: 'CreateTenantWithContext failed: Bad Request'
 2025/02/06 07:20:31 Terraform apply | severity: error
 2025/02/06 07:20:31 Terraform apply | resource: ibm_logs_router_tenant
 2025/02/06 07:20:31 Terraform apply | operation: create
 2025/02/06 07:20:31 Terraform apply | component:
 2025/02/06 07:20:31 Terraform apply |   name: github.com/IBM-Cloud/terraform-provider-ibm
 2025/02/06 07:20:31 Terraform apply |   version: 1.74.0
 2025/02/06 07:20:31 Terraform apply | ---

Since we are not exposing the ability in the module to set the name, we need to ensure the computed name meets above criteria.

@terraform-ibm-modules-ops
Copy link
Contributor

🎉 This issue has been resolved in version 3.4.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants