Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Make job docker container name meaningful. (#12503)
Closes #9584. This now follows the Kubernetes name convention of <image-name>-<job-id>-<attempt-number>. This should make things easier for oss users doing oncall for docker deployments. Mirror the Kubernetes name convention: https://github.com/airbytehq/airbyte/blob/master/airbyte-workers/src/main/java/io/airbyte/workers/process/KubeProcessFactory.java#L180 We can probably reuse the naming functions across both processes. We are going to do more changes in this area around making things more ergonomic (e.g. instead of always using sync, we want to replace this with the actual operation - discover, check connection etc.) so we'll do refactoring in that PR. I want to unblock this in the time being since open source users are having difficulty debugging now.
- Loading branch information