Improve ironic driver logging configurability

[Michael Davies]

As part of review https://review.openstack.org/#/c/111425/19/nova/virt/ironic/driver.py it was suggested that logging configurability be addressed in the ironic driver.

A number of different viewpoints exist regarding whether the ironic driver should be independently configurable; and whether there should be a way to turn down the "chatyness" of the driver for operator sanity.

This is raised a bug so that the right solution for nova can be considered and implemented independently.

[Michael Still (mikal)]

The python logging framework already allows configuring log levels on a per module basis. Doesn't that cover our needs here?

[Jim Rollenhagen (jim-rollenhagen)]

Yes, we should go back and deprecate/remove this option in favor of the logging framework.

[Markus Zoeller (markus_z) (mzoeller)]

@Rodolfo: Are you still working on this? It's been >4 weeks until you set yourself as assignee but I cannot find a patch for that.

[Rodolfo Alonso (rodolfo-alonso-hernandez)]

Sorry, I was working with ironic with no time for this.

My apologies.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/312006
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=a924b1db46149d2928731f59afb7fef18deed54d
Submitter: Jenkins
Branch: master

commit a924b1db46149d2928731f59afb7fef18deed54d
Author: Markus Zoeller <email address hidden>
Date: Tue May 3 11:35:47 2016 +0200

    config: remove deprecated ironic.client_log_level

    The config option "client_log_level" of the "ironic" section was
    deprecated with commit d6baa2ae4d7f18dc2e4845af25c21b95767f46ef since
    the Kilo release and this change removes this option.

    Closes-Bug: 1365230
    Change-Id: I605d4ca5b3924ecf013cbf7e228e713b864d7aa4

[Davanum Srinivas (DIMS) (dims-v)]

This issue was fixed in the openstack/nova 14.0.0.0b1 development milestone.