Bug 7152 - Check NFS-Shares, fails with to long path-names
Summary: Check NFS-Shares, fails with to long path-names
Status: RESOLVED FIXED
Alias: None
Product: CTDB 2.5.x or older
Classification: Unclassified
Component: ctdb (show other bugs)
Version: unspecified
Hardware: x64 Linux
: P3 trivial
Target Milestone: ---
Assignee: Michael Adam
QA Contact: Michael Adam
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-02-18 07:01 UTC by Thomas Sesselmann
Modified: 2010-02-23 04:17 UTC (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Thomas Sesselmann 2010-02-18 07:01:23 UTC
exportfs reports two lines if the path is to long, but replace this in
/etc/ctdb/events.d/60.nfs  will help:

old:
        # and that its directories are available
        [ "$CTDB_NFS_SKIP_SHARE_CHECK" = "yes" ] || {
            exportfs | grep -v '^#' | grep '^/' |
            sed -e 's/[[:space:]]*[^[:space:]]*$//' |
            ctdb_check_directories
        } || exit $?

new:
        # and that its directories are available
        [ "$CTDB_NFS_SKIP_SHARE_CHECK" = "yes" ] || {
            exportfs | grep '^/' |
            sed -e 's/[[:space:]]\+[^[:space:]]*$//' |
            ctdb_check_directories
        } || exit $?
Comment 1 Michael Adam 2010-02-18 09:47:55 UTC
Good catch - thanks!
I am going to incorporate this.

If you have (or set up) a git checkout of the ctdb.git repository
(git://git.samba.org/sahlberg/ctdb.git), then you could attach this
change as a git-formattet patch (git format-patch).
Then you would get proper credits for the change!

Cheers - Michael
Comment 2 Michael Adam 2010-02-23 04:17:27 UTC
I have brought the corresponding patch to master.
This will be fixed in ctdb version 1.0.114