Code review comment for lp:~ibmcharmers/charms/trusty/ibm-db2/trunk

Revision history for this message
Suchitra Venugopal (suchvenu) wrote :

> Overall this looks good. I encountered a couple of small rough edges from the
> ibm-base layer, which can be addressed there. But I did notice that the ibm-
> db2.client.sshconfigured state is not set per-consumer; once any consumer
> sends SSH keys, all consumers will assume that keys have been sent, and once
> any one consumer departs, all consumers will be blocked even if they
> previously sent SSH keys. I don't know if that was intended, but it seems
> like that state should be set by the interface layer instead, such as in the
> dismiss() method.

Hi Cory,
 I have done the changes as per your suggestions. Please review once and let me know.

Thanks,
 Suchitra

« Back to merge proposal