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

Prevent over-provisioning #622

Merged
merged 1 commit into from
Feb 22, 2018
Merged

Conversation

pjdarton
Copy link
Member

Jenkins can keep on asking for more executors even though it has enough slaves promised to it that will satisfy demand once they come online.
We already keep track of the number of slaves that we are in the process of creating but have yet to give to Jenkins, so if we include that count when determining demand, we can ensure we do not provision more
containers than Jenkins really wanted.

Note: This is mostly useful if the docker host is being slow at creating containers, causing Jenkins to ask again before we've created the last bunch.

Jenkins can keep on asking for more executors even though it has enough
slaves promised to it that will satisfy demand once they come online.
We already keep track of the number of slaves that we are in the process
of creating but have yet to give to Jenkins, so if we include that count
when determining demand, we can ensure we do not provision more
containers than permitted.
@pjdarton
Copy link
Member Author

@ndeloof I've not had much of a chance to test this beyond ensuring that it doesn't seem to do any harm ... but I think this is a pretty important stability enhancement.
I'm going to be away until March 5th so, if you have the time ;-) then please do test this and #623 and see if you agree.
(Normally, I'd be in a position to try these out at work for a while and then I'd merge them if all seemed well, but as I'm going away for a week then that won't happen any time soon. So, if you've got the time... if not, I'll pick up where I left off when I get back)

@ndeloof ndeloof merged commit 47c038a into jenkinsci:master Feb 22, 2018
@pjdarton
Copy link
Member Author

Personally I wasn't going to press the "merge" button on this until I was sure it was a good change, but I guess as long as we don't release it until we can be sure that it's all good then that'll suffice.

@ndeloof
Copy link
Contributor

ndeloof commented Feb 22, 2018

git revert is your best friend if something sounds weird during pre-release testing :)

@pjdarton
Copy link
Member Author

pjdarton commented Feb 22, 2018

OK, I'll take your word for it - personally I'm not hugely comfortable with git beyond "commit" and "push" :-)

FYI I've added a few RFEs (#624, #625, #626) that I intend to start work sometime next month (probably in that order). If you have any opinions on them then please do share - I don't want to take things in a direction you disapprove of, and I'd also really value having a second opinion - it's all too easy to get "tunnel vision" and end up going down the wrong path sometimes...

@pjdarton pjdarton deleted the stop-over-provisioning branch March 6, 2018 18:38
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.

2 participants