frr icon indicating copy to clipboard operation
frr copied to clipboard

zebra: fix help handler for 'mpls label global-block'

Open pguibert6WIND opened this issue 1 year ago • 7 comments

The help handler refers to segment routing. Fix it.

Fixes: 1b6d5c7e0873 ("bgpd, zebra: Implement BGP Prefix-SID IETF draft")

pguibert6WIND avatar Oct 10 '23 12:10 pguibert6WIND

Is this really for segment routing?

BGP segment routing.

pguibert6WIND avatar Oct 13 '23 12:10 pguibert6WIND

unrelated failures ci:rerun

riw777 avatar Oct 24 '23 10:10 riw777

Is this really for segment routing?

BGP segment routing.

Don't you use this command to create a label pool for IS-IS, though?

riw777 avatar Oct 31 '23 14:10 riw777

Is this really for segment routing?

BGP segment routing.

Don't you use this command to create a label pool for IS-IS, though?

actually, no. This command is specific to BGP segment routing MPLS.

pguibert6WIND avatar Nov 02 '23 07:11 pguibert6WIND

As I read the code, the global label block is a generic bit of code that can be used by anyone using FEC's. Which is more than just BGP right? What is the problem that you are trying to solve? How would this help people attempting to use this command for say ldpd?

I just wanted to clarify what that label block was for.

which is more than just BGP right? You are right. I would say, actually, more than just BGP prefix SID cases. (as with LDP, I don't see the use case).

How would this help people attempting to use this command for say ldpd? I need to think of a rework of this pull request to :

  • either add a description that covers all FEC use cases
  • or limit the usage of the global block to FEC with prefix-SID.

pguibert6WIND avatar Nov 02 '23 07:11 pguibert6WIND

@pguibert6WIND still working on this?

riw777 avatar Jan 16 '24 13:01 riw777

@frrbot autoclose in 1 month

riw777 avatar Feb 13 '24 14:02 riw777