| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
| |
When deleting a secret that has consumers, the
--force parameter must be specified in the CLI.
Change-Id: I49d19ac843d5c805fd7f533d07a3a719ce9a1104
|
|
|
|
|
|
|
|
|
|
| |
This adds the Python API for secret consumers,
similar to the container consumers API.
Also adding functional tests for both secret
and container consumers.
Co-Authored-By: Grzegorz Grasza <xek@redhat.com>
Change-Id: I024db27d12ea33713bda0273d8748577cc89a38d
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch introduces a new `microversion` parameter to the factory
method for the Client class. The parameter is used to set
`default_microversion` in the HTTPClient.
Co-Authored-By: Andre Aranha <afariasa@redhat.com>
Co-Authored-By: Grzegorz Grasza <xek@redhat.com>
Depends-On: https://review.opendev.org/c/openstack/keystoneauth/+/865439
Change-Id: I2a0a5315daca8ce5bd6164cebbd2f917348c7675
|
|
|
|
|
|
|
|
| |
... because u'...' is equivalent to '...' in Python 3. Also, removed
the unicode prefix from example outputs because it is no longer shown
in Python 3.
Change-Id: I1f8fbec1d711659872cb07d2dcef9e33d5a9d21a
|
|
|
|
|
|
|
|
| |
Python 2 is no longer supported, so we no longer need the six library.
This change removes usage of the library as well as reference to it
(and Python 2) from the documentation.
Change-Id: I6328b11dcad54f70f64ecff53eb60708e34351cf
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
As discussed in TC PTG[1] and TC resolution[2], we are
dropping the lower-constraints.txt file and its testing.
We will keep lower bounds in the requirements.txt file but
with a note that these are not tested lower bounds and we
try our best to keep them updated.
This patch also fixes a broken test by updating the maximum
payload size to match the current maximum in Barbican. [3]
This patch also skips an additional broken test. It will be
fixed in a separate patch.
[1] https://etherpad.opendev.org/p/tc-zed-ptg#L326
[2] https://governance.openstack.org/tc/resolutions/20220414-drop-lower-constraints.html#proposal
[3] https://opendev.org/openstack/barbican/src/branch/stable/yoga/barbican/common/config.py#L33
Change-Id: Id192b8eef2c2518b8a89f200727bd03c47dbd07e
|
|
|
|
|
|
|
|
|
|
|
| |
Port the functional job to the native Zuul v3 format,
and remove a few old wrapper scripts which are not needed
anymore (the tests can be run locally through `tox -efunctional`
as well).
Also, switch to the Victoria run time for unit tests.
Co-Authored-By: Luigi Toscano <ltoscano@redhat.com>
Change-Id: Ibce326044c240d6c4e7f1ca3c3ac50a18282a86d
|
|
|
|
|
|
|
|
|
| |
The repo is Python 3 now, so update hacking to version 3.1.0 which
supports Python 3.
Fix problems found.
Change-Id: I54f9f7c5f6ee87ee886658388541f45e23c7546d
|
|
|
|
|
|
|
|
|
|
| |
- Fix incorrect type handling of secret payload in py3
(it should be bytes instead of str by default)
- Fix py3 compatibility of test codes
- Update expiration date so that resources are created with valid
expiration.
Change-Id: I4935f601f87e9c49499da1034a320eee2e655b4d
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch adds a --file (-F) flag to both 'secret store' and 'secret
get' commands to work around a cliff formatter limitation when dealing
with binary data. This flag can be used to store a payload directly
from a file so that a user doesn't have to do any base64 juggling of
their data.
The flag can also be used to save a payload directly to a file so we can
avoid UTF-8 and ASCII encoding errors thrown by the cliff formatters
that expect text data.
Original Review: https://review.openstack.org/#/c/388981
Co-Authored-By: douglas.mendizabal@rackspace.com
Change-Id: Iee7e2bbb95238a9bbb9e8e6124fe1663da377939
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Currenlty, some gates is being failed with logs like this
"""
b'Response: {"description": "Provided object does not match schema
\'Secret\': \'expiration\' is before current time.
Invalid property: \'expiration\'", "title": "Bad Request", "code": 400}'
"""
So it is necessary to update the time to create secrets successfully.
Change-Id: I4f2d3817d9a2380919d16e76ef5dcba3ec62e5a1
|
|\ |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Probably the most common format for documenting arguments is reST field
lists [1]. This change updates some docstrings to comply with the field
lists syntax.
[1] http://sphinx-doc.org/domains.html#info-field-lists
Change-Id: I1e4221e886a75177c4607817a78dbcda37dfa57a
|
|/
|
|
| |
Change-Id: Ie5f8d146430d6a94c22ad0f5175f492ea1985d19
|
|
|
|
|
|
|
|
|
| |
Add v1 directory for APIs, CLIs and API testcases.
Currently, v1 is supported and the default api version. If version is not
specified when constructing barbicanclient, default api version will be used.
Implements: blueprint refactor-barbicanclient
Change-Id: Ib91e7049de007f4d4254abcd4c125b4dc7e03c55
|
|\ |
|
| |
| |
| |
| |
| |
| |
| |
| | |
Exposes the secret_type, created, updated, expiration, and sort as
filters for listing secrets.
Change-Id: I583f27f91cb3c6bdb23438dff6b539407b4005ed
Depends-On: I95b5390ef24a754f66ccdb9ccde34cd0820b02fd
|
|/
|
|
|
|
|
|
|
| |
Switch both client and functional test to use generic auth plugin.
Also keep keystone auth_uri as one option so we needn't add extra
version to the endpoint and let keystoneauth handle this.
Change-Id: I5535d4d459496eec9052131b43615e6056e249e8
Closes-bug: #1691480
|
|
|
|
|
|
|
|
| |
We just run pep8 test on barbicanclient directory and functionaltests
directory is not included. This patch extends pep8 test coverage and
removes the script for running pep8 test.
Change-Id: I2b4fe6fadd69884c749a91b390297600bfa40182
|
|
|
|
|
|
|
|
|
| |
DevStack was updated so that the port can't be used
with the keystone URI anymore.
See I46294fb24e3c23fa19fcfd7d6c9ee8a932354702
Change-Id: I808240d534be5e4d50f3f51e6b8318cbffec9c96
|
|
|
|
|
| |
Change-Id: I9f2c72e89bd7305277d7a531c719e02423e1dccb
Closes-Bug: #1651047
|
|
|
|
|
|
|
|
|
|
|
|
| |
1.As mentioned in [1], we should avoid using six.iteritems to achieve
iterators. We can use dict.items instead, as it will return iterators
in PY3 as well. And dict.items/keys will more readable.
2.In py2, the performance about list should be negligible, see the
link [2].
[1] https://wiki.openstack.org/wiki/Python3
[2] http://lists.openstack.org/pipermail/openstack-dev/2015-June/066391.html
Change-Id: I093160f746f346871c136043a8fa48c9491fdc1a
|
|\ |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The deletion of the secrets wasn't happening correctly in the
clean-up methods for the CLI functional tests because items
were being deleted from the lists as the list was being iterated
over.
Includes general clean-up of the entities being created and deleted.
Change-Id: I87d0ef93505ee46ab8314a2562b5d4ab59717649
|
|\ \
| |/
|/| |
|
| |
| |
| |
| |
| |
| |
| |
| | |
Openstack common has a wrapper for generating uuids. We should
use that function when generating uuids for consistency.
Change-Id: I95a4a8c8e9b4501d4f9796309beb50971f2a7222
Closes-Bug: #1082248
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Re-adding the skip decorator because the content type still cannot be
null. I98a558862eed039ed6f33fbee474bdb86767313e was supposed to fix it,
but something is still broken, and a few extra checks in the test show
this is true.
Change-Id: Icb5c0e28bf0199962d2082143099905d4fb63685
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
| |
The smoke test that ran the CLI command "acl delete"
with a null entity_ref was failing with an exception
in the cliff library. This test is invalid, since
the CLI parser would not accept this command without
an entity_ref.
This test failed more gracefully with cliff 2.3.0, but
in 2.4.0 cliff throws an exception that causes this
test to start failing.
Change-Id: If91d4966eff77dad7c8569e309134dc540edc258
|
|\ |
|
| |
| |
| |
| |
| |
| | |
TrivialFix
Change-Id: I4e9265439856b5947a657f8419334d19ebb0a07f
|
|\ \ |
|
| |/
| |
| |
| |
| |
| |
| |
| | |
Keystoneauth was extracted from keystoneclient. This CR replaces usage
of keystoneclient in favor of keystoneauth.
Co-Authored-By: Clenimar Filemon <clenimar@lsd.ufcg.edu.br>
Change-Id: I4262bed1e595d58f31fe80c85e3692e4ca2840fa
|
| |
| |
| |
| |
| |
| |
| | |
The correct order expected by testtools is assertEqual(expected, observed).
Change-Id: I8457af6e15a9000dd6c54ad8199331e75bd3c7fd
Partial-Bug: #1259292
|
|\ \
| |/
|/| |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
assertEqual expects that the arguments provided to it should be
(expected, observed). If a particluar order is kept as a convention,
then it helps to provide a cleaner message to the developer if Unit
Tests fail. The following patch fixes this issue.
Change-Id: I5209e5889cfe62c66dc675c00e3ccaf22410f637
Partial-Bug: #1259292
|
|/
|
|
|
|
|
|
|
|
|
|
|
| |
``acl_users`` can be either a set or a list. As long as
Python sets are not JSON serializable, cast them to lists
in functional tests.
This way we avoid gate failures like [1].
[1]
http://logs.openstack.org/46/319446/7/check/gate-python-barbicanclient-devstack-dsvm/cb80e0c/console.html#_2016-08-04_15_38_51_649177
Change-Id: I6b8601f7168120cf0e15716cc22b8834ddb127e8
|
|
|
|
|
|
|
|
|
|
| |
When using KMIP backend, some barbican client functional
tests failed in payload length validation. Fixed the test case
payload data, for client/v1/functional/test_secrets and
cli/v1/smoke/test_secret
Change-Id: Ic0abe8b6094d838fb4ec33cf22a5ca4c0450de31
Closes-Bug: #1570166
|
|
|
|
|
|
|
|
|
| |
When using KMIP backend, some barbican client functional
tests failed in payload length validation. Fixed the test case
payload data.
Change-Id: I9d0b2d04662b3a7399603b9eff20d9daf406e069
Closes-Bug: #1570166
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
assertEqual expects that the arguments provided to it
should be (expected, observed). If a particluar order
is kept as a convention, then it helps to provide a cleaner
message to the developer if Unit Tests fail.
The following patch fixes this issue.
TrivialFix
Change-Id: I6a8e11df30516f6be9a745b1f0f68ef3f2825fcf
Closes-Bug: #1259292
|
|
|
|
|
|
|
| |
Import six.moves.urllib.parse as urlparse for python3 compatible.
Change-Id: I0f28f01a54daaa690cd890540fd4edc3b32411d1
Closes-Bug: #1279611
|
|
|
|
|
|
|
|
|
|
| |
Use "keystoneclient.exceptions" instead of the deprecated
"keystoneclient.openstack.common.apiclient.exceptions".
ref: https://github.com/openstack/python-keystoneclient/blob/master/
keystoneclient/openstack/common/apiclient/exceptions.py#L25
Change-Id: Ic0fb631aa9f5f3cab2d9829420da746e37d0b0eb
|
|
|
|
| |
Change-Id: I6e022008940819c479afc040441e0e5e029d3bd6
|
|
|
|
|
|
| |
Change-Id: Id03e05edd300ae141a70115bb30d9e998a501ec2
Depends-On: I98a558862eed039ed6f33fbee474bdb86767313e
Closes-bug: #1425667
|
|
|
|
|
|
|
|
|
|
| |
Added functional tests for CLI commands.
Added functional tests for client API.
Modified ACL client API functional test not to use behavior class
as per recent change in secrets, containers, and orders.
Change-Id: Ibacf33e834daa2c70ef1b0032495bbe443957cac
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The client currently only accepts '3.0' which directly conflicts with
other clients (like OSC). It is not currently possible to export
OS_IDENTITY_API_VERSION and have it work properly for both clients.
In addition, the way it is written now "DEFAULT" is not really a
proper default, as 2.0 is the fallback. This change resolves that
inconsistency.
Closes-bug: #1496597
Change-Id: I0d829e46fc251eb7762912c7f89f79cb267e6abe
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is the first implementation of the barbicanclient plugin for the
OpenStack CLI. Since we chose to use cliff in for the client, the
integration with the OCC is quite seamless. Only change that needed to
be done was that, since the OCC uses a ClientManager class to fetch the
specific libraries for the components, we needed to simulate that in the
barbicanclient.
Currently, the plugin can be used if the barbican server is configured
with keystone authentication. This is because currently there is no way
for us to set the X-Project-Id header manually from the client (if the
plugin is being used) but that functionality is out of the scope of this
first implementation.
Change-Id: I37fd158af24e785bc0b7125c6b4c1a9795927a10
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Removes the behaviors folder from functionaltests.client.v1.
Must wait until the following patches are merged:
https://review.openstack.org/#/c/213414/
https://review.openstack.org/#/c/213419/
https://review.openstack.org/#/c/213420/
https://review.openstack.org/#/c/217280/
https://review.openstack.org/#/c/217298/
https://review.openstack.org/#/c/217312/
Change-Id: I64509adb05a6061d7d86ab6db26f140e7068a1c8
|
|\ |
|
| |
| |
| |
| |
| |
| |
| | |
Will remove the test behaviors abstraction from orders smoke tests.
Tests will be rewritten to use the barbican-client directly.
Change-Id: I5d60c01055cf31f6dbbb24bae5f57c04c453d5f5
|
|\ \ |
|