You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@apisix.apache.org by "neil85 (via GitHub)" <gi...@apache.org> on 2023/10/31 08:02:09 UTC
[PR] Feat:configure ssl and adding ingress class [apisix-helm-chart]
neil85 opened a new pull request, #657:
URL: https://github.com/apache/apisix-helm-chart/pull/657
(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] feat: support configure gateway.tls and adding ingress class [apisix-helm-chart]
Posted by "AlinsRan (via GitHub)" <gi...@apache.org>.
AlinsRan merged PR #657:
URL: https://github.com/apache/apisix-helm-chart/pull/657
--
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] Feat:configure ssl and adding ingress class [apisix-helm-chart]
Posted by "lingsamuel (via GitHub)" <gi...@apache.org>.
lingsamuel commented on code in PR #657:
URL: https://github.com/apache/apisix-helm-chart/pull/657#discussion_r1378491519
##########
charts/apisix-ingress-controller/README.md:
##########
@@ -161,6 +170,7 @@ The same for container level, you need to set:
| labelsOverride | object | `{}` | Override default labels assigned to Apache APISIX ingress controller resource |
| nameOverride | string | `""` | Default values for apisix-ingress-controller. This is a YAML-formatted file. Declare variables to be passed into your templates. |
| nodeSelector | object | `{}` | |
+| plugins | list | `[]` | Customize the list of APISIX plugins to enable. By default, APISIX's default plugins are automatically used. See [config-default.yaml](https://github.com/apache/apisix/blob/master/conf/config-default.yaml) |
Review Comment:
Is this auto-generated? I can't find this field in values.yaml
--
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] Feat:configure ssl and adding ingress class [apisix-helm-chart]
Posted by "neil85 (via GitHub)" <gi...@apache.org>.
neil85 commented on code in PR #657:
URL: https://github.com/apache/apisix-helm-chart/pull/657#discussion_r1378578122
##########
charts/apisix-ingress-controller/README.md:
##########
@@ -161,6 +170,7 @@ The same for container level, you need to set:
| labelsOverride | object | `{}` | Override default labels assigned to Apache APISIX ingress controller resource |
| nameOverride | string | `""` | Default values for apisix-ingress-controller. This is a YAML-formatted file. Declare variables to be passed into your templates. |
| nodeSelector | object | `{}` | |
+| plugins | list | `[]` | Customize the list of APISIX plugins to enable. By default, APISIX's default plugins are automatically used. See [config-default.yaml](https://github.com/apache/apisix/blob/master/conf/config-default.yaml) |
Review Comment:
my bad i updated helm-docs
--
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