[GitHub] [apisix-dashboard] juzhiyuan commented on issue #1237: bug: Failed to sync schema with APISIX via schema-sync.lua

2021-01-06 Thread GitBox
juzhiyuan commented on issue #1237: URL: https://github.com/apache/apisix-dashboard/issues/1237#issuecomment-755950318 cc @nic-chen @starsz This is an automated message from the Apache Git Service. To respond to the

[GitHub] [apisix-dashboard] imjoey opened a new issue #1237: bug: Failed to sync schema with APISIX via schema-sync.lua

2021-01-06 Thread GitBox
imjoey opened a new issue #1237: URL: https://github.com/apache/apisix-dashboard/issues/1237 # Bug report ## Describe the bug I would get the following errors at running `./api/build-tools/schema-sync.sh` to sync schema definitions with APISIX. ``` lua:

[GitHub] [apisix-dashboard] imjoey removed a comment on issue #1216: bug: module 'resty.radixtree' not found when call command `http://schema-sync.sh/`

2021-01-06 Thread GitBox
imjoey removed a comment on issue #1216: URL: https://github.com/apache/apisix-dashboard/issues/1216#issuecomment-755928093 @starsz @juzhiyuan I'm afraid that this issue has not been completely fixed. With the latest code from master branch, I still got the following errors: ```

[GitHub] [apisix-dashboard] juzhiyuan commented on issue #1233: docker build fail

2021-01-06 Thread GitBox
juzhiyuan commented on issue #1233: URL: https://github.com/apache/apisix-dashboard/issues/1233#issuecomment-755946725 cc @idbeta This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [apisix] gewei2000 commented on issue #3174: request help: prometheus not support filter by route

2021-01-06 Thread GitBox
gewei2000 commented on issue #3174: URL: https://github.com/apache/apisix/issues/3174#issuecomment-755944964 @spacewander I have already visit route many times, but metrics api `/apisix/prometheus/metrics` can not receive any route data;

[GitHub] [apisix-dashboard] fukiki opened a new issue #1236: bug: Online Debug can not use key-value to enter the integer params

2021-01-06 Thread GitBox
fukiki opened a new issue #1236: URL: https://github.com/apache/apisix-dashboard/issues/1236 # Bug report ## Describe the bug When Online Debug, using key-value to enter `Header Params`, the params validation failed.

[GitHub] [apisix] spacewander commented on issue #1710: bug: Invalid JSON encoding when exporting logs with method body

2021-01-06 Thread GitBox
spacewander commented on issue #1710: URL: https://github.com/apache/apisix/issues/1710#issuecomment-755932907 The problem can't be reproduced in the master now. This is an automated message from the Apache Git Service. To

[GitHub] [apisix-dashboard] juzhiyuan merged pull request #1235: doc: add changelog for v2.3

2021-01-06 Thread GitBox
juzhiyuan merged pull request #1235: URL: https://github.com/apache/apisix-dashboard/pull/1235 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[apisix-dashboard] branch v2.3 updated: doc: add changelog for v2.3 (#1235)

2021-01-06 Thread juzhiyuan
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a commit to branch v2.3 in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git The following commit(s) were added to refs/heads/v2.3 by this push: new d518b47 doc: add changelog for v2.3

[GitHub] [apisix-dashboard] tokers opened a new pull request #1235: doc: add changelog for v2.3

2021-01-06 Thread GitBox
tokers opened a new pull request #1235: URL: https://github.com/apache/apisix-dashboard/pull/1235 Please answer these questions before submitting a pull request - Why submit this pull request? - [ ] Bugfix - [ ] New feature provided - [ ] Improve performance - Related

[GitHub] [apisix] Firstsawyou commented on issue #3211: Feature: implement p2c balancer with random, ewma, leastloaded

2021-01-06 Thread GitBox
Firstsawyou commented on issue #3211: URL: https://github.com/apache/apisix/issues/3211#issuecomment-755930741 If the performance is better, then welcome to submit PR. This is an automated message from the Apache Git

[GitHub] [apisix-dashboard] imjoey commented on issue #1163: Langauge selection issue in Dashboard

2021-01-06 Thread GitBox
imjoey commented on issue #1163: URL: https://github.com/apache/apisix-dashboard/issues/1163#issuecomment-755928643 > This command should under the `apisix-docker` repo, right? @juzhiyuan Yep. This is an automated

[GitHub] [apisix-dashboard] imjoey commented on issue #1216: bug: module 'resty.radixtree' not found when call command `http://schema-sync.sh/`

