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

refactor(deps): ImagePullPolicy updated in install.go #326

Merged

Conversation

rootxrishabh
Copy link
Member

Signed-off-by: rootxrishabh risrock02@gmail.com

fixes: #307

This lets karmor pull images present locally when installing karmor using the --local flag. This is useful when you want to avoid unnecessary image pulls and speed up pod startup times.

Signed-off-by: rootxrishabh <risrock02@gmail.com>
@kranurag7 kranurag7 requested a review from Vyom-Yadav June 5, 2023 17:56
@daemon1024 daemon1024 self-requested a review June 5, 2023 18:29
Copy link
Member

@daemon1024 daemon1024 left a comment

Choose a reason for hiding this comment

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

LGTM 🚀 Thanks for the PR

@daemon1024
Copy link
Member

@DelusionalOptimist can we keep this handled once we move to new KubeArmor controller? and Helm not sure what exactly is the roadmap for installation.

@rksharma95 how will this be handled when we have Operator based deployment. Would we need these options in the Operator CRD?

Copy link
Contributor

@nyrahul nyrahul left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@rksharma95
Copy link
Contributor

@rksharma95 how will this be handled when we have Operator based deployment. Would we need these options in the Operator CRD?

yes we'll need to handle it separately if we want to have this option in operator based deployment, we would require to add this option in operator crd.

@DelusionalOptimist
Copy link
Member

Folks, this is what the roadmap to helm looks like - kubearmor/KubeArmor#1256
Please add any other points.

@DelusionalOptimist
Copy link
Member

@daemon1024 is this good to merge?

@daemon1024
Copy link
Member

Yep

@DelusionalOptimist DelusionalOptimist merged commit 4d184e4 into kubearmor:main Jun 26, 2023
@rootxrishabh rootxrishabh deleted the UpdateImagePullPolicy branch June 27, 2023 17:48
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.

karmor install --local does not set ImagePullPolicy to IfNotPresent for all containers
5 participants