-
Notifications
You must be signed in to change notification settings - Fork 71
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
filepromoter: Warn on (but don't block) unauthenticated operations #417
filepromoter: Warn on (but don't block) unauthenticated operations #417
Conversation
This is a partial revert of a previous commit which enforced authentication for operations on destination filestores. Without this, presubmits will fail because they're running in an untrusted environment (without access to the prod file promotion svc acct). Signed-off-by: Stephen Augustus <[email protected]>
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: justaugustus 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 |
Signed-off-by: Stephen Augustus <[email protected]>
78062c1
to
32c3c59
Compare
/assign @puerco @ameukam @Verolop @cpanato @saschagrunert |
/lgtm |
What type of PR is this?
/kind bug
What this PR does / why we need it:
(Part of kubernetes/k8s.io#2624, #413.)
This is a partial revert of a previous commit (kpromo: Improve service account usage/retrieval #412) which enforced
authentication for operations on destination filestores. Without
this, presubmits will fail because they're running in an untrusted
environment (without access to the prod file promotion svc acct).
Signed-off-by: Stephen Augustus [email protected]
Which issue(s) this PR fixes:
Special notes for your reviewer:
Example presubmit failure from kubernetes/k8s.io#2704 (also visible in kubernetes/k8s.io#2663):
With this PR, we should see logs similar to this in presubmit:
Does this PR introduce a user-facing change?