[GitHub] [apisix-ingress-controller] tao12345666333 commented on issue #787: request help: ApisixRoute apiversion Inconsistency

2021-12-07 Thread GitBox
tao12345666333 commented on issue #787: URL: https://github.com/apache/apisix-ingress-controller/issues/787#issuecomment-988578931 I mean you are using * https://github.com/apache/apisix-helm-chart/blob/master/charts/apisix-ingress-controller/crds/customresourcedefinitions.yaml

[GitHub] [apisix-ingress-controller] tao12345666333 commented on pull request #772: feat: implement pluginconfig clients (#638)

2021-12-07 Thread GitBox
tao12345666333 commented on pull request #772: URL: https://github.com/apache/apisix-ingress-controller/pull/772#issuecomment-988576890 #788 has fixed the e2e error. It's green! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [apisix-ingress-controller] GhangZh commented on issue #787: request help: ApisixRoute apiversion Inconsistency

2021-12-07 Thread GitBox
GhangZh commented on issue #787: URL: https://github.com/apache/apisix-ingress-controller/issues/787#issuecomment-988572975 > And which CRD configuration are you using? 1. Yes, deleted then create. 2.I using apisix.apache.org/v2beta1 ApisixRoute. -- This is an automated

[GitHub] [apisix] qq54903099 commented on pull request #5732: feat: consumer-restriction add reject-message

2021-12-07 Thread GitBox
qq54903099 commented on pull request #5732: URL: https://github.com/apache/apisix/pull/5732#issuecomment-988565449 how to chose reviewers? -- 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] shaobo322 commented on a change in pull request #5738: feat(plugin): jwt-token plugin allow user defined token parameter name

2021-12-07 Thread GitBox
shaobo322 commented on a change in pull request #5738: URL: https://github.com/apache/apisix/pull/5738#discussion_r764601380 ## File path: apisix/plugins/jwt-auth.lua ## @@ -45,6 +45,7 @@ local consumer_schema = { properties = { key = {type = "string"},

[GitHub] [apisix-dashboard] GhangZh closed issue #2223: request help :apisix-dashboard Dockerfile and build.sh error

2021-12-07 Thread GitBox
GhangZh closed issue #2223: URL: https://github.com/apache/apisix-dashboard/issues/2223 -- 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:

[GitHub] [apisix-dashboard] GhangZh commented on issue #2223: request help :apisix-dashboard Dockerfile and build.sh error

2021-12-07 Thread GitBox
GhangZh commented on issue #2223: URL: https://github.com/apache/apisix-dashboard/issues/2223#issuecomment-988557283 > Hi @GhangZh, may I know if this issue gets resolved? Yes, thank you both for your help @juzhiyuan @zaunist , then I'll turn off this issue -- This is an

[GitHub] [apisix] leslie-tsang commented on issue #5733: request help: sometimes connection refused while using service ip to connect the upstream

2021-12-07 Thread GitBox
leslie-tsang commented on issue #5733: URL: https://github.com/apache/apisix/issues/5733#issuecomment-988556034 > Hi @leslie-tsang, could you give some ideas on how to debug this issue? Yes for sure. > ### Issue description > **error log** > > ``` > [error]

[GitHub] [apisix] yxudong opened a new issue #4301: bug: skywalking not work when upsteam timeout

2021-12-07 Thread GitBox
yxudong opened a new issue #4301: URL: https://github.com/apache/apisix/issues/4301 ### Issue description At present, sw_tracer:finish() will be called in body_filter_by_lua phase. But in some case, the http request will not execute body_filter_by_lua phase. so this function

[GitHub] [apisix] wu-sheng closed issue #4301: bug: skywalking not work when upsteam timeout

2021-12-07 Thread GitBox
wu-sheng closed issue #4301: URL: https://github.com/apache/apisix/issues/4301 -- 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:

[GitHub] [apisix] wu-sheng commented on issue #4301: bug: skywalking not work when upsteam timeout

2021-12-07 Thread GitBox
wu-sheng commented on issue #4301: URL: https://github.com/apache/apisix/issues/4301#issuecomment-988548156 ![image](https://user-images.githubusercontent.com/5441976/145162308-af4de142-2ccc-4581-b93d-1c6d15a01b3c.png) 2 fixes are code related. It is 7 months since last release, if

[GitHub] [apisix] spacewander commented on issue #5596: request help: customize Nginx variable in the configuration

