Skip to content

Conversation

@kolluria
Copy link
Contributor

@kolluria kolluria commented May 19, 2025

Summary:

This MR updates the documentation to provide clear instructions to deploy CNS Manager on vSphere Supervisor 8.0.

Change List:

  1. README is updated with latest information
  2. svMotion which is a supported feature for Vanilla k8s clusters is removed from Supervisor as WCP mobility operator can be used for the same.
  3. suspend and resume volume provisioning endpoints are removed as they are not supported on Supervisors
  4. client-sdk is generated with the latest spec
  5. Examples are updated to reflect the latest spec
  6. Service type in the deployment is converted to a LoadBalancer from a NodePort
  7. Deployment instructions are updated to work accurately with Supervisor

Note: Spec was generated using the following -

curl -X POST \ 
  https://generator3.swagger.io/api/generate \
  -H 'content-type: application/json' \
  -d '{
  "specURL" : "https://raw.githubusercontent.com/kolluria/cloud-native-storage-self-service-manager/refs/heads/update-documentation/deploy/swagger-template.yaml",
  "lang" : "go",
  "type" : "CLIENT",
  "codegenVersion" : "V3"
}' --output sdk.zip

@kolluria kolluria force-pushed the update-documentation branch from a111c37 to ab25fb6 Compare May 21, 2025 09:28
@kolluria kolluria self-assigned this May 21, 2025
@kolluria kolluria requested review from a team and vdkotkar May 21, 2025 09:31
@vdkotkar
Copy link

@kolluria Instead of keeping 6 commits, will you rebase and keep 1 commit to keep commit history clean?

@kolluria
Copy link
Contributor Author

@kolluria Instead of keeping 6 commits, will you rebase and keep 1 commit to keep commit history clean?

@vdkotkar yeah. I will do that before we merge. I was just incrementally making changes.

@kolluria kolluria changed the title Update documentation Update documentation in accordance with vSphere 8.0 May 21, 2025
@kolluria kolluria force-pushed the update-documentation branch from f968ecd to e890f93 Compare May 21, 2025 18:14
@kolluria kolluria changed the title Update documentation in accordance with vSphere 8.0 Update documentation in accordance with vSphere Supervisor May 21, 2025
@kolluria kolluria requested a review from vdkotkar May 21, 2025 18:15
@AciDCooL
Copy link

AciDCooL commented May 23, 2025

And here i'm sitting like a dumbass deploying it in a tkc :)

Edit:
Or is the namespace part in this comment meant as a vsphere namespace? and thus a tkc:
CNS manager needs to be deployed in one of the namespaces running on the Supervisor in the vCenter.

The namespace part and the word "on" instead of "in" is trowing me off. But it makes sense that is should be the supervisor since the supervisor has the volume handles for the vSAN part.

Had to reference and delete 144 fcd's by hand today, what a mess this system they implemented. Maybe cool if it works. But if you hit a bug; Like this timeout issue with snapshots in a tkc. Big oof. This is by far the worst experience I ever had with storage.

And if this is the case to deploy on the supervisor then there is allot of info missing for vSphere supervisor. For instance howto install and expose this? Supervisor can only be accessed from a vCenter server to gain the privileged needed. And there is no middleware for a lb normally. Not even sure this will work from a network perspective if nsx is involved.
Than should it be a vsphere supervisor service? And hopefully not as a vsphere pod since since that in turn will not work with vds backed networks when using nsx. Ohh how this has become a total disaster. Ill stick with my primitive govc commands for now and hopefully we'll resolve the snapshot bug soon.

@kolluria kolluria force-pushed the update-documentation branch from 76918a6 to 4b07a16 Compare September 8, 2025 12:47
Removed endpoints and related models that are not supported with supervisor clusters
Removed DatastoreOperations tag as it's no longer needed
Generated the client-sdk using the latest spec
Updated README with the deployment instructions for vSphere Supervisor 8.0
Updated the service type to a LoadBalancer from Nodeport in the deploy templates
Updated deployment instructions to align with Supervisor
Updated basic auth secret creation to use SHA512 instead of less secure MD5
@kolluria kolluria force-pushed the update-documentation branch from 5026e89 to 04e1c10 Compare September 11, 2025 09:19
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