ci: drop the unused apisix helm repo from CI and release - #352
Conversation
`helm repo add apisix https://charts.apiseven.com` ran in five places across ci.yaml and release.yaml, but no chart consumes it: every dependency in every Chart.yaml and Chart.lock resolves to bitnami or jaegertracing. It was pure failure surface — `helm repo add` fetches index.yaml, and that host is a redirect to apache.github.io served from an IP the runners reach unreliably. It timed out today and failed a release run.
|
Warning Review limit reachedYou’ve reached a temporary PR review limit under our Fair Usage Limits Policy. Next review available in: 32 minutes Limit details: You’ve used the included review currently available. Your 60 included PR review attempts over the past 7 days set your current allowance at 1 review per hour. You’re in a promotional period — use the checkbox below to run this review for free:
On-demand reviews are free for the next 31 days. After that, they cost $0.25 per reviewed file. How can I continue?Run this review now using the option above, or comment You can also wait for the limit to reset, then comment An organization admin can change what happens after included review limits in Billing. How do review limits work?CodeRabbit enforces per-developer PR review limits within each organization. For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
Comment |
The release run on #351 failed at the "Add Helm repo" step:
Nothing in this repo uses that repository. Every
dependencies:entry in everyChart.yamland everyChart.lockresolves tocharts.bitnami.comorjaegertracing.github.io— theapisixrepo has no consumer, inhelm dependencyresolution or anywhere else. It was added in five places (four in
ci.yaml, one inrelease.yaml) and each one is a hard failure point, becausehelm repo addfetchesindex.yamlbefore it will accept the URL.It is also an alias rather than a real repo:
https://charts.apiseven.comis a 301to
https://apache.github.io/apisix-helm-chart/, served from an IP the runners reachunreliably. So every CI and release run has been paying a network round trip to a
host it does not need, with an outage of that host able to fail a release outright —
which is what happened on #351.
Removes all five occurrences. Verified that the remaining shell chains are still
valid (
sh -nover everyrun:block) and that both workflow files still parse.