-
Notifications
You must be signed in to change notification settings - Fork 144
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
[DOCS] Clarify AI Assistant will spawn ML nodes #3687
base: 8.12
Are you sure you want to change the base?
Conversation
Clarify AI Assistant will spawn ML nodes
A documentation preview will be available soon: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for updating the docs. Just some minor changes to follow Elastic style and avoid what grammarians call an unclear antecedent.
@lucabelluccini Do we want to port this change to the serverless docs, too? |
Co-authored-by: DeDe Morton <dede.morton@elastic.co>
Hello @dedemorton - thanks for the review. I think we need some inputs from engineering as:
|
@dgieselaar Can you respond to Lucas' comments here so we can finalize these changes? Thanks! (My apologies if I'm pinging the wrong person here. I'm still trying to figure out who knows what on the team.) |
I assume you meant to say 'disabling AI Assistant / KB' ? The AI Assistant will not explicitly scale down ML nodes if it is disabled.
I'm unfamiliar with how autoscaling works in ML, but happy to ask around: https://elastic.slack.com/archives/C2AJLJHMM/p1711029351315459 |
@CoenWarmer Can you help me interpret the response to the question about autoscaling? The response includes a link to a document that "could be out of date" and the code for autoscaling, so I'm still not sure what, if anything, I need to say in the serverless docs. |
==== | ||
In {ecloud} or {ece}, if you have Machine Learning autoscaling enabled, Machine Learning nodes will be started when using the knowledge base and AI Assistant. Therefore using these features will incur additional costs. | ||
==== | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
==== | |
In {ecloud} or {ece}, if you have Machine Learning autoscaling enabled, Machine Learning nodes will be started when using the knowledge base and AI Assistant. Therefore using these features will incur additional costs. | |
==== | |
==== | |
In {ecloud} or {ece}, if you have Machine Learning autoscaling enabled, Machine Learning nodes will be started when using the knowledge base and AI Assistant. Therefore using these features will incur additional costs. | |
Note that disabling the AI Assistant will not result in ML nodes being scaled down. | |
==== | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@lucabelluccini Is this what you had in mind?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that's great @dedemorton
I would love to get confirmation by engineering whether ML nodes will scale down.
ATM, it seems that once ML jobs are stopped or finish, ML nodes can be automatically scaled down.
But AI Assistant is not using ML Jobs but other ML features...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Following up here. Did we ever figure out if ML nodes scale down? This comment https://github.com/elastic/support-tech-lead/issues/1392#issuecomment-2030864999 says we do.
Clarify AI Assistant will spawn ML nodes
To the doc writers: