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

[JENKINS-32924] Add LogRotator to Workflow Multibranch #746

Closed
wants to merge 2 commits into from

Conversation

meip
Copy link
Contributor

@meip meip commented Feb 13, 2016

Jira ticket: https://issues.jenkins-ci.org/browse/JENKINS-32924

Add a LogRotator to Workflow Multibranch Jobs to rotate build artifacts.

@jenkinsadmin
Copy link
Member

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

@daspilker
Copy link
Member

Sorry for the late feedback. I'm running Jenkins 1.625 with Pipeline Multibranch 1.14 and Branch API 1.7. I can't find a setting for the strategy in the UI. What do I need to do to test this? Or has this become obsolete?

@daspilker
Copy link
Member

daspilker commented May 10, 2016

I made the BranchSourcesContext extensible (see #846) so that it will work with the Automatically Generated DSL. The embedded API viewer shows several options for strategy, but the UI does not show any strategy.

@meip
Copy link
Contributor Author

meip commented May 13, 2016

I think this property is gone from the "root" config and should better be configured via Pipeline script. The Pipeline Multibranch is a bit messed up with properties inherited to branches. See discussion here:
https://issues.jenkins-ci.org/browse/JENKINS-30519
https://issues.jenkins-ci.org/browse/JENKINS-34005

An example to do it within Jenkinsfile is here: https://github.com/jenkinsci/workflow-multibranch-plugin/blob/master/src/test/java/org/jenkinsci/plugins/workflow/multibranch/JobPropertyStepTest.java

As you can see there should also be an option to set it with the Config File Provider Plugin.

Considering these two options, and your remarks that it is no more possible to configure logRotation for Multibranch via Gui, I don't know if we should purge this PR?

@daspilker
Copy link
Member

Ah, thanks for the pointers. Since logRotator is supposed to be configured in Jenkinsfile and not on the multi-branch job, I'm closing this PR.

@daspilker daspilker closed this May 13, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants