Skip to content

HDDS-16109. Update upgrade/xcompat tests to use 2.2.1 - #11133

Open
jojochuang wants to merge 1 commit into
apache:masterfrom
jojochuang:HDDS-16109
Open

HDDS-16109. Update upgrade/xcompat tests to use 2.2.1#11133
jojochuang wants to merge 1 commit into
apache:masterfrom
jojochuang:HDDS-16109

Conversation

@jojochuang

Copy link
Copy Markdown
Contributor

Summary

  • Add old_client_2_2_1 to xcompat clients.yaml
  • Append 2.2.1 to old_versions in lib.sh
  • Run upgrade test from 2.2.1 instead of 2.2.0

Jira

https://issues.apache.org/jira/browse/HDDS-16109

Depends on HDDS-16108 / apache/ozone-docker#96 for apache/ozone:2.2.1-rocky image.

Test plan

  • xcompat CI passes after Docker image is published

Made with Cursor

Add 2.2.1 client image to xcompat compose and run upgrade test from 2.2.1.

Co-authored-by: Cursor <cursoragent@cursor.com>
Change-Id: I108e633cca22aa6b3f177dea7f16d84404b49720
Copilot AI lite review requested due to automatic review settings August 27, 2026 22:33

Copilot AI left a comment

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.

Copilot wasn't able to review any files in this pull request.


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@adoroszlai adoroszlai left a comment

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.

Thanks @jojochuang for the patch.

current_version="${OZONE_CURRENT_VERSION}"
# TODO: debug acceptance test failures for client versions 1.0.0 on secure clusters
old_versions="1.1.0 1.2.1 1.3.0 1.4.1 2.0.0 2.1.1 2.2.0" # container is needed for each version in clients.yaml
old_versions="1.1.0 1.2.1 1.3.0 1.4.1 2.0.0 2.1.1 2.2.0 2.2.1" # container is needed for each version in clients.yaml

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.

I think testing with latest patch release from each minor release line is enough:

Suggested change
old_versions="1.1.0 1.2.1 1.3.0 1.4.1 2.0.0 2.1.1 2.2.0 2.2.1" # container is needed for each version in clients.yaml
old_versions="1.1.0 1.2.1 1.3.0 1.4.1 2.0.0 2.1.1 2.2.1" # container is needed for each version in clients.yaml

old_client_2_2_0 may also be removed from clients.yaml.

run_test ha non-rolling-upgrade 2.2.0 "$OZONE_CURRENT_VERSION"
# run_test ha non-rolling-upgrade 2.1.1 "$OZONE_CURRENT_VERSION"
run_test ha non-rolling-upgrade 2.2.1 "$OZONE_CURRENT_VERSION"
# run_test ha non-rolling-upgrade 2.2.0 "$OZONE_CURRENT_VERSION"

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.

Let's keep this (disabled) for previous minor release:

Suggested change
# run_test ha non-rolling-upgrade 2.2.0 "$OZONE_CURRENT_VERSION"
# run_test ha non-rolling-upgrade 2.1.1 "$OZONE_CURRENT_VERSION"

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.

3 participants