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

Reimplemented munge authentication. New version relies on the munge libr... #68

Merged
merged 1 commit into from
Feb 28, 2013

Conversation

lflis
Copy link

@lflis lflis commented Feb 15, 2013

...ary instead of exec resulting in better server performance and stability. Most of code contributed by Mariusz Mamonski from PSNC. Old code can still be used with --enable-munge-exec.

After applying this patch autoreconf -i is needed in order to use updated configure.ac

Code have been running successuly here in Cyfronet for more than one year now. (~17k cores)

…ibrary instead of exec resulting in better server performance and stability. Most of code contributed by Mariusz Mamonski from PSNC. Old code can still be used with --enable-munge-exec.
@lflis
Copy link
Author

lflis commented Feb 15, 2013

Hi Ken,

This patch is for 2.5 only. I haven't tried it with 4.x

In the commit there is README.munge file included which explains things a bit.
Basically there we have added new code between #define blocks and left the old code untouched.
Functions have been reimplemented to make use of munge library instead of executable but behaviour is identical to the old ones.

New code is enabled by default when building with --enable-munge-auth. If somebody needs old implementation
it is enough to specify --enable-munge-exec instead.

If you need more information please specify what exactly would be needed

Cheers.

LKF

knielson added a commit that referenced this pull request Feb 28, 2013
Reimplemented munge authentication. New version relies on the munge libr...
@knielson knielson merged commit 023e700 into adaptivecomputing:2.5-dev Feb 28, 2013
@knielson
Copy link
Contributor

Reminder to Adaptive, we need to update the docs for this.

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