Yuvipanda has submitted this change and it was merged. Change subject: k8s: Ensure /etc/kubernetes is present wherever required ......................................................................
k8s: Ensure /etc/kubernetes is present wherever required Change-Id: I932664528562393f0e03e5ad4997ec7007a18d94 --- M modules/k8s/manifests/infrastructure_config.pp M modules/k8s/manifests/kubelet.pp 2 files changed, 8 insertions(+), 10 deletions(-) Approvals: Yuvipanda: Verified; Looks good to me, approved diff --git a/modules/k8s/manifests/infrastructure_config.pp b/modules/k8s/manifests/infrastructure_config.pp index 3bab4c1..b628c89 100644 --- a/modules/k8s/manifests/infrastructure_config.pp +++ b/modules/k8s/manifests/infrastructure_config.pp @@ -1,4 +1,11 @@ class k8s::infrastructure_config { + file { '/etc/kubernetes': + ensure => directory, + owner => 'root', + group => 'root', + mode => '0755', + } + $users = hiera('k8s_users') # Ugly HACK! $client_token = inline_template("<%= @users.select { |u| u['name'] == 'client-infrastructure' }[0]['token'] %>") @@ -8,5 +15,6 @@ owner => 'root', group => 'root', mode => '0400', + require => File['/etc/kubernetes'], } } diff --git a/modules/k8s/manifests/kubelet.pp b/modules/k8s/manifests/kubelet.pp index d1e7834..ccc6b67 100644 --- a/modules/k8s/manifests/kubelet.pp +++ b/modules/k8s/manifests/kubelet.pp @@ -6,16 +6,6 @@ include ::k8s::infrastructure_config file { [ - '/etc/kubernetes/', - '/etc/kubernetes/manifests', - ]: - ensure => directory, - owner => 'root', - group => 'root', - mode => '0755', - } - - file { [ '/var/run/kubernetes', '/var/lib/kubelet', ] : -- To view, visit https://gerrit.wikimedia.org/r/291617 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I932664528562393f0e03e5ad4997ec7007a18d94 Gerrit-PatchSet: 4 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: Yuvipanda <yuvipa...@wikimedia.org> Gerrit-Reviewer: Gehel <gleder...@wikimedia.org> Gerrit-Reviewer: Yuvipanda <yuvipa...@wikimedia.org> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits