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 dev container #375

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

Conversation

constambeys
Copy link

fix dev container fails

1 package is looking for funding
run npm fund for details
./scripts/setup.sh: line 13: cd: ./scripts/..: No such file or directory

Copy link
Collaborator

@chemelli74 chemelli74 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @constambeys and thx for contributing!

In order to install the plugin outside the git folder, we need to change directory.
Current container works fine for me on several different projects.
We need to investigate what's wrong on your scenario.

@constambeys
Copy link
Author

I am new to this project but I am interested in how it works !

Dev container does not build, The directory is already changed in line 5 to the root why changing again !

@chemelli74
Copy link
Collaborator

Dev container does not build, The directory is already changed in line 5 to the root why changing again !

Because you need to install in different path the different components.

Back to your issue:

  1. What's is not working exactly ?
  2. How are you creating/starting the devcontainer ?
  3. Which is the docker version you are running ?
  4. Is the host Windows or Linux ?

@constambeys
Copy link
Author

Screenshot from 2024-10-28 10-19-23

  1. See attachment !
    
  2. Reopen folder in container
    
  3. Docker Engine v27.2.0
    
  4. Tried on both Fedora Linux and Windows 11
    

@chemelli74
Copy link
Collaborator

  1. Reopen folder in container
    

You should use instead
image

@constambeys
Copy link
Author

Still the same !!!

Maybe you should add some debug messages to understand what is going on !

@wuwentao
Copy link
Owner

wuwentao commented Nov 5, 2024

the screenshot should not the root cause, and the error should be exist before this screenshot.
we need to find out the first error step, and it may not report error or exit 1, so the jobs continue running, and you got this error at the end of the scripts.

you can check the error before it, there is detail log for devcontainer build job, vscode will print all.
if you still can't fix it, maybe you can upload devcontainer log as a file.

@wuwentao
Copy link
Owner

wuwentao commented Nov 5, 2024

I have test with my windows PC with WSL2, try to run dev container jobs: [Rebuild devcontainer without cache], and got the same error.

Running the postCreateCommand from devcontainer.json...

[379457 ms] Start: Run in container: /bin/sh -c scripts/setup.sh
pre-commit installed at .git/hooks/pre-commit
pre-commit installed at .git/hooks/commit-msg

it start to run this script , and got an error with it:

[389501 ms] [15:28:52] Error while preparing to install dependencies and extension packs of the extension: ryanluker.vscode-coverage-gutters
[389503 ms] [15:28:52] Error while installing extensions: 40482AC94B7F0000:error:0A000119:SSL routines:ssl3_get_record:decryption failed or bad record mac:../deps/openssl/openssl/ssl/record/ssl3_record.c:622:

[389504 ms] [15:28:52] Failed: 40482AC94B7F0000:error:0A000119:SSL routines:ssl3_get_record:decryption failed or bad record mac:../deps/openssl/openssl/ssl/record/ssl3_record.c:622:

so, the result is :

added 11 packages in 46s

1 package is looking for funding
  run `npm fund` for details
scripts/setup.sh: line 13: cd: scripts/..: No such file or directory

@wuwentao
Copy link
Owner

wuwentao commented Nov 5, 2024

a ttach the full log as below:
devcontainer.log

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.

3 participants