Skip to content

Feature/503 too many open files issue

Julien Andrieux requested to merge feature/503-too-many-open-files-issue-ng into develop

Describe the MR

In the Probes, move all the resource opening calls from the Configure() function to the Execute() function.

Why was this MR needed?

To allow a defer call on resource closing functions when the Probe exits. If we don't do that, the connections to resources may never be closed, which could lead us to a leaking situation.

Extra-care points?

The resource connection that where defined in the Probe structure are removed, because they are only needed in the Execute() function.

The MR also updates dependencies of the build process: go 1.12.5 and node 10.x.

Does this MR meet the acceptance criteria?

  • Documentation created/updated
  • CHANGELOG updated
  • Unit Tests added/updated

What are the relevant issue numbers?

Closes #503 (closed)

License and Developer Certificate of Origin

  • By contributing to Pantomath SAS, You accept and agree to the following terms and conditions for Your present and future Contributions submitted to Pantomath SAS. Except for the license granted herein to Pantomath SAS and recipients of software distributed by Pantomath SAS, You reserve all right, title, and interest in and to Your Contributions. All Contributions are subject to the following Developer Certificate of Origin and License terms.
Edited by Julien Andrieux

Merge request reports