Skip to content

BYO RHEL docs need to be updated to include RHEL 7.9 support#28703

Merged
mburke5678 merged 1 commit intoopenshift:masterfrom
mburke5678:BZ-1918082
Jan 27, 2021
Merged

BYO RHEL docs need to be updated to include RHEL 7.9 support#28703
mburke5678 merged 1 commit intoopenshift:masterfrom
mburke5678:BZ-1918082

Conversation

@mburke5678
Copy link
Copy Markdown
Contributor

@mburke5678 mburke5678 commented Jan 20, 2021 •
edited
Loading

@mburke5678 mburke5678 added this to the Next Release milestone Jan 20, 2021
@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Jan 20, 2021
@openshift-docs-preview-bot
Copy link
Copy Markdown

openshift-docs-preview-bot commented Jan 20, 2021

The preview will be available shortly at:

@mburke5678
Copy link
Copy Markdown
Contributor Author

mburke5678 commented Jan 20, 2021

@sdodson PTAL

@sdodson
Copy link
Copy Markdown
Member

sdodson commented Jan 20, 2021

/lgtm
Did a quick search and couldn't find any other references to these version ranges so this seems to be good. Thanks.

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jan 20, 2021
@sdodson
Copy link
Copy Markdown
Member

sdodson commented Jan 26, 2021

/lgtm cancel
@katherinedube pointed out other references to 7.6 where as the linked solution mentions 7.7 and later in 4.5+, lets update that.
https://github.com/openshift/openshift-docs/blame/master/modules/installation-requirements-user-infra.adoc#L135

@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Jan 26, 2021
@openshift-ci-robot openshift-ci-robot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Jan 26, 2021
@mburke5678
Copy link
Copy Markdown
Contributor Author

mburke5678 commented Jan 26, 2021

@sdodson @katherinedube
Updated the doc referenced in the BZ to RHEL 7.7 and later.

@openshift-ci-robot openshift-ci-robot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Jan 26, 2021
@mburke5678
Copy link
Copy Markdown
Contributor Author

mburke5678 commented Jan 26, 2021

@sdodson @katherinedube Updated the supported versions for 4.6 and 4.7 (impending) to RHEL 7.8-7.9. PTAL

cc. @sferich888 @codyhoag

[1] https://github.com/openshift/openshift-docs/pull/28703/files#diff-2f3534bfedc94d5ef24b4845354ac2e1cf0ad53bf4bab0f5764acb82bfa9fb17R19

@mburke5678
Copy link
Copy Markdown
Contributor Author

mburke5678 commented Jan 26, 2021

Changes to 4.5 breanch: #28853

@sdodson
Copy link
Copy Markdown
Member

sdodson commented Jan 26, 2021

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jan 26, 2021
@mburke5678
Copy link
Copy Markdown
Contributor Author

mburke5678 commented Jan 26, 2021

@gpei Can you please take a look at this PR and my PR for 4.5: #28853

@gpei
Copy link
Copy Markdown

gpei commented Jan 27, 2021

LGTM, RHEL-7.8 and RHEL-7.9 are covered in QE's testing in OCP-4.6 and OCP-4.7

bergerhoffer
bergerhoffer reviewed Jan 27, 2021
View reviewed changes
Copy link
Copy Markdown
Contributor

@bergerhoffer bergerhoffer left a comment

Choose a reason for hiding this comment

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

LGTM!

@bergerhoffer bergerhoffer added peer-review-needed Signifies that the peer review team needs to review this PR 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 Jan 27, 2021
@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Jan 27, 2021

New changes are detected. LGTM label has been removed.

@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Jan 27, 2021
@netlify
Copy link
Copy Markdown

netlify Bot commented Jan 27, 2021

Deploy preview for osdocs ready!

Built with commit 325d52a

https://deploy-preview-28703--osdocs.netlify.app

@mburke5678 mburke5678 mentioned this pull request Jan 27, 2021
Merged
@mburke5678 mburke5678 merged commit 6feedf0 into openshift:master Jan 27, 2021
@mburke5678 mburke5678 deleted the BZ-1918082 branch January 27, 2021 15:30
@mburke5678
Copy link
Copy Markdown
Contributor Author

mburke5678 commented Jan 27, 2021 •
edited by openshift-ci-robot
Loading

/cherrypick enterprise-4.6

@openshift-cherrypick-robot openshift-cherrypick-robot mentioned this pull request Jan 27, 2021
Merged
@openshift-cherrypick-robot
Copy link
Copy Markdown

openshift-cherrypick-robot commented Jan 27, 2021 •
edited by openshift-ci-robot
Loading

@mburke5678: new pull request created: #28879

Details

In response to this:

/cherrypick 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.

@mburke5678
Copy link
Copy Markdown
Contributor Author

mburke5678 commented Jan 27, 2021 •
edited by openshift-ci-robot
Loading

/cherrypick enterprise-4.7

@openshift-cherrypick-robot
Copy link
Copy Markdown

openshift-cherrypick-robot commented Jan 27, 2021 •
edited by openshift-ci-robot
Loading

@mburke5678: new pull request created: #28880

Details

In response to this:

/cherrypick enterprise-4.7

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 openshift-cherrypick-robot mentioned this pull request Jan 27, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bergerhoffer bergerhoffer bergerhoffer left review comments

Assignees

@sdodson sdodson

Labels

branch/enterprise-4.6 branch/enterprise-4.7 peer-review-done Signifies that the peer review team has reviewed this PR size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Milestone

Next Release

Development

Successfully merging this pull request may close these issues.

7 participants

@mburke5678 @openshift-docs-preview-bot @sdodson @gpei @openshift-ci-robot @openshift-cherrypick-robot @bergerhoffer