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

BATS tests Dockerfile #44

Merged
merged 3 commits into from Aug 20, 2018

Conversation

Projects
None yet
2 participants
@patthiel
Contributor

patthiel commented Aug 13, 2018

  • Adds Dockerfile to run BATS tests

  • Adds dockerignore file so that builds/dist files are ignored when creating the docker image

  • This PR was based off the domains test branch, I will rebase once that PR is merged. The main files of concern here are Dockerfile-bats .dockerignore and README.rst

Testing

Please remember these TESTS WILL DELETE ALL RESOURCES (linodes/volumes/domains/nodebalancers/etc/etc) associated with your linode account!

  • Follow the instructions for running via docker in the README

cc. @Dorthu

&& make install SPEC=${SPEC} \
&& cd dist \
&& pip install --user $(ls) \
&& echo -n "[DEFAULT]\ntoken = ${TOKEN}" > /root/.linode-cli

This comment has been minimized.

@Dorthu

Dorthu Aug 20, 2018

Member

This config file will work, but a real config file would have keys for image, type, and region

@Dorthu

Dorthu Aug 20, 2018

Member

This config file will work, but a real config file would have keys for image, type, and region

@Dorthu

Dorthu approved these changes Aug 20, 2018

@Dorthu Dorthu merged commit 7afed95 into linode:master Aug 20, 2018

@patthiel patthiel deleted the patthiel:bats-docker branch Aug 20, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment