rmetzger commented on a change in pull request #12268:
URL: https://github.com/apache/flink/pull/12268#discussion_r429909137
##########
File path: tools/azure-pipelines/jobs-template.yml
##########
@@ -121,15 +128,34 @@ jobs:
continueOnError: true # continue the build even if the cache fails.
condition: not(eq('${{parameters.test_pool_definition.name}}', 'Default'))
displayName: Cache Maven local repo
+
- script: |
echo "##vso[task.setvariable variable=JAVA_HOME]$JAVA_HOME_11_X64"
echo "##vso[task.setvariable variable=PATH]$JAVA_HOME_11_X64/bin:$PATH"
displayName: "Set to jdk11"
condition: eq('${{parameters.jdk}}', 'jdk11')
+
- script: sudo sysctl -w kernel.core_pattern=core.%p
displayName: Set coredump pattern
+
# Test
- - script: STAGE=test ${{parameters.environment}}
./tools/azure-pipelines/azure_controller.sh $(module)
+ - script: |
+ ./tools/azure-pipelines/unpack_build_artifact.sh
+ export DEBUG_FILES="$AGENT_TEMPDIRECTORY/debug_files"
Review comment:
I considered it, but I had the feeling that these steps are still short
enough to keep them in the template.yml.
If you think it would improve readability, can move them into script.
----------------------------------------------------------------
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
