Skip to content

Support for cavium platform: initial commit#16

Merged
qiluo-msft merged 5 commits intosonic-net:masterfrom
NStetskovych-zz:cavium_platform_supp
Oct 7, 2016
Merged

Support for cavium platform: initial commit#16
qiluo-msft merged 5 commits intosonic-net:masterfrom
NStetskovych-zz:cavium_platform_supp

Conversation

@NStetskovych-zz
Copy link
Copy Markdown

Add changes for cavium platform support

# Dbus
- name: Install dbus
become: true
apt: pkg=dbus
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Who requires dbus? Is it required for all platform?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Copy link
Copy Markdown
Contributor

@qiluo-msft qiluo-msft Oct 6, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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
~$

Copy link
Copy Markdown
Author

@NStetskovych-zz NStetskovych-zz Oct 6, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like we really don`t need it. Will remove in next commit. Thanks for pointing it out.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool! Please go ahead.

sensor_hwskus: [ "ACS-S6000", "ACS-MSN2700", "Force10-S6000" ]

cavium_hwskus: [ "AS7512", "XP-SIM" ]
## Note:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Place 3 [vendor]_hwsku together?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now it used to distinguish vendor platform i.e when: sonic_hwsku in mellanox_hwskus . Afraid to break other vendor configuration.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I mean move 3 lines of code closer. sensor_hwskus is not parallel with others.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks!

Nadiya.Stetskovych added 3 commits October 6, 2016 23:50
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
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No need to 'become'

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, removed.

changed_when: False

- name: Add mgmt ip rule if it does not exist
become: true
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No need to 'become'

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed. Looks like was needed by previous ansible version.

Signed-off-by: Nadiya.Stetskovych <Nadiya.Stetskovych@cavium.com>
@qiluo-msft qiluo-msft merged commit 49eee2d into sonic-net:master Oct 7, 2016
devin-ai-integration bot added a commit to bpar9/sonic-mgmt that referenced this pull request Jan 14, 2026
- 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>
devin-ai-integration bot added a commit to bpar9/sonic-mgmt that referenced this pull request Jan 14, 2026
- 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>
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
[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
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
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>
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
… 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)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants