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

feature: Support specify the image matching rule file to move the container images too #180

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

yidaqiang
Copy link
Contributor

@yidaqiang yidaqiang commented Mar 10, 2023

feature: Support specify the locally image matching rule file to move the container images too.

when I setting the relocateContainerImages: true property in the config file i.e

# leverage .relok8s-images.yaml file inside the Charts to move the container images too
relocateContainerImages: true
source:
   ...
target:
   ...

In order for this option to work it is required that the source Helm Charts includes a .relok8s-images.yaml file with information about where to find the images inside chart.
In fact, We have a lot of chart left over from history, Which Can't modify. To be able to synchronize these charts, I add the imageHintsFile: "example/image-hints.yaml" property in the config file i.e

# leverage locally image-hints.yaml file to move the container images too
relocateContainerImages: true
source:
  repo:
    imageHintsFile: "example/image-hints.yaml"
   ...
target:
   ...

Now we can synchronize chart contains container images without .relok8s-images.yaml file

…ontainer images too, when setting the "relocateContainerImages: true" property in the config file
@yidaqiang yidaqiang changed the title Support specify the image matching rule file the Charts to move the c… feature: Support specify the image matching rule file to move the container images too Mar 10, 2023
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.

1 participant