Removal of apt/yum cookbooks from tests #169

Merged
merged 1 commit into from Feb 16, 2017

Conversation

Projects
None yet
3 participants
@artem-sidorenko
Member

artem-sidorenko commented Feb 13, 2017

They produce broken builds and we do not need them anymore:
https://travis-ci.org/dev-sec/chef-ssh-hardening/jobs/201236096

In the same time we need to run apt-get update at the start of chef-run
as ubuntu does not have any apt repo information. This is the reason
for introduction of test cookbook.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Feb 13, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling 1a2af9b on artem-sidorenko:apt-yum into e03f161 on dev-sec:master.

Coverage Status

Coverage remained the same at 100.0% when pulling 1a2af9b on artem-sidorenko:apt-yum into e03f161 on dev-sec:master.

Removal of apt/yum cookbooks from tests
They produce broken builds and we do not need them anymore:
https://travis-ci.org/dev-sec/chef-ssh-hardening/jobs/201236096

In the same time we need to run apt-get update at the start of chef-run
as ubuntu does not have any apt repo information. This is the reason
for introduction of test cookbook.
@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Feb 13, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling cf1b473 on artem-sidorenko:apt-yum into e03f161 on dev-sec:master.

Coverage Status

Coverage remained the same at 100.0% when pulling cf1b473 on artem-sidorenko:apt-yum into e03f161 on dev-sec:master.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Feb 13, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling 56972dd on artem-sidorenko:apt-yum into e03f161 on dev-sec:master.

Coverage Status

Coverage remained the same at 100.0% when pulling 56972dd on artem-sidorenko:apt-yum into e03f161 on dev-sec:master.

@atomic111

This comment has been minimized.

Show comment
Hide comment
@atomic111

atomic111 Feb 16, 2017

Member

@artem-sidorenko thanks for removing the old stuff

Member

atomic111 commented Feb 16, 2017

@artem-sidorenko thanks for removing the old stuff

@atomic111 atomic111 merged commit b19230f into dev-sec:master Feb 16, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 100.0%
Details

@artem-sidorenko artem-sidorenko deleted the artem-forks:apt-yum branch Feb 16, 2017

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