Skip to content

docs/devel: add CI monitoring information for developers#1221

Merged
k8s-ci-robot merged 1 commit into
kubernetes:masterfrom
mtulio:doc-devel-monitor
Sep 4, 2025
Merged

docs/devel: add CI monitoring information for developers#1221
k8s-ci-robot merged 1 commit into
kubernetes:masterfrom
mtulio:doc-devel-monitor

Conversation

@mtulio
Copy link
Copy Markdown
Contributor

@mtulio mtulio commented Aug 7, 2025

What type of PR is this?

/kind documentation

What this PR does / why we need it:

Update docs/development.md with useful information about monitoring dashboards to investigate CI build issues.

With this change, developers will have more information about failures in their PRs and CI test infrastructure, allowing them to investigate and distinguish between CI infrastructure issues and code-related failures.

The main motivation is to:

  • Decrease the total time for proposal readiness
  • Reduce maintainer time spent reviewing CI-related known issues
  • Maintain stability and confidence in the development process

Which issue(s) this PR fixes:

Fixes #

Related to:

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

NONE

Update development.md with useful information about monitoring
dashboards to investigate CI build issues.
@k8s-ci-robot k8s-ci-robot added release-note-none Denotes a PR that doesn't merit a release note. kind/documentation Categorizes issue or PR as related to documentation. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. labels Aug 7, 2025
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

This issue is currently awaiting triage.

If cloud-provider-aws contributors determine this is a relevant issue, they will accept it by applying the triage/accepted label and provide further guidance.

The triage/accepted label can be added by org members by writing /triage accepted in a comment.

Details

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-sigs/prow repository.

@k8s-ci-robot k8s-ci-robot requested review from dims and hakman August 7, 2025 14:09
@k8s-ci-robot k8s-ci-robot added the needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. label Aug 7, 2025
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

Hi @mtulio. Thanks for your PR.

I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

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-sigs/prow repository.

@k8s-ci-robot k8s-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Aug 7, 2025
@mtulio
Copy link
Copy Markdown
Contributor Author

mtulio commented Aug 26, 2025

/assign @elmiko @kmala

@kmala
Copy link
Copy Markdown
Member

kmala commented Aug 29, 2025

/ok-to-test

@k8s-ci-robot k8s-ci-robot added ok-to-test Indicates a non-member PR verified by an org member that is safe to test. and removed needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. labels Aug 29, 2025
@kmala
Copy link
Copy Markdown
Member

kmala commented Aug 29, 2025

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Aug 29, 2025
@jcpowermac
Copy link
Copy Markdown

/lgtm

@k8s-ci-robot
Copy link
Copy Markdown
Contributor

@jcpowermac: changing LGTM is restricted to collaborators

Details

In response to this:

/lgtm

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-sigs/prow repository.

Copy link
Copy Markdown
Contributor

@elmiko elmiko left a comment

Choose a reason for hiding this comment

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

/lgtm

@elmiko
Copy link
Copy Markdown
Contributor

elmiko commented Sep 4, 2025

/approve

@k8s-ci-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: elmiko

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

The pull request process is described 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

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 4, 2025
@k8s-ci-robot k8s-ci-robot merged commit 43543b3 into kubernetes:master Sep 4, 2025
8 checks passed
@mtulio mtulio deleted the doc-devel-monitor branch September 4, 2025 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/documentation Categorizes issue or PR as related to documentation. lgtm "Looks good to me", indicates that a PR is ready to be merged. needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. ok-to-test Indicates a non-member PR verified by an org member that is safe to test. release-note-none Denotes a PR that doesn't merit a release note. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants