Support building multi-asic component #3856
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* indicate 'virsh' should be run as sudo * add instructions to get DHCP lease for SSH connections Signed-off-by: Lawrence Lee <[email protected]> (cherry picked from commit 4afe564)
* prepare slave.mk to build multi-asic services * move systemd services into single and multi instance folders * prepare sonic_debian_extension to copy over single and multi instance services * detect single or multi-asic platform in systemd-sonic-generator and enable correct services Signed-off-by: Lawrence Lee <[email protected]> (cherry picked from commit 0fdb595)
…ulti-instance container support [build_templates]: * add network namespace creation/linking to docker_image_ctl.j2 * add support for multi-instance containers to docker_image_ctl.j2 * add support for hostnames in multi-asic containers * don't try and copy single instance version of namespace service in sonic_debian_extension [rules]: * add rule for creating network namespace docker * removes "--net=host" from DOCKER_RUN_OPTS in all other docker rules (it now lives in docker_image_ctl.j2) [build]: * retry docker build without squashing if squashing fails [platform/vs]: add network interfaces to virtual switch * add 32 more network interfaces, for a total of 64 Signed-off-by: Lawrence Lee <[email protected]> (cherry picked from commit ba7b7e0)
* add script to configure virtual switch internal topology * create service to manage topology * install topology service in sonic_debian_extension Signed-off-by: Lawrence Lee <[email protected]> (cherry picked from commit bf7b16d)
* remove extra line from slave.mk * change namespace names to 'asic#' Signed-off-by: Lawrence Lee <[email protected]> (cherry picked from commit 95bef6e)
platform specific conf file during boot up.
creation and use database docker network for multi asic platform.
lguohan
reviewed
Dec 7, 2019
| @@ -0,0 +1,17 @@ | |||
| [Unit] | |||
| Description=Internal topology service | |||
| [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] | |||
There was a problem hiding this comment.
why hard code numbers here?
There was a problem hiding this comment.
Fixed to dynamically update dependency based on number of asics
lguohan
reviewed
Dec 7, 2019
lguohan
reviewed
Dec 7, 2019
lguohan
reviewed
Dec 7, 2019
lguohan
reviewed
Dec 7, 2019
|
can you make description more informative? refer to PR that is not going to be merged is not a good idea. |
lguohan
reviewed
Dec 7, 2019
lguohan
reviewed
Dec 7, 2019
lguohan
reviewed
Dec 7, 2019
lguohan
reviewed
Dec 7, 2019
lguohan
reviewed
Dec 7, 2019
lguohan
reviewed
Dec 7, 2019
only for multi-asic vs platform
[asic.conf]: Added default asic.conf file for vs platform
docker_image_ctl.j2
|
retest this please |
|
retest vsimage please |
1 similar comment
|
retest vsimage please |
lguohan
reviewed
Jan 23, 2020
| NET="host" | ||
| else | ||
| {%- if docker_container_name == "database" %} | ||
| NET="bridge" |
There was a problem hiding this comment.
you are using tab, but other places you are using space in the same file. can you change to space and make the spacing consistent acrosss the file?
There was a problem hiding this comment.
Corrected spacing
lguohan
approved these changes
Jan 23, 2020
|
retest vsimage please |
1 similar comment
|
retest vsimage please |
abdosi
pushed a commit
that referenced
this pull request
Feb 4, 2020
- move single instance services into their own folder - generate Systemd templates for any multi-instance service files in slave.mk - detect single or multi-instance platform in systemd-sonic-generator based on asic.conf platform specific file. - update container hostname after creation instead of during creation (docker_image_ctl) - run Docker containers in a network namespace if specified - add a service to create a simulated multi-ASIC topology on the virtual switch platform Signed-off-by: Lawrence Lee <[email protected]> Signed-off-by: Suvarna Meenakshi <[email protected]>
| if [ x"$HOSTNAME" != x"$CURRENT_HOSTNAME" ]; then | ||
| updateHostName "$HOSTNAME" | ||
| fi | ||
| postStartAction | ||
| updateHostName "$HOSTNAME" |
There was a problem hiding this comment.
Sorry for late comment.
Why it is needed to call updateHostName even though it is called in the condition right above?
| echo "Failed to docker run" >&1 | ||
| exit 4 | ||
| } | ||
|
|
||
| preStartAction | ||
| docker start {{docker_container_name}} | ||
| postStartAction | ||
| updateHostName "$HOSTNAME" |
There was a problem hiding this comment.
same question here
pphuchar
pushed a commit
to SONIC-DEV/sonic-buildimage
that referenced
this pull request
Mar 9, 2020
- move single instance services into their own folder - generate Systemd templates for any multi-instance service files in slave.mk - detect single or multi-instance platform in systemd-sonic-generator based on asic.conf platform specific file. - update container hostname after creation instead of during creation (docker_image_ctl) - run Docker containers in a network namespace if specified - add a service to create a simulated multi-ASIC topology on the virtual switch platform Signed-off-by: Lawrence Lee <[email protected]> Signed-off-by: Suvarna Meenakshi <[email protected]>
CharlieChenEC
added a commit
to CharlieChenEC/sonic-buildimage
that referenced
this pull request
Mar 20, 2020
… "--net" options in the docker create command In the file "files/build_templates/docker_image_ctl.j2", it adds the option "--net" to the docker create command through the commit "abe7ef7e2e2e1215c97cee19a83aab0b130cebe5" (sonic-net#3856). Remove the "--net" option in "docker-syncd-brcm-rpc.mk" to avoid specifying duplicated "--net" options. Signed-off-by: charlie_chen <[email protected]>
lguohan
pushed a commit
that referenced
this pull request
Mar 22, 2020
…m syncd rpc (#4296) In the file "files/build_templates/docker_image_ctl.j2", it adds the option "--net" to the docker create command through the commit "abe7ef7e2e2e1215c97cee19a83aab0b130cebe5" (#3856). Remove the "--net" option in "docker-syncd-brcm-rpc.mk" to avoid specifying duplicated "--net" options. Signed-off-by: charlie_chen <[email protected]>
tiantianlv
pushed a commit
to SONIC-DEV/sonic-buildimage
that referenced
this pull request
Apr 24, 2020
- move single instance services into their own folder - generate Systemd templates for any multi-instance service files in slave.mk - detect single or multi-instance platform in systemd-sonic-generator based on asic.conf platform specific file. - update container hostname after creation instead of during creation (docker_image_ctl) - run Docker containers in a network namespace if specified - add a service to create a simulated multi-ASIC topology on the virtual switch platform Signed-off-by: Lawrence Lee <[email protected]> Signed-off-by: Suvarna Meenakshi <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Single asic build was generated and loaded on the device to make sure the changes don't impact single asic image.
- Description for the changelog
- A picture of a cute animal (not mandatory but encouraged)