Informasjonsforvaltning / fdk-issue-tracker

An issue tracking repository for data.norge.no
1 stars 0 forks source link

Setup paths in liveness/readiness probes #543

Open valosnah opened 2 years ago

valosnah commented 2 years ago

Flere av applikasjonene mangler path i probespec. Dette skaper problemer hvis f.eks. ingressen slettes og opprettes på nytt, da healthchecks blir opprettet basert på denne specen. Hvis path ikke finnes, defaulter den til / , men vi har flere applikasjoner som er satt opp slik at healthcheck skal gå mot /ready, og / ikke gir et forventet svar. GCP vil da bli kranglete og svare 502 for aktuell path, selv om applikasjonen egentlig virker.

Har stort sett løst dette ved å oppdatere healthchecken manuelt hvis nødvendig, da den er mer eller mindre statisk. Ser nå at dette burde være definert i spec, slik at vi eliminerer behov for manuelt arbeid.

Eksempel (fdk-harvest-admin):

      livenessProbe:
            httpGet:
              path: /api/datasources
              port: 8080
          (...)
       readinessProbe:
            httpGet:
              path: /api/datasources
              port: 8080
valosnah commented 2 years ago

Healthchecks styres av backendconfig

KjerstiSteien commented 9 months ago

@valosnah er dette noe som burde prioriteres?

KjerstiSteien commented 2 months ago

@valosnah fortsatt relevant?

valosnah commented 1 month ago

@KjerstiSteien Den er ikke veldig nødvendig å prioritere, men det kan være kjekt/naturlig å verifisere at dette er på plass ifm utføringen av https://github.com/Informasjonsforvaltning/fdk-issue-tracker/issues/769