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

debug(rpc): witness and build logging #2974

Closed
wants to merge 2 commits into from
Closed

Conversation

conorsch
Copy link
Contributor

@conorsch conorsch commented Sep 6, 2023

Adding some debug logging statements to the witness and build logic, to aid in tracing a failure encountered during interchaintest integration. The WitnessAndBuild RPC never returns: pclientd balloons in memory consumption until oomkilled. Let's isolate where in the witness-and-build logic it's getting stuck.

@conorsch conorsch force-pushed the debug-witness-and-build branch from 1ac2967 to aa0b87b Compare September 6, 2023 15:38
@conorsch conorsch temporarily deployed to smoke-test September 6, 2023 15:38 — with GitHub Actions Inactive
Adding some debug logging statements to the witness and build logic,
to aid in tracing a failure encountered during interchaintest
integration. The WitnessAndBuild RPC never returns: pclientd balloons in
memory consumption until oomkilled. Let's isolate where in the
witness-and-build logic it's getting stuck.

assemble txp in stages
We need a custom dockerfile to support feature branches in
interchaintest, due to strangelove-specific tooling.
@conorsch conorsch force-pushed the debug-witness-and-build branch from aa0b87b to 80c4c35 Compare September 6, 2023 22:19
@conorsch conorsch temporarily deployed to smoke-test September 6, 2023 22:19 — with GitHub Actions Inactive
@conorsch
Copy link
Contributor Author

conorsch commented Sep 8, 2023

This was in pursuit of #2977. The real fix was #2980, so closing this PR without merging.

@conorsch conorsch closed this Sep 8, 2023
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.

1 participant