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.

Sign up for GitHub

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

Jump to bottom

Terminal crashes on showing docker status #2434

Closed
belyaev-andrey opened this issue Aug 15, 2019 · 6 comments
Closed

Terminal crashes on showing docker status #2434

belyaev-andrey opened this issue Aug 15, 2019 · 6 comments
Labels
Area-TerminalConnection Issues pertaining to the terminal<->backend connection interface Issue-Bug It either shouldn't be doing this or needs an investigation. Needs-Attention The core contributors need to come back around and look at this ASAP. Needs-Tag-Fix Doesn't match tag requirements Product-Terminal The new Windows Terminal. Severity-Crash Crashes are real bad news.

Comments

@belyaev-andrey
Copy link

Environment

Windows Terminal (Preview)
Version: 0.3.2171.0

Platform ServicePack Version      VersionString                                                                         
-------- ----------- -------      -------------                                                                          
Win32NT             10.0.18362.0 Microsoft Windows NT 10.0.18362.0                                                     

Docker for Windows 
Version 2.1.0.1(37199)
Channel stable

h2 database image running in docker

Steps to reproduce

  1. Start Windows Terminal (Preview) with PowerShell as a default environment
  2. Open Linux Ubuntu tab as a second tab
  3. In the second tab execute ls command
  4. Close the second tab
  5. In the first tab execute docker ps command

Expected behavior

Docker list of running images is shown

Actual behavior

CONTAINER ID is printed in the console and then the terminal window closes.

Logs attached: Terminal.zip
@ghost ghost added Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting Needs-Tag-Fix Doesn't match tag requirements labels Aug 15, 2019
@zadjii-msft
Copy link
Member

So I'm going to go out on a limb here - this doesn't repro if you take out steps 2, 3, and 4, right? Just running docker ps works fine, correct?

@zadjii-msft zadjii-msft added the Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something label Aug 15, 2019
@belyaev-andrey
Copy link
Author

Correct. If I just execute docker ps in the power shell without opening the linux terminal, it works fine.

@ghost ghost added Needs-Attention The core contributors need to come back around and look at this ASAP. and removed Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something labels Aug 15, 2019
@zadjii-msft
Copy link
Member

Fantastic. I'm 80-90% sure that this is a dupe of #the-bug-fixed-by #2198, and it might also be #2394

@DHowett-MSFT DHowett-MSFT added Area-TerminalConnection Issues pertaining to the terminal<->backend connection interface Product-Terminal The new Windows Terminal. Severity-Crash Crashes are real bad news. Issue-Bug It either shouldn't be doing this or needs an investigation. and removed Needs-Attention The core contributors need to come back around and look at this ASAP. Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting labels Aug 19, 2019
@ghost ghost removed the Needs-Tag-Fix Doesn't match tag requirements label Aug 19, 2019
@zadjii-msft
Copy link
Member

This might also be #2525 or even #2375

@DHowett-MSFT
Copy link
Contributor

It's been a very long time since this was reported. Is it still happening?

@DHowett-MSFT DHowett-MSFT added the Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something label May 12, 2020
@belyaev-andrey
Copy link
Author

Looks OK to me now.

@ghost ghost added Needs-Attention The core contributors need to come back around and look at this ASAP. Needs-Tag-Fix Doesn't match tag requirements and removed Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something labels May 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Area-TerminalConnection Issues pertaining to the terminal<->backend connection interface Issue-Bug It either shouldn't be doing this or needs an investigation. Needs-Attention The core contributors need to come back around and look at this ASAP. Needs-Tag-Fix Doesn't match tag requirements Product-Terminal The new Windows Terminal. Severity-Crash Crashes are real bad news.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@belyaev-andrey @DHowett-MSFT @zadjii-msft