Skip to content
This repository has been archived by the owner on Feb 8, 2024. It is now read-only.

EOS-27247: Remove shared storage framework #711

Merged
merged 5 commits into from
Jan 21, 2022
Merged

EOS-27247: Remove shared storage framework #711

merged 5 commits into from
Jan 21, 2022

Conversation

vastradparayya
Copy link
Contributor

Signed-off-by: Parayya Vastrad [email protected]

Problem Statement

  • Remove shared storage framework and move shared storage to experiments

Design

Coding

  • Coding conventions are followed and code is consistent [Y/N]: Y
  • Confirm All CODACY errors are resolved [Y/N]: Y

Testing

  • Confirm that Test Cases are added (for both the cases, fix and feature) [Y/N]: NA
  • Confirm Test Cases cover Happy Path, Non-Happy Path and Scalability [Y/N]: NA
  • Confirm Testing was performed with installed RPM [Y/N]: Y

Review Checklist

Before posting the PR please ensure

  • PR is self reviewed
  • Is there a change in filename/package/module or signature [Y/N]: N
  • If yes for above point, Is a notification sent to all other cortx components [Y/N] NA
  • Jira is updated
  • Check if the description is clear and explained.
  • Check Acceptance Criterion is defined.
  • All the tests performed should be mentioned before Resolving a JIRA.
  • Verification needs to be done before marked as Closed/Verified

Documentation

  • Changes done to WIKI / Confluence page

@vastradparayya
Removed shared_storage module
95ef34c 
Signed-off-by: Parayya Vastrad <[email protected]>
sachinpunadikar
sachinpunadikar approved these changes Jan 18, 2022
View reviewed changes
Copy link
Contributor

@sachinpunadikar sachinpunadikar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sachinpunadikar sachinpunadikar merged commit 975daf8 into Seagate:main Jan 21, 2022
suryakumar1024 pushed a commit to suryakumar1024/cortx-utils that referenced this pull request Mar 21, 2022
@vastradparayya @suryakumar1024
Removed shared_storage module (Seagate#711)
056fd53 
Signed-off-by: Parayya Vastrad <[email protected]>

Co-authored-by: Sachin Punadikar <[email protected]>
Signed-off-by: suryakumar.kumaravelan <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@rohit-k-dwivedi rohit-k-dwivedi rohit-k-dwivedi approved these changes

@sachinpunadikar sachinpunadikar sachinpunadikar approved these changes

@ajaysrivas ajaysrivas Awaiting requested review from ajaysrivas

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@vastradparayya @sachinpunadikar @rohit-k-dwivedi