Bug 2215828

Cloned this bug from the below BZ for RHCS 5, in order to add the same changes to the Block Device Guide in RHCS 6 documentation as well.

+++ This bug was initially created as a clone of Bug #2019951 +++

Describe the issue:
block-device guide - add information that image resync pulls image afresh

Chapter/Section Number and Title:
5.7.8. Image resynchronization
5.8.7.1. Fail back to the primary storage cluster

Product Version:
RHCS 5
Environment Details:

Any other versions of this document that also needs this update:
RHCS 4
RHCS 3
RHCS 2
(Not sure whether we are still backporting changes to 2,3, needs to be done if we are )

Additional information:
https://www.mail-archive.com/ceph-users@lists.ceph.com/msg53451.html

--- Additional comment from Vasishta on 2021-11-03 17:11:00 UTC ---

Document URL:
Sorry the chapter numbers in Description is of 4.x guide

https://access.redhat.com/documentation/en-us/red_hat_ceph_storage/5/html/block_device_guide/mirroring-ceph-block-devices#image-resynchronization_block
https://access.redhat.com/documentation/en-us/red_hat_ceph_storage/5/html/block_device_guide/mirroring-ceph-block-devices#fail-back-to-the-primary-storage-cluster_block

--- Additional comment from Vasishta on 2021-11-03 17:15:46 UTC ---

On Wed, Nov 3, 2021 at 11:17 AM Deepika Upadhyay <dupadhya> wrote:

    Yes, Indeed Vashishta! I don't think so we have it worked out yet. 
    I think we do have a trello for that though: https://trello.com/c/lj37Glr2/440-rbd-mirror-improve-sync-via-only-updating-out-of-sync-objects 

    On Tue, Nov 2, 2021 at 10:54 PM Vasishta Shastry <vashastr> wrote:

        Hi Deepika,

        Thanks for trackers.
        Just to summarise, resync (both journal based and snapshot based) still recreates the image (fetch complete image from primary)

--- Additional comment from Vasishta on 2021-11-18 13:03:11 UTC ---

Hi Ranjini,

Can we please add another small information -
After the steps where we provide information on resync, we can mention that snapshot schedule added at image level would be removed as resync creates image afresh.
More info can be found in - https://bugzilla.redhat.com/show_bug.cgi?id=2020618

Regards,
Vasishta

--- Additional comment from Fionn Kelleher on 2021-12-09 18:03:14 UTC ---

Hi Vasishta,

The requested changes have been made to the RHCS 4 and RHCS 5 guides. The relevant MRs and preview links are below for your technical review.

RHCS 5
------
MR: https://gitlab.cee.redhat.com/red-hat-ceph-storage-documentation/doc-Red_Hat_Ceph_Storage_5-Block_Device_Guide/-/merge_requests/75

Preview:
  * "Image resynchronization"
    https://jenkins.dxp.redhat.com/job/CCS/job/ccs-mr-preview/14721/artifact/preview/index.html#image-resynchronization_block
  * "Fail back to the primary storage cluster"
    https://jenkins.dxp.redhat.com/job/CCS/job/ccs-mr-preview/14721/artifact/preview/index.html#fail-back-to-the-primary-storage-cluster_block


RHCS 4
------
MR: https://gitlab.cee.redhat.com/red-hat-ceph-storage-documentation/doc-Red_Hat_Ceph_Storage_4-Block_Device_Guide/-/merge_requests/86

Preview:
  * "Image resynchronization"
    https://jenkins.dxp.redhat.com/job/CCS/job/ccs-mr-preview/14783/artifact/preview/index.html#image-resynchronization_block
  * "Fail back to the primary storage cluster"
    https://jenkins.dxp.redhat.com/job/CCS/job/ccs-mr-preview/14783/artifact/preview/index.html#fail-back-to-the-primary-storage-cluster_block

Kind regards,
Fionn

--- Additional comment from Fionn Kelleher on 2021-12-09 18:17:45 UTC ---

Apologies for erroneously requesting a NEEDINFO from you, Vasishta!
Veera, please see the last message for the proposed fix and let me know if anything should be changed.

Regards,
Fionn

--- Additional comment from Veera Raghava Reddy on 2021-12-10 05:18:54 UTC ---

This is fine. You can move to ON_QA and QE will verify the updates.

--- Additional comment from Sunny Kumar on 2021-12-13 15:18:18 UTC ---

Hi Fionn,

Looks good to me.

Thanks.

--- Additional comment from Mary Frances Hull on 2021-12-13 17:02:47 UTC ---

Hi Fionn,

Nice work on this! I've left my comments in the merge request. 

Let me know if you have any questions.


Mary Frances

--- Additional comment from Fionn Kelleher on 2022-01-12 13:20:57 UTC ---

Hi Mary Frances,

I committed most of the changes requested; seeking a second review of the MRs as I left some comments there also.

Thanks for the thorough review!

Fionn

--- Additional comment from Mary Frances Hull on 2022-01-12 13:29:19 UTC ---

Hi Fionn,

Is there a different MR? I only see commits from December in the above MR links and only one commit for the RHCS 5 guide.

Thanks!

Mary Frances

--- Additional comment from Fionn Kelleher on 2022-01-12 13:59:05 UTC ---

Hi Mary Frances,

You're right there! I'd be grateful if you could take a look at my RHCS 4 commits from December and
double check you're happy with the changes made.

My intent was to update the RHCS 5 guide MR following this review, as the requested changes spanned
the two MRs. Let me know if that makes sense for you, or if you'd prefer I can push the changes to
RHCS 5's guide now too.

Fionn

--- Additional comment from Mary Frances Hull on 2022-01-12 14:21:36 UTC ---

Hi Fionn,

Can you add them both here for review at one time? 

Thanks!

Mary Frances

--- Additional comment from Red Hat Bugzilla on 2022-12-01 00:24:36 UTC ---

remove performed by PnT Account Manager <pnt-expunge>

--- Additional comment from Red Hat Bugzilla on 2022-12-31 19:24:20 UTC ---

remove performed by PnT Account Manager <pnt-expunge>

--- Additional comment from Red Hat Bugzilla on 2022-12-31 19:50:29 UTC ---

remove performed by PnT Account Manager <pnt-expunge>

--- Additional comment from Red Hat Bugzilla on 2022-12-31 19:51:02 UTC ---

remove performed by PnT Account Manager <pnt-expunge>

--- Additional comment from Red Hat Bugzilla on 2023-01-01 06:03:27 UTC ---

remove performed by PnT Account Manager <pnt-expunge>

--- Additional comment from Red Hat Bugzilla on 2023-01-01 08:29:34 UTC ---

Account disabled by LDAP Audit

--- Additional comment from Red Hat Bugzilla on 2023-01-01 08:45:19 UTC ---

Account disabled by LDAP Audit

--- Additional comment from Red Hat Bugzilla on 2023-01-01 08:49:05 UTC ---

Account disabled by LDAP Audit

--- Additional comment from Vasishta on 2023-03-08 17:32:34 UTC ---

Since 5.3z1 is GA, moving bz to next available release
Please change based on the requirement.

--- Additional comment from Rivka Pollack on 2023-06-04 10:50:28 UTC ---

I see that there is an old MR with changes in it.
https://gitlab.cee.redhat.com/red-hat-ceph-storage-documentation/doc-Red_Hat_Ceph_Storage_5-Block_Device_Guide/-/merge_requests/75/diffs
Does this MR contain all needed changes for this ticket? 
I am not sure why it wasn't merged as this was before I was on the writing team, but I'd like to assess getting this in for 5.3z4 commits.
Thanks

--- Additional comment from Manasa on 2023-06-05 06:44:41 UTC ---

The changes look okay to me at a first glance, you could please move the bug to ON_QA. I will verify it.