| Commit message (Collapse) | Author | Age | Files | Lines |
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
We do not need tox_install.sh, pip can handle constraints itself
and install the project correctly. Thus update tox.ini and remove
the now obsolete tools/tox_install.sh file.
This follows https://review.openstack.org/#/c/508061 to remove
tools/tox_install.sh.
Change-Id: I1e72cf7084ed0b6fcd3c5df1f31b3fa6c7734d31
|
| |\ \ |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Adding 'tap_enabled' parameters to port-pair-group.
If '--enable-tap' set to True, port pairs in port-pair-group will
be deployed as passive Tap service function.
Change-Id: I7267b7ce782f040d0951a3b522d183931debe818
|
| |\ \ \ |
|
| | |/ /
| | |
| | |
| | |
| | |
| | |
| | | |
Co-Authored-By: Igor Duarte Cardoso <igor.duarte.cardoso@intel.com>
Partial-Bug: #1587486
Depends-On: I372da15f99f3cbfb7ffd1d8bf87a79bc56180afe
Change-Id: Ie54da56d2388cb375bccd883c111c5f87e293047
|
| |\ \ \ |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Port pair groups order must be kept so we cannot use a set to filter
duplicates. Else this will change the chain order
Also fixed for flow classifiers: though loosing the order does not
currently trigger a visible change, this restores the neutron CLI
behaviour
Expand help messages to mention the port pair groups order
Change-Id: Ia6e342fda2c739a3553a8736837bcacf00a2d030
Closes-Bug: #1724187
|
| |\ \ \ \
| |/ / / |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Service function parameters could not be set on a port pair, and "None"
correlation value was not accepted
Chain parameters were passed incorrectly to server on a port chain
On a port pair group, lb_fields must be passed to the API with an
underscore
Fix check on empty dictionary parameters (we get [], not None)
Also fix related unit tests
Change-Id: Ie60c609b8d33731a7fed732d97ed92982523956b
Closes-Bug: #1719584
|
| |\ \ \ \
| |/ / /
| | | /
| |_|/
|/| | |
|
| | |/
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Setting a flow classifier or port pair group currently fails,
additionally with multiple flow classifiers only the last one is added
* Fix adding multiple items in the same command
* Update unit test to try setting multiple flow classifiers
Change-Id: Ifbc24d84022c9893d3348fb27b20206fa464d8b2
Closes-Bug: #1716942
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Release notes are version independent, so remove version/release
values. We've found that projects now require the service package
to be installed in order to build release notes, and this is entirely
due to the current convention of pulling in the version information.
Release notes should not need installation in order to build, so this
unnecessary version setting needs to be removed.
This is needed for new release notes publishing, see
I56909152975f731a9d2c21b2825b972195e48ee8 and the discussion starting
at
http://lists.openstack.org/pipermail/openstack-dev/2017-November/124480.html
.
Change-Id: Ie3be535be2de4c27114233cff34f1bb70ca33e46
|
| |/
|
|
|
|
|
|
|
|
|
| |
Neutron is transitioning its CLI to OSC, should update dynamic routing
CLIs to align with OSC plugin.
Change-Id: Ic1a633ca103d9e97e28ddf23e0f528e0261ce497
Co-Authored-By: Ryan Tidwell <ryan@6tidwells.com>
Co-Authored-By: Roey Chen <roeyc@vmware.com>
Partially-Implements: blueprint bgp-spinout
Partial-Bug: #1560003
|
| |
|
|
|
|
|
|
|
|
|
| |
Rename all list_ functions to follow the expected naming format
expected by find_resource() calls: "list_%s" % cmd_resource_plural
Rename for consistency the other SFC commands to include the "_sfc_"
prefix
Change-Id: Ie01692d5cbe928b042c7e8052295a781ac205764
Related-Bug: #1716942
|
| |
|
|
|
|
|
| |
Zuulv3 uses "zuul" user instead of "jenkins", $USER should be set to the
correct value in all cases
Change-Id: Iae78d694a97978144de542608212c77dfe69a390
|
| |\ |
|
| | |
| |
| |
| | |
Change-Id: I57041f72eadc70d2befcd7ce92b3a3b394da4cc6
|
| |/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
neutronclient func test is broken due to missing fully support
of keystone v3 API. It seems keystone v2 API has been disbaled recently
and this triggers the test failures.
There are two causes.
The one is because tempest.lib.cli.base supports only keystone v2 auth info.
There is an on-going review in tempest, but until the next
tempest release we need a workaround. The workaround is to pass
keystone v3 related arguments in neutron() method.
'flags' argument allows us to pass extra auth info to the CLI.
The other reason is because HTTPClient (the legacy python binding class)
supports only keystone v2 API. keystone v3 API is provided via SessionClient.
I believe it is not a good idea to keep the two types of python binding
classes in future. It looks like a time to drop HTTPClient.
As a start, this commit drops HTTPClient func tests.
Note that SessionClient supports keystone v2 as well.
Closes-Bug: #1721553
Related-Bug: #1719687
Change-Id: I6f843e1412400bb1dfb4fc2352fc5cc4c3b837a4
|
| |
|
|
| |
Change-Id: I036c81a2cc84e615d8dd7936ceee60812260abba
|
| |
|
|
| |
Change-Id: I031afd7bc7943d04c098934b30eaf51efa6986ae
|
| |
|
|
| |
Change-Id: Iea16b097e17dfe3244f6e99abfdf8d6a7a1e6179
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Add missing correlation type "mpls" to Port Pair create
command's help (PortPairCreate).
The merged Port Pair command was still using the older help
description, before MPLS correlation support was merged to
networking-sfc.
Also add unit test coverage for correlation type "mpls".
Closes-Bug: #1708968
Change-Id: I708345b25b600ca94504725f1559988ae29413b8
|
| |/
|
|
|
|
|
|
|
| |
Mocking non-existing attribute causes UT faillure.
As a result, neutronclient UT fails depending on
executing order of tests.
Change-Id: Id047527fba9e908938f7157781fb0e36e76011c7
Closes-Bug: #1709652
|
| |
|
|
| |
Change-Id: Iaec435d5f56b48469fff6d53247e9bbf7feb21c3
|
| |
|
|
| |
Change-Id: I39d32b8aea40ec62fd7a4544dc8c99d17b2d4190
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This patch is the first in the series of patches which tracks the
migration of 'public' attribute to 'shared' in FWaaS v2.
Co-Authored-By: Reedip <reedip.banerjee@nectechnologies.in>
Closes-Bug: #1676922
Depends-On: I0be5fca27c9696714ba8b91de2098448c5a18265
Change-Id: Ibc3b90e28ae82ccc8ed044b12d83e97907f1dfb8
|
| |/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
shell.COMMANDS is not intended to be public.
However, 8 projects using neutron CLI extenions access this variable
in their UT. I am honestly surprised on this, but it is the real.
Neutron CLI is now deprecated, so it is better not to break them.
This commit populates shell.COMMANDS dict based on registered commands.
The following is my testting results with 'neutron net-list'.
I ran this 10 times and the average user time is as follows:
before 1.096 sec
after 1.112 sec
More dynamic loading can be explored but I think
this overhead can be acceptable.
Closes-Bug: #1706573
Change-Id: Idcbc64e6f709a666829e0c2985a03aa4517ccf77
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Current code displays "<type 'type'>[port]:xx" for
source/destination ports:
$ openstack sfc flow classifier list -c Summary -f value
protocol: UDP,
<type 'type'>[port]: 192.168.0.14/32[any:any],
<type 'type'>[port]: 192.168.0.16/32[any:any],
neutron_source_port: 5c196983-c319-41d7-9af0-02301ed85513,
neutron_destination_port: None,
l7_parameters: {}
Change-Id: Ia5c772d48b22d4007dad8645cbfd2ea81cbb80ad
Closes-Bug: #1706379
|
| |\ |
|
| | |
| |
| |
| | |
Change-Id: Ia07630b2626781d89b1957a96706e22a9e28674f
|
| |\ \ |
|
| | | |
| | |
| | |
| | |
| | |
| | | |
Implements: blueprint openstackclient-cli-porting
Change-Id: Ifeb62bad26ffeb0bb8b548c56d2d6a620a922f78
|
| | |/
|/|
| |
| | |
Change-Id: I0126609c076ee715b84d357b007ffbef6e0ebfe8
|
| |\ \ |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | | |
Openstack common has a wrapper for generating uuids.
We should use that function to generate uuids for consistency.
Change-Id: I86ae28e6042d1fac2b08540add8a9e23c9254c86
|
| | | |
| | |
| | |
| | | |
Change-Id: I607d35de1ff5d26747629a5f1dd5c114808b5ed7
|
| |/ /
| |
| |
| | |
Change-Id: Idcc7808ae5af7fc42dafc0fc04c3ca1d69239288
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This replaces the doc-migration work. We can avoid the maintenance
cost by generating CLI reference for neutron CLI like this.
This patch introduces a new sphinx directive to render CLI
global options to achieve the goal. This can be migrated to
cliff sphinxext later.
Change-Id: I7089df3f7fc6475ebdd1e71221492baf12f18226
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Recently cliff introduced a sphinx extension directive named autoprogram-cliff
which genarates command-line help automatically. By using it, we no longer
need to write command-line help document separately.
Also fixes minor string substitution issue in a BGPVPN command.
Note that the new cliff release with the directive is not released yet,
but there are several number of OSC plugin commands proposed and
having a local copy would help such developers.
Change-Id: I6b1aee89f406ac449fbc43e210c4ca7ad901b19b
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
* Move OSC first as neutron CLI is deprecated
* Add more information on OSC usage (links to embeded command list
and the mapping guide)
* Add deprecation notice for neutron CLI
This commit also makes toctree structure simpler
At now, we have two index page unlinked from the top page.
I thought it is good first, but it potentially makes things tricky.
Let's remove such tweaks for easy maintenance.
Change-Id: I7fd79f465fa6c04bf5b13f6578796d4c2d01edfd
|
| |\ \ |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
It allows us to generate CLI references automatically using cliff.sphinxext
(which will be available in the next release of cliff).
This allows us to avoid maintaining the in-tree CLI refernece manually.
Change-Id: I3950ed8cf50508210c8b3270724e283aea8e1673
|
| |\ \ \
| |/ / |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The new simple configuration way was introduced in
openstackdocstheme 1.11.0. This commit introduces the new way.
Also drops unnecessary sphinx configurations like latex,
texinfo and manpages.
Change-Id: I587eb8cbd3e64c65dfc7247a01eca90af6adad4e
|
| |\ \ \ |
|
| | |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
In reviews we usually check import grouping but it is boring.
By using flake8-import-order plugin, we can avoid this.
It enforces loose checking so it sounds good to use it.
This flake8 plugin is already used in tempest.
Note that flake8-import-order version is pinned to avoid unexpected
breakage of pep8 job.
Setup for unit tests of hacking rules is tweaked to disable
flake8-import-order checks. This extension assumes an actual file
exists and causes hacking rule unit tests.
Change-Id: I61e683ab0119e4ae90b7107f0690528d789e3875
|
| |/ /
| |
| |
| |
| |
| | |
Part of doc-migration work
Change-Id: I956975ab28be163ca0b2024456360595e649d740
|