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

Enh: added strict object name conflict policy #2010

Merged

Conversation

geektophe
Copy link
Collaborator

This patch adds a global parameter named conflict_policy to control the shinken behavior when there's name conflict between objects.

When more than one object are created with the same name and no definition order has been specified, the default behavior is to raise a warning message and keep the last definition.

When enabling strict conflict naming policy by setting the global parameter conflict_policy to strict, this results in an error that prevents the configuration from being loaded.

This is typically used to enforce more precise objects definition, and avoid silent collisions lading to unexpected behavior.

@geektophe geektophe force-pushed the enh_strict_object_coflict_policy branch 4 times, most recently from 8835ec9 to 74e9f0e Compare March 26, 2021 16:30
This patch adds a global parameter named `conflict_policy` to control
the shinken behavior when there's name conflict between objects.

When more than one object are created with the same name and no definition
order has been specified, the default behavior is to raise a warning message
and keep the last definition.

When enabling strict conflict naming policy by setting the global parameter
`conflict_policy` to `strict`, this results in an error that prevents the
configuration from being loaded.

This is typically used to enforce more precise  objects definition, and avoid
silent collisions lading to unexpected behavior.
@geektophe geektophe force-pushed the enh_strict_object_coflict_policy branch from 74e9f0e to d680640 Compare May 6, 2021 17:14
@geektophe geektophe merged commit fc4a0fe into shinken-solutions:master May 10, 2021
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.

None yet

1 participant