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

readme: -H generates input as-list not output as-list #116

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

yu-re-ka
Copy link
Contributor

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
Contributor 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
Contributor 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.

2 participants