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.
--
Mailing list: https://launchpad.net/~canonical-hw-cert
Post to : [email protected]
Unsubscribe : https://launchpad.net/~canonical-hw-cert
More help : https://help.launchpad.net/ListHelp
