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.

Sign up for GitHub

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

Jump to bottom

Document spot instance behaviour #508

Merged
merged 3 commits into from Mar 29, 2023
Merged

Document spot instance behaviour #508

merged 3 commits into from Mar 29, 2023

Conversation

jucor
Copy link
Contributor

@jucor jucor commented Mar 24, 2023 •
edited

Describe the extra options available when using spot instances in the autoscaler.
Pasted and slightly edited the very helpful explanations from @jkhenning in the #clearml-community Slack: https://clearml.slack.com/archives/CTK20V944/p1678877442269469
Thanks!

@ainoam
Copy link
Collaborator

ainoam commented Mar 26, 2023

Thanks for the help @jucor.
Please add a single indentation to the new items as they are conditional on configuring spot instance.

@jucor
Copy link
Contributor Author

jucor commented Mar 27, 2023

@ainoam Done.

@ainoam
Copy link
Collaborator

ainoam commented Mar 28, 2023

Thanks @jucor.
Looks like you mistakenly indented the 'Availability Zone' instead of 'Regular Instance Rollback Timeout'?

@jucor
Redo indent
326cc93 
Now with the right lines.
@jucor
Copy link
Contributor Author

jucor commented Mar 28, 2023

@ainoam Re-done.

@ainoam
Copy link
Collaborator

ainoam commented Mar 29, 2023

Many thanks @jucor!!

@ainoam ainoam merged commit a869990 into allegroai:main Mar 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@jucor @ainoam