Skip to content
This repository was archived by the owner on Apr 15, 2026. It is now read-only.

DISPATCH-1130 Expose link priority#389

Closed
ErnieAllen wants to merge 2 commits into
apache:masterfrom
ErnieAllen:ernie-DISPATCH-1130
Closed

DISPATCH-1130 Expose link priority#389
ErnieAllen wants to merge 2 commits into
apache:masterfrom
ErnieAllen:ernie-DISPATCH-1130

Conversation

@ErnieAllen
Copy link
Copy Markdown
Contributor

Adds priority to router.link management entity in the schema.
Connects link.priority to schema attribute.
Adds priority column to qdstat -l

@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #389 into master will decrease coverage by 0.03%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #389      +/-   ##
==========================================
- Coverage   85.06%   85.02%   -0.04%     
==========================================
  Files          73       73              
  Lines       16523    16525       +2     
==========================================
- Hits        14055    14051       -4     
- Misses       2468     2474       +6
Impacted Files Coverage Δ
src/router_core/agent_link.c 63.63% <100%> (+0.41%) ⬆️
src/router_core/core_timer.c 94.64% <0%> (-1.79%) ⬇️
src/router_core/modules/core_test_hooks.c 89.54% <0%> (-1.37%) ⬇️
src/router_core/router_core.c 92.05% <0%> (-0.34%) ⬇️
src/container.c 76.55% <0%> (-0.2%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d1d6e92...a81784b. Read the comment docs.

@ganeshmurthy
Copy link
Copy Markdown
Contributor

@ErnieAllen This change looks good. As part of this PR, can you also please add a test to system_tests_qdstat to verify that the correct link priority is showing up? Thanks.

@ErnieAllen ErnieAllen closed this Oct 11, 2018
@ErnieAllen ErnieAllen deleted the ernie-DISPATCH-1130 branch October 11, 2018 11:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants