Skip to content
This repository has been archived by the owner on Apr 2, 2024. It is now read-only.

Fix: Do not collect telemetry if set to off #1612

Conversation

Harkishen-Singh
Copy link
Member

@Harkishen-Singh Harkishen-Singh commented Sep 1, 2022

Fixes: #1413

Signed-off-by: Harkishen-Singh harkishensingh@hotmail.com

This commits updates the telemetry engine to start, irrespective of
whether timescaledb.telemetry_level is on or off. However,
collection of telemetry happens only when the telemetry_level is on.

This allows the telemetry engine to either start telemetry collection
if it was disabled in the starting, but enabled later on, or stop
telemetry collection if it was enabled in the starting but disabled
later on.

Description

Merge requirements

Please take into account the following non-code changes that you may need to make with your PR:

  • CHANGELOG entry for user-facing changes
  • Updated the relevant documentation

@Harkishen-Singh Harkishen-Singh self-assigned this Sep 1, 2022
@Harkishen-Singh Harkishen-Singh requested a review from a team as a code owner September 1, 2022 12:34
@Harkishen-Singh Harkishen-Singh force-pushed the telemetry_engine_respect_db_state branch 2 times, most recently from a613eb6 to 8ac89fc Compare September 2, 2022 07:37
@Harkishen-Singh Harkishen-Singh added this to the 0.15.0 milestone Sep 12, 2022
@Harkishen-Singh Harkishen-Singh force-pushed the telemetry_engine_respect_db_state branch 2 times, most recently from a1512f2 to 676f4bb Compare September 14, 2022 09:50
Signed-off-by: Harkishen-Singh <harkishensingh@hotmail.com>
@Harkishen-Singh Harkishen-Singh enabled auto-merge (rebase) September 14, 2022 10:05
@Harkishen-Singh Harkishen-Singh merged commit efec5ae into timescale:master Sep 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Promscale connector doesn't recheck if telemetry is still on/off in the DB
2 participants