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

Updated dependencies #4

Merged
merged 1 commit into from
Apr 8, 2022
Merged

Updated dependencies #4

merged 1 commit into from
Apr 8, 2022

Conversation

tom-fleming
Copy link
Contributor

Updated dependencies to include reactphp/socket#258, which is a suspected cause of silent failures in our container.

@tom-fleming tom-fleming merged commit 87f54df into main Apr 8, 2022
@tom-fleming tom-fleming deleted the update-dependencies branch April 8, 2022 21:27
@clue
Copy link

clue commented Dec 17, 2023

@tom-fleming This looks like an interesting one. I'd love to learn more about the specific silent failures you mentioned. I filed a pull request earlier this week (reactphp/socket#311) that might be connected to the other ticket you mentioned (that's how I came across this PR). Please let me know if this could be helpful.

@tom-fleming
Copy link
Contributor Author

tom-fleming commented Mar 7, 2024

Hi @clue Thanks for your comment. We haven't experienced this issue for quite some time, and it looks like you've since resolved the other issue, but to provide some context, this project is used to stream logs to New Relic within an ECS container. What would happen is that we would occasionally (maybe once every 1-4 weeks) stop seeing logs show up in the container or in New Relic, and this behavior was not detected as an issue with our healthcheck at the time. Restarting the container, which would restart the application, would allow log streaming to continue for awhile. We were unable to identify any spikes in CPU, memory, or storage issues.

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

Successfully merging this pull request may close these issues.

2 participants