Ema has submitted this change and it was merged. Change subject: Remove Varnishkafka APT pinning ......................................................................
Remove Varnishkafka APT pinning Bug: T150660 Change-Id: If49fd22de7037ee3d2fbd2b50161a6b2a59ff421 --- M modules/role/manifests/cache/kafka.pp 1 file changed, 0 insertions(+), 11 deletions(-) Approvals: Ema: Verified; Looks good to me, approved diff --git a/modules/role/manifests/cache/kafka.pp b/modules/role/manifests/cache/kafka.pp index 2e67ad4..4a7891b 100644 --- a/modules/role/manifests/cache/kafka.pp +++ b/modules/role/manifests/cache/kafka.pp @@ -7,17 +7,6 @@ # NOTE: This is used by inheriting classes role::cache::kafka::* $kafka_brokers = $kafka_config['brokers']['array'] - apt::pin { 'varnishkafka': - ensure => 'absent', - pin => '', - priority => '', - } - - # Make the Varnishkafka class depend on APT pinning. We want to ensure - # varnishkafka is not apt-get installed before the pinning file is - # created/removed. - Apt::Pin['varnishkafka'] -> Class['Varnishkafka'] - # Make sure varnishes are configured and started for the first time # before the instances as well, or they fail to start initially... Service <| tag == 'varnish_instance' |> -> Varnishkafka::Instance <| |> -- To view, visit https://gerrit.wikimedia.org/r/322911 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: If49fd22de7037ee3d2fbd2b50161a6b2a59ff421 Gerrit-PatchSet: 2 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: Ema <e...@wikimedia.org> Gerrit-Reviewer: BBlack <bbl...@wikimedia.org> Gerrit-Reviewer: Elukey <ltosc...@wikimedia.org> Gerrit-Reviewer: Ema <e...@wikimedia.org> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits