Skip to content

fix: panic for incomplete metric in eviction controller#1083

Merged
ryanzhang-oss merged 15 commits intoAzure:mainfrom
Arvindthiru:evictionMetricsTest
Mar 20, 2025
Merged

fix: panic for incomplete metric in eviction controller#1083
ryanzhang-oss merged 15 commits intoAzure:mainfrom
Arvindthiru:evictionMetricsTest

Conversation

@Arvindthiru
Copy link
Copy Markdown
Contributor

@Arvindthiru Arvindthiru commented Mar 16, 2025

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

@Arvindthiru Arvindthiru marked this pull request as ready for review March 17, 2025 00:46
@Arvindthiru Arvindthiru changed the title test: add tests for eviction complete metric fix: panic for incomplete metric in eviction controller Mar 17, 2025
Comment thread pkg/controllers/clusterresourceplacementeviction/controller.go Outdated
Comment thread pkg/controllers/clusterresourceplacementeviction/controller_test.go Outdated
Comment thread pkg/controllers/clusterresourceplacementeviction/controller.go Outdated
Comment thread pkg/controllers/clusterresourceplacementeviction/controller.go Outdated
@ryanzhang-oss ryanzhang-oss merged commit a73306d into Azure:main Mar 20, 2025
16 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.

2 participants