feat: add latest flagd config fields, bump deps#818
Merged
Conversation
9527eb0 to
509dc40
Compare
This PR adds configuration added to flagd since the last upgrade: - context-value (--context-value / -X): static key-value pairs for evaluation context - header-to-context mappings (--context-from-header / -H): map HTTP headers to context keys - CORS origins (--cors-origin / -C): allowed origins for the OFREP endpoint - OFREP port (--ofrep-port / -r): configurable port for the OFREP service - Bump flagd from v0.12.4 to v0.15.2 - Bump flagd-proxy from v0.7.4 to v0.9.4 No e2e tests were added since none of these impact RBAC, topology, or deployment; just added unit tests that assert the associated startup args are correctly passed. Signed-off-by: Todd Baert <todd.baert@dynatrace.com>
509dc40 to
ca5dc33
Compare
There was a problem hiding this comment.
Code Review
This pull request enhances the FeatureFlagSource CRD by adding support for static context values, header-to-context mappings, CORS origin configuration, and a custom OFREP port. These new fields are integrated into the flagd sidecar injection logic and standalone deployment controller. Furthermore, the PR updates the default images for flagd to v0.15.2 and flagd-proxy to v0.9.4 across the codebase, documentation, and Helm charts. Feedback was provided regarding the manual modification of generated code in zz_generated.deepcopy.go, which should instead be handled by the project's generation tooling.
This was referenced Apr 10, 2026
6d53e09 to
f3fa361
Compare
Signed-off-by: Todd Baert <todd.baert@dynatrace.com>
f3fa361 to
dd8d6e0
Compare
toddbaert
commented
Apr 10, 2026
beeme1mr
approved these changes
Apr 13, 2026
lukas-reining
approved these changes
Apr 13, 2026
Member
lukas-reining
left a comment
There was a problem hiding this comment.
Looks good to me, I can find no issue!
This was referenced Apr 10, 2026
Merged
Merged
Y0sh1dk
pushed a commit
to Y0sh1dk/open-feature-operator
that referenced
this pull request
Apr 22, 2026
Signed-off-by: Todd Baert <todd.baert@dynatrace.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR adds configuration added to flagd since the last upgrade:
No e2e tests were added since none of these impact RBAC, topology, or deployment; just added unit tests that assert the associated startup args are correctly passed.