Skip to content

fix bug for compile sonic-platform-common caused by enable pytest#7431

Merged
lguohan merged 1 commit intosonic-net:masterfrom
CentecNetworks:fix_bug_for_compile_sonic-platform-common
Apr 28, 2021
Merged

fix bug for compile sonic-platform-common caused by enable pytest#7431
lguohan merged 1 commit intosonic-net:masterfrom
CentecNetworks:fix_bug_for_compile_sonic-platform-common

Conversation

@LuiSzee
Copy link
Contributor

@LuiSzee LuiSzee commented Apr 26, 2021

Why I did it

compile sonic-platform-common failed by unit test

How I did it

add depend on PYTHON_SWSSCOMMON

How to verify it

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012

Description for the changelog

A picture of a cute animal (not mandatory but encouraged)

@LuiSzee
Copy link
Contributor Author

LuiSzee commented Apr 26, 2021

"PDDF_SUPPORT" : "y"
"MULTIARCH_QEMU_ENVIRON" : "n"

"SONIC_DPKG_CACHE_METHOD" : "none"

[ FAIL LOG START ] [ target/python-wheels/sonic_platform_common-1.0-py2-none-any.whl ]

/usr/lib/python2.7/dist-packages/six.py:709: in exec_
exec("""exec code in globs, locs""")
tests/sfputilhelper_test.py:6: in
from sonic_platform_base.sonic_sfp import sfputilhelper
sonic_platform_base/sonic_sfp/sfputilhelper.py:20: in
raise ImportError("%s - required module not found" % str(e))
E ImportError: No module named swsscommon.swsscommon - required module not found - required module not found
---- generated xml file: /sonic/src/sonic-platform-common/test-results.xml -----

---------- coverage: platform linux2, python 2.7.16-final-0 ----------
Name Stmts Miss Cover

sonic_platform_base/init.py 11 0 100%
sonic_platform_base/chassis_base.py 128 80 38%
sonic_platform_base/component_base.py 17 8 53%
sonic_platform_base/device_base.py 19 7 63%
sonic_platform_base/fan_base.py 24 7 71%
sonic_platform_base/fan_drawer_base.py 22 12 45%
sonic_platform_base/module_base.py 104 58 44%
sonic_platform_base/platform_base.py 5 2 60%
sonic_platform_base/psu_base.py 57 33 42%
sonic_platform_base/sfp_base.py 61 31 49%
sonic_platform_base/sonic_eeprom/init.py 0 0 100%
sonic_platform_base/sonic_eeprom/eeprom_base.py

@lguohan
Copy link
Collaborator

lguohan commented Apr 27, 2021

/azp run

@azure-pipelines
Copy link

azure-pipelines bot commented Apr 27, 2021

Azure Pipelines successfully started running 1 pipeline(s).

