The key/value pairs being checked include the quota's id, and the value
is a UUID. Being that this is the only key where the value is not an
integer, this causes issues with Python 3's rules for comparisons
between different types. This manifests as a "Failed to list quota
sizes." error when viewing a region's quota information. This patch
will catch this exception and allow it to continue as before, without
modifying the value.
I opted for this method instead of just skipping 'id' at the start of
the loop since this is closer to the original behavior, and there is
already an explicit check for 'id' at the bottom before adding to the
output. This way, any future keys that also have non-int values will
be handled gracefully as well.
Change-Id: Ib0e53572bfc8600094a6f8483e26bf2d0023ce80
The mock third party library was needed for mock support in py2
runtimes. Since we now only support py36 and later, we don't need it
in lower-constraints.
These changes will help us to drop `mock` from openstack/requirements
Change-Id: I44e4eb34d8e4e30d7a3bb9e1361a259d8f6c9d19
Switch to openstackdocstheme 2.2.1 and reno 3.1.0 versions. Using
these versions will allow especially:
* Linking from HTML to PDF document
* Allow parallel building of documents
* Fix some rendering problems
Update Sphinx version as well.
Disable openstackdocs_auto_name to use 'project' variable as name.
Change pygments_style to 'native' since old theme version always used
'native' and the theme now respects the setting and using 'sphinx' can
lead to some strange rendering.
openstackdocstheme renames some variables, so follow the renames
before the next release removes them. A couple of variables are also
not needed anymore, remove them.
See also
http://lists.openstack.org/pipermail/openstack-discuss/2020-May/014971.html
Change-Id: Ib3db49d1256072975e9f3a9472cb27ca6d9895f6
flake8 new release 3.8.0 added new checks and gate pep8
job start failing. hacking 3.0.1 fix the pinning of flake8 to
avoid bringing in a new version with new checks.
Though it is fixed in latest hacking but 2.0 and 3.0 has cap for
flake8 as <4.0.0 which mean flake8 new version 3.9.0 can also
break the pep8 job if new check are added.
To avoid similar gate break in future, we need to bump the hacking min
version.
- http://lists.openstack.org/pipermail/openstack-discuss/2020-May/014828.html
Change-Id: If6bd35105d281ebf179dbdd4d3636a0419d5cd5c
This repo is now testing only with Python 3, so let's make
a few cleanups:
- Remove python 2.7 stanza from setup.py
- Add requires on python >= 3.6 to setup.cfg so that pypi and pip
know about the requirement
- Update classifiers
- Use newer openstackdocstheme and Sphinx versions
- Build docs, don't publish yet
Change-Id: I2de4e1dbac061002940f80b2cf129dc343343a0d
Now that we are running the Victoria tests that include a
voting py38, we can now add the Python 3.8 metadata to the
package information to reflect that support.
Change-Id: I91cbe83df8580d72bee157c1c430b9f1f69abf3f
Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
The repo is Python 3 now, so update hacking to version 3.0 which
supports Python 3.
Fix problems found.
Remove hacking and friends from lower-constraints, they are not needed
for installation.
Change-Id: I7dd7a16d2b2d75120fa14de1d5c080a1a1c9729d
This updates the docs build command to follow the instructions from the
project team guide.
Use newer TOX_CONSTRAINTS variable.
Change-Id: I4381ae349e2988b2654696086ca5f7159b695b7e
Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
Now that we no longer support py27, we can use the standard library
unittest.mock module instead of the third party mock lib.
Change-Id: Icc42a058851448152ddc86fded6b8808e2e704a4
Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
1. Sync sphinx dependency with global requirements. It caps python 2 since
sphinx 2.0 no longer supports Python 2.7.
2. Remove unncessary "=="
Change-Id: I1d321e17470216b76fca84f21896bc1457a6a92a
fix and enable:
- E121 continuation line under-indented for hanging indent
- E126 continuation line over-indented for hanging indent
- E226 missing whitespace around arithmetic operator
- H306 imports not in alphabetical order
- H401 docstring should not start with a space
Change-Id: I1dab6908652924931e750187380d153d64eba2b6
Add first jobs for this repo - without jobs, nothing can merge.
Disable failing pep8 tests for now.
The repo previously had noop-jobs defined in project-config but that got
removed.
Further jobs and templates can be added later.
Change-Id: I17734aab0aec5e74ecab941f4d142c68184b6700
This matches the new changes to Adjutant as part of
the refactor to split API and Workflow layers.
202 will be the most common response code when dealing
with the creation of new tasks due to future async
task processing.
Change-Id: Ia09a070709b8a2973c9149df91c67af2b69199a0
This repo does not support Python 2 anymore, so we don't need
six for compatibility between Python2 and 3, convert six usage to Python
3 code.
Needed-By: https://review.opendev.org/701743
Change-Id: Ia51c0af5df488069f3048f632f56774dcb4344ef
Horizon test setting no longer depends on nose and related stuffs.
This commit cleans up nose related dependencies.
Filenames of test modules need to start with test_*
so that Django test runner detects them.
There is no test defined but it would be nice to adjust the filename.
Also moves horizon dependencies to requirements.txt.
Change-Id: Ib2f246d76da8a1f5d26a9cf8e9ef27e5f4f62bee
Horizon in Rocky 'fixed' recursive inheritance but this
in turn broke our use of overextends. As such we drop
overextends for Rocky onwards, but this means Adjutant-UI
from Rocky onwards only works with Horizon from Rocky onwards.
This also fixes issues with quota naming cleanup which was
changed in Horizon proper.
Change-Id: I60fa4e8e751bb62760830e37775c42b9900fb59c
When Adjutant has sizes defined for a service that
only exists in certain regions, this can cause errors.
If we come across a region without that service, skip
the service.
Change-Id: Icc6b8cc3aaed408c10dc4fb54c54fe2376cf2a88
This column should have always been roles, this renames that
properly. No point having the data source be called roles
and the column be called role.
Change-Id: I6ce736bfa1449fccf8f9f96d504a236546d76e82
PBR by default gets most files in the repo itself
so a manifest file isn't needed.
"MANIFEST.in generation ensures almost all files
stored in Git, with the exception of .gitignore,
.gitreview and .pyc files, are automatically
included in your distribution."
Change-Id: I5a3dd88e92eee6d7f28d3578e412689fa1f729f4
Adjutant-ui was requesting data for all regions when
it didn't need usage for all regions. This gives us a
way to avoid that.
Change-Id: Iecb474b86c6cd1454f4eb61f7b1326bd65b1dcfa
* add documentation for settings
* clean up documentation
* split role help into own template
* remove old debug print statements
Change-Id: Id764dca96e37627974c77181e73634f461ab10b1