public icon indicating copy to clipboard operation
public copied to clipboard

Ordering of /qos/classifiers/classifier/terms/term

Open panduranganrs opened this issue 2 years ago • 3 comments

Each term has a set of match conditions and an associated action and we could have multiple terms within a classifier. However it is not clear how to order the terms (in case the terms have overlapping match criteria, the order would determine the term and action that gets applied).

For example, in ACL model, we have this clearly defined using sequence-id, but the same isn't clear in case of /qos/classifiers/classifier/terms/term.

panduranganrs avatar Aug 18 '22 13:08 panduranganrs

Thanks for reporting this. I think you are correct that this is missing and is needed.

One could introduce ordered-by user statement to the list per rfc6020 or a sequence-id

Do you have another suggestion?

dplore avatar Aug 26 '22 01:08 dplore

Thanks for the reply. I think it is better to follow the ACL model (using a sequence-id) to be consistent. I don't have any other suggestion.

panduranganrs avatar Aug 26 '22 08:08 panduranganrs

@panduranganrs thanks for the issue. We reviewed on Sept 1 OC community meeting. This sounds like a good addition. Please submit a Pull Request with your proposal and we will get it reviewed. Don't forget to review the contributing guide, which includes a request to include public references to existing implementations related to your PR.

dplore avatar Sep 01 '22 15:09 dplore

This issue is stale because it has been open 180 days with no activity. If you wish to keep this issue active, please remove the stale label or add a comment, otherwise will be closed in 14 days.

github-actions[bot] avatar May 16 '24 02:05 github-actions[bot]