2021-01-06 Thread GitBox
imjoey commented on issue #1216: URL: https://github.com/apache/apisix-dashboard/issues/1216#issuecomment-755928093 @starsz @juzhiyuan I'm afraid that this issue has not been completely fixed. With the latest code from master branch, I still got the following errors: ``` lua:

[GitHub] [apisix] membphis commented on pull request #1971: bugfix: create the health checker in `access` phase, allow to call `y…

2021-01-06 Thread GitBox
membphis commented on pull request #1971: URL: https://github.com/apache/apisix/pull/1971#issuecomment-755926244 we can close this PR after the new PR is created ^_^ This is an automated message from the Apache Git Service.

[GitHub] [apisix-dashboard] LLLLimbo opened a new issue #1234: Cannot read property 'upstream_id' of undefined

2021-01-06 Thread GitBox
imbo opened a new issue #1234: URL: https://github.com/apache/apisix-dashboard/issues/1234 1. I added a route in this way `curl --location --request PUT 'http://192.168.2.102:9080/apisix/admin/routes/1' \ --header 'Content-Type: application/json' \ --header 'X-API-KEY:

[GitHub] [apisix-dashboard] juzhiyuan commented on pull request #1220: feat: add plugin icon

2021-01-06 Thread GitBox
juzhiyuan commented on pull request #1220: URL: https://github.com/apache/apisix-dashboard/pull/1220#issuecomment-755924492 @imjoey Just done  Thanks a lot! This is an automated message from the Apache Git Service. To

[GitHub] [apisix-dashboard] juzhiyuan commented on issue #1163: Langauge selection issue in Dashboard

2021-01-06 Thread GitBox
juzhiyuan commented on issue #1163: URL: https://github.com/apache/apisix-dashboard/issues/1163#issuecomment-755924311 This command should under the `apisix-docker` repo, right? This is an automated message from the Apache

[GitHub] [apisix] LLLLimbo closed issue #3210: bug: Route has no auto-generated id

2021-01-06 Thread GitBox
imbo closed issue #3210: URL: https://github.com/apache/apisix/issues/3210 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

[GitHub] [apisix] spacewander opened a new pull request #3212: test: demonstrate http-logger works fine with JSON body

2021-01-06 Thread GitBox
spacewander opened a new pull request #3212: URL: https://github.com/apache/apisix/pull/3212 Close #1710. Signed-off-by: spacewander ### What this PR does / why we need it: ### Pre-submission checklist: * [x] Did you explain what problem does this PR

[GitHub] [apisix-dashboard] imjoey commented on issue #1233: docker build fail

2021-01-06 Thread GitBox
imjoey commented on issue #1233: URL: https://github.com/apache/apisix-dashboard/issues/1233#issuecomment-755922315 @belongwqz thanks for reporting. We will check ASAP. This is an automated message from the Apache Git

[GitHub] [apisix] hnlq715 opened a new issue #3211: Feature: implement p2c balancer with random, ewma, leastloaded

2021-01-06 Thread GitBox
hnlq715 opened a new issue #3211: URL: https://github.com/apache/apisix/issues/3211 P2c implemention should have better performance than the implemention with for loop when the upstream contains lots of backend servers.

[GitHub] [apisix-dashboard] belongwqz opened a new issue #1233: docker build fail

2021-01-06 Thread GitBox
belongwqz opened a new issue #1233: URL: https://github.com/apache/apisix-dashboard/issues/1233 # Bug report ## Describe the bug docker build fail when build manager-api ## How to Reproduce 1. `docker build -t apisix-dashboard:2.2 --build-arg

[GitHub] [apisix-website] gxthrj merged pull request #127: docs: update release guide

2021-01-06 Thread GitBox
gxthrj merged pull request #127: URL: https://github.com/apache/apisix-website/pull/127 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

[apisix-website] branch master updated (4cee286 -> d92319b)

