Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cloud_storage: downgrade DeleteObject 404 message to debug #8906

Merged
merged 1 commit into from
Feb 16, 2023

Conversation

jcsp
Copy link
Contributor

@jcsp jcsp commented Feb 15, 2023

This message is frequent on GCS backends: INFO is too verbose, and the message does not indicate an issue that users should pay attention to.

This is a bespoke change for v22.3.x, because on dev the code has already been revised in a way that changed this along with more invasive changes.

Backports Required

  • none - not a bug fix
  • none - issue does not exist in previous branches
  • none - papercut/not impactful enough to backport
  • v22.3.x
  • v22.2.x
  • v22.1.x

UX Changes

None

Release Notes

Bug Fixes

  • Harmless "Unexpected NoSuchKey error" log message is downgraded from INFO to DEBUG severity. This message appears frequently on object stores that do not implement the S3 DeleteObject verb in a way that matches the AWS interface, such as GCS.

This message is frequent on GCS backends: INFO is too verbose,
and the message does not indicate an issue that users should pay
attention to.

This is a bespoke change for v22.3.x, because on `dev` the code
has already been revised in a way that changed this along with
more invasive changes.
@jcsp jcsp requested a review from Lazin February 15, 2023 16:57
@jcsp jcsp marked this pull request as ready for review February 16, 2023 09:39
@jcsp
Copy link
Contributor Author

jcsp commented Feb 16, 2023

Test failures:

@jcsp jcsp merged commit 1a1b0b0 into redpanda-data:v22.3.x Feb 16, 2023
@jcsp jcsp deleted the cloud-storage-404-log branch February 16, 2023 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants