yahoo-eng-team team mailing list archive

[Akihiro Motoki <1294166@xxxxxxxxxxxxxxxxxx>]

(Woops,... comment posting failed)

Thanks Ken'ichi,

It is not a tempest bug but a neutron bug (specific plugin bug)
In XML specification, if a tag name contains a colon, a part before a colon is interpreted as namespace.
In this case it is just an attribute name and it is not intended to define a namespace.
This field is specific to NEC plugin and it seems better to rename the attribute name without a colon.
Ideally XML serializer in Neutron should detects (or escapes) these errors, but at now changing name looks a good solution.


** Changed in: tempest
       Status: New => Invalid

** Also affects: neutron
   Importance: Undecided
       Status: New

** Changed in: neutron
   Importance: Undecided => Low

** Summary changed:

- XML test_list_ports_binding_ext_attr does not consider plugin-specific attributes
+ nec plugin fails with XML test_list_ports_binding_ext_attr tempest tests

** Changed in: neutron
     Assignee: (unassigned) => Akihiro Motoki (amotoki)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1294166

Title:
  nec plugin fails with XML test_list_ports_binding_ext_attr tempest
  tests

Status in OpenStack Neutron (virtual network service):
  New
Status in Tempest:
  Invalid

Bug description:
  commit 1ec6e18007691c92fc27235c677d11b0fe1c1f6b in tempest adds a
  validation for neutron port binding extension, but the check is too
  strict. binding:profile attribute in port binding extension is
  designed to allow plugin specific field but the check assumes
  binding:profile attribute implemented in ML2 plugin. This leads to
  test failures with neutron plugins which has different keys in
  binding:profile like NEC plugin or Mellanox plugin (though mellanox
  plugin does not test the related test now).

  The failed test is test_list_ports_binding_ext_attr. It only affects
  XML.

    tempest.api.network.test_ports.PortsAdminExtendedAttrsIpV6TestXML test_list_ports_binding_ext_attr
    tempest.api.network.test_ports.PortsAdminExtendedAttrsTestXML test_list_ports_binding_ext_attr

  Tempest test should allow plugin-specifc binding:profile attribute. If
  not, it should provide an options to disable these tests.

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1294166/+subscriptions