Google
  Web www.spinics.net

[Freeze break Request] Disable bugzilla integration

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]


The bugzilla admins at redhat notified us that they were having load issues
since the bugzilla upgrade caused by infrastructure's hosts.  It seems that
at least one of our scripts is exercising bugs in the new bugzilla codebase.
They've asked us to turn off all of our cron jobs for now so that they can
stabilise the platform and then we'll go through and see if we can identify
the problem query(-ies)  and fix it either on our side or in bugzilla.

Since this is an emergency, we've disabled the cron jobs first -- sending
the patch here for +1's to break freeze and so we know what to revert when
the time comes.

While these are disabled, the following things will not work:

* changes in user status in fas (users joining and leaving packager or
  fedorabugs) will not be reflected to bugzilla
* The cached review status pages will not update
* The easyfix.fedoraproject.org bugs list will not update
* Changes in package ownership will not propogate from the packagedb to
  bugzilla
* New packages won't be added to bugzilla (although they should be able to
  be added to the packagedb and git).

-Toshio
From af0cabefef3d7264050ca75392bfc14156f4c32d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Toshio=20=E3=81=8F=E3=82=89=E3=81=A8=E3=81=BF?= <toshio@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx>
Date: Wed, 23 May 2012 15:21:01 +0000
Subject: [PATCH] Disable bugzilla cron jobs

---
 manifests/services/bugzilla.pp             |    2 +-
 modules/fas/manifests/init.pp              |    1 +
 modules/fedora-packagedb/manifests/init.pp |    1 +
 modules/gather-easyfix/manifests/init.pp   |    1 +
 modules/review-stats/manifests/init.pp     |    1 +
 5 files changed, 5 insertions(+), 1 deletions(-)

diff --git a/manifests/services/bugzilla.pp b/manifests/services/bugzilla.pp
index 21b086e..f55a123 100644
--- a/manifests/services/bugzilla.pp
+++ b/manifests/services/bugzilla.pp
@@ -12,7 +12,7 @@ class bugzilla-no-balance {
         command => "/usr/local/bin/bz-make-components.py 'http://git.fedorahosted.org/git/?p=l10n/owners.git;a=blob_plain;f=owners.list;hb=master' 'http://git.fedorahosted.org/git/?p=docs/owners.git;a=blob_plain;f=owners.list;hb=master' &> /dev/null",
         user => "apache",
         minute => 20,
-        ensure => present,
+        ensure => absent,
         require => Package['fedora-packagedb']
     }
 }
diff --git a/modules/fas/manifests/init.pp b/modules/fas/manifests/init.pp
index f06ef8b..e20ca1f 100644
--- a/modules/fas/manifests/init.pp
+++ b/modules/fas/manifests/init.pp
@@ -372,6 +372,7 @@ class fas::app::nobalance inherits fas::app {
         minute      => 10,
         require     => Package["fas"],
         environment => "MAILTO=root",
+        ensure => "absent",
     }
 
 }
diff --git a/modules/fedora-packagedb/manifests/init.pp b/modules/fedora-packagedb/manifests/init.pp
index 724080c..1244f1f 100644
--- a/modules/fedora-packagedb/manifests/init.pp
+++ b/modules/fedora-packagedb/manifests/init.pp
@@ -87,6 +87,7 @@ class fedora-packagedb::app::nobalance inherits fedora-packagedb::app {
         command => "/usr/bin/pkgdb-sync-bugzilla > /dev/null 2> /dev/null",
         user => "root",
         minute => 10,
+        ensure => "absent",
     }
 }
 
diff --git a/modules/gather-easyfix/manifests/init.pp b/modules/gather-easyfix/manifests/init.pp
index aae2b9e..b0be432 100644
--- a/modules/gather-easyfix/manifests/init.pp
+++ b/modules/gather-easyfix/manifests/init.pp
@@ -85,6 +85,7 @@ class gather-easyfix::build inherits gather-easyfix {
         command => "cd /srv/web/gather-easyfix && /usr/bin/python /usr/local/bin/gather_easyfix.py",
         user    => "apache",
         minute  => [ 0, 30 ],
+        ensure => "absent",
     }
 
 }
diff --git a/modules/review-stats/manifests/init.pp b/modules/review-stats/manifests/init.pp
index 01c51a3..9a016a5 100644
--- a/modules/review-stats/manifests/init.pp
+++ b/modules/review-stats/manifests/init.pp
@@ -57,6 +57,7 @@ class review-stats::build inherits review-stats {
         command => "/usr/bin/python /usr/local/bin/pyGetReviewByFlags.py -t /usr/local/share/review-stats -d /srv/web/review-stats/",
         user    => "apache",
         minute  => [ 0, 30 ],
+        ensure => "absent",
     }
 
 }
-- 
1.7.2.1

Attachment: pgpx1r5A2yJZV.pgp
Description: PGP signature

_______________________________________________
infrastructure mailing list
infrastructure@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/infrastructure

[Home]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Red Hat 9 Bible]     [Fedora Bible]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]

Powered by Linux

Google
  Web www.spinics.net