Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] Manifest files with multiple containers fail to run #533

Open
BenElgar opened this issue Nov 22, 2021 · 1 comment
Open

[BUG] Manifest files with multiple containers fail to run #533

BenElgar opened this issue Nov 22, 2021 · 1 comment
Labels
bug Something isn't working
Milestone

Comments

@BenElgar
Copy link
Contributor

If one attempts to start a manifest file that has more than one container attached, only one of the containers will successfully start:
exo.hcl:

exo = "0.1"
components {
  container "t0" {
    image = "bash"
    command  = "sleep infinity"
  }
  container "t1" {
    image = "bash"
    command  = "sleep infinity"
  }
  container "t2" {
    image = "bash"
    command  = "sleep infinity"
  }
}
❯ dexo run
GUI available at: http://localhost:44643/#/workspaces/ebe2t9ssc4z470eg5t9r0hj0s0
Job URL: http://localhost:44643/#/jobs/jgvwcfydhzxrh3snhe6ad4qemg
applying
⨯ ├─ adding t0  removing existing container {"" '\x00' "" ""}: Error response from daemon: removal of container c9f3ec5543175e2ce375af069daea0de0647c3e68110ddb251f8ab7aadf433a6 is already in progress
✓ ├─ adding t1
⨯ └─ adding t2  removing existing container {"" '\x00' "" ""}: Error response from daemon: removal of container c9f3ec5543175e2ce375af069daea0de0647c3e68110ddb251f8ab7aadf433a6 is already in progress

I added a repro of the problem to the test suite: #532

@BenElgar BenElgar added bug Something isn't working needs triage Incoming request that needs to be triaged and prioritized labels Nov 22, 2021
@BenElgar
Copy link
Contributor Author

BenElgar commented Dec 1, 2021

This can be worked around by adding a container name explicitly. This works fine:

exo = "0.1"
components {
  container "t0" {
    image = "bash"
    command = "sleep infinity"
    container_name = "t0"
  }
  container "t1" {
    image = "bash"
    command = "sleep infinity"
    container_name = "t1"
  }
}

@brandonbloom brandonbloom added this to the Overhaul milestone Feb 28, 2022
@brandonbloom brandonbloom removed overhaul needs triage Incoming request that needs to be triaged and prioritized labels Feb 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants