Skip to content

Conversation

@ryanzhang-oss
Copy link
Contributor

@ryanzhang-oss ryanzhang-oss commented Oct 4, 2025

Description of your changes

  1. Add placement names to the test
  2. remove max log size so we use default (1.8G)
  3. Bump kind cluster version
  4. Add more nodes to the cluster as CI are more powerful with more placement running at the same time
  5. Fix the daemonset image too old (fail to load on my local test)
  6. Fix the max log size to be more reasonable (5G)
  7. Remove v1alpha1 CI run
  8. Add some missing logs that helps debugging at almost no cost
  9. Fix the VMs sku that has no cost issue

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

@ryanzhang-oss ryanzhang-oss changed the title fix the e2e and adjust log fix: fix the e2e and adjust log Oct 4, 2025
@codecov
Copy link

codecov bot commented Oct 4, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Signed-off-by: Ryan Zhang <yangzhangrice@hotmail.com>
Signed-off-by: Ryan Zhang <yangzhangrice@hotmail.com>
Signed-off-by: Ryan Zhang <yangzhangrice@hotmail.com>
@ryanzhang-oss ryanzhang-oss force-pushed the add-applier-debugging branch from b705c25 to a47b825 Compare October 8, 2025 00:04
Signed-off-by: Ryan Zhang <yangzhangrice@hotmail.com>
Signed-off-by: Ryan Zhang <yangzhangrice@hotmail.com>
Signed-off-by: Ryan Zhang <yangzhangrice@hotmail.com>
@ryanzhang-oss ryanzhang-oss force-pushed the add-applier-debugging branch from c02cd41 to cbe1b73 Compare October 9, 2025 23:44
Copy link
Collaborator

@michaelawyu michaelawyu left a comment

Choose a reason for hiding this comment

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

Added a nit comment

Copy link
Collaborator

@michaelawyu michaelawyu left a comment

Choose a reason for hiding this comment

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

Approved just in case we'd like to unblock progress first and fix the nit issue later.

@ryanzhang-oss ryanzhang-oss merged commit 09ad8bf into kubefleet-dev:main Oct 10, 2025
11 of 14 checks passed
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