Skip to content

Commit da88f70

Browse files
sxarichardlau
authored andcommitted
doc: add note on visibility of CI failures to new contributor guide
Signed-off-by: Stewart X Addison <sxa@ibm.com> PR-URL: #64256 Reviewed-By: Richard Lau <richard.lau@ibm.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: Ulises Gascón <ulisesgascongonzalez@gmail.com>
1 parent 20ce359 commit da88f70

1 file changed

Lines changed: 9 additions & 0 deletions

File tree

doc/contributing/first-contributions.md

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -168,6 +168,15 @@ before approving the CI runs. Similar to the review process, be patient and resp
168168
the volunteers' time when you are asking for help to trigger the CI runs, this helps build
169169
trust to make future contributions smoother.
170170

171+
### Q: The Jenkins CI failed, but when I clicked the CI link, it shows "Access Denied". What should I do?
172+
173+
Read access to the Jenkins CI is restricted to a few teams in the Node.js organization. You will
174+
need to ask a member of these teams (for example, any of the collaborators) to show you the details
175+
of the failures. If after a day or so from a collaborator triggering a Jenkins CI your PR shows a
176+
failure in either the `node-test-pull-request` or `node-test-commit-*` checks it's worth adding a
177+
comment to the PR asking what tests are failed, as the collaborator may not realise that you cannot
178+
see the results directly.
179+
171180
### Q: The CI runs showed some failures that seem unrelated to my change. What should I do?
172181

173182
The Node.js CIs are known to be flaky. Consult the daily reports in

0 commit comments

Comments
 (0)