[MediaWiki-commits] [Gerrit] apache-sanity-check (part of apache-graceful-all) change the... - change (operations...wikimedia-task-appserver)

2013-05-15 Thread Dzahn (Code Review) 
Dzahn has submitted this change and it was merged.

Change subject: apache-sanity-check (part of apache-graceful-all) change the 
regex to also restart API servers
..


apache-sanity-check (part of apache-graceful-all)
change the regex to also restart API servers

this should make the apache-graceful-all output clean again,
no more "System failed sanity check" messages.

that is because API servers have VIPs ending in .22 and they
are in dsh groups, so they were NOT restarted. but we want
them to be restarted per email discussion in March

the .21 is actually for rendering servers, but noone actually use it
yet.

1   1H  IN PTR  appservers.svc.eqiad.wmnet.
...
21  1H  IN PTR  rendering.svc.eqiad.wmnet.
22  1H  IN PTR  api.svc.eqiad.wmnet.

Change-Id: I49324bc453c35103359f583472d317e583072c5c
---
M apache-sanity-check
M debian/changelog
2 files changed, 8 insertions(+), 1 deletion(-)

Approvals:
  Dzahn: Verified; Looks good to me, approved



diff --git a/apache-sanity-check b/apache-sanity-check
index fea1b38..60651d8 100644
--- a/apache-sanity-check
+++ b/apache-sanity-check
@@ -8,7 +8,7 @@
 export PATH=/bin:/usr/bin:/sbin:/usr/sbin
 
 # FIXME: move this to PyBal runcommand check, and/or replace with Puppet logic.
-VIP="10\\.2\\.[1-2]\\.\(1\|21\)"
+VIP="10\\.2\\.[1-2]\\.\(1\|22\)"
 if ! /sbin/ip addr | grep -q "$VIP.*lo"; then
echo "System failed sanity check: VIP not configured on lo"
exit 1
diff --git a/debian/changelog b/debian/changelog
index 948ca7d..b4e3753 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,10 @@
+wikimedia-task-appserver (2.9-1) precise-wikimedia; urgency=low
+
+  * make apache-graceful-all also restart API servers by changing
+the apache-sanity-check regex on VIPs
+
+ -- Daniel Zahn   Wed, 15 May 2013 16:05:23 -0700
+
 wikimedia-task-appserver (2.8-1) precise-wikimedia; urgency=low
 
   * Remove scap-1, scap-2, and sync-common scripts so they can be managed

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I49324bc453c35103359f583472d317e583072c5c
Gerrit-PatchSet: 3
Gerrit-Project: operations/debs/wikimedia-task-appserver
Gerrit-Branch: master
Gerrit-Owner: Dzahn 
Gerrit-Reviewer: Dzahn 

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

[MediaWiki-commits] [Gerrit] apache-sanity-check (part of apache-graceful-all) change the... - change (operations...wikimedia-task-appserver)

2013-04-24 Thread Dzahn (Code Review) 
Dzahn has uploaded a new change for review.

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


Change subject: apache-sanity-check (part of apache-graceful-all) change the 
regex to also restart API servers
..

apache-sanity-check (part of apache-graceful-all)
change the regex to also restart API servers

this should make the apache-graceful-all output clean again,
no more "System failed sanity check" messages.

that is because API servers have VIPs ending in .22 and they
are in dsh groups, so they were NOT restarted. but we want
them to be restarted per email discussion in March

the .21 is actually for rendering servers, but noone actually use it
yet.

1   1H  IN PTR  appservers.svc.eqiad.wmnet.
...
21  1H  IN PTR  rendering.svc.eqiad.wmnet.
22  1H  IN PTR  api.svc.eqiad.wmnet.

Change-Id: I49324bc453c35103359f583472d317e583072c5c
---
M apache-sanity-check
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/operations/debs/wikimedia-task-appserver 
refs/changes/68/60768/1

diff --git a/apache-sanity-check b/apache-sanity-check
index fea1b38..60651d8 100644
--- a/apache-sanity-check
+++ b/apache-sanity-check
@@ -8,7 +8,7 @@
 export PATH=/bin:/usr/bin:/sbin:/usr/sbin
 
 # FIXME: move this to PyBal runcommand check, and/or replace with Puppet logic.
-VIP="10\\.2\\.[1-2]\\.\(1\|21\)"
+VIP="10\\.2\\.[1-2]\\.\(1\|22\)"
 if ! /sbin/ip addr | grep -q "$VIP.*lo"; then
echo "System failed sanity check: VIP not configured on lo"
exit 1

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I49324bc453c35103359f583472d317e583072c5c
Gerrit-PatchSet: 1
Gerrit-Project: operations/debs/wikimedia-task-appserver
Gerrit-Branch: master
Gerrit-Owner: Dzahn 

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