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

fix: npm run start-everest should run 'docker compose' command #323

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

fabiolnm
Copy link
Contributor

@fabiolnm fabiolnm commented Jan 11, 2025

A developer who clones the repository today and follow the "Testing with Everest" instructions will be blocked by the error message below. It was possible to trace the root cause to a typo in the npm start-everest script.

npm run start-everest

> @citrineos/workspace@1.5.0 start-everest
> npm run start-everest --prefix ./Server


> @citrineos/server@1.5.0 start-everest
> cd ./everest && cross-env EVEREST_IMAGE_TAG=0.0.16 EVEREST_TARGET_URL=ws://host.docker.internal:8081/cp001 docker-compose up

node:events:485
      throw er; // Unhandled 'error' event
      ^

Error: spawn docker-compose ENOENT
    at ChildProcess._handle.onexit (node:internal/child_process:286:19)
    at onErrorNT (node:internal/child_process:484:16)
    at process.processTicksAndRejections (node:internal/process/task_queues:90:21)
Emitted 'error' event on ChildProcess instance at:
    at ChildProcess._handle.onexit (node:internal/child_process:292:12)
    at onErrorNT (node:internal/child_process:484:16)
    at process.processTicksAndRejections (node:internal/process/task_queues:90:21) {
  errno: -2,
  code: 'ENOENT',
  syscall: 'spawn docker-compose',
  path: 'docker-compose',
  spawnargs: [ 'up' ]
}

@fabiolnm fabiolnm force-pushed the fix-docker-compose-npm-everest-command branch from 123bc8c to 66d5559 Compare January 11, 2025 08:01
@fabiolnm fabiolnm changed the title fix: npm run everest should run 'docker compose' command fix: npm run start-everest should run 'docker compose' command Jan 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant