Skip to content

CTOR-1529: influxdb-metrics-apiv2#280

Open
sdepassio wants to merge 32 commits into
developfrom
CTOR-1529-enhance-influxdb-metrics-apiv-1-to-influxdb-1-metrics-apiv-2
Open

CTOR-1529: influxdb-metrics-apiv2#280
sdepassio wants to merge 32 commits into
developfrom
CTOR-1529-enhance-influxdb-metrics-apiv-1-to-influxdb-1-metrics-apiv-2

Conversation

@sdepassio

@sdepassio sdepassio commented Apr 30, 2025

Copy link
Copy Markdown
Contributor

Description

InfluxDB stream connector using apiv2

Fixes CTOR-1529

Type of change

  • Patch fixing an issue (non-breaking change)
  • New functionality (non-breaking change)
  • Breaking change (patch or feature) that might cause side effects breaking part of the Software

Target serie

  • 22.10.x
  • 23.04.x
  • 23.10.x
  • 24.04.x
  • 24.10.x
  • master

Checklist

Community contributors & Centreon team

  • I have followed the coding style guidelines provided by Centreon
  • I have commented my code, especially new classes, functions or any legacy code modified. (docblock)
  • I have commented my code, especially hard-to-understand areas of the PR.
  • I have rebased my development branch on the base branch (master, maintenance).

@sdepassio sdepassio added the upload-artifacts Upload generated packages as artifacts label Apr 30, 2025
tanguyvda
tanguyvda requested changes May 19, 2025
View reviewed changes
Comment thread centreon-certified/influxdb/influxdb-metrics-apiv2.lua Outdated
Comment thread centreon-certified/influxdb/influxdb-metrics-apiv2.lua
Comment thread centreon-certified/influxdb/influxdb-metrics-apiv2.lua Outdated
Comment thread centreon-certified/influxdb/influxdb-metrics-apiv2.lua Outdated
Comment thread centreon-certified/influxdb/influxdb-metrics-apiv2.lua Outdated
Comment thread centreon-certified/influxdb/influxdb-metrics-apiv2.lua Outdated
Comment thread centreon-certified/influxdb/influxdb-metrics-apiv2.lua Outdated
Comment thread centreon-certified/influxdb/influxdb-metrics-apiv2.lua Outdated
Comment thread centreon-certified/influxdb/influxdb-metrics-apiv2.lua Outdated
Comment thread centreon-certified/influxdb/influxdb-metrics-apiv2.lua
tanguyvda
tanguyvda requested changes May 19, 2025
View reviewed changes
Comment thread modules/centreon-stream-connectors-lib/sc_common.lua
Comment thread modules/centreon-stream-connectors-lib/sc_event.lua
Comment thread modules/centreon-stream-connectors-lib/sc_flush.lua
Comment thread modules/centreon-stream-connectors-lib/sc_logger.lua Outdated
Comment thread modules/centreon-stream-connectors-lib/sc_logger.lua Outdated
Comment thread modules/centreon-stream-connectors-lib/sc_params.lua Outdated
Comment thread modules/centreon-stream-connectors-lib/sc_params.lua Outdated
Comment thread modules/centreon-stream-connectors-lib/sc_metrics.lua
Comment thread modules/centreon-stream-connectors-lib/sc_metrics.lua
Comment thread modules/centreon-stream-connectors-lib/sc_metrics.lua
@sdepassio
sdepassio marked this pull request as ready for review May 22, 2025 13:04
@sdepassio
sdepassio requested review from a team as code owners May 22, 2025 13:04
@sdepassio
sdepassio marked this pull request as draft May 26, 2025 08:43
@sdepassio sdepassio mentioned this pull request Jun 3, 2025
Merged
13 tasks
@sdepassio
sdepassio marked this pull request as ready for review June 17, 2025 08:06
sdepassio and others added 20 commits June 17, 2025 10:08
@sdepassio
sdepassio force-pushed the CTOR-1529-enhance-influxdb-metrics-apiv-1-to-influxdb-1-metrics-apiv-2 branch from 791e61f to bfe3146 Compare June 17, 2025 08:08
@sdepassio
sdepassio requested review from omercier and tanguyvda July 3, 2025 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kduret kduret kduret approved these changes
@tanguyvda tanguyvda Awaiting requested review from tanguyvda
@omercier omercier Awaiting requested review from omercier

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

upload-artifacts Upload generated packages as artifacts

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@sdepassio @tanguyvda @kduret @vuntz