Skip to content

Conversation

@pandpara
Copy link
Contributor

Issue #, if available:

Description of changes:
Add errors metadata in agent debug info with granular sdk client error metrics

Also, adding metrics which tells us how many of RFN exception will
try to create profiling group. ErrorsMetadata is part of
AgentDebugInfo which is part of the Profile. So, everytime we submit
a profile we submit these metrics along with it.

SdkClientErrors will tell us total failures happening because of API
calls. Individual API call failures are captured with the same name
as APIs and the rfne resulting in auto creation of PG is also
captured.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

…r metrics

    Also, adding metrics which tells us how many of RFN exception will
    try to create profiling group. ErrorsMetadata is part of
    AgentDebugInfo which is part of the Profile. So, everytime we submit
    a profile we submit these metrics along with it.

    SdkClientErrors will tell us total failures happening because of API
    calls. Individual API call failures are captured with the same name
    as APIs and the rfne resulting in auto creation of PG is also
    captured.
@pandpara pandpara requested review from gimki and mirelap-amazon and removed request for gimki April 22, 2021 02:39
@mirelap-amazon mirelap-amazon requested a review from a team April 22, 2021 09:01
@pandpara pandpara removed the request for review from mirelap-amazon April 23, 2021 20:43
@pandpara pandpara requested a review from PapaPedro April 26, 2021 17:50
Copy link
Contributor

@PapaPedro PapaPedro left a comment

Choose a reason for hiding this comment

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

Looks good to me, please fix the failing unit test

@pandpara pandpara requested a review from PapaPedro April 27, 2021 05:06
@pandpara pandpara merged commit 77960a2 into aws:main Apr 27, 2021
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.

3 participants