-
Notifications
You must be signed in to change notification settings - Fork 131
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
release-1.16 Pick require affinity for typha #1368
release-1.16 Pick require affinity for typha #1368
Conversation
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.
How did you pick these changes? I'm asking because of the addition in core_controller.go Line 457-464.
If you performed the pick by picking #1308 I think instead you should pick the 3 individual commits that were picked as part of #1308 instead of picking #1308.
I didn't see any other changes that were not changes in the original pick but I'm concerned I might have missed something.
759c692
to
12253b4
Compare
* Allow user to select affinity model for Typha * Regenerate autogenerated files * affinity set correct affinity based on config * Set RequiredDuring as a default for AKS, allow RequiredDuring to be selected * Unit test coverage * Revert temporary script change * Fix spelling * Remove protobuf, gen docs * Make AKS RequiredDuringSchedulingIgnoredDuringExecution a default value * Update comments, update typha affinity if clause for clarity * Typo fix
12253b4
to
eb744dc
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
/merge-when-ready delete-branch |
OK, I will merge the pull request when it's ready, leave the commits as is when I merge it, and delete the branch after I've merged it. |
I'm sorry but I failed delete the branch after merging the pull request. |
Description
pick from #1308 for release-v1.16
For PR author
make gen-files
make gen-versions
For PR reviewers
A note for code reviewers - all pull requests must have the following:
kind/bug
if this is a bugfix.kind/enhancement
if this is a a new feature.enterprise
if this PR applies to Calico Enterprise only.