2021-01-06 Thread kvn
This is an automated email from the ASF dual-hosted git repository. kvn pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/apisix-website.git. from 4cee286 fix: Ingress-controller version number changed back to 0.1.0-rc2 (#128) new e6137e9 docs:

[GitHub] [apisix] spacewander commented on issue #3198: feat(CI): add WIP from the GitHub Marketplace

2021-01-06 Thread GitBox
spacewander commented on issue #3198: URL: https://github.com/apache/apisix/issues/3198#issuecomment-755917698 Err... It seems the semantic checker has already done the same thing. We can't merge a PR if its title begins with `WIP`. There is no need to add another 3rd party application.

[GitHub] [apisix-dashboard] imjoey commented on issue #1163: Langauge selection issue in Dashboard

2021-01-06 Thread GitBox
imjoey commented on issue #1163: URL: https://github.com/apache/apisix-dashboard/issues/1163#issuecomment-755916083 > Hi Guys , > Any update ? Hi @jinusuresh , good to see you and sorry for late. > Hi , This is what used after taking the code from master branch on

[GitHub] [apisix-website] gxthrj commented on a change in pull request #127: docs: update release guide

2021-01-06 Thread GitBox
gxthrj commented on a change in pull request #127: URL: https://github.com/apache/apisix-website/pull/127#discussion_r553132580 ## File path: docs/release-guide.md ## @@ -158,11 +158,11 @@ Committed revision 37434. ### Make source code package and upload to Apache SVN

[GitHub] [apisix-dashboard] juzhiyuan merged pull request #1232: sync v2.3

2021-01-06 Thread GitBox
juzhiyuan merged pull request #1232: URL: https://github.com/apache/apisix-dashboard/pull/1232 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[apisix-dashboard] branch v2.3 updated: sync v2.3 (#1232)

2021-01-06 Thread juzhiyuan
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a commit to branch v2.3 in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git The following commit(s) were added to refs/heads/v2.3 by this push: new ce47759 sync v2.3 (#1232) ce47759 is

[GitHub] [apisix] LLLLimbo commented on issue #3210: bug: Route has no auto-generated id

2021-01-06 Thread GitBox
imbo commented on issue #3210: URL: https://github.com/apache/apisix/issues/3210#issuecomment-755914162 > @imbo > When creating a route in `POST` method, currently, the automatically generated id will not be added to the configuration information. You can try the `PUT` method

[GitHub] [apisix] Firstsawyou commented on issue #3210: bug: Route has no auto-generated id

2021-01-06 Thread GitBox
Firstsawyou commented on issue #3210: URL: https://github.com/apache/apisix/issues/3210#issuecomment-755913157 @imbo When creating a route in `POST` method, currently, the automatically generated id will not be added to the configuration information. You can try the `PUT` method

[GitHub] [apisix] LLLLimbo edited a comment on issue #3210: bug: Route has no auto-generated id

2021-01-06 Thread GitBox
imbo edited a comment on issue #3210: URL: https://github.com/apache/apisix/issues/3210#issuecomment-755910938 > Hi, @imbo. Have you checked the route data in etcd? I checked in etcd just now , it seems exists. But why? ` etcdctl get /apisix/routes/00544244

[GitHub] [apisix] LLLLimbo commented on issue #3210: bug: Route has no auto-generated id

2021-01-06 Thread GitBox
imbo commented on issue #3210: URL: https://github.com/apache/apisix/issues/3210#issuecomment-755910938 > Hi, @imbo. Have you checked the route data in etcd? I checked in etcd just now , it seems exists. But why? ` etcdctl get /apisix/routes/00544244

[GitHub] [apisix-dashboard] imjoey closed pull request #1231: feat: add plugin icon (#1220)(backport to v2.3)

2021-01-06 Thread GitBox
imjoey closed pull request #1231: URL: https://github.com/apache/apisix-dashboard/pull/1231 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

[GitHub] [apisix-dashboard] imjoey commented on pull request #1231: feat: add plugin icon (#1220)(backport to v2.3)

2021-01-06 Thread GitBox
imjoey commented on pull request #1231: URL: https://github.com/apache/apisix-dashboard/pull/1231#issuecomment-755909744 https://github.com/apache/apisix-dashboard/pull/1232 will take in charge. Close this PR now. This is

[GitHub] [apisix-dashboard] jinusuresh commented on issue #1163: Langauge selection issue in Dashboard

2021-01-06 Thread GitBox
jinusuresh commented on issue #1163: URL: https://github.com/apache/apisix-dashboard/issues/1163#issuecomment-755909603 Hi Guys , Any update ? Thanks, Jinu Sureshbabu From: 琚致远 Sent: Friday, January 1, 2021 5:43 AM To:

[GitHub] [apisix] starsz commented on issue #3210: bug: Route has no auto-generated id

2021-01-06 Thread GitBox
starsz commented on issue #3210: URL: https://github.com/apache/apisix/issues/3210#issuecomment-755909344 Hi, @imbo. Have you checked the route data in etcd? This is an automated message from the Apache Git Service. To

[GitHub] [apisix-dashboard] juzhiyuan opened a new pull request #1232: sync v2.3

2021-01-06 Thread GitBox
juzhiyuan opened a new pull request #1232: URL: https://github.com/apache/apisix-dashboard/pull/1232 Please answer these questions before submitting a pull request - Why submit this pull request? - [ ] Bugfix - [x] New feature provided - [ ] Improve performance Sync

[GitHub] [apisix-dashboard] imjoey opened a new pull request #1231: feat: add plugin icon (#1220)(backport to v2.3)

2021-01-06 Thread GitBox
imjoey opened a new pull request #1231: URL: https://github.com/apache/apisix-dashboard/pull/1231 - Why submit this pull request? - [ ] Bugfix - [x] New feature provided - [ ] Improve performance - [x] Backport patches As the preparation for release v2.3, this PR is going

[apisix-dashboard] 02/05: fix: online debug body params support content-type x-www-form-urlencoded (#1201)

2021-01-06 Thread juzhiyuan
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a commit to branch v2.3-sync in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git commit 7862edc75058f405fd6cdb13282a1011748683cd Author: liuxiran AuthorDate: Wed Jan 6 12:32:38 2021 +0800

[apisix-dashboard] 04/05: feat: added E2E test for plugins (#1214)

2021-01-06 Thread juzhiyuan
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a commit to branch v2.3-sync in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git commit 6bc8c8ecaa742d796152a9d6a4bae9115d8580ba Author: 琚致远 AuthorDate: Thu Jan 7 09:42:15 2021 +0800

[apisix-dashboard] 03/05: feat: add tips when plugin type is auth and schemaType is not consumer (#1219)

2021-01-06 Thread juzhiyuan
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a commit to branch v2.3-sync in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git commit fb017217ee1b2def60b116cdcab84a07a460b151 Author: litesun AuthorDate: Wed Jan 6 14:17:55 2021 +0800

[apisix-dashboard] branch v2.3-sync created (now 6990e5b)

2021-01-06 Thread juzhiyuan
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a change to branch v2.3-sync in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git. at 6990e5b feat: add plugin icon (#1220) This branch includes the following new commits: new

[apisix-dashboard] 05/05: feat: add plugin icon (#1220)

2021-01-06 Thread juzhiyuan
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a commit to branch v2.3-sync in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git commit 6990e5b0033172afc4c8f29cf9cd978c8e6bb831 Author: litesun AuthorDate: Thu Jan 7 11:27:08 2021 +0800

[apisix-dashboard] 01/05: fix(be): change bodyParams to string to accept any content-type (#1202)

2021-01-06 Thread juzhiyuan
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a commit to branch v2.3-sync in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git commit 4e4332dd57a743fb8d151fbb0cdc343dea7f642a Author: liuxiran AuthorDate: Wed Jan 6 10:38:56 2021 +0800

[GitHub] [apisix] LLLLimbo opened a new issue #3210: bug: Route has no auto-generated id

2021-01-06 Thread GitBox
imbo opened a new issue #3210: URL: https://github.com/apache/apisix/issues/3210 I create a route in this way `curl --location --request POST 'http://192.168.2.102:9080/apisix/admin/routes' \ --header 'Content-Type: application/json' \ --header 'X-API-KEY:

[GitHub] [apisix-dashboard] imjoey commented on pull request #1220: feat: add plugin icon

2021-01-06 Thread GitBox
imjoey commented on pull request #1220: URL: https://github.com/apache/apisix-dashboard/pull/1220#issuecomment-755904632 @juzhiyuan OK, I could do the cherry pick for this for release v2.3. This is an automated message from

[apisix-dashboard] branch v2.3 updated: fix: sync the preload packages in schema-sync.lua (#1216) (#1221) (#1228)

2021-01-06 Thread juzhiyuan
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a commit to branch v2.3 in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git The following commit(s) were added to refs/heads/v2.3 by this push: new 72f2195 fix: sync the preload

[GitHub] [apisix-dashboard] juzhiyuan merged pull request #1228: fix: sync the preload packages in schema-sync.lua (#1221)(backport to v2.3)

2021-01-06 Thread GitBox
juzhiyuan merged pull request #1228: URL: https://github.com/apache/apisix-dashboard/pull/1228 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [apisix-dashboard] juzhiyuan commented on pull request #1220: feat: add plugin icon

2021-01-06 Thread GitBox
juzhiyuan commented on pull request #1220: URL: https://github.com/apache/apisix-dashboard/pull/1220#issuecomment-755902537 We will check other prs soon This is an automated message from the Apache Git Service. To respond to

[GitHub] [apisix-dashboard] imjoey commented on pull request #1220: feat: add plugin icon

2021-01-06 Thread GitBox
imjoey commented on pull request #1220: URL: https://github.com/apache/apisix-dashboard/pull/1220#issuecomment-755902054 @juzhiyuan Hi, the issue https://github.com/apache/apisix-dashboard/issues/1168 was marked as included in milestone 2.3. Please check that whether this PR requires

[GitHub] [apisix-ingress-controller] tokers commented on pull request #158: chore: version info in Makefile

2021-01-06 Thread GitBox
tokers commented on pull request #158: URL: https://github.com/apache/apisix-ingress-controller/pull/158#issuecomment-755899909 @membphis OK, will check it out. This is an automated message from the Apache Git Service. To

[GitHub] [apisix] Firstsawyou commented on issue #3207: apisix-2.1/apisix/init.lua:89: module 'resty.worker.events' not found:

2021-01-06 Thread GitBox
Firstsawyou commented on issue #3207: URL: https://github.com/apache/apisix/issues/3207#issuecomment-755898630 > would you like to fix this issue? ^_^ Yes, let me try it. This is an automated message from the Apache

[GitHub] [apisix-ingress-controller] membphis opened a new issue #159: chore: implement a script shell to check the version

2021-01-06 Thread GitBox
membphis opened a new issue #159: URL: https://github.com/apache/apisix-ingress-controller/issues/159 original comments: https://github.com/apache/apisix-ingress-controller/pull/158#issuecomment-755897013 here is an example:

[GitHub] [apisix] spacewander commented on a change in pull request #3208: fix: restart failed issue

2021-01-06 Thread GitBox
spacewander commented on a change in pull request #3208: URL: https://github.com/apache/apisix/pull/3208#discussion_r553119731 ## File path: apisix/cli/ops.lua ## @@ -382,11 +382,19 @@ local function start(env, ...) local pid_path = env.apisix_home .. "/logs/nginx.pid"

[GitHub] [apisix-ingress-controller] membphis merged pull request #158: chore: version info in Makefile

2021-01-06 Thread GitBox
membphis merged pull request #158: URL: https://github.com/apache/apisix-ingress-controller/pull/158 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[apisix-ingress-controller] branch master updated: chore: version info in Makefile (#158)

2021-01-06 Thread membphis
This is an automated email from the ASF dual-hosted git repository. membphis pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/apisix-ingress-controller.git The following commit(s) were added to refs/heads/master by this push: new 4935200 chore: version

[GitHub] [apisix-ingress-controller] membphis commented on pull request #158: chore: version info in Makefile

2021-01-06 Thread GitBox
membphis commented on pull request #158: URL: https://github.com/apache/apisix-ingress-controller/pull/158#issuecomment-755897013 we can add a script to check the version when we release new version. here is an example of APISIX:

[GitHub] [apisix] membphis commented on issue #3207: apisix-2.1/apisix/init.lua:89: module 'resty.worker.events' not found:

2021-01-06 Thread GitBox
membphis commented on issue #3207: URL: https://github.com/apache/apisix/issues/3207#issuecomment-755896001 @Firstsawyou would you like to fix this issue? ^_^ This is an automated message from the Apache Git Service. To

[GitHub] [apisix] Firstsawyou commented on issue #3207: apisix-2.1/apisix/init.lua:89: module 'resty.worker.events' not found:

2021-01-06 Thread GitBox
Firstsawyou commented on issue #3207: URL: https://github.com/apache/apisix/issues/3207#issuecomment-755895375 I think this is a bug, and the apisix service should be prohibited from starting when `make run`. Because `make init` has prompted a warning message, but `make run` is executed

[GitHub] [apisix] starsz commented on issue #3209: bug: blacklist does not work

2021-01-06 Thread GitBox
starsz commented on issue #3209: URL: https://github.com/apache/apisix/issues/3209#issuecomment-755893142 Hi, @chenyicai I think you can check the error_log for more details. This is an automated message from the Apache Git

[GitHub] [apisix-ingress-controller] tokers opened a new pull request #158: chore: version info in Makefile

2021-01-06 Thread GitBox
tokers opened a new pull request #158: URL: https://github.com/apache/apisix-ingress-controller/pull/158 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [apisix] Firstsawyou commented on issue #3209: bug: blacklist does not work

2021-01-06 Thread GitBox
Firstsawyou commented on issue #3209: URL: https://github.com/apache/apisix/issues/3209#issuecomment-755888561 Can you provide the configuration information of etcd? This makes it easier for us to troubleshoot problems.

[GitHub] [apisix] dabue merged pull request #3201: feat: Add create_time and update_time in global_rule schema

2021-01-06 Thread GitBox
dabue merged pull request #3201: URL: https://github.com/apache/apisix/pull/3201 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

[apisix] branch master updated: feat: Add create_time and update_time in global_rule schema (#3201)

2021-01-06 Thread dabue
This is an automated email from the ASF dual-hosted git repository. dabue 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 2996416 feat: Add create_time and update_time

[GitHub] [apisix-dashboard] codecov-io commented on pull request #1230: feat: Add backport type in PR template

2021-01-06 Thread GitBox
codecov-io commented on pull request #1230: URL: https://github.com/apache/apisix-dashboard/pull/1230#issuecomment-755875097 # [Codecov](https://codecov.io/gh/apache/apisix-dashboard/pull/1230?src=pr=h1) Report > Merging

[GitHub] [apisix-dashboard] imjoey opened a new pull request #1230: feat: Add backport type in PR template

2021-01-06 Thread GitBox
imjoey opened a new pull request #1230: URL: https://github.com/apache/apisix-dashboard/pull/1230 Signed-off-by: imjoey Please answer these questions before submitting a pull request - Why submit this pull request? - [ ] Bugfix - [x] New feature provided - [ ]

[GitHub] [apisix] moonming commented on issue #3198: feat(CI): add WIP from the GitHub Marketplace

2021-01-06 Thread GitBox
moonming commented on issue #3198: URL: https://github.com/apache/apisix/issues/3198#issuecomment-755873002 @spacewander we have the same right. you can create a jira for apache infra team This is an automated message

[GitHub] [apisix] chenyicai opened a new issue #3209: bug: blacklist does not work

2021-01-06 Thread GitBox
chenyicai opened a new issue #3209: URL: https://github.com/apache/apisix/issues/3209 ### Issue description Configure plugin ip-restriction in dashboard,blacklist does not work I want to not allowed ip 192.168.1.1 access, so i config the ip in dashboard plugin ip-restriction

[GitHub] [apisix] spacewander commented on pull request #3208: fix: restart failed issue

2021-01-06 Thread GitBox
spacewander commented on pull request #3208: URL: https://github.com/apache/apisix/pull/3208#issuecomment-755871051 ``` Checking apisix/cli/ops.lua 1 warning apisix/cli/ops.lua:390:101: line is too long (113 > 100) Total: 1 warning / 0 errors in

[GitHub] [apisix-dashboard] imjoey commented on pull request #1228: fix: sync the preload packages in schema-sync.lua (#1221)(backport to v2.3)

2021-01-06 Thread GitBox
imjoey commented on pull request #1228: URL: https://github.com/apache/apisix-dashboard/pull/1228#issuecomment-755869173 @juzhiyuan please help us attach the `backport` label here. Thank you. This is an automated message

[GitHub] [apisix] azura27 commented on issue #3193: bug: 10 second URL test comes with 1 second api_ctx nil value error

2021-01-06 Thread GitBox
azura27 commented on issue #3193: URL: https://github.com/apache/apisix/issues/3193#issuecomment-755869097 Maybe there is some relation to the stress, but It worked well several days ago. So some other need take into cosnideration too. I test new/delete route A, andin the mean

[GitHub] [apisix-dashboard] juzhiyuan opened a new issue #1229: Architecture for Web

2021-01-06 Thread GitBox
juzhiyuan opened a new issue #1229: URL: https://github.com/apache/apisix-dashboard/issues/1229 # Feature request ## Please describe your feature Here is the latest architecture for the web, we may improve its design step by step.

[GitHub] [apisix-dashboard] imjoey opened a new pull request #1228: fix: sync the preload packages in schema-sync.lua (#1221)(backport to v2.3)

2021-01-06 Thread GitBox
imjoey opened a new pull request #1228: URL: https://github.com/apache/apisix-dashboard/pull/1228 Please answer these questions before submitting a pull request - Why submit this pull request? - [x] Bugfix - [ ] New feature provided - [ ] Improve performance - [x]

[GitHub] [apisix] Firstsawyou edited a comment on issue #3207: apisix-2.1/apisix/init.lua:89: module 'resty.worker.events' not found:

2021-01-06 Thread GitBox
Firstsawyou edited a comment on issue #3207: URL: https://github.com/apache/apisix/issues/3207#issuecomment-755866501 @hifou From your error message, found that your apisix project is located in the `/root` directory. You need to run APISIX in a directory other than `/root`.

[GitHub] [apisix] Firstsawyou edited a comment on issue #3207: apisix-2.1/apisix/init.lua:89: module 'resty.worker.events' not found:

2021-01-06 Thread GitBox
Firstsawyou edited a comment on issue #3207: URL: https://github.com/apache/apisix/issues/3207#issuecomment-755866501 @hifou From your error message, found that your apisix project is located in the `/root` directory. You need to run APISIX in a directory other than `/roo`.

[GitHub] [apisix-dashboard] imjoey commented on pull request #1221: fix: sync the preload packages in schema-sync.lua

2021-01-06 Thread GitBox
imjoey commented on pull request #1221: URL: https://github.com/apache/apisix-dashboard/pull/1221#issuecomment-755867315 > yes! @juzhiyuan OK, please let me backport this. BTW, the label attached should be `need_backport` instead of `backport`. 

[GitHub] [apisix] azura27 commented on issue #3202: bug: server runs apisix also runs Nginx. "apisix restart" command does not work

2021-01-06 Thread GitBox
azura27 commented on issue #3202: URL: https://github.com/apache/apisix/issues/3202#issuecomment-755867187 PR pulled according the reference provided: https://github.com/apache/apisix/pull/3208 This is an automated

[GitHub] [apisix] Firstsawyou edited a comment on issue #3207: apisix-2.1/apisix/init.lua:89: module 'resty.worker.events' not found:

2021-01-06 Thread GitBox
Firstsawyou edited a comment on issue #3207: URL: https://github.com/apache/apisix/issues/3207#issuecomment-755866501 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [apisix] azura27 opened a new pull request #3208: fix : restart failed issue

2021-01-06 Thread GitBox
azura27 opened a new pull request #3208: URL: https://github.com/apache/apisix/pull/3208 ### What this PR does / why we need it: Issue #3202 refer to https://github.com/apache/apisix/pull/2869 ### Pre-submission checklist: * [ ] Did you explain what problem does

[GitHub] [apisix] Firstsawyou commented on issue #3207: apisix-2.1/apisix/init.lua:89: module 'resty.worker.events' not found:

2021-01-06 Thread GitBox
Firstsawyou commented on issue #3207: URL: https://github.com/apache/apisix/issues/3207#issuecomment-755866501 @hifou From your error message, you find that your apisix project is placed in the `/root` directory. You need to run APISIX in a directory other than `/roo`. $ make

[GitHub] [apisix-dashboard] juzhiyuan commented on pull request #1221: fix: sync the preload packages in schema-sync.lua

2021-01-06 Thread GitBox
juzhiyuan commented on pull request #1221: URL: https://github.com/apache/apisix-dashboard/pull/1221#issuecomment-755866436 yes! This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [apisix-dashboard] juzhiyuan opened a new issue #1227: The API for fetching all plugins should return `disable` property

2021-01-06 Thread GitBox
juzhiyuan opened a new issue #1227: URL: https://github.com/apache/apisix-dashboard/issues/1227 # Feature request ## Please describe your feature The data from `plugins?all=true`, only part of the data contains a `disable` property, so the FE has to hack codes.

[GitHub] [apisix-dashboard] imjoey commented on pull request #1226: feat(release): Enable CI trigger for v2.3 branch

2021-01-06 Thread GitBox
imjoey commented on pull request #1226: URL: https://github.com/apache/apisix-dashboard/pull/1226#issuecomment-755865831 @juzhiyuan thank you very much for reviewing. Very glad that we can come to an agreement. Much appreciated for the openness. 

[GitHub] [apisix-dashboard] juzhiyuan merged pull request #1226: feat(release): Enable CI trigger for v2.3 branch

2021-01-06 Thread GitBox
juzhiyuan merged pull request #1226: URL: https://github.com/apache/apisix-dashboard/pull/1226 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[apisix-dashboard] branch v2.3 updated: feat(release): Enable CI trigger for v2.3 branch (#1226)

2021-01-06 Thread juzhiyuan
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a commit to branch v2.3 in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git The following commit(s) were added to refs/heads/v2.3 by this push: new 073a252 feat(release): Enable CI

[GitHub] [apisix-dashboard] imjoey commented on a change in pull request #1226: feat(release): Enable CI trigger for v2.3 branch

2021-01-06 Thread GitBox
imjoey commented on a change in pull request #1226: URL: https://github.com/apache/apisix-dashboard/pull/1226#discussion_r553093499 ## File path: .github/workflows/backend-cli-test.yml ## @@ -3,10 +3,10 @@ name: Backend CLI Test on: push: branches: - - master +

[GitHub] [apisix-dashboard] juzhiyuan commented on a change in pull request #1226: feat(release): Enable CI trigger for v2.3 branch

2021-01-06 Thread GitBox
juzhiyuan commented on a change in pull request #1226: URL: https://github.com/apache/apisix-dashboard/pull/1226#discussion_r553093073 ## File path: .github/workflows/backend-cli-test.yml ## @@ -3,10 +3,10 @@ name: Backend CLI Test on: push: branches: - - master

[apisix-dashboard] branch v2.3 updated (d39951b -> cc9e7de)

2021-01-06 Thread juzhiyuan
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a change to branch v2.3 in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git. from d39951b fix(be): search by status on the route list page is invalid (#1207) (#1224) add cc9e7de

[GitHub] [apisix-dashboard] juzhiyuan merged pull request #1225: feat: Improve consumer for i18n (#1212)(backport to v2.3)

2021-01-06 Thread GitBox
juzhiyuan merged pull request #1225: URL: https://github.com/apache/apisix-dashboard/pull/1225 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [apisix-dashboard] imjoey commented on a change in pull request #1226: feat(release): Enable CI trigger for v2.3 branch

2021-01-06 Thread GitBox
imjoey commented on a change in pull request #1226: URL: https://github.com/apache/apisix-dashboard/pull/1226#discussion_r553092763 ## File path: .github/workflows/backend-cli-test.yml ## @@ -3,10 +3,10 @@ name: Backend CLI Test on: push: branches: - - master +

[GitHub] [apisix-dashboard] juzhiyuan commented on a change in pull request #1226: feat(release): Enable CI trigger for v2.3 branch

2021-01-06 Thread GitBox
juzhiyuan commented on a change in pull request #1226: URL: https://github.com/apache/apisix-dashboard/pull/1226#discussion_r553092664 ## File path: .github/workflows/backend-cli-test.yml ## @@ -3,10 +3,10 @@ name: Backend CLI Test on: push: branches: - - master

[GitHub] [apisix-dashboard] codecov-io commented on pull request #1226: feat(release): Enable CI trigger for v2.3 branch

2021-01-06 Thread GitBox
codecov-io commented on pull request #1226: URL: https://github.com/apache/apisix-dashboard/pull/1226#issuecomment-755863002 # [Codecov](https://codecov.io/gh/apache/apisix-dashboard/pull/1226?src=pr=h1) Report > :exclamation: No coverage uploaded for pull request base

[GitHub] [apisix] Firstsawyou commented on issue #3207: apisix-2.1/apisix/init.lua:89: module 'resty.worker.events' not found:

2021-01-06 Thread GitBox
Firstsawyou commented on issue #3207: URL: https://github.com/apache/apisix/issues/3207#issuecomment-755862168 > @idbeta @Firstsawyou you can make a check > > the steps are fine. If we can not reproduce this issue, can you supply the login information of your machine? @hifou >

[GitHub] [apisix] hifou commented on issue #3207: apisix-2.1/apisix/init.lua:89: module 'resty.worker.events' not found:

2021-01-06 Thread GitBox
hifou commented on issue #3207: URL: https://github.com/apache/apisix/issues/3207#issuecomment-755861706 @membphis 非常感谢,我这边可以提供账号,帮忙看一下。 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [apisix-dashboard] juzhiyuan commented on issue #1194: test: Check for unnecessary wait in FE e2e

2021-01-06 Thread GitBox
juzhiyuan commented on issue #1194: URL: https://github.com/apache/apisix-dashboard/issues/1194#issuecomment-755861073 Those test cases still have improvements, after discussing with @LiteSun, we are going to solve this in M2.4 first.

[GitHub] [apisix] membphis edited a comment on issue #3207: apisix-2.1/apisix/init.lua:89: module 'resty.worker.events' not found:

2021-01-06 Thread GitBox
membphis edited a comment on issue #3207: URL: https://github.com/apache/apisix/issues/3207#issuecomment-755860049 @idbeta @Firstsawyou you can make a check the steps are fine. If we can not reproduce this issue, can you supply the login information of your machine? @hifou

[GitHub] [apisix] membphis commented on issue #3207: apisix-2.1/apisix/init.lua:89: module 'resty.worker.events' not found:

2021-01-06 Thread GitBox
membphis commented on issue #3207: URL: https://github.com/apache/apisix/issues/3207#issuecomment-755860049 @idbeta @Firstsawyou you can make a check the steps are fine. If we can not reproduce this issue, can you supply the login information of your machine? then we can make a

[GitHub] [apisix-dashboard] juzhiyuan commented on a change in pull request #1226: feat(release): Enable CI trigger for v2.3 branch

2021-01-06 Thread GitBox
juzhiyuan commented on a change in pull request #1226: URL: https://github.com/apache/apisix-dashboard/pull/1226#discussion_r553089727 ## File path: .github/workflows/backend-cli-test.yml ## @@ -3,10 +3,10 @@ name: Backend CLI Test on: push: branches: - - master

[apisix-dashboard] branch master updated: feat: add plugin icon (#1220)

2021-01-06 Thread sunyi
This is an automated email from the ASF dual-hosted git repository. sunyi 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 190949c feat: add plugin icon (#1220)

  1   2   3   >