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 2021/01/27 22:11:46 UTC

[GitHub] [trafficserver] bneradt opened a new pull request #7454: Updating to Proxy Verifier v2.0.0

bneradt opened a new pull request #7454:
URL: https://github.com/apache/trafficserver/pull/7454


   We will want to use Proxy Verifier v2.0.0 for it's delayed transaction
   and HTTP/2 to server capabilities. This updates the AuTest extensions to
   properly handle the new version.
   
   
   This is a draft. This PR will be updated once we get the new Proxy Verifier binaries into bintray.


----------------------------------------------------------------
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] SolidWallOfCode commented on pull request #7454: Updating to Proxy Verifier v2.0.0

Posted by GitBox <gi...@apache.org>.
SolidWallOfCode commented on pull request #7454:
URL: https://github.com/apache/trafficserver/pull/7454#issuecomment-771148326


   Is there a way to not have the PV version explicitly defined in different files?


----------------------------------------------------------------
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] bneradt commented on pull request #7454: Updating to Proxy Verifier v2.0.0

Posted by GitBox <gi...@apache.org>.
bneradt commented on pull request #7454:
URL: https://github.com/apache/trafficserver/pull/7454#issuecomment-770076358


   [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 #7454: Updating to Proxy Verifier v2.0.0

Posted by GitBox <gi...@apache.org>.
zwoop commented on pull request #7454:
URL: https://github.com/apache/trafficserver/pull/7454#issuecomment-772902302


   Cherry-picked to v9.1.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] bneradt commented on pull request #7454: Updating to Proxy Verifier v2.0.0

Posted by GitBox <gi...@apache.org>.
bneradt commented on pull request #7454:
URL: https://github.com/apache/trafficserver/pull/7454#issuecomment-770089939


   [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] bneradt commented on pull request #7454: Updating to Proxy Verifier v2.0.0

Posted by GitBox <gi...@apache.org>.
bneradt commented on pull request #7454:
URL: https://github.com/apache/trafficserver/pull/7454#issuecomment-771162933


   > Is there a way to not have the PV version explicitly defined in different files?
   
   Yes, good point. I'll create a `tests/proxy-verifier-version.txt` file with the version and use that in the `prepare_proxy_verifier.sh`, in the `setup.cli.ext` extension, and in the `ci/jenkins/bin/autest.sh` script.


----------------------------------------------------------------
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] bneradt commented on pull request #7454: Updating to Proxy Verifier v2.0.0

Posted by GitBox <gi...@apache.org>.
bneradt commented on pull request #7454:
URL: https://github.com/apache/trafficserver/pull/7454#issuecomment-770081753


   [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] bneradt commented on pull request #7454: Updating to Proxy Verifier v2.0.0

Posted by GitBox <gi...@apache.org>.
bneradt commented on pull request #7454:
URL: https://github.com/apache/trafficserver/pull/7454#issuecomment-770049862


   [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] bneradt commented on pull request #7454: Updating to Proxy Verifier v2.0.0

Posted by GitBox <gi...@apache.org>.
bneradt commented on pull request #7454:
URL: https://github.com/apache/trafficserver/pull/7454#issuecomment-770129903


   [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] bneradt commented on pull request #7454: Updating to Proxy Verifier v2.0.0

Posted by GitBox <gi...@apache.org>.
bneradt commented on pull request #7454:
URL: https://github.com/apache/trafficserver/pull/7454#issuecomment-770146422


   [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] SolidWallOfCode commented on pull request #7454: Updating to Proxy Verifier v2.0.0

Posted by GitBox <gi...@apache.org>.
SolidWallOfCode commented on pull request #7454:
URL: https://github.com/apache/trafficserver/pull/7454#issuecomment-771148090


   I can't set the verbosity level to "info" anymore?


----------------------------------------------------------------
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] bneradt commented on pull request #7454: Updating to Proxy Verifier v2.0.0

Posted by GitBox <gi...@apache.org>.
bneradt commented on pull request #7454:
URL: https://github.com/apache/trafficserver/pull/7454#issuecomment-770041233


   [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] bneradt commented on pull request #7454: Updating to Proxy Verifier v2.0.0

Posted by GitBox <gi...@apache.org>.
bneradt commented on pull request #7454:
URL: https://github.com/apache/trafficserver/pull/7454#issuecomment-771153363


   > I can't set the verbosity level to "info" anymore?
   
   You can. "info" will be what the user gets if they pass `verbose=False`. Currently all tests want debug, and that will almost certainly be the case for the majority of tests moving forward. If an arbitrary level is desired, they can combine `verbose=False` and `other_args="--verbose <level>"`.
   
   The interface in this PR makes the common debug case easy (it's the default) and the rarer case in which just info is wanted easy. The extremely rare case where an arbitrary level is desired (not sure why we'd ever want that in an autest) is also possible.


----------------------------------------------------------------
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] bneradt merged pull request #7454: Updating to Proxy Verifier v2.0.0

Posted by GitBox <gi...@apache.org>.
bneradt merged pull request #7454:
URL: https://github.com/apache/trafficserver/pull/7454


   


----------------------------------------------------------------
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 #7454: Updating to Proxy Verifier v2.0.0

Posted by GitBox <gi...@apache.org>.
zwoop commented on pull request #7454:
URL: https://github.com/apache/trafficserver/pull/7454#issuecomment-778269402


   Cherry-picked to v9.0.x branch.
   
   I also had to include all changes from master in ci/jenkins/bin, because it got maddening with merge conflicts.


----------------------------------------------------------------
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