Azure / acr-builder

Azure Container Registry Build Runner
MIT License
38 stars 35 forks source link

provide clear error message when push step is timed out #566

Open northtyphoon opened 4 years ago

northtyphoon commented 4 years ago

Describe the bug

If push step was timed out, it retried 3 times and return an error message saying the push failed. The error message didn't point out it's a timeout issue.

2020/09/03 07:25:44 Pushing image: XYZ, attempt 2 2020/09/03 07:25:45 Pushing image: XYZ, attempt 3 failed to run step ID: acb_step_1: failed to push images successfully Run ID: cc1rej failed after 16m17s. Error: failed during run, err: exit status 1

shahzzzam commented 4 years ago

"failed to do something successfully" also looks odd. I recommend removing "successfully"