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

fix the issue which when quota is None report error and change filesystem used capacity #599

Merged
merged 10 commits into from
Jun 17, 2021

Conversation

tanjiangyu-ghca
Copy link
Contributor

@tanjiangyu-ghca tanjiangyu-ghca commented Jun 8, 2021

What this PR does / why we need it:
1.fix the issue which when quota is None report error;
2.change filesystem used capacity;
3.change filesystem worm value;

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #

Special notes for your reviewer:

Release note:

@codecov
Copy link

codecov bot commented Jun 8, 2021

Codecov Report

Merging #599 (86cf634) into master (58256ed) will increase coverage by 0.00%.
The diff coverage is 66.66%.

@@           Coverage Diff           @@
##           master     #599   +/-   ##
=======================================
  Coverage   72.09%   72.10%           
=======================================
  Files         141      141           
  Lines       12462    12466    +4     
  Branches     1480     1480           
=======================================
+ Hits         8985     8989    +4     
  Misses       2976     2976           
  Partials      501      501           
Impacted Files Coverage Δ
delfin/drivers/dell_emc/unity/rest_handler.py 50.00% <ø> (ø)
delfin/drivers/dell_emc/unity/unity.py 76.88% <66.66%> (+0.26%) ⬆️

@tanjiangyu-ghca tanjiangyu-ghca changed the title fix the issue which when quota is None report error fix the issue which when quota is None report error and change filesystem used capacity Jun 15, 2021
Copy link
Collaborator

@wisererik wisererik left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@ThisIsClark ThisIsClark left a comment

Choose a reason for hiding this comment

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

LGTM

@ThisIsClark ThisIsClark merged commit bb2cc0e into sodafoundation:master Jun 17, 2021
andrewliu83 referenced this pull request in gh-ca/delfin Jun 18, 2021
@tanjiangyu-ghca tanjiangyu-ghca deleted the unity0608 branch June 30, 2021 06:38
joseph-v pushed a commit that referenced this pull request Dec 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants