Skip to content

Conversation

@maxwelldb
Copy link
Contributor

This PR resolves #23711.

QE should be covered already by #21488 and #22878.

This change addresses openshift-docs issue 23711
openshift#23711
@maxwelldb maxwelldb added this to the Next Release milestone Jul 13, 2020
@maxwelldb maxwelldb requested a review from luis5tb July 13, 2020 13:32
@maxwelldb maxwelldb self-assigned this Jul 13, 2020
@openshift-ci-robot openshift-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Jul 13, 2020
@maxwelldb
Copy link
Contributor Author

Let me know if you have any concerns, @luis5tb.

@openshift-docs-preview-bot

The preview will be available shortly at:

@luis5tb
Copy link
Contributor

luis5tb commented Jul 13, 2020

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jul 13, 2020
@maxwelldb
Copy link
Contributor Author

@openshift/team-documentation PTAL. S review: +1 and -11.

@maxwelldb maxwelldb added the peer-review-needed Signifies that the peer review team needs to review this PR label Jul 13, 2020
@maxwelldb maxwelldb changed the title Remove outdated Kuryr + Octavia compat notes Remove outdated Kuryr + Octavia compatibility notes Jul 16, 2020
for DNS resolution if:

* The {rh-openstack} version is earlier than 16
* The OVN Octavia driver is used
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This bullet no longer applies--OVN Octavia support is improved in 4.5+.

same port. Services that expose the same port to different protocols, like TCP
and UDP, are not supported.

[IMPORTANT]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This admonition no longer applies--OVN Octavia support is improved in 4.5+.

Copy link
Contributor

@adellape adellape left a comment

Choose a reason for hiding this comment

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

👍

@maxwelldb
Copy link
Contributor Author

@adellape Thanks!

@adellape adellape 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 Jul 16, 2020
@maxwelldb maxwelldb merged commit 10f9a95 into openshift:master Jul 16, 2020
@maxwelldb
Copy link
Contributor Author

/cherry-pick enterprise-4.5

@maxwelldb
Copy link
Contributor Author

/cherry-pick enterprise-4.6

@openshift-cherrypick-robot

@maxwelldb: new pull request created: #23878

Details

In response to this:

/cherry-pick enterprise-4.5

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-cherrypick-robot

@maxwelldb: new pull request created: #23879

Details

In response to this:

/cherry-pick enterprise-4.6

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch/enterprise-4.5 branch/enterprise-4.6 lgtm Indicates that a PR is ready to be merged. peer-review-done Signifies that the peer review team has reviewed this PR size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[enterprise-4.5] Update Kuryr and Octavia compatibility statements in OSP installation docs

6 participants