Dashes in dataset names about ansible-role-sanoid HOT 5 CLOSED

Looks like a design decision on my part (I avoid using dashes). I think I'll just need to change the variable to accept a list of mappings.

sanoid_datasets:
  - name: "tank2/docker-volumes"
    recursive: "yes"

from ansible-role-sanoid.

Yes. I don't want to break backwards compatibility. Neither for myself, not for anyone else :)

from ansible-role-sanoid.

That would be awesome, thank you! And I suppose the default name, if none is provided, is the object name, for backwards compatibility?

from ansible-role-sanoid.

The code from https://gitlab.com/radek-sprta/ansible-role-sanoid/-/merge_requests/1 (variable_format branch) should work for you. Can you try it out and let me know?

from ansible-role-sanoid.

I've merged this in and released a new version, since on top of molecule tests passing, I've been dogfooding it and it works fine.

from ansible-role-sanoid.