-
Notifications
You must be signed in to change notification settings - Fork 189
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
Update gitpod to cache_from docker.cdot.systems #3371
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tried this on gitpod, and get:
bash: pnpm: command not found
WARNING: The GIT_COMMIT variable is not set. Defaulting to a blank string.
WARNING: The GIT_COMMIT variable is not set. Defaulting to a blank string.
ERROR: The Compose file './docker/gitpod.yml' is invalid because:
services.nginx.build contains unsupported option: 'ports'
ERROR: The Compose file './docker/gitpod.yml' is invalid because:
services.nginx.build contains unsupported option: 'ports'
[1]- Exit 127 pnpm install
[2]+ Exit 1 docker-compose --env-file .env build $SERVICES
exit
🤙 This task ran as a workspace prebuild
WARNING: The GIT_COMMIT variable is not set. Defaulting to a blank string.
ERROR: The Compose file './docker/gitpod.yml' is invalid because:
services.nginx.build contains unsupported option: 'ports'
bash: pnpm: command not found
You need to bump nginx
's volumes
and ports
one tab-stop to the left so it's in the same level as build
vs. under it.
Also, I'm not sure what's up with the pnpm
not found bit. cc @DukeManh, @Kevan-Y.
I am to blame #2915 We removed Line 1 in 9d86ea7
|
It's almost there now, it kinda built but couldn't connect to elasticsearch. Will try the suggested solution of "Use
|
This is #3369, we can do what is suggested there and add a |
Should I add the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like containers are not running.
.gitpod.Dockerfile
Outdated
@@ -1,4 +1,4 @@ | |||
FROM gitpod/workspace-base | |||
FROM gitpod/workspace-node | |||
|
|||
# Installing Node LTS |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now we are working with workspace-node
, you can remove node
and docker-compose
and nvm
installion below
Still giving me the same error after adding
|
The problem above is that the |
Hi @TDDR, Just wondering if you're still working on this PR. |
94eec21
to
e27efa5
Compare
Issue This PR Addresses
Fixes #3340
Type of Change
Description
We needed to update the GitPod yaml in order to incorporate the move to docker.cdot.systems based images.
Steps to test the PR
Open up GitPod and see if it works.