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
