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

feat(cataas): use y0an/cataas v1 & v2 instead of whoami #24

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

y0an
Copy link
Contributor

@y0an y0an commented May 27, 2020

Parce que tout changer au dernier moment à une certaine saveur 😺

@louiznk
Copy link
Contributor

louiznk commented May 28, 2020

Je plusois l'idée et j'ai cependant 3 objections :

  • sur un browser c'est cool mais sur un curl l'effet visuel est moins intéressant
  • l'image est plus longue pour s’arrêter (ne s’arrête pas sur un SIGTERM mais sur un SIGKILL)
  • l'image est plus grosse (83.6 MB vs 7.4 MB)

@y0an
Copy link
Contributor Author

y0an commented May 28, 2020

  • sur un browser c'est cool mais sur un curl l'effet visuel est moins intéressant

J'ai /api qui affiche la version, je vais y ajouter host + ip

  • l'image est plus longue pour s’arrêter (ne s’arrête pas sur un SIGTERM mais sur un SIGKILL)

J'ai ajouté terminationGracePeriodSeconds: 2 pour sortir plus rapidement, je l'avais mis à 0 mais dans ce cas le rollingupdate et tellement rapide qu'on ne peux pas voir le rolling des pods, mais je peux remettre 0 ou 1. D'ailleurs WTF node ignore SIGTERM?

  • l'image est plus grosse (83.6 MB vs 7.4 MB)

oui 🥺

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