Skip to content

Gregory/eng 784 choreassertion executor assertion executor tracing and#11

Merged
makemake-kbo merged 13 commits intomainfrom
gregory/eng-784-choreassertion-executor-assertion-executor-tracing-and
Jun 25, 2025
Merged

Gregory/eng 784 choreassertion executor assertion executor tracing and#11
makemake-kbo merged 13 commits intomainfrom
gregory/eng-784-choreassertion-executor-assertion-executor-tracing-and

Conversation

@0xgregthedev
Copy link
Copy Markdown
Contributor

No description provided.

@linear
Copy link
Copy Markdown

linear Bot commented Jun 23, 2025

@0xgregthedev 0xgregthedev force-pushed the gregory/eng-784-choreassertion-executor-assertion-executor-tracing-and branch 2 times, most recently from e53e609 to ccbe471 Compare June 24, 2025 16:22
chore: tracing improvements

chore: improve traces in assertion store

chore: tracing improvements

chore: tracing improvements

fix: syntax

fix: comp issue

chore: cleanup tracing

chore: improve tracing data

chore: improve store trace readability

chore: improve tracing

chore: fix tracing

chore: try to fix span

chore: refine state change metadata

chore: use in_scope

chore: improve tracing readability

chore: improve tracing

chore: fix comp issue

chore: add more spans

fix: instrument

chore: abstract execute_assertion_fn

chore: try to fix span

chore: wth

chore: change trace fmt
@0xgregthedev 0xgregthedev force-pushed the gregory/eng-784-choreassertion-executor-assertion-executor-tracing-and branch from 179ceda to 56893c0 Compare June 24, 2025 19:28
@0xgregthedev 0xgregthedev marked this pull request as ready for review June 24, 2025 19:59
Comment thread src/executor/mod.rs
Comment thread src/executor/mod.rs
Comment thread src/executor/mod.rs
Comment thread src/executor/mod.rs Outdated
Comment thread src/executor/mod.rs
Comment thread src/store/assertion_store.rs Outdated
Comment thread src/store/assertion_store.rs
Comment thread src/store/indexer.rs Outdated
Comment thread src/store/indexer.rs Outdated
@fredo
Copy link
Copy Markdown
Contributor

fredo commented Jun 24, 2025

Looks very good! I just had a quick look at extract_assertion_contract . Could you add a couple of logs there in case the contract deployment failed? I remember we had some cases where the root cause was a failed contract deployment which stayed undiscovered for quite some time.

@0xgregthedev
Copy link
Copy Markdown
Contributor Author

Looks very good! I just had a quick look at extract_assertion_contract . Could you add a couple of logs there in case the contract deployment failed? I remember we had some cases where the root cause was a failed contract deployment which stayed undiscovered for quite some time.

f150fcf

Comment thread src/executor/mod.rs
@makemake-kbo makemake-kbo merged commit edd92ec into main Jun 25, 2025
7 checks passed
@fredo fredo deleted the gregory/eng-784-choreassertion-executor-assertion-executor-tracing-and branch June 25, 2025 09:10
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.

3 participants