Skip unauthorized pools in the rbd driver.

[highfly22]

Try to use rexray in the non-privilege account.

[codecov-io]

Codecov Report

Merging #1170 into master will decrease coverage by 0.33%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1170      +/-   ##
==========================================
- Coverage   34.58%   34.25%   -0.34%     
==========================================
  Files          36       36              
  Lines        2362     2362              
==========================================
- Hits          817      809       -8     
- Misses       1441     1450       +9     
+ Partials      104      103       -1

Impacted Files	Coverage Δ
...storage/drivers/storage/vfs/storage/vfs_storage.go	39.54% <0%> (-2.62%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1608180...f65c7df. Read the comment docs.

[akutz]

Hi @highfly22,

I like the change, but I wonder if @codenrhoden thinks it should be configurable? If so, default to skip/error?

Please go ahead and rebase onto master so the PR is up-to-date prior to merge. Thanks!

[codenrhoden]

@highfly22 I would prefer to see this as configurable. If you just blindly ignore the inability to list images in a pool, you have no way to know if it's because your Auth key doesn't allow you access to that specific pool, or if your key is completely invalid.

A config option to enable skipping inaccessible pools would be good, and I would want to see a debug log entry saying that the pool was skipped, such that if you turn on debugging you see it happening.

[akutz]

@highfly22 ping