canonical-hw-cert team mailing list archive

[Nadzeya Hutsko <mp+436237@xxxxxxxxxxxxxxxxxx>]

Looks fine, just a minor question

Diff comments:

> diff --git a/jobs/image-testing/run-image-testing-server.sh b/jobs/image-testing/run-image-testing-server.sh
> index f0e04b2..547f6a9 100755
> --- a/jobs/image-testing/run-image-testing-server.sh
> +++ b/jobs/image-testing/run-image-testing-server.sh
> @@ -197,6 +197,6 @@ if [ -f artifacts/submission.json ]; then
>      cat artifacts/submission.json |jq -r '. | {{results: .results}}' > results.json
>      curl -s "https://testflinger-revcache.canonical.com/v1/results/$JOB_NAME"; -X POST --header "Content-Type: application/json" --data @results.json
>  fi
> -flock /tmp/TRELLO-IMAGE-UPDATE.LOCK certification-dashboard-manager.dashboard-manager --app trello imageupdater -n {name} -i {image_file} -c {image_channel} -v $BUILD_ID --summaryfile artifacts/summary
> +flock /tmp/TRELLO-UPDATE.LOCK certification-dashboard-manager.dashboard-manager --app trello imageupdater -n {name} -i {image_file} -c {image_channel} -v $BUILD_ID --summaryfile artifacts/summary

Is TRELLO_UPDATE_LOCK a common name for all DM executions?

>  echo "Test exit status: $TEST_STATUS"
>  exit $EXITCODE


-- 
https://code.launchpad.net/~jocave/hwcert-jenkins-jobs/+git/hwcert-jenkins-jobs/+merge/436237
Your team hwcert-jenkins is requested to review the proposed merge of ~jocave/hwcert-jenkins-jobs:trello-flock-everywhere into hwcert-jenkins-jobs:master.