[sairedis]: force to disable parallel build for sairedis package#921
Closed
lguohan wants to merge 1 commit intosonic-net:masterfrom
Closed
[sairedis]: force to disable parallel build for sairedis package#921lguohan wants to merge 1 commit intosonic-net:masterfrom
lguohan wants to merge 1 commit intosonic-net:masterfrom
Conversation
Collaborator
|
@lguohan When do you have problems with a parallel build for sairedis? Is it related to some platform? Can you share log with failure? |
Collaborator
Author
|
when doing parellel, it tries to build syncd, syncd-rpc target at the same time, then there is race condition. It fails at the different places each time. You can see following if you use dpkg-buildpackage -us -uc -b -j4 |
Collaborator
|
https://github.com/Azure/sonic-sairedis/blob/master/debian/rules#L25 Can you please try to change to This will make builds in sairedis sequential (first binary-syncd, then binary-syncd-rpc) and we won't have such conflicts |
Contributor
|
@marian-pritsak this seems to change the dependency relationship. but |
Contributor
|
We could just make binary-syncd an order-only prerequisite of binary-syncd-rpc, which will ensure binary-syncd is always built first, but does not imply a dependency between the two: |
Collaborator
Author
|
solve in sonic-net/sonic-sairedis#225 |
yxieca
added a commit
to yxieca/sonic-buildimage
that referenced
this pull request
Jun 4, 2019
Submodule src/sonic-utilities 6b4d1a0..46b5aa8: > [show ip interface] Add support for 'alias' interface naming mode (sonic-net#486) Submodule src/sonic-swss 9c4ae18..a637562: > Suppress storm detect counter increment for ongoing pfc storm case during a warm reboot (sonic-net#869) > Remove *_LEFT fields to allow PFC watchdog to enter fresh into the (sonic-net#897) > Set LAG mtu value based on kernel netlink msg (sonic-net#922) > [warm restart assist] assume vector values could be reordered (sonic-net#921) Signed-off-by: Ying Xie <ying.xie@microsoft.com>
yxieca
added a commit
that referenced
this pull request
Jun 4, 2019
Submodule src/sonic-utilities 6b4d1a0..46b5aa8: > [show ip interface] Add support for 'alias' interface naming mode (#486) Submodule src/sonic-swss 9c4ae18..a637562: > Suppress storm detect counter increment for ongoing pfc storm case during a warm reboot (#869) > Remove *_LEFT fields to allow PFC watchdog to enter fresh into the (#897) > Set LAG mtu value based on kernel netlink msg (#922) > [warm restart assist] assume vector values could be reordered (#921) Signed-off-by: Ying Xie <ying.xie@microsoft.com>
seiferteric
pushed a commit
to project-arlo/sonic-buildimage
that referenced
this pull request
Oct 14, 2019
* Update src/sonic-swss from branch 'broadcom_sonic'
to cd2a2e0504412254d4d44f5f97946921dc246cc6
- Merge 201904 branch to broadcom_sonic branch on Mon Jul 1 13:57:57 PDT 2019
Change-Id: I77bef1ba390171f204e27387bee0226ddab38971
- [debian]: Do not build test when building with real SAI (sonic-net#932)
- swss-orchagent: add new orch for vnet routes/tunnel routes tables in CONFIG_DB (sonic-net#907)
* Vnet route persistence
Signed-off-by: weixi.chen@microsoft.com
- pkill -9 zebra for frr warm restart VS test fix (sonic-net#927)
Signed-off-by: Jipan Yang <jipan.yang@alibaba-inc.com>
- [vnetorch]: Fix tunnel route removal flow for bitmap VNET (sonic-net#912)
Signed-off-by: Volodymyr Samotiy <volodymyrs@mellanox.com>
- [orchagent]: Added support of PFC WD for BFN platform (sonic-net#823)
* [orchagent]: Added support of PFC WD for BFN platform
Signed-off-by: Vitaliy Senchyshyn <vsenchyshyn@barefootnetworks.com>
* Fixed review comments
Signed-off-by: Vitaliy Senchyshyn <vsenchyshyn@barefootnetworks.com>
* Use PFC WD ACL handler for BFN platform
- [aclorch] unittest by gtest (sonic-net#924)
- [orchagent]: Remove try/catch for correct coredump file (sonic-net#790)
Signed-off-by: Jipan Yang <jipan.yang@alibaba-inc.com>
- Set LAG mtu value based on kernel netlink msg (sonic-net#922)
* Update mtu value based on kernel netlink msg
* Push the calculated MTU size into the fvVector
- Address review comment: remove data member m_entriesCreated, which is introduced for dependancy resolution purpose. (sonic-net#839)
Signed-off-by: Wenda Ni <wenni@microsoft.com>
- [vxlanorch] Ambiguous return code for removeNextHopTunnel (sonic-net#880)
Change to return false when isTunnelExists is fail
- [vnet]: Extend Bitmap VNET test with "remove" flows (sonic-net#900)
Signed-off-by: Volodymyr Samotiy <volodymyrs@mellanox.com>
- add dynamic transceiver tuning support (sonic-net#821)
- Remove *_LEFT fields to allow PFC watchdog to enter fresh into the (sonic-net#897)
operational/storm state
Signed-off-by: Wenda Ni <wenni@microsoft.com>
- Fix vlan incremental config and add vs test cases (sonic-net#799)
* Fix vlan incremental config and add vs test cases
Signed-off-by: Jipan Yang <jipan.yang@alibaba-inc.com>
- Suppress storm detect counter increment for ongoing pfc storm case during a warm reboot (sonic-net#869)
* Suppress storm detect counter increment for ongoing pfc storm case during
a warm reboot
Signed-off-by: Wenda Ni <wenni@microsoft.com>
* Comment touch-up
Signed-off-by: Wenda Ni <wenni@microsoft.com>
- [warm restart assist] assume vector values could be reordered (sonic-net#921)
When comparing 2 vectors, assume their elements could be re-ordered.
Signed-off-by: Ying Xie <ying.xie@microsoft.com>
- [test]: Mark some VLAN tests as Stretch only (sonic-net#903)
Signed-off-by: Shu0T1an ChenG <shuche@microsoft.com>
- [aclorch]: Add MIRROR_DSCP table type (sonic-net#906)
Add MIRROR_DSCP table to support creating an ACL mirro table
that only matches DSCP value/mask.
Signed-off-by: Shu0T1an ChenG <shuche@microsoft.com>
- [debian] increment debian compatibility to 10 to enable parallel package build (sonic-net#911)
From debhelper man pages:
"If neither option is specified, debhelper currently defaults to
--parallel in compat 10 (or later) and --no-parallel otherwise."
Signed-off-by: Stepan Blyschak <stepanb@mellanox.com>
- [test]: Skip tests under investigation (sonic-net#919)
- [vstest]: Update the mirror session state table name (sonic-net#917)
Due to the change
c033b23
Fix MIRROR_SESSION table macro name (sonic-net#802)
Signed-off-by: Shu0T1an ChenG <shuche@microsoft.com>
- Ignore neighbor entry with BCAST MAC, check SAI status exists (sonic-net#914)
* Ignore neighbor entry with BCAST MAC, check SAI status exists
* Addressed review comment
- Fix MIRROR_SESSION table macro name (sonic-net#802)
Signed-off-by: Jipan Yang <jipan.yang@alibaba-inc.com>
- [policerorch]: Add PolicerOrch to bundle with mirror session (sonic-net#889)
Now that we could create a policer for the mirror session to
throttle the mirroring traffic.
configuration:
POLICER|NAME:
meter_type:packets|bytes
mode:sr_tcm|tr_tcm|storm_control
cir|DIGITS
cbs|DIGITS
pir|DIGITS
pbs|DIGITS
corlor_source:aware|blind
red_action:drop
yellow_action:drop
green_action:drop
MIRROR_SESSION|NAME:
policer:policer_name
Signed-off-by: Shu0T1an ChenG <shuche@microsoft.com>
madhanmellanox
pushed a commit
to madhanmellanox/sonic-buildimage
that referenced
this pull request
Mar 23, 2020
…net#921) When comparing 2 vectors, assume their elements could be re-ordered. Signed-off-by: Ying Xie <ying.xie@microsoft.com>
patrickmacarthur
pushed a commit
to patrickmacarthur/sonic-buildimage
that referenced
this pull request
Aug 6, 2025
```<br>* 9f3b25a - (HEAD -> 202405) Merge branch '202405' of https://github.com/sonic-net/sonic-buildimage into 202405 (2025-03-18) [Sonic Automation] * 66c15ad - (head/202405) Build fix for ptf-py3 issue on bullseye (sonic-net#22043) (2025-03-14) [mssonicbld] * aa21c6b - sonicbuildimage: Update openssh to openssh_9.2p1-2+deb12u5 (sonic-net#21960) (2025-03-08) [mssonicbld] * 900757d - Add mgmt_type into device metadata YANG model. (sonic-net#21944) (2025-03-06) [mssonicbld] * c9495b0 - [Arista] Change port layout for Arista-7050CX3-32S-C28S4 (sonic-net#21945) (2025-03-06) [mssonicbld] * 04f0e2f - [Nokia-7220 IXR] Update NOKIA 7220 H4, H4-32D, H5-64D platforms (sonic-net#21937) (2025-03-06) [mssonicbld] * eafef39 - [submodule] Update submodule sonic-platform-common to the latest HEAD automatically (sonic-net#21908) (2025-03-04) [mssonicbld] * 932373e - Use pzstd to compress the Docker in SWI slim images (sonic-net#21869) (2025-02-26) [mssonicbld] * f3ddb66 - [Mellanox] Update SDK/FW Version to 4.7.2202/2014.2202 (sonic-net#21761) (2025-02-24) [DavidZagury]<br>```
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
5 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.
sairedis package cannot be built in parellel