Code review comment for lp:~wallyworld/gwacl/prefix-service-match

Revision history for this message
Andrew Wilkins (axwalk) wrote :

Feels like kind of a weird overloading; I kinda think we'd be better off just ditching the use of ListPrefixedHostedServices and use ListHostedServices and regex match in Juju.

But... LGTM, I don't want to hold up the fix.

review: Approve

« Back to merge proposal