Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

clean garbage files in unit tests #206

Merged
merged 1 commit into from
Dec 31, 2020
Merged

Conversation

qinyihust
Copy link
Contributor

What problem does this PR solve?

To save disk capacity for CI, clean RcvSCSTest* folder after snapshot test.

@qinyihust qinyihust closed this Dec 31, 2020
@qinyihust qinyihust reopened this Dec 31, 2020
@qinyihust qinyihust closed this Dec 31, 2020
@qinyihust qinyihust reopened this Dec 31, 2020
system("rm -rf RevSCSTest1");
system("rm -rf RevSCSTest2");
system("rm -rf RevSCSTest3");
system("rm -rf RcvSCSTest.etcd");
Copy link
Member

Choose a reason for hiding this comment

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

    system(std::string("rm -rf " + kTestPrefix + "t.etcd").c_str());
    system(std::string("rm -rf " + kTestPrefix + "1").c_str());
    system(std::string("rm -rf " + kTestPrefix + "2").c_str());
    system(std::string("rm -rf " + kTestPrefix + "3").c_str());

Copy link
Member

Choose a reason for hiding this comment

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

system(std::string("rm -rf " + kTestPrefix + ".etcd").c_str()); 其他测试里的也改下吧,好几个地方不对

@qinyihust qinyihust changed the title clean garbage test files in snapshotcloneserver_recover_test clean garbage files in unit tests Dec 31, 2020
@xu-chaojie xu-chaojie merged commit 984a60e into opencurve:master Dec 31, 2020
ilixiaocui pushed a commit to ilixiaocui/curve that referenced this pull request Feb 6, 2023
cncf/mentoring#206

Signed-off-by: Chris Aniszczyk <caniszczyk@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants