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

nydus-test: fix debug panic #1059

Merged

Conversation

ccx1024cc
Copy link
Contributor

Reproduction: add NYDUS_TEST_VERBOSE=YES to booting envs of pytest.
For example:

    sudo NYDUS_TEST_VERBOSE=YES pytest functional-test/test_layered_image.py::test_basic_read

Output:

    ERROR    root:workload_gen.py:459 Stress read failure, not all arguments converted during string formatting
    Traceback (most recent call last):
      File "/home/morgan/workspace/rust/image-service/contrib/nydus-test/framework/workload_gen.py", line 456, in io_read
        cnt, size, duration = self.read_collected_files(io_duration)
      File "/home/morgan/workspace/rust/image-service/contrib/nydus-test/framework/workload_gen.py", line 404, in read_collected_files
        logging.debug(
      File "/usr/lib/python3.10/logging/__init__.py", line 2148, in debug
        root.debug(msg, *args, **kwargs)
      File "/usr/lib/python3.10/logging/__init__.py", line 1465, in debug
        self._log(DEBUG, msg, args, **kwargs)
      File "/usr/lib/python3.10/logging/__init__.py", line 1624, in _log
        self.handle(record)
      File "/usr/lib/python3.10/logging/__init__.py", line 1634, in handle
        self.callHandlers(record)
      File "/usr/lib/python3.10/logging/__init__.py", line 1696, in callHandlers
        hdlr.handle(record)
      File "/usr/lib/python3.10/logging/__init__.py", line 968, in handle
        self.emit(record)
      File "/usr/local/lib/python3.10/dist-packages/_pytest/logging.py", line 343, in emit
        super().emit(record)
      File "/usr/lib/python3.10/logging/__init__.py", line 1108, in emit
        self.handleError(record)
      File "/usr/lib/python3.10/logging/__init__.py", line 1100, in emit
        msg = self.format(record)
      File "/usr/lib/python3.10/logging/__init__.py", line 943, in format
        return fmt.format(record)
      File "/usr/local/lib/python3.10/dist-packages/_pytest/logging.py", line 114, in format
        return super().format(record)
      File "/usr/lib/python3.10/logging/__init__.py", line 678, in format
        record.message = record.getMessage()
      File "/usr/lib/python3.10/logging/__init__.py", line 368, in getMessage
        msg = msg % self.args
    TypeError: not all arguments converted during string formatting

Signed-off-by: 泰友 cuichengxu.ccx@antgroup.com

Reproduction: add `NYDUS_TEST_VERBOSE=YES` to booting params of pytest.
For example:

```shell
    sudo NYDUS_TEST_VERBOSE=YES pytest functional-test/test_layered_image.py::test_basic_read
```

Output:
```
    ERROR    root:workload_gen.py:459 Stress read failure, not all arguments converted during string formatting
    Traceback (most recent call last):
      File "/home/morgan/workspace/rust/image-service/contrib/nydus-test/framework/workload_gen.py", line 456, in io_read
        cnt, size, duration = self.read_collected_files(io_duration)
      File "/home/morgan/workspace/rust/image-service/contrib/nydus-test/framework/workload_gen.py", line 404, in read_collected_files
        logging.debug(
      File "/usr/lib/python3.10/logging/__init__.py", line 2148, in debug
        root.debug(msg, *args, **kwargs)
      File "/usr/lib/python3.10/logging/__init__.py", line 1465, in debug
        self._log(DEBUG, msg, args, **kwargs)
      File "/usr/lib/python3.10/logging/__init__.py", line 1624, in _log
        self.handle(record)
      File "/usr/lib/python3.10/logging/__init__.py", line 1634, in handle
        self.callHandlers(record)
      File "/usr/lib/python3.10/logging/__init__.py", line 1696, in callHandlers
        hdlr.handle(record)
      File "/usr/lib/python3.10/logging/__init__.py", line 968, in handle
        self.emit(record)
      File "/usr/local/lib/python3.10/dist-packages/_pytest/logging.py", line 343, in emit
        super().emit(record)
      File "/usr/lib/python3.10/logging/__init__.py", line 1108, in emit
        self.handleError(record)
      File "/usr/lib/python3.10/logging/__init__.py", line 1100, in emit
        msg = self.format(record)
      File "/usr/lib/python3.10/logging/__init__.py", line 943, in format
        return fmt.format(record)
      File "/usr/local/lib/python3.10/dist-packages/_pytest/logging.py", line 114, in format
        return super().format(record)
      File "/usr/lib/python3.10/logging/__init__.py", line 678, in format
        record.message = record.getMessage()
      File "/usr/lib/python3.10/logging/__init__.py", line 368, in getMessage
        msg = msg % self.args
    TypeError: not all arguments converted during string formatting
```

Signed-off-by: 泰友 <cuichengxu.ccx@antgroup.com>
@ccx1024cc ccx1024cc requested a review from a team as a code owner February 7, 2023 02:37
@ccx1024cc ccx1024cc requested review from liubin, jiangliu and luodw and removed request for a team February 7, 2023 02:37
@anolis-bot
Copy link
Collaborator

@ccx1024cc , a new test job has been submitted. Please wait in patience. The test job url: https://tone.openanolis.cn/ws/nrh4nnio/test_result/48570

@anolis-bot
Copy link
Collaborator

@ccx1024cc , The CI test is completed, please check result:

Test CaseTest Result
build rust golang image✅ SUCCESS
compile nydusd✅ SUCCESS
compile ctr remote✅ SUCCESS
compile nydus snapshotter✅ SUCCESS
run container with rafs✅ SUCCESS
run container with zran✅ SUCCESS

Congratulations, your test job passed!

@imeoer imeoer merged commit 2ad8675 into dragonflyoss:master Feb 7, 2023
@ccx1024cc ccx1024cc deleted the morgan/fix_nydus_test_debug_log branch March 14, 2023 02:59
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