Dropping windows tests, but keeping cred-scan (#542)

Removing the Windows branch of our testing as we have been experiencing intermittent random failures in windows, which could be the result of a change in image on the test machines.

Closes #541

CHANGELOG.md

@@ -43,7 +43,8 @@ mounting and running matplotblib on some machines. Re-instantiated a disabled te
 
 ### Removed
 
-- ([#520](https://github.com/microsoft/InnerEye-DeepLearning/pull/520)) Disable glaucoma job from Azure pipeline.
+-  ([#542](https://github.com/microsoft/InnerEye-DeepLearning/pull/542)) Removed Windows test leg from build pipeline.
+-  ([#520](https://github.com/microsoft/InnerEye-DeepLearning/pull/520)) Disable glaucoma job from Azure pipeline.
 
 ### Deprecated
 

azure-pipelines/build-pr.yml

@@ -21,7 +21,7 @@ jobs:
     pool:
       vmImage: 'windows-2019'
     steps:
-      - template: build.yaml
+      - template: build_windows.yaml
 
   - job: Linux
     pool:

azure-pipelines/build.yaml

@@ -1,19 +1,6 @@
 steps:
   - template: checkout.yml
 
-  - task: CredScan@3
-    condition: and(succeeded(), eq( variables['Agent.OS'], 'Windows_NT' ))
-
-  - task: PostAnalysis@1
-    condition: and(succeeded(), eq( variables['Agent.OS'], 'Windows_NT' ))
-    displayName: 'Post Analysis'
-    inputs:
-      CredScan: true
-
-  - script: echo %NUMBER_OF_PROCESSORS%
-    condition: and(succeeded(), eq( variables['Agent.OS'], 'Windows_NT' ))
-    displayName: Print processors
-
   - bash: |
       conda env create --file environment.yml --name InnerEye --quiet
       source activate InnerEye

azure-pipelines/build_windows.yaml

@@ -0,0 +1,20 @@
+steps:
+  - template: checkout_windows.yml
+
+  - task: CredScan@3
+    condition: and(succeeded(), eq( variables['Agent.OS'], 'Windows_NT' ))
+
+  - task: PostAnalysis@1
+    condition: and(succeeded(), eq( variables['Agent.OS'], 'Windows_NT' ))
+    displayName: 'Post Analysis'
+    inputs:
+      CredScan: true
+
+  - task: ComponentGovernanceComponentDetection@0
+    condition: succeeded()
+    inputs:
+      scanType: 'Register'
+      verbosity: 'Normal'
+      alertWarningLevel: 'High'
+      failOnAlert: true
+      failOnStderr: true

azure-pipelines/checkout.yml

@@ -4,10 +4,7 @@ steps:
     submodules: true
 
   - bash: |
-      if [ $(Agent.OS) = 'Windows_NT' ]
-      then subdir=Scripts
-      else subdir=bin
-      fi
+      subdir=bin
       echo "Adding this directory to PATH: $CONDA/$subdir"
       echo "##vso[task.prependpath]$CONDA/$subdir"
     displayName: Add conda to PATH
@@ -19,7 +16,6 @@ steps:
       conda list
     displayName: Print conda version and initial package list
 
-  # Linux only; not needed for Windows
   - bash: |
       sudo chown -R $USER /usr/share/miniconda
     condition: and(succeeded(), eq( variables['Agent.OS'], 'Linux' ))

azure-pipelines/checkout_windows.yml

@@ -0,0 +1,17 @@
+steps:
+  - checkout: self
+    lfs: true
+    submodules: true
+
+  - bash: |
+      subdir=Scripts
+      echo "Adding this directory to PATH: $CONDA/$subdir"
+      echo "##vso[task.prependpath]$CONDA/$subdir"
+    displayName: Add conda to PATH
+    condition: succeeded()
+
+  - bash: |
+      conda install conda=4.8.3 -y
+      conda --version
+      conda list
+    displayName: Print conda version and initial package list