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

Conversation

@hessio
Copy link
Contributor

@hessio hessio commented Aug 11, 2022

Addresses changes in PR #1563

Signed-off-by: hessio [email protected]

Problem Statement

  • Feature branch "documentation" had a directory which contained documentation about how parity delustering works
  • Since this is useful information I think it should be on the main repo

Design

  • Moved the documentation to the main branch

Coding

Checklist for Author

  • Coding conventions are followed and code is consistent

Testing

Checklist for Author

  • Unit and System Tests are added
  • Test Cases cover Happy Path, Non-Happy Path and Scalability
  • Testing was performed with RPM

Impact Analysis

Checklist for Author/Reviewer/GateKeeper

  • Interface change (if any) are documented
  • Side effects on other features (deployment/upgrade)
  • Dependencies on other component(s)

Review Checklist

Checklist for Author

  • JIRA number/GitHub Issue added to PR
  • PR is self reviewed
  • Jira and state/status is updated and JIRA is updated with PR link
  • Check if the description is clear and explained

Documentation

Checklist for Author

  • Changes done to WIKI / Confluence page / Quick Start Guide

View rendered doc/CORTX-MOTR-ARCHITECTURE.md

Addresses changes in PR #1563

Signed-off-by: hessio <[email protected]>
Signed-off-by: hessio <[email protected]>
@hessio hessio force-pushed the pdclust-documentation branch from 575b3ac to 2c4936f Compare August 11, 2022 10:50
@stale
Copy link

stale bot commented Aug 16, 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.

@rkothiya
Copy link
Contributor

@hessio please update the problem and the design section, also if you have a ticket then please mention that in the heading.

@stale stale bot removed the needs-attention label Aug 17, 2022
@rkothiya rkothiya added the Exception Allowed to merge without runing ut and st label Aug 17, 2022
@rkothiya
Copy link
Contributor

Since this is documentation change giving an exception

@rkothiya rkothiya merged commit c95d612 into main Aug 17, 2022
kiwionly2 pushed a commit to kiwionly2/cortx-motr that referenced this pull request Aug 30, 2022
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]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla-signed Exception Allowed to merge without runing ut and st

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants