-
-
Notifications
You must be signed in to change notification settings - Fork 11
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
Publish pipeline-steps-doc-generator
and backend-extension-indexer
artifacts to some kind of storage
#3087
Comments
@timja @daniel-beck do you agree with the following proposal?
|
sounds good |
Yes, absolutely, this has been terrible for a long time. We didn't have reports.j.io when this was set up, so this was a pragmatic way to do this.
This has bitten us once or twice; rather than ignoring some bullshit error on startup we have to urgently figure out what was wrong. Every time I complain about the 8 minute startup time of ci.j.io, this dependency from jenkins.io builds is why 😉 |
…eps-doc-generator as part of jenkins-infra/helpdesk#3087 Signed-off-by: Damien Duportal <[email protected]>
…eps-doc-generator as part of jenkins-infra/helpdesk#3087 (#2972)
next step will be to remove the 2 jobs from ci.jenkins.io. |
Both pipeline marked as archived in ci.jenkins.io (removed from the "include list" trait of the GH org scanning named |
To clarify, there is no value in having a public CI there? |
They definitely should have public CI |
ack, reopening to enable again public CI then |
in the process of merging the pipeline files from ci and infra.ci (to provide a public output), I will try to take the process |
…erge) as per jenkins-infra/helpdesk#3087 in order to merge the pipeline we need to use the same label on both controllers.
…erge) (#2499) as per jenkins-infra/helpdesk#3087 in order to merge the pipeline we need to use the same label on both controllers.
Just added back the 2 repositories to ci.jenkins.io in https://ci.jenkins.io/job/Infra/ (updated the GH organization scanning job). Scanning in progress, that should add build checks for jenkins-infra/backend-extension-indexer#46 and jenkins-infra/pipeline-steps-doc-generator#241 |
|
jenkins-infra/helpdesk#3087 fixed the issue that the documentation site build process was unnecessarily dependent on ci.jenkins.io. The zip file is now downloaded from reports.jenkins.io.
jenkins-infra/helpdesk#3087 fixed the issue that the documentation site build process was unnecessarily dependent on ci.jenkins.io. The zip file is now downloaded from reports.jenkins.io.
Service(s)
infra.ci.jenkins.io
Summary
https://github.com/jenkins-infra/jenkins.io/blob/9addc32ac8d366e02a9ec49e07e81ab527547cf4/scripts/fetch-external-resources#L36
and
https://github.com/jenkins-infra/jenkins.io/blob/9addc32ac8d366e02a9ec49e07e81ab527547cf4/scripts/fetch-external-resources#L66
cause site publishing to be reliant on ci.jenkins.io being up.
Which makes ci.jenkins.io a critical path for publishing security advisories and for contributors developing local site builds.
We should decouple these by publishing to some kind of storage.
Either
reports.jenkins.io
or an Azure storage, these need to be published publicly somewhere that any contributors machine can download fromsee also #3086 (comment)
Reproduction steps
No response
The text was updated successfully, but these errors were encountered: