Performance #1035

Open

adamruzicka wants to merge 35 commits into

theforeman:masterfrom

adamruzicka:perf

Commits on Jul 13, 2026

Wait for previous poll response before scheduling the next one

adamruzicka
and
claude
committed
Fetch permissions only on initial load, not on every poll tick

adamruzicka
and
claude
committed
Remove updateJob — its response was never read

adamruzicka
and
claude
committed
Fix host table firing duplicate API calls on mount

adamruzicka
and
claude
committed
Include cancellable in job invocation task, drop separate task fetch

adamruzicka
and
claude
committed
Replace setInterval with setTimeout chain in TemplateInvocation polling

adamruzicka
and
claude
committed
Stop job invocation polling when the job finishes

adamruzicka
and
claude
committed
Drop dead autoRefresh logic — status_label already implies task state

adamruzicka
and
claude
committed
Fix include_permissions being dropped from initial job invocation fetch

adamruzicka
and
claude
committed
Add tests for job invocation polling behaviour

adamruzicka
and
claude
committed
Apply simplifications from code review

adamruzicka
and
claude
committed
Fix TemplateInvocation poll race and drop dead jobId params

adamruzicka
and
claude
committed
Make lints happy
adamruzicka
committed
Poll host table while job is running

adamruzicka
and
claude
committed
Refresh LIST_TEMPLATE_INVOCATIONS on page change and poll

adamruzicka
and
claude
committed
Include task and permissions in API hosts response

adamruzicka
and
claude
committed
Drop list_jobs_hosts — replaced by API hosts endpoint

adamruzicka
and
claude
committed
Gate task and permissions in hosts response behind include_permissions param

adamruzicka
and
claude
committed
Pass jobFinished as prop to JobInvocationHostTable

adamruzicka
and
claude
committed
Simplify polling helpers in actions and host table

adamruzicka
and
claude
committed
Fix polling cleanup inconsistencies and error toast fallbacks

adamruzicka
and
claude
committed
Make lints happy
adamruzicka
committed
Use try(:cancellable?) to handle base ForemanTasks::Task in tests

adamruzicka
and
claude
committed
Trim task data in hosts response to id and cancellable

adamruzicka
and
claude
committed
Unify pollHostTable into makeApiCall

adamruzicka
and
claude
committed
Simplify error toast extraction and reuse task lookup in permissions

adamruzicka
and
claude
committed
Replace redux-mock-store with a real Redux store in polling tests

adamruzicka
and
claude
committed
Add polling test coverage for TemplateInvocation and JobInvocationHostTable

adamruzicka
and
claude
committed
Fix ESLint failures in webpack test and source files

adamruzicka
and
claude
committed
Fix host table staleness on id change and redundant fetches for finished invocations

adamruzicka
and
claude
committed
Use sprintf placeholders for error response in cancelJob toast
adamruzicka
committed
Only fetch permissions on first load, not on every poll
adamruzicka
committed
Hoist host-independent permission checks out of the per-host loop
adamruzicka
committed
Replace module-level pollTimeoutId with useRef in JobInvocationActions
adamruzicka
committed
Add tests and fix nil task in authorized_for for hosts with include_permissions
adamruzicka
committed