-
Notifications
You must be signed in to change notification settings - Fork 304
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
Improve the workflow #40
Conversation
metacpp
commented
Jan 9, 2018
- Added build arguments to setup terraform environment during building image.
- Update the Gemfile to add back missed gems to avoid rake e2e issue.
- Simplify the documentation to run test and build.
… gem in the package and update the readme.
@@ -6,4 +6,6 @@ group :test do | |||
git 'https://github.com/Azure/terramodtest.git' do | |||
gem 'terramodtest', :tag => 'v0.1.0' | |||
end | |||
gem 'test-kitchen', '1.16.0' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do you need these since they are already included with same version on the base image?
@@ -150,7 +150,7 @@ More specifically this provisions: | |||
Test | |||
----- | |||
|
|||
### Configuration Prerequisites |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These things are required, why remove "Prerequisites"?
|
||
```sh | ||
docker run -v ~/.ssh:/root/.ssh/ -v $MODULE_PATH/logs:/tf-test/module/.kitchen -v $MODULE_PATH:/tf-test/module -e ARM_CLIENT_ID -e ARM_TENANT_ID -e ARM_SUBSCRIPTION_ID -e ARM_CLIENT_SECRET -e ARM_TEST_LOCATION -e ARM_TEST_LOCATION_ALT --rm azure-compute rake -f ../Rakefile build |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you don't do the volume mounts then it will never test their code.
``` | ||
|
||
This runs the end to end tests: | ||
|
||
```sh | ||
docker run -v ~/.ssh:/root/.ssh/ -v $MODULE_PATH/logs:/tf-test/module/.kitchen -v $MODULE_PATH:/tf-test/module -e ARM_CLIENT_ID -e ARM_TENANT_ID -e ARM_SUBSCRIPTION_ID -e ARM_CLIENT_SECRET -e ARM_TEST_LOCATION -e ARM_TEST_LOCATION_ALT --rm azure-compute rake -f ../Rakefile e2e | ||
$ docker run -v ~/.ssh:/root/.ssh/ --rm azure-compute /bin/bash -c "rake e2e" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
need the volume mount to map the local cloned code to do e2e test