Skip to content

feat: add configurable cluster domain support#789

Merged
beeme1mr merged 1 commit into
open-feature:mainfrom
kuznero:main
Jan 5, 2026
Merged

feat: add configurable cluster domain support#789
beeme1mr merged 1 commit into
open-feature:mainfrom
kuznero:main

Conversation

@kuznero
Copy link
Copy Markdown
Contributor

@kuznero kuznero commented Nov 23, 2025

Add CLUSTER_DOMAIN environment variable to allow customization of the Kubernetes cluster domain used in flagd-proxy service URIs. The default value remains "cluster.local" to maintain backward compatibility.

This enables support for clusters using non-standard domain configurations.

This PR

  • adds this new feature

Related Issues

Fixes #764

Notes

Follow-up Tasks

How to test

Add CLUSTER_DOMAIN environment variable to allow customization of the Kubernetes cluster domain used in flagd-proxy service URIs. The default value remains "cluster.local" to maintain backward compatibility.

This enables support for clusters using non-standard domain configurations.

Signed-off-by: Roman Kuznetsov <roku@lix.one>
@kuznero kuznero marked this pull request as ready for review November 23, 2025 15:25
Copy link
Copy Markdown
Member

@guidobrei guidobrei left a comment

Choose a reason for hiding this comment

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

Thanks for adding this contribution @kuznero, LGTM.

@toddbaert are there any maintainers assigned to the operator? There are not reviewers assigned to the PR.

@beeme1mr beeme1mr requested a review from toddbaert December 15, 2025 16:21
@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 15, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.33%. Comparing base (499661e) to head (7ff3551).
⚠️ Report is 74 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #789       +/-   ##
===========================================
- Coverage   86.51%   72.33%   -14.19%     
===========================================
  Files          19       30       +11     
  Lines        1587     1923      +336     
===========================================
+ Hits         1373     1391       +18     
- Misses        173      484      +311     
- Partials       41       48        +7     
Files with missing lines Coverage Δ
internal/common/flagdinjector/flagdinjector.go 88.74% <100.00%> (ø)
internal/common/flagdproxy/flagdproxy.go 90.23% <100.00%> (ø)

... and 25 files with indirect coverage changes

Flag Coverage Δ
unit-tests 72.33% <100.00%> (∅)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@kuznero
Copy link
Copy Markdown
Contributor Author

kuznero commented Dec 29, 2025

Any chance this can be merged/finalized?

@beeme1mr beeme1mr merged commit 5e91071 into open-feature:main Jan 5, 2026
18 of 19 checks passed
@beeme1mr
Copy link
Copy Markdown
Member

beeme1mr commented Jan 5, 2026

Any chance this can be merged/finalized?

Sorry for the delay @kuznero. Thanks for the PR.

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.

Flagd-proxy mode assumes cluster.local

3 participants