Skip to content
This repository has been archived by the owner on Dec 13, 2018. It is now read-only.

Integration tests #33

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Integration tests #33

wants to merge 3 commits into from

Conversation

rclark
Copy link
Contributor

@rclark rclark commented Jul 21, 2016

  • assert that the ecs-conex's Dockerfile can build an image
  • assert that running ecs-conex's image can build another repo's image

cc @emilymdubois @jakepruitt

fixes #28
fixes #1

utils.sh Outdated
if [[ -z $role ]]; then
role=$(curl -s http://169.254.169.254/latest/meta-data/iam/security-credentials/) || :
if [[ -z $role ]] && [[ -n ${AWS_ACCESS_KEY_ID} ]]; then
echo "env creds"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note to self: remove these echos

@rclark rclark force-pushed the integration-test branch 5 times, most recently from 8ba3dbe to fc471b0 Compare July 27, 2016 02:34
@rclark
Copy link
Contributor Author

rclark commented Aug 2, 2016

Running these tests repeatedly leads to docker images being clogged up with ecs-conex-test images. Remove them via

docker images | grep 'ecs-conex-test' | awk '{b=$1":"$2; print b}' | xargs docker rmi

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Load AWS credentials from environment Tests
1 participant