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] Revert bgp suppress fib pending #17660

Merged

Conversation

stepanblyschak
Copy link
Collaborator

What I did

Revert FIB suppress.

Work item tracking
  • Microsoft ADO (number only):

Why I did it

Revert FIB suppress due to bugs found.

How I verified it

Boot the image, verify FIB suppress is not enabled.

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

Link to config_db schema for YANG module changes

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

Signed-off-by: Stepan Blyschak <stepanb@nvidia.com>
Signed-off-by: Stepan Blyschak <stepanb@nvidia.com>
Signed-off-by: Stepan Blyschak <stepanb@nvidia.com>
@stepanblyschak stepanblyschak marked this pull request as ready for review January 3, 2024 13:50
@mssonicbld
Copy link
Collaborator

@yxieca PR: #17660 is conflict with MS internal repo
Please complete the following PR by pushing fix commit to sonicbld/conflict_prefix/17660-fix
https://msazure.visualstudio.com/One/_git/Networking-acs-buildimage/pullrequest/9302504
Then comment "/azpw ms_conflict" to rerun PR checker.

@yxieca
Copy link
Contributor

yxieca commented Jan 3, 2024

@stepanblyschak can you help check the test failures?

@yxieca
Copy link
Contributor

yxieca commented Jan 22, 2024

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@yxieca
Copy link
Contributor

yxieca commented Jan 22, 2024

/azpw ms_conflict

@yxieca yxieca merged commit b527372 into sonic-net:202311 Jan 22, 2024
15 of 18 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.

3 participants