search

NetApp / netappdvp

A Docker volume plugin for NetApp storage
96 stars 33 forks source link

Logical validation of values in config.json #105

Closed scaleoutsean closed 6 years ago

scaleoutsean commented 7 years ago

I came across this today, using netapp:17.07 - it seems some values aren't sanity-checked - for example SVIP in the example below:

{
    "version": 1,
    "storageDriverName": "solidfire-san",
    "Endpoint": "https://admin:admin@192.168.160.3/json-rpc/7.0",
    "SVIP": " 10.0.0.7:3260",
    "TenantName": "docker",
    "InitiatorIFace": "default"
}

Clearly this is a valid JSON file, but the SVIP value is incorrect and results in some interesting problems down the road.

This is similar to #56.

adkerr commented 6 years ago

Moved to netapp/trident#85