Merge lp:~blake-rouse/maas/backward-compatible-boot-source-api into lp:~maas-committers/maas/trunk

[Blake Rouse]

Add backward compatible API for boot source on nodegroups.

Access to the boot sources using the api/1.0/nodegroups/uuid/boot-sources was removed as boot sources are now global. This is an issue as it breaks backwards compatibility.

[Jeroen T. Vermeulen (jtv)]

Excellent stuff. My eyes did glaze over a bit towards the end, but sometimes there's just no helping that with API tests.

I try always to find something to consider changing, just to prove that I'm really reviewing the branch, so here goes:

.

The docstring for BootSourcesBackwardHandler.read explains the uuid parameter, but maybe it should just say that it's a cluster uuid that is now ignored?

.

It looks like test_GET_returns_same_boot_source_for_different_node_groups and the two instances of test_GET_returns_same_list_for_different_node_groups each test the same thing 3 times. Does doing it more than once really contribute anything?

review: Approve

[Blake Rouse (blake-rouse)]

Thanks for the review.

.

Fixed the docstring.

.

It just to make very sure!!! :)

=== modified file 'src/maasserver/api/boot_source_selections.py'
--- src/maasserver/api/boot_source_selections.py	2014-08-17 01:07:57 +0000
+++ src/maasserver/api/boot_source_selections.py	2014-08-22 15:30:43 +0000
@@ -94,6 +94,38 @@
         return ('boot_source_selection_handler', (boot_source_id, id))
 
 
+class BootSourceSelectionBackwardHandler(BootSourceSelectionHandler):
+    """Manage a boot source selection.
+
+    It used to be that boot-sources could be set per cluster. Now it can only
+    be set globally for the whole region and clusters. This api is now
+    deprecated, and only exists for backwards compatibility.
+    """
+    deprecated = True
+
+    def read(self, request, uuid, boot_source_id, id):
+        """Read a boot source selection."""
+        return super(BootSourceSelectionBackwardHandler, self).read(
+            request, boot_source_id, id)
+
+    def update(self, request, uuid, boot_source_id, id):
+        """Update a specific boot source selection.
+
+        :param release: The release for which to import resources.
+        :param arches: The list of architectures for which to import resources.
+        :param subarches: The list of subarchitectures for which to import
+            resources.
+        :param labels: The list of labels for which to import resources.
+        """
+        return super(BootSourceSelectionBackwardHandler, self).update(
+            request, boot_source_id, id)
+
+    def delete(self, request, uuid, boot_source_id, id):
+        """Delete a specific boot source."""
+        return super(BootSourceSelectionBackwardHandler, self).delete(
+            request, boot_source_id, id)
+
+
 class BootSourceSelectionsHandler(OperationsHandler):
     """Manage the collection of boot source selections."""
     api_doc_section_name = "Boot source selections"
@@ -134,3 +166,33 @@
             return form.save()
         else:
             raise ValidationError(form.errors)
+
+
+class BootSourceSelectionsBackwardHandler(BootSourceSelectionsHandler):
+    """Manage a boot source selection.
+
+    It used to be that boot-sources could be set per cluster. Now it can only
+    be set globally for the whole region and clusters. This api is now
+    deprecated, and only exists for backwards compatibility.
+    """
+    deprecated = True
+
+    def read(self, request, uuid, boot_source_id):
+        """List boot source selections.
+
+        Get a listing of a boot source's selections.
+        """
+        return super(BootSourceSelectionsBackwardHandler, self).read(
+            request, boot_source_id)
+
+    def create(self, request, uuid, boot_source_id):
+        """Create a new boot source selection.
+
+        :param release: The release for which to import resources.
+        :param arches: The architecture list for which to import resources.
+        :param subarches: The subarchitecture list for which to import
+            resources.
+        :param labels: The label lists for which to import resources.
+        """
+        return super(BootSourceSelectionsBackwardHandler, self).create(
+            request, boot_source_id)

