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

Customized pullPolicy for verifier, tenant, registrar and agent #64

Merged
merged 1 commit into from
Feb 1, 2024

Conversation

maugustosilva
Copy link
Contributor

I have also squeezed in a small fix for Makefile (re: custom kubeconfig)

@@ -161,11 +161,15 @@ global:
# Default image is the from quay, default tag is tag the chart appVersion
image:
repository: quay.io/keylime/keylime_tenant
# Default image pull policy
pullPolicy: IfNotPresent
Copy link

Choose a reason for hiding this comment

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

Wondering whether "always" might not be a better default policy. Especially for people who like "latest" or do quick development.
But certainly having any default pull policy is better than none.

Copy link

@galmasi galmasi left a comment

Choose a reason for hiding this comment

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

LGTM.

The Makefile changes might have been better in a separate PR, but let that not hold merging.

@galmasi galmasi merged commit 0c904f8 into keylime:main Feb 1, 2024
2 checks passed
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