CadolesKube/sso-kustom
15
0
Fork 0
Code Issues 19 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 4 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
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
4 Participants
Notifications
Due Date
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?