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

Fix triple-store socket hangup issue #3718

Open
wants to merge 1 commit into
base: v8/develop
Choose a base branch
from

Conversation

zsculac
Copy link
Contributor

@zsculac zsculac commented Feb 11, 2025

Description

Fix triple store socket hangup issue by replacing communica with axios

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Implemented this logic on testnet nodes 1 and 2 and monitored loggly for "Failed to store Knowledge Collection with the UAL" logs. Since the logic was implemented (10.2. ~17h CET), there has been no occurrence of this error in loggly for respective nodes. Insert logs are present.
  • Implemented this logic on testnet node 1 (11.2. ~9h CET) which previously had failed storage attempts roughly every 1h. Since the logic was implemented, there has been no occurrence of this error in loggly for this node. Insert logs are present.

@zsculac zsculac self-assigned this Feb 11, 2025
@zsculac zsculac added the bug Something isn't working label Feb 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants