openstack/cinder
Code Issues Proposed changes

[api-ref]Host name is not necessary to contain '@'

Browse Source 
As shown in https://zuul.opendev.org/t/openstack/build/e39652d2dc8f4cb394933da8f9a9bb59/log/job-output.txt,
host name may contain '@', but not necessary to contain '@',
like:
Request (VolumeHostsV3RbacTest:test_show_host):
200 GET https://10.208.227.163/volume/v3/67d6a092ee7d4cd4a6b3ae37812bec50/os-hosts/ubuntu-bionic-rax-dfw-0010673901
Response Body: {"host":
                 [{"resource":
                   {"volume_count": "2", "total_volume_gb": "2",
                    "total_snapshot_gb": "1", "project": "(total)",
                    "host": "ubuntu-bionic-rax-dfw-0010673901",
                    "snapshot_count": "1"}},
                  {"resource":
                   {"volume_count": "1", "total_volume_gb": "1",
                     "total_snapshot_gb": "0",
                     "project": "2aba5d9825ac47a7b8c3ea805af34a75",
                     "host": "ubuntu-bionic-rax-dfw-0010673901",
                     "snapshot_count": "0"}}}

Besides, the format 'host@backend' should be that of host_name_backend,
not host.

Change-Id: Id2c0be1e4d0471686674ef781b722087638c4c50
This commit is contained in:
zhufl 2019-09-04 08:41:33 +08:00
parent 0023c2b58b
commit 6ff2c4a1d1
2 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
api-ref/source/v2/parameters.yaml
View File

@ -665,7 +665,7 @@ has_dependent_backups:
host:
description: |
The OpenStack Block Storage host where the
existing volume resides. Host format is ``host@backend``.
existing volume resides.
in: body
required: true
type: string
@ -677,7 +677,8 @@ host_name:
type: string
host_name_1:
description: |
The name of the host that hosts the storage backend.
The name of the host that hosts the storage backend,
may take the format of ``host@backend``.
in: body
required: true
type: string

5
api-ref/source/v3/parameters.yaml
View File

@ -1314,7 +1314,7 @@ has_dependent_backups:
host:
description: |
The OpenStack Block Storage host where the
existing volume resides. Host format is ``host@backend``.
existing volume resides.
in: body
required: true
type: object
@ -1333,7 +1333,8 @@ host_name:
type: string
host_name_backend:
description: |
The name of the host that hosts the storage backend.
The name of the host that hosts the storage backend,
may take the format of ``host@backend``.
in: body
required: true
type: string