Upstart job mounting-glusterfs.conf increases unnecessary 30 seconds in Ubuntu boot

Bug #1465382 reported by Rarylson Freitas
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
glusterfs (Ubuntu)
Invalid
Low
Unassigned
Trusty
Triaged
Low
Unassigned

Bug Description

**Note:** This bug is already reported at upstream. However, as it occurs only in Ubuntu, I thought it makes sense to report it here again. So, if the upstream team do not merge the patch (they are mostly developers from Red Hat and may prioritize problems that affect RedHat like OSes), the Ubuntu team can test it, merge it and propose it to the upstream team.

See: https://bugzilla.redhat.com/show_bug.cgi?id=1231983

--------

Description of problem:

A bug in the mounting-glusterfs.conf file increases unnecessary 30 seconds in the boot in an Ubuntu Server.

The following error is logged in /var/log/upstart/mounting-gluster-*.log: "start: Job failed to start".

The following error is also logged in /var/log/upstart/wait-for-state-mounting-glusterfs-*.log:

"status: Unknown job: static-network-up
start: Unknown job: static-network-up".

For last, another error ("Mount failed") is logged in /var/log/boot.log too.

When not using the nobootwait/nofail flags in fstab, the bug can hang the mount process (and the boot process too). When using the nobootwait/nofail flags, the bug will increase the boot time in about 30 seconds.

Another report from another GlusterFS user can be found at [this link](http://serverfault.com/questions/611462/glusterfs-failing-to-mount-at-boot-with-ubuntu-14-04).

The bug is caused by the following errors:

- There is no need to wait for the network is up. The Ubuntu itself has the _netdev mount flag that will retry the mount for each time that an interface brings up;
- However, it's necessary to wait for the GlusterFS Server daemon (for mounts using localhost);
    - This was implemented in an old commit ([c3bbf6](https://github.com/gluster/glusterfs/commit/c3bbf6aa6c090fd066ab0079aa1c8ae332309d2a)), more precisely in [this link](https://github.com/gluster/glusterfs/blob/c3bbf6aa6c090fd066ab0079aa1c8ae332309d2a/extras/Ubuntu/mounting-glusterfs.conf). However, this commit was overwritten;
- It's wrong to use the wait-for-state upstart task to wait for a signal. It's used to wait for a job. static-network-up is an event signal, and not a job;
    - This is why the "Unknown job: static-network-up" is logged;
- It's wrong, when waiting for a job to be started, not passing the 'WAIT_STATE=running' env var because it's not the default in wait-for-state.

Version-Release number of selected component (if applicable):

Master branch (mainline, commit https://github.com/gluster/glusterfs/commit/5cdbbf34e31758071f597561aa8572cedd56aece) and the glusterfs-server and glusterfs-client packages in Ubuntu 14.04 (maybe the packages in other Ubuntu versions too).

How reproducible:

Every system boot.

Steps to Reproduce:

1. Create a valid volume in a Gluster Server
2. Create an entry in the /etc/fstab file in a Ubuntu 14.04 like:
   gluster1:/dir /var/dir glusterfs defaults,nobootwait,nofail,_netdev 0 0
3. Reboot the system
4. Run 'mount' and check that the volume was mounted
5. Check the boot and upstart logs that shows the problem

To see more easily this problem, it's also possible:

1. Create a valid volume in a Gluster Server
2. Create an entry in the /etc/fstab file in a Ubuntu 14.04 like:
   gluster1:/dir /var/dir glusterfs defaults,nobootwait,nofail,_netdev 0 0
3. Run 'time start mountall'
4. Run 'mount' and check that the volume was mounted
5. Check the upstart logs and the high value time output (about 30 seconds).

Actual results:

Errors logged and 30 seconds of unnecessary waiting.

Expected results:

The volume mounted as long as the desired interface is up (_netdev) and the gluster daemon (if it exists) is up, without errors being logged.

Additional info:

The files that need to be updated are [README.Ubuntu](https://github.com/gluster/glusterfs/commits/master/extras/Ubuntu/README.Ubuntu) and [mounting-glusterfs.conf](https://github.com/gluster/glusterfs/blob/master/extras/Ubuntu/mounting-glusterfs.conf).

A fixed version of 'mounting-glusterfs.conf' will be attached to this bug report.

-----------

Attachment: https://bugzilla.redhat.com/attachment.cgi?id=1039191

Tags: patch trusty
Revision history for this message
Rarylson Freitas (rarylson) wrote :

I made an error when choosing the package. Please ignore the upstart package and consider only the glusterfs package.

Changed in upstart (Ubuntu):
status: New → Invalid
affects: upstart (Ubuntu) → ubuntu
no longer affects: ubuntu
Revision history for this message
Rarylson Freitas (rarylson) wrote :
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "Patch to the mounting-glusterfs.conf file" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]

tags: added: patch
Mathew Hodson (mhodson)
tags: added: trusty
Changed in glusterfs (Ubuntu):
importance: Undecided → Low
James Page (james-page)
Changed in glusterfs (Ubuntu):
status: New → Triaged
Changed in glusterfs (Ubuntu Trusty):
status: New → Triaged
importance: Undecided → Low
Changed in glusterfs (Ubuntu):
status: Triaged → Invalid
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.