-
Notifications
You must be signed in to change notification settings - Fork 58
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
COO-261: Support tolerations and nodeselector in Monitoringstack #540
COO-261: Support tolerations and nodeselector in Monitoringstack #540
Conversation
Hi @PaulFarver. Thanks for your PR. I'm waiting for a rhobs 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-sigs/prow repository. |
/ok-to-test |
@PaulFarver: This pull request references COO-261 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 task to target the "4.17.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. |
Thanks for the PR! Commit messages need to start with |
ccacd18
to
4173d6d
Compare
@simonpasquier thanks for looking at this :) I've updated the commit messages, but I'm unsure what to do about the generated code. When I run |
Yeah, the code generation can be tricky sometimes. Can you try running |
You'd need also to amend the commit messages to remove the upper-case after the |
4173d6d
to
b5efd50
Compare
Yeah. It should be updated now |
/lgtm Thanks! |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: PaulFarver, simonpasquier 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 |
I pushed onto your branch the updated version of |
/lgtm |
rhobs#540) * feat: support tolerations and nodeselectors * chore: generate resources * chore: update bundle --------- Co-authored-by: Simon Pasquier <spasquie@redhat.com>
…bs#540) * feat: support tolerations and nodeselectors * chore: generate resources * chore: update bundle --------- Co-authored-by: Simon Pasquier <spasquie@redhat.com>
Add support for tolerations and nodeselectors in the monitoringstack CRD.