netenglabs / suzieq

Using network observability to operate and design healthier networks
https://www.stardustsystems.net/
Apache License 2.0
780 stars 104 forks source link

controller plugins: deepcopy configuration #868

Closed LucaNicosia closed 1 year ago

LucaNicosia commented 1 year ago

Test inclusion requirements

In case the PR contains an enhancement or a new platform/service support, some tests have to be added for the new functionality:

For additional information about tests, follow this link

Related Issue

Fixes #

Description

It's possible that the same configuration is used in multiple namespaces. If this happens, it's possible that the configuration is modified during the initialization of the plugin (see base_source contructor) causing the next plugins to fail. In order to fix this issue, pass to the deepcopy the object constructors

Type of change

Please delete options that are not relevant.

New Behavior

...

Contrast to Current Behavior

...

Discussion: Benefits and Drawbacks

...

Changes to the Documentation

...

Proposed Release Note Entry

...

Comments

Include additional comments about the this pull request

Double Check