Code review comment for lp:~xfactor973/charms/trusty/ceph/erasure_actions

Revision history for this message
James Page (james-page) wrote :

actions.yaml:

I'd suggest that you provide enums for strings where only a defined set of values is supported - see:

https://jujucharms.com/docs/1.25/authors-charm-actions

This is a feature missing from config options, but we should use them for actions.

« Back to merge proposal