aristanetworks / sonic

Open source drivers and initialization library for Arista platforms running SONiC
GNU General Public License v2.0
25 stars 30 forks source link

Update dependencies for Linux kernel 5.10.46 #32

Closed KostiantynYarovyiBf closed 2 years ago

KostiantynYarovyiBf commented 2 years ago

Why I did it

In preparation for Debian Bullseye, upgrade SONiC's base system to be based on Bullseye, which was released in August 2021.

How I did it

Update Linux kernel 5.10.46 dependencies for debian bullseye in control file.

How to verify it

build sonic image with kernel 5.10 and verify it with this PR for your platform: https://github.com/Azure/sonic-buildimage/pull/8191

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

Description for the changelog

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

Staphylo commented 2 years ago

@zzhiyuan since he is following up on updating the drivers for bullseye. We can simply merge this change on master as doing this would break our ability to make further changes to sonic-buildimage until bullseye has merged in. We might want to create a dedicated branch for 5.10 with the control file. I was however convinced that we no longer needed to upgrade the control file as it was re-written with the expected version. FWIW we can't merge this PR through github and we will instead have to integrate it in our internal SVC.

KostiantynYarovyiBf commented 2 years ago

@Staphylo @zzhiyuan, Can you create a dedicated branch for 5.10? It will be a temporary fix until bullseye has merged in Sonic.

zzhiyuan commented 2 years ago

@KostiantynYarovyiBf This change won't work as the linux kernel version is rewritten in debian/rules. You'd need to delete lines 24 to 27 here: https://github.com/aristanetworks/sonic/blob/master/debian/rules#L24 This is done in our internal version, I'm still waiting for https://github.com/Azure/sonic-buildimage/pull/8890 to merge in before committing to another submodule update for arista (which will include the version change). Thanks

zzhiyuan commented 2 years ago

@KostiantynYarovyiBf if you're building with aristanetworks/sonic set to a certain commit and need the change, I can update the repo with the changes.

KostiantynYarovyiBf commented 2 years ago

@zzhiyuan, Barefoot platform building with aristanetworks/sonic set to a certain commit and need the change.

zzhiyuan commented 2 years ago

@KostiantynYarovyiBf I will push up a change later today, then you can use the commit hash for your build. Thanks.

zzhiyuan commented 2 years ago

@KostiantynYarovyiBf Hi, can you try using the latest master (hash d82bb209f068d7d5e97cfc871a58c739d06aedf1) to build? "debian/rules" should now have the correct logic to update linux version dependency to $(KVERSION_SHORT)-unsigned.

Thanks

KostiantynYarovyiBf commented 2 years ago

@KostiantynYarovyiBf Hi, can you try using the latest master (hash d82bb20) to build? "debian/rules" should now have the correct logic to update linux version dependency to $(KVERSION_SHORT)-unsigned.

Thanks

Fix works, thanks.