[GitHub] [flink] rmetzger commented on a change in pull request #14391: [FLINK-20325][build] Move docs_404_check to CI stage

[GitBox <gi...@apache.org>]

rmetzger commented on a change in pull request #14391:
URL: https://github.com/apache/flink/pull/14391#discussion_r544129022



##########
File path: tools/azure-pipelines/build-apache-repo.yml
##########
@@ -68,6 +68,35 @@ stages:
           run_end_to_end: false
           container: flink-build-container
           jdk: jdk8
+      - job: docs_404_check # run on a MSFT provided machine
+        pool:
+          vmImage: 'ubuntu-16.04'
+        steps:
+          # Skip docs check if this is a pull request doesn't contain documentation change
+          - bash: |
+              source ./tools/azure-pipelines/build_properties.sh
+              contains_docs_pullrequest
+              if [[ "$?" == 0 ]] ; then
+                echo "##[debug]This is a pull request doesn't contain documentation. Skipping docs check."
+                echo "##vso[task.setvariable variable=skip;]1"
+              else
+                echo "##[debug]This is a regular CI build. Continuing ..."

Review comment:
       I rephrased both messages. Since this is scoped to the docs_404_check / "Check if PR contains docs change", it will be clear what's going on. It won't confuse people checking the regular build output, because it won't show there.
   




----------------------------------------------------------------
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:
users@infra.apache.org