You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@trafficserver.apache.org by GitBox <gi...@apache.org> on 2020/05/21 17:12:07 UTC
[GitHub] [trafficserver] traeak opened a new pull request #6817: lua plugin: fix for incorrectly injecting global plugin stats
traeak opened a new pull request #6817:
URL: https://github.com/apache/trafficserver/pull/6817
Fixes a bug where the global lua plugin stats are incorrectly assigned causing a collision with remap plugins.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] traeak removed a comment on pull request #6817: lua plugin: fix for incorrectly injecting global plugin stats
Posted by GitBox <gi...@apache.org>.
traeak removed a comment on pull request #6817:
URL: https://github.com/apache/trafficserver/pull/6817#issuecomment-634152817
[approve ci autest]
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] zwoop commented on pull request #6817: lua plugin: fix for incorrectly injecting global plugin stats
Posted by GitBox <gi...@apache.org>.
zwoop commented on pull request #6817:
URL: https://github.com/apache/trafficserver/pull/6817#issuecomment-637843040
Cherry-picked to v9.0.x branch.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] jvgutierrez commented on pull request #6817: lua plugin: fix for incorrectly injecting global plugin stats
Posted by GitBox <gi...@apache.org>.
jvgutierrez commented on pull request #6817:
URL: https://github.com/apache/trafficserver/pull/6817#issuecomment-632680242
I've tested this on top of our 8.x branch + #6802 + #4924 (to get a clean patch applied) and it solves the issues I've commented on #6802
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] randall commented on pull request #6817: lua plugin: fix for incorrectly injecting global plugin stats
Posted by GitBox <gi...@apache.org>.
randall commented on pull request #6817:
URL: https://github.com/apache/trafficserver/pull/6817#issuecomment-634322461
[approve ci autest]
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] randall merged pull request #6817: lua plugin: fix for incorrectly injecting global plugin stats
Posted by GitBox <gi...@apache.org>.
randall merged pull request #6817:
URL: https://github.com/apache/trafficserver/pull/6817
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] traeak commented on pull request #6817: lua plugin: fix for incorrectly injecting global plugin stats
Posted by GitBox <gi...@apache.org>.
traeak commented on pull request #6817:
URL: https://github.com/apache/trafficserver/pull/6817#issuecomment-634149140
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] shukitchan commented on pull request #6817: lua plugin: fix for incorrectly injecting global plugin stats
Posted by GitBox <gi...@apache.org>.
shukitchan commented on pull request #6817:
URL: https://github.com/apache/trafficserver/pull/6817#issuecomment-633099273
It looks good. 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 the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] traeak commented on pull request #6817: lua plugin: fix for incorrectly injecting global plugin stats
Posted by GitBox <gi...@apache.org>.
traeak commented on pull request #6817:
URL: https://github.com/apache/trafficserver/pull/6817#issuecomment-632720967
with the autest the traffic_ctl metric match is what's causing trouble. How to ensure it runs until the metrics show up?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org