Support for cavium platform: initial commit#16
Conversation
| # Dbus | ||
| - name: Install dbus | ||
| become: true | ||
| apt: pkg=dbus |
There was a problem hiding this comment.
Who requires dbus? Is it required for all platform?
There was a problem hiding this comment.
dbus is required by systemd. Not sure about all platform, we have when condition here:
when: sonic_hwsku == "AS7512". On our pure ONIE image(not configured by ansible) dbus was required by systemctl.
There was a problem hiding this comment.
Tested on one switch (not AS7512). I don't think dbus is needed. Maybe you need 'sudo' before systemctl.
~$ sudo systemctl | head -2
UNIT LOAD ACTIVE SUB DESCRIPTION
proc-sys-fs-binfmt_misc.automount loaded active waiting Arbitrary Executable File Formats File System Automount Point
~$ dpkg -l | grep dbus
~$
There was a problem hiding this comment.
I need to test on "clean" image once I got it. But it seems DBus is widely used by systemd: systemct use DBus to communicate with systemd https://people.debian.org/~stapelberg/docs/systemd-dependencies.html
Uses socket and D-Bus activation for starting services https://wiki.debian.org/systemd#systemd_-_system_and_service_manager
It can be that other vendors got it when build image.
There was a problem hiding this comment.
Looks like we really don`t need it. Will remove in next commit. Thanks for pointing it out.
| sensor_hwskus: [ "ACS-S6000", "ACS-MSN2700", "Force10-S6000" ] | ||
|
|
||
| cavium_hwskus: [ "AS7512", "XP-SIM" ] | ||
| ## Note: |
There was a problem hiding this comment.
Place 3 [vendor]_hwsku together?
There was a problem hiding this comment.
Now it used to distinguish vendor platform i.e when: sonic_hwsku in mellanox_hwskus . Afraid to break other vendor configuration.
There was a problem hiding this comment.
I mean move 3 lines of code closer. sensor_hwskus is not parallel with others.
Signed-off-by: Nadiya.Stetskovych <Nadiya.Stetskovych@cavium.com>
Signed-off-by: Nadiya.Stetskovych <Nadiya.Stetskovych@cavium.com>
Signed-off-by: Nadiya.Stetskovych <Nadiya.Stetskovych@cavium.com>
|
|
||
| # Assign hostname | ||
| - name: Assign hostname | ||
| become: true |
| changed_when: False | ||
|
|
||
| - name: Add mgmt ip rule if it does not exist | ||
| become: true |
There was a problem hiding this comment.
Removed. Looks like was needed by previous ansible version.
Signed-off-by: Nadiya.Stetskovych <Nadiya.Stetskovych@cavium.com>
- Add Test Case #11: Local-Preference with Multiple Paths - Add Test Case #12: Local-Preference with Equal Values (Tie-Breaking) - Add Test Case #13: Local-Preference Interaction with AS-Path - Add Test Case #14: Local-Preference Interaction with MED - Add Test Case #15: Local-Preference Interaction with Weight - Add Test Case sonic-net#16: Local-Preference with Route Flapping - Extend topology support to include t0 and t0-multi-asic - Update scope section to reflect new test coverage Co-Authored-By: bhavani@cisco.com <bhavani@cisco.com>
- Add test_local_preference_multiple_paths (Test Case #11) - Add test_local_preference_equal_values_tiebreaker (Test Case #12) - Add test_local_preference_vs_aspath (Test Case #13) - Add test_local_preference_vs_med (Test Case #14) - Add test_local_preference_vs_weight (Test Case #15) - Add test_local_preference_route_flapping (Test Case sonic-net#16) - Add helper functions for multi-path testing - Update pytestmark to support t0 and t0-multi-asic topologies Co-Authored-By: bhavani@cisco.com <bhavani@cisco.com>
[sonic-linkmgrd][master] submodule update Commits added: 0c23756 Jing Zhang 2022-01-19 Linkmgrd subscribing State DB route event (sonic-net#13) 12b9951 Longxiang Lyu 2021-12-13 Add TLV support to ICMP payload (sonic-net#11) 3eedda3 Longxiang Lyu 2022-01-06 Add missing intermediate states (sonic-net#16) 8da4982 Ying Xie 2022-01-04 [linkmgrd] update README, set coding style guidance (sonic-net#15) a897cf8 Longxiang Lyu 2021-12-13 Improve PR template (sonic-net#16) 6fec701 Jing Zhang 2021-12-06 Add pull request template for linkmgrd repo (sonic-net#9) signed-off-by: Jing Zhang zhangjing@microsoft.com
Signed-off-by: Vivek Reddy <vkarri@nvidia.com> [Nvidia] Enable iproute2 & fix mft build (sonic-net#16) * Enable iproute2 as the SDK is also built Signed-off-by: Vivek Reddy <vkarri@nvidia.com> * [Nvidia] Dont use mkbmdeb method of dkms to build the package Signed-off-by: Vivek Reddy <vkarri@nvidia.com> * Added linux image to the Depends section of mft Signed-off-by: Vivek Reddy <vkarri@nvidia.com> [Nvidia] [Bookworm] Separate KERNEL_MFT into a new target (sonic-net#16782) * [Nvidia] Seperate KERNEL_MFT into a new target because of kernel header dependency Signed-off-by: Vivek Reddy <vkarri@nvidia.com> * Update linux-kernel submodule Signed-off-by: Vivek Reddy <vkarri@nvidia.com> * Fix paralell build problem Signed-off-by: Vivek Reddy <vkarri@nvidia.com> --------- Signed-off-by: Vivek Reddy <vkarri@nvidia.com>
… sensor errors (#25276) <!-- Please make sure you've read and understood our contributing guidelines: https://github.com/Azure/SONiC/blob/gh-pages/CONTRIBUTING.md failure_prs.log Make sure all your commits include a signature generated with `git commit -s` ** If this is a bug fix, make sure your description includes "fixes #xxxx", or "closes #xxxx" or "resolves #xxxx" Please provide the following information: --> #### Why I did it Fix transient errors during bfb install on smartswitch platform. ``` ERR pmon#sensord: Error getting sensor data: mp2975/sonic-net#16: Kernel interface error ``` ##### Work item tracking - Microsoft ADO **(number only)**: #### How I did it Use pre-shutdown procedures before doing a reboot #### How to verify it Installation of bfb image on dpu from switch shouldn't cause errors <!-- If PR needs to be backported, then the PR must be tested against the base branch and the earliest backport release branch and provide tested image version on these two branches. For example, if the PR is requested for master, 202211 and 202012, then the requester needs to provide test results on master and 202012. --> #### Which release branch to backport (provide reason below if selected) <!-- - Note we only backport fixes to a release branch, *not* features! - Please also provide a reason for the backporting below. - e.g. - [x] 202006 --> - [ ] 202205 - [ ] 202211 - [ ] 202305 - [ ] 202311 - [ ] 202405 - [ ] 202411 - [ ] 202505 #### Tested branch (Please provide the tested image version) <!-- - Please provide tested image version - e.g. - [x] 20201231.100 --> - [ ] <!-- image version 1 --> - [ ] <!-- image version 2 --> #### Description for the changelog <!-- Write a short (one line) summary that describes the changes in this pull request for inclusion in the changelog: --> <!-- Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU. --> #### Link to config_db schema for YANG module changes <!-- Provide a link to config_db schema for the table for which YANG model is defined Link should point to correct section on https://github.com/Azure/sonic-buildimage/blob/master/src/sonic-yang-models/doc/Configuration.md --> #### A picture of a cute animal (not mandatory but encouraged)
Add changes for cavium platform support