Skip to content

Commit c499075

Browse files
authored
chore: Update pipeline yaml, replace hardcoded registries to env's (#744)
* fix: replace hardcoded image env's to env's
1 parent 169d7d7 commit c499075

File tree

3 files changed

+13
-11
lines changed

3 files changed

+13
-11
lines changed

.pipelines/e2e-step-template.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -62,9 +62,9 @@ steps:
6262
export CNI_URL=https://$(ARTIFACT_STORAGE).blob.core.windows.net/acn-$(StorageID)/azure-vnet-cni-${{ parameters.clusterDefinitionCniBuildOS }}-amd64-$(Tag)${{ parameters.clusterDefinitionCniBuildExt }}
6363
export CNI_TYPE=${{ parameters.clusterDefinitionCniTypeKey }}
6464
echo CNI type is $CNI_TYPE
65-
cat '${{ parameters.clusterDefinition }}' | jq --arg cnikey $CNI_TYPE --arg cniurl $CNI_URL '.properties.orchestratorProfile.kubernetesConfig[$cnikey]= $cniurl' > '${{ parameters.clusterDefinition }}'.tmp #
65+
cat '${{ parameters.clusterDefinition }}' | jq --arg cnikey $CNI_TYPE --arg cniurl $CNI_URL '.properties.orchestratorProfile.kubernetesConfig[$cnikey]= $cniurl' > '${{ parameters.clusterDefinition }}'.tmp
6666
cat '${{ parameters.clusterDefinition }}'.tmp | jq --arg tag $(Tag) '.properties.orchestratorProfile.kubernetesConfig.azureCNIVersion = $tag' > '${{ parameters.clusterDefinition }}'
67-
cat '${{ parameters.clusterDefinition }}' | jq --arg npmimage $PULL_CONTAINER_REGISTRY/azure-npm:$(Tag)-test '.properties.orchestratorProfile.kubernetesConfig.addons[0].containers[0].image = $npmimage' > '${{ parameters.clusterDefinition }}'.tmp
67+
cat '${{ parameters.clusterDefinition }}' | jq --arg npmimage $IMAGE_REGISTRY/azure-npm:$(Tag)-test '.properties.orchestratorProfile.kubernetesConfig.addons[0].containers[0].image = $npmimage' > '${{ parameters.clusterDefinition }}'.tmp
6868
mv '${{ parameters.clusterDefinition }}'.tmp '${{ parameters.clusterDefinition }}'
6969
echo "Running E2E tests against a cluster built with the following API model:"
7070
cat '${{ parameters.clusterDefinition }}'

.pipelines/pipeline.yaml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -126,9 +126,9 @@ stages:
126126
condition: always()
127127
128128
- script: |
129-
echo '$(DOCKER_PASSWORD)' | docker login containernetworking.azurecr.io -u '$(DOCKER_USERNAME)' --password-stdin
130-
docker tag $PUSH_CONTAINER_REGISTRY/azure-npm:$(EnvironmentalVariables.Tag) $PUSH_CONTAINER_REGISTRY/azure-npm:$(EnvironmentalVariables.Tag)-test
131-
docker push $PUSH_CONTAINER_REGISTRY/azure-npm:$(EnvironmentalVariables.Tag)-test
129+
echo '$(DOCKER_PASSWORD)' | docker login $IMAGE_REGISTRY -u '$(DOCKER_USERNAME)' --password-stdin
130+
docker tag $IMAGE_REGISTRY/azure-npm:$(EnvironmentalVariables.Tag) $IMAGE_REGISTRY/azure-npm:$(EnvironmentalVariables.Tag)-test
131+
docker push $IMAGE_REGISTRY/azure-npm:$(EnvironmentalVariables.Tag)-test
132132
133133
function auto-retry()
134134
{
@@ -139,10 +139,10 @@ stages:
139139
printf "Attempt $i/$attempts - " && "$@" && break || sleep 3 && i=$[$i+1] && false
140140
done
141141
}
142-
auto-retry docker pull mcr.microsoft.com/containernetworking/azure-npm:$(EnvironmentalVariables.Tag)-test
142+
auto-retry docker pull $IMAGE_REGISTRY/azure-npm:$(EnvironmentalVariables.Tag)-test
143143
workingDirectory: "$(modulePath)"
144144
name: "mcrreplication"
145-
displayName: "Push NPM Image and Wait for MCR"
145+
displayName: "Push NPM Image and Wait for Repository"
146146
147147
- script: |
148148
cd ./output

Makefile

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -135,17 +135,19 @@ CNS_IMAGE_ARCHIVE_NAME = azure-cns-$(GOOS)-$(GOARCH)-$(VERSION).$(ARCHIVE_EXT)
135135
CNM_PLUGIN_IMAGE ?= microsoft/azure-vnet-plugin
136136
CNM_PLUGIN_ROOTFS = azure-vnet-plugin-rootfs
137137

138+
IMAGE_REGISTRY ?= acnpublic.azurecr.io
139+
138140
# Azure network policy manager parameters.
139-
AZURE_NPM_IMAGE = containernetworking.azurecr.io/public/containernetworking/azure-npm
141+
AZURE_NPM_IMAGE ?= $(IMAGE_REGISTRY)/azure-npm
140142

141143
# Azure CNI installer parameters
142-
AZURE_CNI_IMAGE = containernetworking.azurecr.io/public/containernetworking/azure-cni-manager
144+
AZURE_CNI_IMAGE = $(IMAGE_REGISTRY)/azure-cni-manager
143145

144146
# Azure vnet telemetry image parameters.
145-
AZURE_VNET_TELEMETRY_IMAGE = containernetworking.azurecr.io/public/containernetworking/azure-vnet-telemetry
147+
AZURE_VNET_TELEMETRY_IMAGE = $(IMAGE_REGISTRY)/azure-vnet-telemetry
146148

147149
# Azure container networking service image paramters.
148-
AZURE_CNS_IMAGE = containernetworking.azurecr.io/public/containernetworking/azure-cns
150+
AZURE_CNS_IMAGE = $(IMAGE_REGISTRY)/azure-cns
149151

150152
VERSION ?= $(shell git describe --tags --always --dirty)
151153
CNS_AI_ID = ce672799-8f08-4235-8c12-08563dc2acef

0 commit comments

Comments
 (0)