Skip to content

Conversation

@stephenlauck
Copy link
Collaborator

No description provided.

@stephenlauck
Copy link
Collaborator Author

fixes issue where build node get's a knife.rb that looks like this:

[root@build workspace]# cat .chef/knife.rb 
node_name                "build.services.com"
log_level                :info
log_location             STDOUT
validation_client_name   "delivery-validator"
validation_key           "/etc/chef/delivery-validator.pem"
chef_server_url          "https://chef.services.com/organizations/delivery"
encrypted_data_bag_secret "/tmp/kitchen/encrypted_data_bag_secret"
trusted_certs_dir         "/etc/chef/trusted_certs"

node_name 'delivery'
log_location 'STDOUT'
client_key '/var/opt/delivery/workspace/.chef/knife.rb/delivery.pem'
trusted_certs_dir '/etc/chef/trusted_certs'

@stephenlauck stephenlauck force-pushed the lauck/fix_runner_knife_rb branch 3 times, most recently from 255731e to 6a445e7 Compare February 16, 2017 03:06
@stephenlauck stephenlauck force-pushed the lauck/fix_runner_knife_rb branch from a648f93 to 0f7bdd5 Compare February 16, 2017 14:34
file "#{workspace}/#{dir}" do
content ensurekv(::File.read('/etc/chef/client.rb'),
node_name: new_resource.chef_user,
log_location: 'STDOUT',
Copy link
Owner

Choose a reason for hiding this comment

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

You need log_location: 'STDOUT' to get the logs into automate.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I'll add that back in and see if it works, I think the permission errors were related to the keys.

@ncerny
Copy link
Owner

ncerny commented Mar 18, 2017

@stephenlauck - Did you test to make sure this is working as intended? I'd like to merge it in if it is.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants