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

[202311] Migrate GNMI table #3138

Merged
merged 1 commit into from
Feb 1, 2024
Merged

Conversation

ganglyu
Copy link
Contributor

@ganglyu ganglyu commented Feb 1, 2024

What I did

We will use GNMI container to replace TELEMETRY container in public repo.
GNMI is using GNMI table for configuration, so we need to migrate configuration in CONFIG_DB.
For L2 mode, there's no GNMI configuration in minigraph, and there's no TELEMETRY table in CONFIG_DB, so we will not generate GNMI table.

Backport #3053

How I did it

If there's no minigraph and golden config, copy GNMI configuration from TELEMETRY table.
If there're minigraph and golden config, copy GNMI configuration from minigraph or golden config.

How to verify it

Run db migrator unit test, test with minigraph and test without minigraph.

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)

@yxieca yxieca merged commit be6224a into sonic-net:202311 Feb 1, 2024
5 checks passed
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.

2 participants