From 3f22ce0ae7fe650a178551bb3ac73b24d7e79c79 Mon Sep 17 00:00:00 2001 From: Nick Walker Date: Tue, 12 Jan 2016 09:32:06 -0800 Subject: [PATCH] Fix hostname on role creation curl command Prior to the this commit, if you were using the code_manager profile on a split install it would incorrectly try to curl the NC api on the master node. After this commit, it should correctly curl the hostname of the NC This has no impact on the functionality of the code on a monolithic PE installation. --- site/profile/manifests/git_webhook/code_manager.pp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/site/profile/manifests/git_webhook/code_manager.pp b/site/profile/manifests/git_webhook/code_manager.pp index 7470e1c..41c9e16 100644 --- a/site/profile/manifests/git_webhook/code_manager.pp +++ b/site/profile/manifests/git_webhook/code_manager.pp @@ -39,7 +39,7 @@ class profile::git_webhook::code_manager { $create_role_creates_file = '/etc/puppetlabs/puppetserver/.puppetlabs/deploy_environments_created' $create_role_curl = @(EOT) /opt/puppetlabs/puppet/bin/curl -k -X POST -H 'Content-Type: application/json' \ - https://<%= $::trusted['certname'] %>:4433/rbac-api/v1/roles \ + https://<%= $classifier_hostname %>:4433/rbac-api/v1/roles \ -d '{"permissions": [{"object_type": "environment", "action": "deploy_code", "instance": "*"}, {"object_type": "tokens", "action": "override_lifetime", "instance": "*"}],"user_ids": [], "group_ids": [], "display_name": "<%= $code_manager_role_name %>", "description": ""}' \ --cert <%= $::settings::certdir %>/<%= $::trusted['certname'] %>.pem \