2021-12-07 Thread GitBox
spacewander commented on issue #5596: URL: https://github.com/apache/apisix/issues/5596#issuecomment-988547683 Just do it. -- 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.

[GitHub] [apisix] spacewander commented on issue #4301: bug: skywalking not work when upsteam timeout

2021-12-07 Thread GitBox
spacewander commented on issue #4301: URL: https://github.com/apache/apisix/issues/4301#issuecomment-988544339 @dmsolr Maybe we can create a new release of skywalking-nginx-lua now? -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [apisix] zaunist commented on pull request #5739: docs(global_rule): update global rule description

2021-12-07 Thread GitBox
zaunist commented on pull request #5739: URL: https://github.com/apache/apisix/pull/5739#issuecomment-988544226 cc @juzhiyuan @leslie-tsang -- 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] zuiyangqingzhou commented on a change in pull request #5728: refine the ua-restriction plugin configuration checksum logic

2021-12-07 Thread GitBox
zuiyangqingzhou commented on a change in pull request #5728: URL: https://github.com/apache/apisix/pull/5728#discussion_r764585950 ## File path: apisix/plugins/ua-restriction.lua ## @@ -34,11 +35,11 @@ local schema = { type = "boolean", default =

[GitHub] [apisix] JiaoMaMJ commented on pull request #5686: feat: add a tool that can help users quickly try your project

2021-12-07 Thread GitBox
JiaoMaMJ commented on pull request #5686: URL: https://github.com/apache/apisix/pull/5686#issuecomment-988541387 > Yes, It is a manual screenshot, just to demonstrate the entrance effect. You can customize the actual display effect. If your project uses Tin, we will also promote

[GitHub] [apisix-dashboard] codecov-commenter commented on pull request #2235: chore(deps): bump actions/upload-artifact from 2.2.4 to 2.3.0

2021-12-07 Thread GitBox
codecov-commenter commented on pull request #2235: URL: https://github.com/apache/apisix-dashboard/pull/2235#issuecomment-988537853 #

[GitHub] [apisix] sekfung commented on issue #5733: request help: sometimes connection refused while using service ip to connect the upstream

2021-12-07 Thread GitBox
sekfung commented on issue #5733: URL: https://github.com/apache/apisix/issues/5733#issuecomment-988537861 BTW,it deploy on k8s cluster -- 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] tzssangglass commented on issue #5596: request help: customize Nginx variable in the configuration

2021-12-07 Thread GitBox
tzssangglass commented on issue #5596: URL: https://github.com/apache/apisix/issues/5596#issuecomment-988537630 Although this can be done. I'd like @spacewander 's way, do it with a special directive in `config-default.yaml`. -- This is an automated message from the Apache Git Service.

[GitHub] [apisix] zaunist commented on issue #4895: docs: https://github.com/apache/apisix/blob/master/docs/zh/latest/architecture-design/global-rule.md

2021-12-07 Thread GitBox
zaunist commented on issue #4895: URL: https://github.com/apache/apisix/issues/4895#issuecomment-988537204 Hi, community. I update the `global-rule` description, please take a look here https://github.com/apache/apisix/pull/5739 -- This is an automated message from the Apache Git

[GitHub] [apisix] spacewander commented on a change in pull request #5738: feat(plugin): jwt-token plugin allow user defined token parameter name

2021-12-07 Thread GitBox
spacewander commented on a change in pull request #5738: URL: https://github.com/apache/apisix/pull/5738#discussion_r764580485 ## File path: apisix/plugins/jwt-auth.lua ## @@ -45,6 +45,7 @@ local consumer_schema = { properties = { key = {type = "string"},

[GitHub] [apisix] juzhiyuan commented on pull request #5738: feat(plugin): jwt-token plugin allow user defined token parameter name

2021-12-07 Thread GitBox
juzhiyuan commented on pull request #5738: URL: https://github.com/apache/apisix/pull/5738#issuecomment-988535082 Hi @shaobo322, thanks for your contribution! And please update your PR's checklist  -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [apisix] zaunist opened a new pull request #5739: doc: update global rule description

2021-12-07 Thread GitBox
zaunist opened a new pull request #5739: URL: https://github.com/apache/apisix/pull/5739 ### What this PR does / why we need it: Global rule doc mistakenly define the scope of plugin.

[GitHub] [apisix-website] SylviaBABY opened a new pull request #797: docs: add APISIX Logger plugin blog

2021-12-07 Thread GitBox
SylviaBABY opened a new pull request #797: URL: https://github.com/apache/apisix-website/pull/797 Changes: add APISIX Logger plugin blog -- 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

[GitHub] [apisix-ingress-controller] tao12345666333 merged pull request #788: fix: ingress LB status records

2021-12-07 Thread GitBox
tao12345666333 merged pull request #788: URL: https://github.com/apache/apisix-ingress-controller/pull/788 -- 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

[apisix-ingress-controller] branch master updated: fix: ingress LB status records (#788)

2021-12-07 Thread zhangjintao
This is an automated email from the ASF dual-hosted git repository. zhangjintao 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 fe4a824 fix: ingress

[GitHub] [apisix] zaunist commented on a change in pull request #5736: docs(ldap-auth): add consumer parameters for ldap

2021-12-07 Thread GitBox
zaunist commented on a change in pull request #5736: URL: https://github.com/apache/apisix/pull/5736#discussion_r764576788 ## File path: docs/en/latest/plugins/ldap-auth.md ## @@ -41,12 +41,13 @@ This authentication plugin use [lualdap](https://lualdap.github.io/lualdap/) plu

[GitHub] [apisix-ingress-controller] tao12345666333 commented on issue #781: request help: The route name generated by default is too long

2021-12-07 Thread GitBox
tao12345666333 commented on issue #781: URL: https://github.com/apache/apisix-ingress-controller/issues/781#issuecomment-988531185 You can using ApisixRoute like this one. ``` apiVersion: apisix.apache.org/v2beta3 kind: ApisixRoute metadata: name: httpbin-route spec:

[GitHub] [apisix] shaobo322 opened a new pull request #5738: feat(plugin): jwt-token plugin allow user defined token parameter name

2021-12-07 Thread GitBox
shaobo322 opened a new pull request #5738: URL: https://github.com/apache/apisix/pull/5738 ### What this PR does / why we need it: fix #5710 ### Pre-submission checklist: * [ ] Did you explain what problem does this PR solve? Or what new features have been

[GitHub] [apisix] xianshun163 commented on issue #3801: request help: support kubernetes as service discovery

2021-12-07 Thread GitBox
xianshun163 commented on issue #3801: URL: https://github.com/apache/apisix/issues/3801#issuecomment-988529804 when use dns, it is differ as the service discovery, the apisix balancer.lua can not get the pod ip. -- This is an automated message from the Apache Git Service. To

[GitHub] [apisix-ingress-controller] tao12345666333 commented on issue #783: request help: possible optimization in the installation documentation

2021-12-07 Thread GitBox
tao12345666333 commented on issue #783: URL: https://github.com/apache/apisix-ingress-controller/issues/783#issuecomment-988529102 #785 has merged. I will close this issue. Thanks! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [apisix-ingress-controller] tao12345666333 closed issue #783: request help: possible optimization in the installation documentation

2021-12-07 Thread GitBox
tao12345666333 closed issue #783: URL: https://github.com/apache/apisix-ingress-controller/issues/783 -- 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,

[GitHub] [apisix-ingress-controller] tao12345666333 commented on issue #784: request help: regex_uri in proxy-rewriter

2021-12-07 Thread GitBox
tao12345666333 commented on issue #784: URL: https://github.com/apache/apisix-ingress-controller/issues/784#issuecomment-988528820 And can you check the route record in Apache APISIX ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [apisix-dashboard] dependabot[bot] opened a new pull request #2235: chore(deps): bump actions/upload-artifact from 2.2.4 to 2.3.0

2021-12-07 Thread GitBox
dependabot[bot] opened a new pull request #2235: URL: https://github.com/apache/apisix-dashboard/pull/2235 Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 2.2.4 to 2.3.0. Release notes Sourced from

[apisix-dashboard] branch dependabot/github_actions/actions/upload-artifact-2.3.0 created (now c640039)

2021-12-07 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/github_actions/actions/upload-artifact-2.3.0 in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git. at c640039 chore(deps): bump actions/upload-artifact

[GitHub] [apisix] Horus-K commented on issue #5737: docs: how to traffic service percentage

2021-12-07 Thread GitBox
Horus-K commented on issue #5737: URL: https://github.com/apache/apisix/issues/5737#issuecomment-988526077 We have a very large user base,I need control plugings(response-rewrite.body) change -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [apisix] juzhiyuan commented on a change in pull request #5736: docs(ldap-auth): add consumer parameters for ldap

2021-12-07 Thread GitBox
juzhiyuan commented on a change in pull request #5736: URL: https://github.com/apache/apisix/pull/5736#discussion_r764570848 ## File path: docs/en/latest/plugins/ldap-auth.md ## @@ -41,12 +41,13 @@ This authentication plugin use [lualdap](https://lualdap.github.io/lualdap/)

[GitHub] [apisix] Horus-K opened a new issue #5737: docs: how to traffic service percentage

2021-12-07 Thread GitBox
Horus-K opened a new issue #5737: URL: https://github.com/apache/apisix/issues/5737 ### Improved or typo fixed. How do I verify the correctness of the plugins ### Solution I want to be able to use traffic-split similar plug-ins to control the percentage of service

[GitHub] [apisix] zaunist commented on issue #5668: docs: ldap-auth plugin's doc is missing Consumer parameters

2021-12-07 Thread GitBox
zaunist commented on issue #5668: URL: https://github.com/apache/apisix/issues/5668#issuecomment-988524341 Hi, @juzhiyuan. I just open a pull request to add consumer parameter, https://github.com/apache/apisix/pull/5736. Please take a look.. -- This is an automated message from the

[GitHub] [apisix-ingress-controller] tao12345666333 commented on issue #787: request help: ApisixRoute apiversion Inconsistency

2021-12-07 Thread GitBox
tao12345666333 commented on issue #787: URL: https://github.com/apache/apisix-ingress-controller/issues/787#issuecomment-988524325 And which CRD configuration are you using? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [apisix] juzhiyuan commented on issue #5573: docs: dont found the usage of etcd in discovery.md

2021-12-07 Thread GitBox
juzhiyuan commented on issue #5573: URL: https://github.com/apache/apisix/issues/5573#issuecomment-988523924 Hi @bzp2010, could you please take a look?  -- 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-ingress-controller] tao12345666333 commented on issue #787: request help: ApisixRoute apiversion Inconsistency

2021-12-07 Thread GitBox
tao12345666333 commented on issue #787: URL: https://github.com/apache/apisix-ingress-controller/issues/787#issuecomment-988523864 `delete` then `create` ? -- 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] zaunist opened a new pull request #5736: docs(ldap-auth): add consumer parameters for ldap

2021-12-07 Thread GitBox
zaunist opened a new pull request #5736: URL: https://github.com/apache/apisix/pull/5736 ### What this PR does / why we need it: Ldap-auth plugin docs are missing parameters when configuring Consumer. see https://apisix.apache.org/docs/apisix/plugins/ldap-auth/ And ldap-auth

[GitHub] [apisix-dashboard] codecov-commenter edited a comment on pull request #2200: test: migrate route_export_test e2e

2021-12-07 Thread GitBox
codecov-commenter edited a comment on pull request #2200: URL: https://github.com/apache/apisix-dashboard/pull/2200#issuecomment-963965563 #

[GitHub] [apisix] juzhiyuan commented on issue #5733: request help: sometimes connection refused while using service ip to connect the upstream

2021-12-07 Thread GitBox
juzhiyuan commented on issue #5733: URL: https://github.com/apache/apisix/issues/5733#issuecomment-988516590 Hi @leslie-tsang, could you give some ideas on how to debug this issue? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [apisix-dashboard] codecov-commenter edited a comment on pull request #2200: test: migrate route_export_test e2e

2021-12-07 Thread GitBox
codecov-commenter edited a comment on pull request #2200: URL: https://github.com/apache/apisix-dashboard/pull/2200#issuecomment-963965563 #

[GitHub] [apisix-dashboard] codecov-commenter edited a comment on pull request #2200: test: migrate route_export_test e2e

2021-12-07 Thread GitBox
codecov-commenter edited a comment on pull request #2200: URL: https://github.com/apache/apisix-dashboard/pull/2200#issuecomment-963965563 #

[GitHub] [apisix-dashboard] codecov-commenter edited a comment on pull request #2200: test: migrate route_export_test e2e

2021-12-07 Thread GitBox
codecov-commenter edited a comment on pull request #2200: URL: https://github.com/apache/apisix-dashboard/pull/2200#issuecomment-963965563 #

[GitHub] [apisix-dashboard] zaunist commented on pull request #2200: test: migrate route_export_test e2e

2021-12-07 Thread GitBox
zaunist commented on pull request #2200: URL: https://github.com/apache/apisix-dashboard/pull/2200#issuecomment-988504604 > Why the PR has too many changes? Sorry, I will fix it -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [apisix-ingress-controller] tao12345666333 commented on pull request #785: docs: Optimize installation documentation

2021-12-07 Thread GitBox
tao12345666333 commented on pull request #785: URL: https://github.com/apache/apisix-ingress-controller/pull/785#issuecomment-988503986 Thanks! -- 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

[GitHub] [apisix-ingress-controller] tao12345666333 merged pull request #785: docs: Optimize installation documentation

2021-12-07 Thread GitBox
tao12345666333 merged pull request #785: URL: https://github.com/apache/apisix-ingress-controller/pull/785 -- 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

[apisix-ingress-controller] branch master updated: docs: Optimize installation documentation (#785)

2021-12-07 Thread zhangjintao
This is an automated email from the ASF dual-hosted git repository. zhangjintao 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 1b2bc34 docs: Optimize

[GitHub] [apisix] shuaijinchao opened a new pull request #5735: fix(log-rotate): after enabling compression collect log exceptions

2021-12-07 Thread GitBox
shuaijinchao opened a new pull request #5735: URL: https://github.com/apache/apisix/pull/5735 (cherry picked from commit 139c3972898f1e45c6d5b1f40143c504116b3486) ### What this PR does / why we need it: ### Pre-submission checklist: * [x] Did you explain

[GitHub] [apisix-ingress-controller] juzhiyuan commented on pull request #785: docs: Optimize installation documentation

2021-12-07 Thread GitBox
juzhiyuan commented on pull request #785: URL: https://github.com/apache/apisix-ingress-controller/pull/785#issuecomment-988475605 LGTM! -- 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] juzhiyuan closed issue #5709: return 504

2021-12-07 Thread GitBox
juzhiyuan closed issue #5709: URL: https://github.com/apache/apisix/issues/5709 -- 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:

[GitHub] [apisix] bzp2010 opened a new pull request #5734: feat: support OPA plugin

2021-12-07 Thread GitBox
bzp2010 opened a new pull request #5734: URL: https://github.com/apache/apisix/pull/5734 ### What this PR does / why we need it: Add OPA plugin to support API access control using OPA services. implement #5714 ### Pre-submission checklist: * [x] Did you

[GitHub] [apisix] sekfung opened a new issue #5733: request help: sometimes connection refused while using service ip to connect the upstream

2021-12-07 Thread GitBox
sekfung opened a new issue #5733: URL: https://github.com/apache/apisix/issues/5733 ### Issue description **error log** ```log [error] 48#48: *2240177 connect() failed (111: Connection refused) while connecting to upstream [lua] balancer.lua:327: run(): failed to pick

[GitHub] [apisix] leslie-tsang commented on issue #5709: return 504

2021-12-07 Thread GitBox
leslie-tsang commented on issue #5709: URL: https://github.com/apache/apisix/issues/5709#issuecomment-988470685 @juzhiyuan Consider solved. Feel free to reopen it if need. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [apisix] qq54903099 opened a new pull request #5732: feat: consumer-restriction add reject-message

2021-12-07 Thread GitBox
qq54903099 opened a new pull request #5732: URL: https://github.com/apache/apisix/pull/5732 ### What this PR does / why we need it: ### Pre-submission checklist: * [ ] Did you explain what problem does this PR solve? Or what new features have been added? *

[GitHub] [apisix] qq54903099 closed pull request #5731: feat: consumer-restriction add reject-message

2021-12-07 Thread GitBox
qq54903099 closed pull request #5731: URL: https://github.com/apache/apisix/pull/5731 -- 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:

[GitHub] [apisix] qq54903099 opened a new pull request #5731: feat: consumer-restriction add reject-message

2021-12-07 Thread GitBox
qq54903099 opened a new pull request #5731: URL: https://github.com/apache/apisix/pull/5731 ### What this PR does / why we need it: Custom return message content ### Pre-submission checklist: * [ ] Did you explain what problem does this PR solve? Or what new

[GitHub] [apisix-dashboard] juzhiyuan commented on issue #1595: feat: Show route ID at LIST and EDIT views in dashboard

2021-12-07 Thread GitBox
juzhiyuan commented on issue #1595: URL: https://github.com/apache/apisix-dashboard/issues/1595#issuecomment-988460587 Oh btw, we have a meeting today, welcome to join in if you have time  see https://github.com/apache/apisix/discussions/5721 -- This is an automated message from the

[GitHub] [apisix-dashboard] juzhiyuan commented on issue #1595: feat: Show route ID at LIST and EDIT views in dashboard

2021-12-07 Thread GitBox
juzhiyuan commented on issue #1595: URL: https://github.com/apache/apisix-dashboard/issues/1595#issuecomment-988460167 Hi @mangoGoForward, you could do it if no PR linked to this issue  If I'm late to reply you, just @ me. -- This is an automated message from the Apache Git Service.

[GitHub] [apisix-ingress-controller] tao12345666333 closed issue #768: request help: path like /api/*/action

2021-12-07 Thread GitBox
tao12345666333 closed issue #768: URL: https://github.com/apache/apisix-ingress-controller/issues/768 -- 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,

[GitHub] [apisix-ingress-controller] tao12345666333 merged pull request #779: feat: support regex in path

2021-12-07 Thread GitBox
tao12345666333 merged pull request #779: URL: https://github.com/apache/apisix-ingress-controller/pull/779 -- 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

[apisix-ingress-controller] branch master updated: feat: support regex in path (#779)

2021-12-07 Thread zhangjintao
This is an automated email from the ASF dual-hosted git repository. zhangjintao 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 4e84eb8 feat: support

[GitHub] [apisix-ingress-controller] tao12345666333 commented on pull request #779: feat: support regex in path

2021-12-07 Thread GitBox
tao12345666333 commented on pull request #779: URL: https://github.com/apache/apisix-ingress-controller/pull/779#issuecomment-988457214 ok I understand your situation, I have added it to v1.4. -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [apisix-dashboard] juzhiyuan commented on issue #2223: request help :apisix-dashboard Dockerfile and build.sh error

2021-12-07 Thread GitBox
juzhiyuan commented on issue #2223: URL: https://github.com/apache/apisix-dashboard/issues/2223#issuecomment-988454236 Hi @GhangZh, may I know if this issue gets resolved? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [apisix] zlhgo commented on issue #5596: request help: customize Nginx variable in the configuration

2021-12-07 Thread GitBox
zlhgo commented on issue #5596: URL: https://github.com/apache/apisix/issues/5596#issuecomment-988449351 ``` diff --git a/apisix/cli/ngx_tpl.lua b/apisix/cli/ngx_tpl.lua index f5fa5d6e..523ae4d7 100644 --- a/apisix/cli/ngx_tpl.lua +++ b/apisix/cli/ngx_tpl.lua @@ -573,6

[GitHub] [apisix] zero-create commented on issue #5723: CPU usage gets higher when Apache APISIX is running a long time

2021-12-07 Thread GitBox
zero-create commented on issue #5723: URL: https://github.com/apache/apisix/issues/5723#issuecomment-988434821 > related to: #5692 > > can you provide packet capture data for the network between apisix and etcd?

[GitHub] [apisix] zero-create commented on issue #5723: CPU usage gets higher when Apache APISIX is running a long time

2021-12-07 Thread GitBox
zero-create commented on issue #5723: URL: https://github.com/apache/apisix/issues/5723#issuecomment-988434211 > 相关: #5692 > > 能否为 apisix 和 etcd 之间的网络提供数据包捕获数据? [etcd1.zip](https://github.com/apache/apisix/files/7672821/etcd1.zip) -- This is an automated message from

[GitHub] [apisix] zero-create removed a comment on issue #5723: CPU usage gets higher when Apache APISIX is running a long time

2021-12-07 Thread GitBox
zero-create removed a comment on issue #5723: URL: https://github.com/apache/apisix/issues/5723#issuecomment-988434211 > 相关: #5692 > > 能否为 apisix 和 etcd 之间的网络提供数据包捕获数据? [etcd1.zip](https://github.com/apache/apisix/files/7672821/etcd1.zip) -- This is an automated message

[GitHub] [apisix] xuminwlt commented on issue #5729: request help: module 'toolkit.json' not found when use test-nginx for unit test

2021-12-07 Thread GitBox
xuminwlt commented on issue #5729: URL: https://github.com/apache/apisix/issues/5729#issuecomment-988432810 Problem solved,clone to the t/ path is right,i have copied it to /t/lib, /t/lib is wrong. -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [apisix] xuminwlt closed issue #5729: request help: module 'toolkit.json' not found when use test-nginx for unit test

2021-12-07 Thread GitBox
xuminwlt closed issue #5729: URL: https://github.com/apache/apisix/issues/5729 -- 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:

[GitHub] [apisix-ingress-controller] lxm commented on pull request #779: feat: support regex in path

2021-12-07 Thread GitBox
lxm commented on pull request #779: URL: https://github.com/apache/apisix-ingress-controller/pull/779#issuecomment-988431614 > Thd code LGTM > > I added the label of `do-not-merge/hold` to it because this feature is not planned to be in the v1.4 version. > > Do you have a

[GitHub] [apisix-ingress-controller] tao12345666333 commented on pull request #779: feat: support regex in path

2021-12-07 Thread GitBox
tao12345666333 commented on pull request #779: URL: https://github.com/apache/apisix-ingress-controller/pull/779#issuecomment-988423022 Thd code LGTM I added the label of `do-not-merge/hold` to it because this feature is not planned to be in the v1.4 version. Do you have a

[GitHub] [apisix] xuminwlt commented on issue #5729: request help: module 'toolkit.json' not found when use test-nginx for unit test

2021-12-07 Thread GitBox
xuminwlt commented on issue #5729: URL: https://github.com/apache/apisix/issues/5729#issuecomment-988419811 > 1. it is not recommended that apisix be placed in the `root` directory; > 2. basically my directory structure is similar to yours. Looking at the search path in the error

[GitHub] [apisix-dashboard] juzhiyuan commented on pull request #2200: test: migrate route_export_test e2e

2021-12-07 Thread GitBox
juzhiyuan commented on pull request #2200: URL: https://github.com/apache/apisix-dashboard/pull/2200#issuecomment-988412954 ![image](https://user-images.githubusercontent.com/2106987/145132607-7a867199-9840-4446-b52f-0d185173bcf4.png) Maybe due to the force-push? -- This is an

[GitHub] [apisix-website] juzhiyuan commented on issue #792: Keep the same language in blog

2021-12-07 Thread GitBox
juzhiyuan commented on issue #792: URL: https://github.com/apache/apisix-website/issues/792#issuecomment-988410907 **Update** I noticed our Medium has some images in Chinese 樂 We will improve them **in the future**. -- This is an automated message from the Apache Git Service. To

[GitHub] [apisix-website] juzhiyuan edited a comment on issue #792: Keep the same language in blog

2021-12-07 Thread GitBox
juzhiyuan edited a comment on issue #792: URL: https://github.com/apache/apisix-website/issues/792#issuecomment-988410907 **Update** I noticed our Medium has some images in Chinese 樂 We will improve them **in the future**. http://medium.com/@apacheapisix -- This is an

[GitHub] [apisix] tokers commented on pull request #5718: docs: refactor Installation Guide

2021-12-07 Thread GitBox
tokers commented on pull request #5718: URL: https://github.com/apache/apisix/pull/5718#issuecomment-988409981 > > How about doing it in another PR? Because this PR's destination is to refactor only Install part. > > I think so +1. -- This is an automated message from the

[GitHub] [apisix] spacewander merged pull request #5715: fix(log-rotate): after enabling compression collect log exceptions

2021-12-07 Thread GitBox
spacewander merged pull request #5715: URL: https://github.com/apache/apisix/pull/5715 -- 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:

[GitHub] [apisix] spacewander closed issue #5658: request help: unable to collect logs when log-rotate plugin is enabled

2021-12-07 Thread GitBox
spacewander closed issue #5658: URL: https://github.com/apache/apisix/issues/5658 -- 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:

[apisix] branch master updated: fix(log-rotate): after enabling compression collect log exceptions (#5715)

2021-12-07 Thread spacewander
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 139c397 fix(log-rotate): after enabling

[GitHub] [apisix] spacewander closed issue #5663: bug: log-rotate.lua

2021-12-07 Thread GitBox
spacewander closed issue #5663: URL: https://github.com/apache/apisix/issues/5663 -- 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:

[GitHub] [apisix] spacewander merged pull request #5717: test(kafka-logger): reduce duplicate sections

2021-12-07 Thread GitBox
spacewander merged pull request #5717: URL: https://github.com/apache/apisix/pull/5717 -- 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:

[apisix] branch master updated: test(kafka-logger): reduce duplicate sections (#5717)

2021-12-07 Thread spacewander
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 bc091ad test(kafka-logger): reduce

[GitHub] [apisix] membphis merged pull request #5716: chore: delete expired inspiration statements

2021-12-07 Thread GitBox
membphis merged pull request #5716: URL: https://github.com/apache/apisix/pull/5716 -- 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:

[apisix] branch master updated: chore: delete expired inspiration statements (#5716)

2021-12-07 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.git The following commit(s) were added to refs/heads/master by this push: new 5f50216 chore: delete expired inspiration

[GitHub] [apisix] tzssangglass commented on pull request #5718: docs: refactor Installation Guide

2021-12-07 Thread GitBox
tzssangglass commented on pull request #5718: URL: https://github.com/apache/apisix/pull/5718#issuecomment-988098157 > How about doing it in another PR? Because this PR's destination is to refactor only Install part. I think so -- This is an automated message from the Apache Git

[GitHub] [apisix] juzhiyuan commented on pull request #5718: docs: refactor Installation Guide

2021-12-07 Thread GitBox
juzhiyuan commented on pull request #5718: URL: https://github.com/apache/apisix/pull/5718#issuecomment-988046347 > are we going to do it in this PR? How about doing it in another PR? Because this PR's destination is to refactor only Install part. -- This is an automated message

[GitHub] [apisix] alex8224 edited a comment on issue #5706: request help: How to route according to the configuration of nacos metadata?

2021-12-07 Thread GitBox
alex8224 edited a comment on issue #5706: URL: https://github.com/apache/apisix/issues/5706#issuecomment-988023902 @tzssangglass **If so, that's pretty good** Generally use the 127.0.0.1:8848/nacos/v1/ns/instance/list?serviceName=xxx=dev interface to get all service instances

[GitHub] [apisix] alex8224 commented on issue #5706: request help: How to route according to the configuration of nacos metadata?

2021-12-07 Thread GitBox
alex8224 commented on issue #5706: URL: https://github.com/apache/apisix/issues/5706#issuecomment-988023902 @tzssangglass **If so, that's pretty good** Generally use the 127.0.0.1:8848/nacos/v1/ns/instance/list?serviceName=xxx=dev interface to get all service instances under a

[GitHub] [apisix] tzssangglass commented on pull request #5718: docs: refactor Installation Guide

2021-12-07 Thread GitBox
tzssangglass commented on pull request #5718: URL: https://github.com/apache/apisix/pull/5718#issuecomment-988000493 > It's my understanding that we can prepare two files, the one for installation and the other one for building from the source. agree. are we going to do it in

[GitHub] [apisix] tzssangglass commented on a change in pull request #5718: docs: refactor Installation Guide

2021-12-07 Thread GitBox
tzssangglass commented on a change in pull request #5718: URL: https://github.com/apache/apisix/pull/5718#discussion_r764071076 ## File path: docs/zh/latest/how-to-build.md ## @@ -94,6 +95,23 @@ sudo yum install -y https://repos.apiseven.com/packages/centos/apache-apisix-rep

[GitHub] [apisix] tzssangglass commented on a change in pull request #5718: docs: refactor Installation Guide

2021-12-07 Thread GitBox
tzssangglass commented on a change in pull request #5718: URL: https://github.com/apache/apisix/pull/5718#discussion_r764070892 ## File path: docs/en/latest/how-to-build.md ## @@ -94,6 +95,23 @@ Please refer to: [Installing Apache APISIX with Helm Chart](https://github.com/a

[GitHub] [apisix] tzssangglass commented on a change in pull request #5718: docs: refactor Installation Guide

2021-12-07 Thread GitBox
tzssangglass commented on a change in pull request #5718: URL: https://github.com/apache/apisix/pull/5718#discussion_r764070493 ## File path: docs/en/latest/how-to-build.md ## @@ -94,6 +95,23 @@ Please refer to: [Installing Apache APISIX with Helm Chart](https://github.com/a

[GitHub] [apisix-dashboard] codecov-commenter edited a comment on pull request #2200: test: migrate route_export_test e2e

2021-12-07 Thread GitBox
codecov-commenter edited a comment on pull request #2200: URL: https://github.com/apache/apisix-dashboard/pull/2200#issuecomment-963965563 #

[GitHub] [apisix] tzssangglass commented on issue #5719: bug: when I use the kafka-logger plugin, failed to fetch the time in kafka

2021-12-07 Thread GitBox
tzssangglass commented on issue #5719: URL: https://github.com/apache/apisix/issues/5719#issuecomment-987946660 pls describe it more clearly. From the information you have provided, I do not know what is `failed to fetch the time in kafka` -- This is an automated message from the Apache

[GitHub] [apisix] tzssangglass commented on issue #5729: request help: module 'toolkit.json' not found when use test-nginx for unit test

2021-12-07 Thread GitBox
tzssangglass commented on issue #5729: URL: https://github.com/apache/apisix/issues/5729#issuecomment-987945840 1. it is not recommended that apisix be placed in the `root` directory; 2. basically my directory structure is similar to yours. Looking at the search path in the error

  1   2   >