Uploaded image for project: 'Beam'
  1. Beam
  2. BEAM-6958

Split Go PostCommit Test results by Runner

Details

    • Improvement
    • Status: Resolved
    • P3
    • Resolution: Fixed
    • None
    • Missing
    • sdk-go, testing
    • None

    Description

      At present the Go SDK only has a single column filled in on the master branch Post-Commit Tests Status testing rubric, which is unclear, and not-ideal.

      Right now the jenkin's Go PostCommit tests trigger the go integration test task, which only runs the tests on Dataflow via a shell script. It doesn't even run the unit tests as per the pre-commit.

      The end goal for this task is to:

      • Have the Go SDK column represent the Go SDK Unit Tests as a post commit.
          * Or better, to avoid pre-commit-run duplication, run the integration tests against the ULR if other runners are doing so.
      • Have the integration tests run against the Dataflow, be represented in the column.

      This will set the basis for adding and the integration tests against other portable runners (Flink, Spark, Python ULR, future portable runners...)

      It looks like there are three bits of work to accomplish here:

      • Adjust the Gradle tasks/task names to accurately represent what they're running against.
      • Add the new Jenkins tasks for each of the runners. (The other languages call these ValidateRunner_<RUNNER> tasks), 
      • Add the cool "badges" to the new Jenkins tasks to the Post Commit rubric.

       

      Attachments

        Activity

          People

            danoliveira Daniel Oliveira
            lostluck Robert Burke
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0h
                0h
                Logged:
                Time Spent - 5h 20m
                5h 20m