Support for platforms based on Barefoot Networks' device by mkbalani · Pull Request #1796 · sonic-net/sonic-buildimage

mkbalani · 2018-06-18T02:11:37Z

**- Incorporate support for platforms based on Barefoot Networks' device **

**- Added port config files and related scripts for the platforms **

**- Verified using SONiC tests (requires the corresponding changes to sairedis and swss) **

Signed-off-by: Wade He <chihen.he@gmail.com>

1. Update sensors.conf. 2. Fixed IO expander init. 3. Fixed PSU EEPROM. 4. Fixed MB EEPROM. 5. Add fancontrol and fan init. 6. Add SYS LED control (sys, fan, fan tray). 7. 2.5V compute and setup max and min. 8. Fixed typo MB eeprom delete address. 9. Remove coretemp to BMC. 10. Add active CPLD. 11. Modify SFP+ GPIO slave address. 12. Modify tmp75 Near Port 32 slave address. Signed-off-by: Wade He <chihen.he@gmail.com>

Signed-off-by: Wade He <chihen.he@gmail.com>

1. Fixed ignore sensors Vbat. 2. Reorg hwmon order. Signed-off-by: Wade He <chihen.he@gmail.com>

Signed-off-by: Wade He <chihen.he@gmail.com>

…age into bf-master

* Fixed blink LED mask set error. Signed-off-by: Wade He <chihen.he@gmail.com>

Address comments

mkbalani · 2018-07-16T01:15:47Z

@lguohan It seems the vs build job is still somehow using an old refpoint for swss. From the PR diff here swss-ref
I can see that the refpoint is correctly dcf6c, however the jenkins build shows below error > assert len(keys) == 1 E AssertionError: assert 3 == 1 E + where 3 = len(('oid:0x700000000058a', 'oid:0x700000000058f', 'oid:0x700000000058e')) which should not be seen with this refpoint

lguohan · 2018-07-16T05:55:01Z

retest this please

lguohan · 2018-07-16T05:56:21Z

...m/barefoot/sonic-platform-modules-ingrasys/debian/sonic-platform-ingrasys-s9180-32x.postinst

@@ -0,0 +1,57 @@
+# Automatically added by dh_systemd_enable


i think this is automatically added by debian, remove from the check-in.

removed file

BFN build break after removing this file. Adding it back for now. We shall have ingrasys folk review them and update accordingly

lguohan · 2018-07-16T05:56:36Z

...rm/barefoot/sonic-platform-modules-ingrasys/debian/sonic-platform-ingrasys-s9180-32x.install

@@ -0,0 +1,4 @@
+lib/systemd/


i think this is automatically added by debian, remove from the check-in.

removed file

same comment as above

lguohan · 2018-07-16T05:56:43Z

platform/barefoot/sonic-platform-modules-ingrasys/debian/sonic-platform-ingrasys-s9180-32x.dirs

@@ -0,0 +1,4 @@
+usr/sbin


i think this is automatically added by debian, remove from the check-in.

removed file

same comment as above

lguohan · 2018-07-16T05:57:37Z

...orm/barefoot/sonic-platform-modules-ingrasys/debian/sonic-platform-ingrasys-s9180-32x.postrm

@@ -0,0 +1,42 @@
+# Automatically added by dh_systemd_start


i think this is automatically added by debian, remove from the check-in.

removed file

same comment as above

lguohan · 2018-07-16T05:57:53Z

...form/barefoot/sonic-platform-modules-ingrasys/debian/sonic-platform-ingrasys-s9180-32x.prerm

@@ -0,0 +1,21 @@
+# Automatically added by dh_systemd_start


this file is automatically added by debian, remove from the check-in.

removed file

BFN build break after removing this file. Adding it back for now. We shall have ingrasys folk review them and update accordingly

lguohan · 2018-07-16T07:57:10Z

retest this please

mkbalani · 2018-07-16T15:57:15Z

retesting build and fixes

mkbalani · 2018-07-16T15:59:24Z

@lguohan BTW azure swss master is still pointing to 762e7da, rather than dcf6c, because of which vs job is failing I think.

* Remove bmc interface from interface jinja template and sample output interfaces file * Add bmc interface at boot time to network interfaces for bfn bmc based platforms

