bneradt commented on PR #12211:
URL: https://github.com/apache/trafficserver/pull/12211#issuecomment-2836865173

   @JakeChampion : Thank you for working on this. It looks like a few autests 
have output expectation issues.
   
   The `compress` autest fails due to zstd beign in the output now. Should be a 
simple gold file update required 
(`src/tests/gold_tests/pluginTest/compress/compress_userver.gold`):
   
   ```
      Run: 31-tr: Failed
        file compress_userver.log : Checking that compress_userver.log matches 
compress_userver.gold - Failed
           Reason: File differences
            Gold File : 
/home/jenkins/workspace/Github_Builds/autest/src/tests/gold_tests/pluginTest/compress/compress_userver.gold
            Data File : /tmp/sandbox/compress/compress_userver.log
            - 0/gzip, deflate, sdch, br
            + 0/zstd, gzip, deflate, sdch, br
            ?   ++++++
            
              0/gzip
              0/br
              0/deflate
            + 0/zstd
            - 1/gzip, deflate, sdch, br
            + 1/zstd, gzip, deflate, sdch, br
            ?   ++++++
            
              1/gzip
              1/br
              1/deflate
            + 1/zstd
            - 2/gzip, deflate, sdch, br
            + 2/zstd, gzip, deflate, sdch, br
            ?   ++++++
            
              2/gzip
              2/br
              2/deflate
            + 2/zstd
            + 3/zstd, gzip, deflate, sdch, br
            + 3/gzip
            + 3/br
            + 3/deflate
            + 3/zstd
              0/gzip;q=0.666
              0/gzip;q=0.666x
              0/gzip;q=#0.666
              0/gzip; Q = 0.666
              0/gzip;q=0.0
              0/gzip;q=-0.1
              0/aaa, gzip;q=0.666, bbb
              0/ br ; q=0.666, bbb
              0/aaa, gzip;q=0.666 , 
              3/gzip
   ```
   
   Also, the `normalized_ae_match_vary_cache` autest fails, it looks like also 
for output expecation update needs:
   
   ```
          file 
/tmp/sandbox/normalized_ae_match_vary_cache/_output/0-tr-Default/stream.stdout.txt
 : There should be no Proxy Verifier violation errors. - Failed
             Reason: Contents of 
/tmp/sandbox/normalized_ae_match_vary_cache/_output/0-tr-Default/stream.stdout.txt
 contains expression: "Violation|Invalid status"
                Details:
                  [INFO]: Equals Violation: Different. Key: "34", Field Name: 
"x-response-identifier", Correct Value: "Zstd-Accept-Encoding", Actual Value: 
"Br-Accept-Encoding" : 80
                  [INFO]: Equals Violation: Different. Key: "34", Field Name: 
"x-cache", Correct Value: "miss", Actual Value: "hit-fresh" : 81
                  [INFO]: Equals Violation: Different. Key: "38", Field Name: 
"x-response-identifier", Correct Value: "Zstd-Accept-Encoding", Actual Value: 
"Br-Accept-Encoding" : 98
   ```


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

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

Reply via email to