search

nais / api

MIT License
1 stars 0 forks source link

Sletting av apper kan gi falsk positiv #64

Open jhrv opened 2 months ago

jhrv commented 2 months ago

https://nav-it.slack.com/archives/C5KUST8N6/p1725951754452269 https://nav-it.slack.com/archives/C5KUST8N6/p1724998208664699

Dette registererer også et audit-event som at det er slettet, og det bør heller ikke skje. Console bør bare gi en feil i dette tilfelle. Hvis spec'en er feil også, bør det varsles om.

image

Audit-logg

image
thokra-nav commented 1 month ago

Vi kan gjøre som kubectl å vente på at sletting er fullført, men vil ha en timeout på 10 sekunder. Problemet med å vente er at dersom den ikke slettes iløpet av den tiden, så vil ressursen fortsatt være markert som slettet i kubernetes. På sikt kan den bli borte (dersom den enten bare er treg, eller det som forhindrer sletting blir fikset).

Så i mitt hode så er det intended behaviour som det er nå, men kan skrive om notisen etter sletting til at "Your app will be deleted"

Det er et metadata felt som inneholder deletion timestamp, vi kan eksponere den og ha en notis på toppen av app siden dersom den er slettet med

This app has been deleted and will soon be gone. If this was done a while ago, please contact the nais team

jhrv commented 1 month ago

Ja så hvis deletion timestamp er satt, så vises den som pending deletion? Og slette-knappen blir gråa ut kanskje?

thokra-nav commented 1 month ago

Jepp, vi kan løse det sånn :)