Skip to content

Conversation

@zhiying-lin
Copy link
Collaborator

Description of your changes

add RP tests

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 Aug 27, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@zhiying-lin zhiying-lin force-pushed the add-apply-strategy-test branch from 06593a4 to e58e742 Compare August 27, 2025 12:01
@zhiying-lin zhiying-lin force-pushed the add-apply-strategy-test branch from 0569c4a to efd10cf Compare August 28, 2025 02:19
Signed-off-by: Zhiying Lin <zhiyingl456@gmail.com>
Signed-off-by: Zhiying Lin <zhiyingl456@gmail.com>
Signed-off-by: Zhiying Lin <zhiyingl456@gmail.com>
Signed-off-by: Zhiying Lin <zhiyingl456@gmail.com>
Signed-off-by: Zhiying Lin <zhiyingl456@gmail.com>
Signed-off-by: Zhiying Lin <zhiyingl456@gmail.com>
@zhiying-lin zhiying-lin force-pushed the add-apply-strategy-test branch from 018018e to a56353f Compare August 28, 2025 02:26
@zhiying-lin zhiying-lin merged commit a7675f8 into kubefleet-dev:main Aug 28, 2025
16 checks passed
@zhiying-lin zhiying-lin deleted the add-apply-strategy-test branch August 28, 2025 05:38
michaelawyu pushed a commit to michaelawyu/kubefleet that referenced this pull request Sep 1, 2025
---------

Signed-off-by: Zhiying Lin <zhiyingl456@gmail.com>
Signed-off-by: michaelawyu <chenyu1@microsoft.com>
michaelawyu added a commit that referenced this pull request Sep 5, 2025
* Minor fixes

Signed-off-by: michaelawyu <chenyu1@microsoft.com>

* (Re-)enabled parallel processing in the work applier

Signed-off-by: michaelawyu <chenyu1@microsoft.com>

* Added unit tests

Signed-off-by: michaelawyu <chenyu1@microsoft.com>

* test: add RP apply strategy tests (#226)

---------

Signed-off-by: Zhiying Lin <zhiyingl456@gmail.com>
Signed-off-by: michaelawyu <chenyu1@microsoft.com>

* test: add taint and toleration tests for RP (#228)

Signed-off-by: michaelawyu <chenyu1@microsoft.com>

* fix: validate if the resource should be placed (#229)

Signed-off-by: Zhiying Lin <zhiyingl456@gmail.com>
Signed-off-by: michaelawyu <chenyu1@microsoft.com>

* test: set up the binding watcher integration tests to wait for cache sync (#231)

* Minor fixes

Signed-off-by: michaelawyu <chenyu1@microsoft.com>

* Minor fixes

Signed-off-by: michaelawyu <chenyu1@microsoft.com>

* Revert the timeout change in #99

Signed-off-by: michaelawyu <chenyu1@microsoft.com>

* Use eventually block

Signed-off-by: michaelawyu <chenyu1@microsoft.com>

* Experimental

Signed-off-by: michaelawyu <chenyu1@microsoft.com>

* Revert experimental changes

Signed-off-by: michaelawyu <chenyu1@microsoft.com>

---------

Signed-off-by: michaelawyu <chenyu1@microsoft.com>

* test: add RP test for join and leave flow (#233)

Signed-off-by: michaelawyu <chenyu1@microsoft.com>

* test: add e2e tests for RP for resource selection (#236)

Signed-off-by: Wantong Jiang <wantjian@microsoft.com>
Signed-off-by: michaelawyu <chenyu1@microsoft.com>

---------

Signed-off-by: michaelawyu <chenyu1@microsoft.com>
Signed-off-by: Zhiying Lin <zhiyingl456@gmail.com>
Signed-off-by: Wantong Jiang <wantjian@microsoft.com>
Co-authored-by: Zhiying Lin <54013513+zhiying-lin@users.noreply.github.com>
Co-authored-by: Wantong <jwtty0919@gmail.com>
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