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(runner-config): ✨ override image pull policy default value #1422

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

yroffin
Copy link

@yroffin yroffin commented Aug 20, 2024

default imagePullPolicy is set to v1.PullIfNotPresent and imagePullPolicy don't exist in CRD

target of this pull request is to add imagePullPolicy in CRD and use it to override default PullIfNotPresent value

@yroffin yroffin changed the title feat(runner-config): ✨ set image pull policy to alwasy when … feat(runner-config): ✨ set image pull policy to always when … Aug 20, 2024
@yroffin yroffin changed the title feat(runner-config): ✨ set image pull policy to always when … feat(runner-config): ✨ override image pull policy default value Aug 21, 2024
@yroffin yroffin force-pushed the setup-runner-img-pull-policy branch from e4ac25b to d60ffff Compare August 21, 2024 09:36
@akselleirv akselleirv self-requested a review September 15, 2024 08:41
Copy link
Collaborator

@akselleirv akselleirv left a comment

Choose a reason for hiding this comment

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

Thank you for the feature and the test! Added some suggestions.

charts/tofu-controller/README.md Outdated Show resolved Hide resolved
api/v1alpha2/reference_types.go Outdated Show resolved Hide resolved
@akselleirv
Copy link
Collaborator

Can you please rebase on main after #1439 has been merged? That should fix the pipeline errors.

@yroffin yroffin force-pushed the setup-runner-img-pull-policy branch from e510ca1 to 3f2a192 Compare September 17, 2024 07:11
@yroffin
Copy link
Author

yroffin commented Sep 17, 2024

@akselleirv job is done, i've rebase my pull request and take in account your propositions

@akselleirv
Copy link
Collaborator

Thanks @yroffin ! Last thing, can you please run make manifests again to generate the CRD.

@yroffin
Copy link
Author

yroffin commented Sep 18, 2024

@akselleirv job done, do you think it will be store in next milestone ? on my project i will be happy to have this feature

@akselleirv
Copy link
Collaborator

@yroffin can you please the pipeline https://github.com/flux-iac/tofu-controller/actions/runs/10917321807/job/30326377700?pr=1422 ?

Regarding a new release, it depends on a follow up in issue #1234

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