Alexandros Kosiaris has submitted this change and it was merged. Change subject: ores::base: Remove data_path ......................................................................
ores::base: Remove data_path nltk is now under /srv/ores/config/submodules/wheels/nltk/ as per https://github.com/wiki-ai/ores-wikimedia-config/commit/b4770646b7d0e6802b86b57b851b40fe470458ea so remove $data_path and nltk handling from ores::base as it is redundant Change-Id: Ib6dff70c777247433470ff2fbb239555232b5c8d --- M modules/ores/manifests/base.pp 1 file changed, 0 insertions(+), 3 deletions(-) Approvals: Alexandros Kosiaris: Verified; Looks good to me, approved diff --git a/modules/ores/manifests/base.pp b/modules/ores/manifests/base.pp index 298bb7a..e296f51 100644 --- a/modules/ores/manifests/base.pp +++ b/modules/ores/manifests/base.pp @@ -2,7 +2,6 @@ $branch = 'deploy', $config_path = '/srv/ores/config', $venv_path = '/srv/ores/venv', - $data_path = '/srv/ores/data', ) { # Let's use a virtualenv for maximum flexibility - we can convert # this to deb packages in the future if needed. We also install build tools @@ -42,8 +41,6 @@ file { [ '/srv/ores', - $data_path, - "${data_path}/nltk", ]: ensure => directory, owner => 'www-data', -- To view, visit https://gerrit.wikimedia.org/r/278950 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib6dff70c777247433470ff2fbb239555232b5c8d Gerrit-PatchSet: 2 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: Alexandros Kosiaris <[email protected]> Gerrit-Reviewer: Alexandros Kosiaris <[email protected]> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
