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

Update remote-state module #66

Merged
merged 7 commits into from
Jun 2, 2023
Merged

Update remote-state module #66

merged 7 commits into from
Jun 2, 2023

Conversation

aknysh
Copy link
Member

@aknysh aknysh commented Jun 2, 2023

what

  • Update remote-state module
  • Add example using bypass=true
  • Update terratest
  • Update Go to 1.20

why

  • If var.bypass is set to true, don't call the datasources, just return the defaults

test

TestExamplesRemoteState 2023-06-02T04:23:42Z logger.go:66: Outputs:
TestExamplesRemoteState 2023-06-02T04:23:42Z logger.go:66: 
TestExamplesRemoteState 2023-06-02T04:23:42Z logger.go:66: remote_state_using_context = {
TestExamplesRemoteState 2023-06-02T04:23:42Z logger.go:66:   "val1" = true
TestExamplesRemoteState 2023-06-02T04:23:42Z logger.go:66:   "val2" = "2"
TestExamplesRemoteState 2023-06-02T04:23:42Z logger.go:66:   "val3" = 3
TestExamplesRemoteState 2023-06-02T04:23:42Z logger.go:66:   "val4" = null
TestExamplesRemoteState 2023-06-02T04:23:42Z logger.go:66: }
TestExamplesRemoteState 2023-06-02T04:23:42Z logger.go:66: remote_state_using_context_ignore_errors = {
TestExamplesRemoteState 2023-06-02T04:23:42Z logger.go:66:   "default_output" = "default-value"
TestExamplesRemoteState 2023-06-02T04:23:42Z logger.go:66: }
TestExamplesRemoteState 2023-06-02T04:23:42Z logger.go:66: remote_state_using_stack = {
TestExamplesRemoteState 2023-06-02T04:23:42Z logger.go:66:   "val1" = true
TestExamplesRemoteState 2023-06-02T04:23:42Z logger.go:66:   "val2" = "2"
TestExamplesRemoteState 2023-06-02T04:23:42Z logger.go:66:   "val3" = 3
TestExamplesRemoteState 2023-06-02T04:23:42Z logger.go:66:   "val4" = null
TestExamplesRemoteState 2023-06-02T04:23:42Z logger.go:66: }
TestExamplesRemoteState 2023-06-02T04:23:42Z logger.go:66: remote_state_with_bypass = {
TestExamplesRemoteState 2023-06-02T04:23:42Z logger.go:66:   "default_output" = "default-value"
TestExamplesRemoteState 2023-06-02T04:23:42Z logger.go:66: }

@aknysh aknysh added the patch A minor, backward compatible change label Jun 2, 2023
@aknysh aknysh requested a review from Nuru June 2, 2023 03:33
@aknysh aknysh self-assigned this Jun 2, 2023
@aknysh aknysh requested review from a team as code owners June 2, 2023 03:33
@aknysh aknysh requested review from jamengual and woz5999 June 2, 2023 03:33
@aknysh
Copy link
Member Author

aknysh commented Jun 2, 2023

/terratest

@aknysh
Copy link
Member Author

aknysh commented Jun 2, 2023

/terratest

@aknysh
Copy link
Member Author

aknysh commented Jun 2, 2023

/terratest

@Nuru Nuru merged commit de7da5b into main Jun 2, 2023
@aknysh aknysh deleted the update-remote-state-module branch June 2, 2023 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch A minor, backward compatible change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants