Skip to content

Document prow alerts#483

Merged
0xmichalis merged 1 commit into
openshift:masterfrom
0xmichalis:more-docs-yay
Dec 6, 2017
Merged

Document prow alerts#483
0xmichalis merged 1 commit into
openshift:masterfrom
0xmichalis:more-docs-yay

Conversation

@0xmichalis
Copy link
Copy Markdown
Contributor

@stevekuznetsov more docs about our alerts

cc @paradigm

Signed-off-by: Michalis Kargakis <mkargaki@redhat.com>
@openshift-ci-robot openshift-ci-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Dec 6, 2017
@0xmichalis 0xmichalis merged commit 19d2164 into openshift:master Dec 6, 2017
@0xmichalis 0xmichalis deleted the more-docs-yay branch December 6, 2017 15:31
@0xmichalis
Copy link
Copy Markdown
Contributor Author

@stevekuznetsov feel free to edit them as you see fit. We need some insight in how to debug Jenkins

cc: @csrwng

Copy link
Copy Markdown
Contributor

@stevekuznetsov stevekuznetsov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SGTM for what we have. For Jenkins we need to sync with Gabe and Ben re: how we can get Prometheus metrics out of the Jenkins masters.

Comment thread README.md
* Slow Jenkins operator sync
* Slow Jenkins pipeline operator sync

These should not be a problem in general but if any of them persists for more than a couple of hours, [`max_goroutines`](https://github.com/openshift/release/blob/ff18182aa0eb849b89e7abd1bc7765ad6d27142f/cluster/ci/config/prow/config.yaml#L7) can be incremented to allow more parallelism in the operators (note that the same option dictates both operators).
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could also be Jenkins master being slow to respond, overloaded, etc?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, I need to craft a query that uses jenkins_requests_latency so we can figure out if Jenkins is the culprit.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Opened #485

Note though that it's unlike we are going to have slow syncs ever since kubernetes/test-infra#5774

derekhiggins pushed a commit to derekhiggins/release that referenced this pull request Oct 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants