-
Notifications
You must be signed in to change notification settings - Fork 34
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!: use v1beta1 in operator logic #539
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #539 +/- ##
==========================================
+ Coverage 77.94% 81.86% +3.92%
==========================================
Files 10 7 -3
Lines 1020 805 -215
==========================================
- Hits 795 659 -136
+ Misses 189 118 -71
+ Partials 36 28 -8
Flags with carried forward coverage won't be shown. Click here to find out more. |
e331851
to
0dbf5b4
Compare
Great progress. Needs to fix some conflicts and address few comments. Once done we can go for the release. Proposal - we should make this a breaking change |
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.
11fa1e0
to
946a73c
Compare
be5eda6
to
c8c6bda
Compare
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
c8c6bda
to
c86833c
Compare
3ada018
to
11df26a
Compare
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
This PR
v1beta1
in operator logicFeatureFlag
controllercommon
packagecommon
packageconfig
folderRelated Issues
Fixes #531
Depends on
open-feature/flagd#997
Notes
Currently the OFO is using custom build images of flagD, this will be chaned after open-feature/flagd#997 will be merged and released