Skip to content

Route: Allow specifying a routing metric value#2051

Open
tim-seoss wants to merge 1 commit intoClusterLabs:mainfrom
tim-seoss:heartbeat-route-add-metric
Open

Route: Allow specifying a routing metric value#2051
tim-seoss wants to merge 1 commit intoClusterLabs:mainfrom
tim-seoss:heartbeat-route-add-metric

Conversation

@tim-seoss
Copy link
Copy Markdown

The Route resource did not previously support setting a routing metric (i.e. a priority which the newly created route will carry). This is a common requirement, and is already supported in agents such as IPsrcaddr and IPaddr2.

@knet-jenkins
Copy link
Copy Markdown

knet-jenkins bot commented May 26, 2025

Can one of the admins check and authorise this run please: https://ci.kronosnet.org/job/resource-agents/job/resource-agents-pipeline/job/PR-2051/1/input

@oalbrigt
Copy link
Copy Markdown
Contributor

retest this please

@oalbrigt
Copy link
Copy Markdown
Contributor

LGTM. I'll leave this open for a week or 2 for others to review and test it.

@tim-seoss
Copy link
Copy Markdown
Author

tim-seoss commented May 27, 2025

Apologies, have just spotted a bug. I'll revise. (correction: I will force-push later today when I have access to my hardware key).

The Route resource did not previously support setting a routing metric
(i.e. a priority which the newly created route will carry). This is a
common requirement, and is already supported in agents such as IPsrcaddr
and IPaddr2.
@tim-seoss tim-seoss force-pushed the heartbeat-route-add-metric branch from 8003fde to 36eb27a Compare May 28, 2025 10:07
@knet-jenkins
Copy link
Copy Markdown

knet-jenkins bot commented May 28, 2025

Can one of the admins check and authorise this run please: https://ci.kronosnet.org/job/resource-agents/job/resource-agents-pipeline/job/PR-2051/3/input

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants