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.

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

AppSignals Metrics and Traces Integration tests #367

Merged
merged 34 commits into from
Oct 30, 2023
Merged

Conversation

nathalapooja
Copy link
Contributor

@nathalapooja nathalapooja commented Oct 25, 2023

Description of the issue

Describe the problem or feature in addition to a link to the issues.

  •  Add integration test to test metrics and traces sent to otlp receiver of the cw agent to the EMF and xray backend
    

Description of changes

  • Creates a golang container with a script for generating the trace id, and sending the trace to the node endpoint on a pod in an EKS cluster and also publishing the metrics using resources attributes defined in client and server json.

  • Performs Traces validation by checking the following annotations in the last 50 minutes


"annotations": {
                    "aws_remote_target": "remote-target",
                    "aws_remote_operation": "remote-operation",
                    "aws_local_service": "service-name",
                    "aws_remote_service": "service-name-remote",
                    "HostedIn_K8s_Namespace": "default",
                    "HostedIn_EKS_Cluster": "cwagent-eks-integ-800b3d9748c1443a",
                    "aws_local_operation": "operation"
                },

-Perform Metrics validation by calling GetMetricData API by passing instructions and namespace.

License

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

Tests

Used cwagent_image_repo and tag from latest beta release to validate the latest appsignals change

@nathalapooja nathalapooja requested a review from a team as a code owner October 25, 2023 20:43
@nathalapooja nathalapooja changed the title Apm traces Apm Metrics and Traces Oct 25, 2023
@nathalapooja nathalapooja changed the title Apm Metrics and Traces AppSignals Metrics and Traces Oct 25, 2023
test/app_signals/traces_test.go Outdated Show resolved Hide resolved
test/metric/app_signals_util.go Show resolved Hide resolved
test/metric/app_signals_util.go Show resolved Hide resolved
test/app_signals/traces_test.go Outdated Show resolved Hide resolved
test/app_signals/traces_test.go Outdated Show resolved Hide resolved
@nathalapooja nathalapooja changed the title AppSignals Metrics and Traces AppSignals Metrics and Traces Integration tests Oct 30, 2023
@nathalapooja nathalapooja merged commit 7fd3bd1 into main Oct 30, 2023
2 checks passed
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.

4 participants