Remove retryable.

Skip second ytelse access check in system context.

Author: oyvind-wedoe

src/main/kotlin/no/nav/klage/oppgave/service/BehandlingService.kt

@@ -667,7 +667,9 @@ class BehandlingService(
         if (tildeltSaksbehandlerIdent != null) {
             //Denne sjekken gjøres kun når det er en tildeling:
 
-            checkYtelseAccess(tildeltSaksbehandlerIdent = tildeltSaksbehandlerIdent, behandling = behandling)
+            if (!systemUserContext) {
+                checkYtelseAccess(tildeltSaksbehandlerIdent = tildeltSaksbehandlerIdent, behandling = behandling)
+            }
 
             //if fagsystem is Infotrygd also do this.
             if (behandling.shouldUpdateInfotrygd()) {

src/main/kotlin/no/nav/klage/oppgave/service/ExternalMottakFacade.kt

@@ -84,6 +84,7 @@ class ExternalMottakFacade(
                 saksbehandlerIdent = saksbehandlerIdent,
             )
         } catch (e: Exception) {
+            logger.error("Klarte ikke å tildele behandling ${behandling.id} til saksbehandlerIdent $saksbehandlerIdent. Feil: $e")
             mottakService.createTaskForMerkantil(
                 behandlingId = behandling.id,
                 reason = "Klarte ikke å tildele behandling ${behandling.id} til saksbehandlerIdent $saksbehandlerIdent. Feilmelding: ${e.message}"
@@ -149,6 +150,4 @@ class ExternalMottakFacade(
             systemUserContext = true,
         )
     }
-
-
 }

src/main/kotlin/no/nav/klage/oppgave/service/KabalInnstillingerService.kt

@@ -4,14 +4,12 @@ import no.nav.klage.kodeverk.Ytelse
 import no.nav.klage.oppgave.clients.kabalinnstillinger.KabalInnstillingerClient
 import no.nav.klage.oppgave.clients.kabalinnstillinger.model.*
 import no.nav.klage.oppgave.domain.klage.Behandling
-import org.springframework.retry.annotation.Retryable
 import org.springframework.stereotype.Service
 
 @Service
 class KabalInnstillingerService(
     private val kabalInnstillingerClient: KabalInnstillingerClient,
 ) {
-    @Retryable
     fun getPotentialSaksbehandlere(behandling: Behandling): Saksbehandlere {
         return kabalInnstillingerClient.searchSaksbehandlere(
             SaksbehandlerSearchInput(
@@ -21,7 +19,6 @@ class KabalInnstillingerService(
         )
     }
 
-    @Retryable
     fun getPotentialMedunderskrivere(behandling: Behandling): Medunderskrivere {
         if (behandling.tildeling == null) {
             return Medunderskrivere(medunderskrivere = emptyList())
@@ -36,7 +33,6 @@ class KabalInnstillingerService(
         )
     }
 
-    @Retryable
     fun getPotentialROL(behandling: Behandling): Saksbehandlere {
         return kabalInnstillingerClient.searchROL(
             ROLSearchInput(
@@ -46,14 +42,12 @@ class KabalInnstillingerService(
     }
 
     //TODO: Bør vi ha et cache her? Kan være et problem om leder gir nye tilganger, kanskje et kortere cache?
-    @Retryable
     fun getTildelteYtelserForSaksbehandler(navIdent: String): List<Ytelse> {
         return kabalInnstillingerClient.getSaksbehandlersTildelteYtelser(navIdent).ytelseIdList.map {
             Ytelse.of(it)
         }
     }
 
-    @Retryable
     fun getTildelteYtelserForEnhet(enhet: String): Set<Ytelse> {
         return kabalInnstillingerClient.getTildelteYtelserForEnhet(enhet).ytelseIdList.map {
             Ytelse.of(it)