Bug 1449387 - Request for upstream inclusion of kmod sanity test
Summary: Request for upstream inclusion of kmod sanity test
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Fedora
Classification: Fedora
Component: kmod
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: kmod development team
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2017-05-09 21:01 UTC by Mike Gahagan
Modified: 2017-05-23 14:41 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-05-23 14:41:17 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
patch to add kmod sanity test to tests/ (13.13 KB, application/mbox)
2017-05-09 21:01 UTC, Mike Gahagan
no flags Details

Description Mike Gahagan 2017-05-09 21:01:05 UTC
Created attachment 1277450 [details]
patch to add kmod sanity test to tests/

Description of problem:
 
There is a proposal to upstream our RHEL test cases into Fedora, as well as
standardize our test interface using Ansible, more details can be found here:
https://fedoraproject.org/wiki/Changes/InvokingTestsAnsible
 
As a result many downstream test cases are being reviewed for upstream use.
I have attached a patch for a downstream kmod sanity test for the
dist-git repo for kmod under the "tests" sub-directory. The initial
step is to port the downstream test, later on will add the required tooling,
Ansible playbook, etc.
 
Can you please review/accept this patch to enable the kmod sanity test for upstream use ?


Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 Josh Boyer 2017-05-09 21:08:55 UTC
Which branch(es)?

Also, just so I'm clear on this, the test isn't expected to work at all at this
point, correct?

Who will maintain the test going forward?  Does that require commit access for
new people?


Note You need to log in before you can comment on or make changes to this bug.