Skip to content

[202205][db_migrator] Add migration of FLEX_COUNTER_DELAY_STATUS during 1911->2205 upgrade + fast-reboot. Add UT.#2870

Merged
yxieca merged 1 commit intosonic-net:202205from
vadymhlushko-mlnx:202205-flex-counter-delay-status-migration
Jun 23, 2023
Merged

[202205][db_migrator] Add migration of FLEX_COUNTER_DELAY_STATUS during 1911->2205 upgrade + fast-reboot. Add UT.#2870
yxieca merged 1 commit intosonic-net:202205from
vadymhlushko-mlnx:202205-flex-counter-delay-status-migration

Conversation

@vadymhlushko-mlnx
Copy link
Contributor

What I did

Add migration of FLEX_COUNTER_DELAY_STATUS attribute of config_db FLEX_COUNTER_TABLE during the SONiC to SONiC upgrade + fast-reboot from older versions 201911 -> 202205.

This change is required for the fast-reboot procedure because without it the counters will be created during the init flow which will waste a lot of resources and cause data plane degradation of more than 30 seconds.

How I did it

Modify the db_migrator.py.

How to verify it

Add UT.

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)

…ng 1911->2211 upgrade + fast-reboot. Add UT.

Signed-off-by: vadymhlushko-mlnx <vadymh@nvidia.com>
@vadymhlushko-mlnx vadymhlushko-mlnx force-pushed the 202205-flex-counter-delay-status-migration branch from 7390758 to 240ec23 Compare June 13, 2023 13:52
@liat-grozovik
Copy link
Collaborator

@vaibhavhd @arfeigin could you please review the changes?

@vaibhavhd
Copy link
Contributor

I don't have permissions to merge to this branch.

@liat-grozovik
Copy link
Collaborator

@yxieca can you please merge this one?

@yxieca yxieca merged commit 1787410 into sonic-net:202205 Jun 23, 2023
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.

4 participants