Skip to content

Commit

Permalink
fix ACR build logs (#319)
Browse files Browse the repository at this point in the history
* suppress ACR builds logs due to Azure/azure-cli#6408

* update aks client image, use --no-format in ACR build
  • Loading branch information
timwebster9 authored Oct 5, 2018
1 parent 195b681 commit a8309a4
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion src/uk/gov/hmcts/contino/azure/Acr.groovy
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ class Acr extends Az {
* stdout of the step
*/
def build(DockerImage dockerImage) {
this.az "acr build -r ${registryName} -t ${dockerImage.getShortName()} -g ${resourceGroup} ."
this.az "acr build --no-format -r ${registryName} -t ${dockerImage.getShortName()} -g ${resourceGroup} ."
}

/**
Expand Down
2 changes: 1 addition & 1 deletion test/uk/gov/hmcts/contino/azure/AcrTest.groovy
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ class AcrTest extends Specification {

then:
1 * steps.sh({it.containsKey('script') &&
it.get('script').contains("az acr build -r ${REGISTRY_NAME} -t ${IMAGE_NAME} -g ${REGISTRY_RESOURCE_GROUP} .") &&
it.get('script').contains("az acr build --no-format -r ${REGISTRY_NAME} -t ${IMAGE_NAME} -g ${REGISTRY_RESOURCE_GROUP} .") &&
it.containsKey('returnStdout') &&
it.get('returnStdout').equals(true)})
}
Expand Down
4 changes: 2 additions & 2 deletions vars/sectionCI.groovy
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ def call(params) {
Builder builder = pipelineType.builder

if (pl.dockerBuild) {
withDocker('hmcts/cnp-aks-client:az-2.0.45-kubectl-1.11.2', null) {
withDocker('hmcts/cnp-aks-client:az-2.0.46-kubectl-1.11.2', null) {
withSubscription(subscription) {
withRegistrySecrets {
def acr = new Acr(this, subscription, env.REGISTRY_NAME, env.REGISTRY_RESOURCE_GROUP)
Expand All @@ -94,7 +94,7 @@ def call(params) {

aksUrl = aksDeploy(dockerImage, params)
log.info("deployed component URL: ${aksUrl}")

githubUpdateDeploymentStatus(deploymentNumber, aksUrl)
}
}
Expand Down

0 comments on commit a8309a4

Please sign in to comment.