Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GET /sessions is missing status information #46

Closed
maheshc01 opened this issue Sep 23, 2022 · 3 comments
Closed

GET /sessions is missing status information #46

maheshc01 opened this issue Sep 23, 2022 · 3 comments
Labels
enhancement New feature or request QoD

Comments

@maheshc01
Copy link
Contributor

GET /sessions api call is missing a status field in its response body which would be used to indicate the current status of the qos session. This would be a very valuable piece of information for the developer and needs to be incorporated.

@hdamker hdamker added bug Something isn't working enhancement New feature or request QoD and removed bug Something isn't working labels Oct 21, 2022
@SyeddR
Copy link
Collaborator

SyeddR commented Oct 21, 2022

This issue is for GET /sessions/{sessionId}

Following values can be considered for session status

  • active: session has been established and active
    -pending_active : session request has been accepted and in the process of establishment
  • delete: session has been deleted
  • pending delete: session request has been accepted and in the process of deletion
  • failed: network has failed to established the session

We probably also need to match the status in notification api. And include description as well

@jlurien
Copy link
Collaborator

jlurien commented Nov 11, 2022

This is related with the discussions for PR #67
A session life cycle and their possible states should be agreed for that.

@hdamker
Copy link
Collaborator

hdamker commented Feb 10, 2023

Closed with merge of PR #67

@hdamker hdamker closed this as completed Feb 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request QoD
Projects
None yet
Development

No branches or pull requests

4 participants