bitnami
[bitnami/nginx] Add `envsubst` to nginx
Description of the change
Addresses #34957
Benefits
Documented in associated issue.
Possible drawbacks
Documented in associated issue.
Applicable issues
Fixes #34957
Additional information
Hi!
Thank you so much for the PR. The verification is not being triggered because of this
Changes should affect to only one asset. You are currently modifying: kubectl nginx
Could you split the PR for kubectl and nginx?
Could you split the PR for kubectl and nginx?
Sure, sorry didn't realize that.
@javsalgar all fixed, should be good to go now on this one, will open a second PR.
Hi @djschny,
As mentioned in #55365 , we have created an internal task to evaluate the gettext-base package addition in all distributions in which we provide NGINX images as part of Tanzu Application Catalog. We will keep you posted.
In the meantime, please extend our image with the required packages on your side.
Hi @djschny,
Sorry for the delay. I have been checking this and I have found we are adding the gettext package instead of gettext-base in other images like appsmith. Can you please update your PR? I don't have any other suggestion/requirement to this.
Sorry @djschny, I have just noticed your changes are for an old version of the Dockerfiles based on debian-11, but we have recently updated the base image to debian-12. I have added the gettext package for the next image revision, so I'm closing this PR. Sorry for the inconveniences.