PaloAltoNetworks / terraform-provider-prismacloudcompute

Terraform provider for Prisma Cloud Compute
https://registry.terraform.io/providers/PaloAltoNetworks/prismacloudcompute/latest
Mozilla Public License 2.0
24 stars 28 forks source link

Configure Provider Using Variable Rather Than Config File #30

Closed hi-artem closed 2 years ago

hi-artem commented 3 years ago

Is your feature request related to a problem?

We have a use case where using config file presents an issue: Running Terraform in automation where we are using Vault as a secret storage. We want to use Vault data source to pull secrets and configure provider. It is problematic with config file.

Describe the solution you'd like

Allow to configure provider the following way:

provider "prismacloudcompute" {
  username = "admin"
  password = "secret"
  console_url = "https://exmaple.com/"
}
hi-artem commented 3 years ago

There is actually a bug preventing this from functioning properly. I opened pr to fix it - https://github.com/PaloAltoNetworks/terraform-provider-prismacloudcompute/pull/31

wfg commented 2 years ago

Was the line fixed in #31 the only problem here?

hi-artem commented 2 years ago

That's right! I can confirm it works on my end now.

wfg commented 2 years ago

Awesome! Closing.

jasonckeating commented 2 years ago

I'm still broken, looks like there's not a new release cut with the fix, can you update?

hi-artem commented 2 years ago

@wfg

wfg commented 2 years ago

I'm still broken, looks like there's not a new release cut with the fix, can you update?

Sure can. We'll push an update Monday that'll include this fix.