Skip to content
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

BZ1586368 Remove references to setting openshift_dns_ip #9875

Closed
wants to merge 1 commit into from

Conversation

sdodson
Copy link
Member

@sdodson sdodson commented Jun 6, 2018

The node config value dnsIP is no longer configurable in 3.10.

/cc @sjenning

The node config value dnsIP is no longer configurable.
@openshift-ci-robot openshift-ci-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jun 6, 2018
@sdodson sdodson requested a review from kalexand-rh June 6, 2018 12:46
Copy link
Contributor

@kalexand-rh kalexand-rh left a comment

Choose a reason for hiding this comment

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

LGTM!

@kalexand-rh
Copy link
Contributor

@sjenning, will you PTAL?

@sjenning
Copy link

LGTM thanks

@kalexand-rh
Copy link
Contributor

@jianlinliu, since you approved https://bugzilla.redhat.com/show_bug.cgi?id=1586368, do you agree with this doc change?

@openshift/team-documentation, PTAL

. If the `*dnsIP*` parameter is omitted from the *_node-config.yaml_*
file, then the value defaults to the kubernetes service IP, which is the first
nameserver in the pod's *_/etc/resolv.conf_* file.
. {product-title} then sets the pod's first namesever to the IP address of
Copy link
Contributor

Choose a reason for hiding this comment

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

s/namesever/nameserver

@kalexand-rh kalexand-rh added the peer-review-needed Signifies that the peer review team needs to review this PR label Jun 19, 2018
@jianlinliu
Copy link

LGTM

@openshift-bot
Copy link

@sdodson: PR needs rebase.

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-bot openshift-bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jun 23, 2018
@kalexand-rh
Copy link
Contributor

@sdodson, will you please rebase?

@openshift/team-documentation, PTAL

@bfallonf
Copy link

LGTM (after typo fix) 🐛

@kalexand-rh kalexand-rh added peer-review-done Signifies that the peer review team has reviewed this PR and removed peer-review-needed Signifies that the peer review team needs to review this PR labels Jun 28, 2018
@adellape
Copy link
Contributor

Rebased and edited via #10537. Closing here.

@adellape adellape closed this Jun 29, 2018
@vikram-redhat vikram-redhat removed this from the Future Release milestone Jul 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
branch/enterprise-3.10 needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. peer-review-done Signifies that the peer review team has reviewed this PR size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants