codecov-io edited a comment on pull request #1381:
URL: https://github.com/apache/apisix-dashboard/pull/1381#issuecomment-768101272
#
[Codecov](https://codecov.io/gh/apache/apisix-dashboard/pull/1381?src=pr&el=h1)
Report
> Merging
[#1381](https://codecov.io/gh/apache/apisix-dashboard/p
codecov-io edited a comment on pull request #1381:
URL: https://github.com/apache/apisix-dashboard/pull/1381#issuecomment-768101272
#
[Codecov](https://codecov.io/gh/apache/apisix-dashboard/pull/1381?src=pr&el=h1)
Report
> Merging
[#1381](https://codecov.io/gh/apache/apisix-dashboard/p
guoqqqi commented on a change in pull request #160:
URL: https://github.com/apache/apisix-website/pull/160#discussion_r565089939
##
File path: website/src/css/customTheme.css
##
@@ -8,10 +8,6 @@
--ifm-color-primary-darker: #9a2521;
--ifm-color-shadow--lw: #7f1e1b;
}
-.na
guoqqqi commented on a change in pull request #160:
URL: https://github.com/apache/apisix-website/pull/160#discussion_r565091046
##
File path: website/src/css/customTheme.css
##
@@ -8,10 +8,6 @@
--ifm-color-primary-darker: #9a2521;
--ifm-color-shadow--lw: #7f1e1b;
}
-.na
guoqqqi commented on a change in pull request #160:
URL: https://github.com/apache/apisix-website/pull/160#discussion_r565089939
##
File path: website/src/css/customTheme.css
##
@@ -8,10 +8,6 @@
--ifm-color-primary-darker: #9a2521;
--ifm-color-shadow--lw: #7f1e1b;
}
-.na
codecov-io commented on pull request #1381:
URL: https://github.com/apache/apisix-dashboard/pull/1381#issuecomment-768101272
#
[Codecov](https://codecov.io/gh/apache/apisix-dashboard/pull/1381?src=pr&el=h1)
Report
> Merging
[#1381](https://codecov.io/gh/apache/apisix-dashboard/pull/138
fukiki commented on pull request #3335:
URL: https://github.com/apache/apisix/pull/3335#issuecomment-768098488
About the bug of `openresty-debug`, there are two suggestions:
1. Add conditions: when OpenResty version is not 1.19 , allow to use Lua 5.1
2. Modify document: introduce the b
juzhiyuan commented on issue #158:
URL: https://github.com/apache/apisix-website/issues/158#issuecomment-768095186
Yes, it's related to V2.
This is an automated message from the Apache Git Service.
To respond to the message,
spacewander commented on a change in pull request #3300:
URL: https://github.com/apache/apisix/pull/3300#discussion_r565077667
##
File path: apisix/balancer/ewma.lua
##
@@ -135,16 +153,23 @@ local function _ewma_find(ctx, up_nodes)
return nil, 'up_nodes trans error'
Yiyiyimu opened a new issue #3434:
URL: https://github.com/apache/apisix/issues/3434
With #3402 got merged, we also need to update the example on grafana
official, as presents in prometheus documentation.
This is an automate
spacewander opened a new pull request #3433:
URL: https://github.com/apache/apisix/pull/3433
Let's use 'done' all the way to make tests happy.
Fix #3432.
Signed-off-by: spacewander
### What this PR does / why we need it:
### Pre-submission checklist:
Jaycean opened a new pull request #1381:
URL: https://github.com/apache/apisix-dashboard/pull/1381
Please answer these questions before submitting a pull request
- Why submit this pull request?
- [ ] Bugfix
- [x] New feature provided
- [ ] Improve performance
- [ ] Backpor
KishaniKandasamy commented on issue #158:
URL: https://github.com/apache/apisix-website/issues/158#issuecomment-768078045
>
>
> **After upgrading to V2, Here is some CSS th
juzhiyuan commented on issue #1380:
URL:
https://github.com/apache/apisix-dashboard/issues/1380#issuecomment-768076672
cc @starsz @LiteSun to know this tip.
This is an automated message from the Apache Git Service.
To respon
juzhiyuan commented on pull request #160:
URL: https://github.com/apache/apisix-website/pull/160#issuecomment-768075802
@guoqqqi Please review when you have time.
This is an automated message from the Apache Git Service.
To r
KishaniKandasamy opened a new pull request #160:
URL: https://github.com/apache/apisix-website/pull/160
Fixes: #158
Changes:
I've updated customTheme.css
This is an automated message from the Apache Git
spacewander commented on pull request #3335:
URL: https://github.com/apache/apisix/pull/3335#issuecomment-768074923
The problem occurs with `openresty-debug`. Since people might use
openresty-debug package, the robust way is to avoid using LuaJIT unless you are
using OpenResty 1.19.
membphis commented on pull request #3335:
URL: https://github.com/apache/apisix/pull/3335#issuecomment-768072871
@spacewander told me that, this is a bug of openresty `1.15` or `1.17` .
do you have any suggestions about this?
```shell
luajit: lj_asm_x86.h:2819: asm_loop_fix
spacewander merged pull request #3417:
URL: https://github.com/apache/apisix/pull/3417
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
This is an automated email from the ASF dual-hosted git repository.
spacewander pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/apisix.git.
from 8d61d31 fix: proxy_set_header Host $host may lose port info (#2079)
add d752b6f feat: allow customizing l
yongboy closed issue #3423:
URL: https://github.com/apache/apisix/issues/3423
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 sp
yongboy commented on issue #3423:
URL: https://github.com/apache/apisix/issues/3423#issuecomment-768063466
Yes :))
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub a
fukiki commented on a change in pull request #3335:
URL: https://github.com/apache/apisix/pull/3335#discussion_r565053974
##
File path: bin/apisix
##
@@ -1,36 +1,40 @@
-#!/usr/bin/env lua
+#!/bin/bash
---
--- Licensed to the Apache Software Foundation (ASF) under one or more
imjoey commented on issue #1380:
URL:
https://github.com/apache/apisix-dashboard/issues/1380#issuecomment-768056790
@Jaycean good catch. I believe that would be very helpful for every
contributor. Your PR is greatly welcome. Thanks.
---
Junnplus edited a comment on pull request #213:
URL:
https://github.com/apache/apisix-ingress-controller/pull/213#issuecomment-768044322
@tokers `installCRDs` of kong is for compatibility helm2. this chart require
helm 3.0+.
see more in [chart best
practices](https://helm.sh/docs/c
imjoey merged pull request #1379:
URL: https://github.com/apache/apisix-dashboard/pull/1379
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 t
This is an automated email from the ASF dual-hosted git repository.
majunjie pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git
The following commit(s) were added to refs/heads/master by this push:
new e362009 fix: update location of man
Firstsawyou commented on pull request #3430:
URL: https://github.com/apache/apisix/pull/3430#issuecomment-768047761
LGTM.
This is an automated message from the Apache Git Service.
To respond to the message, please log on to G
codecov-io edited a comment on pull request #1372:
URL: https://github.com/apache/apisix-dashboard/pull/1372#issuecomment-766654973
#
[Codecov](https://codecov.io/gh/apache/apisix-dashboard/pull/1372?src=pr&el=h1)
Report
> Merging
[#1372](https://codecov.io/gh/apache/apisix-dashboard/p
Junnplus edited a comment on pull request #213:
URL:
https://github.com/apache/apisix-ingress-controller/pull/213#issuecomment-768044322
@tokers `installCRDs` of kong is for compatibility helm2. this chart require
helm 3.0+.
see more in [chart best
practices](https://helm.sh/docs/c
Junnplus edited a comment on pull request #213:
URL:
https://github.com/apache/apisix-ingress-controller/pull/213#issuecomment-768044322
@tokers `installCRDs` of kong is for compatibility helm2. this chart require
helm 3.0+.
see more in
[chart_best_practices](https://helm.sh/docs/c
Junnplus commented on pull request #213:
URL:
https://github.com/apache/apisix-ingress-controller/pull/213#issuecomment-768044322
@tokers `installCRDs` of kong is for compatibility helm2. this chart require
helm 3.0+.
> With the arrival of Helm 3, we removed the old crd-install hook
Jaycean opened a new issue #1380:
URL: https://github.com/apache/apisix-dashboard/issues/1380
# Feature request
## Please describe your feature
- In the process of submitting my contribution, I found that there was no
local documentation for setting up E2E test environment, an
tokers commented on pull request #213:
URL:
https://github.com/apache/apisix-ingress-controller/pull/213#issuecomment-768042163
@Junnplus The installation of CRDs (and the ClusterRole) should be optional,
if we have multiple Ingress controller Deployments, only the first installation
shou
Junnplus commented on a change in pull request #213:
URL:
https://github.com/apache/apisix-ingress-controller/pull/213#discussion_r565038361
##
File path: charts/apisix-ingress-controller/templates/rbac.yaml
##
@@ -157,5 +153,6 @@ roleRef:
name: {{ .Release.Name }}-clusterr
tokers commented on a change in pull request #213:
URL:
https://github.com/apache/apisix-ingress-controller/pull/213#discussion_r565036970
##
File path: charts/apisix-ingress-controller/templates/rbac.yaml
##
@@ -157,5 +153,6 @@ roleRef:
name: {{ .Release.Name }}-clusterrol
tokers commented on a change in pull request #213:
URL:
https://github.com/apache/apisix-ingress-controller/pull/213#discussion_r565036628
##
File path: charts/apisix-ingress-controller/Chart.yaml
##
@@ -15,13 +15,14 @@
# limitations under the License.
#
apiVersion: v2
-nam
tokers commented on a change in pull request #213:
URL:
https://github.com/apache/apisix-ingress-controller/pull/213#discussion_r565036280
##
File path: charts/apisix-ingress-controller/Chart.yaml
##
@@ -15,13 +15,14 @@
# limitations under the License.
#
apiVersion: v2
-nam
tokers commented on issue #211:
URL:
https://github.com/apache/apisix-ingress-controller/issues/211#issuecomment-768039066
@lianghao208 By default APISIX has the source IP limitations, you need to
change the `allow_admin` configuration.
You can see
https://github.com/apache/apisix-
membphis commented on pull request #1245:
URL: https://github.com/apache/apisix-dashboard/pull/1245#issuecomment-768037066
wow, Congratulations this PR was merged
This is an automated message from the Apache Git Service.
To
spacewander merged pull request #2079:
URL: https://github.com/apache/apisix/pull/2079
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
This is an automated email from the ASF dual-hosted git repository.
spacewander pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/apisix.git.
from ea8eef8 fix: when called checker:get_target_status got err, we should
log it~ (#3424)
add 8d61d31 fix: p
codecov-io commented on pull request #1379:
URL: https://github.com/apache/apisix-dashboard/pull/1379#issuecomment-768024996
#
[Codecov](https://codecov.io/gh/apache/apisix-dashboard/pull/1379?src=pr&el=h1)
Report
> Merging
[#1379](https://codecov.io/gh/apache/apisix-dashboard/pull/137
Yiyiyimu commented on issue #3432:
URL: https://github.com/apache/apisix/issues/3432#issuecomment-768024551
@spacewander do you know the reason of this
This is an automated message from the Apache Git Service.
To respond to t
Yiyiyimu opened a new issue #3432:
URL: https://github.com/apache/apisix/issues/3432
### Issue description
According to
https://github.com/apache/apisix/pull/3404/checks?check_run_id=1774158810
```
# Failed test 'TEST 3: reload plugins when attributes changed -
response_body
juzhiyuan merged pull request #1245:
URL: https://github.com/apache/apisix-dashboard/pull/1245
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abov
This is an automated email from the ASF dual-hosted git repository.
juzhiyuan pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git.
from fa080a8 fix: some apis bypassed authentication (#1373)
add 9cb9aa7 feat(BE): Export route from Op
liuxiran opened a new pull request #1379:
URL: https://github.com/apache/apisix-dashboard/pull/1379
Please answer these questions before submitting a pull request
- Why submit this pull request?
- [ ] Bugfix
___
### Bugfix
- How to fix?
update the doc due to the
spacewander commented on issue #3423:
URL: https://github.com/apache/apisix/issues/3423#issuecomment-768013430
Consider solved?
This is an automated message from the Apache Git Service.
To respond to the message, please log o
spacewander merged pull request #3424:
URL: https://github.com/apache/apisix/pull/3424
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
This is an automated email from the ASF dual-hosted git repository.
spacewander pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/apisix.git.
from 2b532f0 feat: proxy-rewrite host support host with port (#3428)
add ea8eef8 fix: when called checker:get_
Junnplus commented on a change in pull request #213:
URL:
https://github.com/apache/apisix-ingress-controller/pull/213#discussion_r565013565
##
File path: charts/apisix-ingress-controller/templates/rbac.yaml
##
@@ -157,5 +153,6 @@ roleRef:
name: {{ .Release.Name }}-clusterr
spacewander commented on issue #3431:
URL: https://github.com/apache/apisix/issues/3431#issuecomment-768007516
> Nginx output filters may be called multiple times for a single request
because response body may be delivered in chunks. Thus, the Lua code specified
by in this directive may al
spacewander closed issue #3431:
URL: https://github.com/apache/apisix/issues/3431
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 th
lianghao208 commented on issue #211:
URL:
https://github.com/apache/apisix-ingress-controller/issues/211#issuecomment-768007123
@tokers I change the admin port to 9180, the controller log show 403:
```bash
[root@VM-187-41-centos ~]# kc logs ingress-controller-79dbd87678-7qzwb -n
ing
tokers commented on a change in pull request #213:
URL:
https://github.com/apache/apisix-ingress-controller/pull/213#discussion_r565008865
##
File path: charts/apisix-ingress-controller/templates/rbac.yaml
##
@@ -157,5 +153,6 @@ roleRef:
name: {{ .Release.Name }}-clusterrol
Junnplus commented on a change in pull request #213:
URL:
https://github.com/apache/apisix-ingress-controller/pull/213#discussion_r565008227
##
File path: charts/apisix-ingress-controller/Chart.yaml
##
@@ -15,13 +15,14 @@
# limitations under the License.
#
apiVersion: v2
-n
zhangpeix opened a new issue #3431:
URL: https://github.com/apache/apisix/issues/3431
### Issue description
### Environment
* apisix version (cmd: `apisix version`):2.2
* OS: (cmd: `uname -a`) centeros7
* OpenResty / Nginx version: (cmd: `nginx -V` or `openresty -V`)
tokers commented on a change in pull request #213:
URL:
https://github.com/apache/apisix-ingress-controller/pull/213#discussion_r565007460
##
File path: charts/apisix-ingress-controller/templates/rbac.yaml
##
@@ -157,5 +153,6 @@ roleRef:
name: {{ .Release.Name }}-clusterrol
yongboy commented on a change in pull request #3424:
URL: https://github.com/apache/apisix/pull/3424#discussion_r565006701
##
File path: apisix/balancer.lua
##
@@ -55,10 +55,12 @@ local function fetch_health_nodes(upstream, checker)
local port = upstream.checks and upstrea
lianghao208 commented on issue #211:
URL:
https://github.com/apache/apisix-ingress-controller/issues/211#issuecomment-768002249
@tokers
```bash
config.yaml: |-
apisix:
node_listen: 9080 # APISIX listening port
enable_heartbeat: true
tokers commented on a change in pull request #213:
URL:
https://github.com/apache/apisix-ingress-controller/pull/213#discussion_r565005993
##
File path: charts/apisix-ingress-controller/templates/rbac.yaml
##
@@ -14,11 +14,7 @@
# See the License for the specific language gove
Junnplus commented on a change in pull request #213:
URL:
https://github.com/apache/apisix-ingress-controller/pull/213#discussion_r565005939
##
File path: charts/apisix-ingress-controller/templates/rbac.yaml
##
@@ -157,5 +153,6 @@ roleRef:
name: {{ .Release.Name }}-clusterr
tokers commented on issue #211:
URL:
https://github.com/apache/apisix-ingress-controller/issues/211#issuecomment-768001332
What's the configuration of apisix (config.y'all), it seems 9080 is not the
admin port.
This is an a
yongboy commented on pull request #3424:
URL: https://github.com/apache/apisix/pull/3424#issuecomment-768000686
Now calling `checker:get_target_status` get error, the log example as below:
```bash
2021/01/27 11:34:52 [error] 8#8: *2315 [lua] balancer.lua:73:
fetch_health_nodes():
spacewander commented on a change in pull request #3424:
URL: https://github.com/apache/apisix/pull/3424#discussion_r565004841
##
File path: apisix/balancer.lua
##
@@ -55,10 +55,12 @@ local function fetch_health_nodes(upstream, checker)
local port = upstream.checks and ups
Junnplus commented on a change in pull request #213:
URL:
https://github.com/apache/apisix-ingress-controller/pull/213#discussion_r565004318
##
File path: charts/apisix-ingress-controller/templates/rbac.yaml
##
@@ -14,11 +14,7 @@
# See the License for the specific language go
tokers commented on a change in pull request #213:
URL:
https://github.com/apache/apisix-ingress-controller/pull/213#discussion_r565001676
##
File path: charts/apisix-ingress-controller/templates/rbac.yaml
##
@@ -157,5 +153,6 @@ roleRef:
name: {{ .Release.Name }}-clusterrol
tokers commented on a change in pull request #213:
URL:
https://github.com/apache/apisix-ingress-controller/pull/213#discussion_r565000280
##
File path: charts/apisix-ingress-controller/templates/rbac.yaml
##
@@ -14,11 +14,7 @@
# See the License for the specific language gove
tokers commented on a change in pull request #213:
URL:
https://github.com/apache/apisix-ingress-controller/pull/213#discussion_r564998406
##
File path: charts/apisix-ingress-controller/Chart.yaml
##
@@ -15,13 +15,14 @@
# limitations under the License.
#
apiVersion: v2
-nam
tokers commented on a change in pull request #213:
URL:
https://github.com/apache/apisix-ingress-controller/pull/213#discussion_r564998406
##
File path: charts/apisix-ingress-controller/Chart.yaml
##
@@ -15,13 +15,14 @@
# limitations under the License.
#
apiVersion: v2
-nam
Junnplus commented on a change in pull request #20:
URL: https://github.com/apache/apisix-helm-chart/pull/20#discussion_r564996717
##
File path: chart/apisix/values.yaml
##
@@ -188,5 +190,5 @@ nodeSelector: {}
tolerations: []
-affinity: {}
-
+affinity:
Review comment:
scn7th commented on pull request #214:
URL:
https://github.com/apache/apisix-ingress-controller/pull/214#issuecomment-767984452
> @scn7th The docs/images/modules.png should be changed since now we
integrated ingress-types to the apisix-ingress-controller.
Ok, let me fix it.
--
Junnplus commented on a change in pull request #213:
URL:
https://github.com/apache/apisix-ingress-controller/pull/213#discussion_r564993640
##
File path: charts/apisix-ingress-controller/Chart.yaml
##
@@ -15,13 +15,14 @@
# limitations under the License.
#
apiVersion: v2
-n
idbeta commented on a change in pull request #3404:
URL: https://github.com/apache/apisix/pull/3404#discussion_r564992390
##
File path: t/chaos/kill-etcd_test.go
##
@@ -0,0 +1,173 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor lice
Junnplus commented on a change in pull request #213:
URL:
https://github.com/apache/apisix-ingress-controller/pull/213#discussion_r564992048
##
File path: charts/apisix-ingress-controller/README.md
##
@@ -0,0 +1,44 @@
+
+
+# Apache APISIX Ingress Controller Helm Chart
+
+## Pr
Junnplus commented on a change in pull request #213:
URL:
https://github.com/apache/apisix-ingress-controller/pull/213#discussion_r564990948
##
File path: charts/apisix-ingress-controller/templates/rbac.yaml
##
@@ -14,11 +14,7 @@
# See the License for the specific language go
yongboy commented on a change in pull request #3424:
URL: https://github.com/apache/apisix/pull/3424#discussion_r564990661
##
File path: apisix/balancer.lua
##
@@ -55,10 +55,12 @@ local function fetch_health_nodes(upstream, checker)
local port = upstream.checks and upstrea
sober-wang commented on issue #3367:
URL: https://github.com/apache/apisix/issues/3367#issuecomment-767980544
No, My means is when the etcd connect is failed ,the apisix pod should print
error log in the stdout or stderr .
apisix server can used error-logger plugin
---
codecov-io edited a comment on pull request #213:
URL:
https://github.com/apache/apisix-ingress-controller/pull/213#issuecomment-767550262
#
[Codecov](https://codecov.io/gh/apache/apisix-ingress-controller/pull/213?src=pr&el=h1)
Report
> Merging
[#213](https://codecov.io/gh/apache/api
codecov-io edited a comment on pull request #213:
URL:
https://github.com/apache/apisix-ingress-controller/pull/213#issuecomment-767550262
#
[Codecov](https://codecov.io/gh/apache/apisix-ingress-controller/pull/213?src=pr&el=h1)
Report
> Merging
[#213](https://codecov.io/gh/apache/api
Firstsawyou commented on pull request #2586:
URL: https://github.com/apache/apisix/pull/2586#issuecomment-767976195
Hi，@tashiya-menuri
The test case has not been fixed yet, do you need help?
This is an automated message
spacewander opened a new pull request #3430:
URL: https://github.com/apache/apisix/pull/3430
Fix #3336
Signed-off-by: spacewander
### What this PR does / why we need it:
### Pre-submission checklist:
* [x] Did you explain what problem does this PR solve? Or w
Junnplus commented on a change in pull request #213:
URL:
https://github.com/apache/apisix-ingress-controller/pull/213#discussion_r564983327
##
File path: charts/apisix-ingress-controller/README.md
##
@@ -0,0 +1,44 @@
+
+
+# Apache APISIX Ingress Controller Helm Chart
+
+## Pr
Junnplus commented on a change in pull request #213:
URL:
https://github.com/apache/apisix-ingress-controller/pull/213#discussion_r564981885
##
File path: charts/apisix-ingress-controller/Chart.yaml
##
@@ -15,13 +15,14 @@
# limitations under the License.
#
apiVersion: v2
-n
juzhiyuan commented on issue #159:
URL: https://github.com/apache/apisix-website/issues/159#issuecomment-767970041
This issue relies on #157, should resolve #157 first.
This is an automated message from the Apache Git Service
Junnplus commented on a change in pull request #213:
URL:
https://github.com/apache/apisix-ingress-controller/pull/213#discussion_r564980836
##
File path: charts/apisix-ingress-controller/templates/rbac.yaml
##
@@ -157,5 +153,6 @@ roleRef:
name: {{ .Release.Name }}-clusterr
spacewander merged pull request #3428:
URL: https://github.com/apache/apisix/pull/3428
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
This is an automated email from the ASF dual-hosted git repository.
spacewander pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/apisix.git
The following commit(s) were added to refs/heads/master by this push:
new 2b532f0 feat: proxy-rewrite host support h
spacewander closed issue #3136:
URL: https://github.com/apache/apisix/issues/3136
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 th
spacewander commented on a change in pull request #3417:
URL: https://github.com/apache/apisix/pull/3417#discussion_r564975536
##
File path: apisix/cli/ops.lua
##
@@ -121,6 +122,19 @@ local function version()
end
+local function get_lua_path(conf)
+if conf then
+
lianghao208 commented on issue #211:
URL:
https://github.com/apache/apisix-ingress-controller/issues/211#issuecomment-767961519
@tokers I just configured the pod ip:port of apisix gateway, the controller
log shows as follow:
```bash
[root@VM-187-41-centos ~]# kc logs ingress-controll
TkClark commented on issue #3426:
URL: https://github.com/apache/apisix/issues/3426#issuecomment-767960994
now useing apisix-dashboard, we were in the testing,
This is an automated message from the Apache Git Service.
To resp
LiteSun commented on a change in pull request #1376:
URL: https://github.com/apache/apisix-dashboard/pull/1376#discussion_r564973620
##
File path: web/cypress/integration/route/create-route-with-upstream.spec.js
##
@@ -88,6 +89,7 @@ context('Create Route with Upstream', () => {
tokers commented on a change in pull request #3415:
URL: https://github.com/apache/apisix/pull/3415#discussion_r564971324
##
File path: .github/workflows/build.yml
##
@@ -54,7 +54,7 @@ jobs:
submodules: recursive
- name: Linux Get dependencies
-run:
tokers commented on a change in pull request #213:
URL:
https://github.com/apache/apisix-ingress-controller/pull/213#discussion_r564965301
##
File path: charts/apisix-ingress-controller/README.md
##
@@ -0,0 +1,44 @@
+
+
+# Apache APISIX Ingress Controller Helm Chart
+
+## Prer
tokers commented on pull request #214:
URL:
https://github.com/apache/apisix-ingress-controller/pull/214#issuecomment-767950552
@scn7th The docs/images/modules.png should be changed since now we
integrated ingress-types to the apisix-ingress-controller.
--
Yiyiyimu commented on a change in pull request #3404:
URL: https://github.com/apache/apisix/pull/3404#discussion_r564963975
##
File path: t/chaos/kill-etcd_test.go
##
@@ -51,6 +52,7 @@ func caseCheck(tc httpTestCase) {
}
if req == nil {
+ fmt.Pri
tokers commented on a change in pull request #3404:
URL: https://github.com/apache/apisix/pull/3404#discussion_r564960763
##
File path: t/chaos/kill-etcd_test.go
##
@@ -51,6 +52,7 @@ func caseCheck(tc httpTestCase) {
}
if req == nil {
+ fmt.Print
tokers commented on pull request #26:
URL: https://github.com/apache/apisix-helm-chart/pull/26#issuecomment-767944599
@TkClark This PR is similar with
https://github.com/apache/apisix-helm-chart/pull/20. Also conflicts should be
resolved.
-
1 - 100 of 252 matches
Mail list logo
