Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[pmon] pcied keeps reporting warning #5819

Closed
bingwang-ms opened this issue Nov 5, 2020 · 3 comments · Fixed by sonic-net/sonic-platform-daemons#144
Closed

[pmon] pcied keeps reporting warning #5819

bingwang-ms opened this issue Nov 5, 2020 · 3 comments · Fixed by sonic-net/sonic-platform-daemons#144

Comments

@bingwang-ms
Copy link
Contributor

Description
pcied in pmon container keeps printing warning logs

Nov  5 05:44:18.057668 str-dx010-acs-4 WARNING pmon#pcied[125]: Failed to load platform-specific PcieUtil module. Falling back to the common implementation

It is generated by following code

try:
            platform_path, _ = device_info.get_paths_to_platform_and_hwsku_dirs()
            platform_plugins_path = os.path.join(platform_path, "plugins")
            sys.path.append(os.path.abspath(platform_plugins_path))
            self.log_warning("plugin path = {}".format(os.path.abspath(platform_plugins_path)))
            from pcieutil import PcieUtil
        except ImportError as e:
            self.log_warning("Failed to load platform-specific PcieUtil module. Falling back to the common implementation")
            try:
                from sonic_platform_base.sonic_pcie.pcie_common import PcieUtil
                platform_pcieutil = PcieUtil(platform_plugins_path)
            except ImportError as e:
                self.log_error("Failed to load default PcieUtil module. Error : {}".format(str(e)), True)
                raise e

I checked the utils in /usr/share/sonic/platform/plugins and found no pcieutil library.

Steps to reproduce the issue:
The log is alway printing even no test case is running.
1.
2.
3.

Describe the results you received:

Describe the results you expected:

Additional information you deem important (e.g. issue happens only occasionally):

**Output of `show version`:**
SONiC Software Version: SONiC.master.469-84d3a260
Distribution: Debian 10.6
Kernel: 4.19.0-9-2-amd64
Build commit: 84d3a260
Build date: Wed Nov  4 00:55:31 UTC 2020
Built by: johnar@jenkins-worker-8

Platform: x86_64-cel_seastone-r0
HwSKU: Celestica-DX010-C32
ASIC: broadcom
Serial Number: DX010F2B118711MS100007
Uptime: 05:55:15 up 11:22,  3 users,  load average: 4.56, 3.05, 2.51

Docker images:
REPOSITORY                    TAG                   IMAGE ID            SIZE
docker-snmp                   latest                e21c75cf3e75        427MB
docker-snmp                   master.469-84d3a260   e21c75cf3e75        427MB
docker-teamd                  latest                93c913c5a017        425MB
docker-teamd                  master.469-84d3a260   93c913c5a017        425MB
docker-sonic-mgmt-framework   latest                244b796dd024        551MB
docker-sonic-mgmt-framework   master.469-84d3a260   244b796dd024        551MB
docker-router-advertiser      latest                c31265cbdcd2        390MB
docker-router-advertiser      master.469-84d3a260   c31265cbdcd2        390MB
docker-platform-monitor       latest                106e63b358ad        503MB
docker-platform-monitor       master.469-84d3a260   106e63b358ad        503MB
docker-lldp                   latest                9ba8b14bb844        456MB
docker-lldp                   master.469-84d3a260   9ba8b14bb844        456MB
docker-dhcp-relay             latest                d48a5f726cd7        397MB
docker-dhcp-relay             master.469-84d3a260   d48a5f726cd7        397MB
docker-database               latest                9af7507f9821        390MB
docker-database               master.469-84d3a260   9af7507f9821        390MB
docker-orchagent              latest                6276adebac5b        439MB
docker-orchagent              master.469-84d3a260   6276adebac5b        439MB
docker-nat                    latest                0a1ee7e92192        428MB
docker-nat                    master.469-84d3a260   0a1ee7e92192        428MB
docker-sonic-telemetry        latest                e387512c848f        459MB
docker-sonic-telemetry        master.469-84d3a260   e387512c848f        459MB
docker-fpm-frr                latest                0a87f992c6a5        441MB
docker-fpm-frr                master.469-84d3a260   0a87f992c6a5        441MB
docker-sflow                  latest                c5edda64f062        429MB
docker-sflow                  master.469-84d3a260   c5edda64f062        429MB
docker-syncd-brcm             latest                bac1974ef6a7        527MB
docker-syncd-brcm             master.469-84d3a260   bac1974ef6a7        527MB    
**Attach debug file `sudo generate_dump`:**

```
(paste your output here)
```
@lguohan
Copy link
Collaborator

lguohan commented Nov 6, 2020

@sujinmkang , can you take a look?

@yxieca
Copy link
Contributor

yxieca commented Jan 6, 2021

@sujinmkang, @jleveque can you assess if this issue is still applicable to latest image?

@sujinmkang
Copy link
Collaborator

I will clean up this message. There is no platforms which have platform api implemented.

sujinmkang added a commit to sonic-net/sonic-platform-daemons that referenced this issue Feb 9, 2021
)

[pcied] Remove unnecessary message and move the configuration path from plugin directory to platform directory

This PR fixes sonic-net/sonic-buildimage#5819 and fixes sonic-net/sonic-buildimage#6437.

Remove unnecessary repeated message
Move the configuration path from plugin directory to platform directory
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants