Conversation
Apoorvkudesia-sumologic
requested review from
a team,
JV0812,
amee-sumo,
jpipkin1,
kimsauce and
mafsumo
as code owners
There was a problem hiding this comment.
@Apoorvkudesia-sumologic, this pull request looks pretty good, except for some broken links and missing screen images. After you fix those, I'll give this one another look.
Co-authored-by: Kim (Sumo Logic) <56411016+kimsauce@users.noreply.github.com>
Co-authored-by: John Pipkin (Sumo Logic) <jpipkin@sumologic.com>
| * Configure an [AWS Kinesis Firehose for Metrics Source](/docs/send-data/hosted-collectors/amazon-aws/aws-kinesis-firehose-metrics-source) (Recommended); or | ||
| * Configure an [Amazon CloudWatch Source for Metrics](/docs/send-data/hosted-collectors/amazon-aws/amazon-cloudwatch-source-metrics) | ||
|
|
||
| * Namespace for **Amazon Sagemaker** Service is **AWS/Sagemaker**. |
There was a problem hiding this comment.
There are multiple namespaces for could watch metrics. I think we should list all ?
|
|
||
| ### Configure field in field schema | ||
|
|
||
| 1. [**New UI**](/docs/get-started/sumo-logic-ui). In the main Sumo Logic menu, select **Data Management**, and then under **Logs** select **Fields**. You can also click the **Go To...** menu at the top of the screen and select **Fields**. <br/>[**Classic UI**](/docs/get-started/sumo-logic-ui-classic). In the main Sumo Logic menu, select **Manage Data > Logs > Fields**. |
There was a problem hiding this comment.
Is this required. This seems to be copied from bedrock since it has modelId ? Also since we are creating all the fields using fields.tf - is any manual creation even required ?
|
|
||
| As part of the app installation process, the following fields will be created by default: | ||
|
|
||
| * `account`: The friendly name or alias assigned to the AWS account. |
There was a problem hiding this comment.
We need to reconsider the fields which will get created automatically. Since some of these fields are not needed to be created explicitly
| | if (!isBlank(endpointname), endpointname, endpointconfigname) as endpoint | ||
| | fields accountid, region, namespace, endpoint | ||
| ``` | ||
|
|
There was a problem hiding this comment.
We also had a metric rule which needs to be in place in this doc.
|
|
||
| ## Viewing the Sagemaker dashboards | ||
|
|
||
| We highly recommend you view these dashboards in the [AWS Observability view](/docs/dashboards/explore-view/#aws-observability) of the AWS Observability solution. |
There was a problem hiding this comment.
Will this be possible as of now ? Since this is not part of aws observability
jpipkin1
left a comment
jpipkin1
left a comment
There was a problem hiding this comment.
Approved, @Apoorvkudesia-sumologic. I made some small updates (see commits "Updates from review" and "Add Amazon Sagemaker to product list").
5 participants
Purpose of this pull request
Select the type of change
Ticket (if applicable)