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

SOFTLAYER-FILE VoluemGet doens't have all the important fields. #19

Open
depohmel opened this issue Jan 30, 2019 · 2 comments · Fixed by kastenhq/ibmcloud-storage-volume-lib#2

Comments

@depohmel
Copy link
Contributor

Here is the example of VoluemGet via Provider

&{VolumeID:62967589 Provider:SOFTLAYER-FILE VolumeType:file ProviderType:ENDURANCE_FILE_STORAGE Capacity:0xc0002c5590 SnapshotSpace:0xc000904210 Iops:0xc0006069f0 Tier:0xc0003ac1b0 Region:us-south Az:sjc03 BillingType: CreationTime:2019-01-29 14:52:23 -0600 -0600 ServiceOffering:<nil> VolumeNotes:map[plugin:ibm-file-plugin-77 region:us-south cluster:d843ddc67ffc4 type:Endurance] LunID: TargetIPAddresses:[]}

And here is an example for the same volume from SL backend

{"capacityGb":20,"createDate":"2019-01-29T14:52:23-06:00","id":62967589,"notes":"{\"plugin\":\"ibm-file-\"type\":\"Endurance\",}","parentVolume":{"accountId":1674595,"capacityGb":20,"createDate":"2019-01-29T18:05:57-06:00",","nasType":"SNAPSHOT","notes":"{some notes here}","serviceProviderId":1,"**serviceResourceBackendIpAddress":"HOSTNAME OF ASERVICE.service.softlayer.com"**,"serviceResourceName":"Storage Type 02 File Aggregate stff-sjc0301a","snapshotSizeBytes":"102400","storageTypeId":"16","upgradableFlag":true,**"username":"USSERNAME"**},"provisionedIops":"40","serviceResource":{"backendIpAddress":"HOSTNAME OF ASERVICE.service.softlayer.com","datacenter":{"name":"sjc03"},,"name":"Storage Type 02 File Aggregate stff-sjc0301a","type":{""}},"snapshotCapacityGb":"20","storageTierLevel":"READHEAVY_TIER","storageType":{"keyName":"ENDURANCE_FILE_STORAGE"},"username":"USSERNAME"}

Important and missing fields are:

serviceResourceBackendIpAddress or backendIpAddress
username

These fields are required if you want ot recreate PVC for K8s snapshot.

Ideally it would be nice to see all the fields whicha re avalible on UI
image

@depohmel
Copy link
Contributor Author

depohmel commented Jan 30, 2019

I might be wrong, but looks like by uncommenting these lines

//networkStorageIscsi.IscsiTargetIpAddresses = storage.IscsiTargetIpAddresses
/* if len(storage.IscsiTargetIpAddresses) > 0 {
some of info will appear in VolumeGet

@pramodgavali
Copy link
Contributor

@depohmel you are correct .. I'll fix it in next change of iteration

depohmel pushed a commit to kastenhq/ibmcloud-storage-volume-lib that referenced this issue Feb 6, 2019
Adding Attributes field for Volume 
The new field would allow adding Provider specific fields without affecting whole Volume struct.
`Username` and `fileNetworkMountAddress` would fix [IBM#19](IBM#19)
@depohmel depohmel reopened this Feb 6, 2019
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 a pull request may close this issue.

2 participants