-
Notifications
You must be signed in to change notification settings - Fork 41
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
signal: add lcow supported signals to windows signal map #98
Conversation
Signed-off-by: Kathryn Baldauf <kabaldau@microsoft.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@samuelkarp Does this PR need another sign off? Is there anything blocking merging it? |
Yes, generally two maintainer LGTMs. |
@thaJeztah could you take a look when you get the chance? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
sorry, I somehow missed this one 😅
This PR addresses the issues discussed in containerd/containerd#6002 to support linux signals for LCOW.
Signed-off-by: Kathryn Baldauf kabaldau@microsoft.com