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

Remove use of soroban high limit override and replace with actual config upgrade #508

Closed
leighmcculloch opened this issue Oct 6, 2023 · 0 comments · Fixed by #513
Closed
Assignees

Comments

@leighmcculloch
Copy link
Member

leighmcculloch commented Oct 6, 2023

The --local network option of the quickstart image starts stellar-core with the TESTING_SOROBAN_HIGH_LIMIT_OVERRIDE config set to true.

This raises the limits of the network to essentially unlimited.

This was done as a work around because the config upgrade script that exists today for stellar-core is complex, has a lot of dependencies that we can't guarantee are up-to-date when new XDR ships, and difficult to use. The quickstart image needs to be able to be built with bleeding edge versions of stellar-core, horizon, and soroban-rpc, and as such it can't be made dependent on SDKs that may take sometime to update with new XDR.

However, using the high limit override has some with its own costs.

It doesn't give us control over limits and the ecosystem has been clear that it needs to be able to test against testnet/pubnet limits locally and not just unlimited limits.

Also, issues have arisen where it appears that quickstart local's limits are behaving oddly in buggy ways in comparison to futrenet/testnet network deployments.

Both of those issues have been discussed here:
https://discord.com/channels/897514728459468821/1155167115142254643

In hindsight I think we made the wrong call to rely on a flag that flipped to unlimited and enabled specialized behavior. We need to be able to replicate testnet/pubnet behavior in quickstart with the --local option, and to do so we should use actual config upgrades to configure soroban.

This issue is dependent on:

cc @anupsdf

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 a pull request may close this issue.

1 participant