# Licensed under the Apache License, Version 2.0 (the "License"); you may
# not use this file except in compliance with the License. You may obtain
# a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
# License for the specific language governing permissions and limitations
# under the License.

from keystone.common.validation import parameter_types


_region_properties = {
    'description': parameter_types.description,
    # NOTE(lbragstad): If a user wants to create a region with a specific
    # value as the 'id', it is done in the request URL, e.g.
    # PUT /regions/us-east. This is not required and if an 'id' is not
    # specified in the request, Keystone will assign a uuid as the 'id'
    # for the region.
    'id': {
        'type': 'string'
    },
    'parent_region_id': {
        'type': ['string', 'null']
    },
    'url': parameter_types.url
}

region_create = {
    'type': 'object',
    'properties': _region_properties,
    'additionalProperties': True
    # NOTE(lbragstad): No parameters are required for creating regions.
}

region_update = {
    'type': 'object',
    'properties': _region_properties,
    'minProperties': 1,
    'additionalProperties': True
}

_service_properties = {
    'enabled': parameter_types.boolean,
    'name': parameter_types.name,
    'type': {
        'type': 'string',
        'minLength': 1,
        'maxLength': 255
    }
}

service_create = {
    'type': 'object',
    'properties': _service_properties,
    'required': ['type'],
    'additionalProperties': True,
}

service_update = {
    'type': 'object',
    'properties': _service_properties,
    'minProperties': 1,
    'additionalProperties': True
}

_endpoint_properties = {
    'enabled': parameter_types.boolean,
    'interface': {
        'type': 'string',
        'enum': ['admin', 'internal', 'public']
    },
    'region_id': {
        'type': 'string'
    },
    'region': {
        'type': 'string'
    },
    'service_id': {
        'type': 'string'
    },
    'url': parameter_types.url
}

endpoint_create = {
    'type': 'object',
    'properties': _endpoint_properties,
    'required': ['interface', 'service_id', 'url'],
    'additionalProperties': True
}

endpoint_update = {
    'type': 'object',
    'properties': _endpoint_properties,
    'minProperties': 1,
    'additionalProperties': True
}

@@ -689,283 +689,107 @@
 d): 
-If a user wants to create a region with a specific%0A    # value as the 'id', it is done in the request URL, e.g.%0A    # PUT /regions/us-east. This is not required and if an 'id' is not%0A    # specified in the request, Keystone will assign a uuid as the 'id'%0A    # for the region
+Regions use ID differently. The user can specify the ID%0A    # or it will be generated automatically
 .%0A  

Fix region schema comment