image cache is logging warnings when not enabled

Bug #1530964 reported by Patrick East
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Undecided
Patrick East

Bug Description

The image cache is logging this warning but it isn't even enabled:

 http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22Unable%20to%20get%20Cinder%20internal%20context%2C%20will%20not%20use%20image-volume%20cache.%5C%22%20AND%20tags%3A%5C%22screen-c-vol.txt%5C%22%20AND%20voting%3A%5C%221%5C%22

While disabled there shouldn't be any log output complaining about the internal tenant being available or not.

We probably don't want this message to be a warning either, there is already one about failing to create the internal context so this is kind of redundant (and verbose).

Changed in cinder:
assignee: nobody → Patrick East (patrick-east)
description: updated
Matt Riedemann (mriedem)
Changed in cinder:
status: New → Confirmed
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/263515

Changed in cinder:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/263515
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=382b56f2fe86659a26b3cdeb20f804245fb6ba6e
Submitter: Jenkins
Branch: master

commit 382b56f2fe86659a26b3cdeb20f804245fb6ba6e
Author: Patrick East <email address hidden>
Date: Mon Jan 4 12:02:50 2016 -0800

    Don’t log warnings for image cache when disabled

    Right now we do a check for the internal tenant and log a warning before
    we check to see if the cache is even turned on for that backend. We
    should be only checking for the internal tenant if the image cache has
    been configured for the backend.

    In addition we shouldn’t log a warning for this since there is already
    a warning for the internal context when we try and create it if it is
    not configured correctly… to reduce the warning spam this change
    switches the log message to an info level.

    Change-Id: Iddc4648543d268a4b3b8d557c94c4d2876654e22
    Closes-Bug: #1530964

Changed in cinder:
status: In Progress → Fix Released
Revision history for this message
Thierry Carrez (ttx) wrote : Fix included in openstack/cinder 8.0.0.0b2

This issue was fixed in the openstack/cinder 8.0.0.0b2 development milestone.

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.