Fix a couple of F-13 bugs

This commit is contained in:
Andrew Beekhof 2010-06-11 13:06:24 +00:00
parent 3a2d7da2b3
commit abdb169577
3 changed files with 64 additions and 3 deletions

40
bug-lf-2401.patch Normal file
View File

@ -0,0 +1,40 @@
changeset: 9848:b7784ef1e32a
parent: 9655:6e6980376f01
user: Andrew Beekhof <andrew@beekhof.net>
date: Mon May 17 17:48:59 2010 +0200
files: crmd/callbacks.c lib/common/membership.c
description:
High: Core: Bug lf#2401 - Backed out changeset 6e6980376f01
The original check was correct, as "member" in this context is only
supposed to refer to comms-level membership.
Additionally, the more agressive check borked the dlm and didn't
actually resolve Bug lf#2401
diff -r 6e6980376f01 -r b7784ef1e32a crmd/callbacks.c
--- a/crmd/callbacks.c Tue Apr 20 10:20:15 2010 +0200
+++ b/crmd/callbacks.c Mon May 17 17:48:59 2010 +0200
@@ -394,7 +394,7 @@ crmd_client_status_callback(const char *
return;
}
- if(safe_str_eq(node, fsa_our_dc) && crm_is_member_active(member) == FALSE) {
+ if(safe_str_eq(node, fsa_our_dc) && safe_str_eq(status, OFFLINESTATUS)){
/* did our DC leave us */
crm_info("Got client status callback - our DC is dead");
register_fsa_input(C_CRMD_STATUS_CALLBACK, I_ELECTION, NULL);
diff -r 6e6980376f01 -r b7784ef1e32a lib/common/membership.c
--- a/lib/common/membership.c Tue Apr 20 10:20:15 2010 +0200
+++ b/lib/common/membership.c Mon May 17 17:48:59 2010 +0200
@@ -39,9 +39,7 @@ gboolean crm_have_quorum = FALSE;
gboolean crm_is_member_active(const crm_node_t *node)
{
- if(node
- && safe_str_eq(node->state, CRM_NODE_MEMBER)
- && (node->processes & crm_proc_crmd)) {
+ if(node && safe_str_eq(node->state, CRM_NODE_MEMBER)) {
return TRUE;
}
return FALSE;

13
doc-validation.patch Normal file
View File

@ -0,0 +1,13 @@
diff -up doc/Pacemaker_Explained/en-US/Book_Info.xml doc/Pacemaker_Explained/en-US/Book_Info.xml
--- doc/Pacemaker_Explained/en-US/Book_Info.xml 2010-05-12 10:45:54.000000000 +0200
+++ doc/Pacemaker_Explained/en-US/Book_Info.xml 2010-06-10 11:52:07.000000000 +0200
@@ -28,8 +28,7 @@
<corpauthor>
<inlinemediaobject>
<imageobject>
- <imagedata fileref="Common_Content/images/title_logo.svg" format="SVG">
- </imagedata>
+ <imagedata fileref="Common_Content/images/title_logo.svg" format="SVG"/>
</imageobject>
</inlinemediaobject>
</corpauthor>

View File

@ -16,7 +16,7 @@
# We generate some docs using Publican, but its not available everywhere
%bcond_without publican
%global specversion 1
%global specversion 2
%global upstream_version Pacemaker-1.1.2
%global upstream_prefix Pacemaker-1-1-
@ -40,11 +40,13 @@
Name: pacemaker
Summary: Scalable High-Availability cluster resource manager
Version: 1.1.2
Release: %{pcmk_release}.1
Release: %{pcmk_release}
License: GPLv2+ and LGPLv2+
Url: http://www.clusterlabs.org
Group: System Environment/Daemons
Source0: http://hg.clusterlabs.org/pacemaker/1.1/archive/%{upstream_version}.tar.bz2
Patch1: bug-lf-2401.patch
Patch2: doc-validation.patch
BuildRoot: %(mktemp -ud %{_tmppath}/%{name}-%{version}-%{release}-XXXXXX)
AutoReqProv: on
Requires(pre): cluster-glue
@ -163,6 +165,8 @@ resource health.
%prep
%setup -q -n %{upstream_prefix}%{upstream_version}
%patch1 -p1
%patch2 -p0
%build
./autogen.sh
@ -171,6 +175,7 @@ resource health.
%{?_without_ais} \
%{?_without_esmtp} \
%{?_without_snmp} \
--disable-ansi \
--docdir=%{pcmk_docdir} \
--localstatedir=%{_var} \
--enable-fatal-warnings=no
@ -298,10 +303,13 @@ rm -rf %{buildroot}
%doc AUTHORS
%changelog
* Fri Jun 11 2010 Andrew Beekhof <andrew@beekhof.net> - 1.1.2-2
- Resolves rhbz#602239 - Added patch to documentation so that it passes validation
- High: Core: Bug lf#2401 - Backed out changeset 6e6980376f01
* Tue Jun 01 2010 Marcela Maslanova <mmaslano@redhat.com> - 1.1.2-1.1
- Mass rebuild with perl-5.12.0
%changelog
* Wed May 12 2010 Andrew Beekhof <andrew@beekhof.net> - 1.1.2-1
- Update the tarball from the upstream 1.1.2 release
+ High: ais: Bug lf#2340 - Force rogue child processes to terminate after waiting 2.5 minutes