Skip to content

[action] [PR:14077] Remove temp golden config after reload config with golden config.#14752

Merged
mssonicbld merged 1 commit intosonic-net:202311from
mssonicbld:cherry/202311/14077
Sep 26, 2024
Merged

[action] [PR:14077] Remove temp golden config after reload config with golden config.#14752
mssonicbld merged 1 commit intosonic-net:202311from
mssonicbld:cherry/202311/14077

Conversation

@mssonicbld
Copy link
Collaborator

Remove temp golden config after reload config with golden config.

Why I did it

test_fallback_to_local_authorization_with_config_reload test case will reload config with AAA enabled.
After this test case, the golden config file will left on device, when some other code reload config, device will enable AAA and block other login again.

Work item tracking
  • Microsoft ADO: 28871521

How I did it

Remove temp golden config after reload config with golden config.

How to verify it

Pass all test case.

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

Remove temp golden config after reload config with golden config.

Link to config_db schema for YANG module changes

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

…nic-net#14077)

Remove temp golden config after reload config with golden config.

Why I did it
test_fallback_to_local_authorization_with_config_reload test case will reload config with AAA enabled.
After this test case, the golden config file will left on device, when some other code reload config, device will enable AAA and block other login again.

Work item tracking
Microsoft ADO: 28871521
How I did it
Remove temp golden config after reload config with golden config.

How to verify it
Pass all test case.
@mssonicbld
Copy link
Collaborator Author

Original PR: #14077

@mssonicbld mssonicbld merged commit 49b3706 into sonic-net:202311 Sep 26, 2024
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