[jira] [Work logged] (BEAM-10527) Python2_PVR_Flink precommit should publish test results to Jenkins

["ASF GitHub Bot (Jira)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/BEAM-10527?focusedWorklogId=465583&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-465583 ]

ASF GitHub Bot logged work on BEAM-10527:
-----------------------------------------

                Author: ASF GitHub Bot
            Created on: 03/Aug/20 08:48
            Start Date: 03/Aug/20 08:48
    Worklog Time Spent: 10m 
      Work Description: mxm commented on pull request #12385:
URL: https://github.com/apache/beam/pull/12385#issuecomment-667895488


   Thanks for fixing the quoting issue!
   
   > I got `flinkCompatibilityMatrixPROCESS` to pass on my machine by escaping the arguments via `${1@Q}`. Apparently whatever shell Jenkins is using does not support this. I will have to find a better solution.
   
   I think that feature only works in newer versions of `bash`. 
   
   > why do we need `flinkCompatibilityMatrixPROCESS` in the first place, when it is not being run anywhere? If it's important, shouldn't we add it to some postcommit?
   
   `flinkCompatibilityMatrixPROCESS` was how we ran the PVR tests to avoid a dependency on the container build (for speed). I was under the assumption that we are still doing that. That probably changed when we added the external transform tests which require the Java container. I agree that we should at least have a post commit.


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


Issue Time Tracking
-------------------

    Worklog Id:     (was: 465583)
    Time Spent: 3h 40m  (was: 3.5h)

> Python2_PVR_Flink precommit should publish test results to Jenkins
> ------------------------------------------------------------------
>
>                 Key: BEAM-10527
>                 URL: https://issues.apache.org/jira/browse/BEAM-10527
>             Project: Beam
>          Issue Type: Improvement
>          Components: testing
>            Reporter: Kyle Weaver
>            Assignee: Kyle Weaver
>            Priority: P2
>          Time Spent: 3h 40m
>  Remaining Estimate: 0h
>
> Right now we only have the logs, which often require scrolling up to see the failure (which itself often requires curl'ing the logs because they are too large for a browser to load comfortably). This causes frequent misunderstandings. For example, folks often mistake errors printed by pipelines that are meant to fail (e.g. test_error_message_includes_stage) for actual test failures.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)