Key 'meta' is absent in bond properties

[Sergey Novikov]

Detailed bug description:
The key 'meta' is absent in bond properties http://paste.openstack.org/show/593147/ .
and it is reason why we cannot check that dpdk or sriov is available for this bond. I did this suggestion due to analogy with iface properties http://paste.openstack.org/show/593150/ . May be, I'm wrong, and there is another way to define availability of dpdk and sriov for corresponding bond. Please, clarify me this moment.

Steps to reproduce:
 1. Setup fuel master node
 2. Bootstrap some node
 3. Assign role "compute" to this node
 4. Choose 2 ifaces for which dpdk is available
 5. Bond them
 6. Get bond properties using REST API call `GET /api/nodes/<node_id>/interfaces/`

Description of the environment:
snapshot #662

[Andriy Popovych (popovych-andrey)]

dpdk, sriov, offloadin_modes for bond programmable attributes. Bond is logical structure which creates on client and then stores in nailgun. All that attributes can be calculated based on NICs meta (bond oflloading_modes are intersection of NIC offloading_modes, dpdk availability relay on NICs dpdk availability in meta etc.). So in any case we should calculate it in client at least first time.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/415448

[Aleksey Kasatkin (alekseyk-ru)]

Andriy, I support your view mostly but AFAIC backend should check offloading_modes as at least CLI user can provide any input actually (and tests use own client). Backend verifies configuration as much as possible, it is natural as it cannot trust clients in most cases (clients do not have whole picture, backend has most part of business logic).

[Aleksey Kasatkin (alekseyk-ru)]

It can be helpful that backend provides meta on GET calls (for the same reason - backend has the most complete picture in it; also, it's easier to maintain meta calculation in one place than in several places).

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/417914

[Aleksey Kasatkin (alekseyk-ru)]

Seems, this does not fix https://bugs.launchpad.net/fuel/+bug/1652076 so it should be treated separately.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/417909
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=98fe7f6de18e272521d2bdc4c2a4207bd877efb8
Submitter: Jenkins
Branch: master

commit 98fe7f6de18e272521d2bdc4c2a4207bd877efb8
Author: Fedor Zhadaev <email address hidden>
Date: Mon Jan 9 19:00:39 2017 +0400

    Add 'meta' to bond properties

    Dynamically generate 'meta' for bond and put there information
    about DPDK availability and structure of available offloading modes.

    Change-Id: Id030dda19b150b56db5624bd45c8f767d0748a47
    Closes-bug: #1652068
    (cherry picked from commit 13be8e85c1d5db7f45893b141d1c8e6ef6eeedab)

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/417914
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=9568c1c155cf8f324228aaba9dda397519bbb39f
Submitter: Jenkins
Branch: stable/newton

commit 9568c1c155cf8f324228aaba9dda397519bbb39f
Author: Fedor Zhadaev <email address hidden>
Date: Mon Jan 9 19:00:39 2017 +0400

    Add 'meta' to bond properties

    Dynamically generate 'meta' for bond and put there information
    about DPDK availability and structure of available offloading modes.

    Change-Id: Id030dda19b150b56db5624bd45c8f767d0748a47
    Closes-bug: #1652068
    (cherry picked from commit 13be8e85c1d5db7f45893b141d1c8e6ef6eeedab)

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/415448
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=b11548c91bfedfd2c9fe652c7b698887d687f976
Submitter: Jenkins
Branch: stable/mitaka

commit b11548c91bfedfd2c9fe652c7b698887d687f976
Author: Fedor Zhadaev <email address hidden>
Date: Wed Dec 28 15:05:19 2016 +0400

    Add 'meta' to bond properties

    Dynamically generate 'meta' for bond and put there information
    about DPDK availability and structure of available offloading modes.

    Change-Id: Id030dda19b150b56db5624bd45c8f767d0748a47
    Closes-bug: #1652068

[Dmitry Belyaninov (dbelyaninov)]

Updated 9.X SWARM test passed since #180 run
https://product-ci.infra.mirantis.net/job/9.x.system_test.ubuntu.bonding_ha_one_controller/

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/fuel-web 11.0.0.0rc1 release candidate.