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

feat: add e2e framework support #654

Open
wants to merge 42 commits into
base: master
Choose a base branch
from
Open

Conversation

xjlgod
Copy link
Contributor

@xjlgod xjlgod commented Apr 15, 2024

Ⅰ. Describe what this PR did

feat: add e2e framework support:

  • creare e2e test builder can build images and prepare test for test by skywalkingInfraE2E.
  • creare e2e test runner can run test by skywalkingInfraE2E.
  • adapt springboot-dubbo-seata case to use this framework.

the details can be seen in readme file in e2e-test dir

Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

@xjlgod xjlgod changed the title [WIP] feat: add e2e framework support feat: add e2e framework support Apr 20, 2024
@funky-eyes funky-eyes closed this May 11, 2024
@funky-eyes funky-eyes reopened this May 11, 2024
@@ -0,0 +1 @@
go install github.com/apache/skywalking-infra-e2e/cmd/e2e@latest
Copy link
Member

Choose a reason for hiding this comment

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

FYI @kezhenxu94 @fgksgf It is great to see another new use case of our e2e tooling.

@xjlgod It is better you lock for a specific commit ID rather than latest, in case of breaking change with development/time.

Copy link
Member

Choose a reason for hiding this comment

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

@xjlgod We can try the test based on the commit id of tag 1.3.0

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok, I have changed the version to v1.3.0 in github action and sh file.

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.

None yet

4 participants