[trafficserver] branch 9.0.x updated: Change the ready checks for ats to use the actual ats port and not the server port. Add ready check on server port for the server

[zw...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

zwoop pushed a commit to branch 9.0.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/9.0.x by this push:
     new 3a2740d  Change the ready checks for ats to use the actual ats port and not the server port. Add ready check on server port for the server
3a2740d is described below

commit 3a2740df9993c3d564d6b7759123a8845f8e9458
Author: Evan Zelkowitz <19...@users.noreply.github.com>
AuthorDate: Fri Sep 27 13:24:47 2019 -0600

    Change the ready checks for ats to use the actual ats port and not the server port. Add ready check on server port for the server
    
    (cherry picked from commit 29b1dc90e276813dd1b62cde881bb8f806065eaa)
---
 tests/gold_tests/headers/syntax.test.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/gold_tests/headers/syntax.test.py b/tests/gold_tests/headers/syntax.test.py
index c10ab02..dbaacb9 100644
--- a/tests/gold_tests/headers/syntax.test.py
+++ b/tests/gold_tests/headers/syntax.test.py
@@ -40,8 +40,8 @@ ts.Disk.remap_config.AddLine(
 
 # Test 1 - 200 Response
 tr = Test.AddTestRun()
-tr.Processes.Default.StartBefore(server)
-tr.Processes.Default.StartBefore(Test.Processes.ts, ready=When.PortOpen(server.Variables.Port))
+tr.Processes.Default.StartBefore(server, ready=When.PortOpen(server.Variables.Port))
+tr.Processes.Default.StartBefore(Test.Processes.ts, ready=When.PortOpen(ts.Variables.port))
 tr.Processes.Default.Command = 'curl -s -D - -v --ipv4 --http1.1 -H " foo: bar" -H "Host: www.example.com" http://localhost:{0}/'.format(ts.Variables.port)
 tr.Processes.Default.ReturnCode = 0
 tr.Processes.Default.Streams.stdout = "syntax.200.gold"