aminueza / terraform-provider-minio

Terraform provider for managing MinIO S3 buckets and IAM Users.
https://registry.terraform.io/providers/aminueza/minio
GNU Affero General Public License v3.0
233 stars 69 forks source link

Add service account values to read and import #525

Closed pjsier closed 10 months ago

pjsier commented 11 months ago

Add service account values to read and import

Adds disable_user and target_user to minio_service_account on import which were previously missing. The more important one is minio_service_account, but in case disable_user presented issues I added it as well. disable_user is not checked for errors since it's pulled from the account status which is already checked.

I didn't include secret_key because it's computed and unless I'm misunderstanding the implementation wouldn't be necessary here.

I may have missed things in the implementation, happy to make changes!

Closing issues

felladrin commented 10 months ago

Thank you, @pjsier!