@lguohan lguohan merged commit 7c79b26 into sonic-net:master Apr 28, 2021
raphaelt-nvidia pushed a commit to raphaelt-nvidia/sonic-buildimage that referenced this pull request May 23, 2021
@LuiSzee @raphaelt-nvidia
[build]: fix bug for compile sonic-platform-common caused by enable p…
39c53cd 
…ytest (sonic-net#7431)

Co-authored-by: Shi Lei <[email protected]>
carl-nokia pushed a commit to carl-nokia/sonic-buildimage that referenced this pull request Aug 7, 2021
@LuiSzee
[build]: fix bug for compile sonic-platform-common caused by enable p…
dd82084 
…ytest (sonic-net#7431)

Co-authored-by: Shi Lei <[email protected]>
LuiSzee added a commit to CentecNetworks/sonic-buildimage that referenced this pull request Aug 20, 2021
@LuiSzee
[build]: fix bug for compile sonic-platform-common caused by enable p…
f87adb6 
…ytest (sonic-net#7431)

Co-authored-by: Shi Lei <[email protected]>
mudsut4ke pushed a commit to mudsut4ke/sonic-buildimage that referenced this pull request Sep 15, 2021
@LuiSzee
[build]: fix bug for compile sonic-platform-common caused by enable p…
de61c7a 
…ytest (sonic-net#7431)

Co-authored-by: Shi Lei <[email protected]>
@LuiSzee LuiSzee deleted the fix_bug_for_compile_sonic-platform-common branch November 22, 2021 05:31
mudsut4ke pushed a commit to SONIC-DEV/sonic-buildimage that referenced this pull request Dec 14, 2021
@LuiSzee
[Celestica]: Add support for Blackstone platform (#262)
068094d 
* [platform/cel]: add blackstone device

* [device/celestica]: add blackstone device cel_blackstone-r0

* [device/celestica-blackstone]: disable services in pmon

* [platform/cel-blackstone]: fix sonic_platform package_dir

* [build]: fix bug for compile sonic-platform-common caused by enable pytest (sonic-net#7431)

Co-authored-by: Shi Lei <[email protected]>

* [platform/cel-blackstone]: fix missing debain files

* [platform/cel-blackstone]: fix invalid init file

* [platform/cel-blackstone]: fix invalid init file

* [platform/cel-blackstone]: add rule to make all kernel module

* [platform/cel-blackstone]: update mapping on fpga driver follow fpga spec

* [platform/cel-blackstone]: enable switch_cpld init

* [platform/cel-blackstone]: update platform_sensors follow ipmi sensors

* [device/celestica-blackstone]: update psuutil follow BMC spec

* [device/celestica-blackstone]: fix typo on eeprom plugin

* [platform/cel-blackstone]: Update FPGA driver follow FPGA rev0.2 spec

* [platform/cel-blackstone]: fix invalid param in FPGA driver

* [platform/cel-blackstone]: fix invalid buses in FPGA driver

* [platform/cel-blackstone]: update init script to enable i2c mapping

* [device/celestica-blackstone]: disable cpld2 in switch cpld driver

* [platform/cel-blackstone]: remove unuse command  on switch CPLD driver

* [platform/cel-blackstone]: remove cpld2 command from switch cpld driver

* [platform/cel-blackstone]: change driver name from switch cpld to misc cpld

* [platform/cel-blackstone]: remove unused variable in misc cpld driver

* [platform/cel-blackstone]: remove unused coment on misc cpld driver

* Revert "[device/celestica-blackstone]: fix typo on eeprom plugin"

This reverts commit c133492.

* Revert "[device/celestica-blackstone]: update psuutil follow BMC spec"

This reverts commit 318b145.

* [device/celestica-blackstone]: update psuutil follow lastest BMC spec

* [device/celestica]: remove unuse files in plugin

* [platform/cel-blackstone]: update misc cpld name device name

* [platform/cel-blackstone]: update misc_cpld device name

* [platform/cel-blackstone]: update misc_cpld driver version

* [platform/cel-blackstone]: update misc_cpld init command

* [platform/cel-blackstone]: update FPGA driver to support BMC I2C

* [platform/cel-blackstone]: update FPGA driver to support BMC I2C

* [platform/cel-blackstone]: update i2c devices driver init command

* [device/celestica-blackstone]: Platform API code initialization

* [device/celestica-blackstone]: update necessary chassis apis

* [platform/cel-blackstone]: add platform api installer

* [platform/cel]: update missing apis installer command

* [device/celestica-blackstone]: update sfputil follow hw spec

* [device/celestica-blackstone]: update skip_syseepromd config for pmon

* [platform/cel-blackstone]: initialize platform APIs

* [device/celestica-blackstone]: update necessary chassis apis

* [platform/cel-blackstone]: add platform api installer

* [platform/cel]: update missing apis installer command

* [device/celestica-blackstone]: update sfputil follow hw spec

* [device/celestica-blackstone]: update skip_syseepromd config for pmon

* [platform/cel]: update api installer script

Co-authored-by: LuiSzee <[email protected]>
Co-authored-by: Shi Lei <[email protected]>
AlanYoush added a commit to CentecNetworks/sonic-buildimage that referenced this pull request Nov 25, 2023
@AlanYoush
[build]: fix bug for compile sonic-platform-common caused by enable p…
2521adf 
…ytest (sonic-net#7431)
centecqianj pushed a commit to CentecNetworks/sonic-buildimage that referenced this pull request Aug 9, 2024
@AlanYoush @centecqianj
[build]: fix bug for compile sonic-platform-common caused by enable p…
2dd7bdb 
…ytest (sonic-net#7431)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lguohan lguohan lguohan approved these changes

@qiluo-msft qiluo-msft qiluo-msft approved these changes

@xumia xumia Awaiting requested review from xumia xumia is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@LuiSzee @lguohan @qiluo-msft