Skip to content

Conversation

@drodriguezhdez
Copy link
Contributor

@drodriguezhdez drodriguezhdez commented Mar 10, 2025

What does this PR do? What is the motivation?

Add the documentation related to the new feature in CI Visibility called "CI Jobs Failure Analysis"

Merge instructions

Merge readiness:

  • Ready for merge

Merge queue is enabled in this repo. To have it automatically merged after it receives the required reviews, create the PR (from a branch that follows the <yourname>/description naming convention) and then add the following PR comment:

/merge

Additional notes

@github-actions github-actions bot added the Images Images are added/removed with this PR label Mar 10, 2025
@joepeeples joepeeples self-requested a review March 11, 2025 14:15
@drodriguezhdez drodriguezhdez marked this pull request as ready for review March 12, 2025 15:28
@drodriguezhdez drodriguezhdez requested review from a team as code owners March 12, 2025 15:28
@drodriguezhdez drodriguezhdez requested review from andrea-mosk and rodrimendoza and removed request for rodrimendoza March 12, 2025 15:28
@joepeeples joepeeples self-assigned this Mar 12, 2025
@joepeeples joepeeples added the editorial review Waiting on a more in-depth review label Mar 12, 2025
Copy link
Contributor

@joepeeples joepeeples left a comment

Choose a reason for hiding this comment

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

I only had time today to do a partial review. I'll finish up with the larger files tomorrow!

Co-authored-by: Joe Peeples <joe.peeples@datadoghq.com>
Copy link
Contributor

@joepeeples joepeeples left a comment

Choose a reason for hiding this comment

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

Finished the review with some suggested edits. Please ping me again when it's revised for a quick final review, thanks!

drodriguezhdez and others added 2 commits March 13, 2025 16:21
Co-authored-by: Joe Peeples <joe.peeples@datadoghq.com>
Copy link
Contributor

@joepeeples joepeeples left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@drodriguezhdez
Copy link
Contributor Author

/merge

@dd-devflow
Copy link

dd-devflow bot commented Mar 31, 2025

View all feedbacks in Devflow UI.
2025-03-31 11:53:22 UTC ℹ️ Start processing command /merge


2025-03-31 11:53:27 UTC ℹ️ MergeQueue: pull request added to the queue

The expected merge time in master is approximately 17m (p90).


2025-03-31 12:08:21 UTC ℹ️ MergeQueue: This merge request was merged

@dd-mergequeue dd-mergequeue bot merged commit 447e213 into master Mar 31, 2025
16 of 18 checks passed
@dd-mergequeue dd-mergequeue bot deleted the drodriguezhdez/add_public_docs_log_summarization branch March 31, 2025 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

editorial review Waiting on a more in-depth review Images Images are added/removed with this PR mergequeue-status: done

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants