cloudfoundry / cf-for-k8s

The open source deployment manifest for Cloud Foundry on Kubernetes
Apache License 2.0
300 stars 115 forks source link

Rename allow-istio-control-plane network policy #475

Closed dolfolife closed 4 years ago

dolfolife commented 4 years ago

174810340

Co-authored-by: Rodolfo Sanchez srodolfo@vmware.com

Thanks for contributing to cf-for-k8s!

We've designed this PR template to speed up the PR review and merge process - please use it.

WHAT is this change about?

Rename a network-policy to better describe the scope of it

Does this PR introduce a change to config/values.yml?

no

Acceptance Steps

When I have a cluster with cf-for-k8s 
And I get the network policies in all namespaces `kubectl get netpol -A  | grep allow-istio-control-plane`
Then I can see network policies called only-allow-istio-control-plane.
And I do not see network policies called allow-istio-control-plane 

Tag your pair, your PM, and/or team

@cloudfoundry/cf-for-k8s-networking

cf-gitbot commented 4 years ago

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/175021318

The labels on this github issue will be updated when the story is started.

cf-rel-int-status-bot commented 4 years ago

Hello friend, it looks like your pull request has failed one or more of our checks. Please take a look! :eyes:

cf-rel-int-status-bot commented 4 years ago

Hello friend, it looks like your pull request has failed one or more of our checks. Please take a look! :eyes:

cf-rel-int-status-bot commented 4 years ago

Hello friend, it looks like your pull request has failed one or more of our checks. Please take a look! :eyes: