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

Add support for ttl-except and hop-limit-except to the Juniper platform #325

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

Conversation

nward
Copy link

@nward nward commented Sep 10, 2022

Add support for the ttl-except and hop-limit-except match conditions to the Juniper platform.

Have added tests, and all passing fine.

Have added docs.

Generates valid config.

I have created hop-limit-except as a single value rather than a range, as Juniper does not (to my knowledge) support a range for this option.

@google-cla
Copy link

google-cla bot commented Sep 10, 2022

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@nward nward changed the title Add support for ttl-except to the Juniper platform Add support for ttl-except and hop-limit-except to the Juniper platform Sep 10, 2022
@nward
Copy link
Author

nward commented Sep 10, 2022

I'm unable to sign the CLA as my org (SearchLight) because https://cla.developers.google.com/clas doesn't let me switch to another Google account I'm logged in as (I have.. 9 or so), so it does not recognise that I am the owner of the group.

I have attempted to switch accounts on https://developers.google.com and it does not take effect on the CLA URL above.

I have signed the CLA personally, and I am a Director of the org.

¯_(ツ)_/¯

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.

1 participant