feat(shipping): delay international shipments via shippingSlowdown feature flag by tlee768 · Pull Request #3354 · open-telemetry/opentelemetry-demo

tlee768 · 2026-05-07T16:21:02Z

Summary

Add shippingSlowdown feature flag (default off) that introduces a 10-second delay for non-US shipping requests when enabled
Integrate with flagd via OFREP protocol, with OpenTelemetry tracing on flag evaluations
Add FLAGD_HOST and FLAGD_OFREP_PORT env vars and flagd dependency to docker-compose
Add comprehensive tests with injectable flag checker and configurable slowdown duration

Closes #346

Test plan

cargo test passes in src/shipping
docker compose up starts shipping service with flagd dependency
Verify no delay for US addresses regardless of flag state
Verify 10s delay for non-US addresses when shippingSlowdown flag is enabled
Verify no delay for non-US addresses when flag is disabled

Assisted-by: This contribution was prepared with the assistance of an AI development tool.

linux-foundation-easycla · 2026-05-09T20:31:33Z

The committers listed above are authorized under a signed CLA.

✅ login: tlee768 / name: tlee439 (024b8a2, 28192f5, 2cab5e2, 2cad430, 2d5aad3, 2e3539d, 30aeeac, 3108414, 3501c1c, 361df7b, 3de2ef0, 5476d2a, 672c4d5, 6e3b78a, 76fa075, 7b6b22e, 82945d7, 9297a87, 9450555, 9afc649, ac5de46, b5ff292, cabb716, cae3f18, d666969, d93c9b2)
✅ login: tlee768 / name: tlee768 (9b9884d)

ps48 · 2026-05-12T05:03:06Z

@tlee768 thanks for the contribution, can you please fix the markdownlint errors:

Command to run

  # Markdown lint (requires npm install first)
  make markdownlint

  # Or directly:
  npm install
  npx --no -p markdownlint-cli markdownlint -c .markdownlint.yaml <file>

Errors from the CI

 ./src/shipping/README.md:35:16 error MD060/table-column-style Table column style [Table pipe does not align with header for style "aligned"]
./src/shipping/README.md:35:26 error MD060/table-column-style Table column style [Table pipe does not align with header for style "aligned"]
./src/shipping/README.md:35:58 error MD060/table-column-style Table column style [Table pipe does not align with header for style "aligned"]
./src/shipping/README.md:36:31 error MD060/table-column-style Table column style [Table pipe does not align with header for style "aligned"]
./src/shipping/README.md:36:62 error MD060/table-column-style Table column style [Table pipe does not align with header for style "aligned"]

Copilot

Pull request overview

Adds a new shippingSlowdown feature flag (default off) to the Rust Shipping service to introduce an artificial delay for international (non‑US) /ship-order requests when enabled, using flagd’s OFREP REST API and emitting tracing around evaluations.

Changes:

Introduce a flagd OFREP client and a FlagChecker abstraction to evaluate shippingSlowdown (with a test mock path).
Apply a configurable slowdown delay in ship_order for non‑US addresses when the flag is enabled, and add request-path tests.
Wire flagd dependency + env vars into docker compose files and add the flag definition to demo.flagd.json.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
src/shipping/src/shipping_service/feature_flag.rs	Adds a flagd OFREP HTTP client with OTel-instrumented requests and safe fallbacks.
src/shipping/src/shipping_service.rs	Adds `FlagChecker`, injects it + a slowdown duration into `ship_order`, and adds tests for US/international behavior.
src/shipping/src/main.rs	Registers `FlagChecker` and default slowdown duration into Actix app state.
src/shipping/README.md	Documents the new feature flag and related environment variables.
src/flagd/demo.flagd.json	Adds the `shippingSlowdown` flag definition (default variant `off`).
docker-compose.yml	Passes flagd OFREP env vars to shipping and adds `flagd` to `depends_on`.
docker-compose.minimal.yml	Same as full compose: env var pass-through and `flagd` dependency for shipping.
CHANGELOG.md	Notes the new shipping feature flag integration.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

puckpuck

Hey @tlee768 thanks for this, and I really want this to happen as a feature flag for the demo. There is an official flagd provider for rust that we should use here instead of a custom client and enum to manage the feature flag evaluation.

puckpuck · 2026-05-19T17:00:38Z

Instead of a shippingSlowdown boolean flag, I think a intlShippingSlowdown integer flag with a default value of 0 is a better idea, will keep the code cleaner, and allow for more flexibility with the flag.

Doing this allows us to remove all DEFAULT_SLOWDOWN and the slowdown function parameter, since this is pulled from the featureflag itself now. If the flag doesn't exists we default it to 0 (no sleep).

tlee768 · 2026-05-19T21:29:07Z

Instead of a shippingSlowdown boolean flag, I think a intlShippingSlowdown integer flag with a default value of 0 is a better idea, will keep the code cleaner, and allow for more flexibility with the flag.

