Skip to content

[TUI] Configurable tick interval#1669

Merged
martin-g merged 3 commits intoapache:mainfrom
martin-g:configurable-tick-interval
May 8, 2026
Merged

[TUI] Configurable tick interval#1669
martin-g merged 3 commits intoapache:mainfrom
martin-g:configurable-tick-interval

Conversation

@martin-g
Copy link
Copy Markdown
Member

@martin-g martin-g commented May 8, 2026

Which issue does this PR close?

N/A

Rationale for this change

Make it possible to use a custom tick interval for the UI updates.

What changes are included in this PR?

  • Added a new setting in the TUI config: tick_interval with default value of 2000ms.
  • Change the config dir to dirs::config_local_dir()
  • Rename the config file from .../ballista/config.yaml to .../ballista/tui.yaml

Example full paths:

|Platform | Example                                  |
| ------- | ---------------------------------------- |
| Linux   | /home/alice/.config/ballista/tui.yaml                      |
| macOS   | /Users/Alice/Library/Application Support/ballista/tui.yaml |
| Windows | C:\Users\Alice\AppData\Local\ballista\tui.yaml             |

All config settings could be overwritten with env vars:

Are there any user-facing changes?

The config file is renamed and a new setting is added.

martin-g added 2 commits May 8, 2026 11:18
Rename the config file from `.../ballista/config.yaml` to
`.../ballista/tui.yaml`

Full paths:
```
|Platform | Example                                  |
| ------- | ---------------------------------------- |
| Linux   | /home/alice/.config/ballista/tui.yaml                      |
| macOS   | /Users/Alice/Library/Application Support/ballista/tui.yaml |
| Windows | C:\Users\Alice\AppData\Local\ballista\tui.yaml             |
```

Override with env vars:
* BALLISTA_HTTP_TIMEOUT=30000
* BALLISTA_SCHEDULER_URL=https://example.com:50050
* BALLISTA_TICK_INTERVAL=1000
pub scheduler: SchedulerSettings,
pub http: HttpSettings,
/// How often to refresh the UI. In millis.
pub tick_interval: u64,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

tick_interval_ms maybe ?

Log the configuration as DEBUG at app start
@martin-g martin-g merged commit c65b41a into apache:main May 8, 2026
17 checks passed
@martin-g martin-g deleted the configurable-tick-interval branch May 8, 2026 14:00
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