openBackhaul / MicroWaveDeviceInventory

Physical and logical inventory of the MW SDN Domain
Apache License 2.0
5 stars 1 forks source link

Observing 500 internal-server-error when user tried to create link-port #900

Closed Ziabatcha closed 5 months ago

Ziabatcha commented 5 months ago

User tried to create new link port after link creation with min-of-rest data, Observing 500 internal server error

url:
/core-model-1-4:network-control-domain=cache/link=link006007/link-port=1

Body:
{
  "core-model-1-4:link-port": [
    {
      "local-id": "1",
      "link-port-direction": "core-model-1-4:PORT_DIRECTION_INPUT",
      "logical-termination-point": "513250006+LTP-MWPS-TTP-ODU-A"
    }
  ]
}
nardecchia-siae commented 5 months ago

Link should be created not only using the min-of-rest but with the complete structure. For example:

image

With this structure the Api is working correctly.

Do you mean with this issue that id the generic part is missing we should return a different error type?

Ziabatcha commented 5 months ago

If we are using above template, is two different UUID will get created (i.e. one uuid for min-of-rest and another uuid for generic). Is this expected ? then which UUID we will give in POST-/v1/provide-list-of-parallel-links

kmohr-soprasteria commented 5 months ago

I clarified this with Thorsten and created issue #911 with an explanation on generic and minimum for rest sub-schemas.

So in summary: a link should be only created according to either one of the definitions, but not with both at once. For testing the link-port service, it would suffice to write a testLink to the cache following the generic definition with some sample linkId, then the port can be added.

Ziabatcha commented 5 months ago

As per the discussion, Link-port creation will be applicable for non-air-interface port. Current version is not supported for non-air-interface port.