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

terraform-provider: fix parsing api_server_cert_sans #2758

Merged
merged 3 commits into from
Dec 27, 2023

Conversation

3u13r
Copy link
Member

@3u13r 3u13r commented Dec 20, 2023

Context

Proposed change(s)

  • Don't double quote cert sans
  • Improve Terraform provider examples

Additional info

  • Any additional information or context

Checklist

  • Add labels (e.g., for changelog category)
  • Is PR title adequate for changelog?
  • Link to Milestone

@3u13r 3u13r added the bug fix Fixing a bug label Dec 20, 2023
@3u13r 3u13r added this to the v2.15.0 milestone Dec 20, 2023
@3u13r 3u13r requested review from msanft and elchead December 20, 2023 14:51
Copy link

netlify bot commented Dec 20, 2023

Deploy Preview for constellation-docs canceled.

Name Link
🔨 Latest commit 53ac030
🔍 Latest deploy log https://app.netlify.com/sites/constellation-docs/deploys/658309e6b418e40008b64a6f

@3u13r 3u13r changed the title Terraform Provider: Fix setting api_server_cert_sans Terraform Provider: Fix parsing api_server_cert_sans Dec 20, 2023
@msanft msanft changed the title Terraform Provider: Fix parsing api_server_cert_sans terraform-provider: fix parsing api_server_cert_sans Dec 20, 2023
@3u13r 3u13r requested a review from msanft December 20, 2023 15:32
Copy link
Contributor

Coverage report

Package Old New Trend
terraform-provider-constellation/internal/provider 4.80% 4.70% ↘️

@@ -70,6 +70,7 @@ module "aws_infrastructure" {
image_id = data.constellation_image.bar.image.reference
region = local.region
zone = local.zone
internal_load_balancer = false
debug = false
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we remove debug, since we don't really support this for customers?

@3u13r 3u13r merged commit 2f10223 into main Dec 27, 2023
9 checks passed
@3u13r 3u13r deleted the fix/tf-provider/cert-sans branch December 27, 2023 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug fix Fixing a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants