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] envinfo hang on node:alpine #81

Closed
gengjiawen opened this issue Jan 10, 2019 · 10 comments · Fixed by #82 · May be fixed by #267
Closed

[Bug] envinfo hang on node:alpine #81

gengjiawen opened this issue Jan 10, 2019 · 10 comments · Fixed by #82 · May be fixed by #267

Comments

@gengjiawen
Copy link
Collaborator

docker run -it node:alpine /bin/sh
npm i -g envinfo && envinfo
@SimenB
Copy link

SimenB commented Jan 10, 2019

Interestingly, it does output if you hit ^C

$ docker run -it node:alpine npx envinfo
npx: installed 1 in 1.353s
^C
  System:
    OS: Linux 4.9 Alpine Linux undefined
    CPU: (6) x64 Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz
    Memory: 316.87 MB / 1.95 GB
    Container: Yes
    Shell: Unknown - /bin/ash
  Binaries:
    Node: 11.6.0 - /usr/local/bin/node
    Yarn: 1.12.3 - /usr/local/bin/yarn
    npm: 6.5.0-next.0 - /usr/local/bin/npm

It does hang up until hitting ^C, though. And running it from /bin/sh within the image doesn't output even after ^C. I have no idea why...

@tabrindle
Copy link
Owner

Looking into this now.

@tabrindle
Copy link
Owner

using docker run -it -e ENVINFO_DEBUG=trace node:8 npx envinfo I found the issue.

getShellInfo is hanging, possibly because of the type of shell - /bin/ash.

Working on a fix.

@gengjiawen
Copy link
Collaborator Author

@tabrindle This occurred again, can you reopen this ?

@tabrindle tabrindle reopened this Aug 27, 2019
@tabrindle
Copy link
Owner

confirmed.

@borgogelli
Copy link

+1

@ryhinchey
Copy link
Collaborator

@tabrindle it looks like the work you did to fix this bug got reverted at some point. I'll work on a PR adding it back soon!

@ryhinchey
Copy link
Collaborator

this issue can be closed given #147 got merged

@karlhorky
Copy link

I guess the Alpine Linux tests were removed again in Feb 2024?

@karlhorky
Copy link

karlhorky commented Oct 15, 2024

It seems like envinfo is hanging again on Alpine Linux (I could also confirm this myself):

@gengjiawen @tabrindle should this issue (#81) be reopened or should it be continued in #266?

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 a pull request may close this issue.

6 participants