summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJan Pokorný <jpokorny@redhat.com>2016-01-28 17:36:57 +0100
committerJan Pokorný <jpokorny@redhat.com>2016-02-01 23:54:13 +0100
commit31eb05c147dc928d5c78ecf9e303b3d999e2ecd4 (patch)
treea5239c2abefb7e5a4da1b1170d20e2edb07e27a9
parent47249473962210a420d2061ff58865a637b053cb (diff)
downloadclufter-31eb05c147dc928d5c78ecf9e303b3d999e2ecd4.tar.gz
clufter-31eb05c147dc928d5c78ecf9e303b3d999e2ecd4.tar.xz
clufter-31eb05c147dc928d5c78ecf9e303b3d999e2ecd4.zip
filters/ccs-revitalize[rm]: omit "failure" params when noop
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
-rw-r--r--filters/cluster/rm/__init__.py31
1 files changed, 31 insertions, 0 deletions
diff --git a/filters/cluster/rm/__init__.py b/filters/cluster/rm/__init__.py
index e9ddf16..f6c2f2f 100644
--- a/filters/cluster/rm/__init__.py
+++ b/filters/cluster/rm/__init__.py
@@ -342,6 +342,37 @@ ccs_revitalize = '''\
</xsl:when>
<xsl:otherwise>
<xsl:copy>
+ <xsl:choose>
+ <!-- omit __max_failures + __failure_expire_time when
+ used without any real effect -->
+ <xsl:when test="not(@__max_failures)
+ !=
+ not(@__failure_expire_time)
+ or
+ number(@__max_failures) &lt;= 0
+ or
+ number(@__failure_expire_time) &lt; 0">
+ <xsl:message>
+ <xsl:value-of select="concat('WARNING: __max_failures',
+ ' and __failure_expire_time',
+ ' must be specified in pair',
+ ' and both of them has to be',
+ ' positive numbers; not',
+ ' satisfied, hence dropped',
+ '(', name(),')')"/>
+ </xsl:message>
+ <xsl:apply-templates select="@*[
+ not(
+''' + (
+ xslt_is_member('name()',
+ ('__max_failures',
+ '__failure_expire_time'))
+) + ''')]"/>
+ </xsl:when>
+ <xsl:otherwise>
+ <xsl:apply-templates select="@*"/>
+ </xsl:otherwise>
+ </xsl:choose>
<xsl:apply-templates select="@*|node()"/>
</xsl:copy>
</xsl:otherwise>