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

Module vyos_firewall_rules doesn't delete rules on replaced state #247

Closed
mlk-89 opened this issue Mar 18, 2022 · 2 comments
Closed

Module vyos_firewall_rules doesn't delete rules on replaced state #247

mlk-89 opened this issue Mar 18, 2022 · 2 comments

Comments

@mlk-89
Copy link
Contributor

mlk-89 commented Mar 18, 2022

SUMMARY

I think the module vyos_firewall_rules should delete rules when using replaced.

This bug only affect source and destination.

ISSUE TYPE
  • Bug Report
COMPONENT NAME

vyos_firewall_rules (replaced state)

ANSIBLE VERSION

ansible [core 2.12.1]
configured module search path = ['/home//.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
ansible python module location = /home//.local/lib/python3.8/site-packages/ansible
ansible collection location = /home/maxime/.ansible/collections:/usr/share/ansible/collections
executable location = /home/maxime/.local/bin/ansible
python version = 3.8.10 (default, Nov 26 2021, 20:14:08) [GCC 9.3.0]
jinja version = 2.10.1
libyaml = True


##### COLLECTION VERSION
<!--- Paste verbatim output from "ansible-galaxy collection list <namespace>.<collection>"  between the quotes
for example: ansible-galaxy collection list community.general
-->

Collection Version


vyos.vyos 2.8.0


##### STEPS TO REPRODUCE
<!--- Describe exactly how to reproduce the problem, using a minimal test-case -->

<!--- Paste example playbooks or commands between quotes below -->
```yaml
  vyos.vyos.vyos_firewall_rules:
    config:
  - afi: ipv4
    rule_sets:
      - name: test4
        description: IPv4 INBOUND rule set
        default_action: accept
        rules:
          - number: 101
            action: reject
            source:
              group:
                network_group: "{{ public_group_ipv4['public']['name'] }}"
  state: replaced

EXPECTED RESULTS
vyos@nf-pont-client1a# show firewall name test4
 default-action accept
 description "IPv4 INBOUND rule set"
 rule 101 {
     action reject
     source {
         group {
             network-group public_group_ipv4
         }
     }
 }

ACTUAL RESULTS
vyos@nf-pont-client1a# show firewall name test4
 default-action accept
 description "IPv4 INBOUND rule set"
 rule 101 {
     action reject
     destination {
         group {
             network-group public_group_ipv4
         }
     }
     source {
         group {
             network-group public_group_ipv4
         }
     }
 }

@mlk-89
Copy link
Contributor Author

mlk-89 commented Mar 18, 2022

Hi, I've created a PR for this issue #248

@andamasov
Copy link
Member

moved to vyos.dev
https://vyos.dev/T6825

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants