Getting cl_prefix_check included in Ansible

Archives Ansible Developer
1

Hey Ansible Dev Team. Awhile ago Stanley and I at Cumulus created a bunch of modules. I think the first one that has no dependencies on Cumulus Linux (as in it will work on any Linux distro that has the ip route command) is finished. Right now its located at https://github.com/CumulusNetworks/cumulus-linux-ansible-modules/blob/master/library/cl_prefix_check

I suggest (based on feedback from others) that we rename it prefix_check. I don’t care what the name is as long as it gets pushed into main. This module allows me to check routes easily (if they are present or absent) and adds the ability of a timeout to give their route table to remove or add the route based on the user’s desired outcome. This was based on our distro (Cumulus Linux) running Quagga but this module is not dependent on our distro, or quagga. A user could make changes with static routes or any route engine (Quagga, Bird, etc). This gives Ansible the ability to check route changes on a live change and now not only can Ansible provision the network, it can now move traffic around like a controller.

My question is what do I have to do to get to the next phase? How do I push this into main the correct way. Reading http://docs.ansible.com/developing_modules.html#getting-your-module-into-ansible suggested I email this list :slight_smile:

Thanks!

-Sean Cavanaugh
Cumulus Networks

2

Hi Sean,

We no longer include switch management modules in core ansible (because it’s something the vendors can maintain better than us, and it sometimes gets weird), but we would encourage you to submit this to Galaxy.

In doing so, I’d recommend registering an Ansible Galaxy account called “CumulusLinux” or whatever, and then we’re going to have a “showcase” page soon that highlights some interesting vendor and non-vendor modules that will make these super easy to find.

(Note that we’ve also removed modules of some other hardware vendors in the routing/switching category that used to be in core - we’ll continue to include load balancer stuff like F5, A10, etc, as those are a bit different of a use case).

Still very glad to see this!