Skip to content

[BMC] Configure features and services for SONiC BMC#27726

Draft
roger-nexthop wants to merge 1 commit into
sonic-net:masterfrom
nexthop-ai:roger.sonic-bmc-feature-enablement
Draft

[BMC] Configure features and services for SONiC BMC#27726
roger-nexthop wants to merge 1 commit into
sonic-net:masterfrom
nexthop-ai:roger.sonic-bmc-feature-enablement

Conversation

@roger-nexthop

Copy link
Copy Markdown

Why I did it

SONiC BMC only runs essential services and does not run all of the features that SONiC would on a switch. This patch configures feature enablement for the targeted set on SONiC BMC and removes feature delay that would delay bringing up services (e.g. pmon) that could delay switch CPU bringup for air cooled platforms.

Work item tracking
  • Microsoft ADO (number only):

How I did it

Updated init-cfg.json.j2 with the set of BMC features. BMC features are selectively enabled with feature delay off; rest of features that are not applicable to BMC are disabled.
Also disabled route-counter.service and route-counter.timer that are not applicable to BMC.

How to verify it

Verify feature set for BMC is only enabled with feature delay off on SONiC BMC (show feature status).

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

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

Tested branch (Please provide the tested image version)

master

Description for the changelog

Enabled selected services/features for SONiC BMC

Link to config_db schema for YANG module changes

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

Signed-off-by: Roger Liao <roger@nexthop.ai>
@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).

@judyjoseph

Copy link
Copy Markdown
Contributor

@oleksandrivantsiv @YairRaviv f.y.i

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