LinyuYao1021 commented on pull request #14737:
URL: https://github.com/apache/flink/pull/14737#issuecomment-793552614


   > What we know is that this approach works well for the S3 tests.
   
   Okay, I got it. Theoretically, variables transformation should work. Then 
the problem should still come from condition check. I've put the following code 
back to package in the latest commit. 
   ```
   SECRET_S3_ACCESS_KEY: $[variables.IT_CASE_S3_ACCESS_KEY]
   SECRET_S3_SECRET_KEY: $[variables.IT_CASE_S3_SECRET_KEY]
   ```
   Meanwhile, I changed
   ```
   if [ -n "$IT_CASE_GLUE_SCHEMA_ACCESS_KEY" ] && [ -n 
"$IT_CASE_GLUE_SCHEMA_SECRET_KEY" ]; then
     run_test "AWS Glue Schema Registry nightly end-to-end test" 
"$END_TO_END_DIR/test-scripts/test_glue_schema_registry.sh"
   fi
   ```
   to
   ```
   if [[ -n "$IT_CASE_GLUE_SCHEMA_ACCESS_KEY" ]] && [[ -n 
"$IT_CASE_GLUE_SCHEMA_SECRET_KEY" ]]; then
     run_test "AWS Glue Schema Registry nightly end-to-end test" 
"$END_TO_END_DIR/test-scripts/test_glue_schema_registry.sh"
   fi
   ```


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to