openBackhaul / AirInterfacePowerSaver

Pranay's first Application on the MW SDN Controller
Apache License 2.0
1 stars 1 forks source link

Need for clarification : /v1/delete-link-from-power-saving-status-table service behavior #102

Open IswaryaaS opened 7 months ago

IswaryaaS commented 7 months ago

In AIPS specification, we have a service /v1/delete-link-from-power-saving-status-table. This service is allowed to send two types of positive responses , one is 200 (when the entry is not eligible to be deleted in Elasticsearch) and 204 (when the entry is successfully deleted in Elasticsearch).

But, we could see that the description provided for /v1/delete-link-from-power-saving-status-table in AIPS specification and diagram 705_deleteLinkFromPowerSavingStatusTable are quite different.

As per diagram 705_deleteLinkFromPowerSavingStatusTable, the entry shall not be deleted if deviation-from-original-state-list != empty, and deviation-from-original-state-list shall be sent as response. But in specification (OAS file), the entry is not expected to be deleted if module-to-restore-original-state-list != empty and module-to-restore-original-state-list shall be sent in response for 200.

Kindly let us know the expected behavior regarding this service.

kmohr-soprasteria commented 2 months ago

As diagram and OAS should be aligned, I will create a separate issue, to modify the diagram accordingly. Please implement as specified in the OAS.