
<html>

<head>

<meta http-equiv="content-type" content="text/html; charset=UTF-8">

</head>

<body><p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">Hi,</p>

<br /><p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">Dnia poniedziałek, 26 kwietnia 2021 17:48:08 CEST Martin Kopec pisze:</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> Hi everyone,</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> </p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> I would like to further discuss the topics we covered with the neutron team</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> during</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> the PTG [1].</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> </p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> * adding address_group API capability</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> It's tested by tests in neutron-tempest-plugin. First question is if tests</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> which are</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> not directly in tempest can be a part of a non-add-on marketing program?</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> It's possible to move them to tempest though, by the time we do so, could</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> they be</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> marked as advisory?</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> </p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> * Shall we include QoS tempest tests since we don't know what share of</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> vendors</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> enable QoS? Could it be an add-on?</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> These tests are also in neutron-tempest-plugin, I assume we're talking about</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> neutron_tempest_plugin.api.test_qos tests.</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> If we want to include these tests, which program should they belong to? Do</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> we wanna</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> create a new one?</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> </p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> [1] https://etherpad.opendev.org/p/neutron-xena-ptg</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> </p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> Thanks,</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> --</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> Martin Kopec</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> Senior Software Quality Engineer</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> Red Hat EMEA</p>

<br /><p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">First of all, sorry that it took so long for me but I finally looked into Neutron related tests and capabilities and I think we can possibly add few things there:</p>

<br /><p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">- <span style="background-color:#ffffff;"><span style="font-family:-apple-system;"><span style="color:#202020;">For "networks-security-groups-CRUD" we can add "address_groups" API. It is now supported by ML2 plugin [1]. In the neutron-tempest-plugin we just have some scenario test [2] but we would probably need also API tests for that, correct?</span></span></span></p>

<br /><p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">- For networks-l3-CRUD we can optionally add port_forwarding API. This can be added by service plugin [3] so it may not be enabled in all deployments. But maybe there is some "optional feature" category in the RefStack, and if so, this could be included there. Tests for that are in neutron-tempest-plugin [4] and [5].</p>

<br /><p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">- There are also 2 other service plugins, which I think could be included as "optional feature" in the RefStack, but IMO don't fit exactly in any of the existing groups. Those are QoS [6] and Trunks [7]. Tests for both are in the neutron-tempest-plugin as well: Qos: [8] and [9], Trunk [10], [11] and [12].</p>

<br /><p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">Please let me know what do You think about it and if that would be ok and if You want me to propose some patches with that or maybe You will propose them.</p>

<br /><p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">[1] <a href="https://review.opendev.org/c/openstack/neutron-lib/+/741784">https://review.opendev.org/c/openstack/neutron-lib/+/741784</a></p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">[2] <a href="https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/777833">https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/777833</a></p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">[3] <a href="https://github.com/openstack/neutron/blob/master/neutron/services/portforwarding/pf_plugin.py">https://github.com/openstack/neutron/blob/master/neutron/services/portforwarding/pf_plugin.py</a></p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">[4] <a href="https://github.com/openstack/neutron-tempest-plugin/blob/master/neutron_tempest_plugin/api/test_port_forwardings.py">https://github.com/openstack/neutron-tempest-plugin/blob/master/neutron_tempest_plugin/api/test_port_forwardings.py</a></p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">[5] <a href="https://github.com/openstack/neutron-tempest-plugin/blob/master/neutron_tempest_plugin/api/test_port_forwarding_negative.py">https://github.com/openstack/neutron-tempest-plugin/blob/master/neutron_tempest_plugin/api/test_port_forwarding_negative.py</a></p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">[6] <a href="https://github.com/openstack/neutron/blob/master/neutron/services/qos/qos_plugin.py">https://github.com/openstack/neutron/blob/master/neutron/services/qos/qos_plugin.py</a></p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">[7] <a href="https://github.com/openstack/neutron/blob/master/neutron/services/trunk/plugin.py">https://github.com/openstack/neutron/blob/master/neutron/services/trunk/plugin.py</a></p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">[8] <a href="https://github.com/openstack/neutron-tempest-plugin/blob/master/neutron_tempest_plugin/api/test_qos.py">https://github.com/openstack/neutron-tempest-plugin/blob/master/neutron_tempest_plugin/api/test_qos.py</a></p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">[9] <a href="https://github.com/openstack/neutron-tempest-plugin/blob/master/neutron_tempest_plugin/api/test_qos_negative.py">https://github.com/openstack/neutron-tempest-plugin/blob/master/neutron_tempest_plugin/api/test_qos_negative.py</a></p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">[10] <a href="https://github.com/openstack/neutron-tempest-plugin/blob/master/neutron_tempest_plugin/api/test_trunk.py">https://github.com/openstack/neutron-tempest-plugin/blob/master/neutron_tempest_plugin/api/test_trunk.py</a></p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">[11] <a href="https://github.com/openstack/neutron-tempest-plugin/blob/master/neutron_tempest_plugin/api/test_trunk_details.py">https://github.com/openstack/neutron-tempest-plugin/blob/master/neutron_tempest_plugin/api/test_trunk_details.py</a></p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">[12] <a href="https://github.com/openstack/neutron-tempest-plugin/blob/master/neutron_tempest_plugin/api/test_trunk_negative.py">https://github.com/openstack/neutron-tempest-plugin/blob/master/neutron_tempest_plugin/api/test_trunk_negative.py</a></p>

<br /><p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">-- </p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">Slawek Kaplonski</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">Principal Software Engineer</p>

<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">Red Hat</p>

</body>

</html>