-
Notifications
You must be signed in to change notification settings - Fork 50
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: parse new featureflagconfiguration CRD structure #205
feat: parse new featureflagconfiguration CRD structure #205
Conversation
Converted to draft to ensure it's merged at the same time as the OFO component. Ready for review |
The tests/linting fails because the PR needs updating to include the version of OFO that includes the CRD spec change. |
Signed-off-by: Skye Gill <gill.skye95@gmail.com>
Signed-off-by: Skye Gill <gill.skye95@gmail.com>
362c65b
to
84bcfcb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM pending the OFO merge, obviously.
@@ -40,28 +40,32 @@ func (k *Sync) Source() string { | |||
} | |||
|
|||
func (k *Sync) Fetch(ctx context.Context) (string, error) { | |||
state := "{}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👨🍳 💋 🤌
Not needed |
This PR
Parses the featureflagconfiguration CRD structure changes (more detail here).
Related Issues
Fixes #204
Notes
To be release in conjunction with open-feature/open-feature-operator#207
Follow-up Tasks
How to test