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

Do not wait for checkpoint from previous build #10

Merged

Conversation

@mbektchiev
Copy link
Contributor

mbektchiev commented May 12, 2015

Fixes an issue with result archiver getting stuck for a long time in concurrent builds.
A similar issue has been fixed in the JUnit Jenkins plugin:

https://issues.jenkins-ci.org/browse/JENKINS-10234
jenkinsci/jenkins@90ff9f8

@jenkinsadmin

This comment has been minimized.

Copy link
Member

jenkinsadmin commented May 12, 2015

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

Fixes an issue with result archiver getting stuck for a long time in concurrent builds.
A similar issue has been fixed in the JUnit Jenkins plugin:

https://issues.jenkins-ci.org/browse/JENKINS-10234
jenkinsci/jenkins@90ff9f8
@mbektchiev mbektchiev force-pushed the mbektchiev:bektchiev/dont-block-for-previous-build branch from 1a00b97 to 1e89e02 May 12, 2015
@kinow

This comment has been minimized.

Copy link
Member

kinow commented May 12, 2015

Thanks for the PR @mbektchiev ! Will try to take a look at this issue tonight. Released CCM plug-in last night, maybe we can release NUnit this week too with this fix 😄

@mbektchiev

This comment has been minimized.

Copy link
Contributor Author

mbektchiev commented Jun 5, 2015

Hi @kinow, have you managed to take a look at this fix? We've been using it for almost a month now and haven't found any problems with it so far.

@kinow

This comment has been minimized.

Copy link
Member

kinow commented Jun 5, 2015

Hi @mbektchiev, not yet, just finished the mozilla science global sprint. Opening my Eclipse workspace to see if I can try it right now.

kinow added a commit that referenced this pull request Jun 5, 2015
…ous-build

Do not wait for checkpoint from previous build
@kinow kinow merged commit e45e236 into jenkinsci:master Jun 5, 2015
1 check passed
1 check passed
Jenkins This pull request looks good
Details
@kinow

This comment has been minimized.

Copy link
Member

kinow commented Jun 5, 2015

Looking good!! Thanks @mbektchiev ! Releasing a new version right now.

@kinow

This comment has been minimized.

Copy link
Member

kinow commented Jun 6, 2015

0.17 released. I'm going to try to update the Wiki docs (Wiki has been behaving oddly these past days), and it should be available through the update centre in the next hours. Thanks and sorry for the delay.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.