Address upstream comments for usb0 interface

Remove autogen ingrasys debian files

Revert "Remove autogen ingrasys debian files"

SWI-1509 Buffer and qos config template fix for bfn platforms

lguohan · 2018-07-23T18:11:34Z

retest this please

* Reference only ppg 3,4 in qos files as no profiles are attached to 0,1 in buffer configs

lguohan · 2018-07-24T01:22:59Z

can you fix the vs test by below patch.

diff --git a/platform/vs/tests/conftest.py b/platform/vs/tests/conftest.py
index 9256597..48ad7e1 100644
--- a/platform/vs/tests/conftest.py
+++ b/platform/vs/tests/conftest.py
@@ -54,8 +54,21 @@ class AsicDbValidator(object):
         atbl = swsscommon.Table(self.adb, "ASIC_STATE:SAI_OBJECT_TYPE_ACL_TABLE")
         keys = atbl.getKeys()
 
-        assert len(keys) == 1
-        self.default_acl_table = keys[0]
+        assert len(keys) >= 1
+        # Filter out DTel Acl tables
+        default_table_found = False
+        for k in keys:
+            if default_table_found:
+                break
+            (status, fvs) = atbl.get(k)
+            for item in fvs:
+                if item[0] == "SAI_ACL_TABLE_ATTR_ACL_BIND_POINT_TYPE_LIST":
+                    if 'SAI_ACL_BIND_POINT_TYPE_PORT' in item[1]:
+                        self.default_acl_table = k
+                        default_table_found = True
+                        break
+                    else:
+                        break
 
         atbl = swsscommon.Table(self.adb, "ASIC_STATE:SAI_OBJECT_TYPE_ACL_ENTRY")
         keys = atbl.getKeys()

lguohan · 2018-07-24T15:19:07Z

retest this please

…t#1796) efa2ff6 [show][platform summary] Add chassis type in the platform summary output(sonic-net#1922) a39350c [aclshow] enhance ACL counters to work with FC infrastructure (sonic-net#1858) ed88013 [sonic-package-manager] fix registry requests failing when no service field in Bearer fields (sonic-net#1921) 00b6045 [VS test] Increase test timeout (sonic-net#1924) Signed-off-by: Vivek Reddy Karri <vkarri@nvidia.com>

