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

[conflict][config] no op if Golden Config is invalid #3503

Closed
wants to merge 2 commits into from

Conversation

wen587
Copy link
Contributor

@wen587 wen587 commented Aug 27, 2024

What I did

How I did it

How to verify it

Previous command output (if the output of a command-line utility has changed)

New command output (if the output of a command-line utility has changed)

### What I did
Fix  sonic-net#3164
Check Golden Config earlier before service is down.
#### How I did it
Move the check at the begining
#### How to verify it
Unit test
ADO: 27941719

What I did
Improve Golden Config workflow and make sure no op if invalid config detected

How I did it
Add table dependency check right after Golden Config path is enabled

How to verify it
Unit test
@wen587 wen587 closed this Aug 28, 2024
@wen587 wen587 deleted the 3367_cflt branch August 28, 2024 06:07
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.

1 participant