Skip to content

[TRIVIAL] Fix playground configs#4108

Merged
jmg-duarte merged 1 commit intomainfrom
jmgd/playground/configs
Jan 30, 2026
Merged

[TRIVIAL] Fix playground configs#4108
jmg-duarte merged 1 commit intomainfrom
jmgd/playground/configs

Conversation

@jmg-duarte
Copy link
Copy Markdown
Contributor

@jmg-duarte jmg-duarte commented Jan 30, 2026

Description

Estimators were expecting different strings and the tx gas was missing from the driver

Changes

  • Remove Native from gas estimators
  • Add "Driver" to the native price estimators
  • Add tx-gas-limit to the driver config

How to test

Run docker compose and check if autopilot, orderbook and driver are up

Use Driver price estimator, Web3-only gas estimation, disable monitoring services, and set 45M gas limit.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@jmg-duarte jmg-duarte requested a review from a team as a code owner January 30, 2026 12:31
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request applies several configuration fixes to the playground docker-compose files and the driver.toml configuration. The changes correct estimator names and add a missing transaction gas limit. A high-severity issue with a duplicated environment variable in one of the compose files has been identified and should be addressed.

Comment thread playground/docker-compose.non-interactive.yml
@jmg-duarte jmg-duarte added this pull request to the merge queue Jan 30, 2026
Merged via the queue into main with commit 64cd00b Jan 30, 2026
19 checks passed
@jmg-duarte jmg-duarte deleted the jmgd/playground/configs branch January 30, 2026 13:17
@github-actions github-actions Bot locked and limited conversation to collaborators Jan 30, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants