You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@apisix.apache.org by "nic-6443 (via GitHub)" <gi...@apache.org> on 2025/08/29 03:13:21 UTC
[PR] chore: upgrade etcd chart to latest version and use latest image [apisix-helm-chart]
nic-6443 opened a new pull request, #868:
URL: https://github.com/apache/apisix-helm-chart/pull/868
(no comment)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore: upgrade etcd chart to latest version and use latest image [apisix-helm-chart]
Posted by "nic-6443 (via GitHub)" <gi...@apache.org>.
nic-6443 commented on PR #868:
URL: https://github.com/apache/apisix-helm-chart/pull/868#issuecomment-3235799278
> Why do we need to use latest? Different etcd chart versions are tied to specific etcd application versions, and an older APISIX chart may not be fully compatible with the newest etcd release. Using latest also introduces unpredictability in deployments.
@AlinsRan Due to Bitnami's adjustment of the open-source strategy for Docker images: https://github.com/bitnami/containers/issues/83267, non-latest images will be unavailable for a period of time in the future.
> Bitnami will begin deprecating support for non-hardened, Debian-based software images in its free tier and will gradually remove non-latest tags from the public catalog. As a result, community users will have access to a reduced number of hardened images. These images are published only under the “latest” tag and are intended for development purposes
Therefore, unless we replace the etcd image with a historical image from the docker.io/bitnamilegacy repository, we can only use the latest image. Considering that our current requirement for etcd in the apisix helm chart is for quickly experiencing apisix and is only used for development and testing purposes without providing production guarantees for the etcd helm chart itself, we ultimately chose to use the latest image.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore: upgrade etcd chart to latest version and use latest image [apisix-helm-chart]
Posted by "AlinsRan (via GitHub)" <gi...@apache.org>.
AlinsRan commented on PR #868:
URL: https://github.com/apache/apisix-helm-chart/pull/868#issuecomment-3235637898
Why do we need to use latest? Different etcd chart versions are tied to specific etcd application versions, and an older APISIX chart may not be fully compatible with the newest etcd release. Using latest also introduces unpredictability in deployments.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore: upgrade etcd chart to latest version and use latest image [apisix-helm-chart]
Posted by "nic-6443 (via GitHub)" <gi...@apache.org>.
nic-6443 commented on code in PR #868:
URL: https://github.com/apache/apisix-helm-chart/pull/868#discussion_r2309035698
##########
.github/workflows/ci.yaml:
##########
@@ -29,11 +29,12 @@ jobs:
--name ct-lint \
--volume $PWD:/workdir \
--workdir /workdir/charts/apisix \
- quay.io/helmpack/chart-testing:v3.4.0 sh -c 'helm dependency update \
+ quay.io/helmpack/chart-testing:v3.13.0 sh -c 'helm dependency update \
&& cd ../.. \
&& helm repo add bitnami https://charts.bitnami.com/bitnami \
&& helm repo add apisix https://charts.apiseven.com \
&& ct lint \
+ --validate-maintainers=false \
Review Comment:
ref: https://github.com/helm/chart-testing/issues/464#issuecomment-1870661766
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore: upgrade etcd chart to latest version and use latest image [apisix-helm-chart]
Posted by "nic-6443 (via GitHub)" <gi...@apache.org>.
nic-6443 merged PR #868:
URL: https://github.com/apache/apisix-helm-chart/pull/868
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org