google/digitalbuildings

etags break the instance validator

bsimmons-onboard opened this issue · 4 comments

Describe the bug
The instance validator doesn't work on entities that include etags.

To Reproduce
minimal.yaml
6a99999c-05c8-4a01-8c4a-99f256be4c7f:
code: Test-Building
etag: d7b7777e
type: FACILITIES/BUILDING

python instance_validator.py -i minimal.yaml

while parsing a mapping unexpected key not in schema 'etag' in "<unicode string>", line 3, column 1: etag: d7b7777e ^ (line: 3)

OS:
Ubuntu

This has to do with a conflict between the CONFIG_METADATA operation and the existence of etags. Looking into it.

@charbull, I can confirm that adding the UPDATE operation allowed me to run the instance_validator on an building_config with etags.

cool, thank you for confirming. closing for now, feel free to reopen if needed.