chore: Prevent unnecessary API calls to get container execution details

[morgan-wowk]

Description

Before

After

Optimized container execution state and log fetching by preventing unnecessary API calls for containers in expected non-execution states. Added status-based conditional fetching that skips requests when containers are in states like INVALID, UNINITIALIZED, QUEUED, WAITING_FOR_UPSTREAM, or SKIPPED, where execution records and logs are not expected to exist. Refined log polling logic to only poll for actively running containers (RUNNING, PENDING, CANCELLING) while still allowing log fetches for completed states that may have logs.

Related Issue and Pull requests

Type of Change

• Improvement

Checklist

• I have tested this does not break current pipelines / runs functionality

Screenshots (if applicable)

Test Instructions

1. Navigate to task details for containers with statuses: INVALID, UNINITIALIZED, QUEUED, WAITING_FOR_UPSTREAM, or SKIPPED
2. Verify that no container execution state or log API calls are made for these statuses
3. Confirm that containers with RUNNING, PENDING, or CANCELLING statuses actively poll for logs every 5 seconds
4. Test that completed containers (SUCCEEDED, FAILED, SYSTEM_ERROR, CANCELLED) fetch logs once without polling
5. Verify that the ExecutionDetails and Logs components still render correctly for all status types

Additional Comments

This change reduces unnecessary network requests and prevents potential errors when trying to fetch execution state or logs for containers that haven't been launched or were terminated before execution began. The CANCELLED status is intentionally handled carefully since tasks cancelled mid-run may have logs uploaded by the backend before termination.

[github-actions]

🎩 To tophat this PR:

You can add the following URL parameter to your browser to tophat this PR:

`?tophat_location=reduce-status-checks/e0cc8ea`

---

[morgan-wowk]

• chore: Prevent unnecessary API calls to get container execution details #1893 👈 (View in Graphite)
• master

This stack of pull requests is managed by Graphite. Learn more about stacking.

[morgan-wowk]

I have updated this to respect that Cancelled might have some start and end time, or logs. So we will still request information for cancelled states to account for mid-run cancellations.

[maxy-shpfy]

I tophatted and seems no regression, works as intended - will take another look soon

[morgan-wowk]

@maxy-shpfy Just a friendly nudge. No rush. Still a desired change.

[camielvs]

Appears to be working

[morgan-wowk]

Merge activity

• Mar 31, 6:09 PM UTC: A user started a stack merge that includes this pull request via Graphite.
• Mar 31, 6:10 PM UTC: Graphite rebased this pull request as part of a merge.
• Mar 31, 6:12 PM UTC: @morgan-wowk merged this pull request with Graphite.