Skip to content

Fix Lvol connect API response - #1024

Merged
boddumanohar merged 1 commit into
mainfrom
lvol-connect-return-error
May 6, 2026
Merged

Fix Lvol connect API response#1024
boddumanohar merged 1 commit into
mainfrom
lvol-connect-return-error

Conversation

@boddumanohar

@boddumanohar boddumanohar commented May 6, 2026

Copy link
Copy Markdown
Member

return the actual error message as a part of the response API

Without this change this is what we see on the CSI driver side

oner is provisioning volume for claim "default/spdkcsi-pvc"
  Warning  ProvisioningFailed    4s (x5 over 11s)  csi.simplyblock.io_manohar-exra-node-1_b19c2b1d-d063-492f-a7bd-e76027ac06d0  failed to provision volume with StorageClass "simplyblock-simplyblock-cluster-pool4": rpc error: code = Internal desc = json: cannot unmarshal bool into Go value of type []*util.LvolConnectResp

Now this is the response

{"error":"Volume 4f533b12-dedc-452d-afaf-a507acce6705 has allowed hosts configured; --host-nqn is required","results":[],"status":false}

Comment thread simplyblock_web/api/v2/volume.py Fixed
@boddumanohar
boddumanohar force-pushed the lvol-connect-return-error branch from eadbc2d to 5c655ff Compare May 6, 2026 12:54
@boddumanohar
boddumanohar force-pushed the lvol-connect-return-error branch from 5c655ff to 5f5bd2a Compare May 6, 2026 13:26
@boddumanohar
boddumanohar merged commit 42e53ac into main May 6, 2026
8 of 9 checks passed
@boddumanohar
boddumanohar deleted the lvol-connect-return-error branch May 6, 2026 13:39
schmidt-scaled pushed a commit that referenced this pull request May 10, 2026
EbiRider pushed a commit that referenced this pull request May 13, 2026
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.

3 participants