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.
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
Source admin role should be granted to KF service accounts. #965
Conversation
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ankushagarwal The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest |
* Source admin permission was granted to cloud services account but that service account is only used by deployment manager and shouldn't need source admin. * We want the Kubeflow service accounts to have access to source admin so that we can push the ksonnet app to source control and so that users can access their code in pods.
/test all |
@ankushagarwal Could you re-lgtm please? I needed to sync to get tests to pass. |
/lgtm |
…#965) * Source admin permission was granted to cloud services account but that service account is only used by deployment manager and shouldn't need source admin. * We want the Kubeflow service accounts to have access to source admin so that we can push the ksonnet app to source control and so that users can access their code in pods.
Source admin permission was granted to cloud services account but
that service account is only used by deployment manager and shouldn't
need source admin.
We want the Kubeflow service accounts to have access to source admin
so that we can push the ksonnet app to source control and so that users
can access their code in pods.
/assign @ankushagarwal
This change is