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

Don't wait for Spot pool nodes #72

Merged
merged 1 commit into from Aug 10, 2018
Merged

Conversation

aermakov-zalando
Copy link
Contributor

@aermakov-zalando aermakov-zalando commented Aug 10, 2018

Spot nodes can be terminated at any time, so we don't need to wait for replacement node to come up during the upgrade. This also allows CLM to proceed when AWS doesn't have capacity instead of getting stuck forever.

Signed-off-by: Alexey Ermakov <alexey.ermakov@zalando.de>
@aermakov-zalando
Copy link
Contributor Author

👍

@szuecs
Copy link
Member

szuecs commented Aug 10, 2018

Please check the travis build:

$ dep ensure -vendor-only
grouped write of manifest, lock and vendor: error while writing out vendor tree: failed to write dep tree: failed to export github.com/alecthomas/template: failed to fetch source for https://github.com/alecthomas/template: unable to get repository: Cloning into '/home/travis/gopath/pkg/dep/sources/https---github.com-alecthomas-template'...
POST git-upload-pack (165 bytes)
remote: Counting objects: 59, done.        
error: RPC failed; curl 56 GnuTLS recv error (-9): A TLS packet with unexpected length was received.
fatal: The remote end hung up unexpectedly
fatal: early EOF
fatal: unpack-objects failed
: command failed: [git clone --recursive -v --progress https://github.com/alecthomas/template /home/travis/gopath/pkg/dep/sources/https---github.com-alecthomas-template]: exit status 128

@aermakov-zalando
Copy link
Contributor Author

@szuecs I think Travis is having a bad day today.

@njuettner
Copy link
Contributor

👍

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.

None yet

3 participants