openvstorage / framework-alba-plugin

The Framework ALBA plugin extends the OpenvStorage GUI with functionality to manage ASDs (Alternate Storage Daemon) and Seagate Kinetic drives.
Other
2 stars 3 forks source link

Linking alba backends requires domain-guid. API code disagrees #296

Closed openvstorage-ci closed 7 years ago

openvstorage-ci commented 7 years ago

From @JeffreyDevloo on December 5, 2016 10:24

Problem description

Domain_guid is not a required parameter in the metadata. Code ovs/lib/alba.py -- line 1185:

'backend_info': (dict, {'domain_guid': (str, Toolbox.regex_guid, False),

However, if this parameter is not given, the api will return 'domain_guid'. eg:

        task_guid = api.post(
            api='/alba/backends/{0}/link_alba_backends'
                .format(BackendHelper.get_alba_backend_guid_by_name(globalbackend_name)),
            data=data
        )

        task_result = api.wait_for_task(task_id=task_guid, timeout=timeout)

        if not task_result[0]:
            error_msg = "Linking backend `{0}` to global backend `{1}` has failed with error '{2}'".format(
                albabackend_name, globalbackend_name, task_result[1])
            BackendSetup.LOGGER.error(error_msg)
            raise RuntimeError(error_msg)

Gave:

RuntimeError: Linking backend `mybackend02` to global backend `mybackend-global` has failed with error ''domain_guid''

Packages

- openvstorage                         2.7.7-rev.4395.5099afe-1        amd64        openvStorage
- openvstorage-backend                 1.7.7-rev.851.85b61b8-1         amd64        openvStorage Backend plugin
- openvstorage-backend-core            1.7.7-rev.851.85b61b8-1         amd64        openvStorage Backend plugin core
- openvstorage-backend-webapps         1.7.7-rev.851.85b61b8-1         amd64        openvStorage Backend plugin Web Applications
- openvstorage-core                    2.7.7-rev.4395.5099afe-1        amd64        openvStorage core
- openvstorage-hc                      1.7.7-rev.851.85b61b8-1         amd64        openvStorage Backend plugin HyperConverged
- openvstorage-sdm                     1.6.7-rev.478.a1849c5-1         amd64        Open vStorage Backend ASD Manager
- openvstorage-webapps                 2.7.7-rev.4395.5099afe-1        amd64        openvStorage Web Applications

Copied from original issue: openvstorage/framework#1240

kvanhijf commented 7 years ago

https://github.com/openvstorage/framework-alba-plugin/pull/297 --> openvstorage-backend-1.7.7-rev.853.050d474

JeffreyDevloo commented 7 years ago

Steps