Skip to content

Remove overly verbose ShouldNotCreatePolicy when linkerd is enabled but enforcement is disabled#582

Merged
amitlicht merged 1 commit intomainfrom
amitlicht/remove_reason_shouldnotcreatepolicy
Apr 1, 2025
Merged

Remove overly verbose ShouldNotCreatePolicy when linkerd is enabled but enforcement is disabled#582
amitlicht merged 1 commit intomainfrom
amitlicht/remove_reason_shouldnotcreatepolicy

Conversation

@amitlicht
Copy link
Copy Markdown
Contributor

Description

This PR removes an overly verbose warning-level event with reason ShouldNotCreatePolicy, reported when linkerd support is enabled but enforcement is disabled. This is a valid situation and no warning events should be omitted in that case.

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

@amitlicht amitlicht marked this pull request as ready for review April 1, 2025 08:27
@amitlicht amitlicht requested a review from zohar7ch April 1, 2025 08:27
@amitlicht amitlicht enabled auto-merge (squash) April 1, 2025 08:29
@amitlicht amitlicht merged commit e76ea21 into main Apr 1, 2025
21 checks passed
@amitlicht amitlicht deleted the amitlicht/remove_reason_shouldnotcreatepolicy branch April 1, 2025 08:46
@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