diff options
| author | Yuriy Andamasov <yuriy@vyos.io> | 2026-07-03 19:16:09 +0300 |
|---|---|---|
| committer | Yuriy Andamasov <yuriy@vyos.io> | 2026-07-03 19:16:09 +0300 |
| commit | 238894482e89cce374ca98763f74a424aa81e2fe (patch) | |
| tree | ee7518b617062c3462c155b3a5d49b82f69bbaf2 | |
| parent | ad3518c119641d830a6ed96c9f35de0a078e70e6 (diff) | |
| download | vyos-1x-238894482e89cce374ca98763f74a424aa81e2fe.tar.gz vyos-1x-238894482e89cce374ca98763f74a424aa81e2fe.zip | |
smoketest: T9048: guard the retry read in get_process_cmdline() too
The re-resolve path read /proc/<pid>/cmdline without defaultonfailure,
so a repeated PID race raised an unhandled exception (test ERROR)
instead of failing with a clear assertion message.
🤖 Generated by [robots](https://vyos.io)
| -rw-r--r-- | smoketest/scripts/cli/base_interfaces_test.py | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/smoketest/scripts/cli/base_interfaces_test.py b/smoketest/scripts/cli/base_interfaces_test.py index fd63e1124..4772c7202 100644 --- a/smoketest/scripts/cli/base_interfaces_test.py +++ b/smoketest/scripts/cli/base_interfaces_test.py @@ -246,7 +246,9 @@ class BasicInterfaceTest: pid = process_named_running(process_name, cmdline=interface, timeout=PROCESS_WAIT_TIMEOUT) self.assertTrue(pid) - cmdline = read_file(f'/proc/{pid}/cmdline') + cmdline = read_file(f'/proc/{pid}/cmdline', defaultonfailure='') + self.assertTrue(cmdline, + f'{process_name} cmdline unreadable after PID re-resolve') return cmdline def test_dhcp_disable_interface(self): |