[warm/fast reboot] refactoring advanced-reboot script#769
Merged
yxieca merged 18 commits intosonic-net:masterfrom Jan 10, 2019
Merged
[warm/fast reboot] refactoring advanced-reboot script#769yxieca merged 18 commits intosonic-net:masterfrom
yxieca merged 18 commits intosonic-net:masterfrom
Conversation
The arp_responder might be enabled by another test with different configuration but left running. To guarantee the latest configuration is used, always restart arp_responder. Signed-off-by: Ying Xie <ying.xie@microsoft.com>
Signed-off-by: Ying Xie <ying.xie@microsoft.com>
Create the pool earlier so other code could share it. Signed-off-by: Ying Xie <ying.xie@microsoft.com>
Signed-off-by: Ying Xie <ying.xie@microsoft.com>
When some part of the test doesn't work properly, these variables got picked up to generated exception because they were not initialized. Initializing them just so that the failure won't be that they are not initialized. The script still fails and hopefully that would be less confusing. Signed-off-by: Ying Xie <ying.xie@microsoft.com>
Signed-off-by: Ying Xie <ying.xie@microsoft.com>
…ructure Signed-off-by: Ying Xie <ying.xie@microsoft.com>
Signed-off-by: Ying Xie <ying.xie@microsoft.com>
Signed-off-by: Ying Xie <ying.xie@microsoft.com>
Signed-off-by: Ying Xie <ying.xie@microsoft.com>
Signed-off-by: Ying Xie <ying.xie@microsoft.com>
We have up to 3 concurrent tasks so far. Reduce the pool size to 5 to reduce resource footprint and leave some space for more parallel tasks in the future. Signed-off-by: Ying Xie <ying.xie@microsoft.com>
CPU -> control plane ASIC -> data plane
Signed-off-by: Ying Xie <ying.xie@microsoft.com>
…rency Signed-off-by: Ying Xie <ying.xie@microsoft.com>
Signed-off-by: Ying Xie <ying.xie@microsoft.com>
Signed-off-by: Ying Xie <ying.xie@microsoft.com>
pavel-shirshov
approved these changes
Jan 10, 2019
kazinator-arista
pushed a commit
to kazinator-arista/sonic-mgmt
that referenced
this pull request
Mar 4, 2026
[ci]: download artifacts from master branch (sonic-net#768) Do not create fabric port if mapping is not available (sonic-net#769) [syncd] Comparison logic log also current attr value on set operation (sonic-net#763) Add fabric port test to vslib (sonic-net#737) [ci]: use sonicbld pool (sonic-net#766) [tests] Remove exit command blocking all tests to run (sonic-net#765) [vslib]: adapt macsec sai 1.7.1 (sonic-net#755) Add support for SAI_SWITCH_ATTR_AVAILABLE_IPMC_ENTRY needed by CRM (sonic-net#756) Signed-off-by: Danny Allen <daall@microsoft.com>
kazinator-arista
pushed a commit
to kazinator-arista/sonic-mgmt
that referenced
this pull request
Mar 4, 2026
sonic-swss * 32d3297 2019-01-24 | use sai_stat_id_t for new SAI header file (sonic-net#769) sonic-sairedis * fdfcefd 2019-04-24 | [SAI] Advance submodule to v1.4.1 (sonic-net#450) (HEAD -> 201811, origin/201811) [Marian Pritsak] * 006df41 2019-02-27 | [vslib]: Add support of bitmap VNET (bmtor) SAI API (sonic-net#424) [Volodymyr Samotiy] * ec5e187 2019-04-23 | [SAI]: Move SAI pointer to v1.4 (sonic-net#447) [Shuotian Cheng] * 6b9c32e 2019-03-04 | [SAI] Update SAI pointer (sonic-net#431) [Stepan Blyshchak] * f078998 2019-02-22 | Fix compilation issues in stretch docker with gcc-6.3 (sonic-net#426) [Stepan Blyshchak] * 0540e59 2019-01-31 | Update SAI pointer (sonic-net#414) [Marian Pritsak] * 9084920 2019-01-18 | Update SAI pointer to latest master (sonic-net#408) [Marian Pritsak] * fbfe48b 2019-06-24 | Revert "[SAI header] upgrade SAI header to version v1.3.7 (sonic-net#445)" [Guohan Lu] Signed-off-by: Guohan Lu <gulv@microsoft.com>
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Approach
How did you do it?
At high level, this change addresses a few issues I encountered when running fast-reboot test. As it stands now, the script hasn't been fully adapted to support warm-reboot yet.
Issues addressed:
Approach taking:
How did you verify/test it?