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.

Sign up for GitHub

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

Prevent leaking file descriptor during snapshotting and provide better logging of errors #19093

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Prevent leaking file descriptor during snapshotting and provide bette…

a1d5008
Select commit
Loading
Failed to load commit list.
Open

Prevent leaking file descriptor during snapshotting and provide better logging of errors #19093

Prevent leaking file descriptor during snapshotting and provide bette…
a1d5008
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Dec 20, 2024 in 0s

47.61% of diff hit (within 80.00% threshold of 68.85%)

View this Pull Request on Codecov

47.61% of diff hit (within 80.00% threshold of 68.85%)

Annotations

Check warning on line 61 in client/v3/snapshot/v3_snapshot.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

client/v3/snapshot/v3_snapshot.go#L61 

Added line #L61 was not covered by tests

Check warning on line 69 in client/v3/snapshot/v3_snapshot.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

client/v3/snapshot/v3_snapshot.go#L69 

Added line #L69 was not covered by tests

Check warning on line 80 in client/v3/snapshot/v3_snapshot.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

client/v3/snapshot/v3_snapshot.go#L80 

Added line #L80 was not covered by tests

Check warning on line 93 in client/v3/snapshot/v3_snapshot.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

client/v3/snapshot/v3_snapshot.go#L93 

Added line #L93 was not covered by tests

Check warning on line 100 in client/v3/snapshot/v3_snapshot.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

client/v3/snapshot/v3_snapshot.go#L100 

Added line #L100 was not covered by tests

Check warning on line 106 in client/v3/snapshot/v3_snapshot.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

client/v3/snapshot/v3_snapshot.go#L106 

Added line #L106 was not covered by tests

Check warning on line 109 in client/v3/snapshot/v3_snapshot.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

client/v3/snapshot/v3_snapshot.go#L109 

Added line #L109 was not covered by tests
View more details on Codecov