Looks like PFCWD unit tests are failing with "Root privileges are required for this operation". Not sure why this wasn't seen before. Looks like @smaheshm modified these recently. Maybe he can help troubleshoot?

Looks like PFCWD unit tests are failing with "Root privileges are required for this operation". Not sure why this wasn't seen before. Looks like @smaheshm modified these recently. Maybe he can help troubleshoot?

Let me take a look. Looks like there's a difference between how sonic-buildimage repo runs tests vs sonic-utilities repo.

need to fix the unit test failure.

there are about 23 commits picked up in sonic-utilities repo, the description only listed 19 commits.

https://sonic-jenkins.westus2.cloudapp.azure.com/job/vs/job/buildimage-vs-image-pr/8207/

issue caused by this pr. 6202a81

reverted.

07:11:55 Sunday 27 December 2020 15:11:54 +0000 (0:00:00.059) 0:00:21.107 *******
07:12:51 fatal: [vlab-01]: FAILED! => {"changed": true, "cmd": "config load_minigraph -y", "delta": "0:00:47.452274", "end": "2020-12-27 15:12:41.680371", "msg": "non-zero return code", "rc": 1, "start": "2020-12-27 15:11:54.228097", "stderr": "Warning: Stopping telemetry.service, but it can still be activated by:\n telemetry.timer\nTraceback (most recent call last):\n File "/usr/local/bin/config", line 8, in \n sys.exit(config())\n File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 764, in call\n return self.main(*args, **kwargs)\n File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 717, in main\n rv = self.invoke(ctx)\n File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 1137, in invoke\n return _process_result(sub_ctx.command.invoke(sub_ctx))\n File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 1137, in invoke\n return _process_result(sub_ctx.command.invoke(sub_ctx))\n File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 956, in invoke\n return ctx.invoke(self.callback, **ctx.params)\n File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 555, in invoke\n return callback(*args, **kwargs)\n File "/usr/local/lib/python3.7/dist-packages/click/decorators.py", line 17, in new_func\n return f(get_current_context(), *args, **kwargs)\n File "/usr/local/lib/python3.7/dist-packages/config/main.py", line 1673, in reload\n sonic_version_file = device_info.get_sonic_version_file()\nAttributeError: module 'sonic_py_common.device_info' has no attribute 'get_sonic_version_file'", "stderr_lines": ["Warning: Stopping telemetry.service, but it can still be activated by:", " telemetry.timer", "Traceback (most recent call last):", " File "/usr/local/bin/config", line 8, in ", " sys.exit(config())", " File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 764, in call", " return self.main(*args, **kwargs)", " File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 717, in main", " rv = self.invoke(ctx)", " File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 1137, in invoke", " return _process_result(sub_ctx.command.invoke(sub_ctx))", " File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 1137, in invoke", " return _process_result(sub_ctx.command.invoke(sub_ctx))", " File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 956, in invoke", " return ctx.invoke(self.callback, **ctx.params)", " File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 555, in invoke", " return callback(*args, **kwargs)", " File "/usr/local/lib/python3.7/dist-packages/click/decorators.py", line 17, in new_func", " return f(get_current_context(), *args, **kwargs)", " File "/usr/local/lib/python3.7/dist-packages/config/main.py", line 1673, in reload", " sonic_version_file = device_info.get_sonic_version_file()", "AttributeError: module 'sonic_py_common.device_info' has no attribute 'get_sonic_version_file'"], "stdout": "Executing stop of service telemetry...\nExecuting stop of service swss...\nExecuting stop of service lldp...\nExecuting stop of service pmon...\nExecuting stop of service bgp...\nRunning command: /usr/local/bin/sonic-cfggen -H -m -j /etc/sonic/init_cfg.json --write-to-db\nRunning command: pfcwd start_default\nRunning command: /usr/local/bin/sonic-cfggen -d -y /etc/sonic/sonic_version.yml -t /usr/share/sonic/templates/sonic-environment.j2,/etc/sonic/sonic-environment\nRunning command: config qos reload --no-dynamic-buffer", "stdout_lines": ["Executing stop of service telemetry...", "Executing stop of service swss...", "Executing stop of service lldp...", "Executing stop of service pmon...", "Executing stop of service bgp...", "Running command: /usr/local/bin/sonic-cfggen -H -m -j /etc/sonic/init_cfg.json --write-to-db", "Running command: pfcwd start_default", "Running command: /usr/local/bin/sonic-cfggen -d -y /etc/sonic/sonic_version.yml -t /usr/share/sonic/templates/sonic-environment.j2,/etc/sonic/sonic-environment", "Running command: config qos reload --no-dynamic-buffer"]}

Looks like PFCWD unit tests are failing with "Root privileges are required for this operation". Not sure why this wasn't seen before. Looks like @smaheshm modified these recently. Maybe he can help troubleshoot?

Let me take a look. Looks like there's a difference between how sonic-buildimage repo runs tests vs sonic-utilities repo.

sonic-net/sonic-utilities#1340