Skip to content
This repository has been archived by the owner on Nov 8, 2022. It is now read-only.

Expose plugin load timeout as a configurable option #1191

Merged
merged 1 commit into from
Sep 12, 2016
Merged

Expose plugin load timeout as a configurable option #1191

merged 1 commit into from
Sep 12, 2016

Conversation

obourdon
Copy link
Contributor

@obourdon obourdon commented Sep 9, 2016

Fixes #1164

Summary of changes:

  • Expose plugin load timeout in plugin manager so that the plugin can take more time to load on a busy system

Testing done:

  • successfully tested on our environment where 12 plugins are autoloaded with an additional auto-loaded task which was failing due to this issue

@intelsdi-x/snap-maintainers

@obourdon
Copy link
Contributor Author

Could you please review this PR and tell me if I need to rework some parts ? Thanks

cc: @intelsdi-x/snap-maintainers

@jcooklin
Copy link
Collaborator

LGTM

@jcooklin jcooklin merged commit 9a05d66 into intelsdi-x:master Sep 12, 2016
@obourdon obourdon deleted the plugin-load-timeout branch September 20, 2016 15:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@obourdon @jcooklin