Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rc 2021.02.09 #2647

Merged
merged 10 commits into from
Feb 9, 2021
Merged

Rc 2021.02.09 #2647

merged 10 commits into from
Feb 9, 2021

Conversation

dccurtis
Copy link
Contributor

@dccurtis dccurtis commented Feb 9, 2021

No description provided.

blentz and others added 10 commits February 3, 2021 12:16
* reapply application-side text field size check as a mixin
* Mixin changed to only validate TextField

* protection against non-str values
* COST-954: add AnyUserAccess; small refactor
* add beta param to user_access API
Sources stability with new application-extra flow
* COST-900: Create the storage api.

Co-authored-by: Douglas Curtis <[email protected]>
@codecov
Copy link

codecov bot commented Feb 9, 2021

Codecov Report

Merging #2647 (373ba63) into stable (422de7c) will increase coverage by 0.3%.
The diff coverage is 97.6%.

@@           Coverage Diff            @@
##           stable   #2647     +/-   ##
========================================
+ Coverage    94.5%   94.8%   +0.3%     
========================================
  Files         232     282     +50     
  Lines       16442   21665   +5223     
  Branches     1826    2455    +629     
========================================
+ Hits        15533   20533   +5000     
- Misses        582     680     +98     
- Partials      327     452    +125     

Copy link
Contributor

@lcouzens lcouzens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

=========================== short test summary info ============================
FAILED tests/rest_api/v1/test_aws_org_units.py::test_api_aws_org_units_returned_data_validation[org_unit_with_region]
FAILED tests/rest_api/v1/test_ocp_on_cloud_storage_reports.py::test_api_ocp_on_cloud_storage_validate_data_integrity[clusters]
FAILED tests/rest_api/v1/test_ocp_on_cloud_storage_reports.py::test_api_ocp_on_cloud_storage_validate_data_integrity[regions]
FAILED tests/rest_api/v1/test_source.py::test_api_aws_source_crud - wait_for....
FAILED tests/rest_api/v1/test_source.py::test_api_azure_source_crud - wait_fo...
ERROR tests/rest_api/v1/test_cost_model.py::test_api_cost_model_negative_tiered_rate_unbound_mid
ERROR tests/rest_api/v1/test_user.py::test_api_settings_negative_tag_enablement
= 5 failed, 2249 passed, 245 skipped, 769 deselected, 1 warning, 2 errors in 2659.40s (0:44:19) =
Crud tests and errors are hitting a caching issue that I am investigating currently, OU test is missing data from the api and can be investigated later, as for integrity tests they are off by about 10 units where 10.0 = abs((148932.0 - 148942.0)) probably a timezone nise issue we can pick up later too. Otherwise this is ready to rock!

@dccurtis dccurtis merged commit db8ec4b into stable Feb 9, 2021
@dccurtis dccurtis deleted the rc-2021.02.09 branch February 9, 2021 19:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants