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

Add sonic-bmp submodule #20780

Merged
merged 3 commits into from
Nov 14, 2024
Merged

Add sonic-bmp submodule #20780

merged 3 commits into from
Nov 14, 2024

Conversation

FengPan-Frank
Copy link
Contributor

Why I did it

Add sonic-bmp into build-image flow.

Work item tracking
  • Microsoft ADO (number only):27588893

How I did it

update submodule file

How to verify it

build

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)

@liushilongbuaa
Copy link
Contributor

You need to clone submodule to its path locally and checkout initial commit.
Then git add its path and .gitmodules.

@FengPan-Frank FengPan-Frank added the BMP Bgp Monitor Container label Nov 13, 2024
@FengPan-Frank
Copy link
Contributor Author

/azpw ms_conflict

1 similar comment
@FengPan-Frank
Copy link
Contributor Author

/azpw ms_conflict

@FengPan-Frank
Copy link
Contributor Author

/azpw ms_conflict

@qiluo-msft qiluo-msft merged commit d4f646b into sonic-net:master Nov 14, 2024
22 checks passed
aidan-gallagher pushed a commit to aidan-gallagher/sonic-buildimage that referenced this pull request Nov 16, 2024
#### Why I did it
Add sonic-bmp into build-image flow.

#### How I did it
update submodule file

#### How to verify it
build
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BMP Bgp Monitor Container
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants