Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sletting av apper kan gi falsk positiv #64

Open
jhrv opened this issue Aug 30, 2024 · 3 comments
Open

Sletting av apper kan gi falsk positiv #64

jhrv opened this issue Aug 30, 2024 · 3 comments

Comments

@jhrv
Copy link
Contributor

jhrv commented Aug 30, 2024

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
Copy link
Contributor

thokra-nav commented Sep 10, 2024

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
Copy link
Contributor Author

jhrv commented Sep 10, 2024

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

@thokra-nav
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants