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

CORTX-33964: Improvement in code coverage job #2099

Merged
merged 1 commit into from
Aug 25, 2022

Conversation

rkothiya
Copy link
Contributor

Problem:
With the previous job if one of the test failed then
rest of the jobs were not executed.

Solution:
Running tests individually instead of a group.
Ignoring any failures that occur.

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

Problem Statement

If one of the test failed then rest of the jobs are not executed.

Design

Running tests individually instead of a group.
Ignoring any failures that occur.

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

Problem:
With the previous job if one of the test failed then
rest of the jobs were not executed.

Solution:
Running tests individually instead of a group.
Ignoring any failures that occur.

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

Giving an exception as this is a stand alone script and UT/ST or premerge jobs do not test this.

@rkothiya rkothiya added the Exception Allowed to merge without runing ut and st label Aug 25, 2022
@rkothiya rkothiya merged commit b23be2c into Seagate:main Aug 25, 2022
kiwionly2 pushed a commit to kiwionly2/cortx-motr that referenced this pull request Aug 30, 2022
Problem:
With the previous job if one of the test failed then
rest of the jobs were not executed.

Solution:
Running tests individually instead of a group.
Ignoring any failures that occur.

Signed-off-by: Rinku Kothiya <rinku.kothiya@seagate.com>
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