=== modified file 'src/maasserver/api/boot_sources.py'
--- src/maasserver/api/boot_sources.py	2014-08-16 15:43:01 +0000
+++ src/maasserver/api/boot_sources.py	2014-08-22 15:30:43 +0000
@@ -112,6 +112,35 @@
         return ('boot_source_handler', (id, ))
 
 
+class BootSourceBackwardHandler(BootSourceHandler):
+    """Manage a boot source.
+
+    It used to be that boot-sources could be set per cluster. Now it can only
+    be set globally for the whole region and clusters. This api is now
+    deprecated, and only exists for backwards compatibility.
+    """
+    deprecated = True
+
+    def read(self, request, uuid, id):
+        """Read a boot source."""
+        return super(BootSourceBackwardHandler, self).read(request, id)
+
+    def update(self, request, uuid, id):
+        """Update a specific boot source.
+
+        :param url: The URL of the BootSource.
+        :param keyring_filename: The path to the keyring file for this
+            BootSource.
+        :param keyring_filename: The GPG keyring for this BootSource,
+            base64-encoded data.
+        """
+        return super(BootSourceBackwardHandler, self).update(request, id)
+
+    def delete(self, request, uuid, id):
+        """Delete a specific boot source."""
+        return super(BootSourceBackwardHandler, self).delete(request, id)
+
+
 class BootSourcesHandler(OperationsHandler):
     """Manage the collection of boot sources."""
     api_doc_section_name = "Boot sources"
@@ -148,3 +177,34 @@
                 status=httplib.CREATED)
         else:
             raise ValidationError(form.errors)
+
+
+class BootSourcesBackwardHandler(BootSourcesHandler):
+    """Manage the collection of boot sources.
+
+    It used to be that boot-sources could be set per cluster. Now it can only
+    be set globally for the whole region and clusters. This api is now
+    deprecated, and only exists for backwards compatibility.
+    """
+    deprecated = True
+
+    def read(self, request, uuid):
+        """List boot sources.
+
+        Get a listing of a cluster's boot sources.
+
+        :param uuid: This is deprecated, only exists for backwards
+            compatibility. Boot sources are now global for all of MAAS.
+        """
+        return super(BootSourcesBackwardHandler, self).read(request)
+
+    def create(self, request, uuid):
+        """Create a new boot source.
+
+        :param url: The URL of the BootSource.
+        :param keyring_filename: The path to the keyring file for
+            this BootSource.
+        :param keyring_data: The GPG keyring for this BootSource,
+            base64-encoded.
+        """
+        return super(BootSourcesBackwardHandler, self).create(request)

=== modified file 'src/maasserver/api/doc.py'
--- src/maasserver/api/doc.py	2014-08-16 05:43:33 +0000
+++ src/maasserver/api/doc.py	2014-08-22 15:30:43 +0000
@@ -47,13 +47,16 @@
     """
     p_has_resource_uri = lambda resource: (
         getattr(resource.handler, "resource_uri", None) is not None)
+    p_is_not_deprecated = lambda resource: (
+        getattr(resource.handler, "deprecated", False))
     for pattern in resolver.url_patterns:
         if isinstance(pattern, RegexURLResolver):
             accumulate_api_resources(pattern, accumulator)
         elif isinstance(pattern, RegexURLPattern):
             if isinstance(pattern.callback, Resource):
                 resource = pattern.callback
-                if p_has_resource_uri(resource):
+                if p_has_resource_uri(resource) and \
+                        not p_is_not_deprecated(resource):
                     accumulator.add(resource)
         else:
             raise AssertionError(

=== modified file 'src/maasserver/api/tests/test_boot_source_selections.py'
--- src/maasserver/api/tests/test_boot_source_selections.py	2014-08-17 01:16:55 +0000
+++ src/maasserver/api/tests/test_boot_source_selections.py	2014-08-22 15:30:43 +0000
@@ -41,6 +41,22 @@
     )
 
 
+def get_boot_source_selection_backward_uri(
+        boot_source_selection, nodegroup=None):
+    """Return a boot source's URI on the API."""
+    if nodegroup is None:
+        nodegroup = factory.make_node_group()
+    boot_source = boot_source_selection.boot_source
+    return reverse(
+        'boot_source_selection_backward_handler',
+        args=[
+            nodegroup.uuid,
+            boot_source.id,
+            boot_source_selection.id,
+        ]
+    )
+
+
 class TestBootSourceSelectionAPI(APITestCase):
 
     def test_handler_path(self):
