CadolesKube/sso-kustom
14
0
Fork 0
Code Issues 17 Pull Requests 4 Projects Releases Wiki Activity

feat(pods): change image pull policy from Always to IfNotPresent #53

Merged
pcaseiro merged 1 commits from feat/pull-policy-IfNotPresent into unstable 2024-12-16 14:07:37 +01:00
Conversation 1 Commits 1 Files Changed 3 +5 -5
lgourvenec commented 2024-12-13 14:21:12 +01:00
Owner
Copy Link

Pour : la politique de pull d'image "Always" va toujours (sauf si le hash est donné dans le nom de l'image) vérifier avec le registry si l'image derrière le tag est la bonne. Seulement, si le registry tombe, le pod crash en boucle alors que l'image peut être déjà téléchargée sur le noeud k8s. Il me semble que cela peut arriver lors d'un scaling up par exemple (je n'arrive pas à le prouver cela dit)
Contre : il faut faire attention à ne pas écraser une image déjà poussée derrière un tag. En dev, cela peut peut-être embêtant.

Pendant coté mse-kustom : CNOUS/mse-kustom#261

Pour : la politique de pull d'image "Always" va toujours (sauf si le hash est donné dans le nom de l'image) vérifier avec le registry si l'image derrière le tag est la bonne. Seulement, si le registry tombe, le pod crash en boucle alors que l'image peut être déjà téléchargée sur le noeud k8s. Il me semble que cela peut arriver lors d'un scaling up par exemple (je n'arrive pas à le prouver cela dit) Contre : il faut faire attention à ne pas écraser une image déjà poussée derrière un tag. En dev, cela peut peut-être embêtant. Pendant coté mse-kustom : https://forge.cadoles.com/CNOUS/mse-kustom/issues/261
lgourvenec added 1 commit 2024-12-13 14:21:13 +01:00 
Otherwise, we have a SPOF with the image registry used
lgourvenec requested review from mlamalle 2024-12-13 15:05:32 +01:00
lgourvenec requested review from vfebvre 2024-12-13 15:05:39 +01:00
mlamalle approved these changes 2024-12-16 09:59:58 +01:00
pcaseiro approved these changes 2024-12-16 14:07:01 +01:00
pcaseiro merged commit c97266c272 into unstable 2024-12-16 14:07:37 +01:00
pcaseiro deleted branch feat/pull-policy-IfNotPresent 2024-12-16 14:07:39 +01:00
vfebvre approved these changes 2024-12-16 14:07:43 +01:00
pcaseiro commented 2024-12-16 14:09:43 +01:00
Owner
Copy Link

Nouveau tag : https://forge.cadoles.com/CadolesKube/sso-kustom/releases/tag/v1.0.24-unstable

Nouveau tag : https://forge.cadoles.com/CadolesKube/sso-kustom/releases/tag/v1.0.24-unstable
Sign in to join this conversation.
Reviewers
No reviewers
mlamalle
pcaseiro
vfebvre
Labels
Clear labels   
bug

Something is not working

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
question

More information is needed

  
wontfix

This won't be fixed

No Label
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
4 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: CadolesKube/sso-kustom#53
No description provided.
Delete Branch "feat/pull-policy-IfNotPresent"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?