Skip to content

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.

@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