Skip to content

[build] Fix debian mirror snapshot timestamp not work issue. (#21758) #21767

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

Merged
merged 1 commit into from
Feb 21, 2025

Conversation

liushilongbuaa
Copy link
Contributor

@liushilongbuaa liushilongbuaa commented Feb 18, 2025

Why I did it

PR21366 wants to use debian snapshot instead of version pinning to improve image quality. But it missed this change. Build is using latest debian snapshot instead of validated snapshot.
Now validated snapshot is '20250130'. details in this link. But we can find build is using '20250216', which is a bad snapshot. It breaks SONiC build since Feb 16.

Get:18 http://packages.trafficmanager.net/snapshot/debian-security/20250216T001420Z bullseye-security/non-free amd64 Packages [1164 B]
Get:19 http://packages.trafficmanager.net/snapshot/debian-security/20250216T001420Z bullseye-security/main amd64 Packages [346 kB]
Fetched 17.9 MB in 5s (3789 kB/s)
Reading package lists...
E: Failed to fetch http://packages.trafficmanager.net/snapshot/debian-security/20250216T001420Z/dists/bullseye-security/main/source/Sources.xz  File has unexpected size (234932 != 234820). Mirror sync in progress? [IP: 13.107.246.71 80]
   Hashes of expected file:
    - Filesize:234820 [weak]
    - SHA256:99cbb6795504058c2b3b2355ad128d8ca7879c136ba8fc00f2813ea4d5e7d147
   Release file created at: Sat, 15 Feb 2025 17:04:32 +0000
E: Some index files failed to download. They have been ignored, or old ones used instead.
Work item tracking
  • Microsoft ADO (number only):

How I did it

Fix debian mirror snapshot timestamp generate script.

How to verify it

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)

…et#21758)

Why I did it
Recently microsoft owned debian mirror has an issue. It is using the latest timestamp '20250216'. And this snapshot is a bad snapshot. Bydesign we should use timestamp '20250130', which passed PR validation.

Get:18 http://packages.trafficmanager.net/snapshot/debian-security/20250216T001420Z bullseye-security/non-free amd64 Packages [1164 B]
Get:19 http://packages.trafficmanager.net/snapshot/debian-security/20250216T001420Z bullseye-security/main amd64 Packages [346 kB]
Fetched 17.9 MB in 5s (3789 kB/s)
Reading package lists...
E: Failed to fetch http://packages.trafficmanager.net/snapshot/debian-security/20250216T001420Z/dists/bullseye-security/main/source/Sources.xz  File has unexpected size (234932 != 234820). Mirror sync in progress? [IP: 13.107.246.71 80]
   Hashes of expected file:
    - Filesize:234820 [weak]
    - SHA256:99cbb6795504058c2b3b2355ad128d8ca7879c136ba8fc00f2813ea4d5e7d147
   Release file created at: Sat, 15 Feb 2025 17:04:32 +0000
E: Some index files failed to download. They have been ignored, or old ones used instead.
By design, we should use timestamp in https://github.com/sonic-net/sonic-buildimage/blob/master/files/build/versions/default/versions-mirror

How I did it
Fix debian mirror snapshot timestamp generate script.
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liushilongbuaa liushilongbuaa marked this pull request as ready for review February 18, 2025 05:23
@liushilongbuaa liushilongbuaa requested review from wangxin and removed request for lguohan, qiluo-msft and xumia February 19, 2025 03:28
@wangxin wangxin merged commit 31a86d8 into sonic-net:202311 Feb 21, 2025
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants