Exit if run was not queued

[galargh]

I've noticed a situation where the action would hang forever (6hrs) if it failed to queue a run.

That's why I think we should exit early if queuing a run failed. I decided to check if TGID was empty after scheduling to determine if it worked.

The alternative approach would be to set -e on the entire entrypoint.sh script. Personally, I'd prefer that but I do not know testground CLI well enough to judge if there are any situation in which we'd like to tolerate non-zero exit codes - which would prevent me from introducing this patch fix quickly.

Let me know what you think :)

Testing

Failure case: https://github.com/galargh/go-ipfs/runs/5231923598?check_suite_focus=true
Success case: https://github.com/galargh/go-ipfs/runs/5232065515?check_suite_focus=true

[lidel]

cc @coryschwartz thoughts?

[galargh]

@coryschwartz have you had a chance to look at it?

[galargh]

I created the repo fork in testground org and merged this change in there - https://github.com/testground/testground-github-action.

[coryschwartz]

Sorry about the delay!

But you know, it's probably better that this live in the testground repo anyway -- thanks for doing that.

I'll point to that repo in the README.

Thanks.