We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Pushing this out to a separate issue so that we can get #20 merged.
When using make docker_run, we should source the test config so that CLOUDSDK_AUTH_CREDENTIAL_FILE_OVERRIDE is set in the shell environment.
make docker_run
CLOUDSDK_AUTH_CREDENTIAL_FILE_OVERRIDE
diff --git i/Makefile w/Makefile index 6e16919..73ff8bf 100644 --- i/Makefile +++ w/Makefile @@ -119,7 +119,7 @@ docker_run: docker run --rm -it \ -v $(CURDIR):/cftk/workdir \ ${DOCKER_IMAGE_KITCHEN_TERRAFORM}:${DOCKER_TAG_KITCHEN_TERRAFORM} \ - /bin/bash + /bin/bash --rcfile ${TEST_CONFIG_FILE_LOCATION} .PHONY: docker_create docker_create: docker_build_terraform docker_build_kitchen_terraform
Alternately, we can set the environment variables within the tests, as such:
ENV['CLOUDSDK_AUTH_CREDENTIAL_FILE_OVERRIDE'] = File.expand_path( File.join("../..", credentials_path), __FILE__)
Though this feels somewhat brittle.
The text was updated successfully, but these errors were encountered:
This has been addressed for a while, closing.
Sorry, something went wrong.
No branches or pull requests
Pushing this out to a separate issue so that we can get #20 merged.
When using
make docker_run
, we should source the test config so thatCLOUDSDK_AUTH_CREDENTIAL_FILE_OVERRIDE
is set in the shell environment.Alternately, we can set the environment variables within the tests, as such:
Though this feels somewhat brittle.
The text was updated successfully, but these errors were encountered: