-
Notifications
You must be signed in to change notification settings - Fork 827
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
roles: remove stale prow binding cleanup
- Loading branch information
Showing
1 changed file
with
1 addition
and
6 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -113,12 +113,7 @@ for prj; do | |
|
||
color 6 "Ensure stale role bindings have been removed from e2e project: ${prj}" | ||
( | ||
# TODO(https://github.com/kubernetes/k8s.io/issues/1661): remove once verified as consistent | ||
color 6 "group:[email protected] should not have roles/viewer (ref: https://github.com/kubernetes/k8s.io/issues/1661)" | ||
ensure_removed_project_role_binding "${prj}" "group:[email protected]" "roles/viewer" | ||
# TODO(https://github.com/kubernetes/k8s.io/issues/299): remove once smarter logic folded into ensure_project | ||
color 6 "user:* should not have roles/owner for projects (ref: https://github.com/kubernetes/k8s.io/issues/299)" | ||
ensure_removed_project_role_binding "${prj}" "[email protected]" "roles/owner" | ||
echo "no stale bindings slated for removal" | ||
) 2>&1 | indent | ||
|
||
color 6 "Enabling APIs necessary for kubernetes e2e jobs to use e2e project: ${prj}" | ||
|