Skip to content

Download matching OpenFeature provider for Java development tests - #8184

Closed
hhan2024 wants to merge 1 commit into
mainfrom
fix/java-dev-openfeature
Closed

hhan2024 wants to merge 1 commit into
mainfrom
fix/java-dev-openfeature

Conversation

@hhan2024

@hhan2024 hhan2024 commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

Motivation

Java development CI downloads the agent from S3 but resolves dd-openfeature from Maven. After DataDog/dd-trace-java#12844 changed the gateway API, that combination fails provider initialization with NoClassDefFoundError: FeatureFlaggingGateway$ConfigListener, including in PR #7853's Java job.

CI evidence

Datadog job history across all branches, October 19–26 shows the passing and failing executions of System Tests (java, dev) / parametric / parametric (2).

DataDog/dd-trace-java#12844 merged on October 26 at 10:59:02 UTC−04:00 (14:59:02 UTC). PR #7853's failing job used 1.69.0-SNAPSHOT+de1eb9c722, the exact upstream merge commit. Independent PRs #8130 and #8179 also failed with the same missing FeatureFlaggingGateway$ConfigListener exception in their archived server logs:

The later history provides a direct comparison between this fix branch and other branches still using the incompatible artifacts. All times below are job completion times on October 26, UTC−04:00:

Time PR / job Result
12:59:31 #8182 — Java Lambda SCA Failed: FFE provider startup
13:13:09 #8184 — this fix Passed: 402 passed, 28 skipped, 134 xfailed, 4 xpassed
13:16:58 #8115 — Node SSR helpers Failed: FFE provider startup
13:17:59 #8183 — backend interface changes Failed: FFE provider startup

The passing execution belongs to this fix branch (fix/java-dev-openfeature, commit 9a6fa747e), confirming the Java job succeeds with the compatible agent/provider pair while other branches continue failing.

Changes

Download dd-openfeature.jar alongside the Java development agent from the same LIBRARY_TARGET_BRANCH (default: master). Existing Java build support picks up the custom jar automatically. A failed provider download stops setup instead of falling back to an incompatible released provider. Java Lambda keeps its existing agent-only download.

Validation

  • Downloaded both artifacts and built the Java parametric image; the build confirmed use of the custom agent and OpenFeature provider.
  • All 29 serial-ID metadata and dynamic-evaluation tests passed with Java 1.69.0-SNAPSHOT+c3ea2067b1.
  • ./format.sh --check passed, including shellcheck.
  • Downloader smoke checks passed for default/custom branches, provider download failure, and Java Lambda behavior.

The first integration run exhausted temporary disk space; after recovering space, the full focused rerun passed.

Load dd-openfeature.jar from the selected agent build branch so provider initialization does not mix released gateway APIs with the development tracer.

Environment: Datadog workspace

Co-Authored-By: Codex GPT-6 <noreply@localhost>
@github-actions

Copy link
Copy Markdown
Contributor

CODEOWNERS have been resolved as:

utils/scripts/load-binary.sh                                            @DataDog/system-tests-core

@hhan2024
hhan2024 marked this pull request as ready for review October 26, 2026 17:15
@hhan2024
hhan2024 requested a review from a team as a code owner October 26, 2026 17:15
@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review 🔄 Running since 2026-10-26T17:15:54.774701Z 9a6fa74 Draft marked ready
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@sameerank sameerank left a comment

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.

I need this fix too 😬

@datadog-prod-us1-3

datadog-prod-us1-3 Bot commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

Pipelines  Tests

🎉 All green!

🧪 All tests passed
❄️ No new flaky tests detected

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 9a6fa74 | Docs | View more details | Give us feedback!

@cbeauchesne

Copy link
Copy Markdown
Collaborator

#8188 has fixed it.

@hhan2024

Copy link
Copy Markdown
Contributor Author

closing bc already fixed

@hhan2024 hhan2024 closed this Oct 27, 2026
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.

4 participants