Skip to content

Conversation

@yu-re-ka
Copy link

No description provided.

@job
Copy link
Member

job commented Sep 30, 2024

Can you also update this one? https://github.com/bgp/bgpq4/blob/main/bgpq4.8

@job
Copy link
Member

job commented Sep 30, 2024

It seems there is a discrepancy between documentation and behaviour:

feather$ fgrep 'origin as-list' *
main.c: printf(" -H number : generate origin as-lists (JunOS only)\n");

feather$ bgpq4 -J -H 15562 AS-SNIJDERS
policy-options {
replace:
 as-list-group NN {
  as-list a0 members 15562;
  as-list a1 members [ 112 234 267 8952 12654 31451 39765 41996 ];
  as-list a2 members [ 43997 44854 48603 57782 60927 199036 202314 205956 ];
  as-list a3 members [ 210089 212121 215198 ];
 }
}

The keyword as-path-origin does not appear in the code:

feather$ fgrep -r as-path-origin *
CHANGES:    - Add support for the new Junos as-path-origins feature
README.md:> generate output as-list for JunOS 21.3R1+ `as-path-origin` filter (JunOS only)
bgpq4.8:.Em as-path-origin

@yu-re-ka
Copy link
Author

Hi,

the command works as intended. Assuming we generated our as-list for use with the as-path-origins statement with bgpq4 -J -H 9136 -l AS9136_ORIGINS AS-WOBCOM, the following policy, which includes as-path-origin AS9136_ORIGINS must be specified by the user:

policy-statement POLICY_AS9136 {
    term FILTER_LISTS {
        from {
            as-path-origins as-list-group AS9136_ORIGINS;
        }
        then next policy;
    }
    then reject;
}

The -f and -G options would not generate a policy-statement or term either. The documentation never states that an as-path-origins statement will be generated.

@yu-re-ka
Copy link
Author

yu-re-ka commented Dec 2, 2024

Have you had a chance to take review this with the additional context?

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.

3 participants