Skip to content

Apiserverpprof #30589

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
Draft
benluddy wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

Apiserverpprof #30589

benluddy wants to merge 6 commits into from
+175 −0

Conversation

@benluddy
Copy link
Contributor

@benluddy benluddy commented Dec 10, 2025 •
edited
Loading

this is just #30587 with the endpoint changed to fetch heap profiles

dgoodwin and others added 6 commits December 9, 2025 11:48
@dgoodwin
Add apiserverpprof monitortest to poll for CPU data
eb40226 
Part of investigating the apiserver outage bug on-going. This collects
timestamps pprof files every so many seconds and stores in the artifacts
dir. This should allow much better insight into the apiserver at times
of outages.
@dgoodwin
30 seconds not 10 for timeout
4492ae7
@dgoodwin
Longer poll interval
209fc36
@dgoodwin
Continuous poll every 15s
da95c9c
@dgoodwin
Memory efficiency with immediate writes
eef37c9
@benluddy
grab heap profiles instead
2cfdf83
@openshift-ci-robot
Copy link

openshift-ci-robot commented Dec 10, 2025

Pipeline controller notification
This repo is configured to use the pipeline controller. Second-stage tests will be triggered either automatically or after lgtm label is added, depending on the repository configuration. The pipeline controller will automatically detect which contexts are required and will utilize /test Prow commands to trigger the second stage.

For optional jobs, comment /test ? to see a list of all defined jobs. To trigger manually all jobs from second stage use /pipeline required command.

This repository is configured in: automatic mode

@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Dec 10, 2025
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Dec 10, 2025

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@benluddy
Copy link
Contributor Author

benluddy commented Dec 10, 2025

/payload-aggregate periodic-ci-openshift-release-master-nightly-4.21-e2e-aws-ovn-upgrade-fips 4

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Dec 10, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: benluddy
Once this PR has been reviewed and has the lgtm label, please assign stbenjam for approval. For more information see the Code Review Process.

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

Details 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

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Dec 10, 2025

@benluddy: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

  • periodic-ci-openshift-release-master-nightly-4.21-e2e-aws-ovn-upgrade-fips

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/0e95ab90-d55d-11f0-83ca-8f7878e22bd8-0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@benluddy @openshift-ci-robot @dgoodwin