@@ -125,6 +141,107 @@
         self.assertEqual(httplib.FORBIDDEN, response.status_code)
 
 
+class TestBootSourceSelectionBackwardAPI(APITestCase):
+
+    def test_handler_path(self):
+        self.assertEqual(
+            '/api/1.0/nodegroups/uuid/boot-sources/3/selections/4/',
+            reverse(
+                'boot_source_selection_backward_handler',
+                args=['uuid', '3', '4']))
+
+    def test_GET_returns_boot_source(self):
+        self.become_admin()
+        boot_source_selection = factory.make_boot_source_selection()
+        response = self.client.get(
+            get_boot_source_selection_backward_uri(boot_source_selection))
+        self.assertEqual(httplib.OK, response.status_code)
+        returned_boot_source_selection = json.loads(response.content)
+        boot_source = boot_source_selection.boot_source
+        # The returned object contains a 'resource_uri' field.
+        self.assertEqual(
+            reverse(
+                'boot_source_selection_handler',
+                args=[
+                    boot_source.id,
+                    boot_source_selection.id]
+            ),
+            returned_boot_source_selection['resource_uri'])
+        # The other fields are the boot source selection's fields.
+        del returned_boot_source_selection['resource_uri']
+        # All the fields are present.
+        self.assertItemsEqual(
+            DISPLAYED_BOOTSOURCESELECTION_FIELDS,
+            returned_boot_source_selection.keys())
+        self.assertThat(
+            boot_source_selection,
+            MatchesStructure.byEquality(**returned_boot_source_selection))
+
+    def test_GET_returns_same_boot_source_for_different_node_groups(self):
+        self.become_admin()
+        boot_source_selection = factory.make_boot_source_selection()
+        for _ in range(3):
+            nodegroup = factory.make_node_group()
+            response = self.client.get(
+                get_boot_source_selection_backward_uri(
+                    boot_source_selection, nodegroup))
+            self.assertEqual(httplib.OK, response.status_code)
+            returned_boot_source_selection = json.loads(response.content)
+            del returned_boot_source_selection['resource_uri']
+            self.assertThat(
+                boot_source_selection,
+                MatchesStructure.byEquality(**returned_boot_source_selection))
+
+    def test_GET_requires_admin(self):
+        boot_source_selection = factory.make_boot_source_selection()
+        response = self.client.get(
+            get_boot_source_selection_backward_uri(boot_source_selection))
+        self.assertEqual(httplib.FORBIDDEN, response.status_code)
+
+    def test_DELETE_deletes_boot_source_selection(self):
+        self.become_admin()
+        boot_source_selection = factory.make_boot_source_selection()
+        response = self.client.delete(
+            get_boot_source_selection_backward_uri(boot_source_selection))
+        self.assertEqual(httplib.NO_CONTENT, response.status_code)
+        self.assertIsNone(reload_object(boot_source_selection))
+
+    def test_DELETE_requires_admin(self):
+        boot_source_selection = factory.make_boot_source_selection()
+        response = self.client.delete(
+            get_boot_source_selection_backward_uri(boot_source_selection))
+        self.assertEqual(httplib.FORBIDDEN, response.status_code)
+
+    def test_PUT_updates_boot_source_selection(self):
+        self.become_admin()
+        boot_source_selection = factory.make_boot_source_selection()
+        ubuntu_os = UbuntuOS()
+        new_release = factory.pick_release(ubuntu_os)
+        new_values = {
+            'release': new_release,
+            'arches': [factory.make_name('arch'), factory.make_name('arch')],
+            'subarches': [
+                factory.make_name('subarch'), factory.make_name('subarch')],
+            'labels': [factory.make_name('label')],
+        }
+        response = self.client_put(
+            get_boot_source_selection_backward_uri(
+                boot_source_selection), new_values)
+        self.assertEqual(httplib.OK, response.status_code)
+        boot_source_selection = reload_object(boot_source_selection)
+        self.assertAttributes(boot_source_selection, new_values)
+
+    def test_PUT_requires_admin(self):
+        boot_source_selection = factory.make_boot_source_selection()
+        new_values = {
+            'release': factory.make_name('release'),
+        }
+        response = self.client_put(
+            get_boot_source_selection_backward_uri(
+                boot_source_selection), new_values)
+        self.assertEqual(httplib.FORBIDDEN, response.status_code)
+
+
 class TestBootSourceSelectionsAPI(APITestCase):
     """Test the the boot source selections API."""
 
