Skip to content

WIP Prefer IPv6 hostIP on bootstrap IPv6 deployments#4788

Closed
osherdp wants to merge 1 commit intoopenshift:masterfrom
osherdp:osherdp/bz1931974
Closed

WIP Prefer IPv6 hostIP on bootstrap IPv6 deployments#4788
osherdp wants to merge 1 commit intoopenshift:masterfrom
osherdp:osherdp/bz1931974

Conversation

@osherdp
Copy link
Copy Markdown

@osherdp osherdp commented Mar 24, 2021

/hold
/cc @osherdp

@celebdor @osherdp
Prefer IPv6 hostIP on bootstrap IPv6 deployments
d756e6e 
Kubelet can end up choosing IPv4 addresses as its hostIP, which after:

https://github.com/openshift/cluster-kube-apiserver-operator/pull/1042/files

mean that the kubernetes API service ends up with an endpoint being an
IPv4 address.  This breaks the kubernetes API loadbalancer in the
service network and prevent the deployment from succeeding.
@openshift-ci-robot
Copy link
Copy Markdown
Contributor

openshift-ci-robot commented Mar 24, 2021

@osherdp: GitHub didn't allow me to request PR reviews from the following users: osherdp.

Note that only openshift members and repo collaborators can review this PR, and authors cannot review their own PRs.

Details

In response to this:

/hold
/cc @osherdp

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.

@openshift-ci-robot openshift-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. labels Mar 24, 2021
@openshift-ci-robot
Copy link
Copy Markdown
Contributor

openshift-ci-robot commented Mar 24, 2021

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

openshift-ci-robot commented Mar 24, 2021

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign staebler after the PR has been reviewed.
You can assign the PR to them by writing /assign @staebler 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

@staebler
Copy link
Copy Markdown
Contributor

staebler commented Mar 24, 2021

Is this PR any different than #4756?

@osherdp
Copy link
Copy Markdown
Author

osherdp commented Mar 24, 2021

Only rebased. Wanted to test something
Anyway, closing

@osherdp osherdp closed this Mar 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@osherdp @openshift-ci-robot @staebler @celebdor