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

Dynamic IP handling #2

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Dynamic IP handling #2

wants to merge 3 commits into from

Conversation

Fraddles
Copy link

Hi There,
I knew I should have figured out how to make a PR sooner :)
I made some changes to the script to handle devices that have been given an Alias, but not a fixed IP, ie they are on DHCP. A few devices do not register nice hostnames with DHCP, this allows those names to be overridden with an alias.

  • The use_fixedip field appears to always be present for a 'configured' client, the fixed_ip field is not always present.
  • If the configured client has a fixed IP that will be used, otherwise the IP will be retrieved from the DHCP leases file.
    There is probably better ways to query the DHCP leases, but this seems to work fine on my small network.
    Disclaimer: The sum total of my Python experience is as long as it took to figure out how to make these changes :)
    Cheers.

Query DHCP leases for clients that only have an alias not an actual fixed IP.
Change from 'fixed_ip' to 'use_fixedip' field in client record to handle configured clients without a saved fixed IP.
Added handling for configured clients that have no current fixed or DHCP IP, ie offline in the case of DHCP clients.
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

Successfully merging this pull request may close these issues.

1 participant