Bug 597112 - get-e2uuid should use blkid instead of "tune2fs -l" to get filesystem UUID
Summary: get-e2uuid should use blkid instead of "tune2fs -l" to get filesystem UUID
Alias: None
Product: Virtualization Tools
Classification: Community
Component: libguestfs   
(Show other bugs)
Version: unspecified
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Richard W.M. Jones
QA Contact:
Depends On:
Blocks: 627833
TreeView+ depends on / blocked
Reported: 2010-05-28 07:05 UTC by Yufang Zhang
Modified: 2010-08-27 05:16 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 627833 (view as bug list)
Last Closed: 2010-06-02 21:32:50 UTC
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)

Description Yufang Zhang 2010-05-28 07:05:32 UTC
Description of problem:
Currently, get-e2uuid gets filesystem UUID via "tune2fs -l". However, this doesn't work for JBD(journaling block device): whenever you try to get uuid of a JBD, you get a error that tells "Filesystem has unsupported feature(s)". In another way, UUID could be gotten via blkid, for both JBD or filesystems such as ext2/3/4. So it would be better to replace "tune2fs -l" with "blkid" in the implementation of get-e2uuid. 

Version-Release number of selected component (if applicable):
><fs> version 
major: 1
minor: 2
release: 7

How reproducible:

Steps to Reproduce:
1. ><fs> mke2journal 1024 /dev/VG/jounal
2. ><fs> get-e2uuid /dev/VG/jounal
libguestfs: error: get_e2uuid: tune2fs: Filesystem has unsupported feature(s) while trying to open /dev/VG/jounal
Couldn't find valid filesystem superblock.

Actual results:

Expected results:
get-e2uuid should return UUID successfully for JBD

Additional info:

Comment 1 Richard W.M. Jones 2010-06-01 14:39:07 UTC
Patches posted upstream:

Comment 2 Richard W.M. Jones 2010-06-02 21:32:50 UTC
This patch has been pushed upstream.

If this needs to be fixed for RHEL 5 / 6, please clone
this bug and adjust the Product accordingly.

Note You need to log in before you can comment on or make changes to this bug.