errose28 commented on code in PR #3262:
URL: https://github.com/apache/ozone/pull/3262#discussion_r852429890
##########
hadoop-ozone/dist/src/main/compose/xcompat/test.sh:
##########
@@ -73,6 +73,45 @@ test_cross_compatibility() {
KEEP_RUNNING=false stop_docker_env
}
+test_ec_cross_compatibility() {
+ echo "Running Erasure Coded storage backward compatibility tests."
+ local cluster_versions_with_ec="1.3.0"
+ local non_ec_client_versions="1.0.0 1.1.0 1.2.1"
+
+ for cluster_version in ${cluster_versions_with_ec}; do
+ export COMPOSE_FILE=new-cluster.yaml:clients.yaml
cluster_version=${cluster_version}
Review Comment:
new-cluster.yaml is running the current code in the ozone-runner image
regardless of the value of this `cluster_version` variable. If we want to try
all cluster versions that support EC, we would need one line before the loop to
test against the current version in new-cluster.yaml, then iterate the past
versions in this loop with old-cluster.yaml. See lines 118-124 farther down in
this file for an example.
--
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]
