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

[AWS Farget] Set dimension fields #6733

Merged
merged 5 commits into from Jul 7, 2023
Merged

[AWS Farget] Set dimension fields #6733

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
b88b0a6
Set dimension fields.
constanca-m Jun 28, 2023
b6f15a5
Update changelog.
constanca-m Jun 28, 2023
f2e2ac4
Add container.name as dimension
constanca-m Jul 6, 2023
d12e805
Merge branch 'main' into aws-fargate-dimensions
constanca-m Jul 7, 2023
4ad79d7
Resolve conflicts
constanca-m Jul 7, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
5 changes: 5 additions & 0 deletions packages/awsfargate/changelog.yml
View file
Open in desktop
@@ -1,4 +1,9 @@
# newer versions go on top
- version: 0.2.2
changes:
- description: Set dimension fields.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This sets dimension fields, but also agent.id. Should we mention it here?

Suggested change
- description: Set dimension fields.
- description: Set `dimension`, and `agent.id` fields.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

agent.id was already present, but this time, it is just explicitly set on the ecs.yml. I can add that. What do you think is best @dmathieu?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mentioning that it's now explicitly set sounds good to me.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @dmathieu , I updated the changelog to include that

type: enhancement
link: https://github.com/elastic/integrations/pull/6733
- version: 0.2.1
changes:
- description: Added categories and/or subcategories.
Expand Down
5 changes: 5 additions & 0 deletions packages/awsfargate/data_stream/task_stats/fields/ecs.yml
View file
Open in desktop
Expand Up @@ -44,10 +44,15 @@
description: ECS container name
- name: container.labels.com_amazonaws_ecs_task-arn
type: keyword
dimension: true
description: ECS task ARN
- name: container.labels.com_amazonaws_ecs_task-definition-family
type: keyword
description: ECS task definition family
- name: container.labels.com_amazonaws_ecs_task-definition-version
type: keyword
description: ECS task definition version
# agent
- name: agent.id
external: ecs
dimension: true
1 change: 1 addition & 0 deletions packages/awsfargate/docs/README.md
View file
Open in desktop
Expand Up @@ -302,6 +302,7 @@ If you want to learn more about Amazon ECS metrics, take a look at the blog post
| Field | Description | Type |
|---|---|---|
| @timestamp | Event timestamp. | date |
| agent.id | Unique identifier of this agent (if one exists). Example: For Beats this would be beat.id. | keyword |
| awsfargate.task_stats.cluster_name | Cluster name | keyword |
| awsfargate.task_stats.cpu.core | | object |
| awsfargate.task_stats.cpu.core.\*.norm.pct | Percentage of time per CPU core normalized by the number of CPU cores. | scaled_float |
Expand Down
2 changes: 1 addition & 1 deletion packages/awsfargate/manifest.yml
View file
Open in desktop
@@ -1,7 +1,7 @@
format_version: 1.0.0
name: awsfargate
title: AWS Fargate
version: 0.2.1
version: 0.2.2
license: basic
description: Collects metrics from containers and tasks running on Amazon ECS clusters with Elastic Agent.
type: integration
Expand Down