Skip to content

feat: add webhook configuration for cluster resource placement eviction#70

Merged
britaniar merged 6 commits intokubefleet-dev:mainfrom
britaniar:evictionWebhook
May 22, 2025
Merged

feat: add webhook configuration for cluster resource placement eviction#70
britaniar merged 6 commits intokubefleet-dev:mainfrom
britaniar:evictionWebhook

Conversation

@britaniar
Copy link
Member

Description of your changes

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

Signed-off-by: Britania Rodriguez Reyes <britaniar@microsoft.com>
@codecov
Copy link

codecov bot commented May 20, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!

Signed-off-by: Britania Rodriguez Reyes <britaniar@microsoft.com>
@britaniar britaniar marked this pull request as ready for review May 20, 2025 18:02
@britaniar britaniar requested a review from Arvindthiru May 20, 2025 18:02
britaniar added 2 commits May 22, 2025 10:27
Signed-off-by: Britania Rodriguez Reyes <britaniar@microsoft.com>
Signed-off-by: Britania Rodriguez Reyes <britaniar@microsoft.com>
britaniar added 2 commits May 22, 2025 11:17
Signed-off-by: Britania Rodriguez Reyes <britaniar@microsoft.com>
Signed-off-by: Britania Rodriguez Reyes <britaniar@microsoft.com>
Copy link
Member

@Arvindthiru Arvindthiru left a comment

Choose a reason for hiding this comment

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

LGTM

@britaniar britaniar merged commit 9a979fc into kubefleet-dev:main May 22, 2025
17 checks passed
@britaniar britaniar deleted the evictionWebhook branch May 22, 2025 21:02
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.

2 participants