@@ -198,3 +315,93 @@
                 'boot_source_selections_handler',
                 args=[boot_source.id]), params)
         self.assertEqual(httplib.FORBIDDEN, response.status_code)
+
+
+class TestBootSourceSelectionsBackwardAPI(APITestCase):
+    """Test the the boot source selections API."""
+
+    def get_uri(self, boot_source, nodegroup=None):
+        if nodegroup is None:
+            nodegroup = factory.make_node_group()
+        return reverse(
+            'boot_source_selections_backward_handler',
+            args=[nodegroup.uuid, boot_source.id])
+
+    def test_handler_path(self):
+        self.assertEqual(
+            '/api/1.0/nodegroups/uuid/boot-sources/3/selections/',
+            reverse(
+                'boot_source_selections_backward_handler',
+                args=['uuid', '3']))
+
+    def test_GET_returns_boot_source_selection_list(self):
+        self.become_admin()
+        boot_source = factory.make_boot_source()
+        selections = [
+            factory.make_boot_source_selection(boot_source=boot_source)
+            for _ in range(3)]
+        # Create boot source selections in another boot source.
+        [factory.make_boot_source_selection() for _ in range(3)]
+        response = self.client.get(self.get_uri(boot_source))
+        self.assertEqual(httplib.OK, response.status_code, response.content)
+        parsed_result = json.loads(response.content)
+        self.assertItemsEqual(
+            [selection.id for selection in selections],
+            [selection.get('id') for selection in parsed_result])
+
+    def test_GET_returns_same_list_for_different_node_groups(self):
+        self.become_admin()
+        boot_source = factory.make_boot_source()
+        selections = [
+            factory.make_boot_source_selection(boot_source=boot_source)
+            for _ in range(3)]
+        # Create boot source selections in another boot source.
+        [factory.make_boot_source_selection() for _ in range(3)]
+        for _ in range(3):
+            nodegroup = factory.make_node_group()
+            response = self.client.get(self.get_uri(boot_source, nodegroup))
+            self.assertEqual(
+                httplib.OK, response.status_code, response.content)
+            parsed_result = json.loads(response.content)
+            self.assertItemsEqual(
+                [selection.id for selection in selections],
+                [selection.get('id') for selection in parsed_result])
+
+    def test_GET_requires_admin(self):
+        boot_source = factory.make_boot_source()
+        response = self.client.get(self.get_uri(boot_source))
+        self.assertEqual(httplib.FORBIDDEN, response.status_code)
+
+    def test_POST_creates_boot_source_selection(self):
+        self.become_admin()
+        boot_source = factory.make_boot_source()
+        ubuntu_os = UbuntuOS()
+        new_release = factory.pick_release(ubuntu_os)
+        params = {
+            'release': new_release,
+            'arches': [factory.make_name('arch'), factory.make_name('arch')],
+            'subarches': [
+                factory.make_name('subarch'), factory.make_name('subarch')],
+            'labels': [factory.make_name('label')],
+        }
+        response = self.client.post(self.get_uri(boot_source), params)
+        self.assertEqual(httplib.OK, response.status_code)
+        parsed_result = json.loads(response.content)
+
+        boot_source_selection = BootSourceSelection.objects.get(
+            id=parsed_result['id'])
+        self.assertAttributes(boot_source_selection, params)
+
+    def test_POST_requires_admin(self):
+        boot_source = factory.make_boot_source()
+        ubuntu_os = UbuntuOS()
+        new_release = factory.pick_release(ubuntu_os)
+        params = {
+            'release': new_release,
+            'arches': [factory.make_name('arch'), factory.make_name('arch')],
+            'subarches': [
+                factory.make_name('subarch'), factory.make_name('subarch')],
+            'labels': [factory.make_name('label')],
+        }
+        response = self.client.post(self.get_uri(boot_source), params)
+        self.assertEqual(httplib.FORBIDDEN, response.status_code)

