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
hammer: osdc/Objecter: If osd full, it should pause read op which w/ rwordered flag #13784
Conversation
d609dab
to
5100c5e
Compare
retest this please |
ac3d3a1
to
ffa2f84
Compare
ffa2f84
to
83af8cd
Compare
f14e8b8
to
34e2485
Compare
FULL_TRY = try to do the op; ENOSPC if it results in a net increase is space. client must cope. FULL_FORCE = do the op even if it consumes space. The MDS will use this. We should restrict this based on the OSD cap (* vs w, probably). Signed-off-by: Sage Weil <sage@redhat.com> (cherry picked from commit 95055e7)
c039b27
to
64658ac
Compare
resend writes after pool loses full flag Signed-off-by: xinxin shu <xinxin.shu@intel.com> (cherry picked from commit dbcf2e4) Conflicts: src/osdc/Objecter.cc src/osdc/Objecter.h
64658ac
to
2b0646a
Compare
retest this please |
1 similar comment
retest this please |
@@ -410,6 +410,8 @@ enum { | |||
pool uses pool snaps */ | |||
CEPH_OSD_FLAG_REDIRECTED = 0x200000, /* op has been redirected */ | |||
CEPH_OSD_FLAG_KNOWN_REDIR = 0x400000, /* redirect bit is authoritative */ | |||
CEPH_OSD_FLAG_FULL_TRY = 0x800000, /* try op despite full flag */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@shinobu-x (NIT) Please try to use title of PR < 50 characters for future PRs.
https://github.com/ceph/ceph/blob/master/SubmittingPatches.rst#title-of-pull-requests-and-title-of-commits
@@ -898,7 +898,8 @@ bool Objecter::ms_dispatch(Message *m) | |||
|
|||
void Objecter::_scan_requests(OSDSession *s, | |||
bool force_resend, | |||
bool force_resend_writes, | |||
bool cluster_full, | |||
map<int64_t, bool> *pool_full_map, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@shinobu-x Looks indent is incorrect
@@ -2357,8 +2379,9 @@ bool Objecter::is_pg_changed( | |||
|
|||
bool Objecter::target_should_be_paused(op_target_t *t) | |||
{ | |||
const pg_pool_t *pi = osdmap->get_pg_pool(t->base_oloc.pool); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@shinobu-x (NIT) Coding guide says about const names:
https://google.github.io/styleguide/cppguide.html#Variable_Names
Your thought please?
Hammer is EOL. |
http://tracker.ceph.com/issues/19141