bugfix: propagate eds changes when annotations change.

[yuval-k]

As the lb hash propagates to EndpointsVersion, it needs to include cluster name as well, until we hopfully stop appending upstream hashes to cluster names soon

BOT NOTES:
resolves https://github.com/solo-io/solo-projects/issues/7301

[solo-changelog-bot]

Issues linked to changelog:
https://github.com/solo-io/solo-projects/issues/7301

[github-actions]

Visit the preview URL for this PR (updated for commit 596641d):

https://gloo-edge--pr10411-yuval-k-annotate-svc-ulxjd139.web.app

_{(expires Wed, 04 Dec 2024 15:48:16 GMT)}

_{🔥 via Firebase Hosting GitHub Action 🌎}

_{Sign: 77c2b86e287749579b7ff9cadb81e099042ef677}

[sam-heilbron]

Do you think it's worth it to add an e2e test to ensure this doesn't regress in the future since it's a core piece of functionality?

[sam-heilbron]

Do you think it's worth it to add an e2e test to ensure this doesn't regress in the future since it's a core piece of functionality?

We agreed that an e2e test would be valuable. However, this PR fixes the bug and disables a very flakey test. So our plan is:

• Yuval will merge this code as is, without an e2e test
• Yuval will work on fixing the flaky test
• Sam/Yuval will pair on writing an e2e test