-
Notifications
You must be signed in to change notification settings - Fork 28
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
fix(agent): should respect taint-toleration #68
fix(agent): should respect taint-toleration #68
Conversation
eb310ac
to
aa95566
Compare
The Harvester backend needs to adapt this change, tracked in harvester/harvester#3702. |
4710297
to
fa3517d
Compare
Ref: 57, Longhorn-5614 Signed-off-by: Chin-Ya Huang <chin-ya.huang@suse.com>
Ref: 57, Longhorn-5614 Signed-off-by: Chin-Ya Huang <chin-ya.huang@suse.com>
Ref: 57, Longhorn-5614 Signed-off-by: Chin-Ya Huang <chin-ya.huang@suse.com>
Signed-off-by: Chin-Ya Huang <chin-ya.huang@suse.com>
fa3517d
to
ff87893
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.
LGTM
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
Operator: corev1.TolerationOpExists, | ||
}, | ||
}, | ||
Tolerations: a.sbm.getTaintToleration(), |
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.
Just a note: This will change the current behavior if no taint tolerations are specified since it was to tolerate all taints.
#57
longhorn/longhorn#5614