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

Travis and kitchen-ec2 testing #181

Merged
merged 1 commit into from Dec 9, 2016

Conversation

Projects
None yet
2 participants
@alexpop
Collaborator

alexpop commented Dec 8, 2016

Uses IAM policy that can only create, tag and terminate own ec2 instances

@alexpop alexpop self-assigned this Dec 8, 2016

Enable Travis testing in ec2
Signed-off-by: Alex Pop <apop@chef.io>

@alexpop alexpop changed the title from WIP: travis and kitchen-ec2 testing to Travis and kitchen-ec2 testing Dec 9, 2016

its(['total']) { should be > 60 }
end
# guard the elastic_hits array from index out of bounds exception
if elastic_hits.length == 2

This comment has been minimized.

@chris-rock

chris-rock Dec 9, 2016

Collaborator

Should we add an error if the length is not 2?

This comment has been minimized.

@alexpop

alexpop Dec 9, 2016

Collaborator

I was thinking that this will provide the error:

describe elastic_hits.length do
    it { should eq 2 }
end

This comment has been minimized.

@chris-rock

chris-rock Dec 9, 2016

Collaborator

You are right!

its(['total']) { should be > 60 }
end
# guard the elastic_hits array from index out of bounds exception
if elastic_hits.length == 2

This comment has been minimized.

@chris-rock

chris-rock Dec 9, 2016

Collaborator

You are right!

@chris-rock chris-rock merged commit e49ce7b into master Dec 9, 2016

2 checks passed

DCO This commit has a DCO Signed-off-by
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@chris-rock chris-rock deleted the ap/travis-kitchen branch Dec 9, 2016

@chris-rock chris-rock removed the in progress label Dec 9, 2016

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