Skip to content

Override bootconf env variable from installer.conf#27740

Open
nats-nokia wants to merge 1 commit into
sonic-net:masterfrom
nats-nokia:dev_bmc_bootconf
Open

Override bootconf env variable from installer.conf#27740
nats-nokia wants to merge 1 commit into
sonic-net:masterfrom
nats-nokia:dev_bmc_bootconf

Conversation

@nats-nokia

Copy link
Copy Markdown

Why I did it

Update bootconf uboot env variable only if it is defined in device specific installer.conf. This change will help to preserve default bootconf set by uboot on platforms that do not define bootconf in installer.conf

Work item tracking
  • Microsoft ADO (number only):

How I did it

Updated platform_arm64.conf

How to verify it

Create a build sonic-aspeed-arm64.bin by including these changes and verify the bootconf is updated only if defined in installer.conf

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

  • 202305
  • 202311
  • 202405
  • 202411
  • 202505
  • 202511
  • 202605

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

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

Signed-off-by: Natarajan Subbiramani <natarajan.subbiramani@nokia.com>
@nats-nokia nats-nokia requested a review from lguohan as a code owner June 5, 2026 14:08
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@nats-nokia

Copy link
Copy Markdown
Author

@Pavan-Nokia @chander-nexthop @judyjoseph
Created the PR to skip fw_setenv bootconf if not defined in installer.conf as discussed in BMC WG meeting, .

@mssonicbld

Copy link
Copy Markdown
Collaborator

This PR has backport request for branch(es): 202605.
Added label(s) for branch(es) 202605.

---Powered by SONiC BuildBot

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.

2 participants