Repository navigation
Add agentless and standalone OpenFeature smoke tests - #12846
Conversation
This comment has been minimized.
This comment has been minimized.
🟢 Java Benchmark SLOs — All performance SLOs passed
PR vs. master results
Commit: Load and DaCapo benchmarks can be triggered manually in the GitLab pipeline. Results will appear in the Benchmarking Platform UI after completion. |
5e778ce to
ed48374
Compare
ed48374 to
68623d7
Compare
68623d7 to
a264ffd
Compare
a264ffd to
117a212
Compare
117a212 to
7289f2d
Compare
7289f2d to
4db3136
Compare
4db3136 to
caabcf2
Compare
caabcf2 to
8738bc6
Compare
239cf99 to
841c43f
Compare
dc45c3e to
0aef5b5
Compare
0aef5b5 to
8223463
Compare
442e327 to
17833c4
Compare
5f96f72 to
716f7a1
Compare
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 0cc336db05
ℹ️ 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".
There was a problem hiding this comment.
The standalone smoke suite fails before its application starts: its launcher omits the scope-diagnostics placeholder without supplying the required opt-out. Consequently, none of its evaluation scenarios can run.
🤖 Bits Code Review · Commit 0cc336d
There was a problem hiding this comment.
More details
Missing OpenTelemetry dependencies in the offline cache prevented targeted tests from reaching the OpenFeature scenarios, so runtime behavior remains unverified.
🤖 Bits Code Review · Commit 018613e
Extract the shared OpenFeature smoke test scenarios and run them against the SDK alone, the SDK with the agent in agentless mode, and the SDK with the agent through Remote Configuration. Document the new SDK layout in ARCHITECTURE.md. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Describe the SDK setup, its configuration, the features by setup, the migration from earlier versions and the design notes for contributors. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The agent bridge ships with dd-java-agent 1.69.0, and the providers released before 1.69.0 return the caller defaults with it. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
(cherry picked from commit 76d1a62)
There was a problem hiding this comment.
More details
Confidence is limited to static analysis of configuration, event delivery, and shutdown handling; the smoke tests were not executed.
🤖 Bits Code Review · Commit 2e078e9
What Does This Do
Extracts the shared OpenFeature smoke test scenarios into
AbstractOpenFeatureSmokeTestand runs them in three setups: SDK only (standalone), SDK + agent in agentless mode, and SDK + agent with Remote Configuration. UpdatesARCHITECTURE.md, and adds thedd-openfeatureSDK README: setup, configuration, features by setup, migration from earlier versions and design notes for contributors.Motivation
Covers each supported setup of the standalone SDK (APMLP-1512).
Stack
The full change is in #12709. It was split into this stack to ease the review.
Jira ticket: APMLP-1512
🤖 Generated with Claude Code