Skip to content

fix(bm-server-components): preserve boolean type in OS install custom…#22742

Merged
Carlolis merged 1 commit into
release/dedicated-fix-os-installfrom
fix/PUBM-52327-fix-bool-OS-install
Apr 22, 2026
Merged

fix(bm-server-components): preserve boolean type in OS install custom…#22742
Carlolis merged 1 commit into
release/dedicated-fix-os-installfrom
fix/PUBM-52327-fix-bool-OS-install

Conversation

@Carlolis
Copy link
Copy Markdown
Contributor

@Carlolis Carlolis commented Apr 14, 2026

ref: #PUBM-52327

getCustomizations() was converting all input values to strings via .toString(), causing enableLacpBonding to be sent as "true" instead of true. The API rejects this with "[customizations.enableLacpBonding] Given data is not valid for type boolean".

Description

Ticket Reference: #PUBM-52327

Additional Information

@Carlolis Carlolis requested review from a team as code owners April 14, 2026 14:22
@github-actions github-actions Bot added the bug Something isn't working label Apr 14, 2026
@Carlolis Carlolis force-pushed the fix/PUBM-52327-fix-bool-OS-install branch 3 times, most recently from 2cf1bf3 to f0b7311 Compare April 20, 2026 09:26
…izations

getCustomizations() was converting all input values to strings via .toString(),
causing enableLacpBonding to be sent as "true" instead of true. The API rejects
this with "[customizations.enableLacpBonding] Given data is not valid for type boolean".

ref: #PUBM-52327

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
Signed-off-by: Charles Ilieff <[email protected]>
@Carlolis Carlolis force-pushed the fix/PUBM-52327-fix-bool-OS-install branch from f0b7311 to 289878b Compare April 21, 2026 07:34
@Carlolis Carlolis changed the base branch from master to release/dedicated-fix-os-install April 22, 2026 08:25
@Carlolis Carlolis merged commit 032e5fe into release/dedicated-fix-os-install Apr 22, 2026
13 of 14 checks passed
@Carlolis Carlolis deleted the fix/PUBM-52327-fix-bool-OS-install branch April 22, 2026 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants