New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[release-4.12] OCPBUGS-8024: Backport fixes to resume gathering CI disruption data for SLB and image registry #27759
[release-4.12] OCPBUGS-8024: Backport fixes to resume gathering CI disruption data for SLB and image registry #27759
Conversation
Discovered that we are getting no data for image registry since Dec 7th due to PR openshift#27574 which shut off the entire monitor if we have less than the required 100 runs to appear in the file of allowances to enforce. I guess at this time, we did not have enough runs with image registry for 4.13. (which is unusual) Skip the test, but let the monitor run.
Previous attempt did not work, as we were still bypassing FrameworkEventIntervals() which is needed to add the test summary, which eventually results in the AdditionalEvents json files in artifacts, which are then merged into the overall events and disruption data files. Without calling it, we still get no disruption data uploaded. Move the nil check into the ExpectNoDisruption function, and allow FrameworkEventIntervals to run as it should.
@dgoodwin: No Bugzilla bug is referenced in the title of this pull request. In 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 kubernetes/test-infra repository. |
@dgoodwin: This pull request references Jira Issue OCPBUGS-8024, which is invalid:
Comment The bug has been updated to refer to the pull request using the external bug tracker. In 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 kubernetes/test-infra repository. |
/jira refresh |
@dgoodwin: This pull request references Jira Issue OCPBUGS-8024, which is valid. The bug has been moved to the POST state. 6 validation(s) were run on this bug
In 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 kubernetes/test-infra repository. |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dgoodwin, xueqzhan 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 |
@dgoodwin: This pull request references Jira Issue OCPBUGS-8024, which is valid. 6 validation(s) were run on this bug
In 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 kubernetes/test-infra repository. |
@dgoodwin: No Bugzilla bug is referenced in the title of this pull request. Retaining the bugzilla/valid-bug label as it was manually added. In 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 kubernetes/test-infra repository. |
@dgoodwin: This pull request references Jira Issue OCPBUGS-8024, which is valid. 6 validation(s) were run on this bug
In 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 kubernetes/test-infra repository. |
@dgoodwin: No Bugzilla bug is referenced in the title of this pull request. Retaining the bugzilla/valid-bug label as it was manually added. In 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 kubernetes/test-infra repository. |
@dgoodwin: The following tests failed, say
Full PR test history. Your PR dashboard. 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. |
/test e2e-aws-ovn-serial |
@dgoodwin: All pull requests linked via external trackers have merged: Jira Issue OCPBUGS-8024 has been moved to the MODIFIED state. In 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 kubernetes/test-infra repository. |
[ART PR BUILD NOTIFIER] (beta) This PR has been included in build 2398560 for distgit openshift-enterprise-tests. |
These fixes were applied in 4.13 early this year, but the backport was missed, as such we are not gathering SLB or image registry disruption numbers in 4.12.
No product changes, just CI data gathering.
TRT-825