=== modified file 'src/maasserver/api/tests/test_boot_sources.py'
--- src/maasserver/api/tests/test_boot_sources.py	2014-08-16 15:43:01 +0000
+++ src/maasserver/api/tests/test_boot_sources.py	2014-08-22 15:30:43 +0000
@@ -34,6 +34,14 @@
         args=[boot_source.id])
 
 
+def get_boot_source_backward_uri(boot_source, nodegroup=None):
+    if nodegroup is None:
+        nodegroup = factory.make_node_group()
+    return reverse(
+        'boot_source_backward_handler',
+        args=[nodegroup.uuid, boot_source.id])
+
+
 class TestBootSourceAPI(APITestCase):
 
     def test_handler_path(self):
@@ -104,6 +112,92 @@
         self.assertEqual(httplib.FORBIDDEN, response.status_code)
 
 
+class TestBootSourceBackwardAPI(APITestCase):
+
+    def test_handler_path(self):
+        self.assertEqual(
+            '/api/1.0/nodegroups/uuid/boot-sources/3/',
+            reverse('boot_source_backward_handler', args=['uuid', '3']))
+
+    def test_GET_returns_boot_source(self):
+        self.become_admin()
+        boot_source = factory.make_boot_source()
+        response = self.client.get(get_boot_source_backward_uri(boot_source))
+        self.assertEqual(httplib.OK, response.status_code)
+        returned_boot_source = json.loads(response.content)
+        # The returned object contains a 'resource_uri' field.
+        self.assertEqual(
+            reverse(
+                'boot_source_handler',
+                args=[boot_source.id]
+            ),
+            returned_boot_source['resource_uri'])
+        # The other fields are the boot source's fields.
+        del returned_boot_source['resource_uri']
+        # All the fields are present.
+        self.assertItemsEqual(
+            DISPLAYED_BOOTSOURCE_FIELDS, returned_boot_source.keys())
+        self.assertThat(
+            boot_source,
+            MatchesStructure.byEquality(**returned_boot_source))
+
+    def test_GET_returns_same_boot_source_no_matter_the_nodegroup(self):
+        self.become_admin()
+        boot_source = factory.make_boot_source()
+        for _ in range(3):
+            nodegroup = factory.make_node_group()
+            response = self.client.get(
+                get_boot_source_backward_uri(boot_source, nodegroup))
+            self.assertEqual(httplib.OK, response.status_code)
+            returned_boot_source = json.loads(response.content)
+            del returned_boot_source['resource_uri']
+            self.assertThat(
+                boot_source,
+                MatchesStructure.byEquality(**returned_boot_source))
+
+    def test_GET_requires_admin(self):
+        boot_source = factory.make_boot_source()
+        response = self.client.get(get_boot_source_backward_uri(boot_source))
+        self.assertEqual(httplib.FORBIDDEN, response.status_code)
+
+    def test_DELETE_deletes_boot_source(self):
+        self.become_admin()
+        boot_source = factory.make_boot_source()
+        response = self.client.delete(
+            get_boot_source_backward_uri(boot_source))
+        self.assertEqual(httplib.NO_CONTENT, response.status_code)
+        self.assertIsNone(reload_object(boot_source))
+
+    def test_DELETE_requires_admin(self):
+        boot_source = factory.make_boot_source()
+        response = self.client.delete(
+            get_boot_source_backward_uri(boot_source))
+        self.assertEqual(httplib.FORBIDDEN, response.status_code)
+
+    def test_PUT_updates_boot_source(self):
+        self.become_admin()
+        boot_source = factory.make_boot_source()
+        new_values = {
+            'url': 'http://example.com/',
+            'keyring_filename': factory.make_name('filename'),
+        }
+        response = self.client_put(
+            get_boot_source_backward_uri(boot_source), new_values)
+        self.assertEqual(httplib.OK, response.status_code)
+        boot_source = reload_object(boot_source)
+        self.assertAttributes(boot_source, new_values)
+
+    def test_PUT_requires_admin(self):
+        boot_source = factory.make_boot_source()
+        new_values = {
+            'url': 'http://example.com/',
+            'keyring_filename': factory.make_name('filename'),
+        }
+        response = self.client_put(
+            get_boot_source_backward_uri(boot_source), new_values)
+        self.assertEqual(httplib.FORBIDDEN, response.status_code)
+
+
 class TestBootSourcesAPI(APITestCase):
     """Test the the boot source API."""
 
