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] Rely on automatic flash size detection for Lodoga #8608

Merged
merged 1 commit into from
Aug 27, 2021

Conversation

Staphylo
Copy link
Collaborator

Why I did it

Lodoga actually has a 8GB storage device.
LodogaSsd variant has a 30GB SSD drive.
However, in boot0 both were mishandled and assigned 4GB for legacy reasons.

How I did it

Remove the hardcoding of the flash size and let boot0 autodetect the available space.

How to verify it

Check that /proc/cmdline and make sure that logs_inram isn't set.

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106

202012 is needed and 202106 is desirable.

Description for the changelog

Rely on automatic flash size detection for Lodoga product

@Staphylo Staphylo requested a review from lguohan as a code owner August 26, 2021 20:32
@lguohan lguohan merged commit 48ba459 into sonic-net:master Aug 27, 2021
qiluo-msft pushed a commit that referenced this pull request Aug 27, 2021
Lodoga actually has a 8GB storage device.
LodogaSsd variant has a 30GB SSD drive.
However, in boot0 both were mishandled and assigned 4GB for legacy reasons.

Remove the hardcoding of the flash size and let boot0 autodetect the available space.
judyjoseph pushed a commit that referenced this pull request Sep 2, 2021
Lodoga actually has a 8GB storage device.
LodogaSsd variant has a 30GB SSD drive.
However, in boot0 both were mishandled and assigned 4GB for legacy reasons.

Remove the hardcoding of the flash size and let boot0 autodetect the available space.
@Staphylo Staphylo deleted the master-lodoga-flashsize branch December 6, 2022 15:03
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.

5 participants