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

Make e2e test itself in github action #116

Merged
merged 7 commits into from
Sep 4, 2023
Merged

Make e2e test itself in github action #116

merged 7 commits into from
Sep 4, 2023

Conversation

chunriyeqiongsaigao
Copy link
Contributor

This PR introduces several directories and files, and their relationships are depicted as follows:

  • skywalking-infra-e2e/
    • .github/workflows/
      • e2e-test.yaml
    • test/e2e/
      • e2e.yaml
      • docker-compose.yaml
      • fail-fast/
        • verify.yaml
        • expected.yaml
        • internal/
          • expected.yaml
      • non-fail-fast/
        • verify.yaml
        • expected.yaml
        • internal/
          • expected.yaml

This PR solely introduces directories and files (e2e configuration file & github-action configuration file) without altering the existing functionality of e2e. The purpose of this PR is to implement Use Infra E2E to test itself.

This PR currently implements the fail-fast and non-fail-fast testing modes in the non-concurrency mode only. The plan is to implement the fail-fast and non-fail-fast testing modes in the concurrency mode in the subsequent PRs.

@wu-sheng wu-sheng requested a review from fgksgf September 3, 2023 09:01
@wu-sheng wu-sheng added this to the 1.3.0 milestone Sep 3, 2023
@wu-sheng wu-sheng added the enhancement New feature or request label Sep 3, 2023
jobs:
build:

name: Build
Copy link
Member

Choose a reason for hiding this comment

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

Should this rename to e2e? Build seems strange.

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, Thanks very much !

Copy link
Member

@fgksgf fgksgf left a comment

Choose a reason for hiding this comment

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

Currently, <feature>/expected.yaml and <feature>/internal/expected.yaml seems a bit confusing, how about exchange them and move <feature>/verify.yaml to <feature>/internal/verify.yaml?

.github/workflows/e2e-test.yaml Show resolved Hide resolved
@fgksgf fgksgf changed the title add :make e2e test itself in github-action Make e2e test itself in github action Sep 4, 2023
Copy link
Member

@fgksgf fgksgf left a comment

Choose a reason for hiding this comment

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

LGTM, look forward to adding other test cases 👍 @chunriyeqiongsaigao

@fgksgf fgksgf merged commit 935338e into apache:main Sep 4, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants