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

Problem in conjunction with configuration history plugin #88

Closed
christofdamian opened this issue May 20, 2015 · 6 comments
Closed

Problem in conjunction with configuration history plugin #88

christofdamian opened this issue May 20, 2015 · 6 comments

Comments

@christofdamian
Copy link

I am using https://wiki.jenkins-ci.org/display/JENKINS/JobConfigHistory+Plugin and together with ghprb it seems to generate a backup of the ghprb configuration on every pull request. With 1.2M each this quickly generated 19GB of data on our installation at cloudbees.
I am not sure what makes ghprb different from other plugins in this respect.

@KostyaSha
Copy link
Member

Jenkins core and ghprb code design makes this. Jenkins store data in deseralised xmls and JobConfigHistory just saving any changes, that are constant in ghprb plugin.

@christofdamian
Copy link
Author

For now I removed JobConfigHistory, because I can't live without ghprb

@benpatterson
Copy link
Member

Hi @christofdamian We also use both the ghprb plugin and the jobHistory plugin. We encountered this problem, because it was making a significant dent in our disk usage.

We got around it by putting a pattern in the master configuration. Specifically, we went to Manage Jenkins > Configure > JobConfigHistory > Advanced, and put a pattern in the "System configuration exclude file pattern" field.

I honestly don't remember if there were any pre-filled values in that field, but here's our entire field as an FYI: queue|nodeMonitors|UpdateCenter|global-build-stats|GhprbTrigger. I don't remember if it was pre-filled when we adjusted it...but you can see the last pipe handles this plugin. A couple of the others there were also taking up unnecessary space (at least, for us).

@christofdamian
Copy link
Author

thanks @benpatterson ! I'll try that, sounds like a good and easy solution.

@bjoernhaeuser
Copy link
Contributor

@christofdamian are you interested in providing a documentation update for this issue?

@bjoernhaeuser
Copy link
Contributor

For now i am closing this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants