martin-schulze-e2m commented on code in PR #916:
URL: https://github.com/apache/apisix-helm-chart/pull/916#discussion_r2905851654
##########
charts/apisix-ingress-controller/README.md:
##########
@@ -133,7 +133,8 @@ The same for container level, you need to set:
| deployment.annotations | object | `{}` | Add annotations to Apache APISIX
ingress controller resource |
| deployment.image.pullPolicy | string | `"IfNotPresent"` | |
| deployment.image.repository | string | `"apache/apisix-ingress-controller"`
| |
-| deployment.image.tag | string | `"2.0.1"` | |
+| deployment.image.tag | string | `"2.0.0-rc5"` | |
Review Comment:
The CI tests failed due to mismatches between the committed README.md and
the helm-docs generated one. This change should fix the CI error:
```suggestion
| deployment.image.tag | string | `"2.0.1"` | |
```
Alternatively, run `helm-docs` locally and commit that @ArnaudCHT.
##########
charts/apisix-ingress-controller/README.md:
##########
@@ -133,7 +133,8 @@ The same for container level, you need to set:
| deployment.annotations | object | `{}` | Add annotations to Apache APISIX
ingress controller resource |
| deployment.image.pullPolicy | string | `"IfNotPresent"` | |
| deployment.image.repository | string | `"apache/apisix-ingress-controller"`
| |
-| deployment.image.tag | string | `"2.0.1"` | |
+| deployment.image.tag | string | `"2.0.0-rc5"` | |
Review Comment:
The CI tests failed due to mismatches between the committed README.md and
the helm-docs generated one. This change should fix the CI error:
```suggestion
| deployment.image.tag | string | `"2.0.1"` | |
```
Alternatively, you can run `helm-docs` locally and commit that @ArnaudCHT.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]