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

CORTX-29985: DTM integration test #2062

Merged
merged 1 commit into from
Aug 9, 2022
Merged

Conversation

rkothiya
Copy link
Contributor

@rkothiya rkothiya commented Aug 8, 2022

This script will compile the code with dtm-enable flag
and then run dtm integration test.

Signed-off-by: Rinku Kothiya rinku.kothiya@seagate.com

Problem Statement

Enabled DTM integration test for pre-merge job.

Design

The script compiles the code with dtm-enable flag and runs dtm integration test.

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

This script will compile the code with dtm-enable flag
and then run dtm integration test.

Signed-off-by: Rinku Kothiya <rinku.kothiya@seagate.com>
@rkothiya
Copy link
Contributor Author

rkothiya commented Aug 9, 2022

As this is a standalone script and does not impact any of the features of motr, giving an exception for pre-merge jobs.

@rkothiya rkothiya added the Exception Allowed to merge without runing ut and st label Aug 9, 2022
@rkothiya rkothiya merged commit 4f948c4 into Seagate:main Aug 9, 2022
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.

4 participants