Fix resource loss during CS merge and improve OperandConfig creation by Daniel-Fan · Pull Request #2762 · IBM/ibm-common-service-operator

refactor: preserve base resources during CS merge to prevent loss

Daniel-Fan

committed

test(operandconfig): add tests for resource array merging logic

Daniel-Fan

committed

refactor: enhance debug logging with klog.V(2) for resource existence checks and CPU limit clearing.

Daniel-Fan

committed

refactor: Enhance type safety and readability in operand config processing by introducing helper functions for map access and string field extraction.

Daniel-Fan

committed

refactor: reimplement `mergeResourceArrays` to preserve base resource ordering and append CS-only resources.

Daniel-Fan

committed

refactor: extract common service configuration conversion logic into a new `convertServiceConfigs` helper function.

Daniel-Fan

committed

refactor: transition config merger from global variable to Bootstrap struct field

Daniel-Fan

committed

refactor: Streamline common service config processing by directly iterating the object list and calling `ExtractCommonServiceConfigs`.

Daniel-Fan

committed

Remove comments and klog.Info statements about single-stage OperandConfig creation.

Daniel-Fan

committed

refactor: update MergeConfigs to accept servicesNs directly instead of through the CommonServiceReconciler.

Daniel-Fan

committed

test: Add unit tests for `MergeConfigs` covering empty CommonService, feature extraction, invalid base configuration, and namespace handling.

Daniel-Fan

committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix resource loss during CS merge and improve OperandConfig creation#2762

Fix resource loss during CS merge and improve OperandConfig creation#2762
qpdpQ merged 11 commits intoIBM:masterfrom
Daniel-Fan:fix-opcon

Commits on Mar 6, 2026

refactor: preserve base resources during CS merge to prevent loss

test(operandconfig): add tests for resource array merging logic

refactor: enhance debug logging with klog.V(2) for resource existence checks and CPU limit clearing.

refactor: Enhance type safety and readability in operand config processing by introducing helper functions for map access and string field extraction.

refactor: reimplement `mergeResourceArrays` to preserve base resource ordering and append CS-only resources.

refactor: extract common service configuration conversion logic into a new `convertServiceConfigs` helper function.

refactor: transition config merger from global variable to Bootstrap struct field

refactor: Streamline common service config processing by directly iterating the object list and calling `ExtractCommonServiceConfigs`.

Remove comments and klog.Info statements about single-stage OperandConfig creation.

refactor: update MergeConfigs to accept servicesNs directly instead of through the CommonServiceReconciler.

test: Add unit tests for `MergeConfigs` covering empty CommonService, feature extraction, invalid base configuration, and namespace handling.