Skip to content

ingrasys-s9100: Add ingrasys switch s9100#117

Closed
kaiyu22 wants to merge 1 commit intosonic-net:masterfrom
kaiyu22:master
Closed

ingrasys-s9100: Add ingrasys switch s9100#117
kaiyu22 wants to merge 1 commit intosonic-net:masterfrom
kaiyu22:master

Conversation

@kaiyu22
Copy link

@kaiyu22 kaiyu22 commented Feb 8, 2017

  • Adding switch configuration
  • Platform-Monitor - Adding fan control and sensor

Signed-off-by: Sam Yang yang.kaiyu@gmail.com

* Adding switch configuration
* Platform-Monitor - Adding fan control and sensor

Signed-off-by: Sam Yang <yang.kaiyu@gmail.com>
Copy link
Contributor

@lguohan lguohan left a comment

Choose a reason for hiding this comment

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

I'll be focusing on the other PR you have submitted in the buildimage as the some assumptions on the mgmt repo are changing by introducing this one image. Therefore, some files should be move to buildimage.

Let's first finish that one. after that we'll come back to this one.

@@ -0,0 +1,12 @@
# Configuration file generated by pwmconfig, changes will be lost
Copy link
Contributor

Choose a reason for hiding this comment

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

move this to buildimage repo.

Copy link
Author

@kaiyu22 kaiyu22 Feb 13, 2017

Choose a reason for hiding this comment

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

I saw other vendors also placed sensor file in mgmt and buildimage repo. Do I put the sensor file in here and buildimage repo as well? Or as you said, only put the file in buildimage repo.

@@ -0,0 +1,35 @@
# libsensors configuration file
Copy link
Contributor

Choose a reason for hiding this comment

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

this is in buildimage, so remove it from here.

Copy link
Author

@kaiyu22 kaiyu22 Feb 13, 2017

Choose a reason for hiding this comment

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

Does port config file has to be removed from mgmt, and move to buildiamge repo? Or place port config in both of them?

@lguohan
Copy link
Contributor

lguohan commented Jun 13, 2017

built in one image, this patch is no longer needed.

@lguohan lguohan closed this Jun 13, 2017
nhe-NV pushed a commit to nhe-NV/sonic-mgmt that referenced this pull request May 12, 2025
sonic-net#117)

<!--
Please make sure you've read and understood our contributing guidelines;
https://github.com/sonic-net/SONiC/blob/gh-pages/CONTRIBUTING.md

Please provide following information to help code review process a bit easier:
-->
### Description of PR
<!--
- Please include a summary of the change and which issue is fixed.
- Please also include relevant motivation and context. Where should reviewer start? background context?
- List any dependencies that are required for this change.
-->

Summary:
Fixes # (issue)

### Type of change

<!--
- Fill x for your type of change.
- e.g.
- [x] Bug fix
-->

- [ ] Bug fix
- [ ] Testbed and Framework(new/improvement)
- [ ] New Test case
 - [x] Skipped for non-supported platforms
- [ ] Test case improvement

### Back port request
- [ ] 202012
- [ ] 202205
- [ ] 202305
- [ ] 202311
- [ ] 202405
- [ ] 202411

### Approach
#### What is the motivation for this PR?
The vxlan-decap testcases cannot be supported in some testbeds without port-channel

#### How did you do it?
Adapting the test parameters on both portchannel and non-portchannel scenarioes.

#### How did you verify/test it?
Run it on non-portchannel platform:

```
==================================================================================================================================== test session starts =====================================================================================================================================
platform linux -- Python 3.8.10, pytest-7.4.0, pluggy-1.5.0
ansible: 2.13.13
rootdir: /data/sonic-mgmt-int/tests
configfile: pytest.ini
plugins: ansible-4.0.0, metadata-3.1.1, html-4.1.1, xdist-1.28.0, forked-1.6.0, repeat-0.9.3, allure-pytest-2.8.22
collecting ...
vxlan/test_vxlan_decap.py::test_vxlan_decap[NoVxLAN] PASSED
vxlan/test_vxlan_decap.py::test_vxlan_decap[Enabled] PASSED
vxlan/test_vxlan_decap.py::test_vxlan_decap[Removed] PASSED

====================================================================================================================================== warnings summary ======================================================================================================================================
../../../usr/local/lib/python3.8/dist-packages/paramiko/transport.py:236
 /usr/local/lib/python3.8/dist-packages/paramiko/transport.py:236: CryptographyDeprecationWarning: Blowfish has been deprecated
 "class": algorithms.Blowfish,

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
------------------------------------------------------------------------------------------------------- generated xml file: /data/sonic-mgmt-int/tests/logs/vxlan/test_vxlan_decap.xml -------------------------------------------------------------------------------------------------------
----------------------------------------------------------------------------------------------------------------------------------- live log sessionfinish -----------------------------------------------------------------------------------------------------------------------------------
12:55:46 __init__.pytest_terminal_summary L0067 INFO | Can not get Allure report URL. Please check logs
========================================================================================================================== 3 passed, 1 warning in 299.34s (0:04:59) ==========================================================================================================================

```

#### Any platform specific information?

#### Supported testbed topology if it's a new test case?

### Documentation
<!--
(If it's a new feature, new test case)
Did you update documentation/Wiki relevant to your implementation?
Link to the wiki page?
-->
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
…submodule head (sonic-net#11761)

linkmgrd:
* 476f85e 2022-08-17 | Update linkmgr health after getting default route update (sonic-net#117) (HEAD -> 202205, github/202205) [Longxiang Lyu]
* fc589e9 2022-08-17 | Use `table` to toggle peer forwarding state (sonic-net#108) (sonic-net#120) [Longxiang Lyu]
* bcb5a56 2022-08-17 | Fix azure pipeline (sonic-net#118) (sonic-net#121) [Longxiang Lyu]

swss:
* ef3a601 2022-08-17 | [muxorch] Returning true if nbr in skip_neighbor_ in isNeighborActive() (sonic-net#2415) (HEAD -> 202205) [Nikola Dancejic]

sairedis:
* aed01cd 2022-08-12 | Fix: missing sonic-db-cli in docker-sonic-vs image (sonic-net#1072) (sonic-net#1104) (github/202205) [Hua Liu]

platform-daemon:
* 5a68073 2022-08-01 | Xcvrd changes to support 400G ZR configuration (sonic-net#270) (HEAD -> 202205) [Prince George]

swsssdk:
* ca785a2 2022-06-01 | Remove sonic-db-cli (sonic-net#122) (HEAD -> 202205, origin/202205) [Hua Liu]

Signed-off-by: Ying Xie <ying.xie@microsoft.com>

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
03936ea (HEAD -> 202111, origin/202111) define index for recirc port (sonic-net#118)
d48f750 [port_util] Fix issue: port_util.get_vlan_interface_oid_map should not raise exception when DB has not RIF data (sonic-net#117)
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
commit 5f7cb77230fceb1b7fd30c57a70d0cd05cd6dd95 (HEAD -> 201911, origin/201911)
Author: Sumukha Tumkur Vani <sumukhatv@outlook.com>
Date:   Wed Jul 27 16:51:38 2022 -0700

    Use 201911/stretch dependencies for build (sonic-net#118)

commit e3809523050df75ec18bf31a5dc3a2e595d58a14
Author: Sumukha Tumkur Vani <sumukhatv@outlook.com>
Date:   Wed Jul 27 15:17:12 2022 -0700

    Change response message for conflicting VNI (sonic-net#117)

    Ref: sonic-net/sonic-restapi#99

Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants