Skip to content

Move third-party S3 and backend selection to Configure step - #3

Closed
TimothyAsirJeyasing wants to merge 2 commits into
rh-8574-mirrorpeer-pre-pair-validation-redesignfrom
agnostic-dr-wizard-configure-step
Closed

Move third-party S3 and backend selection to Configure step#3
TimothyAsirJeyasing wants to merge 2 commits into
rh-8574-mirrorpeer-pre-pair-validation-redesignfrom
agnostic-dr-wizard-configure-step

Conversation

@TimothyAsirJeyasing

Copy link
Copy Markdown
Owner

Stacked on the Submariner Create DRPolicy wizard PR:
red-hat-storage#2975

This change updates the Create DRPolicy wizard for Agnostic / multi-backend DR UX:
After you select two clusters, the wizard always continues to Configure.
Both clusters are Data Foundation (ODF): Configure shows Submariner / cluster-pair validation. You proceed once that looks good.
Third-party only (no usable ODF on the pair): Configure skips Submariner pairing UI and asks for S3 replication-site details (with the third-party storage warning).
Hybrid (mixed backends): Configure first shows Data Foundation vs Third-party tiles. Choosing DF continues with Submariner pre-pair; choosing third-party continues with the S3 form.

Validate ACM-managed and upstream Submariner before policy creation, surface Globalnet warnings, and align the Data Foundation flow with the four-step UX.

Signed-off-by: Timothy Asir Jeyasingh <tim.gluster@gmail.com>
Keep Clusters focused on pair selection and validation, and surface Data Foundation pre-pair, third-party S3, and hybrid backend tiles on Configure so the Agnostic DR wizard matches the multi-backend UX.

Also align wizard footer button order with other wizards, reuse GreenCheckCircleIcon, share isFilled, and use Submariner condition constants for failure messages.

Signed-off-by: Timothy Asir Jeyasingh <tim.gluster@gmail.com>
Repository owner deleted a comment from cursor Bot Jul 21, 2026
Repository owner deleted a comment from coderabbitai Bot Jul 21, 2026
@TimothyAsirJeyasing

Copy link
Copy Markdown
Owner Author
Screenshot 2026-07-20 at 11 47 05 PM Screenshot 2026-07-20 at 11 46 37 PM Screenshot 2026-07-20 at 11 46 03 PM Screenshot 2026-07-20 at 11 45 44 PM Screenshot 2026-07-20 at 11 45 25 PM

@TimothyAsirJeyasing
TimothyAsirJeyasing force-pushed the rh-8574-mirrorpeer-pre-pair-validation-redesign branch 2 times, most recently from 10ee53c to 12e8396 Compare July 22, 2026 13:10
@TimothyAsirJeyasing

Copy link
Copy Markdown
Owner Author

Superseded by #5 (rebased stack: no DF vs third-party choice tiles; Configure shows Submariner or S3 from auto-detected backend).

@TimothyAsirJeyasing

Copy link
Copy Markdown
Owner Author

Closing in favor of #5.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant