Skip to content

Unrelated pending edits: CI config, AGENTS.md, NativeBuildPlugin doc#643

Merged
jbachorik merged 1 commit into
mainfrom
jb/check_me_out
Jul 9, 2026
Merged

Unrelated pending edits: CI config, AGENTS.md, NativeBuildPlugin doc#643
jbachorik merged 1 commit into
mainfrom
jb/check_me_out

Conversation

@jbachorik

Copy link
Copy Markdown
Collaborator

What does this PR do?:
Splits out three pre-existing local modifications that were unrelated to the
reference-chains work in progress on another branch, so they can be reviewed
independently:

  • .gitlab/reliability/.gitlab-ci.yml
  • AGENTS.md
  • doc/build/NativeBuildPlugin.md

Motivation:
These edits were sitting in the working tree from prior, separate work and
shouldn't be bundled into the reference-chains PR.

Additional Notes:
No functional relationship between the three files beyond "all pending at
the same time" - please review each independently.

How to test the change?:
Docs/CI-config only; no code changes.

For Datadog employees:

  • If this PR touches code that signs or publishes builds or packages, or handles
    credentials of any kind, I've requested a security review (run the dd:platform-security-review
    skill, or file a request via the PSEC review form).
    bewaire also runs automatically on every PR.
  • This PR doesn't touch any of that.
  • JIRA: [N/A]

Unsure? Have a question? Request a review!

🤖 Generated with Claude Code

Splitting out pre-existing local modifications unrelated to the
reference-chains work, for separate review.
@jbachorik jbachorik added the AI label Jul 8, 2026
@dd-octo-sts

dd-octo-sts Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

CI Test Results

Run: #29035344857 | Commit: 56e5adf | Duration: 17m 0s (longest job)

All 32 test jobs passed

Status Overview

JDK glibc-aarch64/debug glibc-amd64/debug musl-aarch64/debug musl-amd64/debug
8 - - -
8-ibm - - -
8-j9 - -
8-librca - -
8-orcl - - -
11 - - -
11-j9 - -
11-librca - -
17 - -
17-graal - -
17-j9 - -
17-librca - -
21 - -
21-graal - -
21-librca - -
25 - -
25-graal - -
25-librca - -

Legend: ✅ passed | ❌ failed | ⚪ skipped | 🚫 cancelled

Summary: Total: 32 | Passed: 32 | Failed: 0


Updated: 2026-07-09 17:19:03 UTC

@jbachorik
jbachorik marked this pull request as ready for review July 9, 2026 16:58
@jbachorik
jbachorik requested a review from a team as a code owner July 9, 2026 16:58
@jbachorik
jbachorik merged commit e22b8f9 into main Jul 9, 2026
115 of 127 checks passed
@jbachorik
jbachorik deleted the jb/check_me_out branch July 9, 2026 16:58
@github-actions github-actions Bot added this to the 1.47.0 milestone Jul 9, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: e7ae433755

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread AGENTS.md
Comment on lines +397 to +398
Files to touch when upgrading the build/target JDK, and the Gradle 9 breaking-API reference
table: see [doc/build/JdkUpgrades.md](doc/build/JdkUpgrades.md).

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Restore the missing JDK upgrade guide

This replacement points the removed JDK upgrade checklist to doc/build/JdkUpgrades.md, but that file is not present in doc/build and rg JdkUpgrades only finds this new link. As a result, the detailed build/target JDK upgrade instructions deleted from AGENTS.md are no longer reachable for anyone following the project guidance.

Useful? React with 👍 / 👎.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant