Merge lp://staging/~semiosis/ubuntu/precise/glusterfs/fix-for-876648 into lp://staging/ubuntu/precise/glusterfs

Proposed by Louis Zuckerman
Status: Needs review
Proposed branch: lp://staging/~semiosis/ubuntu/precise/glusterfs/fix-for-876648
Merge into: lp://staging/ubuntu/precise/glusterfs
Diff against target: 68 lines (+29/-1)
5 files modified
debian/changelog (+7/-0)
debian/control (+2/-1)
debian/glusterfs-server.mounting-glusterfs.upstart (+7/-0)
debian/glusterfs-server.upstart (+10/-0)
debian/rules (+3/-0)
To merge this branch: bzr merge lp://staging/~semiosis/ubuntu/precise/glusterfs/fix-for-876648
Reviewer Review Type Date Requested Status
Clint Byrum (community) Needs Fixing
Review via email: mp+90587@code.staging.launchpad.net

Description of the change

added two upstart jobs to glusterfs-server package:
- glusterfs-server.conf: the glusterd management daemon (replaces debian initscript)
- mounting-glusterfs.conf: blocks mounting glusterfs filesystems until glusterfs-server is started

To post a comment you must log in.
Revision history for this message
Clint Byrum (clint-fewbar) wrote :

Wow this is great, thanks so much for submitting.

One nit-pick.. you need to run 'update-maintainer' in the package root so that the Maintainer is changed to the ubuntu developers, since we have patched the package. Just run that, and bzr commit/push, and it will be ready to be sponsored.

review: Needs Fixing
22. By Louis Zuckerman

updated maintaner to ubuntu developers

Revision history for this message
Louis Zuckerman (semiosis) wrote :

> Wow this is great, thanks so much for submitting.
>
> One nit-pick.. you need to run 'update-maintainer' in the package root so that
> the Maintainer is changed to the ubuntu developers, since we have patched the
> package. Just run that, and bzr commit/push, and it will be ready to be
> sponsored.

Done. I also fixed a typo in the mounting-glusterfs description.

23. By Louis Zuckerman

fixed a typo in the mounting-glusterfs description

Unmerged revisions

23. By Louis Zuckerman

fixed a typo in the mounting-glusterfs description

22. By Louis Zuckerman

updated maintaner to ubuntu developers

21. By Louis Zuckerman

Added Ubuntu upstart jobs providing support for local glusterfs mounts
(LP: #876648)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches