Skip to content

Conversation

@anarnold97
Copy link
Contributor

@anarnold97 anarnold97 commented Aug 16, 2024

Cherry pick to 4.13

Jira

  • OADP-4657 - [DOC] Release Notes for OADP 1.4.0 for OCP 4.13
    • Release notes added to 4.13 to address the ACM issue. However, when @adellape was reviewing, there were so many corrections, rather than make them for just 4.13, I created this PR for the sake of uniformity. See this PR for further details. This created the need for the cherry pick

Version(s):

  • OCP 4.13 +

Link to docs preview:

QE review:

@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Aug 16, 2024
@openshift-ci-robot
Copy link

openshift-ci-robot commented Aug 16, 2024

@anarnold97: This pull request references OADP-4657 which is a valid jira issue.

In response to this:

Cherry pick to 4.13

Jira

  • OADP-4657 - [DOC] Release Notes for OADP 1.4.0 for OCP 4.13
    • Release notes added to 4.13 to address the ACM issue. However, when @adellape was reviewing, there were so many corrections, rather than make them for just 4.13, I created this PR for the sake of uniformity.

See this PR for further details.

Version(s):

  • OCP 4.13 +

Link to docs preview:

QE review:

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 openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci openshift-ci bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Aug 16, 2024
@ocpdocs-previewbot
Copy link

ocpdocs-previewbot commented Aug 16, 2024

@openshift-ci-robot
Copy link

openshift-ci-robot commented Aug 16, 2024

@anarnold97: This pull request references OADP-4657 which is a valid jira issue.

In response to this:

Cherry pick to 4.13

Jira

  • OADP-4657 - [DOC] Release Notes for OADP 1.4.0 for OCP 4.13
    • Release notes added to 4.13 to address the ACM issue. However, when @adellape was reviewing, there were so many corrections, rather than make them for just 4.13, I created this PR for the sake of uniformity. See this PR for further details. This created the need for the cherry pick

Version(s):

  • OCP 4.13 +

Link to docs preview:

QE review:

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 openshift-eng/jira-lifecycle-plugin repository.

@anarnold97
Copy link
Contributor Author

/label oadp

@openshift-ci openshift-ci bot added the OADP Label for all OADP PRs label Aug 16, 2024
@anarnold97
Copy link
Contributor Author

/label peer-review-needed

@openshift-ci openshift-ci bot added the peer-review-needed Signifies that the peer review team needs to review this PR label Aug 16, 2024
@agantony
Copy link
Contributor

/label peer-review-in-progress

@openshift-ci openshift-ci bot added the peer-review-in-progress Signifies that the peer review team is reviewing this PR label Aug 19, 2024
@anarnold97 anarnold97 force-pushed the 4.13-OADP-1-4-release-notes-update-4.13-review branch from 7d206ed to 7fbac1e Compare August 19, 2024 11:10
@anarnold97
Copy link
Contributor Author

Updated to include topic map

Signed-off-by: A.Arnold <anarnold@redhat.com>
@anarnold97 anarnold97 force-pushed the 4.13-OADP-1-4-release-notes-update-4.13-review branch from 6d8aa3a to 682666a Compare August 19, 2024 11:24
@openshift-ci
Copy link

openshift-ci bot commented Aug 19, 2024

@anarnold97: all tests passed!

Full PR test history. Your PR dashboard.

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-sigs/prow repository. I understand the commands that are listed here.

Copy link
Contributor

@agantony agantony left a comment

Choose a reason for hiding this comment

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

Added a few suggestions for your conisderation; overall lgtm!

* xref:../../../operators/admin/olm-upgrading-operators.adoc#olm-changing-update-channel_olm-upgrading-operators[Updating installed Operators]

[id="oadp-converting-dpa-to-new-version-1-4-0_{context}"]
=== Converting DPA to the new version
Copy link
Contributor

Choose a reason for hiding this comment

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

Upgrading DPA ....?

[id="oadp-converting-dpa-to-new-version-1-4-0_{context}"]
=== Converting DPA to the new version

To upgrade from OADP 1.3 to 1.4, no Data Protection Application (DPA) changes are required.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
To upgrade from OADP 1.3 to 1.4, no Data Protection Application (DPA) changes are required.
To upgrade from OADP 1.3 to 1.4, you do not need to make any changes to the Data Protection Application (DPA).

Copy link
Contributor

Choose a reason for hiding this comment

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

To keep the customer action center here, adding the suggested verbiage is needed: https://redhat-documentation.github.io/supplementary-style-guide/#_principle_1_customer_focus_and_action_orientation

[id="verifying-upgrade-1-4-0_{context}"]
=== Verifying the upgrade

