Code review comment for ~newell-jensen/maas:lp1789521

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

Tested the latest version. Looking better, but I now see this when composing in the UI:

    Pod unable to compose machine: too many values to unpack (expected 2)

Testing composition from the command-line results in:

    # maas admin pod compose 1
    Unable to compose machine because: Failed talking to pod: check_network_maas_dhcp_enabled() takes 3 positional arguments but 4 were given

So, still some bugs to fix - and tests to enhance...

« Back to merge proposal