Skip to content

feat(feature-flags): generalize driver-side cache - #496

Draft
cathleeny wants to merge 3 commits into
mainfrom
cathleeny/general-driver-flags
Draft

cathleeny wants to merge 3 commits into
mainfrom
cathleeny/general-driver-flags

Conversation

@cathleeny

@cathleeny cathleeny commented Oct 6, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Move and generalize the existing telemetry feature-flag cache into internal/featureflags; retain all returned values and add six typed getters.
  • Share values by workspace ID, with normalized host as a fallback, and use authenticated transport supplied by each caller.
  • Honor the connector-service TTL, falling back to 15 minutes when it is missing or invalid.

Validation

  • Full unit suite: go test ./... passed.
  • Feature-flag and telemetry tests passed with the race detector.
  • Cold-load/workspace and stale-reader concurrency tests passed for 30 repeated race-detector runs.
  • The new stale-reader regression test failed before the fix and passed afterward.
  • go vet ./... and golangci-lint passed (0 issues).

Signed-off-by: Cathleen Yan <58714163+cathleeny@users.noreply.github.com>
Signed-off-by: Cathleen Yan <58714163+cathleeny@users.noreply.github.com>
Signed-off-by: Cathleen Yan <58714163+cathleeny@users.noreply.github.com>

This branch has not been deployed

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant