Skip to content

Kill Process Runtime Policies Support #531

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

Jump to bottom
Merged
merged 4 commits into from
Jul 19, 2024
Merged

Kill Process Runtime Policies Support #531

merged 4 commits into from
Jul 19, 2024

Conversation

IgorEulalio
Copy link
Contributor

@IgorEulalio IgorEulalio commented Jul 10, 2024 •
edited
Loading

No description provided.

@IgorEulalio IgorEulalio requested a review from a user July 10, 2024 14:52
ravinadhruve10
ravinadhruve10 approved these changes Jul 16, 2024
View reviewed changes
Copy link
Contributor

@ravinadhruve10 ravinadhruve10 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change LGTM. Good to also have another pair of eyes from one of the resource owners.

ombellare
ombellare approved these changes Jul 16, 2024
View reviewed changes
Copy link
Contributor

@ombellare ombellare left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, removed the FR ticket number from pull request heading as per sysdiglabs convention

@ombellare ombellare changed the title Fr 2029 kill process runtime policies support Kill Process Runtime Policies Support Jul 16, 2024
@IgorEulalio IgorEulalio merged commit 6787bc6 into master Jul 19, 2024
@IgorEulalio IgorEulalio deleted the FR-2029-kill-process-runtime-policies-support branch July 19, 2024 00:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ombellare ombellare ombellare approved these changes

@ravinadhruve10 ravinadhruve10 ravinadhruve10 approved these changes

@kmvachhani kmvachhani Awaiting requested review from kmvachhani

@rosenbloomb-sysdig rosenbloomb-sysdig Awaiting requested review from rosenbloomb-sysdig rosenbloomb-sysdig is a code owner

@miguelgordo miguelgordo Awaiting requested review from miguelgordo

@ivanlysiuk-sysdig ivanlysiuk-sysdig Awaiting requested review from ivanlysiuk-sysdig ivanlysiuk-sysdig is a code owner

@daniel-almeida daniel-almeida Awaiting requested review from daniel-almeida daniel-almeida is a code owner

@jbainbridgesysdig jbainbridgesysdig Awaiting requested review from jbainbridgesysdig jbainbridgesysdig is a code owner

@hila-krut-sysdig hila-krut-sysdig Awaiting requested review from hila-krut-sysdig

@yaminSapir yaminSapir Awaiting requested review from yaminSapir

@chen-shmilovich-sysdig chen-shmilovich-sysdig Awaiting requested review from chen-shmilovich-sysdig

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@IgorEulalio @ombellare @ravinadhruve10