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

mimic: rgw: when using nfs-ganesha to upload file, rgw es sync module get failed #27972

Merged
merged 1 commit into from May 7, 2019

Conversation

@smithfarm
Copy link
Contributor

commented May 6, 2019

http://tracker.ceph.com/issues/37348

Note: this backports the rest of #24492 to mimic. The first commit was backported by @yehudasa as part of #26517

rgw: ES sync: wrap all the decode bls in try block
since decode can throw, wrap all the try block

Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
(cherry picked from commit 12b12cc)

Conflicts:
	src/rgw/rgw_sync_module_es.cc
- mimic uses val.begin() where master has val.cbegin()

@smithfarm smithfarm self-assigned this May 6, 2019

@smithfarm smithfarm added this to the mimic milestone May 6, 2019

@smithfarm smithfarm requested review from yehudasa, theanalyst and cbodley May 6, 2019

@smithfarm smithfarm added rgw and removed core labels May 6, 2019

@smithfarm smithfarm changed the title mimic: when using nfs-ganesha to upload file, rgw es sync module get failed mimic: rgw: when using nfs-ganesha to upload file, rgw es sync module get failed May 6, 2019

@yuriw

This comment has been minimized.

Copy link
Contributor

commented May 6, 2019

@cbodley

This comment has been minimized.

Copy link
Contributor

commented May 7, 2019

@theanalyst @yehudasa can you please review this one?

@cbodley cbodley merged commit d507a9b into ceph:mimic May 7, 2019

4 checks passed

Docs: build check OK - docs built
Details
Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details

@smithfarm smithfarm deleted the smithfarm:wip-37348-mimic branch May 10, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.