(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

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp

Reply via email to