3PAR drivers are logging version numbers a lot

Bug #1447697 reported by Walt Boring
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Low
Walt Boring

Bug Description

When the 3PAR drivers were refactored to remove the local file locks, the login mechanism to create the client to talk to the array, has the artifact of logging the 3par common version and the client version on every driver entry point.

The driver should only be logging that information at volume manager init_host time. Not every driver entry point.

2015-04-22 21:09:31.290 INFO cinder.volume.drivers.san.hp.hp_3par_common [req-183f37ba-ba19-4887-83e7-c09ebda9613d None] HP3PARCommon 2.0.40, hp3parclient 3.2.0

Tags: 3par driver
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

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

Changed in cinder:
assignee: nobody → Walt Boring (walter-boring)
status: New → In Progress
Changed in cinder:
importance: Undecided → Low
Changed in cinder:
milestone: none → liberty-1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/176905
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=5b9ee831ad0c372e31bca22710196397e151a951
Submitter: Jenkins
Branch: master

commit 5b9ee831ad0c372e31bca22710196397e151a951
Author: Walter A. Boring IV <email address hidden>
Date: Thu Apr 23 09:34:35 2015 -0700

    3PAR don't log version numbers every stats update

    After the 3PAR drivers were refactored to remove the local
    file locks, the login mechanism was logging the common and
    client version numbers on every driver entry point.

    This patch removes that logging except at driver startup.

    Change-Id: Ib7732817a45cbdfdf975df13dbaf9bc0462a1dc0
    Closes-Bug: #1447697

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in cinder:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: liberty-1 → 7.0.0
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.