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

[20.10] Revert "Ignore SIGURG on Linux." #2987

Merged

Conversation

thaJeztah
Copy link
Member

@thaJeztah thaJeztah commented Mar 1, 2021

for 20.10, this fixes moby/moby#42093 docker 20.10.4 on linux and windows breaks docker-compose run -T
for 20.10, this fixes #2989 docker start -a does not start containers

This reverts commit 3c87f01 (#2960) which was a backport of #2929

This commit introduced two regressions;

Reverting for now, while we dig deeper into what's causing the regression.

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

This reverts commit 3c87f01.

This commit introduced two regressions;

- spurious "Unsupported signal: <nil>. Discarding."
- docker start --attach hanging if the container does not
  have a TTY attached

Reverting for now, while we dug deeper into what's causing
the regression.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@tiborvass
Copy link
Collaborator

I think with #2986 and #2990 we no longer need the revert.

@tiborvass tiborvass merged commit 55c4c88 into docker:20.10 Mar 2, 2021
@thaJeztah thaJeztah deleted the 20.10_revert_backport_ignore_sigurg branch March 2, 2021 18:35
kaosagnt added a commit to kaosagnt/boot2docker-xfs-ng that referenced this pull request Mar 3, 2021
Revert docker/cli#2960 to fix hanging in docker start --attach and remove
spurious “Unsupported signal: . Discarding." messages
[docker/cli#2987](docker/cli#2987)
kaosagnt added a commit to kaosagnt/toolbox2docker that referenced this pull request Mar 3, 2021
https://docs.docker.com/engine/release-notes/

2021-03-02 Client
Revert docker/cli#2960 to fix hanging in docker start --attach and
remove spurious “Unsupported signal: . Discarding." messages docker/cli#2987
This was referenced Mar 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants