IPA fails to record OSError failure with iscsi init

Bug #1715466 reported by Julia Kreger
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ironic-python-agent
Fix Released
Medium
Julia Kreger

Bug Description

An exception catch in ironic_python_agent/extensions/iscsi.py for some reason has ProcessExecutionError from oslo, and OSError from python and expects identical data structures which do not exist.

OSError can be raised in cases, such as where a program just doesn't exist on disk.

Changed in ironic-python-agent:
importance: Undecided → Medium
status: New → In Progress
assignee: nobody → Julia Kreger (juliaashleykreger)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ironic-python-agent (master)

Fix proposed to branch: master
Review: https://review.openstack.org/501386

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic-python-agent (master)

Reviewed: https://review.openstack.org/501386
Committed: https://git.openstack.org/cgit/openstack/ironic-python-agent/commit/?id=308cddc1bb5d4ae8bb9f462d64ee6754f2d14d8d
Submitter: Zuul
Branch: master

commit 308cddc1bb5d4ae8bb9f462d64ee6754f2d14d8d
Author: Julia Kreger <email address hidden>
Date: Wed Sep 6 19:15:06 2017 +0000

    Fix OSError catch

    The _execute helper was catching OSError, except it
    was expecting the same attributes as ProcessExecutionError,
    which is incorrect.

    Added a separate catch and unit test to ensure that we
    at least properly catch and convert the error instead of
    raising an error about an attribute that does not exist.

    Change-Id: Id47715a5657478e4d9dd10ea7f360b1ededa27de
    Closes-Bug: #1715466

Changed in ironic-python-agent:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/ironic-python-agent 3.0.0

This issue was fixed in the openstack/ironic-python-agent 3.0.0 release.

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.