Skip to content

[9.0] test: skip backward-compatibility assertions for APIs dropped in v9.1 - #8756

Open
chrisburr wants to merge 1 commit into
DIRACGrid:rel-v9r0from
chrisburr:fix/bc-tests-skip-apis-removed-in-9.1
Open

[9.0] test: skip backward-compatibility assertions for APIs dropped in v9.1#8756
chrisburr wants to merge 1 commit into
DIRACGrid:rel-v9r0from
chrisburr:fix/bc-tests-skip-apis-removed-in-9.1

Conversation

@chrisburr

Copy link
Copy Markdown
Member

The "Backward Compatibility" CI job runs this branch's tests against a server
installed from a more recent branch. Two v9.0 tests exercise APIs that v9.1
deliberately removed (#8473 dropped ResourceManagement.AccountingCache, #8446
dropped nodes/queues from the RSS along with Publisher.getNodeStatuses), so they
cannot pass against a 9.1 server.

No need for release notes.

@chrisburr
chrisburr requested a review from fstagni as a code owner August 27, 2026 10:12
"""
DowntimeCache table
"""
if not serverIsOlderThan(rmClient, "9.1"):

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
if not serverIsOlderThan(rmClient, "9.1"):
if serverIsOlderThan(rmClient, "9.1"):

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

That would be incorrect I think?

@fstagni fstagni changed the title test: skip backward-compatibility assertions for APIs dropped in v9.1 [9.0] test: skip backward-compatibility assertions for APIs dropped in v9.1 Aug 27, 2026
@fstagni

fstagni commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

For completeness you should also have this added to integration

@chrisburr

Copy link
Copy Markdown
Member Author

Why? it's not needed in integration as these tests don't exist there

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