search

mistifyio / go-zfs

Go wrappers for ZFS commands
Apache License 2.0
128 stars 66 forks source link

Remove command name from debug message joinedArgs #61

Closed rgynn closed 6 years ago

rgynn commented 7 years ago

Remove command name from joinedArgs as path to command is already printed (making it confusing to debug)

encountered error getting children of zfs filesystem: zpool_default/docker: exit status 1: "/usr/sbin/zfs get -t all -Hp all -r zpool_default/docker" => cannot open '-r': dataset does not exist
encountered error refreshing zfs watcher: exit status 1: "/usr/sbin/zfs get -t all -Hp all -r zpool_default/docker" => cannot open '-r': dataset does not exist

Instead of:

encountered error getting children of zfs filesystem: zpool_default/docker: exit status 1: "/usr/sbin/zfs zfs get -t all -Hp all -r zpool_default/docker" => cannot open '-r': dataset does not exist
encountered error refreshing zfs watcher: exit status 1: "/usr/sbin/zfs zfs get -t all -Hp all -r zpool_default/docker" => cannot open '-r': dataset does not exist
mmlb commented 6 years ago

Thanks @hajhatten, sorry it took so long!