Intersection of baremetal network IP range and Ironic IP range (Neutron L3 settings) is not validated
Bug #1569790 reported by
Julia Aranovich
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Fuel for OpenStack |
Confirmed
|
Medium
|
Fuel UI Team | ||
Newton |
Confirmed
|
Medium
|
Fuel UI Team |
Bug Description
Reproduced on 10.0 master.
Steps to reproduce in Fuel UI:
1. create environment (default wizard settings)
2. got to Settings tab and enable Ironic service
3. go to Networks tab and check Use whole CIDR for Baremetal network in default node network group
4. click SaveSettings button
Expected result: the changes can not be saved because of an intersection of baremetal IP range with Ironic IP range from Neutron L3 parameters. Nailgun should return an appropriate error.
Actual result: there is no error from Nailgun, changes are saved successfully.
Fuel UI should also validate this case as well as it is done for Public network IP range and Floating IP range intersection.
Changed in fuel: | |
importance: | Undecided → Medium |
milestone: | none → 10.0 |
Changed in fuel: | |
assignee: | Fuel Python (Deprecated) (fuel-python) → Krzysztof Szukiełojć (kszukielojc) |
Changed in fuel: | |
assignee: | Krzysztof Szukiełojć (kszukielojc) → Fuel UI Team (fuel-ui) |
Changed in fuel: | |
assignee: | Fuel UI Team (fuel-ui) → Alexander Ignatyev (aignatyev) |
status: | Confirmed → In Progress |
Changed in fuel: | |
assignee: | Alexander Ignatyev (aignatyev) → nobody |
Changed in fuel: | |
assignee: | nobody → Fuel UI Team (fuel-ui) |
status: | In Progress → Confirmed |
Changed in fuel: | |
milestone: | 10.0 → 10.1 |
Changed in fuel: | |
milestone: | 10.1 → 11.0 |
To post a comment you must log in.
Need to add such check into Nailgun (NetworkCheck class). Now we have partial validation here: NetworkCheck. neutron_ check_l3_ addresses_ not_match_ subnet_ and_broadcast but it does not cover this case.
There should also be a check for intersection with gateways (one is set in Neutron L3 and another is added in case of using multiple node groups).