Skip to content
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

Missing status label for GitHub check #2493

Closed
danielleadams opened this issue Dec 8, 2020 · 2 comments
Closed

Missing status label for GitHub check #2493

danielleadams opened this issue Dec 8, 2020 · 2 comments
Labels
ci-change PSA of configuration changes

Comments

@danielleadams
Copy link
Member

danielleadams commented Dec 8, 2020

When I opened a PR in nodejs/node (nodejs/node#36435), I started the node-test-pull-request job and one of the checks that I received is pictured in the following:

Screen Shot 2020-12-08 at 6 00 32 AM

The Details link to: https://ci.nodejs.org/job/node-test-commit-linux-containered/nodes=ubuntu1804_sharedlibs_debug_x64/23997/. The check says ${STATUS_LABEL} instead of the label that would have been received from Jenkins.

I'm not sure if this is a bug in GitHub or missing data that is updating the PR.

Update: Once the issue was resolved with the test, the status check is not getting marked as resolved/passed.

@richardlau
Copy link
Member

richardlau commented Apr 1, 2021

I've shuffled around how the GitHub status labels are named in the node-test-commit-linux-containered job so that STATUS_LABEL is set as early as possible in the job. This should hopefully prevent edge cases were the job fails before STATUS_LABEL is set in future jobs.

@richardlau richardlau added the ci-change PSA of configuration changes label Apr 1, 2021
@danielleadams
Copy link
Member Author

thanks @richardlau!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-change PSA of configuration changes
Projects
None yet
Development

No branches or pull requests

2 participants