Skip to content

Conversation

@zzhlogin
Copy link
Contributor

@zzhlogin zzhlogin commented Sep 9, 2024

Issue description:

Description of changes:
This PR enabled ECS end to end test for Java and Python including workflows, terraform and validation setups. The change follows same procedure for other platform: EC2, EKS and K8S.

Rollback procedure:
Revert this commit. No further revert action needed.

<Can we safely revert this commit if needed? If not, detail what must be done to safely revert and why it is needed.>

Ensure you've run the following tests on your changes and include the link below:
Example of successful workflows running:
Java: https://github.com/aws-observability/aws-application-signals-test-framework/actions/runs/10906916872
Python: https://github.com/aws-observability/aws-application-signals-test-framework/actions/runs/10906916891/job/30269312152

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Copy link
Member

@bjrara bjrara left a comment

Choose a reason for hiding this comment

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

Overall looks good to me. Do we want to hold the PR until the artifacts are released?

# This is the 1st commit message:

Test concurency.

# The commit message #2 will be skipped:

# Dummy commit

# The commit message #3 will be skipped:

# Dummy commit

# The commit message #4 will be skipped:

# Dummy commit

# The commit message #5 will be skipped:

# Dummy commit

# The commit message #6 will be skipped:

# add push.

# The commit message #7 will be skipped:

# Dummy commit

# The commit message #8 will be skipped:

# Dummy commit

# The commit message #9 will be skipped:

# Dummy commit

# The commit message #10 will be skipped:

# Add push condition.

# The commit message #11 will be skipped:

# Dummy commit
@zzhlogin zzhlogin force-pushed the ecs-no-call-zzhlogin branch from 6c96df5 to 1e36f50 Compare September 13, 2024 19:31
# This is the 1st commit message:

Test concurency.

# The commit message #2 will be skipped:

# Dummy commit

# The commit message #3 will be skipped:

# Dummy commit

# The commit message #4 will be skipped:

# Dummy commit

# The commit message #5 will be skipped:

# Dummy commit

# The commit message #6 will be skipped:

# add push.

# The commit message #7 will be skipped:

# Dummy commit

# The commit message #8 will be skipped:

# Dummy commit

# The commit message #9 will be skipped:

# Dummy commit

# The commit message #10 will be skipped:

# Add push condition.

# The commit message #11 will be skipped:

# Dummy commit
@zzhlogin zzhlogin force-pushed the ecs-no-call-zzhlogin branch from 2342b7c to 2c6acd4 Compare September 17, 2024 17:31
@zzhlogin zzhlogin merged commit 328e079 into main Sep 17, 2024
1 check passed
@zzhlogin zzhlogin deleted the ecs-no-call-zzhlogin branch September 17, 2024 22:17
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