Code review comment for lp:~gz/maas/1.2_add_db_kernel_params

Revision history for this message
John A Meinel (jameinel) wrote :

The actual change seems good. But I'm thinking there should be a docstring somewhere that needs updating as well. And maybe some sort of test about setting & getting the value?

review: Approve

« Back to merge proposal