Skip to content

Prevent issue report on read_object_md if the object doesn't exist #8347

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

Conversation

tangledbytes
Copy link
Member

Explain the changes

This PR prevents issue report when an object doesn't exist in a target bucket. It is quite common and OK for clients to issue HeadObject against an object that doesn't exist in the bucket. This should not be counted as an issue with the namespace store. Counting it as an issue leads to it being marked as REJECTED in k8s.

This situation gets even stranger when we account for the fact that our namespace_monitor does take into account such scenarios which leads to NamespaceStore continuously switching between OPTIMAL and REJECTED.

And it gets worse when NooBaa internal replication engine is the entity issuing HeadObject. In such scenarios it isn't clear what is causing the NS to get into REJECTED state at all as the customer might not be even touching the bucket by themselves.

Issues: Fixed #xxx / Gap #xxx

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=2277298

  • Doc added/updated
  • Tests added

@tangledbytes tangledbytes force-pushed the utkarsh/fix/incorrect-namespace-store-issue-report branch from b7fc59e to 5cc06aa Compare September 12, 2024 08:34
Signed-off-by: Utkarsh Srivastava <[email protected]>

add todo comment to be able to handle GLACIER objects

Signed-off-by: Utkarsh Srivastava <[email protected]>
@tangledbytes tangledbytes force-pushed the utkarsh/fix/incorrect-namespace-store-issue-report branch from 5cc06aa to ea8d44c Compare September 12, 2024 09:04
@tangledbytes tangledbytes merged commit 25bbf89 into noobaa:master Sep 12, 2024
10 checks passed
@tangledbytes
Copy link
Member Author

tangledbytes commented Sep 12, 2024

Tested entire flow manually on K8s. Merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants