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.

Sign up for GitHub

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

Jump to bottom

Bug: Logical Interface IPv6 output_filter not being set #117

Closed
arjenvri opened this issue Feb 10, 2021 · 0 comments · Fixed by #118
Closed

Bug: Logical Interface IPv6 output_filter not being set #117

arjenvri opened this issue Feb 10, 2021 · 0 comments · Fixed by #118
Assignees
@jeremmfr
Labels
bug Something isn't working
Milestone
v1.13.0

Comments

@arjenvri
Copy link
Contributor

Just to report a minor bug in the logical interface ipv6 output_filter code.
Currently the output_filter is never configured properly.

Seems that here
https://github.com/jeremmfr/terraform-provider-junos/blob/master/junos/resource_interface_logical.go#L781

There is a copy-paste error calling the filter_input variable instead of filter_output.
@jeremmfr jeremmfr added the bug Something isn't working label Feb 11, 2021
@jeremmfr jeremmfr self-assigned this Feb 11, 2021
jeremmfr added a commit that referenced this issue Feb 11, 2021
@jeremmfr
Fixes #117: filter_output not set with good argument for family inet6
271be7a 
in junos_interface_logical resource
@jeremmfr jeremmfr mentioned this issue Feb 11, 2021
Merged
jeremmfr added a commit that referenced this issue Feb 11, 2021
@jeremmfr
Merge pull request #118 from jeremmfr/issue-117
4155f95 
Fixes #117: filter_output not set with good argument for family inet6
@jeremmfr jeremmfr mentioned this issue Feb 11, 2021
Merged
@jeremmfr jeremmfr added this to the v1.13.0 milestone Dec 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jeremmfr jeremmfr

Labels
bug Something isn't working
Projects
None yet
Milestone
v1.13.0
2 participants
@jeremmfr @arjenvri