From 3feb4750e7d6206729d8abe6dc4d9b17e2948863 Mon Sep 17 00:00:00 2001 From: Prathamesh Musale Date: Thu, 2 Apr 2026 08:33:16 +0000 Subject: [PATCH] fix: external-stack sync test sed pattern and config assertion The spec file uses YAML format (key: value) not env format (key=value), so the sed substitution never matched. Also update the config log check to expect UPDATED since the spec was modified by the sync test. Co-Authored-By: Claude Opus 4.6 --- tests/external-stack/run-test.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/external-stack/run-test.sh b/tests/external-stack/run-test.sh index 54c6f18f..f6c313ae 100755 --- a/tests/external-stack/run-test.sh +++ b/tests/external-stack/run-test.sh @@ -135,7 +135,7 @@ original_marker_content=$(<$test_data_marker) # Verify deployment file exists and preserve its cluster ID original_cluster_id=$(grep "cluster-id:" "$test_deployment_dir/deployment.yml" 2>/dev/null || echo "") # Modify spec file to simulate an update -sed -i.bak 's/CERC_TEST_PARAM_1=PASSED/CERC_TEST_PARAM_1=UPDATED/' $test_deployment_spec +sed -i.bak 's/CERC_TEST_PARAM_1: PASSED/CERC_TEST_PARAM_1: UPDATED/' $test_deployment_spec # Run sync to update deployment files without destroying data $TEST_TARGET_SO_STACK deploy create --spec-file $test_deployment_spec --deployment-dir $test_deployment_dir --update # Verify the spec file was updated in deployment dir @@ -179,7 +179,7 @@ else exit 1 fi # Check the config variable CERC_TEST_PARAM_1 was passed correctly -if [[ "$log_output_3" == *"Test-param-1: PASSED"* ]]; then +if [[ "$log_output_3" == *"Test-param-1: UPDATED"* ]]; then echo "deployment config test: passed" else echo "deployment config test: FAILED"