@@ -190,3 +284,105 @@
         response = self.client.post(
             reverse('boot_sources_handler'), params)
         self.assertEqual(httplib.FORBIDDEN, response.status_code)
+
+
+class TestBootSourcesBackwardAPI(APITestCase):
+    """Test the the boot source API."""
+
+    def get_uri(self, nodegroup=None):
+        if nodegroup is None:
+            nodegroup = factory.make_node_group()
+        return reverse(
+            'boot_sources_backward_handler', args=[nodegroup.uuid])
+
+    def test_handler_path(self):
+        self.assertEqual(
+            '/api/1.0/nodegroups/uuid/boot-sources/',
+            reverse('boot_sources_backward_handler', args=['uuid']))
+
+    def test_GET_returns_boot_source_list(self):
+        self.become_admin()
+        sources = [
+            factory.make_boot_source() for _ in range(3)]
+        response = self.client.get(self.get_uri())
+        self.assertEqual(httplib.OK, response.status_code, response.content)
+        parsed_result = json.loads(response.content)
+        self.assertItemsEqual(
+            [boot_source.id for boot_source in sources],
+            [boot_source.get('id') for boot_source in parsed_result])
+
+    def test_GET_returns_same_list_for_different_node_groups(self):
+        self.become_admin()
+        sources = [
+            factory.make_boot_source() for _ in range(3)]
+        for _ in range(3):
+            nodegroup = factory.make_node_group()
+            response = self.client.get(self.get_uri(nodegroup))
+            self.assertEqual(
+                httplib.OK, response.status_code, response.content)
+            parsed_result = json.loads(response.content)
+            self.assertItemsEqual(
+                [boot_source.id for boot_source in sources],
+                [boot_source.get('id') for boot_source in parsed_result])
+
+    def test_GET_requires_admin(self):
+        response = self.client.get(self.get_uri())
+        self.assertEqual(httplib.FORBIDDEN, response.status_code)
+
+    def test_POST_creates_boot_source_with_keyring_filename(self):
+        self.become_admin()
+
+        params = {
+            'url': 'http://example.com/',
+            'keyring_filename': factory.make_name('filename'),
+            'keyring_data': '',
+        }
+        response = self.client.post(self.get_uri(), params)
+        self.assertEqual(httplib.CREATED, response.status_code)
+        parsed_result = json.loads(response.content)
+
+        boot_source = BootSource.objects.get(id=parsed_result['id'])
+        # boot_source.keyring_data is returned as a read-only buffer, test
+        # it separately from the rest of the attributes.
+        self.assertEqual('', bytes(boot_source.keyring_data))
+        del params['keyring_data']
+        self.assertAttributes(boot_source, params)
+
+    def test_POST_creates_boot_source_with_keyring_data(self):
+        self.become_admin()
+
+        params = {
+            'url': 'http://example.com/',
+            'keyring_filename': '',
+            'keyring_data': (
+                factory.make_file_upload(content=sample_binary_data)),
+        }
+        response = self.client.post(self.get_uri(), params)
+        self.assertEqual(httplib.CREATED, response.status_code)
+        parsed_result = json.loads(response.content)
+
+        boot_source = BootSource.objects.get(id=parsed_result['id'])
+        # boot_source.keyring_data is returned as a read-only buffer, test
+        # it separately from the rest of the attributes.
+        self.assertEqual(sample_binary_data, bytes(boot_source.keyring_data))
+        del params['keyring_data']
+        self.assertAttributes(boot_source, params)
+
+    def test_POST_validates_boot_source(self):
+        self.become_admin()
+
+        params = {
+            'url': 'http://example.com/',
+        }
+        response = self.client.post(self.get_uri(), params)
+        self.assertEqual(httplib.BAD_REQUEST, response.status_code)
+
+    def test_POST_requires_admin(self):
+        params = {
+            'url': 'http://example.com/',
+            'keyring_filename': '',
+            'keyring_data': (
+                factory.make_file_upload(content=sample_binary_data)),
+        }
+        response = self.client.post(self.get_uri(), params)
+        self.assertEqual(httplib.FORBIDDEN, response.status_code)

