Err, I missed something here. Disregard the PR. I see now that is just a name that Bacula requires and the actual storage resource also has an address which makes this look redundant in the storage resource and just plain wrong in the pool resource when in fact it's all OK.

The real problem, as I just discovered, is that the storage named passed in my bacula::director::pool resource did not match the default name given to the storage.conf. I'll need to look closer at the code and unify the setting in Hiera. I've got things working now, just not organized as I'd like.

Let me know if I can help. I can give sample hiera values for what I'm using on which nodes if that's helpful.