Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Final backports for 2.1.9 release #3713

Open
wants to merge 13 commits into
base: 2.1
Choose a base branch
from
Open

Conversation

kgaillot
Copy link
Contributor

These relate to two things:

  • The ClusterLabs.org website was updated, and various locations changed
  • cts-lab broke on systems with python3-dateutil <2.7.0, which we still support

kgaillot and others added 13 commits October 28, 2024 15:55
For older versions, the site will redirect the old URL to the new one.
The updated website no longer references it. Pacemaker Explained's
fencing chapter is sufficient to replace it.
This adds XML support as well, which is somewhat pointless since this
program doesn't really see a lot of use, but it's consistent with the
rest of our command line tools.

Additionally, here the XML output uses text nodes instead of putting
everything into attributes.  This is because the tool can potentially
output multiple messages, one after the other.  We never have all the
data we need at once, so using text nodes is a cleaner way to structure
the code.

Fixes T109
Now that we have a program whose name does not begin with crm_ doing XML
output, we need to update the regex to fix up the request= attribute.
We support OSes with older versions of dateutil
@kgaillot
Copy link
Contributor Author

@clumens , can you review this one? Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants