Use the new boot-resources endpoint in MAAS 2 support #4962

Merged

jujubot merged 13 commits into juju:maas2 from voidspace:maas2-use-boot-resources

Apr 4, 2016

voidspace commented Apr 1, 2016

Change fetching boot images to use the new boot-resources endpoint for MAAS 2, with associated refactoring and test support.

(Review request: http://reviews.vapour.ws/r/4401/)

Michael Foord and others added some commits Mar 31, 2016


      Use maasController.BootResources if using MAAS 2

9c9cf64


      Test for SupportedArchitectures in MAAS 2 error case

de35bb7


      Add test for maasEnviron.SupportedArchitectures with MAAS 2

Added fakeBootResource, changed fakeController so it could have an
error or a slice of boot resources.

ce15ffd


      Disentangle the MAAS 1v2 logic in SupportedArchitectures

Rather than having a branch in SupportedArchitectures for the
nodeArchitectures fallback and then another in allBootImages, base it
all on one branch in SupportedArchitectures.

37c9b06


      Refactored allBootImages and allBootImages2 to be about architectures

Rather than deduping boot images and then doing another pass to dedupe to architectures.

I'm not sure this is worthwhile, it's mostly refactoring the old code path.

0154b77


      Add API version constants

354a8fc


      Change var to const

b37e841


      Review changes

e702c3c


      Merge branch 'maas2-use-controller' into maas2-use-boot-resources

7e7739c


      Merge changing handling of boot images to architectures.

fcd0788


      Remove duplication in SupportedArchitectures, fix doc comments

000d5f3


      Merge remote-tracking branch 'upstream/maas2' into maas2-use-boot-res…

…ources

531ffcc


      Use set.NewStrings

1b172dd

voidspace commented Apr 4, 2016

$$merge$$

Contributor

jujubot commented Apr 4, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

jujubot merged commit `5d70bff` into juju:maas2 Apr 4, 2016

voidspace deleted the voidspace:maas2-use-boot-resources branch Oct 6, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

You can't perform that action at this time.