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

Start startup_tsa_tsb.service only if services.conf file exists #20315

Merged
merged 1 commit into from
Nov 13, 2024

Conversation

saksarav-nokia
Copy link
Contributor

@saksarav-nokia saksarav-nokia commented Sep 20, 2024

Why I did it

Fix #20049

Currently, the startup_tsa_tsb.service is started for all the platforms and the service exists if startup-tsa-tsb.conf doesn't exist for the platform.

Work item tracking
  • Microsoft ADO (number only):

How I did it

Don't add startup_tsa_tsb.service to /etc/sonic/generated_services.conf.
Instead add it to services.conf if the platform supports startup_tsa_tsb.service.

How to verify it

1)Verified in Nokia platform that the startup_tsa_tsb.service starts only when the services.conf file exists

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

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)

arlakshm
arlakshm previously approved these changes Oct 7, 2024
Signed-off-by: saksarav <sakthivadivu.saravanaraj@nokia.com>
@lguohan lguohan merged commit 5681eb9 into sonic-net:master Nov 13, 2024
23 checks passed
mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Nov 14, 2024
…c-net#20315)

Fix sonic-net#20049

Currently, the startup_tsa_tsb.service is started for all the platforms and the service exists if startup-tsa-tsb.conf doesn't exist for the platform.

Signed-off-by: saksarav <sakthivadivu.saravanaraj@nokia.com>
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202405: #20806

mssonicbld pushed a commit that referenced this pull request Nov 15, 2024
Fix #20049

Currently, the startup_tsa_tsb.service is started for all the platforms and the service exists if startup-tsa-tsb.conf doesn't exist for the platform.

Signed-off-by: saksarav <sakthivadivu.saravanaraj@nokia.com>
aidan-gallagher pushed a commit to aidan-gallagher/sonic-buildimage that referenced this pull request Nov 16, 2024
…c-net#20315)

Fix sonic-net#20049

Currently, the startup_tsa_tsb.service is started for all the platforms and the service exists if startup-tsa-tsb.conf doesn't exist for the platform.

Signed-off-by: saksarav <sakthivadivu.saravanaraj@nokia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Leverage ssg platform services.conf support for startup_tsa_tsb service
7 participants