Git blame points to https://chromium-review.googlesource.com/c/316970/7/site_utils/gs_offloader.py

Allen any chance this could be related to the virtualenv??

Here's the source context around the failure:
    if psutil:
        proc = psutil.Process()
        logging.debug('Set process to ionice IDLE')
        proc.ionice(psutil.IOPRIO_CLASS_IDLE)

'psutil' in turn is set like so:
try:
    # Does not exist, nor is needed, on moblab.
    import psutil
except ImportError:
    psutil = None


So, either
 1) Something changed in the 'psutil' module that broke this
    code, or
 2) Until recently, 'psutil' simply wasn't being imported.

I don't think gs_offloader use virtualenv?

I tried the import in my workstation and cautotest, seems psutil package was updated recently and it no longer has the method ionice

The psutil module itself changed.

On the lab server, the version of psutil is 1.2.1, and the ionice() function was renamed to set_ionice()

If you run in chroot, the version of psutil is 2.2.1. And it has the ionice() function, I think we should upgrade the psutil

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/627a75ebc4cc2f755ccad17d1c9f92c91cc2e776

commit 627a75ebc4cc2f755ccad17d1c9f92c91cc2e776
Author: Simran Basi <sbasi@google.com>
Date: Wed Feb 08 19:13:27 2017

gs_offloader: Move psutil import to after import common

We seem to be having an issue with the version of psutil on
our goobuntu machines. Move the import of psutil to after
import common to use the build_externals copy of psutil.

BUG= chromium:690089 
TEST=On goobuntu machine.

Change-Id: I09d2991f275e5f1941a4c70b1bc07bd15dc4224f
Reviewed-on: https://chromium-review.googlesource.com/439629
Reviewed-by: Dan Shi <dshi@google.com>
Tested-by: Simran Basi <sbasi@chromium.org>

[modify] https://crrev.com/627a75ebc4cc2f755ccad17d1c9f92c91cc2e776/site_utils/gs_offloader.py

> Move the import of psutil to after
import common to use the build_externals copy of psutil.

Oh dear, I hadn't thought of that.  That's horrifying.

For reference, the bad CL is here:
https://chromium-review.googlesource.com/#/c/434968/

+pprabhu

Oops, sorry about this. Did not see it coming.
Maybe test push needs a "health check" on gs_offloader (and other services?)?

Pri-0 bugs are critical regressions or serious emergencies, and this bug has not been updated in three days. Could you please provide an update, or adjust the priority to a more appropriate level if applicable?

If a fix is in active development, please set the status to Started.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot