Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG]: Creating StorageClass for replication failed with unmarshal error #968

Closed
yutaro-satomi opened this issue Sep 4, 2023 · 5 comments
Assignees
Labels
area/csm-replication Issue pertains to the CSM Replication module type/bug Something isn't working. This is the default label associated with a bug issue.
Milestone

Comments

@yutaro-satomi
Copy link

How can the Team help you today?

Details: ?
We installed CSI Driver for Dell PowerStore and CSM for Replication on OpenShift.
When trying to create a StorageClass for replication, the following error occurred.

$ oc create -f powerstore-replication-ext4.yaml
Error from server (BadRequest): error when creating "powerstore-replication-ext4.yaml": StorageClass in version "v1" cannot be handled as a StorageClass: json: cannot unmarshal bool into Go struct field StorageClass.parameters of type string

The StorageClass Manifest was created based on the sample on GitHub.
The same error will occur even if the sample manifest is applied as is.

We don't know the cause, so please let us know how to deal with it.

Here is our environment details:

  • PowerStore 1200T
  • OpenShift Container Platform 4.12.28
  • CSI Driver for Dell PowerStore 2.7.0
  • CSM for Replication 1.5.0

Kind regards.

@yutaro-satomi yutaro-satomi added the type/question Ask a question. This is the default label associated with a question issue. label Sep 4, 2023
@donatwork
Copy link
Contributor

Looks like line 34 (replication.storage.dell.com/isReplicationEnabled: true) should be a string. Will need to check the public docs but you could try enclosing the value in quotes to see if you get past the error. I'll take a look to see how it is used.

@brianlucey
Copy link

Line should read :

replication.storage.dell.com/isReplicationEnabled: "true"

@santhoshatdell
Copy link
Contributor

Right. Only the PowerStore driver sample file seems to have this issue, other driver repository sample files are correct. I will see if I can correct the PowerStore sample file.
Please refer to the CSM-Docs for any detail needed for any of these inputs at https://dell.github.io/csm-docs/docs/replication/deployment/powerstore/

Also provide the bool value for replication.storage.dell.com/ignoreNamespaces within quotes.

@yutaro-satomi
Copy link
Author

After enclosing the bool values in quotes as per the documentation, the StorageClass was successfully created.
This issue has been resolved, so it can be closed.
Thank you for your support.

@shaynafinocchiaro shaynafinocchiaro added type/bug Something isn't working. This is the default label associated with a bug issue. area/csm-replication Issue pertains to the CSM Replication module and removed type/question Ask a question. This is the default label associated with a question issue. labels Sep 6, 2023
@shaynafinocchiaro shaynafinocchiaro added this to the v1.8.0 milestone Sep 6, 2023
@shaynafinocchiaro shaynafinocchiaro changed the title [QUESTION]: Creating StorageClass for replication failed with unmarshal error [BUG]: Creating StorageClass for replication failed with unmarshal error Sep 6, 2023
@santhoshatdell
Copy link
Contributor

Sample file has been corrected as part of PR dell/csi-powerstore#272

@santhoshatdell santhoshatdell self-assigned this Sep 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/csm-replication Issue pertains to the CSM Replication module type/bug Something isn't working. This is the default label associated with a bug issue.
Projects
None yet
Development

No branches or pull requests

5 participants