Skip to content

fix(upstart): Install inotify-tools if using upstart#42

Merged
bflad merged 1 commit into
sous-chefs:masterfrom
jayofdoom:jay/FixInotify
Jan 2, 2014
Merged

fix(upstart): Install inotify-tools if using upstart#42
bflad merged 1 commit into
sous-chefs:masterfrom
jayofdoom:jay/FixInotify

Conversation

@jayofdoom

Copy link
Copy Markdown
Contributor

My primary concern is that it appears inotify-tools is only available in RHEL via EPEL? Given they don't use upstart though, is this an issue? I don't really run any redhat-based OSes so it's a bit hard for me to tell.

@bflad

bflad commented Jan 2, 2014

Copy link
Copy Markdown
Contributor

Great catch. Surprised I haven't run into this during testing. Anything special about your Ubuntu setup? :)

While you could theoretically use Upstart init_type on RHEL family machines, I don't think we need to necessarily support it since CentOS/RHEL 6 is primarily sysv (by default) and 7 will be primarily systemd (by default assuming they're following Fedora's footsteps - I haven't messed around with the 7 Beta yet). I guess we'll see if anyone really wants it; it wouldn't be too hard to add.

bflad added a commit that referenced this pull request Jan 2, 2014
fix(upstart): Install inotify-tools if using upstart
@bflad
bflad merged commit bd64a8f into sous-chefs:master Jan 2, 2014
@jayofdoom

Copy link
Copy Markdown
Contributor Author

@bflad While I have this running on real machines, most of the issues I'm filing is against a local vagrant VM based on Ubuntu Precise w/the Saucy kernel installed. Basically all it's doing is installing omnibus chef, and running a small recipe that loads all these. I intend on using chef-docker to manage all this in production as well, hence why I've been finding+fixing every little thing I see in dev to lessen impact when I take this to my production environment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants