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: use correct objv_tracker for bucket instance #4535

Merged
merged 1 commit into from Sep 21, 2015

Conversation

xinxinsh
Copy link

@xinxinsh xinxinsh commented May 4, 2015

When trying to create a bucket that already existed, use the
objv_tracker of the newly created instance, and not of the original
bucket.

Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
(cherry picked from commit fe158ec)
@xinxinsh xinxinsh self-assigned this May 4, 2015
@xinxinsh xinxinsh added this to the firefly milestone May 4, 2015
@ghost
Copy link

ghost commented May 12, 2015

@yehudasa does this backport look good to merge ? It passed a run of the firefly rgw suite ( see http://tracker.ceph.com/issues/11090#rgw for details ).

@ghost
Copy link

ghost commented Jul 8, 2015

@yehudasa ping ?

@athanatos athanatos added rgw and removed core labels Jul 10, 2015
@smithfarm
Copy link
Contributor

@yehudasa This has passed another round of integration testing as detailed in http://tracker.ceph.com/issues/11644

@smithfarm
Copy link
Contributor

@yehudasa This PR is included in the integration branch http://tracker.ceph.com/issues/11644 that just passed another rgw suite http://pulpito.ceph.com/smithfarm-2015-08-10_12:30:50-rgw-firefly-backports---basic-multi/ - is it OK to merge?

@ghost
Copy link

ghost commented Sep 2, 2015

@yehudasa ping ?

@smithfarm
Copy link
Contributor

Passed a rados suite, too.

@smithfarm smithfarm assigned yehudasa and unassigned xinxinsh Sep 7, 2015
yehudasa added a commit that referenced this pull request Sep 21, 2015
rgw: use correct objv_tracker for bucket instance
@yehudasa yehudasa merged commit bb8b422 into ceph:firefly Sep 21, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants