-
Notifications
You must be signed in to change notification settings - Fork 29
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
fix(azure): close the reader after reading #186
Conversation
12cc0b7
to
af4371e
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #186 +/- ##
=========================================
- Coverage 7.88% 7.87% -0.01%
=========================================
Files 23 23
Lines 2005 2006 +1
=========================================
Hits 158 158
- Misses 1821 1822 +1
Partials 26 26
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Can you create a bug ticket for the fix? |
af4371e
to
38bffea
Compare
Add issue number/link here. Thank you. |
Also add ticket number in the commit message |
ref: longhorn/longhorn 9281 Signed-off-by: James Lu <james.lu@suse.com>
38bffea
to
b2687f0
Compare
Which issue(s) this PR fixes:
Issue # longhorn/longhorn#9281
What this PR does / why we need it:
Close the reader after reading.
Special notes for your reviewer:
Additional documentation or context