Skip to content
This repository has been archived by the owner on May 3, 2024. It is now read-only.

CORTX-33718: Disable 37protocol till release time #2030

Merged
merged 8 commits into from
Aug 4, 2022
Merged
Show file tree
Hide file tree
Changes from 4 commits
Commits
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
9 changes: 8 additions & 1 deletion .xperior/testds/motr-single_tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -490,7 +490,14 @@ Tests:
- id : 37protocol
script : 'm0 run-st 37protocol'
dir : src/scripts
executor : Xperior::Executor::MotrTest
# This ST confirms that no BE structures are changed
# in the new version of the code since that would
# create a mismatch of BE structures and thus cause
# corruption in case of upgrades. During development
# phase the BE structures are assumed to be modified
# and to avoid this ST from failing it is better to disable
# the ST until a GA release is done
kanchan-chaudhari marked this conversation as resolved.
Show resolved Hide resolved
executor : Xperior::Executor::Skip
sandbox : /var/motr/root/sandbox.st-37protocol
groupname: 01motr-single-node
polltime : 30
Expand Down
5 changes: 5 additions & 0 deletions st/protocol
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,11 @@ popd
echo "Comparing '$proto_dev' and '$proto_current' files"
$diff_cmd $proto_dev $proto_current
exit_code=$?
if [ $exit_code -ne 0 ]; then
echo -n "37protocol ST may fail if there are some local changes done to "
echo -n "the BE structures. please confirm these are intended changes to "
kanchan-chaudhari marked this conversation as resolved.
Show resolved Hide resolved
echo "the BE structures in the new version before ignoring this error"
kanchan-chaudhari marked this conversation as resolved.
Show resolved Hide resolved
fi

# remove test files
rm -rf $test_dir 2>&1> /dev/null
Expand Down