push-test-metrics: hard coded periodic names

[dhiller]

What happened:
Periodic names that are looked at are hard coded:

project-infra/robots/push-test-metrics/main.go

Line 57 in 16ce7a1

jobNamesToScan = jobNames{

This generates test times from old job runs and ridicules the values.

What you expected to happen:
Job names should be deferred from the kubevirt-periodics.yaml
Use

project-infra/vendor/sigs.k8s.io/prow/pkg/config/config.go

Line 1678 in 1e74906

func ReadJobConfig(jobConfig string, yamlOpts ...yaml.JSONOpt) (JobConfig, error) {

func to read the config.

How to reproduce it (as minimally and precisely as possible):
See link, see k8s lane names in grafana panel: https://grafana.ci.kubevirt.io/d/gY4hNc5Iz/kubevirt-kubevirt-e2e-test-runtimes?orgId=1&editPanel=6

Additional context:
n/a

/help
/good-first-issue

[kubevirt-bot]

@dhiller:
This request has been marked as suitable for new contributors.

Guidelines

• No Barrier to Entry
• Clear Task
• Solution Explained
• Provides Context
• Identifies Relevant Code
• Gives Examples
• Ready to Test
• Goldilocks priority
• Up-To-Date

For more details on the requirements of such an issue, please see here and ensure that they are met.

If this request no longer meets these requirements, the label can be removed
by commenting with the /remove-good-first-issue command.

Details

In response to this:

What happened:
Periodic names that are looked at are hard coded:

project-infra/robots/push-test-metrics/main.go

Line 57 in 16ce7a1

jobNamesToScan = jobNames{

This generates test times from old job runs and ridicules the values.

What you expected to happen:
Job names should be deferred from the kubevirt-periodics.yaml

How to reproduce it (as minimally and precisely as possible):
See link, see k8s lane names in grafana panel: https://grafana.ci.kubevirt.io/d/gY4hNc5Iz/kubevirt-kubevirt-e2e-test-runtimes?orgId=1&editPanel=6

Additional context:
n/a

/help
/good-first-issue

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

[dhiller]

/triage accepted

[dhiller]

/priority backlog

[Jaydeep869]

@dhiller I’d like to work on this. I plan to replace the hard coded default job list in push test metrics with names read from the kubevirt periodics config, while keeping explicit job name flags as an override. Please assign it to me if that is the preferred workflow.