test-infra icon indicating copy to clipboard operation
test-infra copied to clipboard

WIP: add release-team-jobs/periodics.yaml

Open Priyankasaggu11929 opened this issue 3 years ago • 3 comments

What this PR does / why we need it:

The PR adds a periodic Prow job periodic-sync-enhancements-github-project-beta-1-25 that will sync all open KEP issues from the k/enhancements repository under a current release milestone (say v1.25) to an Enhancements GitHub Project Beta board (similar to https://github.com/orgs/kubernetes/projects/87).

Which issue(s) this PR fixes:

Part of https://github.com/kubernetes/community/issues/6735

Priyankasaggu11929 avatar Jul 14 '22 14:07 Priyankasaggu11929

/hold

Remove hold after PR https://github.com/kubernetes/sig-release/pull/1968 is merged

Priyankasaggu11929 avatar Jul 14 '22 14:07 Priyankasaggu11929

@ameukam need help with the following error.

What needs to be added to the job to meet the requirements of the k8s-infra-prow-build-trusted CI policy?

Screenshot 2022-09-09 at 10 09 40 PM

Priyankasaggu11929 avatar Sep 09 '22 16:09 Priyankasaggu11929

@ameukam need help with the following error.

What needs to be added to the job to meet the requirements of the k8s-infra-prow-build-trusted CI policy?

Screenshot 2022-09-09 at 10 09 40 PM

I think you have a location issue. Any prowjob running on k8s-infra-prow-build-trusted should be in config/jobs/kubernetes/sig-k8s-infra/trusted/

ameukam avatar Sep 09 '22 16:09 ameukam

Thanks so much for the help, @ameukam. Changing the job location worked. All required tests are passing now (except one optional - pull-test-infra-prow-checkconfig-beta which seems to be failing recently for all PRs)

Arnaud, need your help with a final review & get the merged asap. It is required for the current 1.26 release cycle's Enhancements tracking process.

cc: @leonardpahlke

Priyankasaggu11929 avatar Sep 10 '22 03:09 Priyankasaggu11929

Thanks for the quick turnaround! /lgtm

rhockenbury avatar Sep 10 '22 03:09 rhockenbury

Removing hold as PR https://github.com/kubernetes/sig-release/pull/1968 is now merged.

/unhold

Priyankasaggu11929 avatar Sep 10 '22 06:09 Priyankasaggu11929

@Priyankasaggu11929: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
pull-test-infra-prow-checkconfig-beta 2ba02116fc7f513e697cd3250da47109561325ed link false /test pull-test-infra-prow-checkconfig-beta

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

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/test-infra repository. I understand the commands that are listed here.

k8s-ci-robot avatar Sep 13 '22 10:09 k8s-ci-robot

@ameukam, I have updated the PR addressing the above comments. PTAL.

cc: @rhockenbury

Priyankasaggu11929 avatar Sep 13 '22 10:09 Priyankasaggu11929

/approve /skip pull-test-infra-prow-checkconfig-beta

ameukam avatar Sep 13 '22 10:09 ameukam

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ameukam, Priyankasaggu11929

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

k8s-ci-robot avatar Sep 13 '22 10:09 k8s-ci-robot

/lgtm

rhockenbury avatar Sep 13 '22 17:09 rhockenbury

@Priyankasaggu11929: Updated the job-config configmap in namespace default at cluster test-infra-trusted using the following files:

  • key release-team-periodics.yaml using file config/jobs/kubernetes/sig-k8s-infra/trusted/sig-release-release-team-jobs/release-team-periodics.yaml

In response to this:

What this PR does / why we need it:

The PR adds a periodic Prow job periodic-sync-enhancements-github-project-beta-1-25 that will sync all open KEP issues from the k/enhancements repository under a current release milestone (say v1.25) to an Enhancements GitHub Project Beta board (similar to https://github.com/orgs/kubernetes/projects/87).

Which issue(s) this PR fixes:

Part of https://github.com/kubernetes/community/issues/6735

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/test-infra repository.

k8s-ci-robot avatar Sep 13 '22 17:09 k8s-ci-robot

@ameukam - ~~Is there anything else required for the testgrid dashboard? I'm not seeing the sig-release-release-team-periodics dashboard on the sig-release testgrid?~~

Also, it looks like the first run of the job failed - https://prow.k8s.io/log?job=periodic-sync-enhancements-github-project-beta-1-26&id=1569743073582256128.

Would you be able to help assist in debugging this?

rhockenbury avatar Sep 13 '22 17:09 rhockenbury

https://github.com/kubernetes/sig-release/pull/2023 fixes the failure.

palnabarun avatar Sep 13 '22 18:09 palnabarun