Skip to content

[action] [PR:25398] [docker] Fix Docker commands hanging before docker.service starts#25503

Merged
mssonicbld merged 1 commit intosonic-net:202511from
mssonicbld:cherry/202511/25398
Mar 12, 2026
Merged

[action] [PR:25398] [docker] Fix Docker commands hanging before docker.service starts#25503
mssonicbld merged 1 commit intosonic-net:202511from
mssonicbld:cherry/202511/25398

Conversation

@mssonicbld
Copy link
Collaborator

Why I did it

Fix Docker socket activation broken since PR #2417 (January 2019). Docker commands executed before docker.service starts hang indefinitely because -H unix:// breaks systemd socket activation.

Fixes #25383

Work item tracking
  • Microsoft ADO (number only):

How I did it

Changed -H unix:// to -H fd:// in files/docker/docker.service.conf. Added documentation comments explaining why this flag is required to prevent future regression.

How to verify it

Verified on VS image:

  • docker.socket is active and triggers docker.service
  • dockerd runs with -H fd:// flag
  • Socket activation works correctly

Which release branch to backport (provide reason below if selected)

  • 202305
  • 202311
  • 202405
  • 202411
  • 202505
  • 202511

Reason: This is a bug fix that has existed since January 2019. All active release branches are affected and should receive this fix.

Tested branch (Please provide the tested image version)

  • master : fix_docker-socket-activation-25383.0-b01504e6c (sonic-vs.img.gz)

Description for the changelog

Fix Docker socket activation hang by using -H fd:// instead of -H unix:// in docker.service.conf

Link to config_db schema for YANG module changes

N/A

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

michael-sum-LEpfefQf4rU-unsplash

#### Why I did it
Fix Docker socket activation broken since PR sonic-net#2417 (January 2019). Docker commands executed before docker.service starts hang indefinitely because -H unix:// breaks systemd socket activation.

**Fixes sonic-net#25383**

##### Work item tracking
- Microsoft ADO **(number only)**:

#### How I did it
Changed `-H unix://` to `-H fd://` in  `files/docker/docker.service.conf`. Added documentation comments explaining why this flag is required to prevent future regression.

#### How to verify it

<!--
If PR needs to be backported, then the PR must be tested against the base branch and the earliest backport release branch and provide tested image version on these two branches. For example, if the PR is requested for master, 202211 and 202012, then the requester needs to provide test results on master and 202012.
-->

Verified on VS image:
- docker.socket is active and triggers docker.service
- dockerd runs with -H fd:// flag
- Socket activation works correctly

#### Which release branch to backport (provide reason below if selected)

<!--
- Note we only backport fixes to a release branch, *not* features!
- Please also provide a reason for the backporting below.
- e.g.
- [x] 202006
-->

- [x] 202305
- [x] 202311
- [x] 202405
- [x] 202411
- [x] 202505
- [x] 202511

**Reason:** This is a bug fix that has existed since January 2019. All active release branches are affected and should receive this fix.

#### Tested branch (Please provide the tested image version)

<!--
- Please provide tested image version
- e.g.
- [x] 20201231.100
-->

- [x] master : `fix_docker-socket-activation-25383.0-b01504e6c` (sonic-vs.img.gz)

#### Description for the changelog

Fix Docker socket activation hang by using -H fd:// instead of -H unix:// in docker.service.conf

<!--
 Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.
-->

#### Link to config_db schema for YANG module changes
<!--
Provide a link to config_db schema for the table for which YANG model
is defined
Link should point to correct section on https://github.com/Azure/sonic-buildimage/blob/master/src/sonic-yang-models/doc/Configuration.md
-->
N/A

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

![michael-sum-LEpfefQf4rU-unsplash](https://github.com/user-attachments/assets/a4f71f80-b11c-49f4-a562-26d63b89fdbe)
@mssonicbld
Copy link
Collaborator Author

Original PR: #25398

@vmittal-msft
Copy link
Contributor

/azpw run

@mssonicbld
Copy link
Collaborator Author

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld merged commit 2ed2850 into sonic-net:202511 Mar 12, 2026
17 checks passed
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.

2 participants