Doing this allows us to remove all DEFAULT_SLOWDOWN and the slowdown function parameter, since this is pulled from the featureflag itself now. If the flag doesn't exists we default it to 0 (no sleep).

Done

tlee768 · 2026-05-21T17:08:15Z

@puckpuck LMK of any further issues

…wn feature flag Closes open-telemetry#346 Assisted-by: This contribution was prepared with the assistance of an AI development tool.

Assisted-by: This contribution was prepared with the assistance of an AI development tool.

… slowdown duration Assisted-by: This contribution was prepared with the assistance of an AI development tool.

…ependency to compose Assisted-by: This contribution was prepared with the assistance of an AI development tool.

Assisted-by: This contribution was prepared with the assistance of an AI development tool.

…w OTel API Assisted-by: This contribution was prepared with the assistance of an AI development tool.

Assisted-by: This contribution was prepared with the assistance of an AI development tool.

awc::Client is !Send + !Sync (uses Rc internally), so it cannot be stored in a struct captured by the Send + Sync FlagChecker closure. Instead, store one Client per actix worker thread using thread_local!, initialized lazily on first use. Cloning the Client is cheap (bumps an Rc, not the pool). Assisted-by: This contribution was prepared with the assistance of an AI development tool.

The #[instrument] span was never bridged to OTel since tracing-opentelemetry is not wired up, so the span and its fields were silently dropped. Assisted-by: This contribution was prepared with the assistance of an AI development tool.

Assisted-by: This contribution was prepared with the assistance of an AI development tool.

Replace closure-based FlagChecker type alias with an enum that supports generic evaluate::<T>() calls, enabling non-bool flags in the future. Assisted-by: This contribution was prepared with the assistance of an AI development tool.

…r methods Assisted-by: This contribution was prepared with the assistance of an AI development tool.

…gd crate Use FlagdProvider from open-feature-flagd instead of a hand-rolled HTTP client, removing the FlagdClient wrapper and FlagChecker enum. The ship_order handler now takes dyn FeatureProvider directly, and tests use MockFeatureProvider from the open-feature crate. Assisted-by: This contribution was prepared with the assistance of an AI development tool.

Use the crate's default gRPC resolver (port 8013) instead of REST/OFREP (port 8016). Update docker-compose files to pass FLAGD_PORT instead of FLAGD_OFREP_PORT to the shipping service. Assisted-by: This contribution was prepared with the assistance of an AI development tool.

… up immediately The default RPC resolver wraps evaluations in an LRU cache with a 60s TTL, causing demo.flagd.json edits to be invisible until the cache expires. Setting cache_settings: None bypasses the cache and makes every evaluation a fresh gRPC unary call, matching the behaviour of the old OFREP client. Assisted-by: This contribution was prepared with the assistance of an AI development tool.

Narrow open-feature-flagd to the rpc feature only (drops reqwest, datalogic-rs, notify, semver, etc.). Remove manual FLAGD_HOST/PORT reads since FlagdOptions::default() already reads them from env. Assisted-by: This contribution was prepared with the assistance of an AI development tool.

Short-circuit the && so the gRPC call to flagd only happens when the address is outside the US. Assisted-by: This contribution was prepared with the assistance of an AI development tool.

…lShippingSlowdown flag The flag value itself now carries the delay in seconds (default 0), removing the need for DEFAULT_SLOWDOWN and the injected duration parameter. Assisted-by: This contribution was prepared with the assistance of an AI development tool.

puckpuck

Thank you very much for this @tlee768

tlee768 requested a review from a team as a code owner May 7, 2026 16:21

github-actions Bot added docs-update-required Requires documentation update helm-update-required Requires an update to the Helm chart when released labels May 7, 2026

tlee768 marked this pull request as draft May 7, 2026 16:23

tlee768 force-pushed the feat/shipping-slowdown-flag branch 3 times, most recently from 9feecdc to 82945d7 Compare May 12, 2026 03:19

tlee768 marked this pull request as ready for review May 12, 2026 03:22

osullivandonal requested a review from Copilot May 13, 2026 13:19

Copilot started reviewing on behalf of osullivandonal May 13, 2026 13:21 View session

Copilot AI reviewed May 13, 2026

View reviewed changes

Comment thread src/shipping/src/shipping_service.rs Outdated

Comment thread src/shipping/src/shipping_service.rs Outdated

puckpuck requested changes May 14, 2026

View reviewed changes

Comment thread src/shipping/src/shipping_service/feature_flag.rs Outdated

tlee768 force-pushed the feat/shipping-slowdown-flag branch from 44e8cde to 3501c1c Compare May 15, 2026 03:36

tlee768 requested a review from puckpuck May 15, 2026 15:13

tlee768 force-pushed the feat/shipping-slowdown-flag branch from fb321e7 to 7b6b22e Compare May 18, 2026 19:30

tlee768 mentioned this pull request May 18, 2026

ShippingService should check FeatureFlagService if shipping outside the US takes a while #346

Closed

tlee768 added 9 commits May 22, 2026 21:14

feat(shipping): delay international shipments based on shippingSlowdo…

608094c

…wn feature flag Closes open-telemetry#346 Assisted-by: This contribution was prepared with the assistance of an AI development tool.

feat(shipping): add OTel tracing to feature flag evaluation

8e59cc8

Assisted-by: This contribution was prepared with the assistance of an AI development tool.

test(shipping): add ship_order tests with injectable flag checker and…

4161526

… slowdown duration Assisted-by: This contribution was prepared with the assistance of an AI development tool.

test(shipping): add ship_order tests with injectable flag checker and…

3f72c60

… slowdown duration Assisted-by: This contribution was prepared with the assistance of an AI development tool.

feat(shipping): add FLAGD_HOST, FLAGD_OFREP_PORT env vars and flagd d…

7783f6e

…ependency to compose Assisted-by: This contribution was prepared with the assistance of an AI development tool.

docs: add shippingSlowdown feature flag entry to CHANGELOG

ec43e5d

Assisted-by: This contribution was prepared with the assistance of an AI development tool.

fix(shipping): check HTTP status before parsing feature flag response

f5be5e6

Assisted-by: This contribution was prepared with the assistance of an AI development tool.

docs(shipping): document shippingSlowdown feature flag and env vars

663880c

Assisted-by: This contribution was prepared with the assistance of an AI development tool.

fix(shipping): match all common US country name variants

438e2c8

Assisted-by: This contribution was prepared with the assistance of an AI development tool.

tlee768 added 17 commits May 22, 2026 21:14

test(shipping): remove redundant US country variant tests

a14bcd9

Assisted-by: This contribution was prepared with the assistance of an AI development tool.

fix(shipping): use tracing crate for feature flag spans instead of ra…

e95ba04

…w OTel API Assisted-by: This contribution was prepared with the assistance of an AI development tool.

fix(shipping): reuse HTTP client for feature flag requests via LazyLock

894db5d

Assisted-by: This contribution was prepared with the assistance of an AI development tool.

refactor(shipping): restructure feature flag as FlagdClient struct

4d64306

Assisted-by: This contribution was prepared with the assistance of an AI development tool.

refactor(shipping): restructure feature flag as FlagdClient struct

4d89f03

Assisted-by: This contribution was prepared with the assistance of an AI development tool.

feat(shipping): log feature flag evaluation result

e06c5d0

Assisted-by: This contribution was prepared with the assistance of an AI development tool.

refactor(shipping): move mock constructors into FlagChecker as builde…

e2dc588

…r methods Assisted-by: This contribution was prepared with the assistance of an AI development tool.

fix linting

1099d95

perf(shipping): skip flag evaluation for US addresses

badd571

Short-circuit the && so the gRPC call to flagd only happens when the address is outside the US. Assisted-by: This contribution was prepared with the assistance of an AI development tool.

puckpuck force-pushed the feat/shipping-slowdown-flag branch from 2efad5f to 03a8b0b Compare May 23, 2026 01:19

puckpuck approved these changes May 23, 2026

View reviewed changes

puckpuck enabled auto-merge May 23, 2026 01:32

puckpuck added this pull request to the merge queue May 23, 2026

Merged via the queue into open-telemetry:main with commit 4ec9379 May 23, 2026
39 checks passed

tlee768 deleted the feat/shipping-slowdown-flag branch May 23, 2026 13:42

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(shipping): delay international shipments via shippingSlowdown feature flag#3354

feat(shipping): delay international shipments via shippingSlowdown feature flag#3354
puckpuck merged 26 commits into
open-telemetry:mainfrom
tlee768:feat/shipping-slowdown-flag

tlee768 commented May 7, 2026 •

edited

Loading

Uh oh!

linux-foundation-easycla Bot commented May 9, 2026 •

edited

Loading

Uh oh!

ps48 commented May 12, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

puckpuck left a comment

Uh oh!

Uh oh!

puckpuck commented May 19, 2026

Uh oh!

tlee768 commented May 19, 2026

Uh oh!

tlee768 commented May 21, 2026

Uh oh!

puckpuck left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

tlee768 commented May 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Test plan

Uh oh!

linux-foundation-easycla Bot commented May 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

ps48 commented May 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

puckpuck left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

puckpuck commented May 19, 2026

Uh oh!

tlee768 commented May 19, 2026

Uh oh!

tlee768 commented May 21, 2026

Uh oh!

puckpuck left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

tlee768 commented May 7, 2026 •

edited

Loading

linux-foundation-easycla Bot commented May 9, 2026 •

edited

Loading

ps48 commented May 12, 2026 •

edited

Loading