Extend PXE & static pool network size

Bug #1283831 reported by Mike Scherbakov on 2014-02-23
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
High
Fuel Library (Deprecated)

Bug Description

> We could short circuit this bug another way by defaulting to a /23 network for PXE? 10.20.0.0/23 instead of 10.20.0.0/24 It would be a couple lines changed in Fuel Menu and in our ISO build script
as from https://bugs.launchpad.net/fuel/+bug/1271571.

Can we do it in 4.1 still?

Mike Scherbakov (mihgen) on 2014-02-23
Changed in fuel:
importance: Undecided → High
status: New → Confirmed
assignee: Matthew Mosesohn (raytrac3r) → Fuel Library Team (fuel-library)
Andrew Woodward (xarses) wrote :

Why? We removed the interfaces * nodes. We now upto 126 addresses currently with the default. This issue above is still going to be seen in the current model until the dnsmasq and fuel use the same range as long as users allocate smaller ranges as in the parent ticket.

I've code ready and waiting for code freeze to lift before working on it again. It needs more work on the unit tests, but functionally works for me.

Mike Scherbakov (mihgen) wrote :

This was requested by Roman - just to give users more IP addresses, so they are even less likely to ever meet this issue, in spite of our big step and reducing IPs to 1-per-node. I personally vote for keeping it as is for now, with 126 for default, and get your code after code freeze.

Is there a reason to keep it to 126 by default? Seems quite low. Let's discuss more during the Monday sync up.

Matthew Mosesohn (raytrac3r) wrote :

This would be accomplished by changing the default netmask for deployment to 255.255.254.0 in fuel-main default kernel parameters for installation. It's not even really a code change.

However, this may break some of our system tests that depend on /24 networks.

Andrew Woodward (xarses) wrote :

I've spoken with Roman, and we will wait for after the code freeze to merge the fix in 1274595. In the meantime we added release-notes to 1271571 to note that by default 126 addresses are available or half of what ever you specify.

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers