Changing network mode to bridge #4
Open
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
pulling from master
Master sync
Merge log plugin
Log messages
Use safe method of querying dictionary
* Added a check to ensure ops-switchd has become active. This will ensure that the OpenSwitch docker image is in a usable state. Signed-off-by: Vasanth Viswanathan <[email protected]>
chg: dev: Add check for ops-switchd to be active
* In legacy frameworks, there is a wait of 60 seconds for ops-switchd to become active. Making this change to stay in sync and ensure we dont timeout sooner. Signed-off-by: Vasanth Viswanathan <[email protected]>
chg: dev: Increase ops-switchd active timeout
chg: dev: Improving logging.
* Modified existing check for cur_hw from cur_cfg_is_set to cur_hw_is_set * Added a new check for cur_cfg column of System table * Moved the check for cur_hw and cur_cfg ahead of ops-switchd is active check. This change is because during OPS boot, all daemons including ops-switchd will only daemonize and become active after cur_cfg is set to 1. So moved the checks to maintain logical flow of events during system boot up. Signed-off-by: Vasanth Viswanathan <[email protected]>
chg: dev: Modifying boot time checks
fix: dev: Adding handler for existing files.
Restd start
chg: dev: adding support for p4simulator images
and adding checks for p4 simulator
chg: dev: Collecting docker coredumps on teardown and adding checks for p4 simulator
This is necessary to make shells register with the node and get information necessary to log properly in the pexpect file.
Operator prompt
fix: dev: Parametrize setup script path.
fix: dev: Refactoring to remove race conditions.
This reverts commit 53bf52b.
Modified P4 boot logic to accept port names in any format when mapping them to interface labels
This mount was used to gather logs from containers and log them in the execution machine, it was an old practice from first topology days and is not required anymore. The removal is mainly because there are some vtysh commands that extract data from local log files, so the logs need to be on the container instead of the execution machine.
Remove switch container volume mount dev/log
fix: dev: Add openswitch_setup script to package.
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.
These changes bring the openswitch in bridge mode to use the default eth0 interface as the oobm interface. This is the method used by the previous framework asn should be a temporary fix while we implement the docker networking patch.