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

Add --enable-soroban-diagnostic-events flag #435

Merged
merged 7 commits into from
Mar 28, 2023
Merged

Conversation

paulbellamy
Copy link
Contributor

@paulbellamy paulbellamy commented Mar 24, 2023

Part of stellar/stellar-cli#471

Add a new quickstart flag to enable diagnostic events.

Closes #436 and #474

Note: This will need core upgraded in order to do anything.

An alternative to an explicit flag would be to just always enable diagnostic events for non-futurenet deployments.

@paulbellamy paulbellamy requested a review from sreuland March 24, 2023 11:12
@sreuland
Copy link
Contributor

@paulbellamy , is there a specific issue/ticket related to this task? Wanted to check on context, thanks.

@paulbellamy
Copy link
Contributor Author

Not directly. We were originally going to do #434, but decided that (because normal events are not logged) diagnostic events should not be logged. We still need a way to enable it for quickstart, though, so added this flag instead.

@sreuland
Copy link
Contributor

I saw the soroban tests failing, the core git ref for soroban needs to be bumped in any case related to soroban release prep, so, should be ok if we do that here, added a commit - 6c9a2e1

@sreuland
Copy link
Contributor

friendbot tx subs are failing when soroban core is enabled, running the newer core version there, gonna try this out locally and see what's up

sreuland and others added 4 commits March 24, 2023 15:26
…cache key based on source branch ref was not invalidating the horizon/friendbot build image cache key when branch had new commits
…hat to latest yet, go:soroban-xdr-next doesn't have that latest xdr yet
…t bump that to latest yet, go:soroban-xdr-next doesn't have that latest xdr yet"

This reverts commit 1d008c1.
@paulbellamy
Copy link
Contributor Author

Depends on stellar/stellar-cli#536, to update the go repo dependency, to support the new TOML field.

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.

Include diagnostic events on soroban enabled core logging
2 participants