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

Add retry mechanism for Opensearch Interactions #499

Draft
wants to merge 19 commits into
base: master
Choose a base branch
from

Conversation

meghana-rajashekar
Copy link
Collaborator

  1. Added retry mechanism while Initalizing Opensearch client which happens in the init() of crypto controller and whenever a config update changes the credentials
  2. Retry mechanism to retry any Opensearch API calls which executes the API and return the rsponse and an error
  3. Made necessary changes across the code base to accept this retry
  4. Update log print on utilities/utilities.go
  5. Update method descriptions

1. Added retry mechanism while Initalizing Opensearch client which happens in the init() of crypto controller and whenever a config update changes the credentials
2. Retry mechanism to retry any Opensearch API calls which executes the API and return the rsponse and an error
3. Made necessary changes across the code base to accept this retry
4. Update log print on utilities/utilities.go
1. Added retry mechanism while Initalizing Opensearch client which happens in the init() of crypto controller and whenever a config update changes the credentials
2. Retry mechanism to retry any Opensearch API calls which executes the API and return the rsponse and an error
3. Made necessary changes across the code base to accept this retry
4. Update log print on utilities/utilities.go
1. Added retry mechanism while Initalizing Opensearch client which happens in the init() of crypto controller and whenever a config update changes the credentials
2. Retry mechanism to retry any Opensearch API calls which executes the API and return the rsponse and an error
3. Made necessary changes across the code base to accept this retry
4. Update log print on utilities/utilities.go
1. Broad cast of updated config.yaml would fail if the current credentials in the config.yaml isn't correct
Therefore separated Broadcast function from Update Encrypted Credentials.
Now Broadcast will happen only when the correct credentials are updated in config.yaml
@meghana-rajashekar meghana-rajashekar marked this pull request as draft March 28, 2023 08:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request Provision
Projects
None yet
Development

Successfully merging this pull request may close these issues.

FetchMetrics failure on restart of opensearch [ISSUE]
1 participant