Verify the installation by following steps from the xref:../../../backup_and_restore/application_backup_and_restore/release-notes/oadp-release-notes-1-2.adoc#verifying-upgrade-1-2-0_oadp-release-notes[Verifying the upgrade] section.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Verify the installation by following steps from the xref:../../../backup_and_restore/application_backup_and_restore/release-notes/oadp-release-notes-1-2.adoc#verifying-upgrade-1-2-0_oadp-release-notes[Verifying the upgrade] section.
To verify the installation, complete the steps in the xref:../../../backup_and_restore/application_backup_and_restore/release-notes/oadp-release-notes-1-2.adoc#verifying-upgrade-1-2-0_oadp-release-notes[Verifying the upgrade] section.

:_template-generated: 2024-06-28
// Module included in the following assemblies:
//
// * backup_and_restore/oadp-1-4-release-notes.adoc
Copy link
Contributor

Choose a reason for hiding this comment

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

Update the correct relative path. Applicable to all modules.

Copy link
Contributor

Choose a reason for hiding this comment

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

This seems like a required change to me

[id="oadp-backing-up-dpa-configuration-1-4-0_{context}"]
= Backing up the DPA configuration

You must back up your current `DataProtectionApplication` (DPA) configuration.
Copy link
Contributor

Choose a reason for hiding this comment

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

Would be nice if we can highlight why we are doing this?

Copy link
Contributor

Choose a reason for hiding this comment

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

Data Protection Application (DPA) ??

Copy link
Contributor

Choose a reason for hiding this comment

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

Is the imperative here maybe something like, "Before upgrading, you must back up your current DataProtectionApplication (DPA) configuration."


* The `velero-plugin-for-csi` code is now available in the Velero code, which means an `init` container is no longer required for the plugin.

* Velero changed client Burst and QPS defaults from 30 and 20 to 100 and 100, respectively.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
* Velero changed client Burst and QPS defaults from 30 and 20 to 100 and 100, respectively.
* Velero has changed the default values for client burst and QPS from 30 and 20 to 100 and 100 respectively.


* Velero changed client Burst and QPS defaults from 30 and 20 to 100 and 100, respectively.

* The `velero-plugin-for-aws` plugin updated default value of the `spec.config.checksumAlgorithm` field in `BackupStorageLocation` objects (BSLs) from `""` (no checksum calculation) to the `CRC32` algorithm. For more information, see link:https://github.com/vmware-tanzu/velero-plugin-for-aws/blob/release-1.10/backupstoragelocation.md[Velero plugins for AWS Backup Storage Location]. The checksum algorithm types are known to work only with AWS.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
* The `velero-plugin-for-aws` plugin updated default value of the `spec.config.checksumAlgorithm` field in `BackupStorageLocation` objects (BSLs) from `""` (no checksum calculation) to the `CRC32` algorithm. For more information, see link:https://github.com/vmware-tanzu/velero-plugin-for-aws/blob/release-1.10/backupstoragelocation.md[Velero plugins for AWS Backup Storage Location]. The checksum algorithm types are known to work only with AWS.
* The `velero-plugin-for-aws` plugin has updated the default value of the `spec.config.checksumAlgorithm` field in `BackupStorageLocation` objects (BSLs) from `""` (no checksum calculation) to the `CRC32` algorithm.
+
For more information, see link:https://github.com/vmware-tanzu/velero-plugin-for-aws/blob/release-1.10/backupstoragelocation.md[Velero plugins for AWS Backup Storage Location].
+
[NOTE]
====
The checksum algorithms only work with AWS.
====

Copy link
Contributor

Choose a reason for hiding this comment

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

Expand AWS??

Copy link
Contributor

Choose a reason for hiding this comment

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

It looks like this is the first mention of AWS on the page, just by a cursory glance. If that's the case then IBMSG recommends spelling out first mention: https://www.ibm.com/docs/en/ibm-style?topic=references-product-service-names#service-names

Comment on lines +64 to +66
Several S3 providers require the `md5sum` to be disabled by setting the checksum algorithm to `""`. Confirm `md5sum` algorithm support and configuration with your storage provider.
+
In OADP 1.4, the default value for BSLs created within DPA for this configuration is `""`. This default value means that the `md5sum` is not checked, which is consistent with OADP 1.3. For BSLs created within DPA, update it by using the `spec.backupLocations[].velero.config.checksumAlgorithm` field in the DPA. If your BSLs are created outside DPA, you can update this configuration by using `spec.config.checksumAlgorithm` in the BSLs.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Several S3 providers require the `md5sum` to be disabled by setting the checksum algorithm to `""`. Confirm `md5sum` algorithm support and configuration with your storage provider.
+
In OADP 1.4, the default value for BSLs created within DPA for this configuration is `""`. This default value means that the `md5sum` is not checked, which is consistent with OADP 1.3. For BSLs created within DPA, update it by using the `spec.backupLocations[].velero.config.checksumAlgorithm` field in the DPA. If your BSLs are created outside DPA, you can update this configuration by using `spec.config.checksumAlgorithm` in the BSLs.
For some S3 providers, you must disable the `md5sum`. To disable the `md5sum`, you must set the checksum algorithm to `""`. Confirm the `md5sum` support and configuration of the `md5sum` algorithm with your storage provider.
+
In OADP 1.4, the default value for BSLs created within DPA for this configuration is `""`. This default value means that the `md5sum` is not checked, which is consistent with OADP 1.3. For BSLs created within DPA, update it by using the `spec.backupLocations[].velero.config.checksumAlgorithm` field in DPA. If your BSLs are created outside of DPA, you can upgrade this configuration by using `spec.config.checksumAlgorithm` in the BSLs.

include::modules/oadp-upgrading-oadp-operator-1-4-0.adoc[leveloffset=+3]
[role="_additional-resources"]
.Additional resources
* xref:../../../operators/admin/olm-upgrading-operators.adoc#olm-changing-update-channel_olm-upgrading-operators[Updating installed Operators]
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
* xref:../../../operators/admin/olm-upgrading-operators.adoc#olm-changing-update-channel_olm-upgrading-operators[Updating installed Operators]
* xref:../../../operators/admin/olm-upgrading-operators.adoc#olm-changing-update-channel_olm-upgrading-operators[Changing the update channel for an Operator]

Comment on lines +52 to +53
[id="changes-oadp-1-3-to-1-4_{context}"]
=== Changes from OADP 1.3 to 1.4
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we make this a concept module?

@agantony
Copy link
Contributor

/remove-label peer-review-in-progress
/remove-label peer-review-needed
/label peer-review-done

@openshift-ci openshift-ci bot added peer-review-done Signifies that the peer review team has reviewed this PR and removed peer-review-in-progress Signifies that the peer review team is reviewing this PR peer-review-needed Signifies that the peer review team needs to review this PR labels Aug 19, 2024
@anarnold97
Copy link
Contributor Author

/label merge-review-needed

@openshift-ci openshift-ci bot added the merge-review-needed Signifies that the merge review team needs to review this PR label Aug 19, 2024
@skrthomas skrthomas added merge-review-in-progress Signifies that the merge review team is reviewing this PR branch/enterprise-4.13 and removed merge-review-needed Signifies that the merge review team needs to review this PR merge-review-in-progress Signifies that the merge review team is reviewing this PR labels Aug 19, 2024
@skrthomas
Copy link
Contributor

@anarnold97 I notice there are many of @agantony 's peer review comments that still need to be addressed. If you choose not to accept, please write a response at least so that merge reviewers will know you've seen and interpreted the feedback. Once you've made your commit with the peer-review feedback incorporated, please add the merge-review-needed label back.

[id="changes-oadp-1-3-to-1-4_{context}"]
=== Changes from OADP 1.3 to 1.4

The Velero server has been updated from version 1.12 to 1.14. Note that there are no changes in the Data Protection Application (DPA).
Copy link
Contributor

Choose a reason for hiding this comment

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

image

IBMSG: To upgrade is to replace software with a new release or fix level of the same program or to replace hardware with a newer model or more powerful technology. An upgrade involves little work on the user’s part.


The Velero server has been updated from version 1.12 to 1.14. Note that there are no changes in the Data Protection Application (DPA).

This changes the following:
Copy link
Contributor

Choose a reason for hiding this comment

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

I agree, being more specific that the update changes the following items is clearer.


* Velero changed client Burst and QPS defaults from 30 and 20 to 100 and 100, respectively.

* The `velero-plugin-for-aws` plugin updated default value of the `spec.config.checksumAlgorithm` field in `BackupStorageLocation` objects (BSLs) from `""` (no checksum calculation) to the `CRC32` algorithm. For more information, see link:https://github.com/vmware-tanzu/velero-plugin-for-aws/blob/release-1.10/backupstoragelocation.md[Velero plugins for AWS Backup Storage Location]. The checksum algorithm types are known to work only with AWS.
Copy link
Contributor

Choose a reason for hiding this comment

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


* Velero changed client Burst and QPS defaults from 30 and 20 to 100 and 100, respectively.

* The `velero-plugin-for-aws` plugin updated default value of the `spec.config.checksumAlgorithm` field in `BackupStorageLocation` objects (BSLs) from `""` (no checksum calculation) to the `CRC32` algorithm. For more information, see link:https://github.com/vmware-tanzu/velero-plugin-for-aws/blob/release-1.10/backupstoragelocation.md[Velero plugins for AWS Backup Storage Location]. The checksum algorithm types are known to work only with AWS.
Copy link
Contributor

Choose a reason for hiding this comment

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

It looks like this is the first mention of AWS on the page, just by a cursory glance. If that's the case then IBMSG recommends spelling out first mention: https://www.ibm.com/docs/en/ibm-style?topic=references-product-service-names#service-names

@anarnold97 anarnold97 closed this Aug 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch/enterprise-4.13 jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. OADP Label for all OADP PRs peer-review-done Signifies that the peer review team has reviewed this PR size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants