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

New deploy-mg causes TSA to be in config_db (and golden), causing test errors when startup_tsa_tsb is disabled #17246

Open
Javier-Tan opened this issue Feb 28, 2025 · 2 comments · May be fixed by #17247
Assignees

Comments

@Javier-Tan
Copy link
Contributor

Javier-Tan commented Feb 28, 2025

Issue Type

Enhancement

Description

Since TSA/TSB was changed to start on swss startup (sonic-net/sonic-buildimage#21587), deploy-mg causes tsa_enabled: true in config_db. This could affect tests

If tsa_enabled is not explicitly set to false and saved to config before running tests, tsa_enabled will remain in the config, which is not desired (and deploy-mg is a common setup step before running nightly tests). Additionally pretest will save this to golden config.

In the case where pretest is run, startup_tsa_tsb service will be disabled, meaning any config reload will cause tsa_enabled to be on indefinitely causing test failures

This also causes "Core dump or config check failed" and triggers config reload, causing 15m per test delay

Issue Details

To recreate this issue:

Run deploy-mg on device and check tsa_enabled in config_db

Impact or Proposed Behavior

In general, tsa_enabled should be false for both config_db.json and golden_config_db.json

Importance or Severity

High

Is it platform specific

generic

Relevant log output

N/A

Output of show version

N/A

Attach files (if any)

N/A

@rlhui
Copy link

rlhui commented Mar 1, 2025

@Javier-Tan , can we please have a more descriptive title for this issue?

@Javier-Tan Javier-Tan changed the title [Bug|Enhancement|Test Gap]: New deploy-mg causes TSA to be in config_db (and golden), causing errors when startup_tsa_tsb is disabled Mar 1, 2025
@Javier-Tan
Copy link
Contributor Author

@rlhui done, sorry not sure why the title didnt save

@Javier-Tan Javier-Tan changed the title New deploy-mg causes TSA to be in config_db (and golden), causing errors when startup_tsa_tsb is disabled New deploy-mg causes TSA to be in config_db (and golden), causing test errors when startup_tsa_tsb is disabled Mar 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants