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

Messagebus fix #39

Merged
merged 2 commits into from
May 17, 2017
Merged

Messagebus fix #39

merged 2 commits into from
May 17, 2017

Conversation

ghoneycutt
Copy link
Collaborator

No description provided.

This makes reading the logic much easier to follow. There are no
functional changes to the module.
oddjobd requires messagebus. The module will install messagebus, though
the service is not started. Without this patch, sssd will not work on
EL6 when oddjobd is being managed, which is the default.
@ghoneycutt
Copy link
Collaborator Author

I upgraded from 8efc626 to v1 and noticed that it introduced a bug regarding the ordering of messagebus and oddjobd. This set of commits is to address that.

@ghoneycutt
Copy link
Collaborator Author

@sgnl05 I'm using this patch in production to allow EL6 systems to continue to work with this module after upgrading. Could you please merge this?

@sgnl05 sgnl05 merged commit 9cca9ec into sgnl05:master May 17, 2017
@sgnl05
Copy link
Owner

sgnl05 commented May 17, 2017

Thanks!

@ghoneycutt
Copy link
Collaborator Author

@sgnl05 could you please tag a new release

@ghoneycutt
Copy link
Collaborator Author

Should be '1.0.1' as this is only a bug fix.

@ghoneycutt ghoneycutt deleted the messagebus_fix branch May 17, 2017 22:42
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