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

oci_ons_notification_topic.bastion_notification is empty tuple #121

Closed
scanlonjts opened this issue Jan 27, 2020 · 5 comments
Closed

oci_ons_notification_topic.bastion_notification is empty tuple #121

scanlonjts opened this issue Jan 27, 2020 · 5 comments
Labels
bug Something isn't working

Comments

@scanlonjts
Copy link

fresh install
setting bastion_enabled = false

Error: Invalid index

on .terraform\modules\base\modules\bastion\datasources.tf line 69, in data "oci_ons_notification_topic" "bastion_notification":
69: topic_id = oci_ons_notification_topic.bastion_notification[0].topic_id
|----------------
| oci_ons_notification_topic.bastion_notification is empty tuple

The given key does not identify an element in this collection value.

should not continue the bastion notification setup if no bastion

Community Note

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
  • Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment

Terraform Version and Provider Version

Affected Resource(s)

Terraform Configuration Files

# Copy-paste your Terraform configurations here - for large Terraform configs,
# please use a service like Dropbox and share a link to the ZIP file. 
# Please remove any sensitive information from configuration files before sharing them. 

Debug Output

Panic Output

Expected Behavior

Actual Behavior

Steps to Reproduce

  1. terraform apply

Important Factoids

References

@scanlonjts scanlonjts added the bug Something isn't working label Jan 27, 2020
@hyder
Copy link
Contributor

hyder commented Jan 29, 2020

Hi,

Thank you for reporting this bug. We'll add another check to ensure the notification is not looked up when bastion is disabled in the base module. In the meantime, can you please try with setting bastion_notification_enabled = false?

@scanlonjts
Copy link
Author

Thanks for the reply. By disabling the notification I was able to setup already. Fixing the logic will prevent others from having to do the same

@hyder
Copy link
Contributor

hyder commented Feb 5, 2020

Hi, it's now fixed in the base module. Can you please test and confirm?

@hyder
Copy link
Contributor

hyder commented Feb 19, 2020

@scanlonjts can you please check?

@hyder
Copy link
Contributor

hyder commented Feb 27, 2020

Closing due to lack of response

@hyder hyder closed this as completed Feb 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants