Skip to content

[Build]: Use one debian mirror config#10274

Merged
xumia merged 4 commits intosonic-net:masterfrom
xumia:fix-dup-mirror
Mar 21, 2022
Merged

[Build]: Use one debian mirror config#10274
xumia merged 4 commits intosonic-net:masterfrom
xumia:fix-dup-mirror

Conversation

@xumia
Copy link
Copy Markdown
Collaborator

@xumia xumia commented Mar 18, 2022

Why I did it

Use one debian mirror config.
The empty config in https://github.com/Azure/sonic-buildimage/blob/master/files/image_config/apt/sources.list overrides the file https://github.com/Azure/sonic-buildimage/blob/master/files/apt/sources.list.amd64 (armhf/arm64), it does not make sense.
All the content in files/image_config/apt is no use, any one wants to add mirror config, please add in files/apt.

How I did it

Remove files/image_config/apt and the reference.

How to verify it

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111

Description for the changelog

Link to config_db schema for YANG module changes

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

@xumia xumia added the Build label Mar 18, 2022
@xumia xumia requested a review from lguohan as a code owner March 18, 2022 06:25
@xumia xumia requested a review from saiarcot895 March 18, 2022 06:26
@xumia xumia requested a review from qiluo-msft March 18, 2022 09:02
@xumia
Copy link
Copy Markdown
Collaborator Author

xumia commented Mar 19, 2022

@antony-rheneus , could you please help review it? It does not break armhf build, can we simplify it? See the checks in #10281
https://dev.azure.com/mssonic/be1b070f-be15-4154-aade-b1d3bfb17054/_build/results?buildId=82352

@xumia
Copy link
Copy Markdown
Collaborator Author

xumia commented Mar 20, 2022

@saiarcot895, could you please help review it?

@xumia xumia merged commit 1017ee6 into sonic-net:master Mar 21, 2022
@xumia xumia deleted the fix-dup-mirror branch March 21, 2022 08:47
xumia added a commit to xumia/sonic-buildimage-1 that referenced this pull request Mar 21, 2022
Why I did it
Use one debian mirror config.
The empty config in https://github.com/Azure/sonic-buildimage/blob/master/files/image_config/apt/sources.list overrides the file https://github.com/Azure/sonic-buildimage/blob/master/files/apt/sources.list.amd64 (armhf/arm64), it does not make sense.
All the content in files/image_config/apt is no use, any one wants to add mirror config, please add in files/apt.

How I did it
Remove files/image_config/apt and the reference.
xumia added a commit that referenced this pull request Mar 21, 2022
Why I did it
Use one debian mirror config.
The empty config in https://github.com/Azure/sonic-buildimage/blob/master/files/image_config/apt/sources.list overrides the file https://github.com/Azure/sonic-buildimage/blob/master/files/apt/sources.list.amd64 (armhf/arm64), it does not make sense.
All the content in files/image_config/apt is no use, any one wants to add mirror config, please add in files/apt.

How I did it
Remove files/image_config/apt and the reference.
xumia added a commit that referenced this pull request Mar 23, 2022
Why I did it
Use one debian mirror config.
The empty config in https://github.com/Azure/sonic-buildimage/blob/master/files/image_config/apt/sources.list overrides the file https://github.com/Azure/sonic-buildimage/blob/master/files/apt/sources.list.amd64 (armhf/arm64), it does not make sense.
All the content in files/image_config/apt is no use, any one wants to add mirror config, please add in files/apt.

How I did it
Remove files/image_config/apt and the reference.
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