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

Jump to bottom

Added PodEvicted default event #1497

Merged
merged 17 commits into from
Jul 17, 2024
Merged

Added PodEvicted default event #1497

merged 17 commits into from
Jul 17, 2024

Conversation

itisallgood
Copy link
Contributor

  • Added on_pod_evicted_enricher action
  • Added default playbook for PodEvicted event
  • Added enrich_pod_with_node_events action

@itisallgood
* Added on_pod_evicted_enricher action
76b06ff 
* Added default playbook for PodEvicted event
* Added enrich_pod_with_node_events action
@itisallgood itisallgood requested a review from RoiGlinik July 11, 2024 10:35
@itisallgood itisallgood self-assigned this Jul 11, 2024
RoiGlinik
RoiGlinik reviewed Jul 11, 2024
View reviewed changes
helm/robusta/values.yaml Outdated Show resolved Hide resolved
@RoiGlinik
Copy link
Contributor

The PR looks good, just left 2 minor fixes and then we can merge this

arikalon1
arikalon1 reviewed Jul 15, 2024
View reviewed changes
helm/robusta/values.yaml Outdated Show resolved Hide resolved
@itisallgood itisallgood requested a review from RoiGlinik July 16, 2024 13:50
@itisallgood itisallgood requested a review from arikalon1 July 16, 2024 13:50
@RoiGlinik RoiGlinik merged commit b974f2d into master Jul 17, 2024
6 checks passed
@RoiGlinik RoiGlinik deleted the add-visibility-out-of-the-box-into-evictions branch July 17, 2024 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@RoiGlinik RoiGlinik RoiGlinik approved these changes

@arikalon1 arikalon1 Awaiting requested review from arikalon1

Assignees

@itisallgood itisallgood

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@itisallgood @RoiGlinik @arikalon1