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

Fjerner busybox fra docker og skrevet om scriptet for å starte ba-sak mot preprod til å fungere uten printeenv #5125

Merged
merged 2 commits into from
Feb 28, 2025

Conversation

stigebil
Copy link
Contributor

💰 Hva skal gjøres, og hvorfor?

Fjerner busybox fra docker og skrevet om scriptet for å starte ba-sak mot preprod til å fungere uten printenv. Dette betyr at man ikke lenger kan bruke kubectl exec for å hente ut secrets. Dette for å gjøre distroless-imaget tryggere.

I stedet for å kjøre printenv, så bruker scriptet
kubectl -n teamfamilie get secrets
For å finnne navn på secret og så

kubectl -n teamfamilie get secret navn-på-secret -o json | jq '.data | map_values(@base64d)' 

kubectl get secret azure-familie-ba-sak-* secreten for å hente det samme.

Utvidet skriptet til å sjekke om naisdevice og gcloud er logget inn. Hvis det ikke er det så vil det komme en melding i terminalvinduet
Screenshot 2025-02-28 at 09 39 59

✅ Checklist

Har du husket alle punktene i listen?

  • Jeg har testet mine endringer i henhold til akseptansekriteriene 🕵️
  • Jeg har config- eller sql-endringer. I så fall, husk manuell deploy til miljø for å verifisere endringene.
  • Jeg har skrevet tester. Hvis du ikke har skrevet tester, beskriv hvorfor under 👇

@stigebil stigebil requested a review from a team as a code owner February 28, 2025 09:26
Copy link
Contributor

@GardOS GardOS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lekkert

@stigebil stigebil added this pull request to the merge queue Feb 28, 2025
Merged via the queue into main with commit 2d63b9b Feb 28, 2025
10 checks passed
@stigebil stigebil deleted the fjern_busybox_docker branch February 28, 2025 11:18
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

Successfully merging this pull request may close these issues.

2 participants