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

rgw: fix the quota check when S3 multipart upload #6460

Closed
wants to merge 1 commit into from

Conversation

Aran85
Copy link
Contributor

@Aran85 Aran85 commented Nov 4, 2015

when the remainder quota of one bucket exceed the maxnum's 95%,then the next file upload can't succeed if it's multipart num more than maxnum's 5%.

chang the quota check timing,only multi post request check quota,multi put will not check obj num again.

Fix: #13223

Signed-off-by: Zengran Zhang zhangzengran@h3c.com

@Aran85
Copy link
Contributor Author

Aran85 commented Nov 6, 2015

for instance :
bucket quota max-objects=100
bucket usage quota = 95
the next upload file will fail if it have 6 multiparts.

@yehudasa could you take a look? thanks!

@smithfarm
Copy link
Contributor

@yehudasa ping

@yehudasa
Copy link
Member

yehudasa commented Dec 1, 2015

@Aran85 I'm not sure this is a good idea. This can be taken advantage of by users that would upload infinite amount of data by creating multipart uploads without completing. We do want to account that data even if the upload is not complete.

@Aran85
Copy link
Contributor Author

Aran85 commented Dec 2, 2015

@yehudasa maybe the --max-size option will works when anyone upload infinite amount of data by creating multipart uploads?

@Aran85
Copy link
Contributor Author

Aran85 commented Dec 28, 2015

ping @yehudasa

@liewegas
Copy link
Member

@Aran85 rebase please? @yehudasa can you take a look?

when the remainder quota of one bucket exceed the maxnum's 95%,then the next file upload can't succeed if it's multipart num more than maxnum's 5%.

chang the quota check timing,only multi post request check quota,multi put will not check obj num again.

Fix: ceph#13223

Signed-off-by: Zengran Zhang <zhangzengran@h3c.com>
@yehudasa
Copy link
Member

@Aran85 looking at it again, I still don't think we should be doing it. The num_objs might not reflect the exact number of complete objects, but that's really ok. It can't really ever be very accurate. When dealing with small numbers it is very noticeable, but practically the num_objs limit should be set to a much higher number, so the inaccuracy due to in-flight uploads is negligible.

@Aran85
Copy link
Contributor Author

Aran85 commented May 20, 2016

@yehudasa can we make a Independent check for every single multi upload's parts num? or close this PR?

@yehudasa
Copy link
Member

@Aran85 not sure if I understand what you mean by independent check for every single multi upload's part num. Maybe when completing / aborting a multipart upload we need to deal with the accounting correctly?

@Aran85
Copy link
Contributor Author

Aran85 commented May 27, 2016

the completing/aborting of multipart upload works well now :). bc i think the rules of quota check a little hard to explain.

@yehudasa
Copy link
Member

yehudasa commented Oct 4, 2016

@Aran85 what is the status of this PR?

@Aran85
Copy link
Contributor Author

Aran85 commented Oct 9, 2016

@yehudasa have no idea about treat it as bug or strategy.

@liewegas
Copy link
Member

Closing this, feel free to reopen if it should be resurrected!

@liewegas liewegas closed this Mar 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
6 participants