x
/
mogan-specs
Code Issues Proposed changes

Change adopt to manage in managing-existing-node spec 

According to the disscusion of IRC meeting, we decide to
use an new 'POST v1/manage','GET v1/manageable_servers' API and
create a new ManageServerController in spec.

Implements: bp manage-existing-bms

Change-Id: Ie77a55183da8f72631f5a0a589a88205c90d5607
This commit is contained in:
wanghao 2017-08-03 11:25:42 +08:00
parent 1e36a74701
commit 2824bc3ab2
1 changed files with 10 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
specs/pike/approved/manage-existing-bms.rst
View File

@ -39,7 +39,7 @@ Proposed change
* Introduce a new admin only API, which supports to query nodes that could * Introduce a new admin only API, which supports to query nodes that could
be managed by mogan. This API will pass down the request to drivers, which be managed by mogan. This API will pass down the request to drivers, which
needs to add a new driver interface `list_adoptable_nodes`, there will be needs to add a new driver interface `get_manageable_nodes`, there will be
driver specified criterias of which nodes are manageable. For ironic, it driver specified criterias of which nodes are manageable. For ironic, it
should be nodes in active state but without instance_uuid associated, and should be nodes in active state but without instance_uuid associated, and
the resource class field should be well set. the resource class field should be well set.
@ -68,12 +68,10 @@ None
REST API impact REST API impact
--------------- ---------------
#. Add a new custom action named 'manage' in ServerController, of course #. Add a new URI named 'manageable_servers' with ManageableServersController,
it is a admin only API:: of course it is a admin only API::
_custom_actions = { POST v1/manageable_servers
'manage': ['POST']
}
#. The management API schema is like this:: #. The management API schema is like this::
@ -104,11 +102,13 @@ REST API impact
'additionalProperties': False, 'additionalProperties': False,
} }
#. Add a new API that will list adoptable nodes which will include all needed #. Add a new API that will list manageable servers which will include all
informations when calling manage API. It may look like this:: needed informations when calling manage API. It may look like this::
GET v1/manageable_servers
{ {
"adoptable_nodes": [ "manageable_servers": [
{ {
"name": "test_server", "name": "test_server",
"ports": [ "ports": [
@ -196,7 +196,7 @@ Unit Testing will be added.
Documentation Impact Documentation Impact
==================== ====================
Docs about adopt/manage servers will be added, including the preparation work Docs about manage servers will be added, including the preparation work
for operator. for operator.
References References