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
Tolerations & Resources Override using .Subscription.Config #1334
Tolerations & Resources Override using .Subscription.Config #1334
Conversation
Hi @umangachapagain. Thanks for your PR. I'm waiting for a operator-framework member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. 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/test-infra repository. |
/ok-to-test |
8430397
to
54eaf6d
Compare
/retest |
2 similar comments
/retest |
/retest |
Updated deployment initializer function to inject tolerations and resources specified in pod configuration of subscription into deployment object. - Toleration will be appended if it does not exist - Resources will be overwritten Signed-off-by: Umanga Chapagain <chapagainumanga@gmail.com>
includes tests for tolerations and resources override Signed-off-by: Umanga Chapagain <chapagainumanga@gmail.com>
54eaf6d
to
f71501a
Compare
/retest |
added examples for Tolerations and Resources spec in Subscription config. Signed-off-by: Umanga Chapagain <chapagainumanga@gmail.com>
/retest |
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.
/lgtm
This looks great, thank you for including clear docs and tests.
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ecordell, umangachapagain 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 Please review the full test history for this PR and help us cut down flakes. |
2 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
18 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/cherry-pick release-4.4 |
@awgreene: #1334 failed to apply on top of branch "release-4.4":
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 kubernetes/test-infra repository. |
@umangachapagain was this feature intentionally planned to delete the resources section from the operator deployment even though the related config section is not defined in the subscription? Based on the code changes and based on my tests if the resources config is not defined in the subscription but it was given in the CSV for the operator then it will be removed from the deployment during installation. We define the resource needs of an operator on the CSV level because in this way you can define it differently for the separate versions. Is this behaviour conceptual? Or just a bug? |
It's a bug. The intention was to override resources spec, but not blindly. Please report a bug with your findings. |
Thanks! |
Description of the change:
(feat) inject 'Tolerations' & 'Resources' from config
Updated deployment initializer function to inject tolerations
and resources specified in pod configuration of subscription
into deployment object.
(test) update e2e test
Motivation for the change:
#1298
Reviewer Checklist
/docs