Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Attempt to allow RHEL5(and derivatives) which use a old chroot #88

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lmorfitt
Copy link

Please see #49 for more details.

This is a attempt to fix the init scripts for RHEL5 (and friends).

Luke

@dartua
Copy link

dartua commented Aug 5, 2015

it works!!! ALELUYA! Many thanks! My OS is oracle linux 5.6.

@lmorfitt
Copy link
Author

lmorfitt commented Aug 6, 2015

@dartua Glad Its working for you. I've tested on RHEL 5.11 and all seems to work for me.

@FlorianHeigl
Copy link

We hit the same issues on SLES11 (or at least I think it's the same).
I'll try soon if this makes issues go away.

What I hated was that it still thinks it sucessfully started [x] done

@FlorianHeigl
Copy link

I think this patch has a typo since it runs sh -c, not su - -c or something like that ... is that intentional in any way?
It appears to be just dropping the user switching

@FlorianHeigl
Copy link

I'm currently using something like this to the same end.

Run the program!

#nice -n ${LS_NICE} chroot --userspec $LS_USER:$LS_GROUP $EXTRA_GROUPS / sh -c "
nice -n ${LS_NICE} sudo -u $LS_USER sh -c "
cd $LS_HOME
ulimit -n ${LS_OPEN_FILES}
exec "$program" $args
" > "${LS_LOG_DIR}/$name.stdout" 2> "${LS_LOG_DIR}/$name.err" &

@lmorfitt
Copy link
Author

@FlorianHeigl - This was a quick hack to drop the user switching, as the chroot option was not available on RHEL5.

I personally no longer have a requirement to support RHEL5 systems and therefore a much better solution may now be available.

@jordansissel
Copy link
Owner

I'd be open to accepting a patch that included user switching support for these older distros. I appreciate your sharing of this quick fix, but it disables user switching, so I don't want to merge it. Hopefully folks who need a quick fix can use this PR, though.

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.

4 participants