Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RegionalDR VP docs #481

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

r2dedios
Copy link

Including docs and diagrams for the RegionalDR VP

Signed-off-by: Alejandro Villegas <[email protected]>
Copy link
Contributor

openshift-ci bot commented Sep 27, 2024

Hi @r2dedios. Thanks for your PR.

I'm waiting for a validatedpatterns member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

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.

@mbaldessari
Copy link
Contributor

This is an automated message:

You can preview this docs PR at http://481.docs-pr.validatedpatterns.io
Note that they get generated every five minutes, so please wait a bit.

@openshift-ci openshift-ci bot added the size/L label Sep 27, 2024
@r2dedios r2dedios changed the title First review RegionalDR VP docs RegionalDR VP docs Sep 27, 2024
@TomerFi
Copy link
Member

TomerFi commented Oct 8, 2024

This is an automated message:

You can preview this docs PR at http://481.docs-pr.validatedpatterns.io Note that they get generated every five minutes, so please wait a bit.

@mbaldessari @r2dedios
I'm sorry, but I haven't found time to review this yet. I want to start today. Do you think we can still use the preview site?

TomerFi and others added 3 commits October 9, 2024 18:12
Signed-off-by: Tomer Figenblat <[email protected]>
docs: update regional dr main doc
Signed-off-by: Alejandro Villegas <[email protected]>
@r2dedios r2dedios marked this pull request as ready for review October 11, 2024 11:13
@r2dedios
Copy link
Author

@mbaldessari Ready to review.

The "HTMLTest" actions is failing due to a redirect loop on "blog/2024-01-16-deploying-mcg-with-cisco-flashstack-portworx/index.html" Should we ignore it?

@mbaldessari
Copy link
Contributor

@mbaldessari Ready to review.

The "HTMLTest" actions is failing due to a redirect loop on "blog/2024-01-16-deploying-mcg-with-cisco-flashstack-portworx/index.html" Should we ignore it?

Yeah that was just a temporary fluke. You do need to drop the aliases: /agof/ though, that is wrong

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

Successfully merging this pull request may close these issues.

3 participants