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

chore: add engine account_info logging #5347

Merged
merged 1 commit into from
Oct 23, 2024
Merged

Conversation

albert-llimos
Copy link
Contributor

@albert-llimos albert-llimos commented Oct 23, 2024

Checklist

Please conduct a thorough self-review before opening the PR.

  • I am confident that the code works.
  • I have written sufficient tests.
  • I have written and tested required migrations.
  • I have updated documentation where appropriate.

Summary

Adding data to error logging in the Engine for account data decoding.

Copy link

codecov bot commented Oct 23, 2024

Codecov Report

Attention: Patch coverage is 0% with 5 lines in your changes missing coverage. Please review.

Project coverage is 71%. Comparing base (6311f8c) to head (36ec7ca).
Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
engine/src/witness/sol/sol_deposits.rs 0% 4 Missing ⚠️
engine/src/witness/sol/nonce_witnessing.rs 0% 1 Missing ⚠️
Additional details and impacted files
@@          Coverage Diff           @@
##            main   #5347    +/-   ##
======================================
- Coverage     71%     71%    -0%     
======================================
  Files        492     492            
  Lines      85750   85617   -133     
  Branches   85750   85617   -133     
======================================
- Hits       61109   60908   -201     
- Misses     21909   21966    +57     
- Partials    2732    2743    +11     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@albert-llimos albert-llimos marked this pull request as ready for review October 23, 2024 07:45
@albert-llimos albert-llimos requested a review from kylezs as a code owner October 23, 2024 07:45
@kylezs kylezs added this pull request to the merge queue Oct 23, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 23, 2024
@kylezs kylezs added this pull request to the merge queue Oct 23, 2024
@kylezs
Copy link
Contributor

kylezs commented Oct 23, 2024

Hit a tracked flaky case, rerunning

Merged via the queue into main with commit 2902103 Oct 23, 2024
48 of 49 checks passed
@kylezs kylezs deleted the chore/engine-solana-info-logging branch October 23, 2024 09:49
kylezs added a commit that referenced this pull request Oct 23, 2024
kylezs added a commit that referenced this pull request Oct 23, 2024
pick: chore: add engine account_info logging #5347
syan095 added a commit that referenced this pull request Oct 23, 2024
* origin/main:
  feat: Submit a slot number alongside nonce (#5297)
  chore: use node version from `.nvmrc` 📌 (#5336)
  chore: add engine account_info logging (#5347)
  chore: replace manual scale encoding for ts-scale (#5335)
  chore: more consistent params in Broker API (#5342)

# Conflicts:
#	engine/src/witness/sol.rs
#	state-chain/pallets/cf-elections/Cargo.toml
#	state-chain/runtime/Cargo.toml
syan095 added a commit that referenced this pull request Oct 29, 2024
…waps-close-accounts

* origin/main: (44 commits)
  fix: expire all previous epochs (#5279)
  feat: add/update contract swaps parameters (#5343)
  chore: add address to solana logging (#5353)
  fix: ignore dust underflows in order fills rpc (#5352)
  chore: consistent naming prewitnessed (#5351)
  feat: engine-runner verifies gpg signature of old dylib when downloaded (#5339)
  feat: tainted transaction reporting (#5310)
  bug: change_utxo not always present (#5340)
  feat: structured error return types for rpcs (#5346)
  chore: unify dependencies to root cargo.toml (#5333)
  feat: Submit a slot number alongside nonce (#5297)
  chore: use node version from `.nvmrc` 📌 (#5336)
  chore: add engine account_info logging (#5347)
  chore: replace manual scale encoding for ts-scale (#5335)
  chore: more consistent params in Broker API (#5342)
  feat: broker can encode btc smart contract call (#5329)
  chore: localnet recreate script can use defaults (#5338)
  feat: witnessing btc smart contract swaps (#5331)
  feat: Solana CCM fallback (#5316)
  fix: scale types for pending ceremonies (#5286)
  ...

# Conflicts:
#	Cargo.lock
#	state-chain/chains/src/sol/api.rs
#	state-chain/pallets/cf-broadcast/src/migrations.rs
#	state-chain/pallets/cf-environment/Cargo.toml
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