Skip to content

[3.6.1] (maybe others). Wrong definition for IpOverEthernetAddressData

I'm looking at 3.6.1-maintenance, but this maybe happens in other branches too.

Inside SOL002-SOL003/src/definitions/SOL002SOL003_def.yaml , the definition for IpOverEthernetAddressData is (I think) incorrect. The condition expressed in

NOTE 2:Exactly one of "fixedAddresses", "numDynamicAddresses" or "ipAddressRange" shall be present.

Should not not apply to IpOverEthernetAddressData, but only to the children of ipAddresses (note how the fields are prefixed with ">" in Table 4.4.1.10c-1 of SOL003 v3.6.1). Therefore this condition should moved under ipAddresses.

I tried to fork and merge-request but It looks I don't have the right permissions. So instead I am attaching the following picture showing the difference (please forget about the "kokoro" lines; they are caused by my internal tool that merges all openapi specs in a single file)

image

To the left, the current status. To the right, how it should be. Please note how the lines must be properly reindented after moving them to the right place.

The original block to move is around line 441 in SOL002-SOL003/src/definitions/SOL002SOL003_def.yaml

Please contact me at jpc@hpe.com if further clarification is needed

Edit: I discovered that the oneOf array has one wrong element. Currently it has ipAddressRangebut the proper name is addressRange (that would be in the last red line at the bottom right of the picture)

CC: @muhammadh

Edited by postlbauer
To upload designs, you'll need to enable LFS and have an admin enable hashed storage. More information