Running network verification on running env may break network

Bug #1399004 reported by Łukasz Oleś
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Maciej Kwiek

Bug Description

During network verification process 8021q module is loaded. When cluster is already installed this module may break OVS.

It should be loaded only on bootstrap image when VLANs are not configured.

https://github.com/stackforge/fuel-web/blob/master/network_checker/network_checker/net_check/api.py#L84

Revision history for this message
Dima Shulyak (dshulyak) wrote :

It makes more sense to use for network verification on ovs-based environments - ovs-vlan test

https://github.com/openvswitch/ovs/blob/master/python/ovstest/tests.py

We may need to modify this code for better automation though

Dmitry Pyzhov (dpyzhov)
tags: added: module-netcheck
Revision history for this message
Dmitry Pyzhov (dpyzhov) wrote :

We are lack of resources in 6.1

Changed in fuel:
milestone: 6.1 → 7.0
Revision history for this message
Nastya Urlapova (aurlapova) wrote :

Let's describe such behaviour in documents.

tags: added: qa-agree-7.0
tags: added: release-notes
Andrew Woodward (xarses)
Changed in fuel:
importance: Medium → High
Revision history for this message
Mike Scherbakov (mihgen) wrote :

Can we have more data on how it affects end users? If it does affect users, can we have warning / ask for confirmation from a user that he want to proceed as a workaround, if we don't have time/resources to fix this in 7.0?

Revision history for this message
Ihor Kalnytskyi (ikalnytskyi) wrote :

Well, it looks like the first thing we have to do is:

* to check that `8021q` is loaded on bootstrap
* to check that `8021q` is loaded right after provisioning (node status is provisioned)
* to check that `8021q` is loaded after deployment

If so, we should just remove loading of `8021q` from the network checker.

Changed in fuel:
status: Confirmed → Triaged
assignee: Fuel Python Team (fuel-python) → Maciej Kwiek (maciej-iai)
Revision history for this message
Maciej Kwiek (maciej-iai) wrote :

`modinfo 8021q` on discovered, provisioned and deployed node says the module is loaded, so I will remove loading the module from network checker.

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/211525

Changed in fuel:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/211525
Committed: https://git.openstack.org/cgit/stackforge/fuel-web/commit/?id=e91647a9cff8c0271d8cbb1181f7dd0dc5f2e411
Submitter: Jenkins
Branch: master

commit e91647a9cff8c0271d8cbb1181f7dd0dc5f2e411
Author: Maciej Kwiek <email address hidden>
Date: Tue Aug 11 14:05:23 2015 +0200

    Loading 8021q module is removed from netcheck

    Loading this module on working environment broke OVS. This module is
    loaded already on bootstrapped, provision and deployed nodes anyway, so
    there is no need for loading it again.

    Change-Id: Icfbae58b10fc1e58e183d5abac05061ae97ea282
    Closes-Bug: #1399004

Changed in fuel:
status: In Progress → Fix Committed
tags: added: on verificationscale
removed: scale
Revision history for this message
Andrey Sledzinskiy (asledzinskiy) wrote :

{u'build_id': u'270', u'build_number': u'270', u'auth_required': True, u'fuel-ostf_sha': u'582a81ccaa1e439a3aec4b8b8f6994735de840f4', u'fuel-library_sha': u'f3780484874f5f4a1831714710ff552f33522915', u'nailgun_sha': u'9a1207d02d943953e130a0a31b9f7aa855cd44db', u'openstack_version': u'2015.1.0-7.0', u'fuel-nailgun-agent_sha': u'd7027952870a35db8dc52f185bb1158cdd3d1ebd', u'fuel-agent_sha': u'082a47bf014002e515001be05f99040437281a2d', u'api': u'1.0', u'python-fuelclient_sha': u'9643fa07f1290071511066804f962f62fe27b512', u'astute_sha': u'ad6d59812b775bc12e7bd7aec8f81374595ffa63', u'fuelmain_sha': u'9ab01caf960013dc882825dc9b0e11ccf0b81cb0', u'feature_groups': [u'mirantis'], u'release': u'7.0', u'release_versions': {u'2015.1.0-7.0': {u'VERSION': {u'build_id': u'270', u'build_number': u'270', u'fuel-library_sha': u'f3780484874f5f4a1831714710ff552f33522915', u'nailgun_sha': u'9a1207d02d943953e130a0a31b9f7aa855cd44db', u'fuel-ostf_sha': u'582a81ccaa1e439a3aec4b8b8f6994735de840f4', u'fuel-nailgun-agent_sha': u'd7027952870a35db8dc52f185bb1158cdd3d1ebd', u'fuel-agent_sha': u'082a47bf014002e515001be05f99040437281a2d', u'api': u'1.0', u'python-fuelclient_sha': u'9643fa07f1290071511066804f962f62fe27b512', u'astute_sha': u'ad6d59812b775bc12e7bd7aec8f81374595ffa63', u'fuelmain_sha': u'9ab01caf960013dc882825dc9b0e11ccf0b81cb0', u'feature_groups': [u'mirantis'], u'release': u'7.0', u'openstack_version': u'2015.1.0-7.0', u'production': u'docker'}}}, u'production': u'docker'}

Changed in fuel:
status: Fix Committed → Fix Released
tags: added: scale
removed: verificationscale
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.