Add feature for bonding/vlan interface in the firewall_interfaces #246
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add vlan interface in the firewall_interfaces
Add bonding interface in the firewall_interfaces
SUMMARY
When using name that contains a dot in the module vyos_firewall_interfaces , we consider this interface as a vlan interface, so this PR include modification on the facts gathering and config file.
Moreover, set the interface type as well when the name begins with bond.
ISSUE TYPE
COMPONENT NAME
plugins/module_utils/network/vyos/config/firewall_interfaces/firewall_interfaces.py
plugins/module_utils/network/vyos/facts/firewall_interfaces/firewall_interfaces.py
ADDITIONAL INFORMATION
Fix this issue: #219
usage:
vyos.vyos.vyos_firewall_interfaces:
config:
- access_rules:
- afi: ipv4
rules:
- name: test1
direction: in
- name: test2
direction: local
name: eth1.10
- access_rules:
- afi: ipv4
rules:
- name: test1
direction: in
- name: test2
direction: local
name: bond0
- access_rules:
- afi: ipv4
rules:
- name: test1
direction: in
- name: test2
direction: local
name: bond0.113