-
Notifications
You must be signed in to change notification settings - Fork 49
Sonic vpp docker image build support #201
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
base: master
Are you sure you want to change the base?
Conversation
|
|
|
/azp run |
|
Azure Pipelines could not run because the pipeline triggers exclude this branch/path. |
| @@ -0,0 +1,25 @@ | |||
| # Copyright 2016 Microsoft, Inc. | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Many of the .mk and .dep already exist in rules/. Can we reuse them?
|
|
||
| $(LIBSAIREDIS)_DEB_BUILD_PROFILES += syncd vs | ||
|
|
||
| # VPP uses the virtual switch syncd since it operates in software dataplane mode |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need to introduce SYNCD_VPP? can we change the caller to SYNCD_VS instead?
| include $(PLATFORM_RULES)/one-image.mk | ||
| include $(PLATFORM_RULES)/onie.mk | ||
| include $(PLATFORM_RULES)/kvm-image.mk | ||
| include $(PLATFORM_RULES)/raw-image.mk |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's the benefit to move them out of rules/?
|
/azp run |
|
Azure Pipelines could not run because the pipeline triggers exclude this branch/path. |
This pr contains all the changes required to build sonic vpp docker image.