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

Fix tests failing for accessing http://repo1.maven.org/ #117

Merged
merged 4 commits into from Mar 7, 2020

Conversation

ikedam
Copy link
Member

@ikedam ikedam commented Feb 29, 2020

https://issues.jenkins-ci.org/browse/JENKINS-61276

It's caused for pom files in zip.
It's difficult to review, then

  • Extract files from the zip file in the source code repository, and construct them into a zip file when running tests.
  • Fix that extracted files to point to https sites.

Comment on lines +16 to +27
<repositories>
<repository>
<id>central</id>
<url>https://repo.maven.apache.org/maven2</url>
</repository>
</repositories>
<pluginRepositories>
<pluginRepository>
<id>central</id>
<url>https://repo.maven.apache.org/maven2</url>
</pluginRepository>
</pluginRepositories>
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added this to all pom files.
Adding this only to the root pom doesn't resolve the problem. It might caused for submodules doesn't inherit the parent module explicitly. (I don't know about detailed behavior of maven)
Adding this only to moduleA might resolve the problem as the artifacts will be downloaded for moduleA, and cached and reused for moduleB, moduleC. But that's just a workaround and not a good solution.

@ikedam ikedam changed the title WIP: Fix tests failing for accessing http://repo1.maven.org/ Fix tests failing for accessing http://repo1.maven.org/ Feb 29, 2020
@ikedam ikedam merged commit 80e5e08 into jenkinsci:master Mar 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
1 participant