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

[Issue #1943] Fix variable access #1945

Merged
merged 1 commit into from
May 7, 2024
Merged

[Issue #1943] Fix variable access #1945

merged 1 commit into from
May 7, 2024

Conversation

coilysiren
Copy link
Collaborator

Summary

Fixes #1943

Time to review: 1 mins

Changes proposed

Changes app_access_policy_arn and migrator_access_policy_arn to not be nested under db_vars

Testing

terraform plan now works in the following folders:

  • infra/analytics/metabase
  • infra/api/service
  • infra/frontend/service

@coilysiren coilysiren marked this pull request as ready for review May 7, 2024 19:37
@coilysiren coilysiren requested a review from aplybeah as a code owner May 7, 2024 19:37
@coilysiren coilysiren merged commit 1ba31ac into main May 7, 2024
7 checks passed
@coilysiren coilysiren deleted the service-db-vars branch May 7, 2024 20:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Task]: Frontend deploys broken
2 participants