Remove dependenies to apt and yum cookbooks. #132

Merged
merged 1 commit into from Oct 24, 2016

Conversation

Projects
None yet
3 participants
@artem-sidorenko
Member

artem-sidorenko commented Oct 21, 2016

The following resources are now completely provided by chef:

  • yum_repository
  • yum_package
  • apt_repository
  • apt_package

Currently you get warnings:

WARN: Chef::Provider::AptRepository already exists!  Cannot create deprecation class for LWRP provider apt_repository from cookbook apt
WARN: AptRepository already exists!  Deprecation class overwrites Custom resource apt_repository from cookbook apt
WARN: Chef::Provider::YumRepository already exists!  Cannot create deprecation class for LWRP provider yum_repository from cookbook yum
WARN: YumRepository already exists!  Deprecation class overwrites Custom resource yum_repository from cookbook yum

Starting from the version v12.14.89 the behavior of yum_repository
seems to be the same like in the cookbook before. So I used this version
as the requirement.

Remove dependenies to apt and yum cookbooks.
The following resources are now completely provided by chef:
 - yum_repository
 - yum_package
 - apt_repository
 - apt_package

Currently you get warnings:
WARN: Chef::Provider::AptRepository already exists!  Cannot create deprecation class for LWRP provider apt_repository from cookbook apt
WARN: AptRepository already exists!  Deprecation class overwrites Custom resource apt_repository from cookbook apt
WARN: Chef::Provider::YumRepository already exists!  Cannot create deprecation class for LWRP provider yum_repository from cookbook yum
WARN: YumRepository already exists!  Deprecation class overwrites Custom resource yum_repository from cookbook yum

Starting from the version v12.14.89 the behaviour of yum_repository
seems to be the same like in the cookbook before. Using this version
as the requirement.
@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Oct 21, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling 6e37167 on artem-sidorenko:apt-yum-removal into 65dcc05 on dev-sec:master.

coveralls commented Oct 21, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling 6e37167 on artem-sidorenko:apt-yum-removal into 65dcc05 on dev-sec:master.

@atomic111

This comment has been minimized.

Show comment
Hide comment
@atomic111

atomic111 Oct 24, 2016

Member

great, thanks @artem-sidorenko for the improvement

Member

atomic111 commented Oct 24, 2016

great, thanks @artem-sidorenko for the improvement

@atomic111 atomic111 merged commit d7409fc into dev-sec:master Oct 24, 2016

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-removal branch Nov 5, 2016

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