Skip to content

Add network policies #1197

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
Open
wants to merge 70 commits into
base: main
Choose a base branch
from
Open

Add network policies #1197

wants to merge 70 commits into from

Conversation

MarekMichali
Copy link
Contributor

@MarekMichali MarekMichali commented Oct 1, 2025 •
edited
Loading

Description

Changes proposed in this pull request:

  • Add network policies
  • Add field networkPoliciesEnabled in btpoperator spec to enable or disable by the user the network policies
  • Add the module label to the sap-btp-service-operator pod
  • Add E2E tests
  • Add docs

Related issue(s)

See #1171

@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 1, 2025

Add one of following labels

- kind/feature -> Use it when you want to submit a new feature

- kind/enhancement -> Use it when you modify or improve an existing feature

- kind/bug -> Use it when you fix a bug

MarekMichali and others added 12 commits October 15, 2025 08:27
@MarekMichali @IwonaLanger
Update docs/contributor/04-10-workflows.md
9b9037c 
Co-authored-by: Iwona Langer <[email protected]>
@MarekMichali @IwonaLanger
Update docs/user/03-15-network-policies.md
9d7b9f3 
Co-authored-by: Iwona Langer <[email protected]>
@MarekMichali @IwonaLanger
Update docs/user/03-15-network-policies.md
6d035c8 
Co-authored-by: Iwona Langer <[email protected]>
@MarekMichali @IwonaLanger
Update docs/user/03-15-network-policies.md
d6cf259 
Co-authored-by: Iwona Langer <[email protected]>
@MarekMichali @IwonaLanger
Update docs/user/03-15-network-policies.md
6ebcc06 
Co-authored-by: Iwona Langer <[email protected]>
@MarekMichali @IwonaLanger
Update docs/user/03-15-network-policies.md
7496f56 
Co-authored-by: Iwona Langer <[email protected]>
@MarekMichali @IwonaLanger
Update docs/user/03-15-network-policies.md
a511dc2 
Co-authored-by: Iwona Langer <[email protected]>
@MarekMichali @IwonaLanger
Update docs/user/03-15-network-policies.md
da3e4e5 
Co-authored-by: Iwona Langer <[email protected]>
@MarekMichali @IwonaLanger
Update docs/user/03-15-network-policies.md
95f3562 
Co-authored-by: Iwona Langer <[email protected]>
@MarekMichali
Refactor networkpolicies controller
a0c5b7f
@MarekMichali
Add reconcile test and deprovisioning test
3e65753
@MarekMichali
Fix docs
c03dd08
IwonaLanger
IwonaLanger reviewed Oct 15, 2025
View reviewed changes
IwonaLanger
IwonaLanger previously approved these changes Oct 15, 2025
View reviewed changes
IwonaLanger
IwonaLanger previously approved these changes Oct 15, 2025
View reviewed changes
szwedm
szwedm previously approved these changes Oct 15, 2025
View reviewed changes
Copy link
Contributor

@szwedm szwedm left a comment

Choose a reason for hiding this comment

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

/lgtm

@MarekMichali MarekMichali dismissed stale reviews from szwedm and IwonaLanger via 72fb413 October 20, 2025 06:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@IwonaLanger IwonaLanger IwonaLanger approved these changes

@szwedm szwedm szwedm left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@MarekMichali @szwedm @IwonaLanger