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

"Error in function call" when enabled = false #42

Open
syphernl opened this issue Aug 18, 2021 · 2 comments
Open

"Error in function call" when enabled = false #42

syphernl opened this issue Aug 18, 2021 · 2 comments
Labels
bug 🐛 An issue with the system

Comments

@syphernl
Copy link
Contributor

syphernl commented Aug 18, 2021

Describe the Bug

After upgrading to the changes in #41 (v0.19.0) with enabled = false running this module results in getting an error:

╷
│ Error: Error in function call
│ 
│   on .terraform/modules/ses/main.tf line 45, in locals:
│   45:   ses_group_name = coalesce(var.ses_group_name, module.this.id)
│     ├────────────────
│     │ module.this.id is ""
│     │ var.ses_group_name is ""
│ 
│ Call to function "coalesce" failed: no non-null, non-empty-string
│ arguments.

It seems that the enabled is not adhered in this particular calculation.

Expected Behavior

No error, since the module is not enabled nothing should be created/generated.

Steps to Reproduce

Steps to reproduce the behavior:

  1. Set enabled = false
  2. Error shown
@syphernl syphernl added the bug 🐛 An issue with the system label Aug 18, 2021
@nitrocode
Copy link
Member

@syphernl is this still an issue with the latest version?

@kurbar
Copy link

kurbar commented Mar 2, 2024

Can confirm this is still an issue in 0.24.0

Usage:

module "ses" {
    source  = "cloudposse/ses/aws"
    version = "0.24.0"

    domain = var.domain
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 An issue with the system
Projects
None yet
Development

No branches or pull requests

3 participants