Project

General

Profile

Actions

Backport #43659

closed

nautilus: metadata is missing in bucket deletion notifications

Added by Nathan Cutler about 4 years ago. Updated almost 4 years ago.

Status:
Resolved
Priority:
Normal
Target version:
Release:
nautilus
Pull request ID:
Crash signature (v1):
Crash signature (v2):


Related issues 1 (0 open1 closed)

Copied from rgw - Bug #43196: metadata is missing in bucket deletion notificationsResolvedYuval Lifshitz

Actions
Actions #1

Updated by Nathan Cutler about 4 years ago

  • Copied from Bug #43196: metadata is missing in bucket deletion notifications added
Actions #2

Updated by Nathan Cutler about 4 years ago

  • Status changed from New to Need More Info
  • Assignee set to Yuval Lifshitz

appears to be a major feature, risk of regression potentially higher than usual

Actions #3

Updated by Nathan Cutler about 4 years ago

  • Assignee changed from Yuval Lifshitz to Casey Bodley

assigning to lead for a judgment call

Actions #4

Updated by Casey Bodley about 4 years ago

from the pr title:

This also fixes an existing issue with metadata info not appearing in deletion requests. Tracker Issue: https://tracker.ceph.com/issues/43196

Yuval, how much of https://github.com/ceph/ceph/pull/31878 would be necessary to backport to fix this issue?

Actions #5

Updated by Yuval Lifshitz about 4 years ago

Casey Bodley wrote:

from the pr title:

This also fixes an existing issue with metadata info not appearing in deletion requests. Tracker Issue: https://tracker.ceph.com/issues/43196

Yuval, how much of https://github.com/ceph/ceph/pull/31878 would be necessary to backport to fix this issue?

The PR contains the fix as well as adding support for tags in notifications and notification filters. Adding object tags support is a small but useful feature, so, IMO, the entire PR could be backported to nautilus.

Actions #6

Updated by Casey Bodley about 4 years ago

  • Assignee changed from Casey Bodley to Yuval Lifshitz

Yuval Lifshitz wrote:

The PR contains the fix as well as adding support for tags in notifications and notification filters. Adding object tags support is a small but useful feature, so, IMO, the entire PR could be backported to nautilus.

Thanks Yuval. We are supporting the bucket notifications feature on nautilus, so backporting fixes and other missing pieces is consistent with that. Would you be willing to prepare the backport, since we're likely to do one downstream anyway?

Actions #7

Updated by Yuval Lifshitz about 4 years ago

Casey Bodley wrote:

Yuval Lifshitz wrote:

The PR contains the fix as well as adding support for tags in notifications and notification filters. Adding object tags support is a small but useful feature, so, IMO, the entire PR could be backported to nautilus.

Thanks Yuval. We are supporting the bucket notifications feature on nautilus, so backporting fixes and other missing pieces is consistent with that. Would you be willing to prepare the backport, since we're likely to do one downstream anyway?

sure. I can do that, however, backporting my PR is blocked by this PR: https://github.com/ceph/ceph/pull/30020

Actions #8

Updated by Nathan Cutler about 4 years ago

sure. I can do that, however, backporting my PR is blocked by this PR: https://github.com/ceph/ceph/pull/30020

correction: blocked by https://github.com/ceph/ceph/pull/32924 (https://github.com/ceph/ceph/pull/30020 was closed)

Actions #9

Updated by Nathan Cutler almost 4 years ago

  • Description updated (diff)
  • Status changed from Need More Info to Resolved
  • Target version set to v14.2.10

This update was made using the script "backport-resolve-issue".
backport PR https://github.com/ceph/ceph/pull/34107
merge commit 3ad2dfa48ba6ff3b4e8104c0184b9daa42d7bc26 (v14.2.9-453-g3ad2dfa48ba)

Actions

Also available in: Atom PDF