Unneccessary log message of neutronclient when polling neutron resources

[Liusheng]

As bug 1499403 reported, there are may warning of regular resource not found when polling neutron, but the patch cannot fix that issue. Because the fix tried to accept exceptions.NotFound and recording a warning message, but actually, the exception maybe exceptions.NeutronClientException, which is superclass of exceptions.NotFound.

[OpenStack Infra (hudson-openstack)]

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

[Rohit Jaiswal (rohit-jaiswal-3)]

Neutron client should raise a NotFound exception when resource not found in neutron: https://github.com/openstack/python-neutronclient/blob/master/neutronclient/v2_0/client.py#L67

[Liusheng (liusheng)]

Hi Rohit, Yeah, I agree, but I guess for now, not all neutronclient API raise NotFound exception when resource not found, because I have update code with master in my environment, I can still find the 404 log message.

[Rohit Jaiswal (rohit-jaiswal-3)]

I think the neutron client 404 is because of neutron client log level in debug mode..

[Liusheng (liusheng)]

Rohit, not exactly, when I debug this issue, I found the 404 message is record by "LOG.warn(e)" statement, [1]

[1] https://github.com/openstack/ceilometer/blob/master/ceilometer/neutron_client.py#L47

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/232862
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=213b585ced83f101e3191318c4787ea4fdcc95ce
Submitter: Jenkins
Branch: master

commit 213b585ced83f101e3191318c4787ea4fdcc95ce
Author: liusheng <email address hidden>
Date: Fri Oct 9 12:40:50 2015 +0800

    Clean some log messages when polling neutron resources

    - Handle the regular resource not found warnings when polling neutron
      resources
    - Set the default log level of neutronclient to INFO to avoid recording
      requests of neutronclient token issuing.

    Change-Id: I94836b3031bae5bee2634b301c67a0c9487112ac
    Closes-Bug: #1504396

[Thierry Carrez (ttx)]

This issue was fixed in the openstack/ceilometer 6.0.0.0b1 development milestone.