sonic-buildimage icon indicating copy to clipboard operation
sonic-buildimage copied to clipboard

Published 20 hours ago • verified publisher icon sonic-net

Made pmon feature delayed flag as jinja template

Open abdosi opened this issue 1 year ago • 4 comments

What I did: To fix: https://github.com/sonic-net/sonic-buildimage/issues/19126 Depends on: https://github.com/sonic-net/sonic-host-services/pull/135 and it's submodule update.

Why I did: PMON was delayed originally for faster uptime of data plane related services in case of fast/warm reboot. This is not needed for T2/SpineRouter . Infact we need PMON to be up asap because of:

  • pmon need to enable asap to detect ASIC's on Supervisor.
  • pmon need to enable asap for bring-up of 400G ports on LC's fast because of CMIS state machine present in PMON.

How I verify: Manual Verification and UT has been added in https://github.com/sonic-net/sonic-host-services/pull/135

abdosi avatar Jul 05 '24 01:07 abdosi

@anamehra @mlok-nokia for viz.

abdosi avatar Jul 05 '24 02:07 abdosi

/Azp run Azure.sonic-buildimage

arlakshm avatar Jul 09 '24 23:07 arlakshm

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Jul 09 '24 23:07 azure-pipelines[bot]

@arlakshm : please reapprove this.

abdosi avatar Jul 13 '24 01:07 abdosi

Fix the issue and advance sonic-host-service sub-module https://github.com/sonic-net/sonic-buildimage/pull/19551

yejianquan avatar Jul 23 '24 07:07 yejianquan

/azp run

wangxin avatar Jul 24 '24 00:07 wangxin

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Jul 24 '24 00:07 azure-pipelines[bot]

https://github.com/sonic-net/sonic-buildimage/pull/19657 merged, closing this one

yejianquan avatar Jul 26 '24 00:07 yejianquan