Fjern ubrukte metrikker

navikt · May 14, 2024 · 79ccb8c · 79ccb8c
1 parent 0912770
commit 79ccb8c
Show file tree

Hide file tree

Showing 6 changed files with 1 addition and 77 deletions.
diff --git a/...structure/src/main/kotlin/no/nav/su/se/bakover/common/infrastructure/metrics/SuMetrics.kt b/...structure/src/main/kotlin/no/nav/su/se/bakover/common/infrastructure/metrics/SuMetrics.kt
@@ -2,9 +2,7 @@ package no.nav.su.se.bakover.common.infrastructure.metrics
 
 import com.github.benmanes.caffeine.cache.Cache
 import io.micrometer.core.instrument.Clock
-import io.micrometer.core.instrument.Counter
 import io.micrometer.core.instrument.Metrics
-import io.micrometer.core.instrument.Tags
 import io.micrometer.core.instrument.binder.cache.CaffeineCacheMetrics
 import io.micrometer.prometheus.PrometheusConfig
 import io.micrometer.prometheus.PrometheusMeterRegistry
@@ -31,58 +29,4 @@ data object SuMetrics {
     fun <K, V> monitorCache(cache: Cache<K, V>, cacheName: String) {
         CaffeineCacheMetrics.monitor(prometheusMeterRegistry, cache, cacheName)
     }
-
-    fun incrementCounter(metricName: String, type: String) {
-        Metrics.counter(
-            metricName,
-            Tags.of("type", type),
-        ).increment()
-    }
-
-    enum class Metrikk(val navn: String) {
-        SØKNAD_MOTTATT("soknad.mottatt"),
-        BEHANDLING_STARTET("behandling.startet"),
-        VEDTAK_IVERKSATT("vedtak.iverksatt"),
-    }
-
-    enum class Søknadstype(val type: String) {
-        PAPIR("papir"),
-        DIGITAL("digital"),
-    }
-
-    enum class Behandlingstype(val type: String) {
-        SØKNAD("soknadsbehandling"),
-        REVURDERING("revurdering"),
-    }
-
-    // TODO metrikkene burde prefikses
-    private fun counter(metrikk: Metrikk, type: String): Counter = Metrics.counter(metrikk.navn, Tags.of("type", type))
-
-    private val søknadPapirMottattCounter = counter(Metrikk.SØKNAD_MOTTATT, type = Søknadstype.PAPIR.type)
-    private val søknadDigitalMottattCounter = counter(Metrikk.SØKNAD_MOTTATT, type = Søknadstype.DIGITAL.type)
-    private val søknadsbehandlingStartetCounter = counter(Metrikk.BEHANDLING_STARTET, type = Behandlingstype.SØKNAD.type)
-    private val revurderingStartetCounter = counter(Metrikk.BEHANDLING_STARTET, type = Behandlingstype.REVURDERING.type)
-    private val vedtakSøknadsbehandlingIverksattCounter = counter(Metrikk.VEDTAK_IVERKSATT, type = Behandlingstype.SØKNAD.type)
-    private val vedtakRevurderingIverksattCounter = counter(Metrikk.VEDTAK_IVERKSATT, type = Behandlingstype.REVURDERING.type)
-
-    fun søknadMottatt(type: Søknadstype) {
-        when (type) {
-            Søknadstype.DIGITAL -> søknadDigitalMottattCounter
-            Søknadstype.PAPIR -> søknadPapirMottattCounter
-        }.increment()
-    }
-
-    fun behandlingStartet(type: Behandlingstype) {
-        when (type) {
-            Behandlingstype.SØKNAD -> søknadsbehandlingStartetCounter
-            Behandlingstype.REVURDERING -> revurderingStartetCounter
-        }.increment()
-    }
-
-    fun vedtakIverksatt(type: Behandlingstype) {
-        when (type) {
-            Behandlingstype.SØKNAD -> vedtakSøknadsbehandlingIverksattCounter
-            Behandlingstype.REVURDERING -> vedtakRevurderingIverksattCounter
-        }.increment()
-    }
 }
diff --git a/web/src/main/kotlin/no/nav/su/se/bakover/web/routes/revurdering/IverksettRevurderingRoute.kt b/web/src/main/kotlin/no/nav/su/se/bakover/web/routes/revurdering/IverksettRevurderingRoute.kt
@@ -7,7 +7,6 @@ import io.ktor.server.routing.post
 import no.nav.su.se.bakover.common.audit.AuditLogEvent
 import no.nav.su.se.bakover.common.brukerrolle.Brukerrolle
 import no.nav.su.se.bakover.common.ident.NavIdentBruker
-import no.nav.su.se.bakover.common.infrastructure.metrics.SuMetrics
 import no.nav.su.se.bakover.common.infrastructure.web.Feilresponser.attestantOgSaksbehandlerKanIkkeVæreSammePerson
 import no.nav.su.se.bakover.common.infrastructure.web.Resultat
 import no.nav.su.se.bakover.common.infrastructure.web.audit
@@ -19,11 +18,7 @@ import no.nav.su.se.bakover.common.infrastructure.web.withRevurderingId
 import no.nav.su.se.bakover.common.serialize
 import no.nav.su.se.bakover.domain.revurdering.RevurderingId
 import no.nav.su.se.bakover.domain.revurdering.RevurderingTilAttestering
-import no.nav.su.se.bakover.domain.revurdering.iverksett.KunneIkkeIverksetteRevurdering
 import no.nav.su.se.bakover.domain.revurdering.service.RevurderingService
-import no.nav.su.se.bakover.web.routes.dokument.tilResultat
-import no.nav.su.se.bakover.web.routes.tilResultat
-import no.nav.su.se.bakover.web.routes.utbetaling.tilResultat
 import vilkår.formue.domain.FormuegrenserFactory
 
 internal fun Route.iverksettRevurderingRoute(
@@ -44,7 +39,6 @@ internal fun Route.iverksettRevurderingRoute(
                     ifRight = {
                         call.sikkerlogg("Iverksatt revurdering med id $revurderingId")
                         call.audit(it.fnr, AuditLogEvent.Action.UPDATE, it.id.value)
-                        SuMetrics.vedtakIverksatt(SuMetrics.Behandlingstype.REVURDERING)
                         call.svar(Resultat.json(HttpStatusCode.OK, serialize(it.toJson(formuegrenserFactory))))
                     },
                 )

diff --git a/web/src/main/kotlin/no/nav/su/se/bakover/web/routes/revurdering/OpprettRevurderingRoute.kt b/web/src/main/kotlin/no/nav/su/se/bakover/web/routes/revurdering/OpprettRevurderingRoute.kt
@@ -7,7 +7,6 @@ import io.ktor.server.routing.post
 import no.nav.su.se.bakover.common.audit.AuditLogEvent
 import no.nav.su.se.bakover.common.brukerrolle.Brukerrolle
 import no.nav.su.se.bakover.common.ident.NavIdentBruker
-import no.nav.su.se.bakover.common.infrastructure.metrics.SuMetrics
 import no.nav.su.se.bakover.common.infrastructure.web.Feilresponser.kunneIkkeOppretteOppgave
 import no.nav.su.se.bakover.common.infrastructure.web.Resultat
 import no.nav.su.se.bakover.common.infrastructure.web.audit
@@ -63,7 +62,6 @@ internal fun Route.opprettRevurderingRoute(
                         ifRight = {
                             call.sikkerlogg("Opprettet en ny revurdering på sak med id $sakId")
                             call.audit(it.fnr, AuditLogEvent.Action.CREATE, it.id.value)
-                            SuMetrics.behandlingStartet(SuMetrics.Behandlingstype.REVURDERING)
                             call.svar(Resultat.json(HttpStatusCode.Created, serialize(it.toJson(formuegrenserFactory))))
                         },
                     )

diff --git a/web/src/main/kotlin/no/nav/su/se/bakover/web/routes/søknad/SøknadRoutes.kt b/web/src/main/kotlin/no/nav/su/se/bakover/web/routes/søknad/SøknadRoutes.kt
@@ -15,7 +15,6 @@ import io.ktor.server.routing.post
 import no.nav.su.se.bakover.common.audit.AuditLogEvent
 import no.nav.su.se.bakover.common.brukerrolle.Brukerrolle
 import no.nav.su.se.bakover.common.ident.NavIdentBruker
-import no.nav.su.se.bakover.common.infrastructure.metrics.SuMetrics
 import no.nav.su.se.bakover.common.infrastructure.web.Feilresponser
 import no.nav.su.se.bakover.common.infrastructure.web.Feilresponser.fantIkkeSak
 import no.nav.su.se.bakover.common.infrastructure.web.Feilresponser.fantIkkeSøknad
@@ -37,7 +36,6 @@ import no.nav.su.se.bakover.domain.søknad.søknadinnhold.FeilVedOpprettelseAvOp
 import no.nav.su.se.bakover.domain.søknad.søknadinnhold.FeilVedOpprettelseAvSøknadinnhold
 import no.nav.su.se.bakover.domain.søknad.søknadinnhold.FeilVedValideringAvBoforholdOgEktefelle
 import no.nav.su.se.bakover.domain.søknad.søknadinnhold.FeilVedValideringAvOppholdstillatelseOgOppholdstillatelseAlder
-import no.nav.su.se.bakover.domain.søknad.søknadinnhold.ForNav
 import no.nav.su.se.bakover.domain.søknadsbehandling.iverksett.avslå.manglendedokumentasjon.AvslåManglendeDokumentasjonCommand
 import no.nav.su.se.bakover.domain.søknadsbehandling.iverksett.avslå.manglendedokumentasjon.KunneIkkeAvslåSøknad
 import no.nav.su.se.bakover.service.søknad.AvslåSøknadManglendeDokumentasjonService
@@ -81,13 +79,6 @@ internal fun Route.søknadRoutes(
                                     { (saksnummer, søknad) ->
                                         call.audit(søknad.fnr, AuditLogEvent.Action.CREATE, null)
                                         call.sikkerlogg("Lagrer søknad ${søknad.id} på sak ${søknad.sakId}")
-                                        SuMetrics.søknadMottatt(
-                                            if (søknad.søknadInnhold.forNav is ForNav.Papirsøknad) {
-                                                SuMetrics.Søknadstype.PAPIR
-                                            } else {
-                                                SuMetrics.Søknadstype.DIGITAL
-                                            },
-                                        )
                                         call.svar(
                                             Resultat.json(
                                                 Created,

diff --git a/.../main/kotlin/no/nav/su/se/bakover/web/routes/søknadsbehandling/SøknadsbehandlingRoutes.kt b/.../main/kotlin/no/nav/su/se/bakover/web/routes/søknadsbehandling/SøknadsbehandlingRoutes.kt
@@ -26,7 +26,6 @@ import no.nav.su.se.bakover.common.ident.NavIdentBruker.Attestant
 import no.nav.su.se.bakover.common.ident.NavIdentBruker.Saksbehandler
 import no.nav.su.se.bakover.common.infrastructure.PeriodeJson
 import no.nav.su.se.bakover.common.infrastructure.PeriodeJson.Companion.toJson
-import no.nav.su.se.bakover.common.infrastructure.metrics.SuMetrics
 import no.nav.su.se.bakover.common.infrastructure.web.Feilresponser
 import no.nav.su.se.bakover.common.infrastructure.web.Feilresponser.attestantOgSaksbehandlerKanIkkeVæreSammePerson
 import no.nav.su.se.bakover.common.infrastructure.web.Feilresponser.fantIkkeBehandling
@@ -108,7 +107,6 @@ internal fun Route.søknadsbehandlingRoutes(
                             {
                                 call.sikkerlogg("Opprettet behandling på sak: $sakId og søknadId: $søknadId")
                                 call.audit(it.second.fnr, AuditLogEvent.Action.CREATE, it.second.id.value)
-                                SuMetrics.behandlingStartet(SuMetrics.Behandlingstype.SØKNAD)
                                 call.svar(Created.jsonBody(it.second, formuegrenserFactory))
                             },
                         )

diff --git a/...v/su/se/bakover/web/routes/søknadsbehandling/iverksett/IverksettSøknadsbehandlingRoute.kt b/...v/su/se/bakover/web/routes/søknadsbehandling/iverksett/IverksettSøknadsbehandlingRoute.kt
@@ -9,7 +9,6 @@ import no.nav.su.se.bakover.common.brukerrolle.Brukerrolle
 import no.nav.su.se.bakover.common.domain.attestering.Attestering
 import no.nav.su.se.bakover.common.ident.NavIdentBruker
 import no.nav.su.se.bakover.common.infrastructure.config.ApplicationConfig
-import no.nav.su.se.bakover.common.infrastructure.metrics.SuMetrics
 import no.nav.su.se.bakover.common.infrastructure.web.Feilresponser
 import no.nav.su.se.bakover.common.infrastructure.web.Resultat
 import no.nav.su.se.bakover.common.infrastructure.web.audit
@@ -59,7 +58,6 @@ internal fun Route.iverksettSøknadsbehandlingRoute(
                         val søknadsbehandling = it.second
                         call.sikkerlogg("Iverksatte behandling med id: $behandlingId")
                         call.audit(søknadsbehandling.fnr, AuditLogEvent.Action.UPDATE, søknadsbehandling.id.value)
-                        SuMetrics.vedtakIverksatt(SuMetrics.Behandlingstype.SØKNAD)
                         call.svar(HttpStatusCode.OK.jsonBody(søknadsbehandling, formuegrenserFactory))
                     },
                 )
@@ -76,6 +74,7 @@ internal fun KunneIkkeIverksetteSøknadsbehandling.tilResultat(): Resultat {
             message = "Simulering fører til feilutbetaling.",
             code = "simulering_fører_til_feilutbetaling",
         )
+
         is KunneIkkeIverksetteSøknadsbehandling.OverlappendeStønadsperiode -> this.underliggendeFeil.tilResultat()
         is KunneIkkeIverksetteSøknadsbehandling.KontrollsimuleringFeilet -> this.underliggende.tilResultat()
     }