Compliance uses ChefDK 1.0.3 #20
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Ohai version 8.11.1 fixed Ohai so that it will detect paravirt amazon instances without hint files. be42b07 removes those hints. |
added 18 commits
November 30, 2016 07:14
Signed-off-by: Nathen Harvey <nharvey@chef.io>
Signed-off-by: Nathen Harvey <nharvey@chef.io>
* Use a generic CentOS 7 Image from the marketplace * Update the ssh user * Bump the version number Signed-off-by: Nathen Harvey <nharvey@chef.io>
Chef will get installed without specifying this Signed-off-by: Nathen Harvey <nharvey@chef.io>
Signed-off-by: Nathen Harvey <nharvey@chef.io>
Signed-off-by: Nathen Harvey <nharvey@chef.io>
Signed-off-by: Nathen Harvey <nharvey@chef.io>
Signed-off-by: Nathen Harvey <nharvey@chef.io>
Signed-off-by: Nathen Harvey <nharvey@chef.io>
Signed-off-by: Nathen Harvey <nharvey@chef.io>
* docker * compat_resource Signed-off-by: Nathen Harvey <nharvey@chef.io>
Signed-off-by: Nathen Harvey <nharvey@chef.io>
Signed-off-by: Nathen Harvey <nharvey@chef.io>
Signed-off-by: Nathen Harvey <nharvey@chef.io>
Templates no longer need to be in the `templates/default` directory. Move them to the `templates` directory. Signed-off-by: Nathen Harvey <nharvey@chef.io>
This allows us to scan the node from Chef Compliance Signed-off-by: Nathen Harvey <nharvey@chef.io>
supports { manage_home: true } on the user resource is deprecated and
will be removed in Chef 13, set manage_home: true instead
Signed-off-by: Nathen Harvey <nharvey@chef.io>
* ChefDK 1.0.3 is installed * AMI version bumped to 1.3.1 * AMI name updated * AMI ID updated in the build and show scripts Signed-off-by: Nathen Harvey <nharvey@chef.io>
nathenharvey
force-pushed
the
nathen/compliance_updates
branch
from
134167a
to
c0214bf
Compare
nathenharvey
changed the title
burtlo
reviewed
Nov 30, 2016
|
|
||
| remote_file "epel-release-6-8.noarch.rpm" do | ||
| source "http://ftp.osuosl.org/pub/fedora-epel/6/i386/epel-release-6-8.noarch.rpm" | ||
| docker_service 'default' do |
There was a problem hiding this comment.
This is nice. The docker cookbook makes that easy.
burtlo
reviewed
Nov 30, 2016
| @@ -15,8 +15,6 @@ | |||
|
|
|||
| chef_ingredient 'chefdk' do | |||
| action :install | |||
| channel :stable | |||
| version '0.17.17' | |||
There was a problem hiding this comment.
Your intention is always to grab the latest?
burtlo
reviewed
Nov 30, 2016
|
|
||
| AMI_ID=ami-614d7476 | ||
|
|
||
| aws ec2 describe-instances --query 'Reservations[*].Instances[*].[InstanceId,ImageId,PublicIpAddress,State.Name,Tags[?Key==`Name`].Value | [0]]' --output text | grep running | grep $AMI_ID |
There was a problem hiding this comment.
Would you be opposed to me moving these into rake tasks after merging?
burtlo
reviewed
Nov 30, 2016
| recursive true | ||
| end | ||
|
|
||
| file "/etc/chef/ohai/hints/ec2.json" do |
There was a problem hiding this comment.
You removed the hints for all the workstations. It's hard for me to see if that was replaced with another resource, if this is unneeded now, or a personal setting.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Use the latest ChefDK for compliance
Fix issues with Docker
Lots of clean-up