8000 Fog Ledger Enclave Health Check by dolanbernard · Pull Request #3530 · mobilecoinfoundation/mobilecoin · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fog Ledger Enclave Health Check #3530

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

Closed
wants to merge 2 commits into from

Conversation

dolanbernard
Copy link
Contributor
  • Use regular get_identity calls to monitor health of Fog Ledger enclave

Motivation

There is an issue that is likely in the enclave code that is causing the enclave to crash in rare cases. This should be fixed, but when this crash happens, the enclave never recovers. This PR adds a simple check to ensure the enclave is running and makes sure it gets restarted if it isn't. While the root cause of the issue is also important, it is very rare and difficult to re-create. The fact that there is no recovery from this is also a problem. This PR fixes the fact that there is no recovery from the enclave either crashing completely or not being in a healthy state.

Future Work

Monitor network logs for this error. Figure out what the cause of this particular crash is and fix it.

@joekottke
Copy link
Contributor

@dolanbernard Shouldn't this be a PR against the release/v5.0 branch? I think this is causing an issue with the container, as what is in master is DCAP focused, I think? And therefor the consensus nodes are coming up with this error: error while loading shared libraries: libsgx_dcap_ql.so.1: cannot open shared object file: No such file or directory

@dolanbernard dolanbernard deleted the feature/ledger-enclave-health-check branch November 30, 2023 20:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3ED0
Development

Successfully merging this pull request may close these issues.

2 participants
0