zentol commented on a change in pull request #11314: [FLINK-16411][AZP] Use
google mvn mirror globally
URL: https://github.com/apache/flink/pull/11314#discussion_r398141636
##########
File path: tools/azure-pipelines/jobs-template.yml
##########
@@ -160,14 +160,17 @@ jobs:
echo "##vso[task.setvariable variable=PATH]$JAVA_HOME_11_X64/bin:$PATH"
displayName: "Set to jdk11"
condition: eq('${{parameters.jdk}}', 'jdk11')
- - script: ./tools/travis/setup_maven.sh
+ - script: |
+ source ./tools/ci/maven-utils.sh
+ setup_maven
+ echo "##vso[task.setvariable variable=M2_HOME]$M2_HOME"
Review comment:
then we may want to explicitly setup maven in the azure_controller.yml /
run-nightly-tests.sh files. Conceptually we _shouldn't_ as it is about setting
up the environment, but having to "manually" pass exported variables seems like
something we will eventually stumble over, and it already raises some questions.
For example, why are we only carrying over the `M2_HOME` variable, and not
`PATH`?
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
