Skip to content

Conversation

vasyharan
Copy link

Updates the APIGateway example with the correct names for error metrics

- name: 4XXError
statistics: [Sum]
- name: 5xx
- name: 5XXError
Copy link
Contributor

Choose a reason for hiding this comment

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

@vasyharan I think 4xx and 5xx are valid metrics for APIGateway v1 (http), while 4XXError and 5XXError are for APIGateway v2 (rest). Maybe we should have two example files?

Copy link
Author

Choose a reason for hiding this comment

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

I didn't realize that these were different versions! Updated to add a new files for v2.

Copy link
Contributor

@cristiangreco cristiangreco left a comment

Choose a reason for hiding this comment

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

Left a comment for clarification.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants