-
Notifications
You must be signed in to change notification settings - Fork 95
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
✨ set agent install namespace from addon deployment config for template type addons #266
✨ set agent install namespace from addon deployment config for template type addons #266
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #266 +/- ##
==========================================
+ Coverage 60.73% 60.91% +0.18%
==========================================
Files 128 128
Lines 13685 13682 -3
==========================================
+ Hits 8311 8335 +24
+ Misses 4608 4584 -24
+ Partials 766 763 -3
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
8b0e30c
to
ad824d3
Compare
62e90d9
to
4377185
Compare
/hold |
a372af7
to
e9a4a6c
Compare
… type addons Signed-off-by: zhujian <jiazhu@redhat.com>
e9a4a6c
to
c524ca7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/approve
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: qiujian16, zhujian7 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 |
/unhold |
Summary
set agent install namespace from addon deployment config for template type addons, so for template type addons, users can use the addonDeploymentConfig API to configure the agent install namespace.
Related issue(s)
Fixes #