[MediaWiki-commits] [Gerrit] operations/puppet[production]: contint: remove zuul from role::ci::slave::browsertests

2016-10-14 Thread Dzahn (Code Review)
Dzahn has submitted this change and it was merged.

Change subject: contint: remove zuul from role::ci::slave::browsertests
..


contint: remove zuul from role::ci::slave::browsertests

Zuul is already provided in the sole caller role::ci::slave::labs

Change-Id: Ida6a358ffd85872c55f9a761b4495c5023a2df3d
---
M modules/role/manifests/ci/slave/browsertests.pp
1 file changed, 0 insertions(+), 2 deletions(-)

Approvals:
  Hashar: Looks good to me, but someone else must approve
  Dzahn: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/modules/role/manifests/ci/slave/browsertests.pp 
b/modules/role/manifests/ci/slave/browsertests.pp
index 5a7d0a4..f41c412 100644
--- a/modules/role/manifests/ci/slave/browsertests.pp
+++ b/modules/role/manifests/ci/slave/browsertests.pp
@@ -8,8 +8,6 @@
 include role::ci::slave::labs::common
 include contint::browsertests
 
-class { '::zuul': }
-
 # For CirrusSearch testing:
 file { '/mnt/elasticsearch':
 ensure => absent,

-- 
To view, visit https://gerrit.wikimedia.org/r/315963
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ida6a358ffd85872c55f9a761b4495c5023a2df3d
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Hashar 
Gerrit-Reviewer: Chad 
Gerrit-Reviewer: Dzahn 
Gerrit-Reviewer: Hashar 
Gerrit-Reviewer: Thcipriani 
Gerrit-Reviewer: jenkins-bot <>

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits


[MediaWiki-commits] [Gerrit] operations/puppet[production]: contint: remove zuul from role::ci::slave::browsertests

2016-10-14 Thread Hashar (Code Review)
Hashar has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/315963

Change subject: contint: remove zuul from role::ci::slave::browsertests
..

contint: remove zuul from role::ci::slave::browsertests

Zuul is already provided in the sole caller role::ci::slave::labs

Change-Id: Ida6a358ffd85872c55f9a761b4495c5023a2df3d
---
M modules/role/manifests/ci/slave/browsertests.pp
1 file changed, 0 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/63/315963/1

diff --git a/modules/role/manifests/ci/slave/browsertests.pp 
b/modules/role/manifests/ci/slave/browsertests.pp
index 5a7d0a4..f41c412 100644
--- a/modules/role/manifests/ci/slave/browsertests.pp
+++ b/modules/role/manifests/ci/slave/browsertests.pp
@@ -8,8 +8,6 @@
 include role::ci::slave::labs::common
 include contint::browsertests
 
-class { '::zuul': }
-
 # For CirrusSearch testing:
 file { '/mnt/elasticsearch':
 ensure => absent,

-- 
To view, visit https://gerrit.wikimedia.org/r/315963
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ida6a358ffd85872c55f9a761b4495c5023a2df3d
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Hashar 

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits