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

upgrade to jcasc test harness #76

Merged
merged 7 commits into from
Jan 20, 2020
Merged

upgrade to jcasc test harness #76

merged 7 commits into from
Jan 20, 2020

Conversation

jetersen
Copy link
Member

@jetersen jetersen commented Jan 13, 2020

JCasc had to create a new dependency that wasn't a tests classifier because maven doesn't allow you to bring dependencies through classifiers.

Upgrading here so that PCT works again,

Tracking:
jenkinsci/bom#164

Note: We'll need a release for PCT please

Copy link
Member

@jglick jglick left a comment

Choose a reason for hiding this comment

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

(once tests pass)

@imod imod self-requested a review January 14, 2020 09:55
@imod
Copy link
Member

imod commented Jan 14, 2020

@Casz tests failed :(

@jetersen
Copy link
Member Author

Let's try bumping the maven plugin 🤔 wagon was updated in newer releases

@jetersen
Copy link
Member Author

@imod

What is this used for?

        <dependency>
            <groupId>com.google.code.findbugs</groupId>
            <artifactId>annotations</artifactId>
            <version>3.0.0</version>
            <scope>provided</scope>
        </dependency>

@imod
Copy link
Member

imod commented Jan 14, 2020

@Casz probably some ancient stuff - don't think its used anymore

@jetersen
Copy link
Member Author

It's like a gift that just give on giving 😆

@jetersen
Copy link
Member Author

hmm need some exclusion magic instead... meaning I need to open my IDE i'll do it later this in the evening

@jetersen
Copy link
Member Author

I have Allow edits from maintainers if you feel like fixing it @imod 🙇

@imod
Copy link
Member

imod commented Jan 14, 2020

I might be able to fix it in the later afternoon - let's see...

@jetersen
Copy link
Member Author

jetersen commented Jan 14, 2020

Hmm seems like a bug tbh

Jenkins core uses commons-io 2.4
Yet maven interceptors is using newer...
https://github.com/jenkinsci/maven-interceptors/blob/master/pom.xml
so is org.apache.maven.shared:maven-shared-utils
perhaps updating Jenkins Core might align the version?

@imod
Copy link
Member

imod commented Jan 14, 2020

sure we can - I don't mind

@jglick
Copy link
Member

jglick commented Jan 14, 2020

maven interceptors

Oof, I thought we were using that stuff only in maven-plugin hell…?

@jetersen
Copy link
Member Author

🤷‍♂ The mavenSettingsMustBeFoundInFreestyleProject test passes locally.

@jetersen
Copy link
Member Author

Seems like a super flaky test.

@imod
Copy link
Member

imod commented Jan 20, 2020

@Casz the test case is quite ok, its actually an infrastructure issue - a required plugin could not been downloaded:

Plugin org.apache.maven.plugins:maven-clean-plugin:2.4.1 or one of its dependencies could not be resolved: Could not transfer artifact org.apache.maven.plugins:maven-clean-plugin:jar:2.4.1 from/to central (http://repo1.maven.org/maven2): Error transferring file: Server returned HTTP response code: 501 for URL: http://repo1.maven.org/maven2/org/apache/maven/plugins/maven-clean-plugin/2.4.1/maven-clean-plugin-2.4.1.jar -> [Help 1]

@imod imod closed this Jan 20, 2020
@imod imod reopened this Jan 20, 2020
@imod
Copy link
Member

imod commented Jan 20, 2020

@Casz ah, for some reason it tries to download the maven-clean-plugin via http from maven-central and thats not supported by sonatype anymore - but to be honest, I have no idea why this is the case, I guess this should be fixed by some infrastructure (it seems not to be caused by the parent: https://repo.jenkins-ci.org/releases/org/jenkins-ci/plugins/plugin/3.56/)

@jetersen
Copy link
Member Author

Could we disable the test for now? As I know it works locally 🤔

@imod
Copy link
Member

imod commented Jan 20, 2020

@Casz yes we can - I'm about to look into it. It seems to be related to the maven build which is triggered by the test case it self. For some reason the maven build triggered by the test case uses http instead of https - but that uses some internal stuff I have never looked at. ...I'm on it, you can disable the test for now.

@imod
Copy link
Member

imod commented Jan 20, 2020

I'm able to reproduce the issue when I do this:

mv ~/.m2/repository/org/apache/maven/plugins ~/.m2/repository/org/apache/maven/plugins_old
mvn verify

@imod
Copy link
Member

imod commented Jan 20, 2020

@Casz I fixed the issue with maven and http/https, but now it fails with an other error I don't quite understand yet - locally its still working fine

@jetersen jetersen changed the title fix jcasc test harness upgrade to jcasc test harness Jan 20, 2020
@jetersen
Copy link
Member Author

@jetersen
Copy link
Member Author

poking CI

@jetersen jetersen closed this Jan 20, 2020
@jetersen jetersen reopened this Jan 20, 2020
@jetersen
Copy link
Member Author

@imod seems we have a green build 🙇

@imod
Copy link
Member

imod commented Jan 20, 2020

yeah, let's get this in then... - thanks!!

@imod imod merged commit 0a38b82 into jenkinsci:master Jan 20, 2020
@jetersen jetersen deleted the fix/jcasc-test-harness branch January 20, 2020 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants