Code review comment for lp:~andreserl/maas/fix_lp1553648

Revision history for this message
Mike Pontillo (mpontillo) wrote :

I read through the thread with Paul here:

https://bugs.launchpad.net/ubuntu/+source/dbconfig-common/+bug/1553757

Just so I understand, why was checking for /etc/dbconfig-common/maas.conf causing a problem? (I guess dbconfig didn't get to fully clean up if the configuration file still existed?)

What do you think about Paul's suggestion to do this to avoid prompting? (might be important for HA?)

    dbc_prio_high=medium

Since it sounds like you've tested this pretty well, I'll go ahead and approve. I have noticed odd dbconfig behavior while purging MAAS before as well, and this change does seem like a good step toward sanity.

review: Approve

« Back to merge proposal