please convert python scripts to Python3

Bug #1440553 reported by Matthias Klose
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Checkbox Provider - Base
Fix Released
High
Sylvain Pineau
plainbox-provider-checkbox (Ubuntu)
Fix Released
High
Sylvain Pineau

Bug Description

There is still the goal to remove Python2 from the desktop images, and to use Python3 only. Please convert the remaining python scripts installed with the gconf2 package to Python3.

$ fgrep -r /usr/bin/python /usr/share/2013.com.canonical.certification:checkbox /usr/lib/2013.com.canonical.certification\:* | fgrep -v python3
/usr/lib/2013.com.canonical.certification:checkbox/bin/glob_test:#!/usr/bin/python
/usr/lib/2013.com.canonical.certification:checkbox/bin/sleep_test:#!/usr/bin/python
/usr/lib/2013.com.canonical.certification:checkbox/bin/pm_test: "{user} ALL=NOPASSWD: /usr/bin/python' "
/usr/lib/2013.com.canonical.certification:checkbox/bin/pm_test:Exec=sudo /usr/bin/python {script} -r {repetitions} -w {wakeup} --hardware-delay {hardware_delay} --pm-delay {pm_delay} --min-pm-time {min_pm_time} --max-pm-time {max_pm_time} --append --total {total} --start {start} --pm-timestamp {pm_timestamp} {silent} --log-level={log_level} --log-dir={log_dir} {pm_operation}

Tags: python3
Revision history for this message
Zygmunt Krynicki (zyga) wrote :

Hey.

We'll look at converting those and uploading a fix to Debian and sync that to Ubuntu as soon as v+1 opens.

Thanks
ZK

Changed in plainbox-provider-checkbox (Ubuntu):
importance: Undecided → High
Zygmunt Krynicki (zyga)
Changed in plainbox-provider-checkbox (Ubuntu):
assignee: nobody → Zygmunt Krynicki (zyga)
Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

Both sleep_test and pm_test are already converted to python3 in trunk.

I'll propose to remove glob_test since:
1. it not used by our whitelists/testplans
2. globs last version was in 2007

Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

The last script manage_compiz_plugin is used to stop unity plugin to test multitouch gestures and is bloked by https://bugs.launchpad.net/compiz/+bug/1310794

Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

manage_compiz_plugin converted to python3. The checkbox provider is clean now :)

Changed in plainbox-provider-checkbox (Ubuntu):
status: New → Triaged
Changed in plainbox-provider-checkbox:
milestone: none → 0.20
assignee: nobody → Sylvain Pineau (sylvain-pineau)
importance: Undecided → High
status: New → Fix Committed
Zygmunt Krynicki (zyga)
Changed in plainbox-provider-checkbox (Ubuntu):
assignee: Zygmunt Krynicki (zyga) → nobody
Changed in plainbox-provider-checkbox:
status: Fix Committed → Fix Released
Revision history for this message
Zygmunt Krynicki (zyga) wrote :

We still depend on python-opencv for screenshot_validation. I think we need to reopen the upstream bug.

Changed in plainbox-provider-checkbox (Ubuntu):
assignee: nobody → Zygmunt Krynicki (zyga)
Changed in plainbox-provider-checkbox:
status: Fix Released → Confirmed
milestone: 0.20 → none
Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

and python3 bindings are only available in opencv 3.0 (only RC1 available so far upstream).

Revision history for this message
Zygmunt Krynicki (zyga) wrote : Re: [Bug 1440553] Re: please convert python scripts to Python3

Let's package that and see if this works. Part of the W cycle is about
getting rid of things like that and by working on all fronts to remove
remnants of python2-only APIs

On Tue, May 5, 2015 at 8:48 PM, Sylvain Pineau
<email address hidden> wrote:
> and python3 bindings are only available in opencv 3.0 (only RC1
> available so far upstream).
>
> --
> You received this bug notification because you are a member of Checkbox
> Bug Wranglers, which is subscribed to plainbox-provider-checkbox in
> Ubuntu.
> https://bugs.launchpad.net/bugs/1440553
>
> Title:
> please convert python scripts to Python3
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/plainbox-provider-checkbox/+bug/1440553/+subscriptions

Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

opencv 3 is gold now but W still provides 2.x versions

Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

http://packages.ubuntu.com/source/xenial/opencv is 2.4.9.1 so no python3 support.
Given that we're only using the opencv test for sru testing in the lab. I would just remove the package from the dependencies (and install it as part of the provisioning process)

Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

Upstream committed a version of this package without the python-opencv dependency. Once released to the hwcert stable ppa we'll update it on debian side.

Changed in plainbox-provider-checkbox (Ubuntu):
assignee: Zygmunt Krynicki (zyga) → Sylvain Pineau (sylvain-pineau)
status: Triaged → In Progress
Changed in plainbox-provider-checkbox:
status: Confirmed → In Progress
milestone: none → 0.25
Changed in plainbox-provider-checkbox:
status: In Progress → Fix Committed
Changed in plainbox-provider-checkbox:
status: Fix Committed → Fix Released
Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

I'm preparing a packaging update for the checkbox provider, the new version will be sync'ed from debian next week

Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

It's finally over. plainbox-provider-checkbox 0.25-1 is python3 only in xenial \o/

Changed in plainbox-provider-checkbox (Ubuntu):
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.