Project

General

Profile

Bug #9418

mon: drop internal-purpose messages from clients without proper caps

Added by Joao Eduardo Luis over 9 years ago. Updated over 8 years ago.

Status:
Resolved
Priority:
Urgent
Assignee:
Joao Eduardo Luis
Category:
Monitor
Target version:
-
% Done:

0%

Source:
other
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Related issues

Copied to Ceph - Backport #12833: mon: drop internal-purpose messages from clients without proper caps Rejected 09/10/2014

History

#1 Updated by Joao Eduardo Luis over 9 years ago

  • Subject changed from mon: drop internal-purpose messages from clients with proper caps to mon: drop internal-purpose messages from clients without proper caps

#2 Updated by Joao Eduardo Luis over 9 years ago

  • Status changed from New to 7

#3 Updated by Sage Weil over 9 years ago

  • Status changed from 7 to Pending Backport

#4 Updated by Loïc Dachary almost 9 years ago

  • Backport set to firefly
  • Regression set to No

Assuming firefly backport is intended because it was set to pending backport prior to hammer release. But cloud not find a trace of the corresponding fix anywhere.

#5 Updated by Loïc Dachary over 8 years ago

pinged joao, hoping to discover the commit to be backported

#6 Updated by Nathan Cutler over 8 years ago

This would appear to be it: https://github.com/ceph/ceph/pull/2560

#7 Updated by Loïc Dachary over 8 years ago

If that's indeed the pull request to be backported, I think it's going to be non trivial. Maybe users who have problems with this can upgrade instead ? We're late in the firefly lifecycle and such a backport is probably not adviseable. It modifies parts of the code base in which we should be super careful about regressions. The risk of a regression because of this patch versus the risk of an upgrade problem : I'm not sure who is the winner :-)

#8 Updated by Nathan Cutler over 8 years ago

  • Status changed from Pending Backport to Resolved
  • Backport deleted (firefly)

I tend to agree with Loic. Dropping the backport for now. If Joao disagrees, I trust he will revert my changes to this issue and the backport tracker issue #12833

Also available in: Atom PDF