fix: failed placement change need to trigger work generator copy to binding - Githubissues

Azure / fleet

Multi-cluster core

MIT License

80 stars 25 forks source link

fix: failed placement change need to trigger work generator copy to binding #879

Closed ryanzhang-oss closed 3 months ago

ryanzhang-oss commented 4 months ago

Description of your changes

fix the case that the failed placement in the work has changed but it's not propagated to the binding by the work generator.
Also remove the periodically check in the CRP controller as the binding will be updated which means the binding watcher also needs to check the failed placement
Move some common string to common

I have:

[x] Run make reviewable to ensure this PR is ready for review.

How has this code been tested

Special notes for your reviewer

Verified that both the new e2e and integration test failed without the fix.