| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
|
|
|
|
|
| |
(fix tabs/spaces issue along the way)
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>
|
|
|
|
|
|
|
|
| |
Previously, the fact that --transform applies also to the symlink
content was missed and resulted in an archive that wasn't directly
usable. Also drop maintenance-specific "vim local rc" file.
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>
|
|
|
|
| |
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
| |
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
| |
... as it is a default in ocf:heartbeat:VirtualDomain anyway.
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
| |
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
|
| |
Resolves: <https://bugzilla.redhat.com/1272592>.
Reported-by: Miroslav Lisik <mlisik@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>
|
|
|
|
| |
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This was actually a conversion bug in that these unused (just declared)
resources were promoted into ones effectively executed under new stack.
Alternative solution would be to keep them included as it used to be
plus set "is-managed" meta-attribute for them to "false", but there is
at least one deeper underlying issue: ccs_flatten is not interested in
these unemployed items in the "resources" stanza at all (they are not
validated, nor they are paired with metadata-provided defaults, etc.).
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
|
|
| |
(RG = resource group)
Interim resolution for <https://bugzilla.redhat.com/1206640>.
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>
|
|
|
|
| |
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>
|
|
|
|
| |
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>
|
|
|
|
|
|
|
| |
... that used to drop non-deprecated cluster properties like
stonith-enabled to be dropped.
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Note that previosly, there was an unintended behavior of filter
chain definition containing a single named block like in:
(ccsflat2cibfinal_chain,
('cib-revitalize',
('cib-meld-templates',
('cib2pcscmd',
('stringiter-combine2' # , ('cmd-wrap' ...
)))))
worked, despite this accidentally led to the expected behavior,
i.e., the same as if the explicit tail sequence (starting with
'cib-revitalize') was appended directly as a sibling of the last
item within ccsflat2cibfinal_chain. Note that abusing this
"feature" is highly discouraged, it's really a bug and, moreover,
it will stop working when you try to apply such nesting recursively
more than once.
Just of a curiosity of when something behaves a bit better than
ever thought of (while making things proportionally worse later on).
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>
|
|
|
|
| |
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>
|
|
|
|
| |
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
|
| |
Older versions of help2man (like 1.36) requires it (sadly, as it
most likely has no value added legal-wise).
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>
|
|
|
|
|
|
|
|
|
|
| |
Currently, it translates deprecated default-resource-stickiness,
is-managed-default and default-action-timeout cluster properties
(options) to resource/operation default properties respectively.
More can arrive later.
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
| |
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
| |
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
| |
Also fix indentation where not per the convention.
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|
|
|
|
|
|
| |
See <https://bugzilla.redhat.com/1225946#c7>.
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>
|
|
|
|
|
|
|
| |
E.g., do not propagate the "implicit" ones like dc-version
or cluster-infrastructure.
Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
|