-
-
Notifications
You must be signed in to change notification settings - Fork 137
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
feat(buildPlugin): switch artifact caching proxy to opt-out #577
feat(buildPlugin): switch artifact caching proxy to opt-out #577
Conversation
@@ -338,11 +338,27 @@ class BuildPluginStepTests extends BaseTest { | |||
} | |||
|
|||
@Test | |||
void test_buildPlugin_with_artifact_caching_proxy_enabled_and_empty_provider_specified() throws Exception { | |||
void test_buildPlugin_with_artifact_caching_proxy_disabled() throws Exception { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Additional case to test when useArtifactCachingProxy
is set to false
End to end test of this PR: jenkinsci/jenkins-infra-test-plugin#63 as a safety net. (edit) success \o/ Ready to go! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. This PR is mergeable once the announced date / time of this change is reached
When this PR will be merged, every plugin using
buildPlugin
will use one of the artifact caching proxy providers instead of repo.jenkins-ci.org and Maven Central to download its Maven artifacts.Successfully tested in jenkinsci/jenkins-infra-test-plugin#62
Close jenkins-infra/helpdesk#2752
Ref: jenkins-infra/helpdesk#3369