Open voor opened 2 years ago
@voor could you combine multiple invocations into one?
kbld -f <(helm template bitnami/postgresql --set metrics.enabled=true --set postgresqlDatabase=bob) \
-f <(helm template bitnami/postgresql --set postgresqlDatabase=bob) \
--imgpkg-lock-output images.yml >/dev/null
Not effectively, since they're also being output to a sample folder and/or part of loops that run at other times.
For the non-helm chart related things, what I've been doing is putting the image as something like:
spec:
initContainers:
- image: change-me-later
And using kbld config:
apiVersion: kbld.k14s.io/v1alpha1
kind: Config
minimumRequiredVersion: 0.24.0
overrides:
- image: change-me-later
newImage: registry.example.com/project/now-i-am-a-container:latest
Describe the problem/challenge you have We have different environments that use different configuration and this shares common manifests, in particular things like Helm charts. We generate our
ImagesLock
file at the same time we output samples that are committed into the repository for readability purposes (sometimes it can be hard to decipher extremely abstracted ytt templates). This loops through various configurations, for example turning on or off a prometheus metrics exporter or a cluster autoscaler. Since these manifests are all part of the same bundle it'd be nice if they had the same pattern of./imgpkg/images.yml
being referenced in the App CR. However, if the "last" sample isn't using a container then that container is excluded from theImagesLock
file since kbld will not append to an existing one.Try it yourself:
First output will look something like this:
Second one will be smaller, missing the metrics exporter (which was removed)
Describe the solution you'd like Add something like
--append-imgpkg-lock-output
that would understand there is most likely already aImagesLock
at the provided location to--imgpkg-lock-output
and instead of rewriting the file it would just look to see if the provided reference already existed, do a no-op, or update it, or add it to the end.Anything else you would like to add: If kbld would process this better as just appending full yml files, so you end up with something like:
that's fine too
Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible" 👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you want to help working on this issue.