Skip to content

fix: PUC-1491 - resolve enroll process issue for reviewing drive config#1674

Merged
RSabounds merged 1 commit intomainfrom
PUC-1491
Feb 16, 2026
Merged

fix: PUC-1491 - resolve enroll process issue for reviewing drive config#1674
RSabounds merged 1 commit intomainfrom
PUC-1491

Conversation

@RSabounds
Copy link
Contributor

@RSabounds RSabounds commented Feb 16, 2026

PUC-1491 handles a discovered issue with the output data in raid drive review:

  1. log output is being presented in the output data that is not handled correctly in argo.
  2. python result is an error exit code due to returning response instead of printing it.

Solving these two verified to result in correct data and return code in python and locally built copy of container image.

local@local:$ docker run --rm -u root -it --env-file env_vars.txt test_image:latest get-raid-
devices --ip-address 1.2.3.4
{"logical_disks": [{"controller": "RAID.SL.1-1", "is_root_volume": true, "physical_disks": ["Disk.Bay.0:Enclosure.Internal.0-1:RAID.SL.1-1", "Disk.Bay.1:Enclosure.Internal.0-1:RAID.SL.1-1"], "raid_level": "1", "size_gb": "MAX"}]}
local@local:
$ echo $?
0

@RSabounds RSabounds marked this pull request as ready for review February 16, 2026 14:58
@RSabounds RSabounds enabled auto-merge February 16, 2026 15:17
Copy link
Contributor

@stevekeay stevekeay left a comment

Choose a reason for hiding this comment

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

Go for it

@RSabounds RSabounds added this pull request to the merge queue Feb 16, 2026
Merged via the queue into main with commit 260d1e8 Feb 16, 2026
63 checks passed
@RSabounds RSabounds deleted the PUC-1491 branch February 16, 2026 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments