Skip to content
This repository has been archived by the owner on Jan 19, 2019. It is now read-only.

COOK-4715 - add upgrade recipe and complete test harness #4

Closed
wants to merge 1 commit into from
Closed

COOK-4715 - add upgrade recipe and complete test harness #4

wants to merge 1 commit into from

Conversation

jtimberman
Copy link
Contributor

With openssl vulnerabilities such as Heartbleed (CVE-2014-0160) and the more recent vulnerability (CVE-2014-0224), it is desirable to keep openssl upgraded. This commit adds an upgrade recipe that will handle upgrading the openssl packages, and also restart any chef-managed services that have SSL bindings.

In order to adequately test the changes, the current complete test harness is added, too. This includes support for test-kitchen and ChefSpec.

With openssl vulnerabilities such as Heartbleed (CVE-2014-0160) and
the more recent vulnerability (CVE-2014-0224), it is desirable to keep
openssl upgraded. This commit adds an `upgrade` recipe that will
handle upgrading the openssl packages, and also restart any
chef-managed services that have SSL bindings.

In order to adequately test the changes, the current complete test
harness is added, too. This includes support for test-kitchen and
ChefSpec.
@jtimberman jtimberman changed the title add upgrade recipe and complete test harness COOK-4715 - add upgrade recipe and complete test harness Jun 10, 2014
@jtimberman
Copy link
Contributor Author

#
include_recipe 'chef-sugar'

node.default['openssl']['packages'] = case
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This attribute is done in the recipe because chef-sugar wouldn't be installed and available when the attributes file is parsed without a custom bootstrap.

jtimberman pushed a commit that referenced this pull request Jun 11, 2014
@someara
Copy link
Contributor

someara commented Jun 11, 2014

merged in a0b2af5

@someara someara closed this Jun 11, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants