Josh Deprez

### Description ### Context ### Changes ### Testing - [ ] Tests have run locally (with `go test ./...`). Buildkite employees may check this if the pipeline has run automatically....

**WIP**: This currently doesn't work. The controller container simply doesn't have permissions to execute a script. (That sounds sensible actually?) So it will have to go in another container. ---...

The later one overrides the previous one, so it can be safely removed.

The more reliable approach is to build a container image containing the hooks you want and then set `BUILDKITE_HOOKS_PATH`. But if that's too much work, a pre-bootstrap script might be...

The `kubernetes` plugin uses the Kubernetes definition of `podSpec`. [Kubernetes defines](https://kubernetes.io/docs/tasks/inject-data-application/define-command-argument-container/#running-a-command-in-a-shell) the `command` and `args` of a `container` to mean a single command entrypoint for the container (`command` is called...

Forgot to fix this in #304

I ran a command similar to following: ```bash createrepo_c \ --no-database \ --unique-md-filenames \ --retain-old-md-by-age=180d \ --update \ --pkglist

I'd like to find a way to Dependabot this And we should probably unblock updating the AWS CLI version, and update it

The main improvement is the removal of `err` as a _field_ from `jobWrapper`. The second is clearer logic for the error path (fallback to failure job).