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

Add E2E testing for Python and Update Java E2E with Latest Release #1181

Merged
merged 2 commits into from
May 18, 2024

Conversation

harrryr
Copy link
Collaborator

@harrryr harrryr commented May 17, 2024

Description of the issue

We need to update the release testing to point to branches containing the latest validator changes (ga-release, ga-python, add-ec2-platform-support) and also create E2E testing for Python artifacts.

The aws-observability/aws-application-signals-test-framework branches are used as follows:

  • Java EKS: ga-release
  • Java EC2 (nonASG): ga-release
  • Java EC2 (ASG): add-ec2-platform-support
  • Python EKS/EC2 (nonASG): ga-python
  • Python EC2 (ASG): add-ec2-platform-for-python-ga

Description of changes

Created new E2E workflow for Python release testing. ADOT artifacts were patched with release candidate ones to sync the artifacts. Also created new workflows to test EC2 with ASGs

Adot Artifacts used:

Test run: https://github.com/aws/amazon-cloudwatch-agent/actions/runs/9132484921

Test run with missing metric aggregation added back into validation schemas: https://github.com/aws/amazon-cloudwatch-agent/actions/runs/9135922894

Rerun with all 6 e2e tests passing: https://github.com/aws/amazon-cloudwatch-agent/actions/runs/9136143337

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

Describe what tests you have done.

Requirements

Before commit the code, please do the following steps.

  1. Run make fmt and make fmt-sh
  2. Run make lint

@harrryr harrryr requested a review from a team as a code owner May 17, 2024 19:18
@majanjua-amzn majanjua-amzn force-pushed the ga-release-testing branch 3 times, most recently from 7b0aafd to a35a2b8 Compare May 18, 2024 00:27
@majanjua-amzn majanjua-amzn force-pushed the ga-release-testing branch 2 times, most recently from 7ecb21b to 30fdbea Compare May 18, 2024 00:55
@harrryr harrryr force-pushed the ga-release-testing branch from 76df048 to 6a5a627 Compare May 18, 2024 02:10
@majanjua-amzn majanjua-amzn merged commit 3a4959b into application_signals_ga May 18, 2024
@majanjua-amzn majanjua-amzn deleted the ga-release-testing branch May 18, 2024 02:34
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