Code review comment for lp:~sberg-l/charms/trusty/nova-compute/tintri-interface

Revision history for this message
Billy Olsen (billy-olsen) wrote :

Skyler, this is definitely down the track of we discussed! Per Ryan's comment above, it'd be quite helpful to see the suite of charms to measure this in the broader context (and to be able to possibly test it), though I think I have a generally good sense of it from our discussions and openstack docs.

A couple of minor comments inline.

Before this gets merged, we need to make sure the tintri subordinate charm is promulgated. Does it exist out there to be able to look at?

review: Needs Fixing

« Back to merge proposal