Skip to content

Conversation

@zohar7ch
Copy link
Contributor

@zohar7ch zohar7ch commented Jun 5, 2025

Description

Use the port defined in the webhook when creating network-policy to allow traffic to webhook service.

Testing

Describe how this can be tested by reviewers. Be specific about anything not tested and reasons why. If this library has unit and/or integration testing, tests should be added for new functionality and existing tests should complete without errors.

Please include any manual steps for testing end-to-end or functionality not covered by unit/integration tests.

Also include details of the environment this PR was developed in (language/platform/browser version).

  • 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

@zohar7ch zohar7ch requested a review from omris94 June 5, 2025 10:15
@zohar7ch zohar7ch force-pushed the zohar7ch/auto-allow-webhook-determine-ports branch from a3c86d5 to 752a939 Compare June 5, 2025 13:31
zohar7ch added 6 commits June 5, 2025 16:41
Use the port defined in the webhook when creating network-policy to
allow traffic to webhook service.
Since this feature might unintentionally block non-webhook traffic to the
same workload as the webhook server, we are introducing it behind a separate
configuration.
This ensures that upgrading to a new version does not disrupt existing
communication.
@zohar7ch zohar7ch force-pushed the zohar7ch/auto-allow-webhook-determine-ports branch from 752a939 to 4c151c2 Compare June 5, 2025 13:42
@zohar7ch zohar7ch merged commit c4265b6 into main Jun 5, 2025
22 checks passed
@zohar7ch zohar7ch deleted the zohar7ch/auto-allow-webhook-determine-ports branch June 5, 2025 14:16
@github-actions github-actions bot locked and limited conversation to collaborators Jun 5, 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.

3 participants