| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
| |
Also reflect the changes in the tests.
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
| |
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
|
|
| |
Previously, it there was a hard-wired dependency making assumptions
about the project vs tests file tree layout. This made tests fail
when different layout was used.
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
| |
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Note that there was a subtle, broken assumption behind how these tests
should work, but this was masked thanks to the result from XSLT always
being a standalone tree (not a set/forest) since the resources were
previously undeployed (passively defined in <resources> of cluster.conf
and not referenced from the active parts). After putting "activating"
the resources by putting them under <service> stanza, the XSLT results
were actually sets of element nodes and while only the first one was
exposed externally, mysterious kabooms around RNG validation emerged.
Proper fix is to rely on a single wrapping level of processing
(following the same rules of root element transformations, notably
this "single result tree only" one) that will make the result of the
actual level of XSLT under test be wrapped with <clufter:snippet>
that is then removed upon attachment to the parent level, which is
OK even if such result contained more trees modulo the wrapping.
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
| |
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
For this monitor operation, either use the interval as designated in the
original rgmanager agent (sticking with the most basic=lowest depth only),
or use the default of 60 seconds (the same is automatically injected as
a default by pcs).
Resolves: <https://bugzilla.redhat.com/1272570>.
Also adjust the (unit) tests respectively.
Reported-by: Miroslav Lisik <mlisik@redhat.com>
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
|
| |
Explicit commands are definitely easier to grasp than, e.g.,
"--clone foo=bar" hidden at the tail of a "resource add" command.
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
|
|
|
| |
Resolves <https://bugzilla.redhat.com/1270740>.
Also extend one of the unit tests to cover clone resource (borrowed from
the reported issue).
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
| |
Also reflect that in the unit test.
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
| |
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
| |
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Based on tri-state whitelist denoting positive propagation:
- 1:1
- 1:1 but emit a warning (the meaning was changing in time)
- rename the subsys
Opened questions:
- should CMAN also be renamed to PCMK?
Also fix affected (but not directly related) unit test.
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Previous one was just splitting into lines within a single group
(also, this was limited to at most one split! that in turn made
it a pretty broken text wrapping). Now it is two-ways allowing
also short-enough groups to be put on the same line, and as many
line splits is made as necessary for a proper text wrapping.
Also update the test counterparts (function run_check_cmd of
run-check script succeeds again after quite a long time!).
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
| |
Fairly unfinished, to be tweaked over time...
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
| |
Also add a unit test targeting this issue.
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
| |
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
| |
Also add the unit test(s).
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
| |
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
|
| |
Also included are respective unit tests (that needed code adjustments
and fixtures spanning several past commits).
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
| |
Likewise, this a "ground preparation" for upcoming new unit test.
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
|
| |
Preparing ground for nested test and unittest not playing well with
symlinked _common.py in such the respective nested directory.
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
|
| |
> WARNING:clufter.formats.command:'command' format: not backed by
> OrderedDict
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
| |
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
| |
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
| |
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
| |
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
... otherwise original quotation is not preserved in, say,
ccs2pcscmd(-flatiron) command due to command format usage in cmd-wrap
filter (--noop=cmd-wrap would serve as a workaround before this fix).
Also add a respective unit test.
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
| |
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
| |
(mainly for posterity)
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
| |
(also unit tests benefit from this)
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
| |
Should be stand-alone for easier reusability in the future...
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
| |
Should be stand-alone for easier reusability in the future...
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
|
| |
... first check being that pcs commands after "auth" are run from actual
intended cluster node.
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
| |
Also adjust the test case.
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
| |
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
| |
Configurable via pcscmd_tmpcib value in command/filter context, feature
disabled when passing empty string.
Uses "pcs -f tmp-cib.xml ..." (per pcscmd_tmpcib value, this filename
is default) for various CIB manipulations, then it is pushed using
"pcs cluster cib-push tmp-cib.xml --config".
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
| |
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
|
|
| |
Also:
- turn off verbose output for the purpose of a unit test
- normalize commas in string interpolations for the snippets
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
|
|
| |
Also:
- output a new-line (as appropriate) + reflect it in the unit test
- turn off verbose output for the purpose of a unit test
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
| |
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
|
|
|
| |
Think of this as 2-to-1 multiplexer (effectively just performing
in-order concatenation of the two inputs).
Also add a unit test.
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
|
| |
This is for upcoming cluster.conf (Pacemaker) + CIB -> sequence of pcs
commands conversion.
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
|
| |
This is for upcoming cluster.conf (Pacemaker) + CIB -> sequence of pcs
commands conversion.
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
| |
"pcs" should be reserved for actual pcs CLI tool commands.
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
|
| |
Also add relevant unit tests.
Reported-by: Patrik Hagara <phagara@redhat.com>
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
| |
Also revamp respective unit tests.
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
... that allows specifying multi-argument option as a single-argument
one with '::' being a logical separator between such args. This is
needed, for instance, when multi-argument option is specified last to
differentiate between such option argument and proper command argument
(only a single argument for an option is normally expected).
Also add a respective unit test.
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
| |
Also fix a typo and add a relevant unit test.
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
| |
Also extend .gitignore whitelist appropriately.
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|