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

Voq HLD update for build error fix #689

Merged
merged 74 commits into from
Nov 18, 2020
Merged

Conversation

vganesan-nokia
Copy link
Contributor

No description provided.

skeesara-nokia and others added 30 commits June 30, 2020 05:15
Updated text reflects discussion on the 07/23/2020 call on Option-2. Additional discussion still pending
Updated for changes in database object names and example configurations
Updated text in "About this manual", "scope"
Removed reference architecture section
Added reference to VOQ Architecture document from Karthik and Eswaran
Additional updates are still pending. Will be completed in the next couple of hours.
Additional updates pending
Updates pending
skeesara-nokia and others added 25 commits September 8, 2020 21:53
This image is not used anymore
This image is not used anymore
This image is not used anymore
This image is not used anymore
This image is not used anymore
This image is not used anymore
This image is not used anymore
This image is not used anymore
Name changes from global_app_db to chassis_app_db
global_app_db name change to chassis_app_db
global_app_db name change to chassis_app_db
global_app_db name change to chassis_app_db
Reversed the order of section 2.2 and 2.3 compared to the previous version.
Fix for build error
@rlhui
Copy link
Contributor

rlhui commented Oct 24, 2020

@vganesan-nokia - wondering why are there so many commits in this PR. Maybe you need to rebase/merge from master branch, or have a fresh branch for this minor change? Thanks.

@vganesan-nokia
Copy link
Contributor Author

vganesan-nokia commented Oct 26, 2020

@vganesan-nokia - wondering why are there so many commits in this PR. Maybe you need to rebase/merge from master branch, or have a fresh branch for this minor change? Thanks.

Raised another PR #694 from a fresh branch that has only this change.

@lguohan lguohan merged commit b8b25fb into sonic-net:master Nov 18, 2020
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.

6 participants