-
Notifications
You must be signed in to change notification settings - Fork 17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Reporter does not detect build number on Semaphore #55
Comments
Hey! I've merged your PR and released it under v0.3.5. Could you please test if it works? |
This is looking good to me, build numbers detected, info all flowing into Coveralls! 🍻 |
After some more usage today, I'm still seeing a problem executing Seems this issue needs to be re-opened. Sample failure with binary:
Sample success with curl:
Since the API response is the same in both cases, I'm not sure how to debug this. |
Hey! Could you add a curl -L https://coveralls.io/coveralls-linux.tar.gz | tar -xz && ./coveralls --debug --done It will print the payload it sends to coveralls.io Looks like there are some parameters that affect coveralls backend. It can be our bug for Semaphore CI. Your debug output would help to investigate the issue. |
Sure, here's the output:
The build_num looks the same as in my curl example that worked:
|
Hey! I've tried to reporoduce this bug but for my repo everything seems to work fine.
Which of these you don't have for your builds? You can check the PR here: mrexox/sentry-sanitizer#8 |
Hey @jaredmoody, do you still have this issue? Can you please check my recent comment about it? |
I tried again and I am seeing Coveralls report back to Github - maybe I just didn't wait long enough the first time 🤷 |
Here's my scenario that's not working:
--parallel
I attempted to call the webhook directly with curl and received a similar message:
Next I changed my first parallel call to directly pass the job id, which did use the $SEMAPHORE_WORKFLOW_ID when reporting with
--parallel
, but then still did not on the--done
step:Finally, what does work is calling
coveralls --parallel --service-job-id=$SEMAPHORE_WORKFLOW_ID
followed by manually calling the done webhook with curl.The text was updated successfully, but these errors were encountered: