Extend nailgun check_network to verify that we have correct numbers of ips in range

Bug #1354803 reported by Gil Meir
52
This bug affects 8 people
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Won't Fix
High
Nikita Zubkov
Newton
Won't Fix
High
Nikita Zubkov

Bug Description

When installing CentOS HA I defined a pool of IP addresses for the public network, amount equal to the number of nodes. Since there should be another address in the pool (note: not-intuitive), when pressing "Deploy changes" I got an error "not enough IP addresses in the pool".
The bug is that at this step all the configuration fields became disabled, and there is no way to edit / add IP addresses.
Validation should be done when pressing "Save settings", not "Deploy changes".

* This might be similar to this bug in the concept, but still a different bug: https://bugs.launchpad.net/fuel/+bug/1353391

{"build_id": "2014-08-06_11-23-14", "ostf_sha": "be71965998364bf8e6415bd38b75c84b63aab867", "build_number": "406", "auth_required": true, "api": "1.0", "nailgun_sha": "f64b06c788e2b92fcb8e678ea6d0c9b86e8d0ab7", "production": "docker", "fuelmain_sha": "124ea87f1ac1c06e27613fe3b31fd5fc6b39e82d", "astute_sha": "99a790ad1b7526cbbd5bf8add0cb2b4e503fccd4", "feature_groups": ["mirantis"], "release": "5.1", "fuellib_sha": "513ec5cdcdef74c7419d5bae967b9edc7da8dbd7"}

Revision history for this message
Gil Meir (gilmeir-d) wrote :
Gil Meir (gilmeir-d)
summary: - No validation on networks tab before deployment
+ UI fields are disabled after pressing "deploy changes" with incorrect
+ values
Revision history for this message
Dima Shulyak (dshulyak) wrote : Re: UI fields are disabled after pressing "deploy changes" with incorrect values

I think we have blueprint somewhere to allocate ip addresses before deployment.
Maybe we can squeeze atleast some basic validation in 5.1 ?

Changed in fuel:
milestone: none → 5.1
assignee: nobody → Fuel Python Team (fuel-python)
importance: Undecided → High
status: New → Confirmed
Dima Shulyak (dshulyak)
summary: - UI fields are disabled after pressing "deploy changes" with incorrect
- values
+ Extend nailgun check_network to verify that we have correct numbers of
+ ips in range
Revision history for this message
Dima Shulyak (dshulyak) wrote :

Looks like we dont have time to add such validation in 5.1, so i will set medium priority and postpone to 6.0

Changed in fuel:
importance: High → Medium
milestone: 5.1 → 6.0
tags: added: release-notes
Changed in fuel:
milestone: 6.0 → 6.1
Dmitry Pyzhov (dpyzhov)
tags: added: feature
Dmitry Pyzhov (dpyzhov)
tags: added: feature-validation
Nikolay Markov (nmarkov)
Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Nikolay Markov (nmarkov)
Revision history for this message
Aleksey Kasatkin (alekseyk-ru) wrote :

Actually, we need to add such checks for all networks. And it should encounter VIPs for Public and Management. It should take into account node groups also.

Dmitry Pyzhov (dpyzhov)
Changed in fuel:
assignee: Nikolay Markov (nmarkov) → Fuel Python Team (fuel-python)
milestone: 6.1 → 7.0
tags: added: qa-agree-7.0
Revision history for this message
Aleksey Kasatkin (alekseyk-ru) wrote :

Validation issue. Does not break deployment. Tech debt. Moved to 8.0.

Changed in fuel:
status: Confirmed → Won't Fix
tags: added: feature-advanced-networking
tags: added: known-issue
Dmitry Pyzhov (dpyzhov)
Changed in fuel:
milestone: 7.0 → 8.0
no longer affects: fuel/8.0.x
Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Ivan Kliuk (ivankliuk)
Dmitry Pyzhov (dpyzhov)
tags: added: area-python
Changed in fuel:
assignee: Ivan Kliuk (ivankliuk) → Fuel Python Team (fuel-python)
Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Bulat Gaifullin (bgaifullin)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (master)

Fix proposed to branch: master
Review: https://review.openstack.org/245941

Changed in fuel:
status: Triaged → In Progress
Changed in fuel:
assignee: Bulat Gaifullin (bgaifullin) → Ivan Kliuk (ivankliuk)
Changed in fuel:
assignee: Ivan Kliuk (ivankliuk) → Bulat Gaifullin (bgaifullin)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/245941
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=4e2d655555e56b730ddd98504d371f85eaa5a0b2
Submitter: Jenkins
Branch: master

commit 4e2d655555e56b730ddd98504d371f85eaa5a0b2
Author: Bulat Gaifullin <email address hidden>
Date: Mon Nov 16 22:38:53 2015 +0300

    prepare_for_deployment was moved to network manager

    The network related function shall be in NetworkManager,
    this makes easier further refactoring.

    Change-Id: I8eb5404509c9fe0a8c367ea119bfb18233941fde
    Partial-Bug: #1354803

Changed in fuel:
milestone: 8.0 → 9.0
Changed in fuel:
assignee: Bulat Gaifullin (bgaifullin) → Fuel Python Team (fuel-python)
Revision history for this message
Nikita Zubkov (zubchick) wrote :

Looks like it's not a high bug, after clicking deployment we still have an error, but network tab is not locked.

Revision history for this message
Nikita Zubkov (zubchick) wrote :

As I said now it's not an issue and you can easily change network settings after getting this error. So won't fix.

Nikita Zubkov (zubchick)
Changed in fuel:
status: In Progress → Won't Fix
tags: removed: release-notes
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.