Network Verification doesn't work with LACP bonds

Hi Gleb, any logs? Other observations? What Fuel version are you running?

There is no logs. I just click on Verify Network and it said that no one vlan works on interfaces. But then I deployed the cloud and everything works fine. I suggested that the reason is we have bonding.

The version is

{
    "build_id": "2014-09-17_21-40-34",
    "ostf_sha": "64cb59c681658a7a55cc2c09d079072a41beb346",
    "build_number": "11",
    "auth_required": true,
    "api": "1.0",
    "nailgun_sha": "eb8f2b358ea4bb7eb0b2a0075e7ad3d3a905db0d",
    "production": "docker",
    "fuelmain_sha": "8ef433e939425eabd1034c0b70e90bdf888b69fd",
    "astute_sha": "f5fbd89d1e0e1f22ef9ab2af26da5ffbfbf24b13",
    "feature_groups": [
        "mirantis"
    ],
    "release": "5.1",
    "release_versions": {
        "2014.1.1-5.1": {
            "VERSION": {
                "build_id": "2014-09-17_21-40-34",
                "ostf_sha": "64cb59c681658a7a55cc2c09d079072a41beb346",
                "build_number": "11",
                "api": "1.0",
                "nailgun_sha": "eb8f2b358ea4bb7eb0b2a0075e7ad3d3a905db0d",
                "production": "docker",
                "fuelmain_sha": "8ef433e939425eabd1034c0b70e90bdf888b69fd",
                "astute_sha": "f5fbd89d1e0e1f22ef9ab2af26da5ffbfbf24b13",
                "feature_groups": [
                    "mirantis"
                ],
                "release": "5.1",
                "fuellib_sha": "d9b16846e54f76c8ebe7764d2b5b8231d6b25079"
            }
        }
    },
    "fuellib_sha": "d9b16846e54f76c8ebe7764d2b5b8231d6b25079"
}

Gleb, sorry, it is current limitation of our checker. LACP check requires a lot of additional logic and has tricky cases. We will implement it eventually.

From my point of view FUEL UI should show some message that bonding can't be checked instead of bunch of errors.

Miroslav,
can you confirm that it's rather the case that verifier doesn't work at all with bonding regardless of its type and shows a bunch of errors instead?

Since we moved to TCP dump based network verification - it would be easier to add message, that bonding verification may not be performed or, depending on current LACP settings, may finish with false error over the working bonding scheme.

Anyway, bonded NICs verification is currently not reliable for all bond settings, we support.

As solution for 6.1 we can filter interfaces in LACP bonds and don't check them with some message (as Gleb said early) . For 7.0 in network verify service(https://blueprints.launchpad.net/fuel/+spec/network-verify-service) we can implement LACP bonds checking.

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

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

commit a0c1271781980c3b2ff8051217d06ae92b3c8e4c
Author: Andriy Popovych <email address hidden>
Date: Fri Apr 10 10:09:35 2015 +0300

    Exclude LACP bonds checking from network verification

    Currently network checker doesn't support
    checking interfaces in bonds and checks it as
    simple interfaces. But in case of LACP L2 bonding
    verification fails on hardware. For 6.1 checking
    interfaces in LACP bonds should be excluded

    DocImpact
    Closes-Bug:#1376908
    Change-Id: Ie43ff2cdf350ab019d474799905a2cc1bb970480

@Andrian, we set status - Fix Released after QA verify.

Verified at Fuel ISO#406
OS Ubuntu, Centos
api: '1.0'
astute_sha: e319b19158fc416d911edf0c06667e810c457b02
auth_required: true
build_id: 2015-05-12_08-34-41
build_number: '406'
feature_groups:
- mirantis
fuel-library_sha: 156fb11bbf3e12e7c73a9a3ac785c9d33d4ac343
fuel-ostf_sha: 51b41cba7572aefa4a98e40fdecdbc05efb2e1ea
fuelmain_sha: 51b86bb24b27742a22b23e2ae3dfc850c47e5fbf
nailgun_sha: 0d077692e813720410c52bc720a8462725446e0d
openstack_version: 2014.2.2-6.1
production: docker
python-fuelclient_sha: af6c9c3799b9ec107bcdc6dbf035cafc034526ce
release: '6.1'
release_versions:
  2014.2.2-6.1:
    VERSION:
      api: '1.0'
      astute_sha: e319b19158fc416d911edf0c06667e810c457b02
      build_id: 2015-05-12_08-34-41
      build_number: '406'
      feature_groups:
      - mirantis
      fuel-library_sha: 156fb11bbf3e12e7c73a9a3ac785c9d33d4ac343
      fuel-ostf_sha: 51b41cba7572aefa4a98e40fdecdbc05efb2e1ea
      fuelmain_sha: 51b86bb24b27742a22b23e2ae3dfc850c47e5fbf
      nailgun_sha: 0d077692e813720410c52bc720a8462725446e0d
      openstack_version: 2014.2.2-6.1
      production: docker
      python-fuelclient_sha: af6c9c3799b9ec107bcdc6dbf035cafc034526ce
      release: '6.1'