a3e34e3 [Auto Techsupport] Event driven Techsupport Changes (#1796) efa2ff6 [show][platform summary] Add chassis type in the platform summary output(#1922) a39350c [aclshow] enhance ACL counters to work with FC infrastructure (#1858) ed88013 [sonic-package-manager] fix registry requests failing when no service field in Bearer fields (#1921) 00b6045 [VS test] Increase test timeout (#1924)

#### What I did sonic-utilities changes required for feature "Event Driven TechSupport Invocation & CoreDump Mgmt". [HLD](sonic-net/SONiC#818 ) Summary of the changes: - Added the AUTO GEN CLI for the CFG DB tables required for this feature - Added the coredump_gen_handler.py & techsupport_cleanup.py scripts. - Added the UT's required for these scripts. - Enhanced coredump-compress & generate-dump scripts

…kers to Bookworm (sonic-net#1796)  #### Why I did it Upgrade two remaining containers that were missed from Bullseye to Bookworm. This appears to be used only by Broadcom, and was missed because it doesn't live in the `platform/broadcom` directory. Also remove `platform/vs/gbsyncd-vs.mk`, which is unused, to prevent any confusion. ##### Work item tracking - Microsoft ADO **(number only)**: 31588961 #### How I did it #### How to verify it  #### Which release branch to backport (provide reason below if selected)  - [ ] 201811 - [ ] 201911 - [ ] 202006 - [ ] 202012 - [ ] 202106 - [ ] 202111 - [ ] 202205 - [ ] 202211 - [ ] 202305 #### Tested branch (Please provide the tested image version)  - [ ]  - [ ]  #### Description for the changelog   #### Link to config_db schema for YANG module changes  #### A picture of a cute animal (not mandatory but encouraged)

wadelnn and others added 30 commits October 5, 2017 09:28

Initial commit

444d83b

Add Ingrasys S9180-32X platform dirver.

f81e484

Signed-off-by: Wade He <chihen.he@gmail.com>

Add bfn.service for init barefoot.

e264261

Signed-off-by: Wade He <chihen.he@gmail.com>

Add bfn script in /etc/init.d/

56743e3

Signed-off-by: Wade He <chihen.he@gmail.com>

Add bfn service in debian

a7bdcec

Signed-off-by: Wade He <chihen.he@gmail.com>

Fixed CPLD switch LED behavior.

178d872

Signed-off-by: Wade He <chihen.he@gmail.com>

[Barefoot Beta] Fixed sensors and hwmon order.

3cb5bea

1. Fixed ignore sensors Vbat. 2. Reorg hwmon order. Signed-off-by: Wade He <chihen.he@gmail.com>

Fixed PSU1 and PSU2 EEPROM order.

edcc31b

Signed-off-by: Wade He <chihen.he@gmail.com>

initial barefoot checkin october 2017

743edc0

update refpoint

2247c9f

update refpoints

161bf76

Merge branch 'rel_6_0' into bf-master

c019a46

update refpoints to bf-master

8f62271

update refpoint

0cd43aa

update refpoint to tested version

ce851cb

change to platform from asic

8aa2359

update refpoint for swss

f5ec304

Merge remote-tracking branch 'origin/rel_6_0' into bf-master

89f64f1

revert core creation setting

33a5004

Merge remote-tracking branch 'azure/master' into bf-master

784d737

update refpoints

aacc252

Merge remote-tracking branch 'origin/rel_6_1' into bf-master

b552178

Merge branch 'bf-master' of github.com:barefootnetworks/sonic-buildim…

00e38c4

…age into bf-master

add telnet for debug shell

bea422b

update refpoints 11/17/17

a848be6

Merge remote-tracking branch 'azure/master' into bf-master

1cc0c95

missed change in file on previous merge

e4131ca

[CPLD] Fixed blink LED issue.

d8debcc

* Fixed blink LED mask set error. Signed-off-by: Wade He <chihen.he@gmail.com>

Update bf_kdrv.c for 6.0.2.39

bd0aff8

mkbalani added 2 commits July 15, 2018 01:38

Merge remote-tracking branch 'azure/master' into address_comments

91b0392

Merge pull request #17 from barefootnetworks/address_comments

93cfeed

Address comments

lguohan reviewed Jul 16, 2018

View reviewed changes

mkbalani added 7 commits July 16, 2018 11:33

Address upstream comments

222cfc0

* Remove bmc interface from interface jinja template and sample output interfaces file * Add bmc interface at boot time to network interfaces for bfn bmc based platforms

Merge pull request #18 from barefootnetworks/add_bmc_intf

11a4b28

Address upstream comments for usb0 interface

Remove autogen ingrasys debian files

3ccdb37

Merge pull request #19 from barefootnetworks/rm_autogen_files

f17e37d

Remove autogen ingrasys debian files

Revert "Remove autogen ingrasys debian files"

25ede9b

Merge pull request #20 from barefootnetworks/revert-19-rm_autogen_files

6ac26d6

Revert "Remove autogen ingrasys debian files"

Buffer and qos config template fix for bfn platforms (#21)

bffd000

SWI-1509 Buffer and qos config template fix for bfn platforms

Fix qos config files for montara & mavericks (#22)

93ddd23

* Reference only ppg 3,4 in qos files as no profiles are attached to 0,1 in buffer configs

Fix vs test (#23)

aa69f9e

lguohan merged commit 93905d3 into sonic-net:master Jul 24, 2018

Conversation

mkbalani commented Jun 18, 2018 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

mkbalani commented Jul 16, 2018

Uh oh!

lguohan commented Jul 16, 2018

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

lguohan commented Jul 16, 2018

Uh oh!

mkbalani commented Jul 16, 2018

Uh oh!

mkbalani commented Jul 16, 2018

Uh oh!

lguohan commented Jul 23, 2018

Uh oh!

lguohan commented Jul 24, 2018

Uh oh!

lguohan commented Jul 24, 2018

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

12 participants

mkbalani commented Jun 18, 2018 •

edited

Loading