-
Notifications
You must be signed in to change notification settings - Fork 44
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
Ansible not updating MX L3 Firewall, but states success #61
Comments
@pokepoke81 - I created a pull request (#62) to fix this problem. Please test. If you need assistance on how to test, let me know and I can help you through. |
I’m new to this so I could use help in learning how to test. Thanks. |
@pokepoke81 Easiest way is to download the file at https://raw.githubusercontent.com/kbreit/ansible-meraki/bugfix/61_mx_firewall/plugins/modules/meraki_mx_l3_firewall.py and copy that into your ansible collections directory. Most likely it's at |
@kbreit took a minute, but I was able to test and it's working now. Again, I'm new at this, so please forgive the beginner questions. How does this bug fix make it into the Galaxy repo so it will show up on my configuration? How long does that take? |
@pokepoke81 Thanks for testing. The first thing that needs to happen is GitHub needs to email me so I know you responded. It didn't so I'm glad I checked back. Note: I meant this sarcastically. Your response was never emailed to me for some reason. How long should it take? Overall, it shouldn't take very long. It's up to me to merge the bugfix (minutes) and post a new version in Galaxy (minutes). I should be able to dedicate some time to it tomorrow. You should be able to follow the collection in Galaxy and be notified every time I do a release. |
@kbreit thank you. I appreciate all your help with this. |
A simple playbook to update firewall rules on an MX device fails to to change, but Ansible reports success and no change. No change occurs on the Meraki Dashboard either. This playbook was used to generate the "Ansible Block traffic to server2" rule, so it did work at one point.
Playbook:
Output:
The text was updated successfully, but these errors were encountered: