Code review comment for lp:~maas-maintainers/juju-core/maas-provider-skeleton

Revision history for this message
Jeroen T. Vermeulen (jtv) wrote :

On point 12, thumper explains:

The new method is environs.config.Validate(). Our Validate() should upcall to that, so that it can do its basic non-specific verification of the proposed config changes. See ec2/config.go, and similar for the openstack and dummy providers.

« Back to merge proposal