-
Notifications
You must be signed in to change notification settings - Fork 71
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!: start command flag refactor #222
feat!: start command flag refactor #222
Conversation
Signed-off-by: James Milligan <james@omnant.co.uk>
This makes sense to me. I think watching multiple k8s-resources is sort of a must have, and this seems like a clean way to do it. I'll test this out a bit, but I like the approach. |
Signed-off-by: James Milligan <james@omnant.co.uk>
Signed-off-by: James Milligan <james@omnant.co.uk>
Signed-off-by: James Milligan <james@omnant.co.uk>
Tested this locally as well and everything works as expected, including updates, merging and warnings about duplicate flags across sources.. The experience is definitely improved. I know some changes based on the OFO review are incoming so I'll wait to approve. |
Signed-off-by: James Milligan <james@omnant.co.uk>
Co-authored-by: Todd Baert <toddbaert@gmail.com> Signed-off-by: James Milligan <75740990+james-milligan@users.noreply.github.com>
Signed-off-by: James Milligan <james@omnant.co.uk>
Signed-off-by: James Milligan <james@omnant.co.uk>
Signed-off-by: James Milligan <james@omnant.co.uk>
Signed-off-by: James Milligan <james@omnant.co.uk>
Signed-off-by: James Milligan <james@omnant.co.uk>
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, nice
Edit: is this not a breaking change?
Signed-off-by: James Milligan <james@omnant.co.uk>
I agree with @skyerus that this is breaking, since it breaks the command line. I think you should add a EDIT: in fact I've done so, and approved. |
Signed-off-by: James Milligan <james@omnant.co.uk>
…ligan/flagd into sync-provider-refactor
🤖 I have created a release *beep* *boop* --- ## [0.2.7](v0.2.5...v0.2.7) (2022-12-02) ### ⚠ BREAKING CHANGES * start command flag refactor ([#222](#222)) ### Features * enable request logging via the --debug flag ([#226](#226)) ([11954b5](11954b5)) * Resurrected the STATIC flag reason. Documented the caching strategy. ([#224](#224)) ([5830592](5830592)) * snap ([#211](#211)) ([c619844](c619844)) * start command flag refactor ([#222](#222)) ([14474cc](14474cc)) ### Miscellaneous Chores * release v0.2.6 ([93cfb78](93cfb78)) * release v0.2.7 ([4a9f6df](4a9f6df)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Addresses changes introduced by flagd 0.2.7 described [here](open-feature/flagd#222). Signed-off-by: Michael Beemer <beeme1mr@users.noreply.github.com>
Signed-off-by: James Milligan james@omnant.co.uk
Corresponding OFO changes here
This PR
--sync-provider
--uri
flags can be passed indicating the use of different existingsync-provider
types, all of which will worksync-provider
;http(s)://
will be passed to the http sync,file://
will be passed to the file path sync and the Kubernetes sync uses the following patterncore.openfeature.dev/{namespace}/{name}
, this will also allow for the Kubernetes sync to watch multipleFeatureFlagConfigurations
from different namespaces.--sync-provider
flag is passed as an argument./flagd start --uri file://etc/flagd/end-to-end.json --uri core.openfeature.dev/test/end-to-end-2
Related Issues
open-feature/open-feature-operator#251
Notes
Follow-up Tasks
How to test