-
Notifications
You must be signed in to change notification settings - Fork 213
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
AGENT-951: Add new security defn for ABI #6784
base: master
Are you sure you want to change the base?
AGENT-951: Add new security defn for ABI #6784
Conversation
Skipping CI for Draft Pull Request. |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: pawanpinjarkar The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@pawanpinjarkar: This pull request references AGENT-951 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.18.0" version, but no target version was set. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
0a96eb3
to
07ae8c0
Compare
/cc @carbonin |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #6784 +/- ##
==========================================
- Coverage 68.74% 68.74% -0.01%
==========================================
Files 249 249
Lines 37439 37448 +9
==========================================
+ Hits 25739 25743 +4
- Misses 9402 9407 +5
Partials 2298 2298
|
@pawanpinjarkar: The following test failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
The agent-based installer needs a new security definition to perform the read-only operations for the user persona
watcher
. The agent-based installer commands such aswait-for
andmonitor
internally uses the read-only personawatcher
which can only make read requests to the endpoints annotated withwatcherAuth
security definition.Other existing authenticators do not support this agent-based installer specific
watcherAuth
security definition.List all the issues related to this PR
What environments does this code impact?
How was this code tested?
Checklist
docs
, README, etc)Reviewers Checklist