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.

Sign up for GitHub

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

wip: t882 #3773

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

wip: t882 #3773

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
375a767
wip: t882
waltdisgrace Dec 18, 2024
a9fde4f
expected output
waltdisgrace Jan 8, 2025
4e90e9b
xml file
waltdisgrace Jan 15, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
54 changes: 52 additions & 2 deletions cts/cts-cli.in
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2933,7 +2933,7 @@ class UpgradeRegressionTest(RegressionTest):
@property
def tests(self):
"""A list of Test instances to be run as part of this regression test."""
resource_cib = """
resource_cib1 = """
<resources>
<primitive id="mySmartFuse" class="ocf" provider="experiment" type="SmartFuse">
<operations>
Expand All @@ -2956,13 +2956,63 @@ class UpgradeRegressionTest(RegressionTest):
<instance_attributes id-ref="mySmartFuse-outputpower-instanceparams"/>
</primitive>
</resources>
"""
resource_cib2 = """
waltdisgrace marked this conversation as resolved.
Show resolved Hide resolved
<cib crm_feature_set="3.19.5" validate-with="pacemaker-1.3" epoch="2" num_updates="0" admin_epoch="0">
<configuration>
<crm_config>
<cluster_property_set id="cib-bootstrap-options">
<nvpair id="cib-bootstrap-options-stonith-enabled" name="stonith-enabled" value="false"/>
</cluster_property_set>
</crm_config>
<nodes/>
<resources/>
<constraints/>
<rsc_defaults>
<meta_attributes id="build-resource-defaults">
<nvpair id="build-resource-stickiness" name="resource-stickiness" value="1"/>
</meta_attributes>
</rsc_defaults>
<acls>
<acl_role id="acl-role"/>
<acl_user id="acl-user">
<role_ref id="acl-role"/>
</acl_user>
</acls>
</configuration>
<status/>
</cib>
"""

t882_tests3 = [
Test("Run cibadmin with upgrade and force flags", "cibadmin -u --force",
update_cib=True),
]

t882_tests0 = [
Test("Run cibadmin with upgrade and force flags", "cibadmin -u --force",
update_cib=True),
]

t882_tests1 = [
Test("Run cibadmin with upgrade and force flags", "cibadmin -u --force -x '{cts_cli_data}/cibadmin_invalid_schema.xml'",
update_cib=True),
]

t882_tests2 = [
Test("Run cibadmin with upgrade and force flags", "cibadmin -u --force -p",
update_cib=True, stdin=resource_cib2),
]

basic_tests = [

TestGroup(t882_tests3,
env={"CIB_file": "{cts_cli_data}/cibadmin_invalid_schema.xml"}),
waltdisgrace marked this conversation as resolved.
Show resolved Hide resolved

waltdisgrace marked this conversation as resolved.
Show resolved Hide resolved
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You're not adding tests0, tests1, or tests2 to basic_tests so they will never be run. I think you were just experimenting with different ways of providing the input for the tests, right? In general, we're doing a pretty good job of testing that already so I think it's okay if you just use a single method here and remove all the others. In that case, you can also get rid of resource_cib2 because it's just duplicating what's in the XML file you add in a later patch.

Test("Set stonith-enabled=false", "crm_attribute -n stonith-enabled -v false",
update_cib=True),
Test("Configure the initial resource", "cibadmin -M -o resources -p",
update_cib=True, stdin=resource_cib),
update_cib=True, stdin=resource_cib1),
Test("Upgrade to latest CIB schema (trigger 2.10.xsl + the wrapping)",
"cibadmin --upgrade --force -V -V",
update_cib=True),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remember to run make to generate a new cts-cli program, and then run cts/cts-cli -s -r acls. This will generate a new regression output file. That change should also be included as a new patch in this PR.

Expand Down