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

Support for input variable, context #28

Closed
dekimsey opened this issue Aug 7, 2020 · 0 comments · Fixed by #34
Closed

Support for input variable, context #28

dekimsey opened this issue Aug 7, 2020 · 0 comments · Fixed by #34

Comments

@dekimsey
Copy link

dekimsey commented Aug 7, 2020

Describe the Feature

Currently, the module declares all of the terraform-null-label inputs independently. It'd be nice if I could simply pass the context and have that be propagated through.

Expected Behavior

Not to have to manually specify every input.

Use Case

module "user" {
  source  = "cloudposse/iam-system-user/aws"
  version = "0.12.0"
  name    = "deploy"
  path    = "/${module.label.id}/"
  
  # doesn't support context (only pass the subset I happen to know are set)
  namespace   = module.label.namespace
  stage       = module.label.stage
  environment = module.label.environment
  tags        = module.label.tags
}

Describe Ideal Solution

module "user" {
  source  = "cloudposse/iam-system-user/aws"
  version = "0.12.0"
  name    = "deploy"
  path    = "/${module.label.id}/"
  context = module.label.context
}

Alternatives Considered

The current implementation is the primary alternative. One could support overriding the given context if a more specific value is set.

Additional Context

None.

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

Successfully merging a pull request may close this issue.

1 participant