Skip to content
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

[flagd] Update the flagd provider to conform to the latest spec #1053

Open
2 tasks
toddbaert opened this issue Oct 31, 2024 · 3 comments
Open
2 tasks

[flagd] Update the flagd provider to conform to the latest spec #1053

toddbaert opened this issue Oct 31, 2024 · 3 comments

Comments

@toddbaert
Copy link
Member

Update the flagd provider to conform to the latest spec: https://flagd.dev/reference/specifications/providers/

For the Java provider, that means:

  • implement retryGraceAttempts param:
    • defaults to 5
    • when provider disconnects, if reconnect attempts <= retryGraceAttempts, provider emits STALE
    • when reconnect attempts > retryGraceAttempts provider is emits ERROR
  • review and align all configuration params as specified in configuration: https://flagd.dev/reference/specifications/providers/#configuration
@toddbaert toddbaert changed the title [flagd] Update the flagd provider to conform to the latest spec: [flagd] Update the flagd provider to conform to the latest spec Oct 31, 2024
@beeme1mr beeme1mr transferred this issue from open-feature/java-sdk Oct 31, 2024
@aepfli
Copy link
Member

aepfli commented Nov 7, 2024

should we maybe split this into smaller tasks based on the todos - everything at once might be overwhelming, but all the todos as little chunks seems to me like good first issues ;)

@guidobrei
Copy link
Member

We should also define the logging behavior during the grace period.
Info logs without exception details during that period should be sufficient to not be too noisy.

@aepfli
Copy link
Member

aepfli commented Dec 2, 2024

Does this supersed #395?

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

No branches or pull requests

3 participants