deploy error logging should be improved

[aeva black]

In some cases, the same error is logged multiple times. This is better than not logging the error at all... but it could be better.

Related to https://bugs.launchpad.net/bugs/1286053

[Dmitry Tantsur (divius)]

Dominik, hi! You've been assigned to this bug for a while already. Could you give a status update or unassign yourself?

[dominik dobruchowski (dominik-dobruchowski)]

Sure! unassigned ;-)

-----Original Message-----
From: <email address hidden> [mailto:<email address hidden>] On Behalf Of Dmitry "Divius" Tantsur
Sent: Thursday, September 25, 2014 1:05 PM
To: Dobruchowski, Dominik
Subject: [Bug 1286374] Re: deploy error logging should be improved

Dominik, hi! You've been assigned to this bug for a while already. Could you give a status update or unassign yourself?

--
You received this bug notification because you are subscribed to the bug report.
https://bugs.launchpad.net/bugs/1286374

Title:
  deploy error logging should be improved

Status in OpenStack Bare Metal Provisioning Service (Ironic):
  In Progress

Bug description:
  In some cases, the same error is logged multiple times. This is better
  than not logging the error at all... but it could be better.

  Related to https://bugs.launchpad.net/bugs/1286053

To manage notifications about this bug go to:
https://bugs.launchpad.net/ironic/+bug/1286374/+subscriptions

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/144771
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=d1f7eddb9e0100f849500589a2675e03350ff67d
Submitter: Jenkins
Branch: master

commit d1f7eddb9e0100f849500589a2675e03350ff67d
Author: Vic Howard <email address hidden>
Date: Fri Jan 2 11:06:12 2015 -0500

    display error logging should be improved

    Removed duplicate error logging

    Change-Id: Id953a79b7db56b4014727d402104a9e998e55255
    Closes-Bug: 1286374

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/150187
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=46067160e6787631fb5a55193a02794449f22ebf
Submitter: Jenkins
Branch: master

commit 46067160e6787631fb5a55193a02794449f22ebf
Author: Chris Krelle <email address hidden>
Date: Mon Jan 26 15:58:05 2015 -0800

    Actively check iSCSI connection after login

    This replaces the sleep command at the end of the login_iscsi fuction
    with an active check and a forced lun update. These two things help ensure
    the connection sees all luns on the iSCSI device before attempting to write
    to it. It also adds a configurable option to allow users to be able to
    customize the number of attempts for their environment.

    Also add some very basic logging to aid in deploy debuging

    Change-Id: I68c9f9f86f5f113bb111c0f4fd83216ae0659d36
    Closes-bug: #1415117
    Partial-Bug: #1286374