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.

Sign up for GitHub

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

Jump to bottom

Fix Reference to Undeclared Variable #169

Merged
merged 2 commits into from
Oct 16, 2023
Merged

Fix Reference to Undeclared Variable #169

merged 2 commits into from
Oct 16, 2023

Conversation

sherifabdlnaby
Copy link
Contributor

@sherifabdlnaby sherifabdlnaby commented Sep 11, 2023 •
edited by milldr
Loading

what

  • Correct variable name for var.auto_tune.cron_schedule

why

  • cron_expression_for_recurrence referred to the wrong variable name, with _ not .

references

@sherifabdlnaby sherifabdlnaby requested review from a team as code owners September 11, 2023 19:19
@sneljo1
Copy link

sneljo1 commented Sep 20, 2023 •
edited
Loading

👍 Also running into this, seems to have been here for a while

@milldr
Copy link
Member

milldr commented Oct 16, 2023

/terratest

@milldr
Copy link
Member

milldr commented Oct 16, 2023

this depends on #167 and #168. We'll get these prioritized internally

@milldr
Copy link
Member

milldr commented Oct 16, 2023

/terratest

@milldr milldr merged commit 1285ccd into cloudposse:main Oct 16, 2023
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@milldr milldr milldr approved these changes

@dotCipher dotCipher Awaiting requested review from dotCipher dotCipher is a code owner automatically assigned from cloudposse/contributors

@woz5999 woz5999 Awaiting requested review from woz5999 woz5999 is a code owner automatically assigned from cloudposse/contributors

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sherifabdlnaby @sneljo1 @milldr