Skip to content

Remove overly verbose event ReasonEnforcementGloballyDisabled in allowexternaltraffic.Off mode & improve event messages#574

Merged
zohar7ch merged 2 commits intomainfrom
orisho/remove_unnecessary_event_and_improve_other_events
Apr 1, 2025
Merged

Remove overly verbose event ReasonEnforcementGloballyDisabled in allowexternaltraffic.Off mode & improve event messages#574
zohar7ch merged 2 commits intomainfrom
orisho/remove_unnecessary_event_and_improve_other_events

Conversation

@orishoshan
Copy link
Copy Markdown
Collaborator

@orishoshan orishoshan commented Mar 16, 2025

Description

This PR removes an overly verbose event with reason ReasonEnforcementGloballyDisabled, reported when external traffic policy support is disabled.
The PR also fixes the formatting of the events (to start with capital case, to meed conventions).

Testing

  • This change adds test coverage for new/changed/fixed functionality

Checklist

  • I have added documentation for new/changed functionality in this PR and in github.com/otterize/docs

@orishoshan orishoshan requested a review from zohar7ch March 16, 2025 14:05
@zohar7ch zohar7ch marked this pull request as ready for review April 1, 2025 08:35
@zohar7ch zohar7ch force-pushed the orisho/remove_unnecessary_event_and_improve_other_events branch from 5eaf0fa to 5d3e98b Compare April 1, 2025 08:35
@zohar7ch zohar7ch merged commit df77a4d into main Apr 1, 2025
21 checks passed
@zohar7ch zohar7ch deleted the orisho/remove_unnecessary_event_and_improve_other_events branch April 1, 2025 13:18
@github-actions github-actions bot locked and limited conversation to collaborators Apr 1, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants