You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@trafficserver.apache.org by "jpeach (via GitHub)" <gi...@apache.org> on 2023/06/05 05:49:24 UTC

[GitHub] [trafficserver] jpeach opened a new pull request, #9774: doc: update the autest README

jpeach opened a new pull request, #9774:
URL: https://github.com/apache/trafficserver/pull/9774

   Remove the mention of bootstrap.py which doesn't seem to exist. Add some basic introduction on how to run tests using the `autest.sh` wrapper 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.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [trafficserver] jpeach merged pull request #9774: doc: update the autest README

Posted by "jpeach (via GitHub)" <gi...@apache.org>.
jpeach merged PR #9774:
URL: https://github.com/apache/trafficserver/pull/9774


-- 
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-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [trafficserver] dragon512 commented on a diff in pull request #9774: doc: update the autest README

Posted by "dragon512 (via GitHub)" <gi...@apache.org>.
dragon512 commented on code in PR #9774:
URL: https://github.com/apache/trafficserver/pull/9774#discussion_r1218645470


##########
tests/README.md:
##########
@@ -40,6 +38,41 @@ To run autest manually, the recommended way is to follow these steps:
 3. **cd gold_tests**: enter the directory containing the test files.
 4. **autest --ats-bin user_ats_bin**: run autest where user_ats_bin is the bin directory in the user's ats directory.
 
+# Running tests

Review Comment:
   You might want to add a note to configure the trafficserver build to install in a location as such ~/ats
   
   I tend to find this helps a lot vs installing on the system itself



-- 
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-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [trafficserver] jpeach commented on pull request #9774: doc: update the autest README

Posted by "jpeach (via GitHub)" <gi...@apache.org>.
jpeach commented on PR #9774:
URL: https://github.com/apache/trafficserver/pull/9774#issuecomment-1583956065

   @dragon512 can you please review the updates?


-- 
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-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org