search

khenidak / dysk

Attach Azure disks in < 1 second. Attach as many as you want. Attach them where ever you want. dysk mounts Azure disks as Linux block devices directly on VMs without dependency on the host.
MIT License
181 stars 13 forks source link

dyskctl list -o json should return [] when no dysks are mounted #36

Closed ehotinger closed 6 years ago

ehotinger commented 6 years ago

dyskctl list -o json should return [] instead of null when no dysks are mounted.

This fixes #20 by initializing dysks in List() with an empty array, so that the json encoder will encode an empty array instead of nil.

ehotinger commented 6 years ago

@khenidak possible to merge this? Need to make a few other small changes to client.go to pull it into our libs. Makes my life easier to avoid conflicts 👍

khenidak commented 6 years ago

Thanks Eric!