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

[Arista] Fix extraction of platform.tar.gz in secureboot #11935

Merged
merged 1 commit into from
Oct 22, 2022

Conversation

Staphylo
Copy link
Collaborator

@Staphylo Staphylo commented Sep 1, 2022

Why I did it

A recent change modified the platform folder of the image into a tarball platform.tar.gz
The boot0 script was modified for regular boot but not secureboot.
This means that the platform/firstime does not get created and rc.local therefore skips initial configuration.

How I did it

Updated the secureboot image extraction logic to properly handle platform.tar.gz

How to verify it

Boot the image on a secureboot enabled product and make sure that rc.local enters the firstime codepath.
Also verify that /host/image-XXX/platform exists.

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

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

Description for the changelog

Fix extraction of platform.tar.gz in secureboot

@Staphylo Staphylo requested a review from lguohan as a code owner September 1, 2022 13:44
@yejianquan
Copy link
Contributor

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Staphylo
Copy link
Collaborator Author

Staphylo commented Sep 5, 2022

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Staphylo Staphylo force-pushed the master-boot0-secureboot-platform branch 2 times, most recently from c4e8de2 to cc0124a Compare September 8, 2022 16:45
@Staphylo
Copy link
Collaborator Author

@prgeor

@Staphylo
Copy link
Collaborator Author

@qiluo-msft

@prgeor prgeor merged commit f39c2ad into sonic-net:master Oct 22, 2022
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.

6 participants