Skip to content

Fix image overrides for scale services#41

Merged
galt-tr merged 3 commits into
bsv-blockchain:masterfrom
galt-tr:imageFix
Dec 10, 2025
Merged

Fix image overrides for scale services#41
galt-tr merged 3 commits into
bsv-blockchain:masterfrom
galt-tr:imageFix

Conversation

@galt-tr
Copy link
Copy Markdown
Contributor

@galt-tr galt-tr commented Dec 10, 2025

This pull request updates the image override logic for several cluster components to ensure that the cluster-level image always takes precedence, regardless of any existing deployment overrides. It also improves the test coverage and reliability for cluster reconciliation, including more thorough validation of image propagation and resource cleanup. Additionally, it enhances the test setup for custom volumes and adds brief delays to allow for resource cleanup and deployment creation.

Image override logic updates:

  • The image override logic in cluster_asset.go, cluster_blockassembly.go, cluster_propagation.go, and cluster_subtreevalidator.go was changed so that the cluster's image always overrides the component's deployment override image, ensuring consistency across all components. [1] [2] [3] [4]

Test improvements for cluster reconciliation:

  • Tests in cluster_controller_test.go now verify that enabling additional components (Propagation, SubtreeValidator, Pruner) correctly propagates the cluster image to those components, and that updating the cluster image triggers reconciliation and updates all relevant components. [1] [2]
  • Added explicit 1-second sleeps after resource deletion and before deployment verification to ensure proper cleanup and resource readiness in tests. [1] [2]

Custom volume setup in tests:

  • Custom volumes in tests are now defined with explicit EmptyDir sources, improving the accuracy and reliability of volume-related test scenarios.

Minor test and naming updates:

  • Changed the resource name in blockvalidator_controller_test.go for clarity, and commented out unused reconciliation logic for future test improvements. [1] [2]
  • Added missing import for time in test files to support new sleep functionality. [1] [2]

@galt-tr galt-tr requested a review from mrz1836 as a code owner December 10, 2025 02:06
@github-actions github-actions Bot added fork-pr PR originated from a forked repository requires-manual-review PR or issue requires manual review by a maintainer or security team labels Dec 10, 2025
@github-actions
Copy link
Copy Markdown

👋 Thanks, @galt-tr!

This pull request comes from a fork. For security, our CI runs in a restricted mode.
A maintainer will triage this shortly and run any additional checks as needed.

  • 🏷️ Labeled: fork-pr, requires-manual-review
  • 👀 We'll review and follow up here if anything else is needed.

Thanks for contributing to bsv-blockchain/teranode-operator! 🚀

@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

@mrz1836 mrz1836 left a comment

Choose a reason for hiding this comment

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

LGTM

@galt-tr galt-tr merged commit 36c1191 into bsv-blockchain:master Dec 10, 2025
40 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fork-pr PR originated from a forked repository requires-manual-review PR or issue requires manual review by a maintainer or security team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants