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

CA-153279: don't complain if there's no tapdisk to signal #145

Closed
wants to merge 1 commit into from
Closed

CA-153279: don't complain if there's no tapdisk to signal #145

wants to merge 1 commit into from

Conversation

tmakatos
Copy link
Contributor

@tmakatos tmakatos commented Dec 2, 2014

When rotating tapdisk logs we blindly tell all tapdisks (even if no
tapdisk exists) to refresh their logs. If no tapdisk exists, an error
message will be logged in /root/dead.letter, which might lead to dom0
root file-system free space implications.

Signed-off-by: Thanos Makatos thanos.makatos@citrix.com

When rotating tapdisk logs we blindly tell all tapdisks (even if no
tapdisk exists) to refresh their logs. If no tapdisk exists, an error
message will be logged in /root/dead.letter, which might lead to dom0
root file-system free space implications.

Signed-off-by: Thanos Makatos <thanos.makatos@citrix.com>
@germanop germanop closed this in c4a722c Sep 23, 2015
martinjorge pushed a commit to martinjorge/blktap that referenced this pull request Sep 23, 2015
When rotating tapdisk logs we blindly tell all tapdisks (even if no
tapdisk exists) to refresh their logs. If no tapdisk exists, an error
message will be logged in /root/dead.letter, which might lead to dom0
root file-system free space implications.

Signed-off-by: Thanos Makatos <thanos.makatos@citrix.com>
Reviewed-by: Chandrika Srinivasan <chandrika.srinivasan@citrix.com>

GitHub: closes xapi-project#145
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.

None yet

2 participants