chore(monitoring): Enable New Relic tracing in select methods (#1602)

spinnaker · Oct 15, 2020 · 5d31d90 · 5d31d90
1 parent 7fe8912
commit 5d31d90
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 1 deletion.
diff --git a/keel-core/keel-core.gradle.kts b/keel-core/keel-core.gradle.kts
@@ -37,9 +37,9 @@ dependencies {
  api("com.github.ben-manes.caffeine:caffeine")
 
  implementation("org.springframework:spring-tx")
-
  implementation ("io.github.resilience4j:resilience4j-kotlin")
  implementation ("io.github.resilience4j:resilience4j-retry")
+ implementation("com.newrelic.agent.java:newrelic-api:6.1.0")
 
  testImplementation(project(":keel-test"))
  testImplementation(project(":keel-core-test"))

diff --git a/...-core/src/main/kotlin/com/netflix/spinnaker/keel/actuation/EnvironmentPromotionChecker.kt b/...-core/src/main/kotlin/com/netflix/spinnaker/keel/actuation/EnvironmentPromotionChecker.kt
@@ -7,6 +7,7 @@ import com.netflix.spinnaker.keel.core.api.EnvironmentArtifactVetoes
 import com.netflix.spinnaker.keel.core.api.PinnedEnvironment
 import com.netflix.spinnaker.keel.persistence.KeelRepository
 import com.netflix.spinnaker.keel.telemetry.ArtifactVersionApproved
+import com.newrelic.api.agent.Trace
 import org.slf4j.LoggerFactory
 import org.springframework.context.ApplicationEventPublisher
 import org.springframework.stereotype.Component
@@ -22,6 +23,7 @@ class EnvironmentPromotionChecker(
 ) {
  private val log by lazy { LoggerFactory.getLogger(javaClass) }
 
+ @Trace(dispatcher=true)
  suspend fun checkEnvironments(deliveryConfig: DeliveryConfig) {
  val pinnedEnvs: Map<String, PinnedEnvironment> = repository
  .pinnedEnvironments(deliveryConfig)

diff --git a/keel-core/src/main/kotlin/com/netflix/spinnaker/keel/actuation/ResourceActuator.kt b/keel-core/src/main/kotlin/com/netflix/spinnaker/keel/actuation/ResourceActuator.kt
@@ -39,6 +39,7 @@ import com.netflix.spinnaker.keel.veto.VetoResponse
 import com.netflix.spinnaker.kork.exceptions.SpinnakerException
 import com.netflix.spinnaker.kork.exceptions.SystemException
 import com.netflix.spinnaker.kork.exceptions.UserException
+import com.newrelic.api.agent.Trace
 import kotlinx.coroutines.async
 import kotlinx.coroutines.supervisorScope
 import org.slf4j.LoggerFactory
@@ -75,6 +76,7 @@ class ResourceActuator(
  private val diffNotActionableEnabled: Boolean
  get() = springEnv.getProperty("keel.events.diff-not-actionable.enabled", Boolean::class.java, false)
 
+ @Trace(dispatcher=true)
  suspend fun <T : ResourceSpec> checkResource(resource: Resource<T>) {
  withTracingContext(resource) {
  val id = resource.id