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

jewel: rgw: default quota fixes #10832

Merged
3 commits merged into from Oct 13, 2016
Merged

jewel: rgw: default quota fixes #10832

3 commits merged into from Oct 13, 2016

Conversation

dang
Copy link
Contributor

@dang dang commented Aug 23, 2016

This is a backport for http://tracker.ceph.com/issues/16447

@dang dang added the rgw label Aug 23, 2016
@smithfarm smithfarm assigned ghost Aug 23, 2016
@smithfarm
Copy link
Contributor

Thanks for the backport! It's missing "cherry picked from" lines, though, and description of how conflicts (if any) were resolved.

@ghost ghost added this to the jewel milestone Aug 24, 2016
@ghost ghost added the documentation label Aug 24, 2016
ghost pushed a commit that referenced this pull request Aug 24, 2016
Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Aug 24, 2016
Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Aug 24, 2016
Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Aug 24, 2016
Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Aug 24, 2016
Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost
Copy link

ghost commented Aug 25, 2016

Hey jenkins, test this please (jenkins general failure)

@prallabh
Copy link
Contributor

thanks @dang

@ghost ghost changed the title jewel - default quota jewel: default quota fixes Aug 25, 2016
@ghost ghost changed the title jewel: default quota fixes DNM: jewel: default quota fixes Aug 25, 2016
@ghost
Copy link

ghost commented Aug 25, 2016

DNM because it could be responsible for the failures at http://tracker.ceph.com/issues/16344#note-38

@dang
Copy link
Contributor Author

dang commented Aug 29, 2016

New update that fixes a bug in the backport found during downstream testing. It passes the QA tests now.

@dang dang changed the title DNM: jewel: default quota fixes jewel: rgw - default quota fixes Aug 31, 2016
@ghost ghost added the bug-fix label Oct 10, 2016
@ghost
Copy link

ghost commented Oct 10, 2016

This backport passed a rgw suite http://tracker.ceph.com/issues/17487#note-3

@dang could you please update this pull request with commits that are obtained with git cherry-pick -x ? They are formatted in a way that agrees with scripts. I see you commented on how you resolved conflicts which is really helpful.

ghost pushed a commit that referenced this pull request Oct 10, 2016
Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Oct 10, 2016
Reviewed-by: Loic Dachary <ldachary@redhat.com>
@dang dang force-pushed the jewel-default-quota branch 3 times, most recently from 11328d7 to 65a0b10 Compare October 12, 2016 15:58
root added 3 commits October 12, 2016 12:01
Fixes http://tracker.ceph.com/issues/16447

Signed-off-by: Pavan Rallabhandi <PRallabhandi@walmartlabs.com>
Signed-off-by: Daniel Gryniewicz <dang@redhat.com>
(cherry picked from commit 400d7c9)
Fixes http://tracker.ceph.com/issues/16447

This was modified to use size in kB, rather than size in bytes, since
Jewel has not yet been converted to use size in bytes.

Signed-off-by: Pavan Rallabhandi <PRallabhandi@walmartlabs.com>
Signed-off-by: Daniel Gryniewicz <dang@redhat.com>
(cherry picked from commit 17d2c17)
Fixes http://tracker.ceph.com/issues/16447

This was modified to use size in kB, rather than size in bytes, since
Jewel has not yet been converted to use size in bytes.

Signed-off-by: Pavan Rallabhandi <PRallabhandi@walmartlabs.com>
Signed-off-by: Daniel Gryniewicz <dang@redhat.com>
(cherry picked from commit e3ab347)
@dang
Copy link
Contributor Author

dang commented Oct 12, 2016

@dachary Done. Sorry for the noise; git was being uncooperative.

ghost pushed a commit that referenced this pull request Oct 13, 2016
Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost
Copy link

ghost commented Oct 13, 2016

It passed the rgw suite http://tracker.ceph.com/issues/17487#note-11.

@ghost ghost merged commit a336110 into jewel Oct 13, 2016
@theanalyst theanalyst changed the title jewel: rgw - default quota fixes "jewel: rgw: default quota fixes" Nov 17, 2016
@theanalyst theanalyst changed the title "jewel: rgw: default quota fixes" jewel: rgw: default quota fixes Nov 17, 2016
@liewegas liewegas deleted the jewel-default-quota branch November 23, 2016 20:14
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants