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

Jenkinsfile: Use -ntp in the Maven Build #4375

Merged
merged 3 commits into from
Nov 28, 2019
Merged

Conversation

jglick
Copy link
Member

@jglick jglick commented Nov 25, 2019

The PR build was polluted with download messages.

Note that the Maven tool is pretty old, so if we add back in Windows builds, it may not work unless we also do #4178.

@oleg-nenashev oleg-nenashev added the skip-changelog Should not be shown in the changelog label Nov 25, 2019
@oleg-nenashev oleg-nenashev changed the title -ntp Jenkinsfile: Use -ntp in the Maven Build Nov 25, 2019
@batmat
Copy link
Member

batmat commented Nov 26, 2019

ignoreUnknownOperations – hudson.cli.PlainCLIProtocolTest test failure.
I suspect test flakiness (?) -- should check other recent builds I guess

image

@jglick
Copy link
Member Author

jglick commented Nov 26, 2019

https://ci.jenkins.io/job/Core/job/jenkins/job/master/1654/testReport/hudson.cli/PlainCLIProtocolTest/Linux_jdk8___Linux_Publishing___ignoreUnknownOperations/ suggests this is a flake. That preceded the merge of #4220 so I do not think there is any relationship.

@fcojfernandez fcojfernandez added the ready-for-merge The PR is ready to go, and it will be merged soon if there is no negative feedback label Nov 28, 2019
@oleg-nenashev oleg-nenashev merged commit 038109d into jenkinsci:master Nov 28, 2019
@jglick jglick deleted the ntp branch December 2, 2019 20:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
internal ready-for-merge The PR is ready to go, and it will be merged soon if there is no negative feedback skip-changelog Should not be shown in the changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants