Project

General

Profile

Actions

Backport #64861

closed

squid: Notification kafka: Persistent messages are removed even when the broker is down and no ack is received

Added by Backport Bot about 2 months ago. Updated about 2 months ago.

Status:
Rejected
Priority:
Normal
Target version:
-
Release:
squid
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Related issues 1 (1 open0 closed)

Copied from rgw - Bug #63335: Notification kafka: Persistent messages are removed even when the broker is down and no ack is receivedPending BackportYuval Lifshitz

Actions
Actions #1

Updated by Backport Bot about 2 months ago

  • Copied from Bug #63335: Notification kafka: Persistent messages are removed even when the broker is down and no ack is received added
Actions #2

Updated by Krunal Chheda about 2 months ago

Why do we need backport for squid, this PR was part of main before the squid was cut

Actions #3

Updated by Yuval Lifshitz about 2 months ago

  • Status changed from New to Rejected

Krunal Chheda wrote:

Why do we need backport for squid, this PR was part of main before the squid was cut

you are right, it is already in squid.

Actions #4

Updated by Krunal Chheda about 2 months ago

@yuval, is the tag v19.0.0 for squid and is this the squid code base (https://github.com/ceph/ceph/blob/v19.0.0/src/rgw/rgw_kafka.cc), if yes then the fix is not part of squid and we may need to backport it.

Actions #5

Updated by Yuval Lifshitz about 2 months ago

Krunal Chheda wrote:

@yuval, is the tag v19.0.0 for squid and is this the squid code base (https://github.com/ceph/ceph/blob/v19.0.0/src/rgw/rgw_kafka.cc), if yes then the fix is not part of squid and we may need to backport it.

i ran "git log" on squid, and saw the commit. so, I think we are good

Actions

Also available in: Atom PDF