Skip to content

test: separate joinleave tests from others#134

Merged
ryanzhang-oss merged 2 commits intokubefleet-dev:mainfrom
jwtty:separate-e2e
Jul 22, 2025
Merged

test: separate joinleave tests from others#134
ryanzhang-oss merged 2 commits intokubefleet-dev:mainfrom
jwtty:separate-e2e

Conversation

@jwtty
Copy link
Collaborator

@jwtty jwtty commented Jul 17, 2025

Description of your changes

Label e2e tests involving member cluster join/leave with "joinleave" label and run them separately in a new pipeline "e2e-tests (joinleave)". This reduces existing e2e-test pipeline execution length.

Fixes #

I have:

  • Run make reviewable to ensure this PR is ready for review.

How has this code been tested

Special notes for your reviewer

@codecov
Copy link

codecov bot commented Jul 17, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!

Copy link
Member

@ryanzhang-oss ryanzhang-oss left a comment

Choose a reason for hiding this comment

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

The serial e2e alone is 60 mins, maybe we can move some of them back or separate them further?

@jwtty jwtty force-pushed the separate-e2e branch 2 times, most recently from e02c646 to 3312479 Compare July 18, 2025 14:17
@jwtty jwtty changed the title test: separate serial tests from parallel tests test: separate joinleave tests from others Jul 21, 2025
jwtty added 2 commits July 21, 2025 03:57
Signed-off-by: Wantong Jiang <wantjian@microsoft.com>
Signed-off-by: Wantong Jiang <wantjian@microsoft.com>
@ryanzhang-oss ryanzhang-oss merged commit e935c34 into kubefleet-dev:main Jul 22, 2025
35 of 40 checks passed
@jwtty jwtty deleted the separate-e2e branch July 22, 2025 20:47
audrastump pushed a commit to audrastump/kubefleet that referenced this pull request Aug 20, 2025
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