=== modified file 'src/maasserver/urls_api.py'
--- src/maasserver/urls_api.py	2014-08-21 19:20:31 +0000
+++ src/maasserver/urls_api.py	2014-08-22 15:30:43 +0000
@@ -29,11 +29,15 @@
     BootResourcesHandler,
     )
 from maasserver.api.boot_source_selections import (
+    BootSourceSelectionBackwardHandler,
     BootSourceSelectionHandler,
+    BootSourceSelectionsBackwardHandler,
     BootSourceSelectionsHandler,
     )
 from maasserver.api.boot_sources import (
+    BootSourceBackwardHandler,
     BootSourceHandler,
+    BootSourcesBackwardHandler,
     BootSourcesHandler,
     )
 from maasserver.api.commissioning_scripts import (
@@ -160,6 +164,14 @@
     BootSourceSelectionHandler, authentication=api_auth)
 boot_source_selections_handler = AdminRestrictedResource(
     BootSourceSelectionsHandler, authentication=api_auth)
+boot_source_backward_handler = AdminRestrictedResource(
+    BootSourceBackwardHandler, authentication=api_auth)
+boot_sources_backward_handler = AdminRestrictedResource(
+    BootSourcesBackwardHandler, authentication=api_auth)
+boot_source_selection_backward_handler = AdminRestrictedResource(
+    BootSourceSelectionBackwardHandler, authentication=api_auth)
+boot_source_selections_backward_handler = AdminRestrictedResource(
+    BootSourceSelectionsBackwardHandler, authentication=api_auth)
 license_key_handler = AdminRestrictedResource(
     LicenseKeyHandler, authentication=api_auth)
 license_keys_handler = AdminRestrictedResource(
@@ -267,4 +279,16 @@
     url(r'^boot-sources/(?P<boot_source_id>[^/]+)/selections/(?P<id>[^/]+)/$',
         boot_source_selection_handler,
         name='boot_source_selection_handler'),
+    url(r'^nodegroups/(?P<uuid>[^/]+)/boot-sources/$',
+        boot_sources_backward_handler, name='boot_sources_backward_handler'),
+    url(r'^nodegroups/(?P<uuid>[^/]+)/boot-sources/(?P<id>[^/]+)/$',
+        boot_source_backward_handler, name='boot_source_backward_handler'),
+    url(r'^nodegroups/(?P<uuid>[^/]+)/boot-sources/(?P<boot_source_id>[^/]+)/'
+        'selections/$',
+        boot_source_selections_backward_handler,
+        name='boot_source_selections_backward_handler'),
+    url(r'^nodegroups/(?P<uuid>[^/]+)/boot-sources/(?P<boot_source_id>[^/]+)/'
+        'selections/(?P<id>[^/]+)/$',
+        boot_source_selection_backward_handler,
+        name='boot_source_selection_backward_handler'),
 )