From 86c6a1f8322d7dfa5740bf2414a369eae6c70ed9 Mon Sep 17 00:00:00 2001
From: Tom Care <tcare@microsoft.com>
Date: Mon, 9 Mar 2020 16:10:56 -0700
Subject: [PATCH] Canary pipeline fixes (#224)

* Add vars template to canary pipeline

* Enable ACR authentication on AKS using a service principal
- Upgrade helm version to 3.1.1
- Remove ACR secret from the abtest-model deployment
---
 .pipelines/abtest.yml                         | 3 ++-
 charts/abtest-model/templates/deployment.yaml | 3 +--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/.pipelines/abtest.yml b/.pipelines/abtest.yml
index 0d64e8c8..cf876181 100644
--- a/.pipelines/abtest.yml
+++ b/.pipelines/abtest.yml
@@ -18,9 +18,10 @@ trigger:
     - ml_service/util/smoke_test_scoring_service.py
 
 variables:
+- template: diabetes_regression-variables-template.yml
 - group: 'devopsforai-aml-vg'
 - name: 'helmVersion'
-  value: 'v3.0.1'
+  value: 'v3.1.1'
 - name: 'helmDownloadURL'
   value: 'https://get.helm.sh/helm-$HELM_VERSION-linux-amd64.tar.gz'
 - name: 'blueReleaseName'
diff --git a/charts/abtest-model/templates/deployment.yaml b/charts/abtest-model/templates/deployment.yaml
index 13572b36..78d01cc4 100644
--- a/charts/abtest-model/templates/deployment.yaml
+++ b/charts/abtest-model/templates/deployment.yaml
@@ -27,5 +27,4 @@ spec:
           containerPort: 5001
         - name: probe
           containerPort: 8086
-      imagePullSecrets:
-        - name: aks-secret     
+