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

Duration in QosProfile and SessionInfo #249

Closed
jlurien opened this issue Dec 20, 2023 · 3 comments
Closed

Duration in QosProfile and SessionInfo #249

jlurien opened this issue Dec 20, 2023 · 3 comments
Labels
enhancement New feature or request Fall24 Relevant for maintenance of Fall24 release QoD

Comments

@jlurien
Copy link
Collaborator

jlurien commented Dec 20, 2023

Problem description
In the latest definition (v0.10.0-rc), we have the option to provide a duration while creating the session and there, the maximum duration of a session is limited to 86400s (1d).
We also have discovery of QoSProfiles by mean of GET /qos-profiles, and there we return per QoSProfile a minDuration and maxDuration, that is no limited. We potentially may allow sessions for up to days.

There is a lack of consistency between both endpints.

Possible evolution

It makes sense to remove the explicit limitation of duration in sessionInfo, and rely on the discovery of QoSProfiles to let the client know about the allowed limits. The same schema for Duration can be aplied in both.

@jlurien jlurien added the enhancement New feature or request label Dec 20, 2023
@hdamker
Copy link
Collaborator

hdamker commented Jan 23, 2024

With last QoD call we have discussed to keep the limit for duration in sessionInfo within v0.10.0 and add a note for the maxDuration parameter within QoSProfile that within v0.10.0 it shouldn't exceed of 86400 seconds/1 day.

The discussion if we should remove the explicit limitation for duration in sessionInfo will be done for the next release, as well as the alignment of the schema for duration.

@hdamker
Copy link
Collaborator

hdamker commented Jan 26, 2024

Removing 'v0.10.0' as the part for this release is temporarily fixed with #256

@hdamker
Copy link
Collaborator

hdamker commented Jun 14, 2024

Fixed with #296

@hdamker hdamker closed this as completed Jun 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Fall24 Relevant for maintenance of Fall24 release QoD
Projects
None yet
Development

No branches or pull requests

2 participants