adoroszlai commented on code in PR #11133:
URL: https://github.com/apache/ozone/pull/11133#discussion_r3878250741


##########
hadoop-ozone/dist/src/main/compose/xcompat/lib.sh:
##########
@@ -24,7 +24,7 @@ source "${COMPOSE_DIR}/../testlib.sh"
 
 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

Review Comment:
   I think testing with latest patch release from each minor release line is 
enough:
   
   ```suggestion
   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`.



##########
hadoop-ozone/dist/src/main/compose/upgrade/test.sh:
##########
@@ -33,8 +33,8 @@ RESULT_DIR="$ALL_RESULT_DIR" create_results_dir
 
 # This is the version of Ozone that should use the runner image to run the
 # code that was built. Other versions will pull images from docker hub.
-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"

Review Comment:
   Let's keep this (disabled) for previous minor release:
   
   ```suggestion
   # run_test ha     non-rolling-upgrade 2.1.1 "$OZONE_CURRENT_VERSION"
   ```



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to