-
Notifications
You must be signed in to change notification settings - Fork 137
Adds EIS info to Security billing #1070
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
Conversation
deploy-manage/cloud-organization/billing/security-billing-dimensions.md
Outdated
Show resolved
Hide resolved
deploy-manage/cloud-organization/billing/security-billing-dimensions.md
Outdated
Show resolved
Hide resolved
Hi @shubhaat , @jamesspi , this ticket is to add EIS information to the security billing dimensions page. Two questions :
|
@gaurav-elastic correct - we should wait on this for now. And yes, "Elastic Managed LLM" is finalized. |
@benironside @jmikell821 as James confirmed above, please pause these changes until the outstanding issues have been resolved and we can continue the rollout of EIS. |
@gaurav-elastic @jamesspi spelling/grammar nit: It should technically be Also, I'll comment out the paragraph, that way when we're ready to publish this, the content will already be there. |
|
||
% Note: Commenting out this section until this is ready to be published. | ||
|
||
% ## Elastic-managed LLM |
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.
@jmikell821, the PLI, or what is shown on the billing usage page is still called 'Elastic Inference Service'. Only the dropdown for Elastic LLM when selecting the LLM in AI Assistant or Attack Discovery is now called Elastic Managed LLM (or Elastic-managed LLM, until James or Shubha confirm). So I recommend that the title of this section be called something like 'Elastic Inference Service - Elastic Managed LLM'. In the body we could replace "default LLM" with "default Elastic Managed LLM."
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 @gaurav-elastic. Since I've merged the PR, I've added a note to #967 as a reminder for when we're ready to publish this.
Addresses #967.
NOTE: We aren't ready to publish these updates yet, so I commented out the paragraph that adds this info and will merge the PR so we'll have the content for when we're ready to publish.