UI: Fix bug where task logs would suddenly switch to another task's logs

[DingoEatingFuzz]

Fixes #8545

Issue
When streaming logs for a task, the logs will suddenly flip to the logs of a different task for the allocation. When this happens, the entire log output is flipped.

Reproduction

1. Run a job with multiple tasks (job file at the bottom)
2. Open the logs for one task in a tab
3. In another tab, open the task detail page of the other tab.
4. Restart the task.
5. Sometimes this will cause the logs of the task being streamed to change to the other tasks

Root cause
A few things are important to know to understand the root cause.

1. Tasks are modeled as Fragments.
2. The task logs page watches the allocation, which means whenever it changes server-side it is reloaded client-side.
3. TaskStates in the API response are a map[string]interface{} that get converted to an array of objects in the UI

So what happens is when the allocation is reloaded, the API response is normalized and pushed into the Ember Data store. When the TaskStates are converted to an array, the order of tasks aren't stable. So when tasks switch places in the array, their respective records in the store are updated as if they are the other. This causes Task2 to become Task1 and vice versa.

Fixing it is simple, just sort the keys when converting objects to arrays to ensure stable model fragments.

Job for Reproduction

job "multi" {
  datacenters = ["dc1"]

  group "cache" {
    task "one" {
      driver = "raw_exec"

      config {
        command = "/bin/bash"
        args = [ "-c", "while true; do echo 'One Fish'; sleep 1; done" ]
      }
    }

    task "two" {
      driver = "raw_exec"

      config {
        command = "/bin/bash"
        args = [ "-c", "while true; do echo 'Two Fish'; sleep 1; done" ]
      }
    }
  }
}

[github-actions]

Ember Asset Size action

As of 3147d5c

Files that got Bigger 🚨:

File	raw	gzip
nomad-ui.js	+28 B	+5 B

Files that stayed the same size 🤷‍:

File	raw	gzip
vendor.js	0 B	0 B
nomad-ui.css	0 B	0 B
vendor.css	0 B	0 B

[github-actions]

Ember Test Audit comparison

	master	3147d5c	change
passes	1385	1385	0
failures	0	0	0
flaky	0	1	+1
duration	7m 33s 611ms	6m 57s 909ms	-35s 702ms

[backspace]

wow, nice sleuthing!! 😯

[github-actions]

I'm going to lock this pull request because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.