OCPBUGS-63307: Fix image policy event intervals and bookkeeping#30991
OCPBUGS-63307: Fix image policy event intervals and bookkeeping#30991haircommander wants to merge 1 commit intoopenshift:mainfrom
Conversation
|
Pipeline controller notification For optional jobs, comment This repository is configured in: automatic mode |
|
@haircommander: This pull request references Jira Issue OCPBUGS-63307, which is valid. The bug has been moved to the POST state. 3 validation(s) were run on this bug
The bug has been updated to refer to the pull request using the external bug tracker. DetailsIn response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughThree event matcher functions in the pathological event library were modified to increase their buffered overlap windows from ±30 seconds to ±20 minutes and a blank line was added after each Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
Scheduling required tests: |
|
The increase the intervals commit looks good. Waiting on possible rework on the deferclean up commit. |
248685a to
46ee5b7
Compare
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: cpmeadors, haircommander The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
1 similar comment
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: cpmeadors, haircommander The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
@haircommander: This pull request references Jira Issue OCPBUGS-63307, which is valid. 3 validation(s) were run on this bug
DetailsIn response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
|
Scheduling required tests: |
|
@haircommander: The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions 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. I understand the commands that are listed here. |
|
/payload-job periodic-ci-openshift-release-main-ci-4.22-e2e-azure-ovn-techpreview-serial |
|
@BhargaviGudi: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/a488aa10-34b9-11f1-9ef8-5eddfc43811b-0 |
|
/payload-job periodic-ci-openshift-release-main-ci-4.21-e2e-azure-ovn-techpreview-serial |
|
@BhargaviGudi: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/b01b0170-34b9-11f1-8baa-b6739f6b7d46-0 |
|
Test case |
All of the other intervals are around 10 minutes. These events are triggered by MCP rollout, which happens for each of the image policy tests. Signed-off-by: Peter Hunt <pehunt@redhat.com>
46ee5b7 to
960bfdf
Compare
|
New changes are detected. LGTM label has been removed. |
|
/payload-job periodic-ci-openshift-release-main-ci-4.22-e2e-azure-ovn-techpreview-serial |
|
@haircommander: trigger 2 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/2d8c3cf0-3505-11f1-9adb-e9ec8aa322fd-0 |
|
Scheduling required tests: |
Assisted by Claude.
All of the other intervals are around 10 minutes. These events are triggered by MCP rollout, which happens for each of the image policy tests.
10 minutes is actually on the lower end for MCP rollout (which is given 20 minutes before timing out), but let's start with this and increase
in the future if necessary