rmetzger opened a new pull request #12268:
URL: https://github.com/apache/flink/pull/12268


   ## What is the purpose of the change
   
   Clean up the CI-related scripts in `tools/`.
   
   
   ## Brief change log
   
   For reviewing this change, I recommend starting from the `job-template.yml` 
file to see how the scripts are connected.
   
   - travis_watchdog.sh used to be a combination of things: test stage control 
(also python test invocation), debug artifact management (mostly uploading 
them), test timeout control. The biggest issue was how the python tests were 
integrated into that file. I moved the "watchdog" functionality into a separate 
file, and created a new `test_controller.sh`.
   - azure_controller.sh used to be the entry point for the CI system, 
controlling the compile stage. I moved most of the stuff into the 
`tools/ci/compile.sh`
   
   
   ## Verifying this change
   
   I have tested timing out builds (both for regular maven/surefire/java 
timeouts and python) to make sure the refactored watchdog works, and exit codes 
are properly forwarded.
   
   Once the PR has reached an acceptable state, I will also test the nightly 
builds on my personal Azure account to make sure the python wheels definition 
works.
   
   The separation of changes into separate commits is not optimal (some YARN 
changes are a bit unrelated in the refactoring commit)
   


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