logger: fail to mount gluster storage shows as warning and not error

Bug #1317573 reported by Dafna Ron
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Low
Flavio Percoco

Bug Description

failure to mount gluster storage appears in the cinder volume log as a warning and not an error
since we would basically not be able to create/delete/attach when we fail to mount I think that this should be logged as ERROR and if possible print the trace in the log.

2014-05-08 18:54:45.306 3121 WARNING cinder.volume.drivers.glusterfs [-] Exception during mounting Unexpected error while running command.
Command: sudo cinder-rootwrap /etc/cinder/rootwrap.conf mount -t glusterfs 10.35.64.104:/gluster2-cinder-tshefi /var/lib/cinder/mnt/8f2d3277bf483ae9bae356bd866fed5e
Exit code: 1
Stdout: 'Mount failed. Please check the log file for more details.\n'
Stderr: ''

Revision history for this message
Eric Harney (eharney) wrote :

It is not appropriate to print a trace for this but we probably should log this as an error rather than a warning.

Mike Perez (thingee)
tags: added: drivers glusterfs
Eric Harney (eharney)
tags: added: nfs
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/95333

Changed in cinder:
assignee: nobody → Flavio Percoco (flaper87)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

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

commit a058e90c5ec33e35c3bc88fed644b598e490e9ad
Author: Flavio Percoco <email address hidden>
Date: Sat May 24 10:12:36 2014 +0200

    Use error instead of warning to log mount exc

    Gluster's driver uses warning to log mount exceptions. This patch
    replaces that warning with an error message.

    Change-Id: I54a10ec8abf460ca9ff7bb009b8d702cfad0fed0
    Closes-Bug: #1317573

Changed in cinder:
status: In Progress → Fix Committed
Mike Perez (thingee)
Changed in cinder:
importance: Undecided → Low
milestone: none → juno-1
Thierry Carrez (ttx)
Changed in cinder:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: juno-1 → 2014.2
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.