Skip to content
This repository has been archived by the owner on Feb 14, 2023. It is now read-only.

Add a draft implementation of Route Integrity #59

Closed
wants to merge 3 commits into from

Conversation

mike1808
Copy link
Contributor

@mike1808 mike1808 commented Aug 4, 2020

Route Integrity Implementation

Refer to proposal doc for more information.

Implementation

It's a simple mutating admission controller which watches for pods in cf-workloads namespace and modifies the serviceAccountName field and creates the service account if it doesn't exist.

Links

Co-authored-by: Nitya Dhanushkodi ndhanushkodi@pivotal.io

* add a Mutating Webhook Admission Controller to watch for pods and
add/update service account

Co-authored-by: Nitya Dhanushkodi <ndhanushkodi@pivotal.io>
@cf-gitbot
Copy link
Collaborator

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

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

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

ndhanushkodi and others added 2 commits August 4, 2020 22:57
Co-authored-by: Mikael Manukyan <mmanukyan@vmware.com>
* SA now will be deleted when StatefulSet is deleted

Co-authored-by: Nitya Dhanushkodi <ndhanushkodi@pivotal.io>
@mike1808 mike1808 added the wontfix This will not be worked on label Aug 10, 2020
@kauana kauana closed this Aug 27, 2020
@kauana
Copy link

kauana commented Aug 27, 2020

We are closing the PR but keeping the route-integrity-draft branch as discussed here.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
wontfix This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants