Skip to content

[8.17] [Feature Flags] Retry provider setup (#214200)#214299

Merged
afharo merged 1 commit intoelastic:8.17from
afharo:backport/8.17/pr-214200
Mar 13, 2025
Merged

[8.17] [Feature Flags] Retry provider setup (#214200)#214299
afharo merged 1 commit intoelastic:8.17from
afharo:backport/8.17/pr-214200

Conversation

@afharo
Copy link
Member

@afharo afharo commented Mar 12, 2025

Backport

This will backport the following commits from main to 8.17:

Questions ?

Please refer to the Backport tool documentation

## Summary

We identified that on some occasions, the Feature Flags provider times
out when setting up, and, since we don't restart the Kibana server, it
never sets it up.

This PR adds a retry logic to try to set the provider in case there's an
error.

cc @pmuellr as he found out about this bug

### Checklist

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 1337c11)

# Conflicts:
#	packages/core/feature-flags/core-feature-flags-server-internal/src/feature_flags_service.ts
#	packages/core/feature-flags/core-feature-flags-server-internal/src/set_provider_with_retries.test.ts
#	packages/core/feature-flags/core-feature-flags-server-internal/src/set_provider_with_retries.ts
@afharo afharo requested a review from kibanamachine as a code owner March 12, 2025 22:42
@afharo afharo added the backport This PR is a backport of another PR label Mar 12, 2025
@afharo afharo enabled auto-merge (squash) March 12, 2025 22:42
@afharo afharo merged commit beb727d into elastic:8.17 Mar 13, 2025
11 checks passed
@afharo afharo deleted the backport/8.17/pr-214200 branch March 13, 2025 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants