Skip to content

[Quagga] Remove Quagga from SONiC#10384

Merged
prsunny merged 1 commit intosonic-net:202012from
liorghub:remove_quagga
Mar 30, 2022
Merged

[Quagga] Remove Quagga from SONiC#10384
prsunny merged 1 commit intosonic-net:202012from
liorghub:remove_quagga

Conversation

@liorghub
Copy link
Contributor

@liorghub liorghub commented Mar 29, 2022

Why I did it

Quagga is no longer being used in SONiC.

Cherry-pick from master PR #7898

How I did it

Remove Quagga git submodule and related code.

How to verify it

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111

Description for the changelog

Link to config_db schema for YANG module changes

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

@liorghub liorghub changed the title Remove quagga from SONiC [Quagga] Remove Quagga from SONiC Mar 29, 2022
@lgtm-com
Copy link

lgtm-com bot commented Mar 29, 2022

This pull request fixes 2 alerts when merging 89d0f08 into 308ecbb - view on LGTM.com

fixed alerts:

  • 2 for Unused import

@liat-grozovik
Copy link
Collaborator

@liorghub can you please refer in the PR description to the original PR from master used as a base for this PR?

@liat-grozovik
Copy link
Collaborator

@prsunny , @xumia could you please check the build failures?

It seems that some has a infra issue. For example:
"System.UnauthorizedAccessException: Access to the path '/agent/_work/1/s/platform/vs/tests/log/test_default_route/log/wtmp' is denied."

@xumia
Copy link
Collaborator

xumia commented Mar 30, 2022

@prsunny , @xumia could you please check the build failures?

It seems that some has a infra issue. For example: "System.UnauthorizedAccessException: Access to the path '/agent/_work/1/s/platform/vs/tests/log/test_default_route/log/wtmp' is denied."

It should be an azp agent issue, please retry it.

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