feat: chart static-app using zerodowntime probes

[devthejo]

No description provided.

[revolunet]

ca devrait pas aller dans helm-charts ?

[devthejo]

ca devrait pas aller dans helm-charts ?

maybe, après c'est très spécifiquement lié au fait d'utiliser des images basée sur notre image nginx, du coup c'est pas super universel non plus, perso j'aurais tendance à mettre dans helm-chart que ce qui est vraiment pertinent pour que des personnes hors fabrique l'utilise et qui sont pas couplé à tout un fonctionnement ...
après ça se discute ....

[revolunet]

c'est quoi l'intéret de rajouter ca par rapport à utiliser notre chart "app" de base ? ca pourrait pas marcher avec le meme chart ?

[devthejo]

c'est quoi l'intéret de rajouter ca par rapport à utiliser notre chart "app" de base ? ca pourrait pas marcher avec le meme chart ?

c'est d'avoir un paramétrage par défaut qui correspont aux probes zero-downtime des nouvelles images nginx (à utiliser donc à partir de la 8.2.0)
ça réduit aussi la verbosité dans les repo (pas besoin de préciser le port)

à la place de voir mettre ça pour pouvoir bénéficier du zero-downtime:

containerPort: 8080
livenessProbe:
  httpGet:
    path: /live
    port: 8080
readinessProbe:
  httpGet:
    path: /ready
    port: 8080
lifecycle:
  preStop:
    exec:
      command: ["/pre-stop.sh"]

on peut mettre simplement

~chart: static-app

ce qui est plus simple

j'ai envisagé d'autres solutions mais ça rendait vraiment compliqué la chart app et le rendait confus

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication