Merge pull request #353 from NHSDigital/AMB-783-fix-monitoring-default-503-status-response

matthewaclark21 · web-flow · commit 439c11c509df · 2021-08-05T09:17:48.000+01:00
AMB-783 Readded changing enabling monitoring to false when _ping or _…
diff --git a/azure/templates/deploy-service.yml b/azure/templates/deploy-service.yml
@@ -300,6 +300,7 @@ steps:
     
       if [[ $ping_endpoint_response != "200" ]]; then
         echo "##vso[task.logissue type=error]Your proxy doesn't have a _ping endpoint therefore we can't monitor this proxy and it should not be released, use the flag 'enable_monitoring=false' if your API doesn't support _ping healthcheck."
+        echo "##vso[task.setvariable variable=enable_ping_monitoring]false"
         exit 1       
       fi 
     displayName: 'Check _ping'
@@ -319,6 +320,7 @@ steps:
       
       if [[ $status_endpoint_response != "200" ]]; then
         echo "##vso[task.logissue type=error]Your proxy doesn't have a _status endpoint therefore we can't monitor this proxy and it should not be released, use the flag 'enable_status_monitoring=false' if your API doesn't support _status healthcheck."
+        echo "##vso[task.setvariable variable=enable_status_monitoring]false"
         exit 1            
       fi
       
