QA Regression Testing

Merge ~arighi/qa-regression-testing:support-python2-and-python3 into qa-regression-testing:master

Proposed by Andrea Righi on 2023-04-11

Status:	Rejected
Rejected by:	Steve Beattie on 2023-07-07
Proposed branch:	~arighi/qa-regression-testing:support-python2-and-python3
Merge into:	qa-regression-testing:master
Diff against target:	52 lines (+16/-4) 4 files modified scripts/kernel-security/errno-read.py (+4/-1) scripts/kernel-security/proc-leaks/dac-bypass.py (+4/-1) scripts/kernel-security/proc-maps/maps-protection.py (+4/-1) scripts/test-kernel-security.py (+4/-1)
Related bugs:	Link a bug report

Reviewer	Review Type	Date Requested	Status
Ubuntu Security Team		2023-04-11	Pending
Review via email: mp+440700@code.staging.launchpad.net

Description of the change

In lunar we are getting some test failures reporting the error:

/usr/bin/env: ‘python2’: No such file or directory

Fix by changing the shebang of the scripts in kernel-security to dynamically detect the availability of python2 or python3.

Revision history for this message

Francis Ginther (fginther) wrote on 2023-04-11:

This overlaps https://code.launchpad.net/~fginther/qa-regression-testing/+git/qa-regression-testing/+merge/440503.

I'd prefer to just convert all of these to python3 as we really have no reason to keep running these with python2 (we have python 3.4 in trusty which is as far back as we would need to run any kernel testing).

I'll propose another MP to address dac-bypass.py and test-kernel-security.py.

Revision history for this message

Francis Ginther (fginther) wrote on 2023-04-12:

I recommend replacing this MP with https://code.launchpad.net/~fginther/qa-regression-testing/+git/qa-regression-testing/+merge/440800 which applies additional 2to3 updates.