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

DHCP add ARP Table Static Entry #173

Open
rtaghawi opened this issue Feb 22, 2025 · 3 comments
Open

DHCP add ARP Table Static Entry #173

rtaghawi opened this issue Feb 22, 2025 · 3 comments

Comments

@rtaghawi
Copy link

rtaghawi commented Feb 22, 2025

first, thank you for this great job!!
i just want to ask if it is possible to add a possibility to set an static ARP entry like it is available on the GUI for dhcp static maping

ARP Table Static Entry. Create an ARP Table Static Entry for this MAC & IP Address pair.

In the GUI there is a checkbox which can be set during creating a static dhcp lease. This help to have faster response if a dhcp static lease is created.

one additional point, in the documentation on the main page you describe to set the Python_interpreter static in the playbook if higher then python 3.8. here you could improve if you also would write that you mean the python running on Pfsense and not on the local system wich ansible is running, i dont have a lot of expiriance in using ansible and this was for me not so clear till i found it out with try and error.

but again, greate job and thank you to maintaining and running this project. this hepls alot!

@opoplawski
Copy link
Contributor

Thanks for the kind words. I think the static arp entries are already possible with the arp_table_static_entry parameter to pfsense_dhcp_static. As for setting the main boolean option - that seems to be present in the proposed module in #140. Does that seem right to you?

I've reworded the README to note that the python version is on the pfSense box, does that seem better?

@rtaghawi
Copy link
Author

rtaghawi commented Mar 2, 2025

Thanks for the response, I'm not a developer so i dont understand what in Module #140 will be changed, but it sounds if this could be what i needed.
Regarding rewording "Current versions of ansible should automatically detect the version of Python the pfSense system." sounds a little bit stargne, maybe better where: "Current versions of ansible should automatically detect the Python version of the pfSense system." :-)

@opoplawski
Copy link
Contributor

Thanks for the readme comment, hopefully better now. Can you try out the current master branch to see if that does what you need?

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

No branches or pull requests

2 participants