Skip to content

Conversation

@bobfuru
Copy link
Contributor

@bobfuru bobfuru commented Jan 4, 2021

Follow up to #28034 and to this comment that brings OCP 4.5 docs up to date for:

  • CSI snapshot controller namespace, as reported in https://bugzilla.redhat.com/show_bug.cgi?id=1906288
  • Use of snapshot apiVersion snapshot.storage.k8s.io/v1beta1
  • Adding modules/persistent-storage-csi-drivers-supported.adoc using the drivers supported up to OCP 4.5

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

The preview will be available shortly at:

@bobfuru bobfuru force-pushed the OSDOCS-1720-bugs-4-5 branch 2 times, most recently from 8537a33 to 3387251 Compare January 4, 2021 22:00
@openshift-ci-robot openshift-ci-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Jan 4, 2021
@bobfuru bobfuru mentioned this pull request Jan 4, 2021
@bobfuru
Copy link
Contributor Author

bobfuru commented Jan 4, 2021

@duanwei33 PTAL for QE

@bobfuru bobfuru force-pushed the OSDOCS-1720-bugs-4-5 branch from 3387251 to c9ca12b Compare January 5, 2021 19:05
@bobfuru
Copy link
Contributor Author

bobfuru commented Jan 5, 2021

@duanwei33 - Removed Cinder, let me know if this looks good now, thanks!

@bobfuru
Copy link
Contributor Author

bobfuru commented Jan 5, 2021

FYI, I've updated this PR and the 4.65 PR (#28331) to conditionally reflect the additional bug fixes that were also introduced in 4.7 (#28034).

Choose a reason for hiding this comment

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

In OCP4.5, aws-ebs and manila csi driver was introduced with 3rd operator, and need user to install manually. Maybe we can have some update here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh, yeah...right. I've modified wording here for 4.5 only, PTAL

@bobfuru bobfuru force-pushed the OSDOCS-1720-bugs-4-5 branch from 28ae990 to 06341e4 Compare January 6, 2021 17:24
Comment on lines +8 to +12
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@duanwei33 - Updated wording here for 4.5 only based on the manual Operator install that is required.

@bobfuru bobfuru added the peer-review-needed Signifies that the peer review team needs to review this PR label Jan 7, 2021
@bobfuru bobfuru force-pushed the OSDOCS-1720-bugs-4-5 branch from 06341e4 to dd94b0f Compare January 7, 2021 20:43
@bobfuru
Copy link
Contributor Author

bobfuru commented Jan 7, 2021

@openshift/team-documentation PTAL

Copy link
Contributor

@sfortner-RH sfortner-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!

@bobfuru bobfuru 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 Jan 7, 2021
@bobfuru bobfuru merged commit 572752c into openshift:enterprise-4.5 Jan 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch/enterprise-4.5 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.

5 participants