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

auth_string is not exposed in module #76

Closed
chicocvenancio opened this issue Mar 17, 2022 · 1 comment · Fixed by #77
Closed

auth_string is not exposed in module #76

chicocvenancio opened this issue Mar 17, 2022 · 1 comment · Fixed by #77
Labels
enhancement New feature or request P4 low priority issues triaged Scoped and ready for work

Comments

@chicocvenancio
Copy link

TL;DR

auth_string is not exposed in module. If auth_enabled is set there is no simple way to get the auth_string.

Expected behavior

auth_string be in the module output

Observed behavior

No response

Terraform Configuration

module "redis" {
  source  = "terraform-google-modules/memorystore/google"
  version = "4.2.0"

  project      = var.project_id
  name         = "api-redis-${var.environment_name}-1"
  display_name = "api-redis-${var.environment_name}"

  region             = var.region
  authorized_network = "${var.environment_name}-vpc"

  auth_enabled = true

  transit_encryption_mode = "DISABLED"
  enable_apis             = var.enable_apis

  labels =  merge(
    var.add_resource_labels, {
    guild     = "infra"
    terraform = true
  })
}

Terraform Version

Terraform v1.1.7
on darwin_amd64
+ provider registry.terraform.io/hashicorp/google v4.13.0
+ provider registry.terraform.io/hashicorp/google-beta v4.14.0

Additional information

No response

@chicocvenancio chicocvenancio added the bug Something isn't working label Mar 17, 2022
@morgante morgante added enhancement New feature or request triaged Scoped and ready for work P4 low priority issues and removed bug Something isn't working labels Mar 18, 2022
@morgante
Copy link
Contributor

This is a feature request, not a bug. We would be happy to review a PR adding it.

JulienBreux added a commit to JulienBreux/terraform-google-memorystore that referenced this issue Mar 22, 2022
morgante pushed a commit that referenced this issue Mar 22, 2022
* feat: Add missing auth_string output. Fix #76

* doc: Generate documentation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request P4 low priority issues triaged Scoped and ready for work
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants