[JENKINS-73242] Don't fail organization scan if a repository errors#790
[JENKINS-73242] Don't fail organization scan if a repository errors#790solonovamax wants to merge 1 commit intojenkinsci:masterfrom
Conversation
|
I have not added a test case for this, as I'm not too familiar with the test framework. also, please do let me know if there is anything I should change/etc. as this is my first contribution to a jenkins plugin. |
|
hey, is there anything blocking this from being merged? I've been forced to use a locally built version of the plugin, as this is a major issue to me, and having to repeatedly go and rebuild on every update is incredibly annoying. |
See [JENKINS-73242](https://issues.jenkins.io/browse/JENKINS-73242). Signed-off-by: solonovamax <solonovamax@12oclockpoint.com>
|
hey, @olamy very sorry to bother you, but I'm mentioning you because you seem to be the person merging most of the recent PRs. would it be possible to get it merged (or get some feedback on what needs to be changed for it to be merged), as this is a major issue for me, forcing me to use my own custom build of the plugin rather than being able to use the normal version. |
Description
Fixes #1487
Don't fail organization scan if a repository errors. See JENKINS-73242 for further information.
Implementation details:
Before attempting any checks against the repository, run
GHRepository.getSource(). This will cause the repository to attempt to populate all the needed data.Sometimes this can fail, such as in the case where a repository was DMCA'd.
If it fails, catch that error and skip the repository.
Submitter checklist
Reviewer checklist
Documentation changes
Users/aliases to notify