Skip to content

NAS-141238 / 27.0.0-BETA.1 / Do not poll network interface status in CLI menu#130

Merged
themylogin merged 1 commit into
masterfrom
NAS-141238
Jun 12, 2026
Merged

NAS-141238 / 27.0.0-BETA.1 / Do not poll network interface status in CLI menu#130
themylogin merged 1 commit into
masterfrom
NAS-141238

Conversation

@themylogin

Copy link
Copy Markdown
Contributor

Running self._build_cli starts a loop that polls network interface unapplied changes status from middleware. This (and polling the terminal width) are the only background activities on an idling CLI menu. Let's postpone starting the polling thread until the TrueNAS CLI is actually requested.

@themylogin themylogin requested a review from yocalebo June 4, 2026 17:29
@bugclerk bugclerk changed the title Do not poll network interface status in CLI menu NAS-141238 / 27.0.0-BETA.1 / Do not poll network interface status in CLI menu Jun 4, 2026
@bugclerk

bugclerk commented Jun 4, 2026

Copy link
Copy Markdown

@themylogin

Copy link
Copy Markdown
Contributor Author

time 1:00

@themylogin themylogin requested a review from a team June 5, 2026 15:27
themylogin added a commit to truenas/middleware that referenced this pull request Jun 5, 2026
…aks in python-prompt-toolkit (#19080)

The actual issue is fixed by truenas/midcli#130
yocalebo pushed a commit to truenas/middleware that referenced this pull request Jun 6, 2026
…s in python-prompt-toolkit (by themylogin) (#19083)

The actual issue is fixed by truenas/midcli#130

Original PR: #19080

Co-authored-by: themylogin <themylogin@gmail.com>
@themylogin themylogin merged commit e3df82f into master Jun 12, 2026
2 checks passed
@themylogin themylogin deleted the NAS-141238 branch June 12, 2026 08:10
@bugclerk

Copy link
Copy Markdown

This PR has been merged and conversations have been locked.
If you would like to discuss more about this issue please use our forums or raise a Jira ticket.

@truenas truenas locked as resolved and limited conversation to collaborators Jun 12, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants