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

plugin: Rewrite the logic of define_plugins_dir #254

Merged
merged 1 commit into from
Sep 25, 2014

Conversation

Spredzy
Copy link
Contributor

@Spredzy Spredzy commented Sep 25, 2014

Currently, there is a duplicate error duplication when
compiling the catalog. This is due to the fact that the
call to define_plugins_dir is called with the exact same
parameter $install_path hence causing the error.

By including the name of the plugin within the name of the resource
we ensure the resource name is unique during catalog compilation.

Currently, there is a duplicate error duplication when
compiling the catalog. This is due to the fact that the
call to `define_plugins_dir` is called with the exact same
parameter `$install_path` hence causing the error.

By including the name of the plugin within the name of the resource
we ensure the resource name is unique during catalog compilation.
@jamtur01
Copy link
Contributor

LGTM

jamtur01 added a commit that referenced this pull request Sep 25, 2014
plugin: Rewrite the logic of define_plugins_dir
@jamtur01 jamtur01 merged commit 4a16ebf into sensu:master Sep 25, 2014
@Spredzy Spredzy deleted the duplicate_resources branch September 26, 2014 03:18
@Spredzy
Copy link
Contributor Author

Spredzy commented Sep 26, 2014

Thanks,

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