Skip to content

Add introduction of Alpha feature EvenPodsSpread to 1.16#82868

Closed
saschagrunert wants to merge 1 commit intokubernetes:masterfrom
openSUSE:even-pods-spread
Closed

Add introduction of Alpha feature EvenPodsSpread to 1.16#82868
saschagrunert wants to merge 1 commit intokubernetes:masterfrom
openSUSE:even-pods-spread

Conversation

@saschagrunert
Copy link
Copy Markdown
Member

@saschagrunert saschagrunert commented Sep 19, 2019

What type of PR is this?

/kind documentation

What this PR does / why we need it:

Adds the missing release notes about EvenPodsSpread to the 1.16 changelog.

Which issue(s) this PR fixes:

Relates to kubernetes/sig-release#801

Special notes for your reviewer:

/cc @Huang-Wei

Does this PR introduce a user-facing change?:

NONE

@k8s-ci-robot k8s-ci-robot added release-note-none Denotes a PR that doesn't merit a release note. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. kind/documentation Categorizes issue or PR as related to documentation. needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. needs-priority Indicates a PR lacks a `priority/foo` label and requires one. labels Sep 19, 2019
@saschagrunert
Copy link
Copy Markdown
Member Author

saschagrunert commented Sep 19, 2019

/sig release

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. sig/release Categorizes an issue or PR as relevant to SIG Release. and removed needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. labels Sep 19, 2019
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

k8s-ci-robot commented Sep 19, 2019

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: saschagrunert
To complete the pull request process, please assign dchen1107
You can assign the PR to them by writing /assign @dchen1107 in a comment when ready.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@saschagrunert saschagrunert mentioned this pull request Sep 19, 2019
Merged
sftim
sftim reviewed Sep 19, 2019
View reviewed changes
Comment thread CHANGELOG-1.16.md Outdated
Copy link
Copy Markdown
Contributor

@sftim sftim Sep 19, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My suggestion for wording:

- Pod spread constraints have been added in alpha. You can use these constraints to control how Pods are spread across the cluster among failure-domains. ([#77327](https://github.com/kubernetes/kubernetes/pull/77327), [#77760](https://github.com/kubernetes/kubernetes/pull/77760), [#77828](https://github.com/kubernetes/kubernetes/pull/77828), [#79062](https://github.com/kubernetes/kubernetes/pull/79062), [#80011](https://github.com/kubernetes/kubernetes/pull/80011), [#81068](https://github.com/kubernetes/kubernetes/pull/81068), [@Huang-Wei](https://github.com/Huang-Wei))

(also made in kubernetes/website#16446)

Copy link
Copy Markdown
Member Author

@saschagrunert saschagrunert Sep 19, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changed as suggested 👍

@saschagrunert
Add introduction of Alpha feature EvenPodsSpread to 1.16
cc97308 
Signed-off-by: Sascha Grunert <sgrunert@suse.com>
@Huang-Wei
Copy link
Copy Markdown
Member

Huang-Wei commented Sep 20, 2019

/close
in favor of #82853

@k8s-ci-robot
Copy link
Copy Markdown
Contributor

k8s-ci-robot commented Sep 20, 2019

@Huang-Wei: Closed this PR.

Details

In response to this:

/close
in favor of #82853

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.

@saschagrunert saschagrunert deleted the even-pods-spread branch September 20, 2019 07:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Huang-Wei Huang-Wei Awaiting requested review from Huang-Wei

1 more reviewer

@sftim sftim sftim left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/documentation Categorizes issue or PR as related to documentation. needs-priority Indicates a PR lacks a `priority/foo` label and requires one. release-note-none Denotes a PR that doesn't merit a release note. sig/release Categorizes an issue or PR as relevant to SIG Release. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@saschagrunert @k8s-ci-robot @Huang-Wei @sftim