Skip to content
This repository was archived by the owner on May 3, 2024. It is now read-only.

doc: fix pdclust link at CORTX-MOTR-ARCHITECTURE.md #1563

Merged
merged 3 commits into from
Aug 29, 2022
Merged

doc: fix pdclust link at CORTX-MOTR-ARCHITECTURE.md #1563

merged 3 commits into from
Aug 29, 2022

Conversation

@johnbent
Copy link
Contributor

@johnbent johnbent commented Mar 24, 2022 •
edited by andriytk
Loading

Fix the link to parity declustered design document.

View rendered doc/CORTX-MOTR-ARCHITECTURE.md

@johnbent
Update CORTX-MOTR-ARCHITECTURE.md
1feb833 
Adding link to parity declustered design document. Why is it in a documentation branch though instead of in main?

Signed-off-by: John Bent <[email protected]>
@cla-bot cla-bot bot added the cla-signed label Mar 24, 2022
andriytk
andriytk reviewed Mar 24, 2022
View reviewed changes
nikitadanilov
nikitadanilov suggested changes Mar 25, 2022
View reviewed changes
@stale
Copy link

stale bot commented Mar 31, 2022

This issue/pull request has been marked as needs attention as it has been left pending without new activity for 4 days. Tagging @nkommuri @mehjoshi @huanghua78 for appropriate assignment. Sorry for the delay & Thank you for contributing to CORTX. We will get back to you as soon as possible.

@stale stale bot removed the needs-attention label Apr 18, 2022
@stale
Copy link

stale bot commented Apr 23, 2022

This issue/pull request has been marked as needs attention as it has been left pending without new activity for 4 days. Tagging @nkommuri @mehjoshi @huanghua78 for appropriate assignment. Sorry for the delay & Thank you for contributing to CORTX. We will get back to you as soon as possible.

@hessio hessio added rose Status: Waiting to be Reviewed PR is waiting for reviewers to review the PR labels May 26, 2022
@stale stale bot removed the needs-attention label May 26, 2022
@stale
Copy link

stale bot commented May 31, 2022

This issue/pull request has been marked as needs attention as it has been left pending without new activity for 4 days. Tagging @nkommuri @mehjoshi @huanghua78 for appropriate assignment. Sorry for the delay & Thank you for contributing to CORTX. We will get back to you as soon as possible.

@cortx-admin
Copy link

cortx-admin commented Jul 28, 2022

Can one of the admins verify this patch?

@stale stale bot removed the needs-attention label Jul 28, 2022
@stale
Copy link

stale bot commented Aug 10, 2022

This issue/pull request has been marked as needs attention as it has been left pending without new activity for 4 days. Tagging @nkommuri @mehjoshi @huanghua78 for appropriate assignment. Sorry for the delay & Thank you for contributing to CORTX. We will get back to you as soon as possible.

hessio added a commit that referenced this pull request Aug 11, 2022
@hessio
Move documentation to main branch
575b3ac 
Addresses changes in PR #1563

Signed-off-by: hessio <[email protected]>
@hessio hessio mentioned this pull request Aug 11, 2022
Merged
12 tasks
hessio added a commit that referenced this pull request Aug 11, 2022
@hessio
Move documentation to main branch
2c4936f 
Addresses changes in PR #1563

Signed-off-by: hessio <[email protected]>
Signed-off-by: hessio <[email protected]>
alfhad pushed a commit to alfhad/cortx-motr that referenced this pull request Aug 18, 2022
@hessio
Move documentation to main branch (Seagate#2070)
c95d612 
Addresses changes in PR Seagate#1563

Signed-off-by: hessio <[email protected]>
Signed-off-by: hessio <[email protected]>

Signed-off-by: hessio <[email protected]>
Signed-off-by: hessio <[email protected]>
@andriytk andriytk changed the title Update CORTX-MOTR-ARCHITECTURE.md doc: fix pdclust link at CORTX-MOTR-ARCHITECTURE.md Aug 22, 2022
@andriytk
Copy link
Contributor

andriytk commented Aug 22, 2022

@mehjoshi, it can be landed now, i think.

@mehjoshi
Copy link

mehjoshi commented Aug 22, 2022

@mehjoshi, it can be landed now, i think.

Sure.
@rkothiya, Could you please merge this PR once the deployment and other checks complete? Thanks.

@rkothiya
Copy link
Contributor

rkothiya commented Aug 29, 2022

giving an exception for pre-merge and ci/cd regression job as this is related to documentation.

@rkothiya rkothiya added the Exception Allowed to merge without runing ut and st label Aug 29, 2022
@rkothiya rkothiya merged commit 781b047 into main Aug 29, 2022
kiwionly2 pushed a commit to kiwionly2/cortx-motr that referenced this pull request Aug 30, 2022
@hessio @kiwionly2
Move documentation to main branch (Seagate#2070)
faaf609 
Addresses changes in PR Seagate#1563

Signed-off-by: hessio <[email protected]>
Signed-off-by: hessio <[email protected]>

Signed-off-by: hessio <[email protected]>
Signed-off-by: hessio <[email protected]>
kiwionly2 pushed a commit to kiwionly2/cortx-motr that referenced this pull request Aug 30, 2022
@johnbent @kiwionly2
Update CORTX-MOTR-ARCHITECTURE.md (Seagate#1563)
389f2c1 
Fix the link to parity declustered design document.

Signed-off-by: John Bent <[email protected]>
@andriytk andriytk deleted the johnbent-patch-4 branch September 1, 2022 09:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@mehjoshi mehjoshi Awaiting requested review from mehjoshi

@madhavemuri madhavemuri Awaiting requested review from madhavemuri

@yeshpal-jain-seagate yeshpal-jain-seagate Awaiting requested review from yeshpal-jain-seagate

@huanghua78 huanghua78 Awaiting requested review from huanghua78

@siningwuseagate siningwuseagate Awaiting requested review from siningwuseagate siningwuseagate is a code owner

@vidyadhar-pinglikar vidyadhar-pinglikar Awaiting requested review from vidyadhar-pinglikar

@shashank-parulekar shashank-parulekar Awaiting requested review from shashank-parulekar

@nkommuri nkommuri Awaiting requested review from nkommuri nkommuri is a code owner

@sg-shankar sg-shankar Awaiting requested review from sg-shankar

@ivan-alekhin ivan-alekhin Awaiting requested review from ivan-alekhin

@sergey-shilov sergey-shilov Awaiting requested review from sergey-shilov

@yatin-mahajan yatin-mahajan Awaiting requested review from yatin-mahajan

@rkothiya rkothiya Awaiting requested review from rkothiya

2 more reviewers

@nikitadanilov nikitadanilov nikitadanilov approved these changes

@andriytk andriytk andriytk approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

cla-signed Exception Allowed to merge without runing ut and st needs-attention rose Status: Waiting to be Reviewed PR is waiting for reviewers to review the PR

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@johnbent @cortx-admin @andriytk @mehjoshi @rkothiya @nikitadanilov @hessio