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

HDDS-7006. Incorrect usedNamespace caused by quota exceeding #3600

Merged
merged 2 commits into from Jul 15, 2022

Conversation

symious
Copy link
Contributor

@symious symious commented Jul 15, 2022

What changes were proposed in this pull request?

The increase of usedNamespace is executed before the check of quota.

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/HDDS-7006

How was this patch tested?

unit test

@symious
Copy link
Contributor Author

symious commented Jul 15, 2022

@adoroszlai @captainzmc @ferhui Could you help to review this PR?

Copy link
Contributor

@kaijchen kaijchen left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for fixing this.

@adoroszlai adoroszlai merged commit 49f2f3b into apache:master Jul 15, 2022
@adoroszlai
Copy link
Contributor

Thanks @symious for the fix, @ferhui and @kaijchen for the review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants