zuul/nodepool
Code Issues Proposed changes

Fix min-ready/max-servers in test configs

Browse Source 
We should never have providers in our test configs that have a
max-servers value that is LESS than the min-ready. This could
potentially cause our tests to hang (and thus fail) if such a
provider were to try and satisfy all of the min-ready requests
because that provider will pause waiting for nodes to free up.

Change-Id: I6a30cacbe97130958c2d22ff56e760549fcc6bc1
This commit is contained in:
David Shrewsbury
2017-03-14 10:18:34 -04:00
parent 691b4c95a7
commit 1fdb050207
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
nodepool/tests/fixtures/node_upload_fail.yaml vendored
View File

@@ -26,7 +26,7 @@ providers:
password: 'fake'
auth-url: 'fake'
project-id: 'fake'
max-servers: 1
max-servers: 2
pool: 'fake'
networks:
- net-id: 'some-uuid'