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
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>
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
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
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
- List view of all notifications (will be split into acknowleged and
unacknowledged)
- Detail view of each notification
- Acknowledge notifications
- Error notifications show up in red
Change-Id: I0800201b42424bfdd3f27dd090502f28e4f54227
* wrong policy in panel
* shift text to below current sizes
* add * for clarify preapproved quotas
Change-Id: I31f2ad99847236b327ec7d51d4feeca4e2de3569
Allows viewing of current quota value, quota sizes and percentage
of use of current quota.
Allows changing of each region's quota.
Change-Id: Ia9f254ffb905b4e8971d84f85aefad164e8a3438
* Ensures that on logouts indicating success (eg after a password
change) the reason message is highlighted in green rather than
red
Change-Id: I0af763975338733ded792755ffac48614aa6f8c1
Closes-Bug: #1708475
* Accessible at /signup/
* Link placed on bottom of login page.
* Submits a simple signup request to Adjutant.
* Added overextends to based Adjutant enabled file.
* Added back button to match login form to forgot password.
* Change forgot password to match signup autofocus logic.
Change-Id: I1628495a3c35186a45061d07d5e392dc63121ee9
* This is mainly to match other places such as the cli tool
where we will refer to these as 'admin tasks' to differentiate
from Mistral tasks.
Change-Id: I23b65e82d640d6a26e3b44a1d5bcce7e49321071
Reissue token is an option for approved tasks, and rerun validation
is an option for active tasks that have not yet been approved and
invalid.
Change-Id: I0147c31ffbc1a003dfb143adf09b75949eba7c58
* Due to the batch actions looking in the table for the actions
the action would fail because tasks in the approved state
would not be present in the default active table.
* Adding a condition to get the correct table actions.
Change-Id: Iac75ecc5ce91952e0f26746045113024aad6cbb2
Adds a panel in the settings dashboard to allow the user to change
their email address, and a token handler for that page.
Change-Id: I4991fa599f6cfbf65f143e5b2227867df6fb86c3
Kevin Masterson