Skip to content
New issue

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

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

Already on GitHub? # to your account

It would be nice to have result output for the image boot check #249

Open
vpa1977 opened this issue Apr 9, 2024 · 2 comments
Open

It would be nice to have result output for the image boot check #249

vpa1977 opened this issue Apr 9, 2024 · 2 comments

Comments

@vpa1977
Copy link

vpa1977 commented Apr 9, 2024

The image boot check, e.g.[1] ignores the exception and the process output.
It would be nice to get result in case of exception, as I am encountering a situation where the machine is booted and I can connect to it manually, but testflinger is failing the check.

[1]

logger.info("Checking if test image booted.")

Copy link

Thank you for reporting us your feedback!

The internal ticket has been created: https://warthogs.atlassian.net/browse/CERTTF-315.

This message was autogenerated

@jocave
Copy link
Collaborator

jocave commented Jul 4, 2024

Adding the following input, in the case of a failed boot check the output looks like:

2024-07-02 17:06:59,014 limerick-zcu106001 INFO: DEVICE CONNECTOR: Checking if test image booted.
Traceback (most recent call last):
  File "/srv/testflinger-agent/limerick-zcu106001/env/bin/testflinger-device-connector", line 8, in <module>
    sys.exit(main())
  File "/srv/testflinger-agent/limerick-zcu106001/env/lib/python3.8/site-packages/testflinger_device_connectors/cmd.py", line 83, in main
    sys.exit(func(args))
  File "/srv/testflinger-agent/limerick-zcu106001/env/lib/python3.8/site-packages/testflinger_device_connectors/devices/__init__.py", line 321, in wrapper
    return func(*args, **kwargs)
  File "/srv/testflinger-agent/limerick-zcu106001/env/lib/python3.8/site-packages/testflinger_device_connectors/devices/muxpi/__init__.py", line 52, in provision
    raise e
  File "/srv/testflinger-agent/limerick-zcu106001/env/lib/python3.8/site-packages/testflinger_device_connectors/devices/muxpi/__init__.py", line 50, in provision
    device.provision()
  File "/srv/testflinger-agent/limerick-zcu106001/env/lib/python3.8/site-packages/testflinger_device_connectors/devices/muxpi/muxpi.py", line 240, in provision
    self.check_test_image_booted()
  File "/srv/testflinger-agent/limerick-zcu106001/env/lib/python3.8/site-packages/testflinger_device_connectors/devices/muxpi/muxpi.py", line 619, in check_test_image_booted
    raise ProvisioningError("Failed to boot test image!")
testflinger_device_connectors.devices.ProvisioningError: Failed to boot test image!

I think it would be better for the user for this traceback to be replaced with details of the what check is performed by the device connector being used, and as request by the OP human understandable details of the exception that occured.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants