Skip to content

Update dependencies for Linux kernel 5.10.46#32

Closed
KostiantynYarovyiBf wants to merge 1 commit intoaristanetworks:masterfrom
KostiantynYarovyiBf:bullseye-upgrade-barefoot-bfn-moduls-arista
Closed

Update dependencies for Linux kernel 5.10.46#32
KostiantynYarovyiBf wants to merge 1 commit intoaristanetworks:masterfrom
KostiantynYarovyiBf:bullseye-upgrade-barefoot-bfn-moduls-arista

Conversation

@KostiantynYarovyiBf
Copy link

@KostiantynYarovyiBf KostiantynYarovyiBf commented Oct 27, 2021

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:
sonic-net/sonic-buildimage#8191

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106

Description for the changelog

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

@KostiantynYarovyiBf KostiantynYarovyiBf marked this pull request as ready for review October 27, 2021 09:59
@Staphylo
Copy link
Member

@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
Copy link
Author

KostiantynYarovyiBf commented Oct 27, 2021

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

@zzhiyuan
Copy link

@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 sonic-net/sonic-buildimage#8890 to merge in before committing to another submodule update for arista (which will include the version change).
Thanks

@zzhiyuan
Copy link

@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
Copy link
Author

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

@zzhiyuan
Copy link

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

@zzhiyuan
Copy link

@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

@KostiantynYarovyiBf
Copy link
Author

@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.

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