summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorjulien zhang <zhang.jun3g@zte.com.cn>2018-01-23 16:03:37 +0000
committerGerrit Code Review <gerrit@opnfv.org>2018-01-23 16:03:37 +0000
commit811ff5abc205d961d45ee00c0b449c47893433ae (patch)
tree413ea2dee5f3ef9378a06d285b60008c401d6673
parent7d2b744815ddbaf494872a0e9b82bdcf40b85ad5 (diff)
parent9f61971b20d02c8d0fdbd8a7034f957a7fb9177a (diff)
Merge "[PDF] [SPEC] Add 'version: 1.0'"
-rw-r--r--config/pdf/pod1.encrypted.yaml2
-rw-r--r--config/pdf/pod1.schema.yaml313
-rw-r--r--config/pdf/pod1.yaml2
-rwxr-xr-xconfig/utils/check-schema.sh2
-rwxr-xr-xconfig/utils/validate_schema.py13
-rw-r--r--labs/arm/pod5.yaml1
-rw-r--r--labs/arm/pod6.yaml1
-rw-r--r--labs/bii/pod1.yaml1
-rw-r--r--labs/ericsson/pod1.yaml1
-rw-r--r--labs/ericsson/pod2.yaml1
-rw-r--r--labs/huawei/pod1.yaml1
-rw-r--r--labs/intel/pod18.yaml1
-rw-r--r--labs/lf/pod2.yaml1
-rw-r--r--labs/lf/pod5.yaml1
-rw-r--r--labs/zte/pod1.yaml1
-rw-r--r--labs/zte/pod2.yaml1
-rw-r--r--labs/zte/pod3.yaml1
-rw-r--r--labs/zte/pod9.yaml1
18 files changed, 209 insertions, 136 deletions
diff --git a/config/pdf/pod1.encrypted.yaml b/config/pdf/pod1.encrypted.yaml
index c1e8f78c..b5c78b50 100644
--- a/config/pdf/pod1.encrypted.yaml
+++ b/config/pdf/pod1.encrypted.yaml
@@ -8,6 +8,8 @@
---
### POD descriptor file ###
+version: 1.0
+
details:
pod_owner: Lab Owner
contact: email@address.com
diff --git a/config/pdf/pod1.schema.yaml b/config/pdf/pod1.schema.yaml
index a602b562..2a96d0b2 100644
--- a/config/pdf/pod1.schema.yaml
+++ b/config/pdf/pod1.schema.yaml
@@ -10,173 +10,216 @@ $schema: 'http://json-schema.org/schema#'
$id: 'https://github.com/opnfv/pharos/blob/master/config/pdf/pod1.yaml'
definitions:
+ ############################################################################
+ # Low-level, general purpose definitions, unversioned
ip_address:
type: 'string' # NOTE: we don't validate this is a valid addr (yet)
mac_address:
type: 'string'
pattern: '^([0-9A-Fa-f]{2}[:-]){5}([0-9A-Fa-f]{2})$'
- # Common node schema for jumpserver, cluster nodes
+ ############################################################################
+ # Mid-level definitions, common schema for jumpserver & cluster nodes
node:
- type: 'object'
- properties:
- type:
- type: 'string'
- enum: ['baremetal', 'virtual']
- vendor:
- type: 'string'
- model:
- type: 'string'
- arch:
- type: 'string'
- enum: ['aarch64', 'x86_64']
- cpus:
- type: 'number'
- cpu_cflags:
- type: 'string'
- cores:
- type: 'number'
- memory:
- type: 'string'
- pattern: '^[0-9]+[GT]B?$'
- required: ['type', 'vendor', 'model', 'arch', 'cpus', 'cpu_cflags', 'cores', 'memory']
- additionalProperties: false
- disks:
- type: 'array'
- items:
+ v1.0:
type: 'object'
properties:
- name:
+ type:
+ type: 'string'
+ enum: ['baremetal', 'virtual']
+ vendor:
type: 'string'
- pattern: '^disk[0-9]+$'
- disk_capacity:
+ model:
type: 'string'
- pattern: '^[0-9]+[MGT]B?$'
- disk_type:
+ arch:
type: 'string'
- enum: ['hdd', 'ssd', 'cdrom', 'tape']
- disk_interface:
+ enum: ['aarch64', 'x86_64']
+ cpus:
+ type: 'number'
+ cpu_cflags:
type: 'string'
- enum: ['sata', 'sas', 'ssd', 'nvme', 'scsi', 'iscsi']
- disk_rotation:
+ cores:
type: 'number'
- enum: [0, 5400, 7200, 10000, 15000]
- required: ['name', 'disk_capacity', 'disk_type', 'disk_interface', 'disk_rotation']
+ memory:
+ type: 'string'
+ pattern: '^[1-9]\d*[MGT]B?$'
+ required: ['type', 'vendor', 'model', 'arch', 'cpus', 'cpu_cflags', 'cores', 'memory']
additionalProperties: false
+ disks:
+ v1.0:
+ type: 'array'
+ items:
+ type: 'object'
+ properties:
+ name:
+ type: 'string'
+ pattern: '^disk[0-9]+$'
+ disk_capacity:
+ type: 'string'
+ pattern: '^[1-9][\d\.]*[MGT]B?$'
+ disk_type:
+ type: 'string'
+ enum: ['hdd', 'ssd', 'cdrom', 'tape']
+ disk_interface:
+ type: 'string'
+ enum: ['sata', 'sas', 'ssd', 'nvme', 'scsi', 'iscsi']
+ disk_rotation:
+ type: 'number'
+ enum: [0, 5400, 7200, 10000, 15000]
+ required: ['name', 'disk_capacity', 'disk_type', 'disk_interface', 'disk_rotation']
+ additionalProperties: false
remote_management:
- type: 'object'
- properties:
- type:
- type: 'string'
- enum: ['ipmi', 'amt']
- versions:
- type: 'array'
- items:
- type: 'number'
- enum: [1.0, 2.0]
- user:
- type: 'string'
- pass:
- type: 'string'
- address:
- $ref: '#/definitions/ip_address'
- mac_address:
- $ref: '#/definitions/mac_address'
- required: ['type', 'versions', 'user', 'pass', 'address', 'mac_address']
- additionalProperties: false
- interfaces:
- type: 'array'
- items:
+ v1.0:
type: 'object'
properties:
- name:
+ type:
+ type: 'string'
+ enum: ['ipmi', 'amt']
+ versions:
+ type: 'array'
+ items:
+ type: 'number'
+ enum: [1.0, 2.0]
+ user:
+ type: 'string'
+ pass:
type: 'string'
- pattern: '^nic[0-9]+$'
- mac_address:
- $ref: '#/definitions/mac_address'
- # Optional
address:
$ref: '#/definitions/ip_address'
- # Optional
- vlan:
- oneOf:
- - type: 'string'
- pattern: '^(native|[1-9][0-9]{0,3})(\|(native|[1-9][0-9]{0,3}))*$'
- - type: 'integer'
- mininum: 1
- maximum: 4095
- # Optional
- speed:
- type: 'string'
- enum: ['1gb', '10gb', '25gb', '40gb']
- # FIXME: mandatory for nodes?
- # Optional
- features:
- type: ['string', 'null']
- pattern: '^((dpdk|sriov)\|?)*$'
- # FIXME: mandatory for nodes?
- required: ['name', 'mac_address']
+ mac_address:
+ $ref: '#/definitions/mac_address'
+ required: ['type', 'versions', 'user', 'pass', 'address', 'mac_address']
additionalProperties: false
-
-# Do not allow any properties not defined here. This lets us catch typos.
-additionalProperties: false
-
-properties:
+ interfaces:
+ v1.0:
+ type: 'array'
+ items:
+ type: 'object'
+ properties:
+ name:
+ type: 'string'
+ pattern: '^nic[0-9]+$'
+ mac_address:
+ $ref: '#/definitions/mac_address'
+ # Optional
+ address:
+ $ref: '#/definitions/ip_address'
+ # Optional
+ vlan:
+ oneOf:
+ - type: 'string'
+ pattern: '^(native|[1-9][0-9]{0,3})(\|(native|[1-9][0-9]{0,3}))*$'
+ - type: 'integer'
+ mininum: 1
+ maximum: 4095
+ # Optional
+ speed:
+ type: 'string'
+ enum: ['1gb', '10gb', '25gb', '40gb']
+ # FIXME: mandatory for nodes?
+ # Optional
+ features:
+ type: ['string', 'null']
+ pattern: '^((dpdk|sriov)\|?)*$'
+ # FIXME: mandatory for nodes?
+ required: ['name', 'mac_address']
+ additionalProperties: false
+ ############################################################################
+ # Top-level PDF blocks, versioned
details:
- type: 'object'
- properties:
- type:
- type: 'string'
- enum: ['production', 'development']
- pod_owner:
- type: 'string'
- contact:
- type: 'string'
- pattern: '^([a-zA-Z0-9_.+-]+@[a-zA-Z0-9-]+\.[a-zA-Z0-9-.]+[,; ]*)+$'
- lab:
- type: 'string'
- location:
- type: 'string'
- link:
- type: 'string'
- required: ['type', 'pod_owner', 'contact', 'lab', 'location', 'link']
- additionalProperties: false
+ v1.0:
+ type: 'object'
+ properties:
+ type:
+ type: 'string'
+ enum: ['production', 'development']
+ pod_owner:
+ type: 'string'
+ contact:
+ type: 'string'
+ pattern: '^([a-zA-Z0-9_.+-]+@[a-zA-Z0-9-]+\.[a-zA-Z0-9-.]+[,; ]*)+$'
+ lab:
+ type: 'string'
+ location:
+ type: 'string'
+ link:
+ type: 'string'
+ required: ['type', 'pod_owner', 'contact', 'lab', 'location', 'link']
+ additionalProperties: false
jumphost:
- type: 'object'
- properties:
- name:
- type: 'string'
- node:
- $ref: '#/definitions/node'
- disks:
- $ref: '#/definitions/disks'
- os:
- type: 'string'
- remote_params: # Optional YAML anchor, contents will be validated below
- type: 'object'
- remote_management:
- $ref: '#/definitions/remote_management'
- interfaces:
- $ref: '#/definitions/interfaces'
- required: ['name', 'node', 'disks', 'os', 'remote_management', 'interfaces']
- additionalProperties: false
- nodes:
- type: 'array'
- items:
+ v1.0:
type: 'object'
properties:
name:
type: 'string'
node:
- $ref: '#/definitions/node'
+ $ref: '#/definitions/node/v1.0'
disks:
- $ref: '#/definitions/disks'
+ $ref: '#/definitions/disks/v1.0'
os:
type: 'string'
- remote_params: # Optional YAML anchor, contents will be validated after inject
+ remote_params: # Optional YAML anchor, contents will be validated below
type: 'object'
remote_management:
- $ref: '#/definitions/remote_management'
+ $ref: '#/definitions/remote_management/v1.0'
interfaces:
- $ref: '#/definitions/interfaces'
- required: ['name', 'node', 'disks', 'remote_management', 'interfaces']
+ $ref: '#/definitions/interfaces/v1.0'
+ required: ['name', 'node', 'disks', 'os', 'remote_management', 'interfaces']
additionalProperties: false
+ nodes:
+ v1.0:
+ type: 'array'
+ items:
+ type: 'object'
+ properties:
+ name:
+ type: 'string'
+ node:
+ $ref: '#/definitions/node/v1.0'
+ disks:
+ $ref: '#/definitions/disks/v1.0'
+ os:
+ type: 'string'
+ remote_params: # Optional YAML anchor, contents will be validated after inject
+ type: 'object'
+ remote_management:
+ $ref: '#/definitions/remote_management/v1.0'
+ interfaces:
+ $ref: '#/definitions/interfaces/v1.0'
+ required: ['name', 'node', 'disks', 'remote_management', 'interfaces']
+ additionalProperties: false
+
+##############################################################################
+# Top-level structure:
+# - define all possible top-level keys, without enforcing a schema for them,
+# just so we can disallow additional properties;
+# - require 'version' and allow dynamically generated 'version_x.y' key;
+properties:
+ # version_x.y keys are auto-added by `validate_schema.py` based on 'version'
+ version_1.0:
+ type: 'boolean'
+ version:
+ type: 'number'
+ enum: [1.0]
+ details:
+ type: 'object'
+ jumphost:
+ type: 'object'
+ nodes:
+ type: 'array'
+required: ['version']
+additionalProperties: false
+
+##############################################################################
+# Schema versioning
+# - based on dynamically added 'version_x.y', require additional PDF blocks
+# and validate them against the proper schema version;
+dependencies:
+ version_1.0:
+ properties:
+ details:
+ $ref: '#/definitions/details/v1.0'
+ jumphost:
+ $ref: '#/definitions/jumphost/v1.0'
+ nodes:
+ $ref: '#/definitions/nodes/v1.0'
+ required: ['details', 'jumphost', 'nodes']
diff --git a/config/pdf/pod1.yaml b/config/pdf/pod1.yaml
index c2d07349..d0cd7931 100644
--- a/config/pdf/pod1.yaml
+++ b/config/pdf/pod1.yaml
@@ -8,6 +8,8 @@
---
### POD descriptor file ###
+version: 1.0
+
details:
pod_owner: Lab Owner
contact: email@address.com
diff --git a/config/utils/check-schema.sh b/config/utils/check-schema.sh
index 321c5ced..61bdec22 100755
--- a/config/utils/check-schema.sh
+++ b/config/utils/check-schema.sh
@@ -16,7 +16,7 @@ RC=0
while IFS= read -r lab_config; do
pdf_cmd="${VALIDATE_SCHEMA} -s ${PDF_SCHEMA} -y ${lab_config}"
echo "###################### ${lab_config} ######################"
- pdf_out=$(${pdf_cmd} |& sed 's|ENC\[PKCS.*\]|opnfv|g')
+ pdf_out=$(${pdf_cmd} |& sed 's|ENC\[PKCS.*\][\\n]*|opnfv|g')
if [ -z "${pdf_out}" ]; then
SUMMARY+=";${lab_config#labs/};OK;\n"
echo "[PDF] [OK] ${pdf_cmd}"
diff --git a/config/utils/validate_schema.py b/config/utils/validate_schema.py
index cb404554..6bdc3bdb 100755
--- a/config/utils/validate_schema.py
+++ b/config/utils/validate_schema.py
@@ -22,6 +22,19 @@ with open(ARGS.yaml) as _:
with open(ARGS.schema) as _:
_SCHEMA = yaml.safe_load(_)
+# Draft 4 (latest supported by py-jsonschema) does not support value-based
+# decisions properly, see related github issue:
+# https://github.com/json-schema-org/json-schema-spec/issues/64
+# Workaround: build 'version_x.y: true' on the fly based on 'version: x.y'
+def schema_version_workaround(node):
+ """Traverse nested dictionaries and handle 'version' key where found."""
+ if 'version' in node:
+ node['version_{0}'.format(node['version'])] = True
+ for item in node.items():
+ if type(item) is dict:
+ schema_version_workaround(item)
+schema_version_workaround(_DICT)
+
_VALIDATOR = jsonschema.Draft4Validator(_SCHEMA)
for error in _VALIDATOR.iter_errors(_DICT):
raise RuntimeError(str(error))
diff --git a/labs/arm/pod5.yaml b/labs/arm/pod5.yaml
index 3708b7f0..3a55678e 100644
--- a/labs/arm/pod5.yaml
+++ b/labs/arm/pod5.yaml
@@ -8,6 +8,7 @@
---
### Enea ARM POD 5 descriptor file ###
+version: 1.0
details:
pod_owner: ENEA AB
contact: dan.lilliehorn@enea.com
diff --git a/labs/arm/pod6.yaml b/labs/arm/pod6.yaml
index dc53888c..3b558164 100644
--- a/labs/arm/pod6.yaml
+++ b/labs/arm/pod6.yaml
@@ -8,6 +8,7 @@
---
### Enea ARM POD 6 descriptor file ###
+version: 1.0
details:
pod_owner: ENEA AB
contact: dan.lilliehorn@enea.com
diff --git a/labs/bii/pod1.yaml b/labs/bii/pod1.yaml
index 44eedc37..40ac58d4 100644
--- a/labs/bii/pod1.yaml
+++ b/labs/bii/pod1.yaml
@@ -8,6 +8,7 @@
---
### This is a BII POD1 descriptor file ###
+version: 1.0
details:
pod_owner: ylong
contact: ylong@biigroup.cn, zjtang@biigroup.cn
diff --git a/labs/ericsson/pod1.yaml b/labs/ericsson/pod1.yaml
index ae04b309..1c2f5ae4 100644
--- a/labs/ericsson/pod1.yaml
+++ b/labs/ericsson/pod1.yaml
@@ -8,6 +8,7 @@
---
### POD descriptor file ###
+version: 1.0
details:
pod_owner: Jose Lausuch
contact: jose.lausuch@ericsson.com
diff --git a/labs/ericsson/pod2.yaml b/labs/ericsson/pod2.yaml
index 8814c42d..4d25af48 100644
--- a/labs/ericsson/pod2.yaml
+++ b/labs/ericsson/pod2.yaml
@@ -1,6 +1,7 @@
---
### POD descriptor file ###
+version: 1.0
details:
pod_owner: Jose Lausuch
contact: jose.lausuch@ericsson.com
diff --git a/labs/huawei/pod1.yaml b/labs/huawei/pod1.yaml
index f48d8239..807c6f3e 100644
--- a/labs/huawei/pod1.yaml
+++ b/labs/huawei/pod1.yaml
@@ -9,6 +9,7 @@
---
### POD descriptor file ###
+version: 1.0
details:
pod_owner: Tianwei Wu
contact: wutianwei1@hauwei.com
diff --git a/labs/intel/pod18.yaml b/labs/intel/pod18.yaml
index b1123b7e..22738910 100644
--- a/labs/intel/pod18.yaml
+++ b/labs/intel/pod18.yaml
@@ -2,6 +2,7 @@
### POD descriptor file ###
+version: 1.0
details:
pod_owner: Jack Morgan
contact: jack.morgan@intel.com
diff --git a/labs/lf/pod2.yaml b/labs/lf/pod2.yaml
index ab60bdb8..219b2a68 100644
--- a/labs/lf/pod2.yaml
+++ b/labs/lf/pod2.yaml
@@ -8,6 +8,7 @@
---
### LF POD 2 descriptor file ###
+version: 1.0
details:
pod_owner: Trevor Bramwell
contact: tbramwell@linuxfoundation.org
diff --git a/labs/lf/pod5.yaml b/labs/lf/pod5.yaml
index 60ab4029..61ca3ebe 100644
--- a/labs/lf/pod5.yaml
+++ b/labs/lf/pod5.yaml
@@ -8,6 +8,7 @@
---
### LF POD 5 descriptor file ###
+version: 1.0
details:
pod_owner: Trevor Bramwell
contact: tbramwell@linuxfoundation.org
diff --git a/labs/zte/pod1.yaml b/labs/zte/pod1.yaml
index 1b4d1ad6..9a4a8a02 100644
--- a/labs/zte/pod1.yaml
+++ b/labs/zte/pod1.yaml
@@ -9,6 +9,7 @@
### ZTE POD 1 descriptor file ###
# refering to https://gerrit.opnfv.org/gerrit/#/c/23727/9/config/pod1.yaml
+version: 1.0
details:
pod_owner: Alex Yang
contact: yangyang1@zte.com.cn
diff --git a/labs/zte/pod2.yaml b/labs/zte/pod2.yaml
index f51d7f79..e5dd5e67 100644
--- a/labs/zte/pod2.yaml
+++ b/labs/zte/pod2.yaml
@@ -9,6 +9,7 @@
### ZTE POD 2 descriptor file ###
# refering to https://gerrit.opnfv.org/gerrit/#/c/23727/9/config/pod1.yaml
+version: 1.0
details:
pod_owner: Alex Yang
contact: yangyang1@zte.com.cn
diff --git a/labs/zte/pod3.yaml b/labs/zte/pod3.yaml
index 063b7dfc..b6768396 100644
--- a/labs/zte/pod3.yaml
+++ b/labs/zte/pod3.yaml
@@ -9,6 +9,7 @@
### ZTE POD 3 descriptor file ###
# refering to https://gerrit.opnfv.org/gerrit/#/c/23727/9/config/pod1.yaml
+version: 1.0
details:
pod_owner: Alex Yang
contact: yangyang1@zte.com.cn
diff --git a/labs/zte/pod9.yaml b/labs/zte/pod9.yaml
index 76d4aa9e..c18d244e 100644
--- a/labs/zte/pod9.yaml
+++ b/labs/zte/pod9.yaml
@@ -8,6 +8,7 @@
---
### ZTE POD 9 descriptor file ###
+version: 1.0
details:
pod_owner: Zhiijang Hu
contact: huzhijiang@zte.com.cn