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

[build] use overlay2 for DinD by default #19078

Merged
merged 1 commit into from
Jun 5, 2024

Conversation

k-v1
Copy link
Contributor

@k-v1 k-v1 commented May 26, 2024

Why I did it

overlay2 is more faster comparing to vfs (more info in #11111)
And we use this option to build SONiC almost two years.
But it may be not obvious for some people that default choice is slow.
I think we should set overlay2 as a default storage driver for DinD.

Work item tracking
  • Microsoft ADO (number only):

How I did it

Change default value of SONIC_SLAVE_DOCKER_DRIVER in rules/config

How to verify it

Build SONiC and compare build time with vfs and overlay2.

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

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

@k-v1
Copy link
Contributor Author

k-v1 commented May 26, 2024

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@lguohan
Copy link
Collaborator

lguohan commented May 31, 2024

@xumia , can you check?

Copy link
Collaborator

@xumia xumia left a comment

Choose a reason for hiding this comment

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

LGTM

@lguohan lguohan merged commit ff39ff6 into sonic-net:master Jun 5, 2024
20 checks passed
@lguohan lguohan added the Build label Jun 5, 2024
@k-v1 k-v1 deleted the use_overlay2_by_default branch June 5, 2024 20:59
arun1355492 pushed a commit to arun1355492/sonic-buildimage that referenced this pull request Jul 26, 2024
overlay2 is more faster comparing to vfs (more info in sonic-net#11111)
And we use this option to build SONiC almost two years.
But it may be not obvious for some people that default choice is slow.
I think we should set overlay2 as a default storage driver for DinD.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants