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

Customize Chilli DHCP Leases #6634

Closed
edospadoni opened this issue Jan 31, 2022 · 5 comments
Closed

Customize Chilli DHCP Leases #6634

edospadoni opened this issue Jan 31, 2022 · 5 comments
Labels
verified All test cases were verified successfully

Comments

@edospadoni
Copy link
Member

In very large environments chilli daemon DHCP refuse to dish out more than 512 leases, we can see in the logs this message:

reached max connections 512!

The DHCP of chilli daemon by default manages until 512 IP addresses, but luckily this value can be customized via the variable maxclients.

Proposed solution

Handle the new varibile through a new config prop called Maxclients that overrides the chilli conf.

See also
nethesis/icaro#158

@edospadoni
Copy link
Member Author

edospadoni commented Jan 31, 2022

Testing

RPMs to install:

Test 1:

  • install on clean machine.
  • verify dedalo runs correctly and check the maxclients attribute on /opt/icaro/dedalo/chilli.conf file is correct.

Test 2:

  • update in a machine with dedalo already installed.
  • verify dedalo runs correctly and check the maxclients attribute on /opt/icaro/dedalo/chilli.conf file is correct.

Test 3:

  • install on clean machine or update a machine with dedalo already installed.
  • change Maxclients prop (config setprop dedalo Maxclients 1024)
  • run the nethserver-dedalo-update event (signal-event nethserver-dedalo-update)
  • verify dedalo runs correctly and check the maxclients attribute on /opt/icaro/dedalo/chilli.conf file is correct.

@edospadoni edospadoni added the testing Packages are available from testing repositories label Jan 31, 2022
@edospadoni edospadoni assigned cotosso and unassigned edospadoni Jan 31, 2022
@Amygos
Copy link
Member

Amygos commented Feb 2, 2022

in 7.9.2009/testing:

@cotosso
Copy link

cotosso commented Feb 2, 2022

Test 1 :OK
dedalo runs flawlessy, maxclients attribute is correct both in db and in the configuration file /opt/icaro/dedalo/chilli.conf

Test 2 :OK
dedalo runs flawlessy

Test 3 :OK
chilli.conf is changed accordingly to the prop in the database

@cotosso cotosso added verified All test cases were verified successfully and removed testing Packages are available from testing repositories labels Feb 3, 2022
@Amygos
Copy link
Member

Amygos commented Feb 3, 2022

in 7.9.2009/updates:

@nethbot
Copy link
Member

nethbot commented Feb 3, 2022

in 7.9.2009/updates:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
verified All test cases were verified successfully
Projects
None yet
Development

No branches or pull requests

5 participants