Project

General

Profile

Actions

Feature #12038

closed

make rgw_fronends more forgiving of whitespace

Added by Sage Weil almost 9 years ago. Updated about 8 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
-
Target version:
-
% Done:

0%

Source:
Development
Tags:
Backport:
hammer
Reviewed:
Affected Versions:
Pull request ID:

Related issues 1 (0 open1 closed)

Copied to Ceph - Backport #14493: hammer: make rgw_fronends more forgiving of whitespaceResolvedLoïc DacharyActions
Actions #1

Updated by Ken Dreyer almost 9 years ago

The original downstream report has a bit more info: https://bugzilla.redhat.com/show_bug.cgi?id=1228845

Actions #2

Updated by Michal Jarzabek almost 9 years ago

I can have a look

Actions #4

Updated by Matt Benjamin over 8 years ago

I've pushed branch wip-civetwarn to :linuxbox2/ceph.git
Yehuda intended to suggest some changes, I don't think he has yet. I've resubmitted it has upstream PR #6944

Actions #5

Updated by Yehuda Sadeh over 8 years ago

I did comment on the PR, or on the commit. But I can't find my comments now. Could it be that it got lost due to re-pushing the code?

Actions #6

Updated by Matt Benjamin over 8 years ago

I've updated PR with what I recall were your original verbal comments:
1. don't warn just because "port" missing
2. refer to upstream feature, not the BZ in the commit msg (and remove comment mentioning it from the code)

Actions #7

Updated by Ken Dreyer over 8 years ago

  • Status changed from New to Fix Under Review
  • Assignee set to Matt Benjamin
Actions #8

Updated by Ken Dreyer over 8 years ago

  • Backport set to hammer
Actions #9

Updated by Ken Dreyer over 8 years ago

  • Status changed from Fix Under Review to Pending Backport
Actions #10

Updated by Loïc Dachary about 8 years ago

  • Copied to Backport #14493: hammer: make rgw_fronends more forgiving of whitespace added
Actions #11

Updated by Loïc Dachary about 8 years ago

  • Status changed from Pending Backport to Resolved
Actions

Also available in: Atom PDF