Skip to content

Conversation

@jaer-tsun
Copy link
Contributor

What this PR does / why we need it:

  • Instead of running telemetry buffer (server) on goroutine, we will spawn a separate process.
  • Adding an option to disable telemetry when starting CNS (-telemetry=false)

@jaer-tsun jaer-tsun requested a review from tamilmani1989 March 29, 2019 21:02
@tamilmani1989 tamilmani1989 changed the title Spawn telemetry process CNS to start telemetry process and connect to it Apr 1, 2019
@jaer-tsun jaer-tsun force-pushed the spawnTelemetryProcess branch from 92f503d to 14bb672 Compare April 2, 2019 18:35
@jaer-tsun jaer-tsun force-pushed the spawnTelemetryProcess branch from 14bb672 to bb0dddd Compare April 2, 2019 21:55
@jaer-tsun jaer-tsun force-pushed the spawnTelemetryProcess branch from bb0dddd to 4c12cb0 Compare April 3, 2019 22:50
@jaer-tsun jaer-tsun force-pushed the spawnTelemetryProcess branch from 4c12cb0 to 4fd8e2b Compare April 4, 2019 20:39
Copy link
Member

@tamilmani1989 tamilmani1989 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@jaer-tsun jaer-tsun merged commit 3043dc9 into Azure:master Apr 8, 2019
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.

2 participants