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.

Sign up for GitHub

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

Jump to bottom

Enabling 15 new LTP tests (PR review completed and merged on lsds/ltp) #765

Closed
wants to merge 7 commits into from
Closed

Enabling 15 new LTP tests (PR review completed and merged on lsds/ltp) #765

wants to merge 7 commits into from

Conversation

hukoyu
Copy link
Collaborator

@hukoyu hukoyu commented Aug 11, 2020 •
edited
Loading

Enabling 15 tests.
The bocking bugs for these tests fixed:

  • chroot01
  • writev06
  • get_robust_list01

PRs reviewed and merged for below tests:

@hukoyu
Copy link
Collaborator Author

hukoyu commented Aug 11, 2020 •
edited
Loading

This PR is blocked by three issues (in order to have pipeline pass to be able to merge this PR)
#767
#768
#772

@hukoyu hukoyu changed the title Enabling 7 LTP tests : chroot01, open12, mmap10, read02, writev06, get_robust_list01, fallocate01 Enabling 7 LTP tests : chroot01, open12, mmap10, writev06, get_robust_list01, fallocate01 Aug 11, 2020
This was referenced Aug 11, 2020
Closed
Closed
@hukoyu hukoyu changed the title Enabling 7 LTP tests : chroot01, open12, mmap10, writev06, get_robust_list01, fallocate01 Enabling 10 LTP tests : chroot01, open12, mmap10, writev06, get_robust_list01, fallocate01, sync_file_range02, renameat01, unshare01, unshare02 Aug 12, 2020
@hukoyu hukoyu requested a review from paulcallen August 12, 2020 18:19
@hukoyu
Copy link
Collaborator Author

hukoyu commented Aug 12, 2020

All of these 10 new LTP tests are passing, verified in pipeline. The failing tests are going to be fixed by @KenGordon that tracked by #772

@hukoyu
Copy link
Collaborator Author

hukoyu commented Aug 13, 2020 •
edited
Loading

@paulcallen all of these tests are passing. Verified in pipeline. We don't need to wait fixing failing LTP tests in order to merge this PR. Paul if you can approve, we can merge this PR and enable those 10 tests. Also PRs in lsds/ltp aready reviewed and merged.

@hukoyu hukoyu changed the title Enabling 10 LTP tests : chroot01, open12, mmap10, writev06, get_robust_list01, fallocate01, sync_file_range02, renameat01, unshare01, unshare02 Enabling 15 new LTP tests (PR review completed and merged on lsds/ltp) Aug 13, 2020
@hukoyu
Copy link
Collaborator Author

hukoyu commented Aug 13, 2020

Enabled 5 more test that PR review process is completed on lsds/ltp. We are enabling total 15 tests here. These tests were reviewed and approved by @davidchisnall and @SeanTAllen . We are just enabling them.

cc @paulcallen @jxyang

@hukoyu hukoyu mentioned this pull request Aug 14, 2020
Closed
@hukoyu
Copy link
Collaborator Author

hukoyu commented Aug 14, 2020

Not enabling failing 5 tests mentioned in #786 that failing after merging #403

Also disabling setresuid04 and setreuid07 until tests fixed.

@hukoyu
Copy link
Collaborator Author

hukoyu commented Aug 14, 2020

After disabling 7 tests and rerunning for enabling 10 tests, there are 5 failures out of 10. I added them with logs to #786

@hukoyu
Copy link
Collaborator Author

hukoyu commented Aug 15, 2020

Dupe of #793

@hukoyu hukoyu closed this Aug 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@paulcallen paulcallen paulcallen approved these changes

@davidchisnall davidchisnall Awaiting requested review from davidchisnall

@SeanTAllen SeanTAllen Awaiting requested review from SeanTAllen

@KenGordon KenGordon Awaiting requested review from KenGordon

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

Successfully merging this pull request may close these issues.
2 participants
@hukoyu @paulcallen