fix: keep comparableResourceVersions when copying a dependent resource's informer configuration

[afalhambra-hivemq]

updateFrom wasn't copying comparableResourceVersions, so it fell back to the default and setting it on a dependent resource had no effect. One line, plus a test that fails without it.

Worth noting it's a real behaviour change for anyone who set it to false on a dependent: they now actually get the temporary resource cache disabled, so read-your-own-write is off for that secondary. That's what they asked for, but they haven't been getting it.

Fixes #3624

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: c4a71291-895d-467d-a51e-96d42da6014d

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[Copilot]

Warning

Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.

Pull request overview

Fixes propagation of comparableResourceVersions when copying informer configuration to dependent resources, ensuring the explicitly configured value is preserved.

Changes:

• Copy comparableResourceVersions in InformerEventSourceConfiguration#updateFrom(...).
• Add a regression test verifying updateFrom preserves comparableResourceVersions instead of falling back to defaults.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/config/informer/InformerEventSourceConfiguration.java	Ensures updateFrom copies comparableResourceVersions from the source informer config.
operator-framework-core/src/test/java/io/javaoperatorsdk/operator/api/config/InformerEventSourceConfigurationTest.java	Adds a regression test that fails without the new copy behavior.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.