Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[release-4.12] OCPBUGS-8024: Backport fixes to resume gathering CI disruption data for SLB and image registry #27759

Merged
merged 3 commits into from
Mar 1, 2023
Merged

[release-4.12] OCPBUGS-8024: Backport fixes to resume gathering CI disruption data for SLB and image registry #27759

merged 3 commits into from
Mar 1, 2023

Commits on Feb 28, 2023

  1. Fix missing disruption uploads. 

    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.
    @dgoodwin
    dgoodwin committed Feb 28, 2023
    Configuration menu
    Copy the full SHA
    ed21280 View commit details
    Browse the repository at this point in the history

  2. Fix missing disruption data again. 

    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
    dgoodwin committed Feb 28, 2023
    Configuration menu
    Copy the full SHA
    7c7d2e2 View commit details
    Browse the repository at this point in the history

  3. Add logging for the job type we determined

    @dgoodwin
    dgoodwin committed Feb 28, 2023
    Configuration menu
    Copy the full SHA
    ccca62d View commit details
    Browse the repository at this point in the history