Abacn commented on code in PR #29270: URL: https://github.com/apache/beam/pull/29270#discussion_r1380396283
########## .test-infra/metrics/sync/github/sync_workflows.py: ########## @@ -47,6 +47,330 @@ GH_NUMBER_OF_WORKFLOW_RUNS_TO_FETCH =\ os.environ['GH_NUMBER_OF_WORKFLOW_RUNS_TO_FETCH'] +# Maps workflows to dashboard category. Any workflows not in one of these lists +# will get auto-mapped to misc. +CORE_JAVA_TESTS = [ + 'PreCommit SQL Java17', Review Comment: Or, let's keep the current inferred category logic and use this list to __overwrite__ the category, instead of expecting every test to have a explicit category. In this way, it may need specify "CORE_INFRA", but most tests in Spark / Flink / Load Test / Performance Test categories can be inferred. Then in most case we still do not need to maintain this category list, while still able to explicitly put a test in a category without rename all tests systematically -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
