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

[202205][Arista] Fix cmdline generation during warm-reboot from 201811/201911 #12371

Merged
merged 2 commits into from
Oct 27, 2022

Conversation

Staphylo
Copy link
Collaborator

Why I did it

The cmdline generation has changed between 201911 and 202012 for Arista products.
Doing a fast/warm reboot from 201811 or 201911 to a newer release would lead to cmdline issues.

How I did it

Cherry-picked the fix #11161 that was applied to 202012 onto 202205

How to verify it

Perform a warm-reboot from 201811/201911 to 202205 to validate.

Description for the changelog

Fix cmdline generation during warm-reboot from 201811/201911

@Staphylo Staphylo requested a review from lguohan as a code owner October 12, 2022 12:33
@Staphylo Staphylo marked this pull request as draft October 13, 2022 13:56
…1/201911 (sonic-net#11161)

Issue fixed: when performing a warm-reboot or fast-reboot from 201811 or 201911 to 202012 the kernel command line contains duplicate information. This issue is related to a change that was made to make 202012 boot0 file more futureproof.
A cold reboot brings everything back into a clean slate though not always desirable.

Changes done:
Added some logic to properly detect the end of the Aboot cmdline when cmdline-aboot-end delimiter is not set (clean case)
Added some logic to regenerate the Aboot cmdline when cmdline-aboot-end is set but duplicate parameters exists before (dirty case). Reorganized some code to handle duplicate parameter handling in the allowlist.
@Staphylo Staphylo force-pushed the 202205-boot0-fix-old-cmdline branch from 32fa9b2 to 9ffabeb Compare October 13, 2022 15:30
@Staphylo Staphylo marked this pull request as ready for review October 13, 2022 15:32
@prgeor
Copy link
Contributor

prgeor commented Oct 26, 2022

@yxieca could you merge? failure doesn't seem to be related to this pr

@yxieca
Copy link
Contributor

yxieca commented Oct 26, 2022

/azp run

@azure-pipelines
Copy link

You have several pipelines (over 10) configured to build pull requests in this repository. Specify which pipelines you would like to run by using /azp run [pipelines] command. You can specify multiple pipelines using a comma separated list.

@yxieca
Copy link
Contributor

yxieca commented Oct 26, 2022

/azp run zure.sonic-buildimage

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@yxieca
Copy link
Contributor

yxieca commented Oct 26, 2022

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@yxieca yxieca merged commit 8e44292 into sonic-net:202205 Oct 27, 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.

3 participants