Currently in devstack README.md path to vagrant file is wrong.
This patch fixes it.
Change-Id: I71bf1c6a0808113afbfa1ebd11f57200e21e3a72
Closes-bug: #1627489
We had tools/tox_install.sh for the case we could depend on neutron
which was not on pypi. In the end, we depend only on
python-neutronclient and as such, this workaround is not necessary and
only makes tox slower.
Change-Id: I558c1dbb2730ef41d3ec7d211f6079f051b53b6d
Signed-off-by: Antoni Segura Puimedon <antonisp@celebdor.com>
Exceptions are raised by the sys.exit() function.
When they are not handled, no stack traceback
is printed in the Python interpreter.
Therefore, when using sys.exit(start())
instead of start() may be more readable and reasonable.
Change-Id: I8887607264c9ead220e66135dab06c9562f0b531
With this change, we are future proofing our code as well as reducing
the amount of external packages to install in Python3 as Netaddr is not
part of python3 nor python2 standard library.
Change-Id: Ie53282f47b294741a5d10ab63ae177ac56c528b4
implements: blueprint netaddr-to-ipaddress
Signed-off-by: Antoni Segura Puimedon <antonisp@celebdor.com>
The python-neutronclient module that we were getting our base test class
from was intended to serve for tests that emulate command line input and
output and, as such, it took control of stdin and stdout.
It is inconvenient to have stdin and stdout captured because that
prevents the developers from printing or attaching debuggers when
running unit tests.
Change-Id: I729652d0b5bf8a852950535cce36195397165581
Closes-bug: #1626542
Signed-off-by: Antoni Segura Puimedon <antonisp@celebdor.com>
The last devstack patch introduced usage of the service project and a
Kuryr user, however, it was with the services role, while Kuryr needs
the 'admin' role.
Closes-bug: #1625482
Change-Id: If517f77c7aec3fad81af101ccc174167f16ae37b
Signed-off-by: Antoni Segura Puimedon <antonisp@celebdor.com>
Return endpoint operational info to libnetwork,
return value is from Neutron port's status.
Closes-Bug: #1620883
Change-Id: Ib5569cfdaa0c5934e2d24e21c27ca4164b8b8d03
For now we could safely remove these options,
once we have complete documentation it could be added easily.
Change-Id: I15f19c1cee2e2a65190d1780b41bdf406634253e
Closes-bug: #1625015
In order to have Docker easily usable for the fullstack testing, we made
it listen to the port 2375. This had an unintended and very bothering
side effect in forcing cli usage to have to pass '-H :2375' to every
signle docker command.
It was driving me crazy, so this should make both the normal cli usage
and the fullstack usage work without hassle.
Change-Id: I0d6162937e018664516de406197ca0a464b4ee71
Signed-off-by: Antoni Segura Puimedon <antonisp@celebdor.com>
Up until this patch, the Docker container for the Kuryr libnetwork
daemon was part of contrib and was being built by checking out the
latest master from github. That is not very helpful when developers
want to build the container to run their latest changes.
With this change, we move the Docker building to the root of the repo so
that it can use the current state of the repo to build and update the
README.rst to contain the information relative to building and using the
container.
Change-Id: Ie9032d62c73d873a29cabe84c64a19df13fdb749
Signed-off-by: Antoni Segura Puimedon <antonisp@celebdor.com>
In order to integrate with Kolla and to support newer deployments that
have only keystonev3 enabled, we leverage keystoneauth1.
I want to thank Fawad for taking a stab at fixing this earlier.
Depends-On: Ia56b22963f5a2130a722403a14ee1a233ac6cff9
Closes-bug: #1622302
Change-Id: Id57420ef94ea9bd71b6581c4cdc24697465a3445
Signed-off-by: Antoni Segura Puimedon <antonisp@celebdor.com>
Co-authored-by: Liping Mao <limao@cisco.com>
Everything in this file is automatically generated by pbr.
There appears to be no good reason to keep it around.
Change-Id: I337e4d573a615280d7cfd95bf52be45176a4dcf4
Common files will be at kuryr-lib. Whatever is there in
kuryr_libnetwork/common is specific to libnetwork so it should
be moved accordingly
Change-Id: I84220aa358375920bd0a135d3eb8c23c7386d947
Closes-bug: #1617521
With the advent of this tool. Generating the config for local usage and
packaging will be much simpler.
Closes-bug: #1619252
Change-Id: I95f8f0ec76fb8a75153f6dac12c6ec1d2f485e8b
Signed-off-by: Antoni Segura Puimedon <antonisp@celebdor.com>
Since commit[1] had renamed kuryr utils to kuryr-libnetwork utils,
but there are some duplicate codes between lib and libnetwork repo.
[1] https://review.openstack.org/#/c/337350
Change-Id: I3dbf737b2a3e23b91a756cc5ff39ff7bbf5a818c
Closes-Bug: #1618012
Move oslo.serialization from requirements to test-requirement.
Sort requirements and test-requirements in alphabetical order.
Change-Id: Ic587843283d58c1df457f198afe7da0b8fac06bf