Thursday, September 1, 2016

[389-commits] ldap/admin

ldap/admin/src/scripts/repl-monitor.pl.in | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 32ee33b2222a9bbc0657ceb912ca3fa74ee27dcc
Author: Mark Reynolds <mreynolds@redhat.com>
Date: Thu Sep 1 12:54:08 2016 -0400

Ticket 48957 - Update repl-monitor to handle new status messages

Bug Description: The replication agreement status messages have changed,
and the repl-monitor script was not updated to reflect
these changes. This lead to the html report incorrectly
color coding a successfull status.

Fix Description: Update the script to ignore "Error (0)"

https://fedorahosted.org/389/ticket/48957

Reviewed by: mreynolds (one line commit rule)

diff --git a/ldap/admin/src/scripts/repl-monitor.pl.in b/ldap/admin/src/scripts/repl-monitor.pl.in
index aa7ab1e..a3efa8e 100755
--- a/ldap/admin/src/scripts/repl-monitor.pl.in
+++ b/ldap/admin/src/scripts/repl-monitor.pl.in
@@ -877,7 +877,7 @@ sub print_consumers
}
my $redfontstart = "";
my $redfontend = "";
- if ($status =~ /error/i) {
+ if ($status !~ /Error \(0\)/i) {
$redfontstart = "<font color='red'>";
$redfontend = "</font>";
}

--
389-commits mailing list
389-commits@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/389-commits@lists.fedoraproject.org

No comments:

Post a Comment