[BEAM-6439] Move Python Validates Runner Flink test to PreCommit

[angoenka]

Please add a meaningful description for your change here

---

Follow this checklist to help us incorporate your contribution quickly and easily:

• Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
• If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.

Post-Commit Tests Status (on master branch)

Lang	SDK	Apex	Dataflow	Flink	Gearpump	Samza	Spark
Go		---	---	---	---	---	---
Java
Python		---			---	---	---

[angoenka]

Run Seed Job

[HuangLED]

No new line at end of file.

[tweise]

See comments on #7514

Please set previous name and also adjust references to this job where necessary (git grep is your friend).

[angoenka]

Run Python_VR PreCommit

[angoenka]

Run Python_VR_Flink PreCommit

[angoenka]

Run Python_VR_Flink PreCommit

[angoenka]

Can't test the job as beam1 is down and all runs are scheduled on beam1.

[angoenka]

Run Seed Job

[angoenka]

Run Seed Job

[angoenka]

Run Seed Job

[angoenka]

Run Seed Job

[angoenka]

Run Python_PVR PreCommit

[angoenka]

Run Python_PVR_Flink PreCommit

[angoenka]

Need to name job to Python_PVR_Flink as beam_PreCommit_Python_VR_Flink_Cron already existed. We can try deleting beam_PreCommit_Python_VR_Flink_Cron and then try to use Python_VR_Flink.
Successful run: https://builds.apache.org/job/beam_PreCommit_Python_PVR_Flink_Phrase/1/

[angoenka]

cc: @tweise @robertwb

[angoenka]

Ping: @tweise @robertwb

[angoenka]

Ping: @tweise @robertwb

[tweise]

git grep 'beam_PostCommit_Python_VR_Flink' - see earlier comment

[angoenka]

Now I see what you mean. Thanks!
Updated the PR comments and readme to remove reference for the old postcommit job.
But I would say that having these run available in post commit and visible in PR gave a lot of confidence in the system.

[tweise]

Hopefully pre-commit check signal will help prevent some of the build breakages we saw lately. But I don't think running all tests of a runners x SDK matrix is feasible going forward. Even just this combination, once it matches the coverage of the Java SDK, is probably no longer suitable for pre-commit.

[tweise]

@angoenka it does not appear that the job actually runs in pre-commit.

https://builds.apache.org/job/beam_PreCommit_Python_PVR_Flink_Commit/