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

[DO NOT MERGE] Update openconfig-qos-interfaces.yang with aged-packets, aged-bytes counters #913

Closed
wants to merge 6 commits into from

Conversation

rszarecki
Copy link
Contributor

@rszarecki rszarecki commented Jul 13, 2023

Summary

Enhance qos stats by additional counter for packet droped due to timeout (aged).

Description

Current drop couter is defined as:

/qos/interfaces/interface/output/queues/queue/state/dropped-pkts
description: Number of packets dropped by the queue due to overrun

Condifering overruning of queue is situatin when there is no available buffer space for packet.

However when there is available buffer space but queue service (de-queue) rate is too-little| next to null, packet would stay in queu for seconds or mitutes. What consumes resources and provides no benefits. Hence aging is purging this packets for NOT overrun queues.

###Vendor implementation
Aging is commonly implemented, but counters are inconsistently exposed to UI

Arista: “show platform fap voq delete” (arista do not provide aggregated per egress interface view)

------------- show platform fap voq delete -------------

 Egress Intf     TC     Ingress Chip     Core     Count      First Occurrence       Last Occurrence  
-------------- ------ ---------------- -------- --------- ----------------------- -------------------
 Et3/1            1     Jericho0            0         1     2022-05-19 18:48:04   2022-05-19 18:48:04
                  0     Jericho0            0         6     2022-05-19 18:48:02   2022-11-09 19:45:32
                  1     Jericho0            1         1     2022-05-19 18:48:04   2022-05-19 18:48:04

Cisco:
Juniper:
Nokia:

@rszarecki rszarecki requested a review from a team as a code owner July 13, 2023 20:20
@rszarecki rszarecki changed the title Update openconfig-qos-interfaces.yang Update openconfig-qos-interfaces.yang with aged-packets|, aged-bytes conter Jul 13, 2023
@OpenConfigBot
Copy link

OpenConfigBot commented Jul 13, 2023

No major YANG version changes in commit 6df4bfa

@rszarecki rszarecki closed this Jul 13, 2023
@rszarecki rszarecki reopened this Jul 13, 2023
@rszarecki rszarecki changed the title Update openconfig-qos-interfaces.yang with aged-packets|, aged-bytes conter [DO NOT MERGE] Update openconfig-qos-interfaces.yang with aged-packets|, aged-bytes conter Jul 13, 2023
@rszarecki rszarecki changed the title [DO NOT MERGE] Update openconfig-qos-interfaces.yang with aged-packets|, aged-bytes conter [DO NOT MERGE] Update openconfig-qos-interfaces.yang with aged-packets, aged-bytes conter Jul 13, 2023
@rszarecki rszarecki changed the title [DO NOT MERGE] Update openconfig-qos-interfaces.yang with aged-packets, aged-bytes conter [DO NOT MERGE] Update openconfig-qos-interfaces.yang with aged-packets, aged-bytes counters Jul 13, 2023
@OpenConfigBot
Copy link

OpenConfigBot commented Jul 20, 2023

Compatibility Report for commit 6df4bfa:
pyangbind@23a7a0d

@rszarecki rszarecki closed this Jul 20, 2023
@rszarecki rszarecki reopened this Jul 20, 2023
@rszarecki
Copy link
Contributor Author

Closing this #PR as it is not immedietly needed

@rszarecki rszarecki closed this Sep 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants