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

Integration seems to have issues with S.M.A.R.T. enabled #124

Open
lukas1906 opened this issue Jun 12, 2023 · 2 comments
Open

Integration seems to have issues with S.M.A.R.T. enabled #124

lukas1906 opened this issue Jun 12, 2023 · 2 comments
Labels
bug Something isn't working planned To be implemented in future release

Comments

@lukas1906
Copy link

lukas1906 commented Jun 12, 2023

Describe the issue

A clear and concise description of what the issue is.
-->When setting up integration with S.M.A.R.T. enabled the integration fails.

How to reproduce the issue

Enable S.M.A.R.T. in the configuration of the integration
Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on '....'
  3. See error

Expected behavior

Integration to connect to my OMV server

Screenshots

Software versions

  • Home Assistant version: HA 2023.6.1
  • openmediavault integration version: v1.4.1
  • openmediavault software version: v6.4.3-1

Diagnostics data

Traceback/Error logs

This error originated from a custom integration.

Logger: homeassistant.config_entries
Source: custom_components/openmediavault/apiparser.py:53
Integration: OpenMediaVault (documentation, issues)
First occurred: 18:31:36 (1 occurrences)
Last logged: 18:31:36

Error setting up entry NAS for openmediavault
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 387, in async_setup
result = await component.async_setup_entry(hass, self)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/openmediavault/init.py", line 35, in async_setup_entry
await controller.async_update()
File "/config/custom_components/openmediavault/omv_controller.py", line 184, in async_update
await self.hass.async_add_executor_job(self.get_smart)
File "/usr/local/lib/python3.11/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/openmediavault/omv_controller.py", line 350, in get_smart
tmp_data = parse_api(
^^^^^^^^^^
File "/config/custom_components/openmediavault/apiparser.py", line 157, in parse_api
data = fill_vals(data, entry, uid, vals)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/openmediavault/apiparser.py", line 289, in fill_vals
data[uid][_name] = from_entry(entry, _source, default=_default)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/openmediavault/apiparser.py", line 53, in from_entry
ret = int(ret)
^^^^^^^^
ValueError: invalid literal for int() with base 10: '16278h+57m+13.531s'

Additional context

To resolve disable S.M.A.R.T. in the configuration of the integration and reboot homeassistant

@lukas1906 lukas1906 added the bug Something isn't working label Jun 12, 2023
@lukas1906 lukas1906 changed the title [Bug] Integration seems to have issues with S.M.A.R.T. enabled Jun 12, 2023
@tomaae
Copy link
Owner

tomaae commented Jun 13, 2023

can you check smart manually and see what kind of field is "16278h+57m+13.531s"?

@lukas1906
Copy link
Author

lukas1906 commented Jun 13, 2023 via email

@tomaae tomaae added the planned To be implemented in future release label Jun 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working planned To be implemented in future release
Projects
None yet
Development

No branches or pull requests

2 participants