Project

General

Profile

Actions

Bug #40183

closed

Save an unnecessary copy of RGWEnv

Added by Mark Kogan almost 5 years ago. Updated about 3 years ago.

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

0%

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

Description

In src/rgw/rgw_rest.h : compute_domain_uri()

auto env = *(s->info.env);

makes a copy of RGWEnv.


Related issues 3 (0 open3 closed)

Copied to rgw - Backport #40759: luminous: Save an unnecessary copy of RGWEnvRejectedActions
Copied to rgw - Backport #40760: nautilus: Save an unnecessary copy of RGWEnvResolvedNathan CutlerActions
Copied to rgw - Backport #40761: mimic: Save an unnecessary copy of RGWEnvResolvedPrashant DActions
Actions #1

Updated by Casey Bodley almost 5 years ago

  • Status changed from In Progress to Fix Under Review
  • Assignee set to Mark Kogan
  • Pull request ID set to 28426
Actions #2

Updated by Casey Bodley almost 5 years ago

  • Backport set to luminous mimic nautilus
Actions #3

Updated by Casey Bodley almost 5 years ago

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

Updated by Nathan Cutler almost 5 years ago

  • Copied to Backport #40759: luminous: Save an unnecessary copy of RGWEnv added
Actions #5

Updated by Nathan Cutler almost 5 years ago

  • Copied to Backport #40760: nautilus: Save an unnecessary copy of RGWEnv added
Actions #6

Updated by Nathan Cutler almost 5 years ago

  • Copied to Backport #40761: mimic: Save an unnecessary copy of RGWEnv added
Actions #7

Updated by Nathan Cutler almost 5 years ago

  • Project changed from Ceph to rgw
Actions #8

Updated by Nathan Cutler about 3 years ago

  • Status changed from Pending Backport to Resolved

While running with --resolve-parent, the script "backport-create-issue" noticed that all backports of this issue are in status "Resolved" or "Rejected".

Actions

Also available in: Atom PDF